Roundup Tracker - Issues

Issue 2551383

classification
Setting same address via REST PUT command results in an error, should pass.
Type: behavior Severity: normal
Components: Database, Web interface, API Versions:
process
Status: fixed fixed
:
: rouilj : rouilj
Priority: normal :

Created on 2024-12-17 22:07 by rouilj, last changed 2024-12-18 03:32 by rouilj.

Messages
msg8231 Author: [hidden] (rouilj) Date: 2024-12-17 22:07
While adding fuzz testing to Roundup, I was testing setting random email addresses via a PUT 
command for the user's address property. I would get a 409 error when setting the email
address to the same value it already had. The error message was correct:

   '"Email address _@A.toyoTa already in use'

But this should be a no-op if the user is setting his/her own email. I tracked this down
to the userauditor.py.

If the email address is in use by the same user who is changing it, don't report
a conflict.

This lead to changing the docs for the REST interface as the result from the PUT
for a property url was not documented properly. The output is more akin to a PUT
to the item including only reporting an attribute if it has changed.
msg8232 Author: [hidden] (rouilj) Date: 2024-12-18 03:32
fixed in changeset:   8218:32aaf5dc562b

which broke python2 testing.

changeset:   8220:818751637b77 works around python2 test breakage.
I didn't spend time trying to make the bad json tests work on py2. Just skipping
the test in that platform.
History
Date User Action Args
2024-12-18 03:32:50rouiljsetstatus: new -> fixed
resolution: fixed
messages: + msg8232
2024-12-17 22:07:22rouiljcreate