Skip to content
This repository was archived by the owner on Apr 4, 2025. It is now read-only.

Refactor ReactiveMongoWebSessionConfiguration #19

Closed
wants to merge 1 commit into from
Closed

Refactor ReactiveMongoWebSessionConfiguration #19

wants to merge 1 commit into from

Conversation

vpavic
Copy link
Contributor

@vpavic vpavic commented Nov 16, 2017

This PR updates ReactiveMongoWebSessionConfiguration to extend SpringWebSessionConfiguration rather than having it imported via @EnableMongoWebSession.

This is required in order for Spring Boot auto-configuration to work properly since it extends the ReactiveMongoWebSessionConfiguration. The same approach is used in the MongoHttpSessionConfiguration and all configuration classes in spring-session project.

This commit updates `ReactiveMongoWebSessionConfiguration` to extend `SpringWebSessionConfiguration` rather than having it imported via `@EnableMongoWebSession`. This is required in order for Spring Boot auto-configuration to work properly.
@gregturn
Copy link
Contributor

I'll merge and polish it this afternoon.

@gregturn
Copy link
Contributor

Resolved via e8d12b5.

Thanks!

@gregturn gregturn closed this Nov 17, 2017
@vpavic vpavic deleted the fix-reactive-config branch November 17, 2017 18:43
@vpavic
Copy link
Contributor Author

vpavic commented Nov 19, 2017

Thanks for merging this Greg - any chance you do a quick RC2?

There are two Boot issues/PRs blocked on this (spring-projects/spring-boot#11055 and spring-projects/spring-boot#11063) so it would be nice if we could get those in early? Note that the Spring Session GA is scheduled for Nov 27th ATM.

@gregturn
Copy link
Contributor

Boot M7 is scheduled for the 28th. Is it not sufficient to wait for Session's GA scheduled on the 27th? (I was planning to have ours GA on the same date)

cc @rwinch

@vpavic
Copy link
Contributor Author

vpavic commented Nov 19, 2017

Yes, I guess that's an option now that Boot has been delayed. Session's GA was originally scheduled after Boot's M7.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants