-
Notifications
You must be signed in to change notification settings - Fork 18
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
fix (webapi): CORS problems #307
Conversation
@github: Github has also some bugs. The last four changes where made by me (@subotic) and not @tobiasschweizer |
This should work now. @tobiasschweizer could you please test it again with Salsah? |
I have a look at it right now. |
implicit def corsRejection = corsRejectionHandler | ||
|
||
/** | ||
* Bring the Knora exception handler into scope, which is responsible for converting our exceptions to HttpResponses |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Brings
Nice job! Now I can delete resources and I get proper error messages on failed attempts to manipulate values. |
Thanks :-) Can I merge? |
Yes, I think so. There is a small typo I found in the documentation you could fix before merging. |
The typo is in code that was deleted ;-) |
Still I want to fix it! joking, sorry, my fault |
* develop: Improve configuration for more concurrent requests (#338) Upgrade to Scala 2.12 (#343) Fix various inconsistencies in knora-base.ttl and docs. (#330) Fix language switching and login/logout in SALSAH (#331) Graph data API operation (#267) Allow connections over HTTPS (#332) Update README.md Add link to GraphDB. fix (salsah): Add missing JavaScript “var”. fix (webapi): Fix RouteUtilV1 so it returns HTML instead of JSON when requested (#326) Ontology and documentation fixes. (#325) fix (salsah): Create multiple links in a resource, and other bug fixes (#315) feature (webapi): Make TransformData fix “test” labels on regions. (#311) Fixes for subclasses and subproperties (#303) Add script to update docs on knora.org (#300) docs: Make trivial change for testing purposes. fix (webapi): CORS problems (#307) Use inference to optimise searches on GraphDB (#301)
Access-Control-Allow-Origin
is missing in response header and thus the browser blocks the answer (in responses with status code other than 200).