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

fix(app/metrics): private repositories may be always ignored #1072

Open
lowlighter opened this issue May 24, 2022 · 4 comments
Open

fix(app/metrics): private repositories may be always ignored #1072

lowlighter opened this issue May 24, 2022 · 4 comments
Labels

Comments

@lowlighter
Copy link
Owner

A lot of users seems to report that their private repositories are not taken into account, even with a repo scope

@github-actions github-actions bot added the Stale label Dec 9, 2022
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 17, 2022
@lowlighter lowlighter removed the Stale label Dec 18, 2022
@lowlighter lowlighter reopened this Dec 18, 2022
@nico-zck
Copy link

I've noticed that the "Most Used Languages" stats for my metrics are weird, while the basic stats look good.It only analyzed 86 commits from thousand commits.
image

When plugin_languages_indepth is enabled, private repositories cannot be cloned even if a token is provided and access is enabled.
The action log of debug: yes as follows:

metrics/compute/XXX/plugins > languages > indepth analyzer > failed to clone https://github.com/XXX/REPO(Error: Cloning into '.'...
  fatal: could not read Username for 'https://github.com/': No such device or address
  )

I'm not sure the relationship between them, but I just want to provide some cues for this bug.
Thanks for your awesome work!

@mia1024
Copy link

mia1024 commented Nov 14, 2023

Hi, I'm trying to set up actions for my own profile and I also ran into the same issues. I took a quick look at the code and the issues seems to be that no token is passed when cloning a repo, causing clones to all private repos to fail:

https://github.com/lowlighter/metrics/blob/master/source/plugins/languages/analyzer/analyzer.mjs#L94

According to this Stackoverflow post, all you need to do is change the URL to include the token.

@ilanRosenbaum
Copy link
Contributor

ilanRosenbaum commented Dec 5, 2023

Encountering the same issue myself

@ilanRosenbaum
Copy link
Contributor

Just created a PR to fix this: #1579

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants