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

[BUG] mismatch between documentation and requests #337

Closed
fmigneault opened this issue Jun 18, 2020 · 0 comments · Fixed by #340
Closed

[BUG] mismatch between documentation and requests #337

fmigneault opened this issue Jun 18, 2020 · 0 comments · Fixed by #340
Assignees
Labels
bug Problem, error, or invalid behaviour doc Documentation improvements or building problem

Comments

@fmigneault
Copy link
Collaborator

fmigneault commented Jun 18, 2020

Documentation indicates some 'current' user routes as valid (example below, maybe others also)
https://pavics-magpie.readthedocs.io/en/latest/api.html#tag/Logged-User%2Fpaths%2F~1users~1current%2Fput

But this is clearly forbidden here:

user_name = ar.get_value_matchdict_checked(request, key="user_name")
ax.verify_param(user_name, param_compare=get_constant("MAGPIE_LOGGED_USER"), not_equal=True,
http_error=HTTPBadRequest, param_name="user_name", content={u"user_name": user_name},
msg_on_fail=s.Service_PUT_BadRequestResponseSchema_ReservedKeyword.description)

@fmigneault fmigneault added bug Problem, error, or invalid behaviour doc Documentation improvements or building problem labels Jun 18, 2020
@fmigneault fmigneault self-assigned this Jun 18, 2020
fmigneault added a commit that referenced this issue Jul 28, 2020
… & fix doc (#337) + add checks for udpate special users (#164)
fmigneault added a commit that referenced this issue Jul 28, 2020
… & fix doc (#337) + add checks for udpate special users (#164)
fmigneault added a commit that referenced this issue Jul 30, 2020
… & fix doc (#337) + add checks for udpate special users (#164)
fmigneault added a commit that referenced this issue Jul 30, 2020
… & fix doc (#337) + add checks for udpate special users (#164)
fmigneault added a commit that referenced this issue Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Problem, error, or invalid behaviour doc Documentation improvements or building problem
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant