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

CORS-middleware-use-explicit-origin-with-credentials #2119

Conversation

Swati-Sneha
Copy link

@Swati-Sneha Swati-Sneha commented Apr 12, 2023

The starting point for contributions should usually be 1832

Simple documentation typos may be raised as stand-alone pull requests, but otherwise please ensure you've discussed your proposal prior to issuing a pull request.

This will help us direct work appropriately, and ensure that any suggested changes have been okayed by the maintainers.

  • Initially raised as discussion #...

@adriangb
Copy link
Member

Can you clarify what this is and link to an issue? Otherwise we’ll have to close this PR

@Swati-Sneha
Copy link
Author

Swati-Sneha commented Apr 13, 2023 via email

@Swati-Sneha Swati-Sneha marked this pull request as draft April 13, 2023 06:27
@Kludex
Copy link
Member

Kludex commented Apr 13, 2023

Please change the title to something more suitable, add a better description, and as I said before: I need to reread the issue, and we need to know what's the behavior of the others web frameworks (flask, django, and maybe others because I believe those two differ in this particular point).

@Swati-Sneha Swati-Sneha changed the title test CORS-middleware-use-explicit-origin-with-credentials Apr 15, 2023
@Kludex
Copy link
Member

Kludex commented Apr 28, 2023

@Swati-Sneha Are you still interested in this? 👀

@Kludex
Copy link
Member

Kludex commented May 31, 2023

@Swati-Sneha I'll be closing this for the time being, feel free to reopen based on our comments. 👍

@Kludex Kludex closed this May 31, 2023
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 this pull request may close these issues.

4 participants