Skip to content

Commit cb365f6

Browse files
authored
Merge branch 'main' into gagik/fix-vscode-test
2 parents 95585cb + dce0b68 commit cb365f6

File tree

25 files changed

+155
-334
lines changed

25 files changed

+155
-334
lines changed

.evergreen.yml

Lines changed: 0 additions & 238 deletions
Large diffs are not rendered by default.

.evergreen/compile-artifact.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ if [ `uname` = Darwin ]; then
2626
# match what Node.js 20 does on their own builder machines
2727
export CFLAGS='-mmacosx-version-min=10.15'
2828
export CXXFLAGS='-mmacosx-version-min=10.15'
29+
export MACOSX_DEPLOYMENT_TARGET=10.15
2930
fi
3031

3132
# The CI machines we have for Windows and x64 macOS are not

.evergreen/evergreen.yml.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1547,7 +1547,7 @@ buildvariants:
15471547
node_js_version: "<% out(variant.nVersion) %>"
15481548
mongosh_skip_node_version_check: "<% out(variant.skipNodeVersionCheck) %>"
15491549
tasks:
1550-
<% for (const test of ALL_UNIT_TESTS) { %>
1550+
<% for (const test of ALL_UNIT_TESTS.filter(test => test.variants.includes(variant.platform))) { %>
15511551
- name: test_<% out(test.id) %>
15521552
<% } %>
15531553
<% } %>

.github/workflows/cron-tasks.yml

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,24 +13,24 @@ jobs:
1313
name: Update automatically generated files
1414
runs-on: ubuntu-latest
1515
steps:
16-
- uses: actions/checkout@v3
16+
- uses: actions/checkout@v4
1717
with:
1818
# don't checkout a detatched HEAD
1919
ref: ${{ github.head_ref }}
2020

2121
# this is important so git log can pick up on
2222
# the whole history to generate the list of AUTHORS
23-
fetch-depth: '0'
23+
fetch-depth: "0"
2424

2525
- name: Set up Git
2626
run: |
2727
git config --local user.email "41898282+github-actions[bot]@users.noreply.github.com"
2828
git config --local user.name "github-actions[bot]"
2929
30-
- uses: actions/setup-node@v2
30+
- uses: actions/setup-node@v4
3131
with:
3232
node-version: ^16.x
33-
cache: 'npm'
33+
cache: "npm"
3434

3535
- name: Install npm@8
3636
run: |
@@ -50,41 +50,36 @@ jobs:
5050
run: |
5151
npm run update-authors
5252
git add AUTHORS \*/AUTHORS
53-
git commit --no-allow-empty -m "chore: update AUTHORS" || true
5453
5554
- name: Generate Error Documentation
5655
run: |
5756
npm run generate-error-overview
5857
mv error-overview.md error-overview.rst packages/errors/generated/
5958
npm run reformat
6059
git add packages/errors/generated
61-
git commit --no-allow-empty -m "chore: update error documentation" || true
6260
6361
- name: Regenerate Evergreen Config
6462
run: |
6563
npm run update-evergreen-config
6664
git add .evergreen.yml
67-
git commit --no-allow-empty -m "chore: update evergreen config" || true
6865
6966
- name: Update Security Test Summary
7067
run: |
7168
npm run update-security-test-summary
7269
git add docs/security-test-summary.md
73-
git commit --no-allow-empty -m "chore: update security test summary" || true
7470
7571
- name: Regenerate CLI usage text in README files
7672
run: |
7773
npm run update-cli-usage-text packages/*/*.md *.md
7874
git add packages/*/*.md *.md
79-
git commit --no-allow-empty -m "chore: update CLI usage text" || true
8075
8176
- name: Create pull request
8277
id: cpr
8378
uses: peter-evans/create-pull-request@v6
8479
with:
8580
commit-message: Update auto-generated files
8681
branch: ci/cron-tasks-update-files
87-
title: 'chore: update auto-generated files'
82+
title: "chore: update auto-generated files"
8883
body: |
8984
- Update auto-generated files
9085

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ variable. For detailed instructions for each of our supported platforms, please
9090
--oidcTrustedEndpoint Treat the cluster/database mongosh as a trusted endpoint
9191
--oidcIdTokenAsAccessToken Use ID tokens in place of access tokens for auth
9292
--oidcDumpTokens[=mode] Debug OIDC by printing tokens to mongosh's output [full|include-secrets]
93+
--oidcNoNonce Don't send a nonce argument in the OIDC auth request
9394

9495
DB Address Examples:
9596

THIRD_PARTY_NOTICES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
The following third-party software is used by and included in **mongosh**.
2-
This document was automatically generated on Sun Nov 17 2024.
2+
This document was automatically generated on Thu Nov 21 2024.
33

44
## List of dependencies
55

package-lock.json

Lines changed: 86 additions & 64 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/arg-parser/AUTHORS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@ Basit <[email protected]>
99
Alena Khineika <[email protected]>
1010
Sergey Petushkov <[email protected]>
1111
Gagik Amaryan <[email protected]>
12+
Nikola Irinchev <[email protected]>

packages/arg-parser/src/arg-mapper.spec.ts

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -450,6 +450,22 @@ describe('arg-mapper.mapCliToDriver', function () {
450450
});
451451
});
452452

453+
context('when cli args have oidcNoNonce', function () {
454+
const cliOptions: CliOptions = {
455+
oidcNoNonce: true,
456+
};
457+
458+
it('maps to oidc skipNonceInAuthCodeRequest', function () {
459+
expect(optionsTest(cliOptions)).to.deep.equal({
460+
driver: {
461+
oidc: {
462+
skipNonceInAuthCodeRequest: true,
463+
},
464+
},
465+
});
466+
});
467+
});
468+
453469
context('when cli args have browser', function () {
454470
it('maps to oidc command', function () {
455471
expect(optionsTest({ browser: '/usr/bin/browser' })).to.deep.equal({

packages/arg-parser/src/arg-mapper.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ function setServerApi<Key extends keyof ServerApi>(
2929
const serverApi =
3030
typeof previousServerApi === 'string'
3131
? { version: previousServerApi }
32-
: { ...previousServerApi } ?? {};
32+
: { ...previousServerApi };
3333
serverApi[key] = value;
3434
return setDriver(i, 'serverApi', serverApi as Required<ServerApi>);
3535
}
@@ -237,6 +237,7 @@ const MAPPINGS: {
237237
v.split(',').filter(Boolean) as OIDCOptions['allowedFlows']
238238
),
239239
oidcIdTokenAsAccessToken: (i, v) => setOIDC(i, 'passIdTokenAsAccessToken', v),
240+
oidcNoNonce: (i, v) => setOIDC(i, 'skipNonceInAuthCodeRequest', v),
240241
browser: (i, v) =>
241242
setOIDC(i, 'openBrowser', typeof v === 'string' ? { command: v } : v),
242243
};

0 commit comments

Comments
 (0)