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

Need to use GET instead of POST for it to work #4

Open
vipergts450 opened this issue Oct 4, 2021 · 1 comment
Open

Need to use GET instead of POST for it to work #4

vipergts450 opened this issue Oct 4, 2021 · 1 comment

Comments

@vipergts450
Copy link

vipergts450 commented Oct 4, 2021

Hi Ian,

Thank you for this great project.

Doing some debugging, and it seems like your code looks at "request.arg" which I believe is populated via GET vs "form" for POST. It works fine when reformatting the curl request as a GET request, and since I'm requiring HTTPS, it shouldn't present a security issue, but I wanted to see if you had noticed this in your usage?

@ianbrown78
Copy link
Owner

Hi Viper,

Thanks for the feedback.
You are very correct and it is something that I should fix in the github repo. Feel free to submit a PR and I will merge it.
I don't see this issue as the version I use is quite different from the one here. :)

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

No branches or pull requests

2 participants