launchpad-reviewers team mailing list archive

Thread
Date

Re: [Merge] ~ilasc/launchpad:bug-1852482 into launchpad:master

To: mp+375614@xxxxxxxxxxxxxxxxxx
From: Ioana Lasc <ioana.lasc@xxxxxxxxxxxxx>
Date: Tue, 19 Nov 2019 11:57:55 -0000
In-reply-to: <157383080450.16442.15693917236301725054.launchpad@ackee.canonical.com>
Reply-to: mp+375614@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Indeed the non-ASCII character in the displayname seems to be the problem Colin.

On a quick test with the local DB Person with id=43, display name=André Luís Lopes and:
   other_user = IStore(Person).find(Person, name='andrelop').one()

unit test on the webservice revealed the same behavior we're seeing in Staging and Prod:

HTTP/1.1 500 Internal Server Error
Content-Length: 147
Content-Type: text/html;charset=utf-8

<html><head><title>GitRepositoryCreatorNotOwner</title></head>
<body><h2>GitRepositoryCreatorNotOwner</h2>
A server error occurred.
</body></html>


Will create a Person now with non-ASCII in displayname for the Unit Test and attempt to figure out where the status gets flipped to 500.




-- 
https://code.launchpad.net/~ilasc/launchpad/+git/launchpad/+merge/375614
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilasc/launchpad:bug-1852482 into launchpad:master.

References

[Merge] ~ilasc/launchpad:bug-1852482 into launchpad:master
From: Ioana Lasc, 2019-11-15