PathOfBuildingCommunity / PathOfBuilding

Offline build planner for Path of Exile.
https://pathofbuilding.community
Other
3.99k stars 2.08k forks source link

'Account is Private' after patch bug #8360

Open vietknight opened 4 hours ago

vietknight commented 4 hours ago

Check version

Check for duplicates

What platform are you running Path of Building on?

Windows

How is Path of Building expected to behave?

Import/Export Build -> Type in username -> Start -> Characters should load

How does Path of Building behave?

Import/Export Build -> Type in username -> Start -> 'Account is private'

How to reproduce the issue

  1. Import/Export Build
  2. Type in username
  3. Press 'Start'
  4. Tried with/without special discriminator value, still did not work
  5. Tried with special POESESSID, still did not work

Character build code

No response

Screenshots

image

image

image

image

image

image

ciceroripi commented 4 hours ago

Same here.

Regisle commented 3 hours ago

If you open the console (ctrl + ~), you should see a 403 response code, this is a known issue and is being looked into.

Sentinent commented 3 hours ago

I did some poking around and the issue is here: https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/dev/src/Classes/ImportTab.lua#L452

local realAccountName = response.body:match("/view%-profile/([^/]+)/characters"):gsub(".", function(c) if c:byte(1) > 127 then return string.format("%%%2X",c:byte(1)) else return c end end)

returns <account name>-<discrim>, which later gets passed to /get-passive-skills and /get-items that causes the issue. A quick dirty hack of adding realAccountName = realAccountName:gsub("-", "%%23") fixes the issue locally for me.

Note that it's the POE site that is introducing the - in <account name>-<discrim>, POB is just a victim here :'(

--

Somewhat tangential, I think support should be added to automatically urlencode the "#" to %23. Much more UX friendly to be able to enter myaccount#1234 instead of myaccount%231234

OcularDisease commented 49 minutes ago

Same problem here

franciskong-lfasystems commented 14 minutes ago

Also having the same issue here. Was working fine last night.

LocalIdentity commented 12 minutes ago

GGG changed the account system in the patch today. We have it fixed on dev and I'm about to write patch notes with the update coming out within the next hour