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

uri: do not use user/password/port with file #1838

Closed
wants to merge 2 commits into from

Conversation

watilde
Copy link

@watilde watilde commented Mar 13, 2018

Related to #1832, but this patch is just for a case arg_check=true.

@k0kubun k0kubun changed the base branch from trunk to master August 15, 2019 17:34
@k0kubun k0kubun requested a review from nurse December 31, 2019 08:45
@nurse
Copy link
Member

nurse commented Jan 3, 2020

04883f1 also introduced a validation for a case arg_check=true.

@nurse nurse closed this Jan 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants