fix: imported fixes 03-09-2026#39512
Conversation
🦋 Changeset detectedLatest commit: 36e1e0b The changes in this PR will be included in the next version bump. This PR includes changesets to release 41 packages
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 |
Co-authored-by: Ricardo Garim <[email protected]>
1a17ceb to
4040c10
Compare
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
There was a problem hiding this comment.
1 issue found across 2 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts">
<violation number="1" location="apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts:78">
P2: `access_token` is only removed when truthy; falsy query values (e.g. empty string) remain in `queryParams`. Delete it unconditionally after extraction.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts
Outdated
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release-7.13.5 #39512 +/- ##
=================================================
Coverage ? 67.80%
=================================================
Files ? 3445
Lines ? 113917
Branches ? 20837
=================================================
Hits ? 77240
Misses ? 34561
Partials ? 2116
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
/backport 7.12.6 |
Co-authored-by: Yasmim Nagat <[email protected]> Co-authored-by: Ricardo Garim <[email protected]> Co-authored-by: Julio Araujo <[email protected]>
|
Pull request #39549 added to Project: "Patch 7.12.6" |
|
/backport 7.11.6 |
Co-authored-by: Yasmim Nagat <[email protected]> Co-authored-by: Ricardo Garim <[email protected]> Co-authored-by: Julio Araujo <[email protected]>
|
Pull request #39550 added to Project: "Patch 7.11.6" |
|
/backport 7.10.9 |
|
Sorry, I couldn't do that backport because of conflicts. Could you please solve them? you can do so by running the following commands: after that just run |
Co-authored-by: Yasmim Nagat <[email protected]> Co-authored-by: Ricardo Garim <[email protected]> Co-authored-by: Julio Araujo <[email protected]>
|
/backport 7.10.9 |
|
Pull request #39552 added to Project: "Patch 7.10.9" |
Co-authored-by: Yasmim Nagat <[email protected]> Co-authored-by: Ricardo Garim <[email protected]> Co-authored-by: Julio Araujo <[email protected]>
Co-authored-by: Yasmim Nagat <[email protected]> Co-authored-by: Ricardo Garim <[email protected]> Co-authored-by: Julio Araujo <[email protected]>
Backport of #39492