Skip to content

feat: allow for authenticating Steam users without official#679

Draft
AnthonyFuller wants to merge 14 commits into
masterfrom
tony/steam-auth
Draft

feat: allow for authenticating Steam users without official#679
AnthonyFuller wants to merge 14 commits into
masterfrom
tony/steam-auth

Conversation

@AnthonyFuller
Copy link
Copy Markdown
Contributor

@AnthonyFuller AnthonyFuller commented Mar 22, 2026

As title. Validates using the app ticket.

Pretty much done aside from the backend implementation (and final polish).

@AnthonyFuller
Copy link
Copy Markdown
Contributor Author

AnthonyFuller commented Mar 22, 2026

Not a clue as to why merging from master causes esbuild to start failing...

@grappigegovert
Copy link
Copy Markdown
Member

Looks good from a quick glance, but I don't see why you would hash the ownership ticket; isn't Map already implemented as a hash map internally?

@AnthonyFuller
Copy link
Copy Markdown
Contributor Author

It is, the hash is more for a reference for when RDIL does the backend implementation so he knows what to hash.

@AnthonyFuller
Copy link
Copy Markdown
Contributor Author

AnthonyFuller commented Mar 28, 2026

Okay, so updating node fixed esbuild failing. Good to know.
EDIT: Apparently not it just successfully built? Very confusing - I must've opened the previous run or something...

@AnthonyFuller
Copy link
Copy Markdown
Contributor Author

I've removed the dependency on steam-appticket and added a util function to parse app tickets (it only parses a subset of the data - the parts we actually use).

Now we just need the backend implementation.

@AnthonyFuller
Copy link
Copy Markdown
Contributor Author

AnthonyFuller commented Mar 30, 2026

Okay, this is basically done now - just requires the backend side of things to be finalised (one or two things to iron out, otherwise that's done too).

Reminder: update the default of steamAuthenticationMethod to BACKEND.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants