Open connection issue on Linux

I'm using Microsoft.Data.SqlClient 1.0.19128.1-preview on Linux RHEL 6 to connect to SQLServer running on Windows.
This is my connection string:
string connetionString Server="SomeComputer\SQLIntegTest,61245;Database=db4;uid=user;pwd=password;Trusted_Connection=No"
with this code:
SqlConnection cnn = new SqlConnection(connetionString);
cnn.Open()
I get the following error.  The same code works on windows.  Any help would be appreciated.
Unhandled Exception: Microsoft.Data.SqlClient.SqlException: Connection Timeout Expired.  The timeout period elapsed during the post-login phase.  The connection could have timed out while waiting for server to complete the login process and respond; Or it could have timed out while attempting to create multiple active connections.  The duration spent while attempting to connect to this server was - [Pre-Login] initialization=118; handshake=271; [Login] initialization=11; authentication=14; [Post-Login] complete=14041;  ---> System.ComponentModel.Win32Exception: Unknown error 258
--- End of inner exception stack trace ---
at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken)
at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)    at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource1 retry, DbConnectionOptions userOptions)    at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource1 retry)
at Microsoft.Data.SqlClient.SqlConnection.Open()
Abort (core dumped)

That is an odd server name in the connection string, maybe change to:
SomeComputer,61245

I tried your suggestion same result works on Windows but not Linux.

Can you resolve the server name from the Linux box?

Yes I use the same connection string with python pyodbc just adding the driver parameter which is not need with c#

Well I was testing with SQL Server 2008 and switched to SQL Server 2016 and that works.  Interestingly the Windows version of Microsoft.Data.SqlClient with both but the Linux version only works with 2016.

@sjc1832
Have you checked with SQL Server 2012 by any chance? Since SQL Server 2008 and 2008 R2 are Out of Support, we wouldn't fix the driver to support them. If the issue happens with SQL Server 2012 as well (which is the oldest of supported SQL Servers) we can consider taking a deeper look.

I have not checked 2012.

I'm facing same issue - i.e. my dotnet core 3 preview8 application is running in Kubernetes cluster (on premise Linux container) which is trying to access remote MSSQL Server 2014 - but hung at connection.Open().
But same works with MSSQL Server 2016.
Connection string : "Data Source=<IP_ADDRESS>,1433;Initial Catalog=ReportServer;Connect Timeout=60;User ID=username;Password=password; Persist Security Info=False;Packet Size=4096"
Docker images used : mcr.microsoft.com/dotnet/core/sdk:3.0.100-preview8 & mcr.microsoft.com/dotnet/core/aspnet:3.0.0-preview8

Hi,
I am running into the same issue. We just upgraded one of our apps to .netcore 3. Connecting to sql works on my windows dev machine, but when we deploy to our staging environment (ECS cluster on AWS - linux machines) it stops working.
The error we get in our logs is: System.InvalidOperationException: Timeout expired. The timeout period elapsed prior to obtaining a connection from the pool. This may have occurred because all pooled connections were in use and max pool size was reached.
The connection string is: Application Name={appName};Server={host};User ID={id};Password={password};Database={dbname};Persist Security Info=false;Connection Lifetime=60
The machine has access to the sql box, if I role the code back to the .net core 2.2 version everything works fine. Its after upgrade to .netcore 3 it has stopped.
The code that fails is simply
using (SqlConnection connection = new SqlConnection(settings.ConnectionString))
{
connection.Open();
}
The sql box is an RDS instance provided by AWS. The Engine version is 13.00.2164.0.v1. I dont seem to be able to see what version of sql server it is under the hood, but it looks like its 2016 (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/CHAP_SQLServer.html)
The EC2 box inside our ECS cluster is Amazon Linux AMI 2018.03.p x86_64 ECS HVM GP2
Docker Images used:
mcr.microsoft.com/dotnet/core/aspnet:3.0
mcr.microsoft.com/dotnet/core/sdk:3.0

Hi @lukebronts

System.InvalidOperationException: Timeout expired. The timeout period elapsed prior to obtaining a connection from the pool. This may have occurred because all pooled connections were in use and max pool size was reached.

Could you tell us the version of Microsoft.Data.SqlClient driver in use?
Please try updating to latest Microsoft.Data.SqlClient version "1.0.19269.1" and confirm if the error occurs again.

Hi @cheenamalhotra
I have tried both 1.0.19269.1 and the current one in preview 1.1.0-preview1.19275.1. Both do not work.
The sql version is Microsoft SQL Server 2016 (RTM-CU2) (KB3182270) - 13.0.2164.0 (X64)   Sep  9 2016 20:13:26   Copyright (c) Microsoft Corporation  Standard Edition (64-bit) on Windows Server 2012 R2 Standard 6.3  (Build 9600: ) (Hypervisor)

Hi @cheenamalhotra
Do you have an update on this issue? Have you managed to find anything?
Thanks

yes, I have the same issue using netcore 2.2 and Microsoft.Data.SqlClient

@mgbee8  Same with us.
asp.net core 3 docker container and MS SQL 2016 (13.0.5216.0) on AWS RDS.
However, this happens only in Test environment.

An exception occurred while iterating over the results of a query for context type 'IdentityServer4.EntityFramework.DbContexts.PersistedGrantDbContext'. Microsoft.Data.SqlClient.SqlException (0x80131904): Execution Timeout Expired. The timeout period elapsed prior to completion of the operation or the server is not responding. ---> System.ComponentModel.Win32Exception (258): Unknown error 258 at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction) at

Maybe there is some relation to new client.

Im having the same issue as @voroninp .
I have a production stack in AWS working fine.  My test stack times out connecting to RDS.
Task running in fargate in private subnet of own vpc.
SQL Express 14.00.3015.40.v1 in same private subnet.
.Net Core 3 in docker mcr.microsoft.com/dotnet/core/aspnet:3.0
I spend 4 hours with AWS support and proved that the network and security setup is correct.  We could get powershell to connect to SQL ok.

