Skip to content

fix: privacy module requests #841

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Feb 27, 2025
Merged

fix: privacy module requests #841

merged 6 commits into from
Feb 27, 2025

Conversation

jkazimierczak
Copy link
Contributor

@jkazimierczak jkazimierczak commented Feb 27, 2025 •
edited
Loading

Purpose

  • Logs error responses when client requests receives 4xx response,
  • Adds config and logic in PrivacyModule to warn user when he posts comment with sensitive data,
  • Fixes mapper to not stringify null,
  • Fixes restrictCommentToGroup to not include mutually exclusive options.

image

Approach

Future work

Merge the PR in helper-messages first, so new message panel is available.

Checklist

  • Included tests
  • Updated documentation in README
    and docs folder
  • Tested in MCTEST-xxx

urielsalis
urielsalis previously approved these changes Feb 27, 2025
View reviewed changes
@urielsalis urielsalis merged commit c599669 into mojira:master Feb 27, 2025
3 checks passed
@jkazimierczak jkazimierczak deleted the fix/privacy-module-requests branch February 27, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@urielsalis urielsalis urielsalis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkazimierczak @urielsalis