in version 2.2., CORSFilter default constructor is not public anymore

[Christophe Thiebaud]

in version 2.2, after this change:

https://bitbucket.org/thetransactioncompany/cors-filter/pull-request/7/constructor-based-initialization/diff

the default constructor of class com.thetransactioncompany.cors.CORSFilter is not public anymore.

this non-public default constructor make subclassing impossible.

but subclassing can be usefull, for instance this class:

https://github.com/aequologica/runciter/blob/master/src/main/java/net/aequologica/neo/geppaequo/cors/CrossOriginResourceSharingFilter.java

uses subclassing to configure the CORSFilter using annotations.

is there any rationale behind having a non public constructor ?

[Vladimir Dzhuvinov]

Sorry, that has been a terrible mistake :)

Thank you for spotting it!

I'll try push a fix later today.

[Vladimir Dzhuvinov]

Done in commit c416a87.

Expect 2.2.1 in Maven Central soon.

Cheers,

Vladimir

[Christophe Thiebaud]

I confirm the issue is fixed with version 2.2.1, which I just grabbed from maven central, tested and even deployed.

thanks for the quick fix!

Christophe

[Vladimir Dzhuvinov]

You're welcome! And thanks for the feedback! I have a clean consciousness now :)