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

Edit suggestions #5

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ViktorLi1988
Copy link

Added Base64 encoding for basicauth - otherwise did not work.
Saved QueryParams in the params varieble OnBeforeRequest, because previously user defined params were not set.
Edited some of the structs to have json.RawMessage because sometimes Meta data can be of different type other than pure string.

@hiscaler
Copy link
Owner

hiscaler commented Nov 8, 2024

request.QueryParam will be foreach and set to url.values, why params use request.QueryParam to default value?
Can you give me a example? Thank you.

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.

3 participants