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

Remove EXPOSE_HEADER option #97

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

rpep
Copy link

@rpep rpep commented Mar 2, 2023

Match functionality from https://github.com/snok/asgi-correlation-id and remove EXPOSE_HEADER

@sondrelg sondrelg requested a review from JonasKs March 3, 2023 08:42
@JonasKs
Copy link
Member

JonasKs commented Mar 3, 2023

Some tests still fail 😊 I'd also love some docs on how to configure CORS😊

@rpep
Copy link
Author

rpep commented Mar 16, 2023

@JonasKs think that's alright now and have added the CORS documentation. I've had a look at failing tests and seems to be unrelated - they're currently failing to get past the install stage due to this:
image

Any ideas?

@JonasKs
Copy link
Member

JonasKs commented Mar 16, 2023

Thanks for the PR 😊

Huh, weird. I'll have to look, but I'm away for a week or so now.

@rpep
Copy link
Author

rpep commented Mar 17, 2023

@JonasKs no worries at all. Give me a ping if you need me to try anything when you’re back :)

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.

2 participants