Open teerapap opened 1 year ago
Just found that some other tests fails. Will resubmit the pull request again
False alarm. It does not relate to this pull request changes.
Some tests failed because I was using JDK17 running mvn test
.
It was about java reflection stuff.
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @58651fd0
at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:354)
at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:297)
at java.base/java.lang.reflect.Method.checkCanSetAccessible(Method.java:199)
at java.base/java.lang.reflect.Method.setAccessible(Method.java:193)
at org.mockito.cglib.core.ReflectUtils$2.run(ReflectUtils.java:57)
at java.base/java.security.AccessController.doPrivileged(AccessController.java:318)
at org.mockito.cglib.core.ReflectUtils.<clinit>(ReflectUtils.java:47)
... 50 more
Changes in detail
consumeHost()
method inSipParser
loosely based on RFC 3261consumeSentBye()
->consumeSentBy()
consumeVia()
method to make use ofconsumeSentBy()
consumeSentBy()
method to make use ofconsumeHost()
consumeGenericParam()
method to loosely handle host in addition to token