Repository navigation
fix(data-classes): use user_uid for the Uid in Transfer Family PosixProfile - #8524
Conversation
|
Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need. |
leandrodamascena
left a comment
There was a problem hiding this comment.
Thanks @LuisFigueroaG, nice catch! I was able to reproduce the swapped Uid and confirm the fix across the EFS scenarios, including uid 0, logical home directories and responses with public keys and policy. Approving!
|
|
Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience! |
|
Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience! |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #8524 +/- ##
========================================
Coverage 96.92% 96.92%
========================================
Files 317 317
Lines 15976 15976
Branches 1413 1413
========================================
Hits 15484 15484
Misses 349 349
Partials 143 143 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|



Issue number: closes #8522
Summary
Changes
build_authentication_response_efsset thePosixProfileUidtouser_gid. It now usesuser_uid. Added a test with different uid and gid values, which fails ondevelop.User experience
Before:
user_gid=1000, user_uid=1001produced{"Gid": 1000, "Uid": 1000}.After: it produces
{"Gid": 1000, "Uid": 1001}, so EFS users get the configured POSIX user id.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.