Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ISPN-10967 CORS rules for localhost don't take offset into account #7606

Merged
merged 1 commit into from Nov 25, 2019

Conversation

gustavocoding
Copy link

@pruivo pruivo self-requested a review November 22, 2019 10:02
@pruivo
Copy link
Member

pruivo commented Nov 22, 2019

@gustavonalle how is it suppose to work if you bind the server to other IP than the localhost?
I bind the server to the wi-fi IP and I'm getting forbidden errors when I try to create a cache.

@pruivo
Copy link
Member

pruivo commented Nov 22, 2019

works fine for localhost!

@gustavocoding
Copy link
Author

@pruivo OOTB it contains rules for localhost only. For other IPs, configuration is needed atm.
https://infinispan.org/docs/stable/titles/rest/rest.html#rest_server_cors

@pruivo pruivo merged commit ba5562f into infinispan:master Nov 25, 2019
@pruivo
Copy link
Member

pruivo commented Nov 25, 2019

@gustavonalle ops! I clicked the merge button too soon. The failure org.infinispan.server.configuration.JsonSerializationTest.testJsonSerialization(JsonSerializationTest.java:198) seems related to your changes!

@gustavonalle thanks for the explanation!

@gustavocoding
Copy link
Author

@pruivo I'll take a look in the failure

@gustavocoding gustavocoding deleted the ISPN-10967_cors_offset branch November 25, 2019 11:39
@gustavocoding
Copy link
Author

@pruivo Fixed in #7616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants