Skip to content

Comments

Create obfuscated_private_ip object when LoginID is provided#845

Merged
yaakov-h merged 1 commit intoSteamRE:masterfrom
xPaw:fix-loginid
May 14, 2020
Merged

Create obfuscated_private_ip object when LoginID is provided#845
yaakov-h merged 1 commit intoSteamRE:masterfrom
xPaw:fix-loginid

Conversation

@xPaw
Copy link
Member

@xPaw xPaw commented May 14, 2020

No description provided.

@JustArchi
Copy link
Contributor

Maybe it'd be a good idea to somehow calculate v6 from LoginID as well in this case? I mean, it's not any real IP for the expected use case, just a value we use for session identification purposes, we might as well do our best guess and supply a value there as well.

@xPaw
Copy link
Member Author

xPaw commented May 14, 2020

The question is whether user should be able to provide their own v6 loginid or not. If not, it could be simply set into the v6 field as bytes (with 96 bits remaining zeroes).

We opted to wait for Steam to actually have IPv6 servers, then we could decide what to do with it.

This PR is a null fix, not a logic change.

@yaakov-h
Copy link
Member

@yaakov-h yaakov-h merged commit c1261ae into SteamRE:master May 14, 2020
@xPaw xPaw deleted the fix-loginid branch May 14, 2020 10:25
@JustArchi
Copy link
Contributor

JustArchi commented May 14, 2020

I was thinking about it too, with the initial idea of "let's just add loginIDv6 then" and realized that it defeats the purpose of loginID - we don't really want to provide fake IP addresses there, only allow multiple sessions from the same machine without going through hoops, in fact, I'd appreciate if that was no longer needed, but I checked a few months ago and sadly it was still in-place.

So my idea is we should just use loginID as arbitrary session number, and how it translates to Steam's backend should remain SK2 implementation detail, as to not give user the false impression of what it's used for.

shady

@yaakov-h
Copy link
Member

with IPv6 we could just assign a million different addresses to one machine... 🤔

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.

3 participants