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

file-access-violation-error #21795

Closed
juanpin-humane opened this issue Apr 24, 2023 · 7 comments · Fixed by #21871
Closed

file-access-violation-error #21795

juanpin-humane opened this issue Apr 24, 2023 · 7 comments · Fixed by #21871
Assignees
Labels
priority-2-high Bugs impacting wide number of users or very important features status:in-progress Someone is working on implementation type:bug Bug fix of existing functionality

Comments

@juanpin-humane
Copy link

How are you running Renovate?

Mend Renovate hosted app on github.com

If you're self-hosting Renovate, tell us what version of Renovate you run.

No response

If you're self-hosting Renovate, select which platform you are using.

None

Was this something which used to work for you, and then stopped?

It used to work, and then stopped

Describe the bug

It was working fine. Now it's throwing an error, but the debug message is not helpful.

Seems that there has been a lot of activity reported around this code in

#21476
#21625

I wonder if some other regression got introduced.

Relevant debug logs

Logs
DEBUG: Matched 3 file(s) for manager html: apps/keycloak-theme/public/index.html, packages/ui/.storybook/preview-head.html, packages/ui/index.html
DEBUG: Matched 7 file(s) for manager npm: apps/center-web/package.json, apps/keycloak-theme/package.json, apps/website/package.json, package.json, packages/api/package.json, packages/auth/package.json, packages/ui/package.json
DEBUG: Matched 2 file(s) for manager nvm: .nvmrc, apps/website/.nvmrc
DEBUG: CircleCI docker image
{
  "depName": "cimg/node",
  "currentValue": "16.20"
}
DEBUG: CircleCI docker image
{
  "depName": "cimg/openjdk",
  "currentValue": "11.0"
}
DEBUG: CircleCI docker image
{
  "depName": "cimg/openjdk",
  "currentValue": "11.0"
}
DEBUG: npm file apps/center-web/package.json has name "@humane/center-web"
DEBUG: Preventing access to path with illegal characters
{
  "path": ".github/workflows/main->production.yml"
}
ERROR: Repository has unknown error
{
  "err": {
    "message": "file-access-violation-error",
    "stack": "Error: file-access-violation-error\n    at ensurePath (/home/ubuntu/renovateapp/node_modules/renovate/dist/util/fs/util.js:21:15)\n    at ensureLocalPath (/home/ubuntu/renovateapp/node_modules/renovate/dist/util/fs/util.js:29:12)\n    at readLocalFile (/home/ubuntu/renovateapp/node_modules/renovate/dist/util/fs/index.js:25:54)\n    at getManagerPackageFiles (/home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/extract/manager-files.js:30:54)\n    at async /home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/extract/index.js:51:30\n    at async Promise.all (index 2)\n    at async extractAllDependencies (/home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/extract/index.js:49:28)\n    at async extract (/home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/process/extract-update.js:106:32)\n    at async extractDependencies (/home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/process/index.js:107:30)\n    at async Object.renovateRepository (/home/ubuntu/renovateapp/node_modules/renovate/dist/workers/repository/index.js:52:15)\n    at async renovateRepository (/home/ubuntu/renovateapp/app/worker/index.js:260:26)\n    at async /home/ubuntu/renovateapp/app/worker/index.js:454:5"
  }
}
DEBUG: Repository result: unknown-error, status: onboarded, enabled: true, onboarded: true
DEBUG: npm file apps/keycloak-theme/package.json has name "@humane/keycloak-theme"
DEBUG: No content found in apps/website/package.json

Have you created a minimal reproduction repository?

I have linked to a minimal reproduction in the description above

@juanpin-humane juanpin-humane added priority-5-triage status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality labels Apr 24, 2023
@viceice viceice added priority-2-high Bugs impacting wide number of users or very important features status:in-progress Someone is working on implementation reproduction:unnecessary and removed status:requirements Full requirements are not yet known, so implementation should not be started priority-5-triage labels Apr 25, 2023
@viceice viceice self-assigned this Apr 25, 2023
@viceice
Copy link
Member

viceice commented Apr 25, 2023

😕

@viceice
Copy link
Member

viceice commented Apr 25, 2023

Isn't > a real invalid file char? 🤔

@juanpin-humane
Copy link
Author

@viceice I will confirm if > is the issue.

@juanpin-humane
Copy link
Author

Confirmed, I renamed the files and all is working again.

@juanpin-humane
Copy link
Author

it's invalid in windows (fat32, ntfs) , linux file systems , git and Mac support it.

@viceice
Copy link
Member

viceice commented Apr 27, 2023

I'll remove the regex check tomorrow and let the filesystem handle any issues.

@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 35.65.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority-2-high Bugs impacting wide number of users or very important features status:in-progress Someone is working on implementation type:bug Bug fix of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants