Skip to content
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

Small fixes #30288

Open
wezell opened this issue Oct 8, 2024 · 4 comments · Fixed by #30289 or #31257
Open

Small fixes #30288

wezell opened this issue Oct 8, 2024 · 4 comments · Fixed by #30289 or #31257
Labels
dotCMS : Security QA : Passed Internal Release : 25.01.30 Sprint: Planned Team : Falcon Type : New Functionality

Comments

@wezell
Copy link
Contributor

wezell commented Oct 8, 2024 •
edited
Loading

User Story

There are a number of small fixes that I have on my desktop that I would like to see merged but do not rise to the level of whole cards.

These include

  • hardening security on the JVMResource
  • less tomcat ClientAbortExeception logging in the CMSFilter
  • tiny change to default in DeleteInactiveLiveWorkingIndicesJob.java
  • fix the LanguageFactoryImpl.java to support - as well as _ to delimit lang-country
  • removing unneed MSSQL query in TemplateFActoryImpl
  • deleting unused jsps in dotCMS/src/main/webapp/html/portlet/ext/director
  • updating example manifests to use the new probe methods.
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(k8) update example k8 manifests to use new endpoints
d887391 
ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(jsp) delete unused jsps
1f4b84b 
ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(debt) delete unused MSSQL sql from TemplateFactoryImpl
20e6193 
ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(logging) do not log ClientAbortException in CMSFilter
febbfee 
ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(security) do not allow core config variable visibility to be cha…
9e98084 
…nged by the system_table

ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
task(lang) allow the LanguageFactory to support dashes, not just unde…
df53a41 
…rscores

ref: #30288
wezell added a commit that referenced this issue Oct 8, 2024
@wezell
test(Util) add tests for NPE for functional UtilMethods.isSet(()->)
2e72628 
ref: #30288
wezell added a commit that referenced this issue Nov 7, 2024
@wezell
fix(lang) updating the isNumeric check
4379c61 
ref: #30288
wezell added a commit that referenced this issue Nov 25, 2024
@wezell
fix(logs) reverting non-important changes so we can merge this thing …
a063cec 
…ref:#30288
@prestonso prestonso removed the Triage label Dec 4, 2024
@prestonso
Copy link

There's a pull request outstanding for this.

@prestonso prestonso added Team: Security Issues related to security and privacy dotCMS : Security labels Dec 4, 2024
@john-thomas-dotcms john-thomas-dotcms added Team : Falcon and removed Team: Security Issues related to security and privacy labels Dec 18, 2024
@john-thomas-dotcms john-thomas-dotcms moved this from New to In Review in dotCMS - Product Planning Dec 18, 2024
@dsilvam
Copy link
Contributor

dsilvam commented Jan 8, 2025

PR: #30289

@github-actions github-actions bot mentioned this issue Jan 10, 2025
Merged
@dsilvam dsilvam moved this from In Review to Internal QA in dotCMS - Product Planning Jan 22, 2025
@github-actions GitHub Actions
Copy link

PRs:

@jcastro-dotcms jcastro-dotcms linked a pull request Jan 28, 2025 that will close this issue
Issue 30288 small fixes - missing change #31257
Merged
@jcastro-dotcms jcastro-dotcms self-assigned this Jan 28, 2025
@jcastro-dotcms jcastro-dotcms moved this from Internal QA to In Review in dotCMS - Product Planning Jan 28, 2025
@github-project-automation github-project-automation bot moved this from In Review to Done in dotCMS - Product Planning Jan 28, 2025
@github-actions github-actions bot mentioned this issue Jan 28, 2025
Merged
@jcastro-dotcms jcastro-dotcms moved this from Done to Internal QA in dotCMS - Product Planning Jan 29, 2025
@github-project-automation github-project-automation bot moved this from Internal QA to Current Sprint Backlog in dotCMS - Product Planning Jan 30, 2025
@jcastro-dotcms
Copy link
Contributor

INTERNAL QA: PASSED ✅

  • Docker image: trunk_4f4d8b6

I was able to check the specific changes related to the dotCMS core itself.

@jcastro-dotcms jcastro-dotcms removed their assignment Jan 30, 2025
@jcastro-dotcms jcastro-dotcms moved this from Current Sprint Backlog to Done in dotCMS - Product Planning Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dotCMS : Security QA : Passed Internal Release : 25.01.30 Sprint: Planned Team : Falcon Type : New Functionality
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 30288 small fixes dotCMS/core
Issue 30288 small fixes - missing change dotCMS/core
5 participants
@prestonso @wezell @dsilvam @jcastro-dotcms @john-thomas-dotcms