Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Token refresh request is being sent after editor is destroyed #17462

Closed
lkszzajac opened this issue Nov 14, 2024 · 0 comments · Fixed by #17466
Closed

Token refresh request is being sent after editor is destroyed #17462

lkszzajac opened this issue Nov 14, 2024 · 0 comments · Fixed by #17466
Labels
package:cloud-services support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@lkszzajac
Copy link

📝 Provide detailed reproduction steps (if any)

  1. Create an editor that sends a request for an auth token to Cloud Services
  2. (optional) In the user-defined token endpoint, modify the lifespan of the token to some low value, like 20s
  3. While using the editor with a valid token, close the endpoint token service to fail the token refresh request.
  4. Immediately after the first failure, destroy the editor using editor.destroy()

✔️ Expected result

Editor is destroyed, no token requests are being made

❌ Actual result

  • token refresh requests are being made every 5s
  • this seems to happen at random
  • opening other secondary editors (Revision History, Comment/Suggestion input etc) may influence the behavior

📃 Other details

  • Browser: n/a
  • OS: n/a
  • First affected CKEditor version: n/a
  • Installed CKEditor plugins: n/a

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@lkszzajac lkszzajac added package:cloud-services support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior. labels Nov 14, 2024
@CKEditorBot CKEditorBot added the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Nov 14, 2024
@CKEditorBot CKEditorBot removed the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Nov 18, 2024
@CKEditorBot CKEditorBot added this to the iteration 80 milestone Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:cloud-services support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants