Lacro59 / playnite-successstory-plugin

This plugin shows the game achievements in Playnite.
https://playnite.link
MIT License
167 stars 29 forks source link

Error When Importing Games Marked Private #469

Closed burnpsy closed 2 months ago

burnpsy commented 6 months ago

This add-on sends an error if a game is added to Playnite that is marked as private on Steam using the recent Steam feature that allows me to do so.

Here is part of the relevant bit of extensions.log, for hopefully obvious reasons I am intentionally leaving out the lines directly before them that list the privated games by name:

01-04 17:19:27.097|INFO |SuccessStory#SteamAchievements:GetAchievements() - IsLocal : False, IsManual : False, HasApiKey: True, SteamIsPrivate: False 01-04 17:19:27.301|INFO |SuccessStory#SteamAchievements:Old 01-04 17:19:27.439|WARN |SuccessStory#SteamAchievements:Steam profil is private 01-04 17:19:27.439|ERROR|SuccessStory#Common:Error on GetPlayerAchievements()|G:\GitHub\Lacro59\playnite-successstory-plugin\source\Clients\SteamAchievements.cs|680 System.Net.WebException: The remote server returned an error: (403) Forbidden.

I'm not sure if anything can be done to get the achievement data in this situation, but at the very least there needs to be a rewording of the error message; my Steam account is not private, but the warning is saying that it is. No, the specific games are private, which is intentional.

epeternally commented 4 months ago

Are you using an API key? If you can't access the achievement list even with a key, I feel like that's a bug on Steam's end.

burnpsy commented 4 months ago

Yes, I am using an API key.

Lacro59 commented 2 months ago

Can you try this? https://github.com/Lacro59/playnite-successstory-plugin/issues/479#issuecomment-2191342743

burnpsy commented 2 months ago

Can you try this? #479 (comment)

Installed this. Behaviour is as follows when used on games marked private:

The error noted in my OP does not show itself when using the test version.

If you go with this as the solution, I'd like to request a button to clear the SuccessStory data for selected games (so I simply have no achievement list for them, rather than it saying my progress is 0).

Lacro59 commented 2 months ago

@burnpsy, can you post your extension.log?

burnpsy commented 2 months ago

Attached. [Removed]

When I tried reproducing the object reference error I saw in my last message, I noticed it did not result in any note in the log, and also refreshing the data on an affected game fixed the issue.

Lacro59 commented 2 months ago

If the game is private, there is only a way to get the achievement: as a private account. I'll show a message to notify it.