Skip to content

v1.1 SDK throws an access violation from MddBootstrapInitialize's logging code if no matching packages are installed #2592 (#2608) - 1.1-servicing#2624

Merged
DrusTheAxe merged 2 commits intorelease/1.1-stablefrom
user/drustheaxe/39995920-2608-bootstrap-telemetry-crash
Jun 17, 2022
Merged

v1.1 SDK throws an access violation from MddBootstrapInitialize's logging code if no matching packages are installed #2592 (#2608) - 1.1-servicing#2624
DrusTheAxe merged 2 commits intorelease/1.1-stablefrom
user/drustheaxe/39995920-2608-bootstrap-telemetry-crash

Conversation

@DrusTheAxe
Copy link
Copy Markdown
Member

@DrusTheAxe DrusTheAxe commented Jun 14, 2022

!cherry-pick 42ba331
!cherry-pick 6198c22

1.1-servicing of v1.1 SDK throws an access violation from MddBootstrapInitialize's logging code if no matching packages are installed #2592 (#2608)

Summary

  • Telemetry char=crash. Needs be string-literal
  • Retrieve the Main package's family name for AppData for the release matching the framework (not always 1.0)
  • Add/Remove the test Main package (DataStore) just like we do the test Framework
  • Fixup test packages with naming more aligned with the actual runtime (test package Names were "blah" but product is "blah-1.2", and we now have code that parses release out of package name for further processing)

For more details see #2608

https://task.ms/39995920 for 1.1-servicing consideration

…ging code if no matching packages are installed #2592 (#2608)

* v1.1 SDK throws an access violation from MddBootstrapInitialize's logging code if no matching packages are installed #2592

* Telemetry char/string. AppData DataStore-1.0.

* Retrieve the *Main* package's family name for AppData

* Add/Remove the test Main package (DataStore) just like we do the test Framework
@DrusTheAxe DrusTheAxe added bug Something isn't working area-DynamicDependencies DynamicDependency for Windows App SDK: enables framework packages for packaged & unpackaged apps servicing-consider-1.1 labels Jun 14, 2022
@DrusTheAxe DrusTheAxe self-assigned this Jun 14, 2022
@ghost ghost added the needs-triage label Jun 14, 2022
@DrusTheAxe
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe DrusTheAxe merged commit 9bfb310 into release/1.1-stable Jun 17, 2022
@DrusTheAxe DrusTheAxe deleted the user/drustheaxe/39995920-2608-bootstrap-telemetry-crash branch June 17, 2022 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-DynamicDependencies DynamicDependency for Windows App SDK: enables framework packages for packaged & unpackaged apps bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants