Skip to content

Update-Process - Change the way the GUID is expected to exist#1349

Merged
Trouffman merged 12 commits into
masterfrom
plugin-update-tweak-1171
Nov 12, 2025
Merged

Update-Process - Change the way the GUID is expected to exist#1349
Trouffman merged 12 commits into
masterfrom
plugin-update-tweak-1171

Conversation

@Trouffman

@Trouffman Trouffman commented Aug 29, 2025

Copy link
Copy Markdown
Collaborator

This PR should address the challenge set on #1171

This would effectively read the GUID set by OBS and never write it.

We will need a way to detect a new "install" maybe with a placeholder value for the update check.

This is because on a FIRST clean OBS launch, the GUID does not exist until there is a "save" event in OBS (settings change or OBS close).

This means that when loading the plugin, that GUID does not exist yet.

The Plugin should never write this because :

  • This is not the plugin configuration space (aka do not mess with others' settings, especially not OBS)
  • This could be used/read but never created or modified.

This can be addressed by :
A. Only read the GUID if it exists, use a fallback & identifiable value if not.

B. Create our own GUID in our own config space.

This PR implement option A.

@Trouffman

Copy link
Copy Markdown
Collaborator Author

The process now only "read" the Install GUID from OBS "General".

If no GUID is detected (meaning that OBS is launched for the first time) this will returned a dummy identifiable GUID (needed for the update process)

This is set to noguiddetectednewfreshinstalldistroavobs for now, it's 40 characters long like the expected GUID.

image

This is also managed in the log entries to be easier to debug :
[distroav] No OBS GUID detected - Assuming this is a new installation

This uses the OBS GUID as soon as it exists (aka when a "save" event occurs.)

@Trouffman
Trouffman marked this pull request as ready for review August 29, 2025 15:44
@Trouffman Trouffman self-assigned this Aug 30, 2025
@Trouffman Trouffman added this to the 6.2.0 milestone Aug 30, 2025
@Trouffman
Trouffman requested a review from paulpv September 7, 2025 22:42
@Trouffman

Copy link
Copy Markdown
Collaborator Author

@paulpv please review this change as it update the behavior you initially put in place.

@Trouffman

Copy link
Copy Markdown
Collaborator Author

Note :
We now only use CalculateFileHash() from shared-update maybe this could be merge into the update.cpp to simplify code / files / deps

@Trouffman
Trouffman requested a review from BitRate27 October 22, 2025 19:53
@Trouffman
Trouffman merged commit 6ee4ff4 into master Nov 12, 2025
9 checks passed
@Trouffman
Trouffman deleted the plugin-update-tweak-1171 branch November 12, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant