Skip to content

GH-3027: Decouple RotationPolicy #3032

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

Merged
merged 1 commit into from
Aug 15, 2019

Conversation

artembilan
Copy link
Member

Fixes #3027

This is a polishing for PR #3029

  • Move KeyDirectory in to the RotationPolicy since it looks like fully
    coupled concept with that abstraction and the class name is so bad for
    the public API
  • Remove AbstractStandardRotationPolicy in favor of a StandardRotationPolicy
    extendability
  • Change ftp.adoc and sftp.adoc to reflect a new API reality
  • Mantion these changes in the whats-new.adoc

Fixes spring-projects#3027

This is a polishing for PR spring-projects#3029

* Move `KeyDirectory` in to the `RotationPolicy` since it looks like fully
coupled concept with that abstraction and the class name is so bad for
the public API
* Remove `AbstractStandardRotationPolicy` in favor of a `StandardRotationPolicy`
extendability
* Change `ftp.adoc` and `sftp.adoc` to reflect a new API reality
* Mantion these changes in the `whats-new.adoc`
@dturanski
Copy link
Contributor

Thanks. LGTM;

@garyrussell garyrussell merged commit 13a8623 into spring-projects:master Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancements to RotatingServerAdvice
3 participants