I spent hours yesterday trying everything to fix this issue and ended up finding that a TLS version mismatch caused the issue on my debian10 server.
Server version:
Linux 4.19.0-6-amd64 #1 SMP Debian 4.19.67-2+deb10u2 (2019-11-11) x86_64
Dotnet core version:
APP#1 -> 3.1 self-contained aspnetcore webapp
APP#2 -> 2.2 self-contained aspnetcore webapp
APP#1 could not connect to SqlServer, same error as others in this thread :
Timeout expired.  The timeout period elapsed prior to obtaining a connection from the pool.  This may have occurred because all pooled connections were in use and max pool size was reached.
at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)\n   at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)\n   at Microsoft.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)\n   at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)\n   at Microsoft.Data.SqlClient.SqlConnection.Open()\n   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenDbConnection(Boolean errorsExpected)\n   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.Open(Boolean errorsExpected)\n   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteReader(RelationalCommandParameterObject parameterObject)\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.InitializeReader(DbContext _, Boolean result)\n   at Microsoft.EntityFrameworkCore.SqlServer.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.MoveNext()\n   at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source)\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)\n   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)\n   at System.Linq.Queryable.First[TSource](IQueryable`1 source)\n

APP#2: Could connect just fine.
I edited the openssl.conf file and rebooted the server, problem solved.
/etc/ssl/openssl.cnf
at the very end of the file, I commented ssl_conf and changed MinProtocol (I don't think MinProtocol matters though)
[default_conf]
#ssl_conf = ssl_sect

[ssl_sect]
system_default = system_default_sect

[system_default_sect]
MinProtocol = TLSv1.1
CipherString = DEFAULT@SECLEVEL=2

dotnetcore 3.1 app can now connect to SQL server just fine.

Hi @avboivin
The exception you posted doesn't seem related to TLS Version change, but rebooting might have solved it since that would flush all open connections. Could you update Microsoft.Data.SqlClient version to 1.1.0 (with explicit package reference) and then confirm if the issue persists.
If you have TLS 1.2 support enabled on Server, client supporting TLS1.2 protocol should be able to connect without any change needed to SSL Configurations.
To know the TLS version in use, you can capture network traces with Wireshark, where you can see the TLS version in pre-login packets where Client and Server perform handshake.

Hi @cheenamalhotra, I just tried uncommenting and rebooting and could reproduce the issue.
Uncommented this line from /etc/ssl/openssl.cnf
ssl_conf = ssl_sect
sudo reboot
I get a connection pool error on my dotnetcore 3.1 apps.
The 3.1 apps use EFcore 3 and microsoft.data.SqlClient 1.1.0
Dotnetcore 2.2 apps still work great and can connect to the DB
Commented the line from /etc/ssl/openssl.cnf again
#ssl_conf = ssl_sect
sudo reboot
Both apps work great and can now connect.
I've never used wireshark so if you can point me to a relevant tutorial I'd be inclined to try it but otherwise that's not something I'm willing to test right now.
Thanks for the quick response !

You can always find online tutorials to use Wireshark, it's a bit tricky to get hands-on, but if you're not facing issues, you may not need to :)

Thanks. It worked for me by adding single line in dockerfile because with dockerfile there is no need for explicit reboot:
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

Example dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base

WORKDIR /app
EXPOSE 80

FROM mcr.microsoft.com/dotnet/core/sdk:3.1 AS build
WORKDIR /src
COPY ["MyApp/MyApp.csproj", "MyApp/"]
RUN dotnet restore "MyApp/MyApp.csproj"
COPY . .
WORKDIR "/src/MyApp"
RUN dotnet build "MyApp.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "MyApp.csproj" -c Release -o /app/publish

FROM base AS final
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "eVision.MyApp.dll"]

We solved the same issue for for SQL Sever 2008 R2 by installing SP4.

Thanks. It worked for me by adding single line in dockerfile because with dockerfile there is no need for explicit reboot:
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

Example dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base

WORKDIR /app
EXPOSE 80

FROM mcr.microsoft.com/dotnet/core/sdk:3.1 AS build
WORKDIR /src
COPY ["MyApp/MyApp.csproj", "MyApp/"]
RUN dotnet restore "MyApp/MyApp.csproj"
COPY . .
WORKDIR "/src/MyApp"
RUN dotnet build "MyApp.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "MyApp.csproj" -c Release -o /app/publish

FROM base AS final
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "eVision.MyApp.dll"]

@iarovyi Thanks for this. I just lost several days trying to figure out what I was doing wrong and then took me a while to find this thread.
@cheenamalhotra I had this issue on MS SQL 2012 running exactly same stack as @iarovyi.
If you need more details let me know.

What is the Connection Timeout given?
If the Connection Timeout is 0 then try to change it to 30.

I'm experiencing the same issue using Microsoft.Data.SqlClient 1.1.1 connecting to SQL Server 2016. The sed command fixes it as a workaround for now, but it seems to be affecting newer SQL Server versions as well

Dear all, I get same error trying to connect to a SQL SERVER 2008. Code works like a charm in windows and mac (with dotnet for mac). But I get an error on the ubuntu docker container.

I'm using .NET core 3.1
last microsoft.sql.client
docker aspnet:3.1-bionic image
changed DEFAULT@SECLEVEL and Mimnprotocol for openssl following other discussions of this repo
tried out to set a lot of different configurations and a lot of way to build a connection string

Do you have any advice?  thanks
Connecting to SQL Server ... Microsoft.Data.SqlClient.SqlException (0x80131904): A connection was successfully established with the server, but then an error occurred during the login process. (provider: TCP Provider, error: 0 - Success)
at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action1 wrapCloseInAction)    at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)    at Microsoft.Data.SqlClient.TdsParserStateObject.ThrowExceptionAndWarning(Boolean callerHasConnectionLock, Boolean asyncClose)    at Microsoft.Data.SqlClient.TdsParserStateObject.ReadSniError(TdsParserStateObject stateObj, UInt32 error)    at Microsoft.Data.SqlClient.TdsParserStateObject.ReadSniSyncOverAsync()    at Microsoft.Data.SqlClient.TdsParserStateObject.TryReadNetworkPacket()    at Microsoft.Data.SqlClient.TdsParserStateObject.TryPrepareBuffer()    at Microsoft.Data.SqlClient.TdsParserStateObject.TryReadByte(Byte& value)    at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)    at Microsoft.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.CompleteLogin(Boolean enlistOK)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean ignoreSniOpenTimeout, TimeoutTimer timeout, Boolean withFailover)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool, SqlAuthenticationProviderManager sqlAuthProviderManager)    at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)    at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)    at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)    at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)
at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)    at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource1 retry, DbConnectionOptions userOptions)
at Microsoft.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource1 retry, DbConnectionOptions userOptions)    at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource1 retry)
at Microsoft.Data.SqlClient.SqlConnection.Open()
at WebApplication4.Controllers.WeatherForecastController.Get() in /src/WebApplication4/Controllers/WeatherForecastController.cs:line 66
ClientConnectionId:c57a691e-9396-4b55-9983-ab20679e9a26

I have the same "hanging sql server connection" problem with this setup:

SQL Server 2012 (SP3)
Docker base image Buster-3.1 (Debian 10)

Before applying the SP3 , the error was about failures around TLS handshake.
After SP3 was installed this error ceased, but now the connections just hangs foverer.
I've tried changing the base images, but all of the dotnet 3.1 images have the same problem.
Also I tried the SECLEVEL and MinTLSSupportedVersion scripts on the Docker imagem construction, no luck either.
This same app can connect normally on Windows, and the container can connect normally on SQL Server 2014 (SP3).

@jean-lourenco
Could you try removing the SECLEVEL scripts and let driver connect with TLS 1.2 if your server has all the updates?

@jean-lourenco
Could you try removing the SECLEVEL scripts and let driver connect with TLS 1.2 if your server has all the updates?

@cheenamalhotra I've tried that too, didn't work.

@albeserra  have you managed to resolved this case? We also had this issue on service running on container.

RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

May i what does this line means? What does it do? TIA

Hi all,
Updating the CA certificates in the base image of container have solve this for our case. See docker file below. I tried many options including @albeserra suggestions and I have remove many things and trimmed to this only command and it seems to what have resolved our case.
FROM mcr.microsoft.com/dotnet/core/runtime:3.1

...
...

RUN update-ca-certificates

...
...

Background:

.NET Core 3.1 app
Linux container, running on Azure Kubernetes/AKS cluster

I've tried changing the base images, but all of the dotnet 3.1 images have the same problem.
Also I tried the SECLEVEL and MinTLSSupportedVersion scripts on the Docker imagem construction, no luck either.

After some days I tried this again and managed to get the container working with the @albeserra sugestion. Idk what I was doing wrong previously, probably not build the container image correctly.

@albeserra have you managed to resolved this case? We also had this issue on service running on container.

RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

May i what does this line means? What does it do? TIA

@rdagumampan It comments out the ssl_conf = ssl_sect line of the open ssl config file.

I've tried changing the base images, but all of the dotnet 3.1 images have the same problem.
Also I tried the SECLEVEL and MinTLSSupportedVersion scripts on the Docker imagem construction, no luck either.

After some days I tried this again and managed to get the container working with the @albeserra sugestion. Idk what I was doing wrong previously, probably not build the container image correctly.

@albeserra have you managed to resolved this case? We also had this issue on service running on container.

RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

May i what does this line means? What does it do? TIA

@rdagumampan It comments out the ssl_conf = ssl_sect line of the open ssl config file.

Hi, no I didn't solve it. We abandoned the idea to build a .NET image for a linux container :( how did you solve it?

I'm using Microsoft.Data.SqlClient 1.0.19128.1-preview on Linux RHEL 6 to connect to SQLServer running on Windows.
This is my connection string:
string connetionString Server="SomeComputer\SQLIntegTest,61245;Database=db4;uid=user;pwd=password;Trusted_Connection=No"
with this code:
SqlConnection cnn = new SqlConnection(connetionString);
cnn.Open()
I get the following error. The same code works on windows. Any help would be appreciated.
Unhandled Exception: Microsoft.Data.SqlClient.SqlException: Connection Timeout Expired. The timeout period elapsed during the post-login phase. The connection could have timed out while waiting for server to complete the login process and respond; Or it could have timed out while attempting to create multiple active connections. The duration spent while attempting to connect to this server was - [Pre-Login] initialization=118; handshake=271; [Login] initialization=11; authentication=14; [Post-Login] complete=14041; ---> System.ComponentModel.Win32Exception: Unknown error 258
--- End of inner exception stack trace ---
at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken)
at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection) at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource1 retry, DbConnectionOptions userOptions) at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource1 retry)
at Microsoft.Data.SqlClient.SqlConnection.Open()
Abort (core dumped)

@sjc1832  did you solve this issue ? could you provide details

@mgbee8 Same with us.
asp.net core 3 docker container and MS SQL 2016 (13.0.5216.0) on AWS RDS.
However, this happens only in Test environment.

An exception occurred while iterating over the results of a query for context type 'IdentityServer4.EntityFramework.DbContexts.PersistedGrantDbContext'. Microsoft.Data.SqlClient.SqlException (0x80131904): Execution Timeout Expired. The timeout period elapsed prior to completion of the operation or the server is not responding. ---> System.ComponentModel.Win32Exception (258): Unknown error 258 at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction) at

Maybe there is some relation to new client.
@voroninp  did you solve this issue ? could you provide details please

@kolesso-programist Alas, I quit that company, so cannod add more details, whether issue was resolved or not.

@kolesso-programist Alas, I quit that company, so cannot add more details on whether issue was resolved or not.

fixed the problem using this command
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf
rejoin the company :)

Thanks. It worked for me by adding single line in dockerfile because with dockerfile there is no need for explicit reboot:
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

Example dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base

WORKDIR /app
EXPOSE 80

FROM mcr.microsoft.com/dotnet/core/sdk:3.1 AS build
WORKDIR /src
COPY ["MyApp/MyApp.csproj", "MyApp/"]
RUN dotnet restore "MyApp/MyApp.csproj"
COPY . .
WORKDIR "/src/MyApp"
RUN dotnet build "MyApp.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "MyApp.csproj" -c Release -o /app/publish

FROM base AS final
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "eVision.MyApp.dll"]

you made my day happier

Thanks. It worked for me by adding single line in dockerfile because with dockerfile there is no need for explicit reboot:
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf

Example dockerfile:
FROM mcr.microsoft.com/dotnet/core/aspnet:3.1 AS base

WORKDIR /app
EXPOSE 80

FROM mcr.microsoft.com/dotnet/core/sdk:3.1 AS build
WORKDIR /src
COPY ["MyApp/MyApp.csproj", "MyApp/"]
RUN dotnet restore "MyApp/MyApp.csproj"
COPY . .
WORKDIR "/src/MyApp"
RUN dotnet build "MyApp.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "MyApp.csproj" -c Release -o /app/publish

FROM base AS final
RUN sed -i '/^ssl_conf = ssl_sect$/s/^/#/' /etc/ssl/openssl.cnf
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "eVision.MyApp.dll"]

Thanks a lot man, you save alot of my 3 days of struggling


Fix PII on referrer parameter

What
When conducting a search on a finder application (e.g. https://www.gov.uk/search/all?), the referrer parameter on the pageView object does not redact PII. (Instructions to recreate the issue have been left on this trello card)
Why
PII issue fix.
Trello card
Visual Changes
N/A

Hi @AshGDS @andysellick, this is a quick fix and I'm sure there's a better way to do this. I was thinking of passing through the referrer parameter to the getReferrer function and the just setting a variable to either the referrer parameter if it existed or document.referrer and then just calling this.PIIRemover.stripPIIWithOverride once instead of twice as I've done here. I was getting some errors though so I pushed up the simplest fix. Hope I've made sense here, happy to discuss.

Also I think the visual regression test failures can be resolved by rebasing against main.


git-pr py3 compatible

Description
This PR...

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

commited manually on a local main and executed git pr 6956 --force

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

LGTM - is this enough or we need to do more?

not sure, It can only be manually tested, it seems and we need it in automation. Have to think of a trick just yet.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4823

testing in #6972

We may need to check/fix the backend Jenkins job or make this script run with either/both py2 and py3 @DaanHoogland @davidjumani
@blueorangutan package

@blueorangutan ui

@rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4865

@DaanHoogland @rohityadavcloud Packaging fails in this but based on my testing and https://github.com/apache/cloudstack/pull/6972 this can be merged


Seg Fault on MacOs

Bevy version
0.9.0
[Optional] Relevant system information
MacOS version: 13.0.1 (22A400)
Intel i7 (not M1/M2)
Rust version:
rustc 1.65.0 (897e37553 2022-11-02)
What you did
I found this on a more complicated program, but I found the same seg fault on this:
use bevy::prelude::*;

fn main() {
    App::new().add_plugins(DefaultPlugins).run();
}

What went wrong
It runs, but crashes in runtime, and gives this error:
Finished dev [optimized + debuginfo] target(s) in 1.70s
     Running `target/debug/bevy_test`
[1]    45941 segmentation fault  cargo run

what were you expecting?

To show the basic window

what actually happened?

A segfault
Additional information
if MinimalPlugins is used instead, then everything works as expected. It's obviously something that is specific to DefaultPlugins
Additionally, I was able to run bevy 0.8.1 fine, but I ran into this issue when switching to 0.9.0
I did some quick searches in winit's and wgpu's repos, and didn't see anything related to seg faults
I'm not 100% sure what commands I can run to get more information, but I'll try what people suggest

Can you try to narrow down which potential plugin is causing this? DefaultPlugins includes quite a few items that could be causing this crash.

Could you run bevy in a debugger and get a backtrace of where it segfaults?

The issue seems to be in the RednerPlugin
use bevy::{prelude::*, render::RenderPlugin, window::WindowPlugin};

fn main() {
    App::new()
        .add_plugin(WindowPlugin::default())
        .add_plugin(AssetPlugin::default())
        .add_plugin(RenderPlugin)
        .run();
}

This seg faults, but if I comment out the RenderPlugin line, then it doesn't. I'll try and look into this some more, but I haven't touched these plugins before

I can reproduce the seg fault on macOS 13.0.1 with bevy 0.9.0 and can also confirm it does not occur on bevy 0.8.1.
If I turn off any optimisations for packages
[profile.dev.package."*"]
opt-level = 1

it the seg fault does not occur. @roboteng could you give that a try and see if that is also the case for you?
I am trying to disable optimisation for specific packages to track down the issue in more detail, but so far I have not been able to get it working without setting the opt-level = 1 for every package. I will try to delve deeper once I get some free time.
I also tried stepping through the code with the debugger, and noticed that the seg fault sometimes does not occur then. So there may be a race condition somewhere. The error seems to occur while gathering and handling the device adapters. I just started out with bevy and rust, so my knowledge is a bit limited. If anyone has any additional pointers to solutions I am all for trying them.

does it segfault also with --release?

No seg fault when running with --release either. I did notice that eventhough it does not seg fault, no window is actually spawned. This is also the case in the optimised regular cargo dev run. I am not sure what other plugins need to be added as a minimum to actually spawn a window though.

Hmm, it seems like it's been fixed. Using just the DefaultPlugins I don't get the error anymore. When I tried with WIndowPlugin, AssetPlugin, and RenderPlugin, I still got an error, but the main point of using DefaultPlugins seems to be fixed. Doing a simple cargo run vs cargo run --release doesn't make a difference either, currently. I think there may still be some issues if you don't include all the correct plugins, but that seems expected to me

Still not fixed for me on Mac (Ventura 13.0.1, 2019). I get segfaults with --release and
[profile.dev.package."*"]
opt-level = 3

Cargo.toml:
[package]
name = "test"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
[profile.dev]
opt-level = 1

[profile.dev.package."*"]
opt-level = 1

[dependencies]
bevy = { version = "0.9.1", features = ["dynamic"] }
bevy-inspector-egui = "0.14.0"

Min example:
use bevy::{prelude::*, render::RenderPlugin};

fn main() {
    App::new()
        .add_plugins(MinimalPlugins)
        .add_plugin(WindowPlugin::default())
        .add_plugin(AssetPlugin::default())
        .add_plugin(RenderPlugin)
        .add_plugin(ImagePlugin::default())
        .run();
}

This example runs fine without the `--release' flag.

This one sounds like a compiler bug to me(My gut is saying it's somewhere deep inside of LLVM), which is way more concerning than just being a bevy bug, this is also why I rarely optimize at level 3(unless it's been well tested against level 2 compiled bins), I think what should probably be done is that the docs should be updated to suggest a more conservative optimization level of either 2 or z with the suggested optimized debug builds.

Still not fixed for me on Mac (Ventura 13.0.1, 2019). I get segfaults with the --release flag and
[profile.dev.package."*"]
opt-level = 3

But this example runs fine without the --release flag and without opt-level = 3:

Does this also happen with just a minimal App::new().run();? Does it happen with non-bevy projects compiled with such a configuration to optimize dev deps?

Does this also happen with just a minimal App::new().run();? Does it happen with non-bevy projects compiled with such a configuration to optimize dev deps? What CPU architecture? One user noted it's only with optimizations but disabled RenderPlugin, can you confirm it only happens with O3 and RenderPlugin?

Thank you very much for you reply. You're right. It's definitely the render plugin. I must have overlooked that. I get everything compiled: a simple "Hello World", your minimal example and my example above - without the render plugin. With the RenderPlugin enabled I get a seg fault.
My specs:
MacOS Ventura <IP_ADDRESS> inch, 2019
CPU: 2.4GHz 8-core Intel Core i9
GPU: AMD Radeon Pro 5600M 8GB
Memory: 32GB 2667MHz DDR4

Seems fixed 🤷♀ I no longer get this error with Rust 1.67.0. Can anyone reproduce it?

fwiw, when a shader is instantiated, Mac compiles it asynchronously in the background using MTLCompilerService - and I've found out that this shader-compiler is... pretty flake'y - it can easily segfault even on simple shaders.
To narrow down the issue, if it happens again, one could open Console.app and look for logs from that service - on my machine (M1 Pro) it likes to crash with something akin to generated non-G1 instruction for G1.

Yeah, I just tried running the breakout bevy example, and that worked without modification. My test project works now too. I'm not sure what fixed it, but I think we can close this


Linux packaged browser failing to launch.

Steps to reproduce:

Clone Safe_browser repo after PR #637
Follow steps https://github.com/maidsafe/safe_browser#compiling to package the browser locally.
Navigate to the release folder and try to launch browser with ./safe-browser

Expected result:
Browser will launch OK
Actual result:
Browser fails to launch, error message is returned: Segmentation fault (core dumped)

Environments:
This has been replicated in Ubuntu 18.04 VM and Linux Mint 19 Cinnamon native machine.

@calumcraig I'll continue to try to reproduce. Until then, I want to look at that core dump.
Do you have a core dump file located at /usr/share/apport/apport?
If not, cat /proc/sys/kernel/core_pattern should give you the location of the dumped memory core.
If there are none, then apport may need to be started first.
systemctl start apport

@calumcraig Do you also receive a segfault with this package? https://github.com/hunterlester/safe_browser/releases/tag/libgconf

@calumcraig [UPDATE]: Scratch that. I'm able to reproduce.

@hunterlester I can see you've been working on this (for me) overnight - I take it you don't need me to do anything further? If you do just let me know.

Can confirm that the roll back of PR #637, PR #641 has resolved this issue. SAFE Browser launches as expected on Linux Mint 19.
Tested on Ubuntu 18.04 and behaviour has returned to the previous - #627 is still seen. Workaround (as described in browser README) resolves the issue: sudo apt-get install libgconf-2-4.


added rxref lookup

The new rxref lookup will search for stack names relative to the current namespace identified in the environment file.
This encourages reuse of stacks in multiple environments within the same AWS account.
Thanks for building some great software!

Hey @discobean - sorry for the delay at looking at this, it got lost in the mix up of work & vacation!  I'll try to get this looked at more today!  Thanks for the contribution!

I've updated adding those tests, let me know how it goes.

Just pushed that updated now.  Thanks!

Really keen on this feature. How long to go until its merged??

🎉  thanks @phobologic

This is just what I was looking for! Thanks for this patch.


When a font name with a space in the style attribute is bind

Version
2.6.14
Reproduction link
https://codepen.io/reefle/pen/RwVBjxa
Steps to reproduce

What is expected?
Font names with spaces should be enclosed in single quotes. Otherwise, the web browser will fail to parse the HTML.

What is actually happening?
Google Chrome 92.0.4515.107（Official Build）

vue is a great project :-)

