-
Notifications
You must be signed in to change notification settings - Fork 48
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
Why submit form values via GET instead of POST? #39
Comments
A GET makes it easy to hack the url. You are welcome to add a POST
configuration to the project by submitting a pull request.
Be warned that although this project works and you are not the only user,
the code itself is dreadful. And I am extremely sorry about that. If I had
time I would totally rewrite it.
…On Sun, 3 Dec 2017, 01:24 Anwar Reddick, ***@***.***> wrote:
This is a great project! But why does the generated UI submit form values
as URL parameters instead in a GET request, instead of as form POST
parameters? Is there a configuration option to make the UI use POST?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#39>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABvsLXvnnyTipJfxCz1PKDjxPKlm2JALks5s8epJgaJpZM4QzhwY>
.
|
Noted, and no need to apologize. I understand how other priorities can get in the way. And I'm actually glad to hear your response. This project has inspired me to consider a fork that uses go templates to provide more modularity. Not sure if I'll have time to pursue that though, or the POST functionality. |
I would also recommend looking into getting the filedescriptor from a grpc method and generating the form dynamically in the javascript or your frontend language of choice. I would use Elm :) |
Sounds like a good idea. I'm at a Typescript shop though so that would probably be my goto frontend. |
fair enough :) |
This is a great project! But why does the generated UI submit form values as URL parameters instead in a GET request, instead of as form POST parameters? Is there a configuration option to make the UI use POST?
The text was updated successfully, but these errors were encountered: