Skip to content

[CAI-291] security improvements #1352

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 8 commits into from
Mar 13, 2025
Merged

[CAI-291] security improvements #1352

merged 8 commits into from
Mar 13, 2025

Conversation

batdevis
Copy link
Collaborator

@batdevis batdevis commented Feb 21, 2025 •
edited
Loading

List of Changes

  • sanitize query input string
  • jwt check

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Chore (nothing changes by a user perspective)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@batdevis batdevis requested a review from a team as a code owner February 21, 2025 14:39
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Feb 21, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 0f19d5b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
chatbot Minor

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

@batdevis batdevis requested a review from mdciri March 3, 2025 08:48
@batdevis batdevis changed the title WIP: [CAI-291] security improvements [CAI-291] security improvements Mar 3, 2025
@github-actions GitHub Actions
Copy link
Contributor

Branch is not up to date with base branch

@batdevis it seems this Pull Request is not updated with base branch.
Please proceed with a merge or rebase to solve this.

mdciri
mdciri approved these changes Mar 13, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 13, 2025 •
edited by atlassian bot
Loading

Jira Pull Request Link

This Pull Request refers to the following Jira issue CAI-291

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 13, 2025 •
edited by atlassian bot
Loading

Jira Pull Request Link

This Pull Request refers to the following Jira issue CAI-291

@batdevis batdevis merged commit 0a549b2 into main Mar 13, 2025
11 checks passed
@batdevis batdevis deleted the chatbot/sanitize-input branch March 13, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcobottaro marcobottaro marcobottaro approved these changes

@mdciri mdciri mdciri approved these changes

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

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

Assignees

@batdevis batdevis

Labels

changeset size/small

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@batdevis @marcobottaro @mdciri