Commit Graph

16 Commits

Author SHA1 Message Date
pixeebot[bot]
97c7a0543c
Hardening suggestions for Stirling-PDF / settings (#4042)
I've reviewed the recently opened PR ([4015 - V2 settings api (Added to
V1)](https://github.com/Stirling-Tools/Stirling-PDF/pull/4015)) and have
identified some area(s) that could benefit from additional hardening
measures.

These changes should help prevent potential security vulnerabilities and
improve overall code quality.

Thank you for your consideration!
🧚🤖  Powered by Pixeebot  

[Feedback](https://ask.pixee.ai/feedback) |
[Community](https://pixee-community.slack.com/signup#/domain-signup) |
[Docs](https://docs.pixee.ai/)
![](https://d1zaessa2hpsmj.cloudfront.net/pixel/v1/track?writeKey=2PI43jNm7atYvAuK7rJUz3Kcd6A&event=PR_HARDENING%7CStirling-Tools%2FStirling-PDF%7C1d1522bd7a9e9eff4cbadcf868304f8c14a130b4)

Co-authored-by: pixeebot[bot] <104101892+pixeebot[bot]@users.noreply.github.com>
2025-07-28 23:10:12 +01:00
Anthony Stirling
5e268fe77e remove logs and fix merge settings 2025-07-28 15:51:27 +01:00
Anthony Stirling
1d1522bd7a test 2025-07-28 14:33:57 +01:00
Anthony Stirling
462ec5b77f ********* 2025-07-28 13:58:41 +01:00
Anthony Stirling
76764e3179 dont show depreciated, do show premium.key 2025-07-28 13:56:22 +01:00
Anthony Stirling
13a7dbbf9e redactiontest 2025-07-28 13:39:15 +01:00
Anthony Stirling
26a6534ae4 remove file endpoint, show delta cahgned to tracking instead 2025-07-28 12:18:19 +01:00
Anthony Stirling
f9d36b985a change file to be json, added delta endpoint 2025-07-28 11:58:10 +01:00
Anthony Stirling
8132f230ef Improve settings file lock handling and add admin file endpoint 2025-07-28 11:34:52 +01:00
Anthony Stirling
b543a72ee9 reduce logs clientside 2025-07-25 15:34:08 +01:00
Anthony Stirling
fc52d14ab1 fixes 2025-07-25 15:23:56 +01:00
Anthony Stirling
b72deaebf0 fixes 2025-07-21 17:08:00 +01:00
Anthony Stirling
af4e20c971 add @Valid 2025-07-21 16:17:49 +01:00
Anthony Stirling
40d2a9015c reduce logging 2025-07-21 15:57:01 +01:00
Anthony Stirling
dbf5de3cee Merge remote-tracking branch 'origin/main' into allowExternalURLs 2025-07-21 15:12:03 +01:00
Ludy
299d52c517
refactor: move modules under app/ directory and update file paths (#3938)
# Description of Changes

- **What was changed:**  
- Renamed top-level directories: `stirling-pdf` → `app/core`, `common` →
`app/common`, `proprietary` → `app/proprietary`.
- Updated all path references in `.gitattributes`, GitHub workflows
(`.github/workflows/*`), scripts (`.github/scripts/*`), `.gitignore`,
Dockerfiles, license files, and template settings to reflect the new
structure.
- Added a new CI job `check-generateOpenApiDocs` to generate and upload
OpenAPI documentation.
- Removed redundant `@Autowired` annotations from `TempFileShutdownHook`
and `UnlockPDFFormsController`.
- Minor formatting and comment adjustments in YAML templates and
resource files.

- **Why the change was made:**  
- To introduce a clear `app/` directory hierarchy for core, common, and
proprietary modules, improving organization and maintainability.
- To ensure continuous integration and Docker builds continue to work
seamlessly with the reorganized structure.
- To automate OpenAPI documentation generation as part of the CI
pipeline.

---

## Checklist

### General

- [x] I have read the [Contribution
Guidelines](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/CONTRIBUTING.md)
- [x] I have read the [Stirling-PDF Developer
Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/DeveloperGuide.md)
(if applicable)
- [ ] I have read the [How to add new languages to
Stirling-PDF](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/HowToAddNewLanguage.md)
(if applicable)
- [x] I have performed a self-review of my own code
- [x] My changes generate no new warnings

### Documentation

- [ ] I have updated relevant docs on [Stirling-PDF's doc
repo](https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/)
(if functionality has heavily changed)
- [ ] I have read the section [Add New Translation
Tags](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/HowToAddNewLanguage.md#add-new-translation-tags)
(for new translation tags only)

### UI Changes (if applicable)

- [ ] Screenshots or videos demonstrating the UI changes are attached
(e.g., as comments or direct attachments in the PR)

### Testing (if applicable)

- [ ] I have tested my changes locally. Refer to the [Testing
Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/DeveloperGuide.md#6-testing)
for more details.
2025-07-14 20:53:11 +01:00