Skip to content

Fix dependabot security alerts#4123

Merged
yifanyang merged 1 commit intomasterfrom
yifany/security-alerts
Sep 21, 2022
Merged

Fix dependabot security alerts#4123
yifanyang merged 1 commit intomasterfrom
yifany/security-alerts

Conversation

@yifanyang
Copy link
Copy Markdown
Contributor

@yifanyang yifanyang commented Sep 21, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 21, 2022 •
edited
Loading

Unit Test Results

   394 files  +  4     394 suites  +4   18m 19s ⏱️ + 1m 24s
4 713 tests +10  4 691 ✔️ +10  22 💤 ±0  0 ❌ ±0 
4 729 runs  +10  4 707 ✔️ +10  22 💤 ±0  0 ❌ ±0 

Results for commit 3f49238. ± Comparison against base commit 523730a.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Copy Markdown
Collaborator

google-oss-bot commented Sep 21, 2022

Size Report 1

Affected Products

No changes between base commit (523730a) and merge commit (0369a0a).

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/G2ksDSqOcq.html

@google-oss-bot
Copy link
Copy Markdown
Collaborator

google-oss-bot commented Sep 21, 2022

Coverage Report 1

Affected Products

  • firebase-database

    Overall coverage changed from 50.18% (523730a) to 50.22% (0369a0a) by +0.04%.

    FilenameBase (523730a)Merge (0369a0a)Diff
    ChildChangeAccumulator.java83.33%96.67%+13.33%
    DefaultPersistenceManager.java74.76%75.73%+0.97%
    QueryParams.java90.36%89.76%-0.60%

  • firebase-firestore

    Overall coverage changed from 44.25% (523730a) to 44.26% (0369a0a) by +0.00%.

    FilenameBase (523730a)Merge (0369a0a)Diff
    SetMutation.java94.44%97.22%+2.78%

  • firebase-messaging

    Overall coverage changed from 85.14% (523730a) to 85.26% (0369a0a) by +0.12%.

    FilenameBase (523730a)Merge (0369a0a)Diff
    FirebaseMessaging.java75.77%76.21%+0.44%
    RequestDeduplicator.java80.00%90.00%+10.00%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/ZhGLAL8UJr.html

@yifanyang yifanyang requested a review from vkryachko September 21, 2022 20:00
@yifanyang yifanyang merged commit 6caf590 into master Sep 21, 2022
@yifanyang yifanyang deleted the yifany/security-alerts branch September 21, 2022 20:10
@firebase firebase locked and limited conversation to collaborators Oct 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@vkryachko vkryachko vkryachko approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

size/XL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yifanyang @google-oss-bot @vkryachko