This is working as expected, you are passing the string sample font, not 'sample font', so Vue will set that. This is similar to not passing the unit to a number


refactor: renaming renaming renaming renaming renaming renaming renaming renaming renaming renaming renaming

Fixes #25

Nice this will be merged btw can u look into the singleplayer issue? its so weird i legitametly cant figure it out

btw can u look into the singleplayer issue? its so weird i legitametly cant figure it out

Currently I can't even get it to compile as I didn't give it enough memory, I've just gave it 1 GB so hopefully it will work now

It compiled!
Also, I set it to 1 GB and it works.


Generalize Energy Message Documentation for Broader Market Use

What's needed?
The Energy message is being used everywhere in our APIs and is mostly related to trading but in various markets like ancillary service markets alongside spot markets. We need to make this more general.
Summary of Changes:
Generalized the Documentation:

Removed specific references to "trading orders" to make the Energy message applicable across various markets, including ancillary service and spot markets.
Clarified the Note: Updated the note to emphasize the use of MWh as a consistent unit of energy across various applications, not limited to trading.
Updated Message Description: Modified the description to represent a unit of energy in a more general context.

This change ensures that the Energy message documentation is universally applicable across all areas where it's used in our APIs.
Proposed solution
// Frequenz definitions of energy.
//
// Copyright 2023 Frequenz Energy-as-a-Service GmbH
//
// Licensed under the MIT License (the "License");
// you may not use this file except in compliance with the License.

