reapit / foundations

Foundations platform mono repo
56 stars 22 forks source link

Redirecting to developers.marketplace instead of localhost #6088

Closed sagarwal3005 closed 2 years ago

sagarwal3005 commented 2 years ago

Describe the bug A clear and concise description of what the bug is. We developed a small app to update a app using reapit scaffolding app and api connection. When we run the app on localhost, after authenticating using reapi user ame and password, it redirects user to the developers link instead of the authenticted platform. The app was working 1.5 years ago but now it does not work. The redirect urls are correct in the app and I updated the urls from old dev.connect to the new developer url. The application is on localhost so please let me know how can I help you to reproduce the issue.

To Reproduce Steps to reproduce the behaviour eg:

  1. Go to '...' localhost:8080
  2. Click on '....' after authentication it should redirect to the authenticated app instead it redirects to Developers link
  3. Scroll down to '....'
  4. See error

Expected behaviour A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem. Please ensure that screenshots do not include sensitive data such as personally identifiable information.

Device (please complete the following information for web issues):

Additional context Add any other context about the problem here.

github-actions[bot] commented 2 years ago

Thank you for taking the time to report a bug. We prioritise bugs depending on the severity and implications, so please ensure that you have provided as much information as possible. If you haven’t already, it really helps us to investigate the bug you have reported if you provide ‘Steps to Replicate’ and any associated screenshots. Please ensure any personal information from the production database is obscured when submitting screenshots. This issue will be reviewed in our weekly refinement sessions and assigned to a specific project board. We may also update the ticket to request additional information, if required. For more information on our processes, please click here

AshDeeming commented 2 years ago

Hi @sagarwal3005 - If I have understood correctly, you created an app using the previous webservice & now need to replicate with Foundations. Please follow the documentation on the developer portal, if you are yet to register, please see the link below; https://developers.reapit.cloud/register

sagarwal3005 commented 2 years ago

Hi, I did register myself on "developers.reapit.cloud" and made changes to the local application. But it's not working. To explain the issue, I recorded a video. Please find attached the video of the issue.

If you notice, the redirect_url passed was localhost:8081 but when the authentication is completed it returns to the https://developers.reapit.cloud/authorize and says page not found.

Not sure what am I doing wrong as it was working fine when I had it installed and modified 2 years back.

Regards Sneha

screen-capture (3).webm https://drive.google.com/file/d/1Rre0OUGGZ5rlN0hCz0fMCR_7RhoIX7oK/view?usp=drive_web

On Mon, Feb 7, 2022 at 8:24 PM Ash @.***> wrote:

Hi @sagarwal3005 https://github.com/sagarwal3005 - If I have understood correctly, you created an app using the previous webservice & now need to replicate with Foundations. Please follow the documentation on the developer portal, if you are yet to register, please see the link below; https://developers.reapit.cloud/register

— Reply to this email directly, view it on GitHub https://github.com/reapit/foundations/issues/6088#issuecomment-1031554394, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEO5ZWPZ2MKJJR47H673K5TUZ7MLXANCNFSM5NWQN4QQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

AshDeeming commented 2 years ago

Hi @sagarwal3005 - What is the name of your app please?

sagarwal3005 commented 2 years ago

GazealApp

On Tue, Feb 8, 2022, 2:46 PM Ash @.***> wrote:

Hi @sagarwal3005 https://github.com/sagarwal3005 - What is the name of your app please?

— Reply to this email directly, view it on GitHub https://github.com/reapit/foundations/issues/6088#issuecomment-1032382214, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEO5ZWKP5NVARISKNMBZULDU2DNORANCNFSM5NWQN4QQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

HollyJoyPhillips commented 2 years ago

@sagarwal3005 For the app 'Gazeal' which is registered under the developer organisation 'Gazeal Limited' it looks like the redirect URL Is 8080/ not 8081 as mentioned above, can you please verify where you are seeing it set as 8081?

sagarwal3005 commented 2 years ago

Hello,

Yes the redirect url is 8080 but the application is also running on 8080. please check the video attached screen-capture (4).webm https://drive.google.com/file/d/1QM5_BTp1EnaQodPRzkCfiDwah0FgO0pY/view?usp=drive_web

On Tue, Feb 8, 2022 at 3:53 PM HollyJoyPhillips @.***> wrote:

@sagarwal3005 https://github.com/sagarwal3005 For the app 'Gazeal' which is registered under the developer organisation 'Gazeal Limited' it looks like the redirect URL Is 8080/ not 8081 as mentioned above, can you please verify where you are seeing it set as 8081?

