<p></p>
<p dir="auto">On further inspection I'm not sure I can see any path which needs this - it's certainly inconsistent that we allow provider but not uid but I'm not sure we need either?</p>
<p dir="auto">If they're not changed then we don't need to update them, and if they are then we're going to take the other branch and redirect based on the raw parameters, and then the auth callback will fill them both in when it completes.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4332#issuecomment-1802438739">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLOC3JLMZPMG624TY3YDPGEJAVCNFSM6AAAAAA7DFNDO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBSGQZTQNZTHE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNQDBZAZ2F5EKHAE2LYDPGEJA5CNFSM6AAAAAA7DFNDO6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTLN4EFG.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/4332/c1802438739</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/4332#issuecomment-1802438739",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4332#issuecomment-1802438739",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>