Code Monkey home page Code Monkey logo

Comments (9)

mertyildiran avatar mertyildiran commented on June 25, 2024 1

hey @mertyildiran

Thanks for the feedback

few follow-up questions: I see you are using chrome 113, may I ask which OS + version are you using?

The reason that I'm asking is because the PublicKeyCredential is a browser API that is used in a dependency of the react sdk (which is another Descope package) for biometrics authentication support

I see in most of the places, it consider a case where the browser is not support it, but there can be a flaw in the way it is done, or maybe a place that it misses to check that it is support it

Browser and OS

Chrome: Version 113.0.5672.92 (Official Build) (64-bit)
Linux distro: Ubuntu 20.04 LTS
Kernel version: 6.0.12-76060012-generic

NPM

react version: 17.0.2
react-scripts version: 5.0.1
@descope/react-sdk version: 1.0.3
webpack version: 5.83.1

package.json:

"browserslist": {
  "production": [
    ">0.2%",
    "not dead",
    "not op_mini all"
  ],
  "development": [
    "last 1 chrome version",
    "last 1 firefox version",
    "last 1 safari version"
  ]
}

tsconfig.json:

{
  "compilerOptions": {
    "outDir": "dist",
    "module": "esnext",
    "lib": [
      "dom",
      "esnext",
      "dom.iterable"
    ],
    "moduleResolution": "node",
    "jsx": "react-jsx",
    "sourceMap": true,
    "declaration": true,
    "allowSyntheticDefaultImports": true,
    "allowJs": true,
    "skipLibCheck": true,
    "strict": false,
    "forceConsistentCasingInFileNames": true,
    "resolveJsonModule": true,
    "isolatedModules": true,
    "noEmit": true,
    "downlevelIteration": true,
    "noFallthroughCasesInSwitch": true,
    "target": "es5",
    "esModuleInterop": true
  },
  "include": [
    "src",
    "src/types"
  ],
  "exclude": [
    "node_modules",
    "dist",
    "example"
  ]
}

from react-sdk.

dorsha avatar dorsha commented on June 25, 2024

Thanks!
We are looking into this.

from react-sdk.

asafshen avatar asafshen commented on June 25, 2024

hey @mertyildiran

Thanks for the feedback

few follow-up questions: I see you are using chrome 113, may I ask which OS + version are you using?

The reason that I'm asking is because the PublicKeyCredential is a browser API that is used in a dependency of the react sdk (which is another Descope package) for biometrics authentication support

I see in most of the places, it consider a case where the browser is not support it, but there can be a flaw in the way it is done, or maybe a place that it misses to check that it is support it

from react-sdk.

mertyildiran avatar mertyildiran commented on June 25, 2024

By the way, it happens specifically when I build the Docker image:

FROM node:16-alpine as builder
WORKDIR /app
COPY package.json package-lock.json ./
RUN npm ci --omit=dev
COPY . .
RUN npm run build

Maybe related to being node:16-alpine I don't know. The local build does not reproduce the issue.

from react-sdk.

asafshen avatar asafshen commented on June 25, 2024

By the way, it happens specifically when I build the Docker image:

FROM node:16-alpine as builder

WORKDIR /app

COPY package.json package-lock.json ./

RUN npm ci --omit=dev

COPY . .

RUN npm run build

Maybe related to being node:16-alpine I don't know. The local build does not reproduce the issue.

Got it, thanks for the tip

from react-sdk.

asafshen avatar asafshen commented on June 25, 2024

@mertyildiran the fix is available in the latest next version @descope/react-sdk@next version, and will be generally available in the next release

to use latest next version you can install it directly:

npm i --save @descope/[email protected]

from react-sdk.

mertyildiran avatar mertyildiran commented on June 25, 2024

@mertyildiran the fix is available in the latest next version @descope/react-sdk@next version, and will be generally available in the next release

to use latest next version you can install it directly:

npm i --save @descope/[email protected]

Thanks for the update @asafshen!

I have upgraded to @descope/[email protected] but it seems like there is a new error now:

Uncaught TypeError: t.get is not a function
    at o (tslib.es6.js:209:23)
    at new n (index-84419228.js:1:5757)
    at n.<anonymous> (createSuper.js:11:24)
    at new n (index-84419228.js:1:12631)
    at rs (react-dom.production.min.js:206:287)
    at Cl (react-dom.production.min.js:251:81)
    at xl (react-dom.production.min.js:250:408)
    at Tl (react-dom.production.min.js:250:278)
    at Sl (react-dom.production.min.js:250:138)
    at ml (react-dom.production.min.js:243:163)

and the login form does not appear.

from react-sdk.

asafshen avatar asafshen commented on June 25, 2024

@mertyildiran thanks fro the quick feedback

I could not reproduce (yet) this error, are you sure that this error comes from the Descope sdk code?

few questions:

  1. Do you have a way to run the application with a sourcemap, so we can see the exception call-stack with proper names/files?
  2. Does the error happen when the code is running both locally and served from a docker container?
  3. I have created a simple application that runs a simple react application with descope inside a docker container (fyi it is running against mine Descope project). can you see if that reproduce there when you run it? can you change the code/create a simple open source example where this reproduces?
    (instruction in the readme file, you can run the project locally or inside a docker iamge)

I'm here for any questions

from react-sdk.

mertyildiran avatar mertyildiran commented on June 25, 2024

@mertyildiran thanks fro the quick feedback

I could not reproduce (yet) this error, are you sure that this error comes from the Descope sdk code?

few questions:

  1. Do you have a way to run the application with a sourcemap, so we can see the exception call-stack with proper names/files?
  2. Does the error happen when the code is running both locally and served from a docker container?
  3. I have created a simple application that runs a simple react application with descope inside a docker container (fyi it is running against mine Descope project). can you see if that reproduce there when you run it? can you change the code/create a simple open source example where this reproduces?
    (instruction in the readme file, you can run the project locally or inside a docker iamge)

I'm here for any questions

@asafshen I have created a new issue here: #301

from react-sdk.

Related Issues (6)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.