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

Add a few things to the negated set of the URL regex #2708

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

cicdw
Copy link
Member

@cicdw cicdw commented Sep 5, 2024

Adds (,", and ) to the negated set at the end of the regex for URLs.

Closes PrefectHQ/prefect#15239

@cicdw cicdw added the fix label Sep 5, 2024
@cicdw cicdw requested a review from a team as a code owner September 5, 2024 21:22
Copy link
Contributor

@znicholasbrown znicholasbrown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@znicholasbrown znicholasbrown merged commit 92e08a4 into main Sep 5, 2024
2 checks passed
@znicholasbrown znicholasbrown deleted the fix-url-regex branch September 5, 2024 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

URLs in logs include trailing parentheses
2 participants