syntax = "proto3";

package frequenz.api.common.v1.market;

import "frequenz/api/common/v1/types/decimal.proto";

// Represents a unit of energy.
//
// !!! note
//     The energy unit is denominated in MWh (Megawatt-hours), which is a unit of
//     energy representing total output over a period. This differs from MW
//     (Megawatts), a unit of power that represents the rate of energy production
//     or consumption. Using MWh allows for consistent measurement across various
//     markets and applications.
message Energy {
  // Energy amount in Megawatt-hours (MWh).
  frequenz.api.common.v1.types.Decimal mwh = 1;
}

Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

Here is also a diff:
--- Original Protobuf File
+++ Updated Protobuf File
@@ -1,6 +1,6 @@
-// Frequenz definitions of energy for electricity trading.
+// Frequenz definitions of energy.
 //
 // Copyright 2023 Frequenz Energy-as-a-Service GmbH
 //
 // Licensed under the MIT License (the "License");
 // you may not use this file except in compliance with the License.
@@ -10,11 +10,12 @@
 package frequenz.api.common.v1.market;

 import "frequenz/api/common/v1/types/decimal.proto";

-// Represents a single unit of electricity.
+// Represents a unit of energy.
 //
 // !!! note
-//     In these trading orders, the unit of energy is denominated in MWh
-//     (Megawatt-hours) as opposed to MW (Megawatts).  MWh is a unit of energy
-//     representing total output over a period, while MW is a unit of power that
-//     represents the rate of energy production or consumption.
+//     The energy unit is denominated in MWh (Megawatt-hours), which is a unit of
+//     energy representing total output over a period. This differs from MW
+//     (Megawatts), a unit of power that represents the rate of energy production
+//     or consumption. Using MWh allows for consistent measurement across various
+//     markets and applications.
 message Energy {
-  // Represents energy unit in Megawatthours (MWh).
+  // Energy amount in Megawatt-hours (MWh).
   frequenz.api.common.v1.types.Decimal mwh = 1;
 }

Done in #267


Can't keep a file on editor when it is deleted

From<EMAIL_ADDRESS>on 2011-01-25T14:49:03Z
What steps will reproduce the problem?

open a file with spyder
delete file (spyder windows has lost focus)
Return focus to spyder
A message pop-ups asking you if you want to keep the file on editor (or something like that).
Click yes What is the expected output? What do you see instead? Nothing happens. The message just pop ups again and again. Pressing on no dismisses the message (and the file). What version of the product are you using? On what operating system? * Spyder 2.0.6

pyflakes 0.4.0
rope 0.9.3
Python 2.7.1
Qt 4.7.1
PyQt 4.8.2
Archlinux x64

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=536

From pierre.raybaut on 2011-05-16T01:26:47Z
Labels: Cat-Editor

From pierre.raybaut on 2011-08-02T03:18:06Z
This is fixed on my local repository: I'll push changesets as soon as I can.
Status: Started

From pierre.raybaut on 2011-08-08T00:56:42Z
This issue was closed by revision be1f3404fdc4 .
Status: Fixed


Join Meeting as ApplicationInstance fails with 400

Describe the issue
We are trying to join an existing group call as an ApplicationInstance but the call cannot be established.
Code Snippet
{
    "callbackUri": "{{callbackUri}}",
    "source": {
        "identity": {
<EMAIL_ADDRESS>"microsoft.graph.identitySet",
            "applicationInstance": {
                "id": "{{applicationInstanceId}}",
<EMAIL_ADDRESS>"microsoft.graph.identity",
                "tenantId": "{{tenantId}}"
            },
            "tenantId": "{{tenantId}}"
        },
        "tenantId": "{{tenantId}}",
<EMAIL_ADDRESS>"microsoft.graph.participantInfo"
    },
    "requestedModalities": [
        "audio"
    ],
    "mediaConfig": {
<EMAIL_ADDRESS>"microsoft.graph.serviceHostedMediaConfig"
    },
    "meetingInfo": {
        "token": "ey...",
<EMAIL_ADDRESS>"microsoft.graph.tokenMeetingInfo"
    },
    "tenantId": "{{tenantId}}",
<EMAIL_ADDRESS>"microsoft.graph.call"
}

Call fails with this response:
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",
  "value": [
    {
<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",
      "changeType": "deleted",
      "resource": "/app/calls/e81f5d00-971f-46c0-9075-7a5d83bd5500",
      "resourceUrl": "/communications/calls/e81f5d00-971f-46c0-9075-7a5d83bd5500",
      "resourceData": {
<EMAIL_ADDRESS>"#microsoft.graph.call",
        "state": "terminated",
        "resultInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.resultInfo",
          "code": 400,
          "subcode": 9999,
          "message": "Enterprise Caller has no TenantId.. DiagCode: 400#10203"
        },
        "meetingInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.tokenMeetingInfo",
          "token": "eyJh..."
        },
        "callChainId": "652489f9-7e1f-4c13-a449-c19053e4bd01"
      }
    }
  ]
}

Expected behavior
I expect to be able to join the meeting, and see the ApplicationInstance as a new participant in the call.
Call ID
CallId: e81f5d00-971f-46c0-9075-7a5d83bd5500
CallChainId: 652489f9-7e1f-4c13-a449-c19053e4bd01
Time: 2021-03-01 09:45:51 UTC
Additional context
I can join the call if I don't set any source or if I set "guest" as a source. Also I tried to add tenantId basically anywhere but I could not get it to work.

Hi, did you manage to make it work?


Porting onedark theme

Please let me know if there is anything else that needs to be done here. Used onedark's airline theme in it's repo as reference.

Are you sure you have not mistaken fg with bg? This looks weird...

Are you sure you have not mistaken fg with bg? This looks weird...

Also you have not set M.command as said in https://github.com/hoob3rt/lualine.nvim/issues/9
Please set it to something, lualine's highlight should fall back to lualine_normal but it is currently not supported. Even
M.command = M.normal

will do fine as long as the highlights make sense with ported colorscheme.

Also you have not set M.command as said in https://github.com/hoob3rt/lualine.nvim/issues/9
Please set it to something, lualine's highlight should fall back to lualine_normal but it is currently not supported. Even
M.command = M.normal

will do fine as long as the highlights make sense with ported colorscheme.

@hoob3rt Thanks for the review! I addressed both comments with another commit, please let me know if there is anything else I should address.

@hoob3rt Thanks for the review! I addressed both comments with another commit, please let me know if there is anything else I should address.

@teddylear looks good now. Please add onedark to README.md and it should be ready to merge.

@teddylear looks good now. Please add onedark to README.md and it should be ready to merge.

@hoob3rt Added theme to list on README.

@hoob3rt Added theme to list on README.


[Feat] Change the language

Target Use Case
Easy to change language
Proposal
Makes it possible to change the language without having to reload the page.

I'd love to be able to do this, but it's impossible at the moment. The best you can do is to not set the language, in which case the maps API will use the locale settings of the user-agent.
Background: For a variety of reasons, the language has to be set when loading the Maps JavaScript API, which is handled by our APIProvider. I initially had an implementation that allowed unloading and reloading the entire Maps API, but that doesn't work anymore since the API also registers web-components that can't be unregistered or re-registered, so if you load the API a second time, it will just fail.
There is an open issue in the Google Maps Platform Issue Tracker, but that has been around for a while, so I wouldn't hold my breath. Still, adding a +1 and maybe a comment on that won't hurt.


Add skip-manim tag when building the docs

List of Changes

This PR allows us to run make html O=-tskip-manim in order to build the documentation without rendering manim code examples.

Motivation
Sometimes when I build the docs, all manim scenes are re-rendered, even if the files they come from have not been modified.
Explanation for Changes
This PR makes it so that running sphinx with the -t skip-manim tag will signal to the manim directive to not render anything. This tag can be used in one of two ways: sphinx-build -M html <source_dir> <dest_dir> -t skip-manim OR by using the makefile: make html O=-tskip-manim. Unfortunately, ugly as it is, this is the only way to pass the -t tag to the underlying sphinx process.
Testing Status
Local pass.
Further Comments
Right now, the output looks ugly. If anyone has any ideas for how to make it prettier, then I'm all ears. However I'm not too concerned with it since the point of the -t skip-manim flag is to be used only for quick-and-dirty things. No need for pretty.

This is a really good and convenient idea!
I like this functionality and think it can be quite useful.
A question about sphinx-build -M html <source_dir> <dest_dir> -t skip-manim :
Does this mean, that source_dir is only one .rst file, like e.g. examples.rst and the rest is ignored and not rendered?
That would be also nice if one works at one specific .rst file to enhance the docs.

This is a really good and convenient idea!
I like this functionality and think it can be quite useful.
A question about sphinx-build -M html <source_dir> <dest_dir> -t skip-manim :
Does this mean, that source_dir is only one .rst file, like e.g. examples.rst and the rest is ignored and not rendered?
That would be also nice if one works at one specific .rst file to enhance the docs.

