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.

Sign up for GitHub

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

Jump to bottom

Add & to allowed characters in sanitized headers. #831

Merged
merged 2 commits into from
Jul 6, 2023
Merged

Add & to allowed characters in sanitized headers. #831

merged 2 commits into from
Jul 6, 2023

Conversation

claudiamurialdo
Copy link
Collaborator

@claudiamurialdo claudiamurialdo commented Jun 20, 2023 •
edited
Loading

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 20, 2023 21:59 — with GitHub Actions Inactive
@claudiamurialdo
Add more characters to log white list to print correctly messages like
06a67ba 
GXLogging.DebugSanitized(log, "HttpContext.Session.setProperty(", key, ")=", value);
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests June 22, 2023 20:43 — with GitHub Actions Inactive
@claudiamurialdo claudiamurialdo merged commit 5c1e2f2 into master Jul 6, 2023
@claudiamurialdo claudiamurialdo deleted the add-amp-to-header-whitelist branch July 6, 2023 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgrampone sgrampone Awaiting requested review from sgrampone

Assignees
No one assigned
Labels
bot closed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@claudiamurialdo @genexusbot