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

TO Go GET /user/current should have 'roleName' not 'rolename' #2535

Merged
merged 1 commit into from
Jul 12, 2018

Conversation

rawlinp
Copy link
Contributor

@rawlinp rawlinp commented Jul 11, 2018

The field needs to be camelcased like it is in the Perl version,
otherwise clients are broken (i.e. TP which checks this field when
determining if a user can automatically fulfill a DSR).

The field needs to be camelcased like it is in the Perl version,
otherwise clients are broken (i.e. TP which checks this field when
determining if a user can automatically fulfill a DSR).
@asfgit
Copy link
Contributor

asfgit commented Jul 11, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2087/
Test PASSed.

@mitchell852 mitchell852 added bug something isn't working as intended Traffic Ops API (golang) regression bug a bug in existing functionality introduced by a new version labels Jul 12, 2018
@mitchell852 mitchell852 merged commit fbe294e into apache:master Jul 12, 2018
@rawlinp rawlinp deleted the fix-current-user-rolename branch July 12, 2018 13:31
@mitchell852 mitchell852 removed the bug something isn't working as intended label Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression bug a bug in existing functionality introduced by a new version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants