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

CLI: Move dotcms-cli.log to .dotcms folder #27809

Closed
nollymar opened this issue Mar 2, 2024 · 2 comments · Fixed by #28353
Closed

CLI: Move dotcms-cli.log to .dotcms folder #27809

nollymar opened this issue Mar 2, 2024 · 2 comments · Fixed by #28353
Labels
Doc : Needs Doc dotCMS : CLI LTS: Excluded Ticket that has been excluded from at least one LTS LTS : Next Ticket that will be added to LTS Merged QA : Approved QA : Passed Internal Release : 24.05.13 Bug Fixing Team : Scout Type : Task

Comments

@nollymar
Copy link
Contributor

nollymar commented Mar 2, 2024

Parent Issue

No response

Task

Currently, the dotcms-cli.log is stored in the cli workspace by default. This default path should be changed and the log should be moved to the ~/.dotcms folder

Proposed Objective

Code Maintenance

Proposed Priority

Priority 3 - Average

Acceptance Criteria

No response

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
@jgambarios jgambarios self-assigned this Apr 24, 2024
jgambarios added a commit that referenced this issue Apr 24, 2024
@jgambarios
#27809 Changing dotcms-cli.log default location
da474c4
@jgambarios jgambarios linked a pull request Apr 24, 2024 that will close this issue
#27809 Changing dotcms-cli.log default location #28353
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 25, 2024
@jgambarios
#27809 Changing dotcms-cli.log default location (#28353)
06347c0
@jgambarios jgambarios removed their assignment Apr 25, 2024
@nollymar nollymar reopened this Apr 25, 2024
@fabrizzio-dotCMS fabrizzio-dotCMS self-assigned this Apr 25, 2024
@fabrizzio-dotCMS
Copy link
Contributor

fabrizzio-dotCMS commented Apr 26, 2024 •
edited

I verified that the login is now written to ${user.home}/.dotcms/dotcms-cli.log
it can also get overridden via environment variables
like this
export QUARKUS_LOG_FILE_PATH=/Users/fabrizzioaraya/Coding/dotcms-cli.log

@fabrizzio-dotCMS fabrizzio-dotCMS removed their assignment Apr 26, 2024
@nollymar nollymar added the LTS : Next Ticket that will be added to LTS label May 1, 2024
@erickgonzalez erickgonzalez added the LTS: Excluded Ticket that has been excluded from at least one LTS label May 3, 2024
@bryanboza
Copy link
Member

Fixed, tested in the last trunk version and now we don't have the log in the CLI files anymore, now it is on the .dotCMS hidden folder

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Doc : Needs Doc dotCMS : CLI LTS: Excluded Ticket that has been excluded from at least one LTS LTS : Next Ticket that will be added to LTS Merged QA : Approved QA : Passed Internal Release : 24.05.13 Bug Fixing Team : Scout Type : Task
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#27809 Changing dotcms-cli.log default location dotCMS/core
5 participants
@jgambarios @bryanboza @erickgonzalez @nollymar @fabrizzio-dotCMS