github-starred/cobalt
2
0
Fork 0
mirror of https://github.com/imputnet/cobalt.git synced 2026-11-11 16:32:57 -06:00
Code Issues 5.8k Packages Projects Releases Wiki Activity

package.json links a git:// dependency that can't be used without authentication #380

New Issue
Closed
opened 2025-11-09 09:47:56 -06:00 by GiteaMirror · 3 comments
GiteaMirror commented 2025-11-09 09:47:56 -06:00
Owner
Copy Link
Copy Source

Originally created by @sertraline on GitHub (Aug 1, 2024).

bug description

This commit https://github.com/imputnet/cobalt/commit/6626d74746b88420ec80e6b707a8d7cd01a4f5d9 has linked youtubei.js to a commit, but the URL starts with git://.

The issue is that unauthenticated git users will be unable to run npm install or yarn install, as Github does not allow unauthenticated users to use the 9418 port.

The solution is to replace git:// with https://.

screenshots

image

Moreover, this commit fails to build. Relevant bug report: https://github.com/LuanRT/YouTube.js/issues/716

image

Originally created by @sertraline on GitHub (Aug 1, 2024). ### bug description This commit https://github.com/imputnet/cobalt/commit/6626d74746b88420ec80e6b707a8d7cd01a4f5d9 has linked `youtubei.js` to a commit, but the URL starts with git://. The issue is that unauthenticated git users will be unable to run `npm install` or `yarn install`, as [Github does not allow unauthenticated users to use the 9418 port](https://github.blog/security/application-security/improving-git-protocol-security-github/). The solution is to replace `git://` with `https://`. ### screenshots ![image](https://github.com/user-attachments/assets/9cd76997-fd27-4988-835b-025e29661be4) Moreover, this commit fails to build. Relevant bug report: https://github.com/LuanRT/YouTube.js/issues/716 ![image](https://github.com/user-attachments/assets/4680d5ed-b087-4c5a-8766-2ae24b5c9e6b)
GiteaMirror added the bug label 2025-11-09 09:47:56 -06:00
GiteaMirror commented 2025-11-09 09:47:58 -06:00
Author
Owner
Copy Link
Copy Source

@wukko commented on GitHub (Aug 1, 2024):

i've updated the dependency to npm package version that includes the needed commit in 50f312f, so the issue should be resolved now

@wukko commented on GitHub (Aug 1, 2024): i've updated the dependency to npm package version that includes the needed commit in [50f312f](https://github.com/imputnet/cobalt/commit/50f312fc71a55e3bfbb32d39fe1f32cf938526a0), so the issue should be resolved now
GiteaMirror commented 2025-11-09 09:47:58 -06:00
Author
Owner
Copy Link
Copy Source

@wukko commented on GitHub (Aug 1, 2024):

lmk if it's actually fixed

@wukko commented on GitHub (Aug 1, 2024): lmk if it's actually fixed
GiteaMirror commented 2025-11-09 09:47:58 -06:00
Author
Owner
Copy Link
Copy Source

@sertraline commented on GitHub (Aug 1, 2024):

it is fixed. Thanks

@sertraline commented on GitHub (Aug 1, 2024): it is fixed. Thanks
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
3rd party app support
bug
can reproduce
can't reproduce
documentation
duplicate
feature request
help wanted
instance hosting help
invalid
main instance issue
need more info
needs fixing
next major
not an issue
pull-request
Mirrored from GitHub Pull Request
 question
region lock issue
security
service request
skill issue
web ui
No labels bug
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
GiteaMirror ninjasurge (Kaehlen Rog)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cobalt#380