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

[aspnet-hello]

From @HaoK on Thursday, June 29, 2017 6:47:53 PM

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 => { })

Copied from original issue: aspnet/Session#174

[aspnet-hello]

From @muratg on Tuesday, September 26, 2017 3:50:16 PM

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

[aspnet-hello]

From @HaoK on Tuesday, September 26, 2017 4:15:47 PM

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

[aspnet-hello]

From @Tratcher on Tuesday, September 26, 2017 4:16:50 PM

We could add the new one.

[aspnet-hello]

From @HaoK on Tuesday, September 26, 2017 4:31:59 PM

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

[muratg]

I don't expect we'll make changes here.