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

Use of Png filter on images results in a 404 #26374

Closed
swicken-dotcms opened this issue Oct 5, 2023 · 3 comments · Fixed by #26381
Closed

Use of Png filter on images results in a 404 #26374

swicken-dotcms opened this issue Oct 5, 2023 · 3 comments · Fixed by #26381
Assignees
@erickgonzalez
Labels
OKR : Customer Support Owned by Scott QA : Approved Release : 22.03.11 Included in LTS patch release 22.03.11 Release : 23.01.8 Included in LTS patch release 23.01.8 Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 Release : 23.12.21 SEO Improvements Team : Bug Fixers Type : Defect

Comments

@swicken-dotcms
Copy link
Contributor

Parent Issue

No response

Problem Statement

When using the Png filter it results in a 404 when trying to access the image

https://demo.dotcms.com/contentAsset/image/5005364a8ce21d15ee81797e66e53bf1/fileAsset/filter/Png

Steps to Reproduce

  1. Start up a demo starter
  2. Attempt to access an image with the Png filter, such as /contentAsset/image/5005364a8ce21d15ee81797e66e53bf1/fileAsset/filter/Png

Acceptance Criteria

Use of the Png filter should correctly generate a Png and not a 404

dotCMS Version

23.01.7/master

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@erickgonzalez erickgonzalez self-assigned this Oct 5, 2023
@yolabingo
Copy link
Contributor

ref Cloud support ticket https://dotcms.zendesk.com/agent/tickets/113624

@erickgonzalez erickgonzalez linked a pull request Oct 5, 2023 that will close this issue
fix: remove redundant close ref#26374 #26381
Merged
@erickgonzalez erickgonzalez reopened this Oct 27, 2023
erickgonzalez added a commit that referenced this issue Oct 31, 2023
@erickgonzalez
#26374 include in 23.01.8
413588b
@erickgonzalez erickgonzalez added Release : 23.01.8 Included in LTS patch release 23.01.8 Release : 22.03.11 Included in LTS patch release 22.03.11 labels Nov 7, 2023
erickgonzalez added a commit that referenced this issue Nov 9, 2023
@erickgonzalez
#26374 include in 22.03.11
06cf8c4
@erickgonzalez erickgonzalez added LTS : Next Ticket that will be added to LTS and removed Next LTS Release labels Nov 9, 2023
@erickgonzalez
Copy link
Contributor

Working as it should

Image

@bryanboza
Copy link
Member

Fixed, tested on master // Docker // FF

Now it is working as expected

Image

erickgonzalez added a commit that referenced this issue Dec 5, 2023
@erickgonzalez
#26374 include in 23.10.24
2455fb1
@erickgonzalez erickgonzalez added Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 and removed LTS : Next Ticket that will be added to LTS labels Dec 5, 2023
@erickgonzalez erickgonzalez added LTS: Excluded Ticket that has been excluded from at least one LTS and removed LTS: Excluded Ticket that has been excluded from at least one LTS labels Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erickgonzalez erickgonzalez

Labels
OKR : Customer Support Owned by Scott QA : Approved Release : 22.03.11 Included in LTS patch release 22.03.11 Release : 23.01.8 Included in LTS patch release 23.01.8 Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 Release : 23.12.21 SEO Improvements Team : Bug Fixers Type : Defect
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: remove redundant close ref#26374 dotCMS/core
6 participants
@yolabingo @bryanboza @erickgonzalez @victoralfaro-dotcms @swicken-dotcms @damen-dotcms