Skip to content

Commit

Permalink
Fix #45
Browse files Browse the repository at this point in the history
  • Loading branch information
@Dadoum
Dadoum authored Feb 3, 2023
1 parent 7b3a488 commit fe8bf4c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/provision/adi.d
View file
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ alias ADIGetIDMSRouting_t = extern(C) int function(ulong*, ulong);
__gshared ADISetIDMSRouting_t pADISetIDMSRouting;
__gshared ADIGetIDMSRouting_t pADIGetIDMSRouting;

string __clientInfo = "<iMac20,2> <Mac OS X;13.1;22C65> <com.apple.AuthKit/1 (com.apple.dt.Xcode/3594.4.19)>";
string __clientInfo = "<MacBookPro17,1> <macOS;12.2.1;21D62> <com.apple.AuthKit/1 (com.apple.dt.Xcode/3594.4.19)>";
public @property string clientInfo() shared {
return __clientInfo;
}
Expand Down

4 comments on commit fe8bf4c

@Dadoum
Copy link
Owner Author

@Dadoum Dadoum commented on fe8bf4c Mar 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should search for another client info

@Dadoum
Copy link
Owner Author

@Dadoum Dadoum commented on fe8bf4c Mar 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay that's trickier because now they expect more than a simple client info change @WhaleHub,
but I got no mac to reverse this so yet we will keep macOS 12.2.1 client info.

@crystall1nedev
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay that's trickier because now they expect more than a simple client info change @WhaleHub

But didn't whalehub (and I actually) have it working fine with the simple client info change? What else is there that needs to be sent?

@crystall1nedev
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's interesting is that I tested the commit that this is supposed to fix vigorously. Docker, natively, on multiple machines, with multiple Apple IDs and even going as far as to use different IPs - they all worked in my testing. I can't really see why it would cause Docker issues or error 401, other than the mishap of pushing Mac OS X instead of macOS, however I can see that in Discord you tested different client infos and found this one to be the most reliable.

Please sign in to comment.