It might also be possible to make sphinx not rebuild .rst files that did not change. But i don't know if it is really the spirit of sphinx.
Also, it seems weird that you can specify not to build manim videos. Is it possible to make a hash of the code to render and then use something like the scene caching, by adding a file keeping track of generated scenes.
I hope you understand what i'm thinking of ^^

This is a really good and convenient idea!
I like this functionality and think it can be quite useful.
A question about sphinx-build -M html <source_dir> <dest_dir> -t skip-manim :
Does this mean, that source_dir is only one .rst file, like e.g. examples.rst and the rest is ignored and not rendered?
That would be also nice if one works at one specific .rst file to enhance the docs.

Unfortunately, I don't know. In almost all cases, the actual command is sphinx-build -M html source/ build/ -t skip-manim (specifying the whole of the source directory).
I agree it would be nice tho, but I don't know how to do it.

Also, it seems weird that you can specify not to build manim videos. Is it possible to make a hash of the code to render and then use something like the scene caching, by adding a file keeping track of generated scenes.
I hope you understand what i'm thinking of ^^

I agree it's messy, but remember this is just a debugging tool, so I personally have no problem with my debugging being messy :wink:. I agree what you suggest would be great, but perhaps we can leave it for a future PR? This was meant as a quick patch for now.

This seems good (and very useful!) to me, I'll just try it out locally before approving this PR. Thanks for your efforts!


Able to pipe query into stdin and get csv on stdout

We use commandline tools to transfer data from postgresql, mysql and sqlserver to (in our case) a postgresql server.
The basic idea is like this (in this case we get data from mysql and import it in postgresql:
# schema.table was created to have the same layout as whatever is getting out of the csv
echo "SELECT ... FROM ...; " | /bin/bash -o pipefail -c "\
MYSQL_PWD=password mysql --user=projecta --host=x.x.x.x --port=xxxx   --skip-column-names | \
PGOPTIONS='--client-min-messages=warning' PGTZ=Europe/Berlin \
psql --username=xxx --host=hostname --no-psqlrc --set ON_ERROR_STOP=on database --echo-queries \
--command=\"COPY schema.table FROM STDIN WITH NULL AS 'NULL'\"";

[we also run all our ETL via sending queries to stdin of a psql call, but to support that in mssql-cli would also mean "import csv on stdin" support to actually get the data into sql server]
For sqlserver, we currently use sqsh  but supports at least the "SQL Script in, csv out"  workflow. As sqsh is unmaintained (and buggy in versions available in ubuntu/debian), we would like to replace this by whatever finally supports at least exporting csv on stdout.

i like to see this as well

cat test_query.sql | { /opt/mssql-tools/bin/sqlcmd -S <server IP> -U <DB user> -P <DB Password> -h-1 -s"," -W -i <(cat -); } | head -n -2

@MrMeemus This is a much needed feature. I am happy to involve in any testing/bug fixing.  Thanks!

I just wanted to see how the work on this was going.  To be able to pass a SQL query through stdin and get results from stdout.  No pressure :-)  Thanks!

A workaround using odbc drivers: https://github.com/kfzteile24/sqpy

Just wanted to see how the progress was going for this enhancement.  Thanks in advance!

Is this still being worked on? It would also be really useful for vim integration for me.

@apisarenco have you found a way to make your dirty method include the column headers too?


Document executable name option in WinDbgX cmd line

WinDbgX.exe [executable]: launch the executable and debug it.
The description is copied from https://docs.microsoft.com/en-us/windows-hardware/drivers/debugger/windbg-command-line-options

@aluhrs13 - Do we support passing an executable name to the WindbgX.exe as we do with the classic debugger?

Yes it is supported.

@aluhrs13 - Thanks!
@tedhudek - This PR is approved and ready to go to live/master.


Today button (.dp-current-location-btn) show the wrong date

Make sure to add all the information needed to understand the bug so that someone can help. If the info is missing we'll add the 'Needs more information' label and close the issue until there is enough information.

[ ] Provide a minimal code snippet, please make sure it is well formatted.
No code required.
[ ] Provide a stackblitz demo / stackblitz example that reproduces the bug.
the current Official Demo has this problem as well
[ ] Provide screenshots where appropriate

[ ] What's the version of Angular you're using?
Whatever demo version is using,
[ ] Does this occur on specific browser?
Tested in Chrome, Edge and Firefox

Thank you,
its fixed now the demo


Allow instance methods to be accessed from the component directly

What is the use case that is requiring this?  Do you have a small gist snippet of the corresponding code that uses these newly exposed instance methods that make them useful?

This would allow the router to be used within a persistent custom container. This is mainly useful if you want globally accessible navigational components.
For instance, you might want a bottom navigation bar that could be clicked at any time to route to certain component.
class Navigation extends Component {
  settingsRoute() {
    return {
      name: "Settings",
      component: Settings,
    };
  }

  toSettings() {
    this.navComponent.instanceMethods.resetToRoute(this.settingsRoute());
  }

  render() {
    return (
      <View>
        <Router
          firstRoute={this.searchRoute()}
          ref={(c) => this.navComponent = c}
        />
        <TouchableHighlight onPress={this.toSettings} >
          <Text> Settings </Text>
        </TouchableHighlight>
      </View>
    );
  }
}

It's a bit simplistic, but the "Settings" tab will not disappear when the route changes, and can even change the route itself! This could be used to allow all sorts of more complex navigational structures.

I have updated this PR to add the methods to the navigation component itself, rather than within the instanceMethods layer.

merged for now.  I think we can do a larger refactor, but this is good for now and to not hold up the other project


http-01-port support

