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

Unsupported URLs [BUG]? #229

Closed
seyuf opened this issue Oct 29, 2019 · 3 comments
Closed

Unsupported URLs [BUG]? #229

seyuf opened this issue Oct 29, 2019 · 3 comments
Labels
need information Requires more info

Comments

@seyuf
Copy link

seyuf commented Oct 29, 2019

The send button is deactivated when url is:
https:user:password-@api.domain.fr
when password is ending with special character (i.e " - "), the url is considered invalid, one can't make a request.

Everything is working fine when entering:
https:user:password@api.domain.fr
or
https:user:password-pwd@api.domain.fr

Error when - at the end of password
error-when-url

The send button should be active e.g
no-error-when-url

@seyuf seyuf changed the title unsupported URL unsupported URLs [BUG] Oct 29, 2019
@seyuf seyuf changed the title unsupported URLs [BUG] unsupported URLs [BUG]? Oct 29, 2019
@seyuf seyuf changed the title unsupported URLs [BUG]? Unsupported URLs [BUG]? Oct 29, 2019
@liyasthomas
Copy link
Member

I couldn't reproduce the bug. By the way, I don't think the URL you mentioned seems to a valid url at all.

@liyasthomas liyasthomas added the need information Requires more info label Oct 30, 2019
@reefqi037
Copy link
Contributor

I think @seyuf meant to send username and password together with the URL in this form:
http://username:password@example.com

If there's - at the end of the password, for example http://username:password-@example.com, Postwoman will consider the URL invalid, thus disabling the send button.

You can achieve the same thing by using Basic Type Authentication function. Type
in username and password in the field and just use http://example.com in the URL instead.

@seyuf
Copy link
Author

seyuf commented Oct 30, 2019

Hi @liyasthomas , what @reefqi037 said, let me know if you need more info?

Hi @reefqi037, thanks for the workaround.

I still think this is an issue though, feel free to close this if you think otherwise.

Great project by the way, many thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need information Requires more info
Projects
None yet
Development

No branches or pull requests

3 participants