Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inaccurate update logic (latest ASFE version updating itself) #324

Open
Botan626 opened this issue Sep 30, 2024 · 14 comments
Open

Inaccurate update logic (latest ASFE version updating itself) #324

Botan626 opened this issue Sep 30, 2024 · 14 comments

Comments

@Botan626
Copy link

During updating ASF from v6.0.7.5 to v6.0.8.0 ASFEnhance was updated from v2.2.3.3 to v2.2.3.3.

2024-10-01 00:43:31|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Local version: 6.0.7.5 | Remote version: 6.0.8.0
2024-10-01 00:43:31|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Fetching checksum from the remote server...
2024-10-01 00:43:32|ArchiSteamFarm-1|INFO|ASF|UpdateASF() This is automated GitHub deployment, human-readable changelog should be available soon. In the meantime please refer to GitHub commits.

2024-10-01 00:43:32|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Downloading new version: 6.0.8.0 (18 MB)... While waiting, consider donating if you appreciate the work being done! :)
2024-10-01 00:43:33|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 0%...
2024-10-01 00:43:35|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 10%...
2024-10-01 00:43:36|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 20%...
2024-10-01 00:43:37|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 30%...
2024-10-01 00:43:38|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 40%...
2024-10-01 00:43:38|ArchiSteamFarm-1|INFO|03|FarmHours() Still farming: 262280
2024-10-01 00:43:39|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 50%...
2024-10-01 00:43:40|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 60%...
2024-10-01 00:43:40|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 70%...
2024-10-01 00:43:41|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 80%...
2024-10-01 00:43:42|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 90%...
2024-10-01 00:43:43|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASF-win-x64.zip 100%...
2024-10-01 00:43:43|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Verifying checksum of the downloaded binary against the one from the remote server...
2024-10-01 00:43:43|ArchiSteamFarm-1|INFO|Microsoft.Hosting.Lifetime|Application is shutting down...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Patching ASF files...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdatePlugins() Checking for plugin updates...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Checking update for ASFAchievementManager plugin...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Checking update for ASFEnhance plugin...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Checking update for CommandAliasPlugin plugin...
2024-10-01 00:44:14|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Checking update for CommandlessRedeem plugin...
2024-10-01 00:44:15|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() No update available for ASFAchievementManager plugin: 1.0.0.1 ≥ 1.0.0.1.
2024-10-01 00:44:15|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() Found ASFEnhance plugin update from version 2.2.3.3 to 2.2.3.3...
2024-10-01 00:44:15|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Updating ASFEnhance plugin...
2024-10-01 00:44:15|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() No update available for CommandAliasPlugin plugin: 1.0.0.2 ≥ 1.0.0.2.
2024-10-01 00:44:15|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() No update available for CommandlessRedeem plugin: 1.0.0.0 ≥ 1.0.0.0.
2024-10-01 00:44:16|ArchiSteamFarm-1|DEBUG|ASF|OnProgressChanged() ASFEnhance 0%...
2024-10-01 00:44:17|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Patching ASF files...
2024-10-01 00:44:17|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Updating ASFEnhance plugin has succeeded, the changes will be loaded on the next ASF launch.
2024-10-01 00:44:21|ArchiSteamFarm-1|INFO|ASF|UpdateASF() Update process finished!
@Botan626
Copy link
Author

@JustArchi
Copy link
Contributor

Not an issue with ASFE - ASF for compatibility reasons will update the plugin to the same version if more than one zip asset is found, since new ASF version could decide to use different asset than the one used in the previous run.

I have no clue why ASFE provides a zip asset per language though 🙂

@Botan626
Copy link
Author

Botan626 commented Sep 30, 2024

I have no clue why ASFE provides a zip asset per language though 🙂

So having only single ASFEnhance.zip archive in ASFE releases would fix it?

@JustArchi
Copy link
Contributor

Yes, although what you're describing is not an issue, plugin can force itself to be updated to the same version at any time it wants, and it should never cause you issues.

@Botan626
Copy link
Author

plugin can force itself to be updated to the same version at any time it wants

I'm pretty sure @chr233 never intended this.

@Botan626
Copy link
Author

Botan626 commented Sep 30, 2024

since new ASF version could decide to use different asset

And if the asset ASF decided to update to is the same as current one?

In this case ASFE is in the ASFEnhance-en-US directory (asset name?), version is the same (2.2.3.3), why updating?

@chr233
Copy link
Owner

chr233 commented Oct 1, 2024

Not an issue with ASFE - ASF for compatibility reasons will update the plugin to the same version if more than one zip asset is found, since new ASF version could decide to use different asset than the one used in the previous run.

I have no clue why ASFE provides a zip asset per language though 🙂

tried use resx for multiple language support, and it will output resource dlls like zh-hans/ASFEnhance.resources.dll, and the plugin will always use the default language, even if define a linguage in String.Format method (tested in ASF 5.x.x.x), so I decide to use this way, not elegant enough, but it works

@JustArchi
Copy link
Contributor

JustArchi commented Oct 1, 2024

All ASF plugins are shipped with multi-lang support and they work fine, so it seems ASFE does something different which is why it doesn't load. You definitely should not have a real need for that.

@chr233
Copy link
Owner

chr233 commented Oct 1, 2024

All ASF plugins are shipped with multi-lang support and they work fine, so it seems ASFE does something different which is why it doesn't load. You definitely should not have a real need for that.

You are right, this's a "historical issues"

@Botan626
Copy link
Author

Botan626 commented Oct 7, 2024

Now ASFE updates to v2.2.4.2.

2024-10-07 22:13:33|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() Found ASFEnhance plugin update from version 2.2.4.0 to 2.2.4.2...
2024-10-07 22:18:17|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Updating ASFEnhance plugin...

but after restart it's v2.2.4.0 again

2024-10-07 22:18:30|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() Found ASFEnhance plugin update from version 2.2.4.0 to 2.2.4.2...
2024-10-07 22:18:30|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() New ASFEnhance plugin version is available! Consider updating yourself!

image

Something's wrong.
It should update to v2.2.4.2, because my ASF is v6.0.8.3 and is Pre-Release, right?

@chr233
Copy link
Owner

chr233 commented Oct 8, 2024

Now ASFE updates to v2.2.4.2.

2024-10-07 22:13:33|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() Found ASFEnhance plugin update from version 2.2.4.0 to 2.2.4.2... 2024-10-07 22:18:17|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() Updating ASFEnhance plugin...

but after restart it's v2.2.4.0 again

2024-10-07 22:18:30|ArchiSteamFarm-1|INFO|ASF|GetTargetReleaseURL() Found ASFEnhance plugin update from version 2.2.4.0 to 2.2.4.2... 2024-10-07 22:18:30|ArchiSteamFarm-1|INFO|ASF|UpdatePlugin() New ASFEnhance plugin version is available! Consider updating yourself!

image

Something's wrong. It should update to v2.2.4.2, because my ASF is v6.0.8.3 and is Pre-Release, right?

just for testing github action, no need to upgrade this release

@chr233
Copy link
Owner

chr233 commented Oct 8, 2024

Now ASFEnhance will publish with all supported language together (english and russion), what's way to name plugin's resource libraries?

  1. .net's default way
    image

  2. named manually
    image

@Botan626
Copy link
Author

Botan626 commented Oct 8, 2024

just for testing github action, no need to upgrade this release

I know it's was for testing, but still it had to update to v2.2.4.2 if I'm correct, since both ASF and ASFE were pre-release.

--

ASFE was successfully updated to v2.2.5.0 now.

@Botan626
Copy link
Author

Botan626 commented Oct 8, 2024

Another thing is PV command output.

image

this line
Current version: 2.2.5.0 New version: 2.2.4.0
could be like
Installed version: 2.2.5.0 Stable version: 2.2.4.0

in order to differentiate between pre-release and stable ASFE versions.

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

No branches or pull requests

3 participants