Hello!
Its possible to support http-01-port protocol to allow non-80 port to be checked?
LetsEncrypt official client support change http port via this parameter (Documentend in http://letsencrypt.readthedocs.io/en/latest/using.html )
Thanks!

Port options are listed under "The following flags are meant for testing purposes only!", I think this only works with specially configured boulder instances and not with anything official. If somebody needs this for testing purposes i'd accept a pull request.


If ... Or ...

Is there any capability to do an If... Or... such as:
Hello {{#if {{Man}} || {{Boy}}Mr.{{#else}}Mrs.{{/if}} Smith!

If there isn't a lot of text in a branch, duplicate the text in an #elseif.
Else, create a IsMale property and use that. That's the best you can do
right now.
On Sep 27, 2016 3:01 PM, "c5racing"<EMAIL_ADDRESS>wrote:

Is there any capability to do an If... Or... such as:
Hello {{#if {{Man}} || {{Boy}}Mr.{{#else}}Mrs.{{/if}} Smith!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jehugaleahsa/mustache-sharp/issues/69, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABTgPuGevdAzlw8P_qY_uvE_SmK7vo0dks5quWgEgaJpZM4KICOB
.

That's what I assumed.  i have substantial duplicated text  The IsMan and IsBoy was just an example so the IsMale suggestion won't work in my case.
Thanks!


[Android] No Pairing Dialog shown in Xamarin Forms

Hello,
first of all sorry for my bad english.
I hope i can explain the Problem as detailed as much without many informations.
We use this plugin to communicate with a custom bluetooth le device. After we try to establish a connection with the device we expect to get a dialog to pair the devices but do not get one.
Our mainproject is in Xamarin.Forms 4.3, we also tested it with the given testproject in Xamarin.Forms 4.2 and get the same behavior.
Maybe there is a know issue or some tips to get more information out of the connection process.
Steps to reproduce

Download Test-Project (Xamarin Forms)

Start on Android-Device

Expected behavior
After the connection is established a Pairing Dialog should Pop up.
You should then insert ther pairing code from the other device and accept.
Actual behavior
The connection is successfully established but no pairing dialog was shown.
Crashlog
Atm no Error or Crash log
Configuration
Version of the Plugin: Testproject from actual Master Branch
Platform: Android 10, Android 9 (Xamarin Forms 4.2)
Device: Pixel 3 XL, OnePlus 5T

Hi @hendriksandberg,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

Hi,
we managed to get to pairing dialog.
We had to choose a specific characteristic out of the connected device and then starts listening on a service.
After a few moments the pairing dialog will popup if the devices isnt paired already.


Improve error handling on manifest missing required columns/wrong schema errors

Open a new issue to improve error message when uploaded manifest does not have all required columns and 'wrong schema' errors are given from backend.

When we try validating manifests that were submitted using a previous version of a schema we get the "wrong schema" error in addition to a message saying "An error has occurred. Check your logs or contact the app author for clarification." This isn't a situation that most people would encounter once the schemas are finalized (we mostly encounter it since we and others have been testing with schemas that were still in flux), but I agree we should have better error handling.It's fine to indicate that the schema used by a manifest is not the right one (i.e. it's wrong) given the selected schema in step 1. But we probably shouldn't be getting "Check your logs..." error. The message "At row 2 value Wrong schema in Wrong schema <> is a required property", can be changed (which is probably the intent of this issue :). If the backend returns "wrong schema", we shouldn't try to interpret at which row and which value the error occurred. Perhaps an informative message would be "The submitted manifest doesn't contain required columns: <<column1, column2, column2>>. Please check that you have selected the correct template type and  ensure your template contains the required columns"; the header row in the preview table can be highlighted with the bright yellow we already use for validation errors.Once we start versioning schemas and including the version information with the metadata we can have more informative messages, but for now this could be ok? Originally posted by @milen-sage in https://github.com/Sage-Bionetworks/data_curator/issues/61#issuecomment-656207143

Now, the error indicates it is wrong schema relevant errors because of missing required columns.
Create a new ticket if we consider to be more specific/descriptive on wrong schema errors, e.g. outdated schema vs wrong column names vs miss required columns.


No warning and doesn't work.

Hello, I installed the plugin on zotero and did the operations specified from the notion. But no content is transferred to notion. There is no message in the form of a warning. Another situation I would like to point out is about the database-id, which consists of 36 characters. When I copied the invite link over Notion, the databse id in the part you specified consists of 32 characters. I tried again and again by watching the video below to establish the connection, but it didn't work.
https://www.youtube.com/watch?v=8IC8-mPbjCo&t=2s&ab_channel=HollyJane

I also have the same issue that nothing is being transferred from Zotero to Notion and the database ID contains 32 characters only.

@SedVi @sanhe374, thanks for reaching out!
You bring up a good point about the database IDs. The IDs are UUIDs, so their "full" format is 36 characters of the form xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx. Notion usually omits the hyphens (as you can see from the link you copied), so you end up with 32 characters. According to their documentation, both the hyphenated and un-hyphenated IDs shoud work. I'll update the README to make this more clear.
And thanks for sharing that video! I had no idea it existed! I noticed someone in the comments seemed to be running into a similar issue, and I wonder if you're encountering the same thing. It sounded like they configured everything but nothing happened, and I believe that's because the sync will only occur when items are added to or modified within a synced collection as described in the How Notero Works section of the README.
The solution for them was to create a new collection in Zotero, configure the Notero preferences to sync it, and then move items into that collection. Another way to solve this is by adding a temporary tag to items as described in the README.
Please let me know if one of those approaches gets things working for you!

Thank you very much for your return. I tried the last method you mentioned and it worked. Now I can export data to notion. This made my work very fast :)

I found my two mistakes that caused me the same trouble - no warning and no sync.

I went to Zotero's main preferences and had no idea what 'enter the required preferences.' meant.  Was clearly my mistake since you said Tools - Notero Preferences, but I just skipped right over that and when to Zotero - Preferences.
I didn't have a collection yet in Zotero. I'm new to the tool. I had a dozen or so sources just in 'My LIbrary'. I had to create a collection, and I could then select a collection to sync in Notero Preferences.

Still isn't working yet, but I'm getting an error now instead of nothing. :)


Add and update the sound docs

soundLevel added to input.md
added onSoundLevelChanged

@ganicke,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Fix expand-availability syntax not to interfere with doc comment parsing

This was our previous convention:
/// Fiddle with the thing.
/*System 0.0.2*/
public func foo()

/// Fiddle with the thing.
/*System 0.0.2, @available(macOS 12.0, iOS 15.0, watchOS 8.0, tvOS 15.0, *)*/
public func foo()

/// Fiddle with the thing.
/*System 0.0.2*/@available(macOS 12.0, iOS 15.0, watchOS 8.0, tvOS 15.0, *)
public func foo()

Unfortunately, regular comments in between a declaration and its preceding doc comments interfere with documentation tools.
Change things to the style below instead:
/// Fiddle with the thing.
@available(/*System 0.0.2*/macOS 10, *)
public func foo()

/// Fiddle with the thing.
@available(/*SwiftSystem 0.0.2: macOS 12.0, iOS 15.0, watchOS 8.0, tvOS 15.0*/macOS 10, *)
public func foo()

/// Fiddle with the thing.
@available(/*SwiftSystem 0.0.2*/macOS 12.0, iOS 15.0, watchOS 8.0, tvOS 15.0, *)
public func foo()

Unfortunately, @available(*) isn’t valid syntax, so we have to add a dummy “macOS 10” version spec. It should have no actual effect.

@swift-ci test

This spelling looks like it shouldn't interfere with DocC parsing the declaration and comments.

I updated these commits to change macOS 10 to iOS 8 -- the latter is the shortest platform version string that matches the minimum possible deployment target for Swift code. (Using a real version number avoids bogus availability showing up in, say, auto-generated API synopses.)
@available(iOS 8, *) is functionally equivalent to not having an @available attribute at all.
An alternative would be to put in all platforms, but I feel @available(macOS 10.9, iOS 8, watchOS 2, tvOS 9, *) would be way too verbose / confusing to consider.

@swift-ci test

@swift-ci test


Layers

Issue opened to keep track of the progress of implementing layers

[x] Add Layer Class
[x] Add LayerStack class

[x] PushLayer
[x] PopLayer
[x] PushOverlay
[x] PopOverlay

[x] Add LayerStack to application
[x] Add Sample

Implemented in #32


Adding Insolation to param set

Purpose
In anticipation of merging the canopy PR, this adds the Insolation params to the param set & adds the Insolation package to the project tomls.
See Issue #152
To-do
Content
Adds Insolation to param set
Adds Insolation to Project tomls
Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

[X] I have read and checked the items on the review checklist.

bors r+

bors r+

bors r+


Navbar link wrong when directory name occurs more than once

I have a directory structure in which names are sometimes used a few times in different places:

Now the breadcrumb links go to:
Home: URL/gallery/%2F
Maarten: URL/gallery/Maarten
2000: URL/gallery/Maarten%2F2000
2000-12-07 Diversen: URL/gallery/Maarten%2F2000%2F2000-12-07%20Diversen
Maarten: URL/gallery/Maarten
Maarten: (not a link)
So the secont 'Maarten' link goes to /gallery/Maarten while I would expect something like /gallery/Maarten%2F2000%2F2000-12-07%20Diversen%2FMaarten
I'm using 1.9.6 docker-nightly

Thanks, the latest push should fix this.


Task on PostgreSQL native client hangs

Description
I have a fairly simple script (it looks dumb but it's just for the purpose of the bug).
`
SELECT 'DEBUT',clock_timestamp();
CREATE
TEMPORARY TABLE test1 AS
SELECT *
FROM tmpJOOOOLANCact;
select * from test1;
`
The CREATE TABLE works but the session hangs on SELECT *. Here's a few observations :

Table test1 has only 30 rows.
If I replace "select * from test1;" by "select * from test1 LIMIT 10;", it works
If I replace "select * from test1;" by "select * from test1 LIMIT 20;", it just hangs as with no limit
If I execute manually the script using psql.exe on a command line, no problem occurs.

Thanks for your help.
Regards.
DBeaver Version
Community Edition <IP_ADDRESS>210092258
Operating System
Windows 10 Enterprise
Database and driver
PostgreSQL 14.2 native client
PostgreSQL 12.5 Server
Steps to reproduce
1 Create task with script above
2 Execute task (=> task hangs)
Additional context
No response

Hi @ladeche,
Please describe how the session hangs. DBeaver stops working, so you can't use UI in any way (completely frozen, unresponsive UI), or is something loading endlessly, but you can click some buttons?

Hi @iraidaa,
Dbeaver UI works fine, I can do whatever I want.
Maybe my case is unclear. I have created a task of "PostgreSQL script" type which is base on PostgreSQL native client. When I launch the task it hangs after the CREATE TABLE. For some reason the last SELECT is not launched. So the task hangs and the session is still opened and idle on the database (I can see it with pg_top). IF I add "LIMIT 10" to my select it works. If I replace "LIMIT 10" by "LIMIT 20", again the task hangs. Very weird.
Regards.

Hi @ladeche,
Could you please try to generate a thread dump and attach it here? Here is the instruction: https://dbeaver.com/docs/wiki/Making-a-thread-dump/
Thank you.

thread-dump.txt
Hi @arhayka,
I started the task (test_tache_pp01_prob) which then was hanging then I generated the thread dump.
psql.exe which was launched by the task is hanging.
While the task is running (ie hanging) I have access to DBeaver UI without any problems.
If I kill manually psql, the task ends with an error.
Thanks and regards.
Laurent.

The issue remains in 23.0.0 with PostgreSQL 12  and PostgreSQL 14.2 native client.
The script added to the task contains only the following query
CREATE
TEMPORARY TABLE test1 AS
SELECT *
FROM actor;
select * from test1;

Table test1 has 203 rows.
Information regarding the task creation

The task hangs while the UI is still fully responsive, and it's possible to click Cancel Operation, which ends with the error

The same task with the limit 10 (or limit 20 or limit 40)  works without any hanging
CREATE
TEMPORARY TABLE test1 AS
SELECT *
FROM actor;
select * from test1
limit 10;

However, the same endless Executing with the limit 100.


exported methods not visible on unexported struct

see; https://godoc.org/github.com/splace/os/xattribs
surely when you don’t export a struct, to force it to be only made in a controlled way, through a constructor, you still do want to see documentation on its exported methods?

This is how godoc (the command) operates, gddo mimics that behavior.
Have you considered either returning an exported type, or an interface that the unexported type implements?

probably should do that now, the constructor has become trivial due to other changes. Just didn't seem like that odd a pattern.
has come up a few times; https://github.com/golang/go/issues/2273


Client must inspect the status field in sechub report and react

Situation
Currently the client only inspects findings and breaks on high findings. Technical errors are only recognized when they have an "artifical" high or critical finding.
Wanted
After #976 the client must check for a status field as well
Solution
When status field is not SUCCESS but FAILED, the client must produce a dedicated error that the job execution failed and return an exit code != 0

With #976 , the getStatus call will return additional messages/job status.
So without needing to download a report, the result and additional messages about the job can be obtained and should be reported by the SecHub client.
Waiting for #976 to be finished before implementing this task.


Can't change pose after changing pose to "sitting"

When i execute /npc attribute my_npc pose sitting, my npc's pose changes to sitting
But when I execute /npc attribute my_npc pose standing, my npc's pose is still sitting
Also when I execute /npc attribute my_npc pose crouching after /npc attribute my_npc pose sitting, my npc's pose is very strange

I think you should add /npc attribute my_npc sitting <true/false> instead of /npc attribute my_npc pose sitting

Which version of FancyNpcs do you use?

2.0.7

Which server software and version do you use? I just tested with paper and folia 1.20.4 and it works fine

No such problem after updating to latest dev build


Questions

Since I am new to the project, I'd like to understand more about the choices made.
The following points emerged while digging the code and debugging the execution.
Didn't go further than this so far.
Would be nice to discuss on these for me to better understand 😇
bin-stub/phparkitect
foreach (array(__DIR__ . '/../../autoload.php', __DIR__ . '/../vendor/autoload.php', __DIR__ . '/vendor/autoload.php') as $file) {
    if (file_exists($file)) {
        define('PHPARKITECT_COMPOSER_INSTALL', $file);

        break;
    }
}

unset($file);

if (!defined('PHPARKITECT_COMPOSER_INSTALL')) {
    fwrite(
        STDERR,
        'You need to set up the project dependencies using Composer:' . PHP_EOL . PHP_EOL .
        '    composer install' . PHP_EOL . PHP_EOL .
        'You can learn all about Composer on https://getcomposer.org/.' . PHP_EOL
    );

    die(1);
}

Why are we looking in different locations for an autoloader?
I get the second part about checking for dependencies being installed 🙂

Arkitect\Rules\RuleChecker::$rules seems to not being used on instanciated object.
in Arkitect\Rules\RuleChecker::build(ClassSet $classSet, ArchRule ...$rules) $rules are passed to the $fileParser in order to be evaluated when a class has been analyzed by php-parser, but never really used in Arkitect\Rules\RuleChecker instance.
Ise there a plan to use that property internally in RuleChecker?

\Arkitect\Analyzer\FileVisitor::enterNode:(26,27) seems to be adding the same dependency twice via the ClassDescriptionBuilder
foreach ($node->implements as $interface) {
    $this->classDescriptionBuilder
         ->addInterface($interface->toString(), $interface->getLine())
         ->addDependency(new ClassDependency($interface->toCodeString(), $interface->getLine()));
}

See ClassDescriptionBuilder::addInterface and ClassDescriptionBuilder::addDependency implementations
class ClassDescriptionBuilder
{
    ...
    public function addInterface(string $FQCN, int $line): self
    {
        $this->addDependency(new ClassDependency($FQCN, $line));
        $this->interfaces[] = FullyQualifiedClassName::fromString($FQCN);

        return $this;
    }

    public function addDependency(ClassDependency $cd): self
    {
        $this->classDependencies[] = $cd;

        return $this;
    }
    ...
}

Is this intended?

Context:
config tests/E2E/fixtures/configMvc.php
class being analyzed App\Controller\CatalogController
In ClassDescriptionBuilder::get the property $this->filePath is empty
public function get(): ClassDescription
{
    $cd = new ClassDescription($this->FQCN, $this->classDependencies, $this->interfaces);
    $cd->setFullPath($this->filePath);

    return $cd;
}

And then it gets set like this $classDescription->setFullPath($currentlyAnalyzedFile->toString()); in RuleChecker:51.
In this case the value gets overridden by Controller.
Is Controller a fullPath?

interface Expression
{
    /**
     * Returns a human readable description of the expression
     * $theClass can be used to add contextual information.
     */
    public function describe(ClassDescription $theClass): Description;

    /**
     * Evaluates the expression for the class passed as parameter.
     * It should adds violations if rule is violated.
     */
    public function evaluate(ClassDescription $theClass, Violations $violations): void;
}

Not completely sure about the interface of the evaluate method which is void.
This means a side effect in the method that modifies the violations.
Would an interface like public function evaluate(ClassDescription $theClass): Violations; be as effective and safer from side effect point of view?
Similar thoughts for
interface ArchRule
{
    public function check(ClassDescription $classDescription, Violations $violations): void;
}

@nelsonkopliku can we close this issue or do you need more answers or discussion about something?

You can close this issue.
The answers should be enough for now. Thanks!


add nix installation option

I've added hoard to the nixpkgs repository so nix users can easily install it!

Awesome! Thanks a lot for your contribution @Builditluc :+1:


Create Homebrew tap for automated installation and upgrading

Discussed in https://github.com/lujjjh/LinearMouse/discussions/34

Originally posted by scmeek December 16, 2021
Homebrew on macOS makes user management of applications like this so easy, including automated upgrading. It's also a godsend when managing "workflows" on different computers. It would be hugely beneficial to add this to Homebrew.

LinearMouse can now be installed via Homebrew:
$ brew install --cask linearmouse --no-quarantine


Add a feature flag to omit ASM dependencies

Compiling the ASM dependencies has turned out to be a major pain point.
This commit adds a feature flag to optionally disable support for all
algorithms that require ASM.

Greate change! Without it I can't build with stable-x86_64-pc-windows-msvc toolchain.


Food stays on the map upon gameover

Fixed by nulling matrix columns and filling rows with 1s upon gameover


ADD: Kimi no na wa ost and full metal panic opening

More songs for this issue #195
This PR is for [Kimi no na wa, Full metal panic].

[ ] This issue is for enhancement/bug fixing.
[x] This issue is an addition to JSON database.

[ ] Pull Request Title is of form Added [Kimi no na wa, Full metal panic].

[ ] This addition is already on ongaku.

[ ] This PR is an improvement to current addition.

[ ] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[ ] This pull request will be closed if I failed to update it even once in a week.

Hey, @SoldierCorp thanks for the PR.
Please check comments

Sure,
So the issue is to use another image for the zen zen song and use a different link for Yumetourou one?

Yea use different images for different tracks.
On Oct 19, 2017 5:50 PM, "Edgardo Ramírez"<EMAIL_ADDRESS>wrote:

Sure,
So the issue is to use another image for the zen zen song and use a
different link for Yumetourou one?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/anshumanv/ongaku/pull/202#issuecomment-337890331, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUCUL_kTurvDPWlHNfFygEF46B3UQ6qIks5stz6GgaJpZM4P_HVe
.

Amazing! 👍

Please add more tracks if you wish. 😄

Absolutely :)


Fix Hetzner CentOS 8 Template

Recently, the Hetzner CentOS 8 based Clusters are not able to launch correctly as the network connection of the protected nodes breaks. This also happens in production, so it is most probably due to a change in the image/distro.

[ ] investigate
[ ] fix

The problem is that after the restart after the roles for the initialization of the protected nodes, someone (NetworkManager?) adds a default route via <IP_ADDRESS> which blocks the protected nodes from accessing the internet.
In CentOS 7, this does not happen...
$ ip route
default via <IP_ADDRESS> dev ens10 proto dhcp metric 100
default via <IP_ADDRESS> dev eth0 proto dhcp metric 101
<IP_ADDRESS>/16 via <IP_ADDRESS> dev ens10 proto dhcp metric 100
<IP_ADDRESS> dev ens10 proto dhcp scope link metric 100
<IP_ADDRESS> dev eth0 proto dhcp scope link metric 101

The 'fix' is to delete the wrong (?) default route after every server start. A further investigation is recommended (see #197)


Fix incorrect representation of BlockIdentifier

CHANGELOG:

Fixed incorrect representation of BlockIdentifier by adding a new function make_optional_params that accounts for incorrectly generated schema for RpcWithOptionalParams
Added tests to ensure correctness of the fix
Closes ticket 1746

bors r+


Clicking on a function should take you to that functions entry point

Clicking on a function always takes you to the top left of the function view window. If functions are particularly "wide", this can mean this view is entirely empty and you need to scroll right to find the functions first basic block.
Clicking on a function window from the list should always take you to a view where you start at the functions first basic block.

It's slightly more complicated than that -- if you were previously looking at a different spot in a function, jumping back to that function takes you to the same spot you were looking at. For example scroll to where you want to look and then double click another function in the side-bar, then back to the first. It should take you back to where you were looking in the first function.
That said, definitely agree that upon first opening a function if there isn't a saved location it should take you somewhere smarter.

Actually, now that I look at it more closely, we should do this already in most instances. I suspect maybe what you're seeing is side-effect of the auto-analysis not completing yet and so our attempts to find a good place to start in the function fail. Ironically, because of the behavior I described above -- even if you navigate to that function again after auto analysis completes (as an aside, navigating to any function automatically moves it to the top of the auto analysis queue), because you had a previous view (off to the side), it saves it.

Makes sense, thanks.

Hmm, actually, when I just tried it (xnu kernel was a good suggestion), the view loaded in the upper left, but when I clicked away to another function and clicked back it did actually work correctly for me. Can you double check and see if maybe you were navigating within the view and then switching away/switching back which might have caused it to save your new view?

Yes, I can replicate this behavior. Load the kernel, find a large function (PE_parse_boot_argn). Open the window, get bank screen. Navigate away, come back and get first basic block this time.
So, I guess the bug is I expect on first click to see the first basic block. Subsequent ones work as expected.

Fixed in dev


CircleCI failing due to SSH key

@conda-forge-admin, please update circle

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I updated the circle-ci deploy key and followed the project.


Re-introduce removed event queues validation

This has a test that would need to pass for the newest_pruned_id logic we had to be correct.
I think probably what we need to do is either (1) have special logic for the webapp to reload after seeing this error like we had before to deal with incorrectly migrated queues or (2) to have a special state for a migrated queue where newest_pruned_event_id is None or something and not compared.
@andersk FYI.

I originally missed this PR before pushing #12938 (which implements (2)), but now I think #12938 might be the only viable approach, so I updated it with a tweaked form of the extra test here.

Closed in favor of #12938.


Make website SSL

It is slightly more complicated on github.io, but Jen Myers has a great write-up here to front it with cloudflare. I believe their shared cert that comes with the free option should work, but would be worth investigating.
https://jenmyers.net/using-https-with-github-pages-and-custom
If the shared cert doesn't work for us, then we could reach out to them (or perhaps even get a free AWS account and use cloudfront) to sponsor coderetreat with a free service that does satisfy our needs.

closing because https://github.com/coderetreat/orga-tasks/issues/1 solves this :+1:


[Words/Family] Exercises 28. the problem is in the word "Новорожденный"

Precondition:
Log in with an existing user. Open: Groups / Speech exercises / Words / Family.
http://<IP_ADDRESS>/groups/2/series/1/subgroup/1/exercise/28/task/28
Steps:
1.Select exercise 28.
2.Repeat the word "Новорожденный"
Actual result: the matrix of words moves out when the word "Новорожденный" is pressed. Buttons go below. the word "Новорожденный" is not spoken
Expected Result: a matrix of words in place. When you click on a word "Новорожденный", it is pronounced
3.Click to solve
Actual result: the word "Новорожденный" is not selected(or selected after repeated pressing on the word). the matrix of words moves out when the word newborn is pressed. Buttons go below.
Expected Result: the word newborn is chosen

corrected


dotnet-publish: Which "publish-related properties" are supported?

Here the docs say:
You can also set publish-related properties by referring to a .pubxml file. For example:
dotnet publish -p:PublishProfile=FolderProfile

I would have assumed for instance that the DeleteExistingFiles property inside .pubxml is a publish-related property but it is ignored by dotnet publish.
Also the LastUsedBuildConfiguration seems to be ignored because dotnet publish still builds and copies the Debug dlls, not the Release dlls although I have specified:
<LastUsedBuildConfiguration>Release</LastUsedBuildConfiguration>

Is LastUsedBuildConfiguration not a publish-related property? I would have guessed that it is.
I think it's not a mistake on my end because the PublishDir setting in .pubxml is honored when I do dotnet publish and the DeleteExistingFiles and LastUsedBuildConfiguration settings are honored when publishing via Visual Studio. Maybe DeleteExistingFiles and LastUsedBuildConfiguration are meant to only be used by Visual Studio but the docs here only mention PublishUrl to be meant to be used by Visual Studio.
I think it would be very helpful to write in the docs which publish-related properties exactly are supported by dotnet publish in order to avoid confusion and maybe also it should be mentioned what the necessary replacements are for unsupported publish-related properties, e.g. --configuration argument has to be used instead of LastUsedBuildConfiguration setting.
Using .NET 7 SDK and tested with a completely new ASP.NET Core Web Application project. I didn't test any other publish-related properties.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 211afdc7-c796-202d-239d-823f2a499dff
Version Independent ID: 9439897e-dcb3-bc2e-5cae-35c1ca2e0681
Content: dotnet publish command - .NET CLI
Content Source: docs/core/tools/dotnet-publish.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@baronfel
Should dotnet publish honor all of the .pubxml properties that Visual Studio publishing honors? Specifically if dotnet publish ignores DeleteExistingFiles and LastUsedBuildConfiguration, is this a bug or just a difference to be documented in the dotnet publish article?

It's really hard to say which properties dotnet publish will ignore because it's entirely determined by the build targets that run as part of the publish operation. It is very likely that, say, SelfContained would be honored in both scenarios, but unfortunately PublishProfiles are also used by VS to store a lot of 'last run' data. Many publish mechanisms used by VS are not runnable from the CLI, so there's no worry about overlap at least.
There's some effort in https://github.com/dotnet/sdk/pull/29817 to bring the CLI more into alignment with VS's behaviors, but LastUsedBuildConfiguration for example may never be used.
There's probably a good documentation split between the 'packaging' aspect of publishing and the 'deployment automation' aspect of publishing - the CLI and VS both do the former, and properties relating to that we should support, but the CLI will probably never do the latter.

@NightProgramming Thanks for the suggestion. I'm adding a paragraph that notes the difference between the VS and the CLI handling of .pubxml properties.


Feat: Add intro for website

Describe the feature
hello, i want to add an intro before the main landpage appears.
i've added this feature in many projects one example is given below.
if you like this feature...kindly assign me the gssoc-ext, hackoctober-fest, level tag to keep learning
thank you
Add ScreenShots
AgroTech AI.webm
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

SwapReads.webm
after cloning it...i didnt see it... so i worked on it and i feel this adds more creativity to this project


not found error in sample api

Hi., Springer and Elsevier (w/o numbers) formats may fail.

This problem is caused by k2pdfopt library cuz of 'Assertion failed' so ask developer of k2pdfopt to this issue please.
URL of k2pdfopt
=>  https://github.com/JohannesBuchner/k2pdfopt


🐛 The EditableComboBox is missing the part where it applies the style to the TextBox.

Describe the bug
The EditableComboBox is missing the part where it applies the style to the TextBox.
Expected behavior
The font size in the TextBox is currently fixed.
Screenshots
line to 167

Fixed by #1121


Generated output missing IDC_ARROW

Actual behavior
Trying to figure out what the current cursor loaded on screen is using GetCursorInfo. I think (though the documentation is unclear) that I can just compare the handle HCURSOR returned with one of the Macro values like IDC_ARROW or IDC_SIZENS as listed here.
However, while I see the generated GetCursorInfo function and HCURSOR structures, I don't see IDC_ARROW being generated.

Expected behavior
Would expect access to generated IDC_ARROW macro to be able to use.
I see that it's used in one of the test projects in this repo as an example, but don't know if that project is run/tested on the CI?
Repro steps

NativeMethods.txt content:

GetClientRect
MapWindowPoints
GetCursorInfo
IDC_ARROW
MAKEINTRESOURCE

Any of your own code that should be shared?

        CURSORINFO curinfo = new()
        {
            cbSize = (uint)Marshal.SizeOf(typeof(CURSORINFO))
        };
        Assert.IsTrue(PInvoke.GetCursorInfo(ref curinfo), "Could not get current mouse cursor.");

        // Not really sure about this part either as can't find docs on this explicit scenario? But first issue is IDC_ARROW not generated.
        Assert.AreEqual(IDC_ARROW, curinfo.hCursor, "Current cursor isn't Arrow cursor.");

Context

CsWin32 version: 0.2.104-beta
Win32Metadata version (if explicitly set by project): ?
Target Framework: uap10.0.17763;net6.0-windows10.0.19041.0
LangVersion: latest

I'm wondering if I need to use LoadCursor to check the handles match? But LoadCursor seems to take a string and not an int as a 2nd argument. I see in the test the IDC_ARROW is cast to char/PCWSTR here:
https://github.com/microsoft/CsWin32/blob/14e48cf5e15420736f6409e66431eafa59a284ab/test/WinRTInteropTest/Program.cs#L39
I figured I could maybe try just doing this as I don't have IDC_ARROW currently:
HCURSOR arrowHandle = PInvoke.LoadCursor(null, "IDC_ARROW");

But I don't think it works, but it also doesn't want to cast the result to an HCURSOR...
As a C# dev needing to call Win32 APIs (especially now with the Windows App SDK), it's pretty confusing to understand how to use these functions still. Though maybe the underlying scenario I'm trying to achieve (check type of current mouse cursor on screen) is the tricky bit? I feel like that should be a common scenario for test development though.

I went back today, and for some reason I do now see PInvoke.IDC_ARROW available, so I'm not sure what was happening.
I did also have to use HINSTANCE to make it work here:
var arrowCursor = PInvoke.LoadCursor(HINSTANCE.Null, PInvoke.IDC_ARROW);

Assert.AreEqual(arrowCursor, curinfo.hCursor, "Cursor wasn't arrow cursor.");

Great. I'll close the issue then.


Remove unecessary for loop

After subsequent refactoring, we no longer test the call and cast names separately.  We resolve first, then test.  So instead of looping, resolve, and if it doesn't exist, calculate the range to report on

Thanks for the review.  I expect this to conflict with https://github.com/dburgener/cascade/pull/202, so I'm going to wait to merge this until after the big and important one has merged.  Then I'll fix up the conflict here so this can get merged.


Simplify KUBECONFIG for tests

What this PR does / why we need it:
makes it possible to run func tests without running
make cluster-up
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm

/approve

/lgtm
/approve


Add Codecov support

Closes #34 as discussed.
One could make an argument for making the :service option madetory, likely with a better syntax, but that would be a backwards compatibility breaking change.

Hi @xendk! Sorry, somehow I missed this PR:(
May I ask you to do a small change and rename :service to :report-type? I think more general parameter name will be good in case if we will support something like text reports on a local machine.

but that would be a backwards compatibility breaking change.

Agree, no need to introduce breaking changes.

@sviridov Done. Though it is a bit odd that setting :report-type suppresses uploading. OTOH, it leaves :service if someone wants to implement uploaders to other services.

Thank you, merged!

It is super confusing that the :report-type option has nothing to do with undercover--determine-report-type and the report-type parameter to undercover-report.

Even more confusing: :report-type does two things (setting the report path and disabling report uploading), none of which have anything to do with the actual report type.
Sorry, but this is bad.


defaults in config file should go under the tests section

Previously we had this:
defaults:
    min_pass_rate: 1.00

tests:
  robustness:
    lowercase:
      min_pass_rate: 0.70
    uppercase:
      min_pass_rate: 0.70

We will now have this:
tests:
  defaults:
    min_pass_rate: 1.00

  robustness:
    lowercase:
      min_pass_rate: 0.70
    uppercase:
      min_pass_rate: 0.70

this issue depends on issue-349

@alytarik Kalyan has fixed this issue while fixing the config for model parameters. So it is completed.


Add KMS policy

Codecov Report

:exclamation: No coverage uploaded for pull request base (aws@6645688). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             aws     #121   +/-   ##
======================================
  Coverage       ?   70.47%
======================================
  Files          ?       68
  Lines          ?     2337
  Branches       ?      354
======================================
  Hits           ?     1647
  Misses         ?      690
  Partials       ?        0

Flag
Coverage Δ

batch-submitter
59.42% <0.00%> (?)

contracts
87.18% <0.00%> (?)

core-utils
53.79% <0.00%> (?)

data-transport-layer
34.06% <0.00%> (?)

message-relayer
83.01% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6645688...49fa158. Read the comment docs.