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

Preflight request with the OPTIONS are needed #2

Open
zbouziane opened this issue Mar 7, 2017 · 2 comments
Open

Preflight request with the OPTIONS are needed #2

zbouziane opened this issue Mar 7, 2017 · 2 comments
Assignees

Comments

@zbouziane
Copy link
Member

Especially in FeedVersionController::register, FeedSourceController::register, ProjectController::register and UserController::register.

@landonreed landonreed self-assigned this Mar 8, 2017
@landonreed
Copy link
Member

@zbouziane, could you elaborate on what you need additional preflight options for? Would a blanket wildcard for all routes (api/manager/*) work or do you need individual options routes specified with the precise http verbs available for that route?

I assume this is to make the proxy work?

@zbouziane
Copy link
Member Author

zbouziane commented Mar 28, 2017

I don't think your wildcard could solve this. It has to do with the CORS paradigm. See the following from mozilla

"The Cross-Origin Resource Sharing standard works by adding new HTTP headers that allow servers to describe the set of origins that are permitted to read that information using a web browser. Additionally, for HTTP request methods that can cause side-effects on user data (in particular; for HTTP methods other than GET, or for POST usage with certain MIME types). The specification mandates that browsers "preflight" the request, soliciting supported methods from the server with an HTTP OPTIONS request header, and then, upon "approval" from the server, sending the actual request with the actual HTTP request method. Servers can also notify clients whether "credentials" (including Cookies and HTTP Authentication data) should be sent with requests."

FYI For me the server and the web app are deployed on two disjoint machines.

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

Successfully merging a pull request may close this issue.

2 participants