Skip to content

fix: update SensitiveFieldWarning prop name in WsseAuth component#6492

Merged
bijin-bruno merged 1 commit intousebruno:mainfrom
abhishek-bruno:fix/bru-2390
Dec 23, 2025
Merged

fix: update SensitiveFieldWarning prop name in WsseAuth component#6492
bijin-bruno merged 1 commit intousebruno:mainfrom
abhishek-bruno:fix/bru-2390

Conversation

@abhishek-bruno
Copy link
Member

@abhishek-bruno abhishek-bruno commented Dec 23, 2025 •
edited by coderabbitai bot
Loading

Description

JIRA

fix: update SensitiveFieldWarning prop name in WsseAuth component

Contribution Checklist:

  • I've used AI significantly to create this pull request
  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

Summary by CodeRabbit

  • Bug Fixes
    • Fixed warning message display for WSSE authentication password field.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 23, 2025 •
edited
Loading

Walkthrough

A prop name in the WsseAuth component's SensitiveFieldWarning invocation has been updated from message to warningMessage for the wsse-password field, aligning with the component's expected API.

Changes

Cohort / File(s) Summary
SensitiveFieldWarning API Update
packages/bruno-app/src/components/RequestPane/Auth/WsseAuth/index.js		 Updated SensitiveFieldWarning prop from message to warningMessage for wsse-password field

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • helloanoop
  • naman-bruno
  • bijin-bruno

Poem

A whisper of change, so light and so small,
One prop name updated, that's all!
From message to warningMessage it flows,
The code marches onward—and now it just knows! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating a prop name in the WsseAuth component, which aligns with the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bijin-bruno bijin-bruno merged commit aeb6b12 into usebruno:main Dec 23, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bijin-bruno bijin-bruno bijin-bruno approved these changes

@helloanoop helloanoop Awaiting requested review from helloanoop helloanoop is a code owner

@lohit-bruno lohit-bruno Awaiting requested review from lohit-bruno lohit-bruno is a code owner

@naman-bruno naman-bruno Awaiting requested review from naman-bruno naman-bruno is a code owner

Assignees

No one assigned

Labels

size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abhishek-bruno @bijin-bruno