Skip to content

OAuth request elements are duplicated in the API console and API request panel #678

Open

Description

@deiteris

Current examples of element and standalone API console include oauth1-authorization and oauth2-authorization handlers in both, the API console and the api-request-editor component which results in OAuth-related request events fired twice.

Expected outcome

OAuth request handlers should be included in either the API console or the request editor.

Actual outcome

OAuth request handlers included in both elements.

Steps to reproduce

  1. Clone the git repo, run npm i and npm start after the installation.
  2. In the opened browser tab, open the "API console as a standalone application" link or the "API console as a web component" link.
  3. Open any API endpoint method and open the request panel.
  4. Inspect the page code in browser's developer tools and search for <oauth2-authorization. See two entries instead of only one.

Additional information

OAuth handlers are included here https://github.com/advanced-rest-client/api-request/blob/stage/src/ApiRequestEditorElement.js#L986
Standalone API console also includes it here https://github.com/mulesoft/api-console/blob/master/src/ApiConsoleApp.js#L429
The API console element includes it here https://github.com/mulesoft/api-console/blob/master/demo/element/app.js#L65

Was this working before?

  • Yes, it was working before
  • No, it was already broken
  • I don't know / not sure

Affected browsers

  • All
  • Chrome
  • Firefox
  • Internet Explorer <\3
  • Edge
  • Safari
  • Other (please, specify)

Activity

  1. changed the title [-]OAuth request elements are duplicated in the API console itself and request panel[/-] [+]OAuth request elements are duplicated in the API console and API request panel[/+] on Mar 8, 2021
  2. deiteris commented on Aug 15, 2021

    @deiteris
    Author

    Any progress on this issue? It's just about removing it from one place or another, and rate limits are no joke in some cases.

    I workarounded this by overriding it with my own _helpersTemplate() in the extending class, but still I guess it should be up to implementer, not the request panel.

  3. jarrodek commented on Aug 16, 2021

    @jarrodek
    Contributor

    These 2 components should only be included in the api-console-app. For any other use case they should be included separately or the events should be handled by the hosting application. @twoplustwoone can you please take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions