Skip to content

Conversation

@SalehBorhani
Copy link
Contributor

@SalehBorhani SalehBorhani commented Jul 23, 2025

📝 New contributors

⚠︎ If the checkbox will not be confirmed - we can't include your contribution in our codebase.

📖 Description

  1. include a summary of the changes and the related issue, eg. Closes #XYZ
  2. Do not make a PR if you can't check all the boxes below

🛠️ Dev Branch Merge Checklist:

Documentation

Testing

  • I have prepared end-to-end tests for all new functionalities
  • I have performed end-to-end tests manually and they work
  • New and existing unit tests pass locally with my changes

Deployment

  • If deployment is affected I have made corresponding/required changes to deployment (Docker, Kubernetes, one-line install)

🏚️ Main Branch Merge Checklist:

Testing

  • I have merged my changes before to dev and the dev checklist is done
  • I have tested all functionalities on the dev instance and they work

Documentation

  • I have made corresponding changes to the user & admin documentation and added new features documentation with screenshots for users/admins

@wojcik91 wojcik91 merged commit 2926791 into DefGuard:main Aug 14, 2025
wojcik91 added a commit that referenced this pull request Sep 10, 2025
* Merge commit from fork

* add redirect URL sanitization

* simplify validation

* bump outdated dependency

* update yanked dependency

* bump version to 1.4.1 (#1419)

* docs: update enterprise features link (#1337)

* Update SECURITY.md

* update dependencies

* update supported versions

---------

Co-authored-by: Saleh Borhani <[email protected]>
Co-authored-by: Robert Olejnik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants