Skip to content

fix: server app should initialize even if FinalizationRegistry is unsupported #8320

New issue

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

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

Already on GitHub? # to your account

Conversation

JoseVSeb
Copy link
Contributor

FinalizationRegistry is required only if releaseOnDeref is provided in config; guard clause in initializeServerApp throws error if it's unsupported.

#8299

…upported

FinalizationRegistry is required only if releaseOnDeref is provided in config; guard clause in initializeServerApp throws error if it's unsupported.

firebase#8299
@JoseVSeb JoseVSeb requested review from a team as code owners June 16, 2024 08:33
Copy link

changeset-bot bot commented Jun 16, 2024

🦋 Changeset detected

Latest commit: d4236c5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/app Patch
@firebase/app-compat Patch
firebase Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@DellaBitta
Copy link
Contributor

Hi @JoseVSeb,

We ended up fixing this slightly differently in #8335 . Thanks for the submission, and it should be fixed in the next release!

@DellaBitta DellaBitta closed this Jun 28, 2024
@JoseVSeb JoseVSeb deleted the bugfix/initialize-server-app-even-if-finalization-registry-unsupported branch June 28, 2024 20:35
@firebase firebase locked and limited conversation to collaborators Jul 29, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants