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

chore: example handling request parameters with httpin #518

Merged
merged 1 commit into from
Nov 11, 2022

Conversation

ggicci
Copy link
Contributor

@ggicci ggicci commented Nov 9, 2022

No description provided.

@emicklei
Copy link
Owner

hi, your contribution clearly is a promotion of your package and demonstration how to integrate with go-restful. A few other examples have the same goal, for different packages of course.

I am willing to accept your PR with the disclaimer that you yourself are responsible for maintaining the example. If at some point in time the example is broken of some period of time then it may be removed from the repository.

@ggicci
Copy link
Contributor Author

ggicci commented Nov 11, 2022

thanks @emicklei for the clarification, i will keep the example updated :)

@emicklei emicklei merged commit 0c527fa into emicklei:v3 Nov 11, 2022
emicklei added a commit that referenced this pull request Aug 5, 2023
* allow multiple samples for Write, issue #514

* update changelog

* chore: example handling request parameters with httpin (#518)

* use path package to join slash fragments #519 (#520)

* update hist

* update example openapi to use 3.10.1

* Add test for client request with and without trailing slash. (#522)

* Add test for client request with and without trailing slash.

* Correction.

* introduce MergePathStrategy

* Revert "introduce MergePathStrategy"

This reverts commit 709cf80.

* introduce MergePathStrategy for #521 #519 (#523)

* introduce MergePathStrategy for #521 #519

* update readme, set default to new strategy, add extra test

* link to security issue

* update change hist

* add hello world with TrimSlashStrategy

* two route example

* examples to show differences #519

* more route examples #519

* add examples for issue519 with path in root

* remove obsolete swagger example

* Update README.md

remover swagger12 mention

* allow multiple samples for Write, issue #514

---------

Co-authored-by: Ggicci <[email protected]>
Co-authored-by: Gerrit <[email protected]>
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