RestComm / jain-sip

Disclaimer: This repository is a git-svn mirror of the project found at http://java.net/projects/jsip whose original repository is developed collaboratively by the Advanced Networking Technologies Division at the National Institute of Standards and Technology (NIST) - an agency of the United States Department of Commerce and by a community of individual and enterprise contributors. TeleStax, Inc. will perform some productization work, new features experimentation branches, etc for its TelScale jSIP product that doesn't concern the community from the main repository hence this git repository.
http://www.restcomm.com/
141 stars 151 forks source link

PAccessNetworkInfo can be parsed as a list #183

Closed xhoaluu closed 6 years ago

xhoaluu commented 6 years ago

https://github.com/RestComm/jain-sip/issues/182

https://telestax.zendesk.com/agent/tickets/35976

xhoaluu commented 6 years ago

jenkin result is aligned with master: https://cxs.restcomm.com/view/VVSSquad/view/JAIN-SIP-Community/job/UPS_JAIN-SIP/93/

jaimecasero commented 6 years ago

im not sure if this header must support multiple values or not. apparently there different spec definitions https://www.ietf.org/mail-archive/web/sipcore/current/msg05804.html

we need to be careful which spec we support here, and what are the impacts for existing apps...

@xhoaluu have you evaluated this from spec point of view, and JAINSIP spec support?

jaimecasero commented 6 years ago

@xhoaluu i think you need to update this https://github.com/RestComm/jain-sip/blob/master/src/gov/nist/javax/sip/message/ListMap.java with new List type so is regsitered in stack as list compatible