Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

fix: correctly propagate the firebase app name #1122

Merged
merged 1 commit into from
Dec 15, 2022

Conversation

mctavish
Copy link
Contributor

The firebase app name is not being correctly set and in at least some cases will prevent the debug agent from connecting to the firebase database.

@mctavish mctavish requested review from a team as code owners December 15, 2022 14:54
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. labels Dec 15, 2022
@codecov
Copy link

codecov bot commented Dec 15, 2022

Codecov Report

Base: 65.88% // Head: 65.54% // Decreases project coverage by -0.33% ⚠️

Coverage data is based on head (bbf1c1a) compared to base (cb3ce24).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1122      +/-   ##
==========================================
- Coverage   65.88%   65.54%   -0.34%     
==========================================
  Files          21       21              
  Lines        1791     1791              
  Branches      355      355              
==========================================
- Hits         1180     1174       -6     
- Misses        524      526       +2     
- Partials       87       91       +4     
Impacted Files Coverage Δ
src/agent/firebase-controller.ts 69.02% <0.00%> (ø)
src/agent/state/inspector-state.ts 87.50% <0.00%> (-1.86%) ⬇️
src/agent/util/utils.ts 95.06% <0.00%> (-1.24%) ⬇️
src/agent/v8/inspector-debugapi.ts 84.16% <0.00%> (-0.46%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mctavish mctavish added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 15, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 15, 2022
@mctavish mctavish added the automerge Merge the pull request once unit tests and other checks pass. label Dec 15, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit 443e902 into googleapis:main Dec 15, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants