Fixed ClassCastException when loading configuration.

[kolkoo]

Found a problem where Publish Delay is attempted to be parsed from a String but it is declared as a Long in the metatype/publisher.xml and we get an exception:

java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.String at com.eclipsesource.jaxrs.publisher.internal.Configuration.getPublishDelay(Configuration.java:78) at com.eclipsesource.jaxrs.publisher.internal.Configuration.updated(Configuration.java:46)

Signed-off-by: IVAN GEORGIEV ILIEV ivan.iliev@musala.com

[hstaudacher]

Thanks for the fix Ivan!

[kolkoo]

Anytime :)

[kolkoo]

One question- will you be able to update the 4.2 release with this fix included if it's not too much trouble?

[hstaudacher]

Actually it's a little trouble :). All maven artifacts need to be updated too and a new sync to maven central is required. Updating the existing 4.2 is not an option. The only option is to publish a 4.2.1.

But to be honest I'm not sure if one fix is worth a service release. What do you think? Is this a blocker for you?

[kolkoo]

Well this bug prevents us from changing any configuration properties as it happens before connector.updateConfiguration is called and our current implementation relies on being able to change root path from /services to /rest. So yes it is a blocker for me :) If we can't have a new release we'd have to go back to 4.1.1

[hstaudacher]

I see. Will try to create the service release this afternoon.

[kolkoo]

Thanks a lot!

[hstaudacher]

4.2.1 is released. In will take a few hours before it's visible in maven central. I only updated the publisher.

[kolkoo]

Thanks again for this.

[hstaudacher]

No problem. Just keep the pull requests coming ;)