Skip to content

Commit

Permalink
fix(@wdio/sauce-service): replace vulnerable ip module with a maintai…
Browse files Browse the repository at this point in the history
…ned package
  • Loading branch information
christian-bromann committed Aug 28, 2024
1 parent 2ab8239 commit 73bdc0f
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 5 deletions.
18 changes: 16 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/wdio-sauce-service/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"@wdio/logger": "8.38.0",
"@wdio/types": "8.40.3",
"@wdio/utils": "8.40.3",
"ip": "^2.0.1",
"address": "^2.0.3",
"saucelabs": "8.0.0",
"webdriverio": "8.40.3"
},
Expand Down
4 changes: 2 additions & 2 deletions packages/wdio-sauce-service/src/launcher.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { performance, PerformanceObserver } from 'node:perf_hooks'

import ip from 'ip'
import { ip } from 'address'
import {
default as SauceLabs,
type SauceLabsOptions,
Expand Down Expand Up @@ -51,7 +51,7 @@ export default class SauceLauncher implements Services.ServiceInstance {
noAutodetect: true,
tunnelIdentifier: sauceConnectTunnelIdentifier,
...this._options.sauceConnectOpts,
noSslBumpDomains: `127.0.0.1,localhost,${ip.address()}` + (
noSslBumpDomains: `127.0.0.1,localhost,${ip()}` + (
this._options.sauceConnectOpts?.noSslBumpDomains
? `,${this._options.sauceConnectOpts.noSslBumpDomains}`
: ''
Expand Down

0 comments on commit 73bdc0f

Please sign in to comment.