Replace UseSession(SessionOptions) with UseSession(Action<SessionOptions>)

[HaoK]

Since the session middleware is using IOptions, we should mirror how we normally configure options... It should instead always use IOptions and apply the action to it afterwards (basically a PostPostConfigureOptions)

We should revisit any other middleware that are using this pattern and using IOptions

UseSession(new SessionOptions()) => UseSession(o => { })

[muratg]

@HaoK is this still a valid issue? Are we going to do this?

[HaoK]

Its a consistency issue, but I'm not sure this is something we could change in a minor release (breaking changes and all)

[Tratcher]

We could add the new one.

[HaoK]

Yeah we could add that across the board, UseXyz(o => { }) everywhere that's currently only taking UseXyz(new XyzOptions())

[aspnet-hello]

This issue was moved to dotnet/aspnetcore#2402