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.

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

Deletion of Unpublished Bundles through API #26201

Closed
Neehakethi opened this issue Sep 20, 2023 · 1 comment · Fixed by #27681
Closed

Deletion of Unpublished Bundles through API #26201

Neehakethi opened this issue Sep 20, 2023 · 1 comment · Fixed by #27681
Assignees
Labels

Comments

@Neehakethi
Copy link
Contributor

Parent Issue

No response

Problem Statement

While trying to delete an unpublished bundle via API, deletion of bundles is failing due to Invalid User error

[20/09/23 17:26:16:750 UTC] WARN rest.WebResource: InitDataObject Error: uri:/api/bundle/ids err:Invalid User [20/09/23 17:26:16:750 UTC] WARN rest.WebResource: Invalid User [20/09/23 17:26:16:750 UTC] WARN rest.WebResource: com.dotcms.rest.WebResource.authenticate(WebResource.java:516) [20/09/23 17:26:25:360 UTC] WARN auth.PrincipalThreadLocal: getName null [20/09/23 17:26:25:360 UTC] INFO util.SecurityLogger: class com.dotcms.cms.login.LoginServiceAPIFactory$LoginServiceImpl : User dotcms.org.1 has successfully login from IP: 172.31.0.1 -- ip:172.31.0.1,user:Admin User [ID: dotcms.org.1][email:[email protected]] [20/09/23 17:26:25:362 UTC] INFO rest.BundleResource: Deleting the bundles: [01HASSW7TN9HPT41S9KSV714SC] by the user: dotcms.org.1 [20/09/23 17:26:25:363 UTC] ERROR business.BundleAPIImpl: The bundle id: 01HASSW7TN9HPT41S9KSV714SC does not exists [20/09/23 17:26:25:366 UTC] ERROR rest.BundleResource: Bundles Deleted Successfully, failed 1 [20/09/23 17:26:25:366 UTC] ERROR rest.BundleResource: Bundles Failed [01HASSW7TN9HPT41S9KSV714SC]

Steps to Reproduce

  1. Login to the demo site
  2. Add some content to a bundle
  3. Try to delete the bundle using {{baseUrl}}/api/bundle/ids
  4. It gives a 200 response, but the bundles are not deleted.
  5. Check the logs (Thr
    Screenshot 2023-09-20 at 1 27 53 PM
    ows an error)
    Screenshot 2023-09-20 at 3 22 19 PM

Acceptance Criteria

We are right now calling the API directly.
Adding a logic to check users permissions, and call that API should solve the issue

dotCMS Version

22.03.9

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

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

https://dotcms.slack.com/archives/CSHTYUR7H/p1695232109476449
https://dotcms.zendesk.com/agent/tickets/113344

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@erickgonzalez erickgonzalez moved this from New to Next 1-3 Sprints in dotCMS - Product Planning Sep 20, 2023
@erickgonzalez erickgonzalez added the OKR : Customer Support Owned by Scott label Sep 20, 2023
@erickgonzalez erickgonzalez moved this from Next 1-3 Sprints to Current Sprint Backlog in dotCMS - Product Planning Oct 12, 2023
@erickgonzalez erickgonzalez moved this from Current Sprint Backlog to Next 1-3 Sprints in dotCMS - Product Planning Oct 12, 2023
@swicken-dotcms swicken-dotcms removed the OKR : Customer Success Owned by Arno label Dec 19, 2023
@erickgonzalez erickgonzalez moved this from Next 1-3 Sprints to Current Sprint Backlog in dotCMS - Product Planning Jan 8, 2024
@dsolistorres dsolistorres moved this from Current Sprint Backlog to In Progress in dotCMS - Product Planning Feb 9, 2024
@dsolistorres dsolistorres linked a pull request Feb 21, 2024 that will close this issue
1 task
@dsolistorres dsolistorres moved this from In Progress to In Review in dotCMS - Product Planning Feb 21, 2024
@erickgonzalez erickgonzalez moved this from In Review to Internal QA in dotCMS - Product Planning Feb 28, 2024
github-merge-queue bot pushed a commit that referenced this issue Feb 28, 2024
@github-project-automation github-project-automation bot moved this from Internal QA to Done in dotCMS - Product Planning Feb 28, 2024
@erickgonzalez erickgonzalez moved this from Done to Internal QA in dotCMS - Product Planning Feb 28, 2024
@erickgonzalez erickgonzalez added the LTS : Next Ticket that will be added to LTS label Feb 29, 2024
@bryanboza
Copy link
Member

We can pass this for now, but we need to continue with some improvements. New card related here #27782

@bryanboza bryanboza moved this from Internal QA to Done in dotCMS - Product Planning Mar 1, 2024
@erickgonzalez erickgonzalez added Next LTS Release Shortlisted of issues that will be included in the upcoming LTS and removed LTS : Next Ticket that will be added to LTS labels Mar 6, 2024
erickgonzalez added a commit that referenced this issue Mar 15, 2024
erickgonzalez added a commit that referenced this issue Mar 22, 2024
@erickgonzalez erickgonzalez added Release : 23.10.24 v7 Included in LTS patch release 23.10.24 v7 Release : 23.01.14 Included in LTS patch release 23.01.14 and removed Next LTS Release Shortlisted of issues that will be included in the upcoming LTS labels Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants