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

Fix setting basic auth for curl to Inkdrop local server #2

Merged
merged 1 commit into from
May 24, 2021

Conversation

picklecillo
Copy link
Contributor

@picklecillo picklecillo commented May 23, 2021

Passing user:password as part of the url for curl was throwing a formatting error because, at least in my case, the user is an email address.

> curl http://[email protected]:[email protected]:19840/notes/?limit=20&keyword=test
curl: (3) URL using bad/illegal format or missing URL

As this has ben deprecated (see here), I switched it to passing the CURLOPT_USERPWD option in order to get the workflow working.

Hope it helps.

Also: This is the related issue -> #1

@picklecillo picklecillo mentioned this pull request May 23, 2021
@craftzdog craftzdog merged commit d97b6cb into inkdropapp:main May 24, 2021
@craftzdog
Copy link
Contributor

Makes sense! I didn't know the format user:password has been deprecated.
Thank you so much for fixing this.

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