— Reply to this email directly, view it on GitHub https://github.com/reapit/foundations/issues/6088#issuecomment-1032447206, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEO5ZWL64TJJJEZSHEG62UTU2DVK5ANCNFSM5NWQN4QQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

HollyJoyPhillips commented 2 years ago

@sagarwal3005 Could you confirm how you have integrated Reapit Connect? It may seem that if you are using connect session, the 'connectLoginRedirectPath' is pointing to the wrong location. This section in the documentation may help: https://foundations-documentation.reapit.cloud/app-development/connect-session#basic-browser-usage

sagarwal3005 commented 2 years ago

Hi, I had created this app 2 years back using the scaffolding app sample that time. Please find below the screenshot of my config file.

[image: image.png]

This does not have a connectLoginRedirectPath. Should I add the value manually here?

On Wed, Feb 9, 2022 at 10:29 PM HollyJoyPhillips @.***> wrote:

@sagarwal3005 https://github.com/sagarwal3005 Could you confirm how you have integrated Reapit Connect? It may seem that if you are using connect session, the 'connectLoginRedirectPath' is pointing to the wrong location. This section in the documentation may help: https://foundations-documentation.reapit.cloud/app-development/connect-session#basic-browser-usage

— Reply to this email directly, view it on GitHub https://github.com/reapit/foundations/issues/6088#issuecomment-1033982991, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEO5ZWN7W3TMS3DJPQAPKRTU2KMORANCNFSM5NWQN4QQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

HollyJoyPhillips commented 2 years ago

@sagarwal3005 Unfortunately, the image you provided is not showing in the above comment.

It’s quite difficult for us to debug this with you seeing as the app was created quite a long time ago. I would suggest using the create react scaffolding app to re scaffold your app which will include connect session by default. This will ensure everything is up to date and using the latest components etc.

You can take your components from the old repo into the new one, then if you have any issues, it will be easier for us to help/for you to debug.

You can see more information here https://foundations-documentation.reapit.cloud/app-development/create-react-app-template and https://foundations-documentation.reapit.cloud/app-development/connect-session

sagarwal3005 commented 2 years ago

Hello,

I did create the new scaffolding app using the link you sent. I updated the redirect url path in index.tsx. Please find the index.tsx file attached to this email. i get an error redirect_mismatch on running the application.

I changed the connectLoginRedirectPath: 'https://localhost:8080/',to '/', '/login', '' but all returns same.

Please let me know what am I missing.

Regards Sneha

On Thu, Feb 10, 2022 at 3:09 PM HollyJoyPhillips @.***> wrote:

@sagarwal3005 https://github.com/sagarwal3005 Unfortunately, the image you provided is not showing in the above comment.

It’s quite difficult for us to debug this with you seeing as the app was created quite a long time ago. I would suggest using the create react scaffolding app to re scaffold your app which will include connect session by default. This will ensure everything is up to date and using the latest components etc.

You can take your components from the old repo into the new one, then if you have any issues, it will be easier for us to help/for you to debug.

You can see more information here https://foundations-documentation.reapit.cloud/app-development/create-react-app-template and https://foundations-documentation.reapit.cloud/app-development/connect-session

— Reply to this email directly, view it on GitHub https://github.com/reapit/foundations/issues/6088#issuecomment-1034699107, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEO5ZWLNUSUC5FXVQP5PVYDU2OBUNANCNFSM5NWQN4QQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

HollyJoyPhillips commented 2 years ago

@sagarwal3005 Unfortunately, the file didn't attach. Please can you add it direct on this github issue rather than replying to the email and i'll take a look.

Thanks.

github-actions[bot] commented 2 years ago

This issue has recently been assigned to our ‘Front-end’ project board for review. All issues are reviewed in a weekly refinement session and where applicable, a comment and associated label will be added. If required, we will add a technical specification to the ticket. Please take the time to review the information. When we're ready to schedule the issue, it will be moved to the relevant column where you can continue to track its progress to completion. For more information on our processes, please click here

willmcvay commented 2 years ago

@sagarwal3005 The redirect path must be exactly the same as those set in your Developer Portal app so if you have http://localhost:8080 set in the app page, the connectLoginRedirectPath should be set to '', not /

willmcvay commented 2 years ago

@sagarwal3005 Can you confirm this has been fixed. Will close the issue off if I don't hear back.