Cédric Luthi

Results 226 comments of Cédric Luthi

I used the [genschsa/mssql-server-linux](https://hub.docker.com/r/genschsa/mssql-server-linux/) image which does it exactly like the postgres and mysql/mysql-server images, i.e. > docker run -p 1433:1433 --name mssql -e 'ACCEPT_EULA=Y' -e 'MSSQL_SA_PASSWORD=Strong(!)Password' **-v $PWD/initdb.d:/docker-entrypoint-initdb.d** -d...

> We did that intentionally to limit the stack trace that was exported. This seems weird to me. Losing some part of the original stack trace will just make it...

Would you mind reconsidering this pull request?

> Its necessary to refresh this token on request when expired. Refreshing is necessary when performing HTTP requests that must include a valid bearer token, but when accessing a property...

Actually I think we could have both: * public string CurrentAccessToken { get; } * public Task RefreshAccessTokenAsync(CancellationToken cancellationToken); Did you get a chance to discuss the matter?

> Are we giving promises that everytime we run into exception, we would be updating the properties? Only when the exception happens in the **user callback**, i.e. the `handler` passed...

> but just adding the dependency to the project isn't going to happen unfortunately This pull request is not about adding the dependency to the Westwind.Globalization project, but about fixing...