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

Adjust easy image to CS token refreshing mechanism #2558

Closed
pjasiun opened this issue Oct 24, 2017 · 0 comments · Fixed by ckeditor/ckeditor5-easy-image#8
Closed

Adjust easy image to CS token refreshing mechanism #2558

pjasiun opened this issue Oct 24, 2017 · 0 comments · Fixed by ckeditor/ckeditor5-easy-image#8
Assignees
Labels
package:easy-image type:task This issue reports a chore (non-production change) and other types of "todos".
Milestone

Comments

@pjasiun
Copy link

pjasiun commented Oct 24, 2017

After this ticket will be closed: https://github.com/ckeditor/ckeditor-cloudservices-core/issues/4 CS will automatically request and update token. That's really cool.

But it means that easy image configuration will have to change. Instead of the token, one will set the URL and the Token class will automatically request for the token.

However, it also means that we need to have a plugin which takes the configuration, create a token class and provide it to all plugins which connect with CS. Otherwise, each plugin will create its own request to refresh the token what will be wrong.

Also, such plugin should not be part of the easy-image repository, because it would be strange that all plugins require ckeditor5-easy-image. So we need a ckeditor5-cloudeservices repository.

In fact, it will be nice because of one more reason: it will fit the convention that the name of the configuration is the name of the plugin which reads this configuration. Now, easy-image is using cloudeServices.token config. After the change cloudeServices.tokenUrl will be the configuration of the cloudeServices plugin.

pjasiun referenced this issue in ckeditor/ckeditor5-easy-image Oct 31, 2017
Other: Aligned code to the new CloudeServices API and use `ckeditor5-cloudservices` package. Closes #7.

Feature: The plugin will now automatically refresh token when it will expire.

BREAKING CHANGE: Easy Image plugin do not use `cloudeservices.token` config anymore. The proper config now is `cloudeservices.tokenUrl`with the URL to the token server.
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-easy-image Oct 9, 2019
@mlewand mlewand added this to the iteration 13 milestone Oct 9, 2019
@mlewand mlewand added status:confirmed type:task This issue reports a chore (non-production change) and other types of "todos". package:easy-image labels Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:easy-image type:task This issue reports a chore (non-production change) and other types of "todos".
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants