feat:RedditSync - Path /u/ requests to use /user/ endpoint

[garpunkal]

Feature description

Patch Reddit Sync to use /user/ endpoint instead of /u/

https://reddit.com/u/spez/about.json (500 error)

vs

https://reddit.com/user/spez/about.json (works fine)

Motivation

Currently the About section in sync is broken for users because the endpoint provides a 500 error.

It's important because sync breaks on the user profile page.

Acknowledgements

• I have checked all open and closed feature requests and this is not a duplicate
• I have chosen an appropriate title.
• All requested information has been provided properly.

[trulow]

Can I offer a $50 USD bounty for this to be patched?

[oSumAtrIX]

Claiming (I already have changes locally that fix it, because I use Sync myself)

[trulow]

@oSumAtrIX

I DM'd you on reddit. Once the fix is live, send me your contact info to send the bounty.

[oSumAtrIX]

Here's the patch in action:

WsaClient_poqZwEQTx1.mp4

[trulow]

@oSumAtrIX

Thanks! I replied to your DM on reddit btw!

[oSumAtrIX]

Implemented with 46d11f3

[DVDAndroid]

I just checked this issue and my unpatched app and is working fine.
Do you think that this patch will be still needed?

[elmagio]

I just checked this issue and my unpatched app and is working fine. Do you think that this patch will be still needed?

Huh that's weird, it also started working again for me, without the new patch. Wasn't working as of an hour ago.

[oSumAtrIX]

It was a known bug on Reddits end confirmed by Reddit devs a couple days ago. It was bound to be fixed like the other login bugs caused by Reddit in the past. (The server returned 500, an internal error). The patch is not needed anymore and can be removed, but it won't make any difference if used.

[trulow]

Does you new patch have a check to see if the app receives a 500 error, to then use /user instead?

If it does, maybe we can still implement it as a backup.

[oSumAtrIX]

No, it simply changes the endpoint to /user. I think it's even the "new" endpoint and /u/ is deprecated but exists for backwards compatibility reasons. Should me mentioned in the docs

[trulow]

I just went over the API docs, and it seems like @oSumAtrIX is correct.

I think we should implement this patch as the API docs only reference /user/ and not /u/

My thinking on this is that it's only a matter of time until /u/ may break again causing another need fix in the future.

What does everyone else think?

[oSumAtrIX]

It won't make a difference. u/ wont break unless Reddit decides to drop backwards compatibility support at which point dozens of other apis would be broken and would need heavy fixing

[garpunkal]

I think we should have the patch available. If /user/ is the canonical URL then it gives the option to patch for future use.

[oSumAtrIX]

I'll keep it for now as it doesn't break anything but maybe the patch should be renamed to what it does, as it's called a fix whereas it doesn't fix current issue