deathandmayhem / jolly-roger

Dead men tell no tales!
MIT License
18 stars 5 forks source link

Make Discord OAuth not require `guilds.join` scope #439

Open zarvox opened 3 years ago

zarvox commented 3 years ago

Some people expressed concern at the fact that jolly-roger requests guilds.join, which allows us to make users members of our guild, when we could instead provide invite links (and require an additional click to accept the guild invite), to allow the user to drive that action rather than requiring a kinda-powerful scope on our tokens.

ebroder commented 3 years ago

The flip side is that without guilds.join we won't have an automatic way to link a JR user back to a Discord account. Unless people link their accounts manually (and based on experience with Slack, I worry that they won't), we won't have any way to do permission management and such.

zarvox commented 3 years ago

I don't think that's necessarily true? We can retain the behavior where you don't get access unless you've linked accounts in jolly-roger. Suppose the following:

zarvox commented 2 years ago

I think we should still do this, especially in light of the move to more hunt-attached behavior for external services rather than server-attached behavior for external services. In the hunt-attached model, I propose: