Raymond Kerckhoffs

> We are facing the same problem with an IdentityServer which does not support CORS for `/conntect/token` requests. > > Most swagger implementations (we are using `NSwag` and `Swashbuckle`) do...

The command `npx playwright install` redirects to https://playwright.azureedge.net/*. Is the install affected by https://devblogs.microsoft.com/dotnet/critical-dotnet-install-links-are-changing and necessary to update the links?

@detule I've pushed a new version of the branch which results in a clean diff. In addition the workaround of #512 has been reverted.

There is one issue while using the _query_timeout_ and _params_. I will check and fix this! **Update:** Issue has been detected, I will commit the fix tomorrow.

A new fix has been pushed which resolves the issue of a query using _params_ and _query_timeout_. Additional tests have been added as well. Pull request is ready for review.

@detule Did you have an opportunity to review and approve the pull request?

@detule and @shrektan Thanks for reviewing! I will update the pull request later on this week. It seems there is always some confusion between a _connection_ timeout and an actual...

@aryoda Thank you for your quick reply! It is indeed the purpose of having a query timeout to stop long running queries (or stored procedures). I did have a look...

@aryoda, @detule, @jimhester I've checked and tested the changes multiple times, but the [next line of code](https://github.com/Odraio/odbc/blob/feature/query_timeout/src/odbc_result.cpp#L51) doesn't seem to use the `query_timeout` of the statement. (Atleast long running queries...

@aryoda I'm not sure which execute() function is called of nanodbc: [L3505 - result execute(connection& conn, const string_type& query, long batch_operations, long timeout)](https://github.com/r-dbi/odbc/blob/main/src/nanodbc/nanodbc.cpp#L3505) [L3517 - result execute(statement& stmt, long batch_operations)](https://github.com/r-dbi/odbc/blob/main/src/nanodbc/nanodbc.cpp#L3517)...