Skip to content

fix: add a go back button on the contents tab (#34749)#36246

Draft
gortiz-dotcms wants to merge 1 commit into
mainfrom
issue-34749-pending-bundles-tab-not-having-close-button
Draft

fix: add a go back button on the contents tab (#34749)#36246
gortiz-dotcms wants to merge 1 commit into
mainfrom
issue-34749-pending-bundles-tab-not-having-close-button

Conversation

@gortiz-dotcms

@gortiz-dotcms gortiz-dotcms commented Jun 19, 2026

Copy link
Copy Markdown
Member

Added a go back button on the contents tab

Captura de pantalla 2026-06-19 a la(s) 3 48 33 p  m

This PR fixes: #34749

@claude

claude Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Claude finished @gortiz-dotcms's task in 42s —— View job


Rollback Safety Analysis

  • Read rollback-unsafe categories reference
  • Get the full PR diff
  • Analyze diff against unsafe categories
  • Apply appropriate label

Result: ✅ Safe to Rollback

The change is limited to contentlet_actions_inc.jsp — a pure UI/JSP modification that adds a "Go Back" button when the user navigates from the publishing queue. It contains no database migrations, no Elasticsearch mapping changes, no data transformations, no API contract changes, and no serialization model changes. None of the rollback-unsafe categories (C-1 through M-4) apply.

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

🤖 Bedrock Review — deepseek.v3.2

[🟡 Medium] dotCMS/src/main/webapp/html/portlet/ext/contentlet/contentlet_actions_inc.jsp:23 — Direct use of referer request parameter without validation or sanitization for inclusion in an HTML attribute (href="<%= referer %>"). This could lead to cross-site scripting (XSS) if the parameter contains malicious data. Should be validated or escaped.

[🟡 Medium] dotCMS/src/main/webapp/html/portlet/ext/contentlet/contentlet_actions_inc.jsp:23 — The condition referer.contains("publishing-queue") is a string match that could be bypassed or matched incorrectly (e.g., a crafted referer URL). This is a weak security control if the referer is being trusted for navigation. Consider a more robust flag or parameter.


Run: #27843117162 · tokens: in: 781 · out: 177 · total: 958

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Safe To Rollback Area : Backend PR changes Java/Maven backend code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Publishing queue: clicking pending bundle redirects to Users/first tab instead of bundle details

1 participant