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

Add debian "testing" release #430

Merged
merged 1 commit into from
Jan 13, 2021
Merged

Add debian "testing" release #430

merged 1 commit into from
Jan 13, 2021

Conversation

sktt
Copy link
Contributor

@sktt sktt commented Jan 12, 2021

Hey there, I had an issue with getting this up and running on debian/testing. Here's a debug log snippet

  MongoMS:MongoBinary MongoBinary options: {"downloadDir":"/home/jnes/TILLY/delta/jest/node_modules/.cache/mongodb-memory-server/mongodb-binaries","platform":"linux","arch":"x64","version":"4.0.14"} +0ms
  MongoMS:getos Trying LSB-Release +0ms
  MongoMS:MongoBinaryDownloadUrl Using "mongodb-linux-x86_64-debian-4.0.14.tgz" as the Archive String +0ms
  MongoMS:MongoBinaryDownloadUrl Using "https://fastdl.mongodb.org" as the mirror +0ms
  MongoMS:MongoBinaryDownload Downloading: "https://fastdl.mongodb.org/linux/mongodb-linux-x86_64-debian-4.0.14.tgz" +0ms
  MongoMS:MongoBinaryDownload trying to download https://fastdl.mongodb.org/linux/mongodb-linux-x86_64-debian-4.0.14.tgz +0ms
Error: Status Code is 403 (MongoDB's 404)

The problem here is not only the version number (using latest gives the same error) but also that it seems debian should be suffixed with its version number too.

Since Debian unstable falls back to 10, I think it should be OK for the slightly less bleeding edge Debian testing to do the same.

Related issue

There's none but one can reproduce by installing debian testing, whose lsb_release -a looks like so

No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux bullseye/sid
Release:        testing
Codename:       bullseye

@codecov
Copy link

codecov bot commented Jan 12, 2021

Codecov Report

Merging #430 (5d28f68) into master (01a6bc6) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #430   +/-   ##
=======================================
  Coverage   78.08%   78.08%           
=======================================
  Files           9        9           
  Lines         981      981           
  Branches      268      277    +9     
=======================================
  Hits          766      766           
+ Misses        214      213    -1     
- Partials        1        2    +1     
Impacted Files Coverage Δ
...ory-server-core/src/util/MongoBinaryDownloadUrl.ts 72.77% <100.00%> (ø)
...ongodb-memory-server-core/src/MongoMemoryServer.ts 92.07% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 634b462...5d28f68. Read the comment docs.

@hasezoey

This comment has been minimized.

@hasezoey hasezoey changed the title Update MongoBinaryDownloadUrl.ts Add debian "testing" release Jan 13, 2021
@hasezoey hasezoey added Distribution support Add / modify an (linux) distribution enhancement labels Jan 13, 2021
@hasezoey hasezoey merged commit e4ffecf into typegoose:master Jan 13, 2021
@github-actions
Copy link

@github-actions
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Distribution support Add / modify an (linux) distribution enhancement released on @beta released Pull Request released | Issue is fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants