QSCC GetBlockByHash fails with marshal error "string field contains invalid UTF-8"

Description
Using qscc to get block by hash using the GetBlockByHash transaction. I am getting the following error as logged by the peer.
failed to invoke chaincode qscc, error: transaction returned with failure: failed to marshal response: string field contains invalid UTF-8

I'm querying it using the Gateway SDK for Go like this (some syntax supressed for clarity):
hashBytes, _ := hex.DecodeString(hash)
args := []string{channelName, string(hashBytes)}

contract.EvaluateTransaction("GetBlockByHash", args...)

Before the error the peer also logs:
2024-10-02 19:05:55.754 UTC 80fa DEBU [blkstorage] retrieveBlockByHash -> retrieveBlockByHash() - blockHash = [[]byte{0x38, 0xe5, 0x3a, 0xba, 0x44, 0x71, 0xa1, 0x1b, 0x78, 0xd7, 0x21, 0x1c, 0xc2, 0x73, 0x56, 0x45, 0xe9, 0x96, 0xee, 0xd8, 0xaf, 0x41, 0xca, 0xf, 0x3, 0x7, 0xc2, 0xb9, 0xa1, 0x1f, 0xbe, 0x70}]
2024-10-02 19:05:55.754 UTC 80fb DEBU [chaincode] handleMessage -> [448dfb03] Fabric side handling ChaincodeMessage of type: ERROR in state ready

One thing to note is that GetBlockByNumber works fine.
This has been observed in my testing of v2.5.3 and v2.5.10.
Steps to reproduce
This has been reproducible locally with Fabric Samples' test-network, on Hyperledger Labs CC-Tools and in production environments. To reproduce it with the test-network, follow the steps.

Deploy the test network in Fabric Samples

cd test-network
./network.sh up createChannel
./network.sh deployCC

Assuming you are on Fabric Samples, replace asset-transfer-basic/application-gateway-go/assetTransfer.go with the following content:

/*
Copyright 2021 IBM All Rights Reserved.

SPDX-License-Identifier: Apache-2.0
*/

package main

import (
	"crypto/x509"
	"encoding/hex"
	"fmt"
	"os"
	"path"
	"time"

	"github.com/hyperledger/fabric-gateway/pkg/client"
	"github.com/hyperledger/fabric-gateway/pkg/identity"
	protos "github.com/hyperledger/fabric-protos-go-apiv2/common"
	"google.golang.org/grpc"
	"google.golang.org/grpc/credentials"
	"google.golang.org/protobuf/proto"
)

const (
	mspID        = "Org1MSP"
	cryptoPath   = "../../test-network/organizations/peerOrganizations/org1.example.com"
	certPath     = cryptoPath +<EMAIL_ADDRESS>	keyPath      = cryptoPath +<EMAIL_ADDRESS>	tlsCertPath  = cryptoPath + "/peers/peer0.org1.example.com/tls/ca.crt"
	peerEndpoint = "dns:///localhost:7051"
	gatewayPeer  = "peer0.org1.example.com"
)

func main() {
	// The gRPC client connection should be shared by all Gateway connections to this endpoint
	clientConnection := newGrpcConnection()
	defer clientConnection.Close()

	id := newIdentity()
	sign := newSign()

	// Create a Gateway connection for a specific client identity
	gw, err := client.Connect(
		id,
		client.WithSign(sign),
		client.WithClientConnection(clientConnection),
		// Default timeouts for different gRPC calls
		client.WithEvaluateTimeout(5*time.Second),
		client.WithEndorseTimeout(15*time.Second),
		client.WithSubmitTimeout(5*time.Second),
		client.WithCommitStatusTimeout(1*time.Minute),
	)
	if err != nil {
		panic(err)
	}
	defer gw.Close()

	// Override default values for chaincode and channel name as they may differ in testing contexts.
	chaincodeName := "qscc"
	if ccname := os.Getenv("CHAINCODE_NAME"); ccname != "" {
		chaincodeName = ccname
	}

	channelName := "mychannel"
	if cname := os.Getenv("CHANNEL_NAME"); cname != "" {
		channelName = cname
	}

	network := gw.GetNetwork(channelName)
	contract := network.GetContract(chaincodeName)

	block, _ := contract.EvaluateTransaction("GetBlockByNumber", "mychannel", "3")

	blockMap, _ := decodeBlock(block)

	blockHash := blockMap["header"].(map[string]interface{})["data_hash"].(string)

	fmt.Println("Block hash:", blockHash)

	hashBytes, _ := hex.DecodeString(blockHash)

	_, err = contract.EvaluateTransaction("GetBlockByHash", "mychannel", string(hashBytes))
	if err != nil {
		panic(fmt.Errorf("failed to evaluate transaction: %w", err))
	}

}

func decodeBlock(b []byte) (map[string]interface{}, error) {
	var block protos.Block

	err := proto.Unmarshal(b, &block)
	if err != nil {
		return nil, err
	}

	blockMap := map[string]interface{}{
		"header": map[string]interface{}{
			"number":        block.Header.Number,
			"previous_hash": hex.EncodeToString(block.Header.PreviousHash),
			"data_hash":     hex.EncodeToString(block.Header.DataHash),
		},
		"metadata": block.Metadata,
	}
	return blockMap, nil
}

// newGrpcConnection creates a gRPC connection to the Gateway server.
func newGrpcConnection() *grpc.ClientConn {
	certificatePEM, err := os.ReadFile(tlsCertPath)
	if err != nil {
		panic(fmt.Errorf("failed to read TLS certifcate file: %w", err))
	}

	certificate, err := identity.CertificateFromPEM(certificatePEM)
	if err != nil {
		panic(err)
	}

	certPool := x509.NewCertPool()
	certPool.AddCert(certificate)
	transportCredentials := credentials.NewClientTLSFromCert(certPool, gatewayPeer)

	connection, err := grpc.NewClient(peerEndpoint, grpc.WithTransportCredentials(transportCredentials))
	if err != nil {
		panic(fmt.Errorf("failed to create gRPC connection: %w", err))
	}

	return connection
}

// newIdentity creates a client identity for this Gateway connection using an X.509 certificate.
func newIdentity() *identity.X509Identity {
	certificatePEM, err := readFirstFile(certPath)
	if err != nil {
		panic(fmt.Errorf("failed to read certificate file: %w", err))
	}

	certificate, err := identity.CertificateFromPEM(certificatePEM)
	if err != nil {
		panic(err)
	}

	id, err := identity.NewX509Identity(mspID, certificate)
	if err != nil {
		panic(err)
	}

	return id
}

// newSign creates a function that generates a digital signature from a message digest using a private key.
func newSign() identity.Sign {
	privateKeyPEM, err := readFirstFile(keyPath)
	if err != nil {
		panic(fmt.Errorf("failed to read private key file: %w", err))
	}

	privateKey, err := identity.PrivateKeyFromPEM(privateKeyPEM)
	if err != nil {
		panic(err)
	}

	sign, err := identity.NewPrivateKeySign(privateKey)
	if err != nil {
		panic(err)
	}

	return sign
}

func readFirstFile(dirPath string) ([]byte, error) {
	dir, err := os.Open(dirPath)
	if err != nil {
		return nil, err
	}

	fileNames, err := dir.Readdirnames(1)
	if err != nil {
		return nil, err
	}

	return os.ReadFile(path.Join(dirPath, fileNames[0]))
}

Under asset-transfer-basic/application-gateway-go/ run

go mod tidy
go mod vendor

Run the application

go run .

The application will GetBlockByNumber with 3 as the number for the block, then it will decode it partially so as to get the hash of the block, and then attempt to fetch the block by hash. This should throw a panic.
panic: failed to evaluate transaction: rpc error: code = Aborted desc = failed to evaluate transaction, see attached details for more info

Check the peer logs

2024-10-02 22:25:40.176 UTC 0113 ERRO [endorser] simulateProposal -> failed to invoke chaincode qscc, error: transaction returned with failure: failed to marshal response: string field contains invalid UTF-8
github.com/hyperledger/fabric/core/chaincode.processChaincodeExecutionResult
        /core/chaincode/chaincode_support.go:188
github.com/hyperledger/fabric/core/chaincode.(*ChaincodeSupport).Execute
        /core/chaincode/chaincode_support.go:162
github.com/hyperledger/fabric/core/endorser.(*SupportImpl).Execute
        /core/endorser/support.go:126
github.com/hyperledger/fabric/core/endorser.(*Endorser).callChaincode
        /core/endorser/endorser.go:120
github.com/hyperledger/fabric/core/endorser.(*Endorser).simulateProposal
        /core/endorser/endorser.go:187
github.com/hyperledger/fabric/core/endorser.(*Endorser).ProcessProposalSuccessfullyOrError
        /core/endorser/endorser.go:409
github.com/hyperledger/fabric/core/endorser.(*Endorser).ProcessProposal
        /core/endorser/endorser.go:350
github.com/hyperledger/fabric/core/handlers/auth/filter.(*expirationCheckFilter).ProcessProposal
        /core/handlers/auth/filter/expiration.go:61
github.com/hyperledger/fabric/core/handlers/auth/filter.(*filter).ProcessProposal
        /core/handlers/auth/filter/filter.go:32
github.com/hyperledger/fabric/internal/pkg/gateway.(*EndorserServerAdapter).ProcessProposal
        /internal/pkg/gateway/gateway.go:43
github.com/hyperledger/fabric/internal/pkg/gateway.(*Server).Evaluate.func1
        /internal/pkg/gateway/evaluate.go:64
runtime.goexit
        /usr/local/go/src/runtime/asm_arm64.s:1223 channel=mychannel txID=54e6bb0b

Upon further investigation @Tubar2 has discovered that the data_hash that we were using to try and get the block is not the hash needed for the function.
We got the function to work correctly by going inside the peer chains folder, opening up the LevelDB data that indexes each block with the hash. With that hash (which is not the data_hash), it works.
The question is: how do I get the correct hash since it doesn't seem to be in the block and taking the SHA256 of the block bytes does not return an existing hash. The answer is that the hash should be taken from the header of the block. Like this:
func BlockHeaderHash(b *protos.BlockHeader) []byte {
    sum := sha256.Sum256(BlockHeaderBytes(b))
    return sum[:]
}

type asn1Header struct {
    Number       *big.Int
    PreviousHash []byte
    DataHash     []byte
}

func BlockHeaderBytes(b *protos.BlockHeader) []byte {
    asn1Header := asn1Header{
        PreviousHash: b.PreviousHash,
        DataHash:     b.DataHash,
        Number:       new(big.Int).SetUint64(b.Number),
    }
    result, err := asn1.Marshal(asn1Header)
    if err != nil {
        // Errors should only arise for types which cannot be encoded, since the
        // BlockHeader type is known a-priori to contain only encodable types, an
        // error here is fatal and should not be propagated
        panic(err)
    }
    return result
}

Because of that, I'll close the issue. Hopefully, this serves for anyone that thinks that data_hash is the actual hash the block is indexed by.


Requesting updated C# code snippets

C# code snippets on page are using old Service Bus SDKs. These should be updated to use new Azure.Messaging.ServiceBus SDK.

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

ID: 13478d1d-11eb-930a-02c4-559a1ef89fa6
Version Independent ID: be36cdf9-1e5e-c47f-e699-652dc42ee2b9
Content: Azure Service Bus access control with Shared Access Signatures - Azure Service Bus
Content Source: articles/service-bus-messaging/service-bus-sas.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

Scratch that. I assumed feature parity with newer SDKs, looks like SAS token authentication is not supported in latest SDK.


useCurrentTime hook

In this PR i'm adding a current time hook. @vogelino if you tell me where to implement it I will. Also creating the tests helped me catch a bug :,) never thought I will admit that tests are useful

Great thx! I think it works well. However, I have a few comments:
1. What happens between 1am and 6pm?

2. Can you use it to set the default selected hour in the hour selector?

3. I would round up only from XX:45 (eg: 14:48 = 15 & 14:35 = 14)

The split is at 3am, I should prob make that clearer with the tests
I'll implement it in the hour selector.
Regarding your third point. Logically, I think don't think it makes a lot of sense, but that's my math brain speaking. On a technical, side, it would also be a pain to implement and in the grand scheme of things not make a big difference imo. If you insist, I'd be happy to adjust it


Error response from daemon: Client sent an HTTP request to an HTTPS server

Have you reviewed our technical documentation and knowledge base? Yes
Question:
Hi,
I have a problem adding a remote environment to Portainer using Portainer API.
In the documentation, there is a step that shows how to add it:
http --form POST my-portainer-server:9443/api/endpoints \
    "Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MSwidXNlcm5hbWUiOiJhZG1pbiIsInJvbGUiOjEsImV4cCI6MTQ5OTM3NjE1NH0.NJ6vE8FY1WG6jsRQzfMqeatJ4vh2TWAeeYfDhP71YEE" \
    Name="test-remote" URL="tcp://<IP_ADDRESS>:2375" EndpointCreationType=1

I've tried it and I have an error:
{
    "message": "Unable to initiate communications with environment",
    "details": "Error response from daemon: Client sent an HTTP request to an HTTPS server."
}

I'm using portainer and portainer agent in docker.
All ports are configured:

9443
8000
9001

I've tried Postman and using HTTP.
In both cases, there is the same error.
Also, I've tried to change EndpointCreationType to 2 but I've got another error:
{
    "message": "Unable to get environment type",
    "details": "Get \"https://<IP_ADDRESS>:9001/ping\": x509: certificate is valid for <IP_ADDRESS>, not <IP_ADDRESS>"
}

Do I miss something?

@solairen
Did you make the changes needed to the command or did you use the defaults?  The IPs will need to reflect your environment.

@solairen
While testing, I was able to get this to respond.  You will need to make sure that the remote docker daemon is exposed on TCP 2375:
http --verify=no --form POST https://<ip>:9443/api/endpoints \
    "X-API-Key:<my-api_key>" \
    Name="test-remote" URL="tcp://<ip>:2375" EndpointCreationType=1

You can change out the API Key for the Auth Bearer that you created.  Please let me know if this works.
Thanks!

@tamarahenson
Ok, I am able to connect when I expose Docker Daemon on port 2375.
But, I have another question.
How can I add portainer-agent by API?
Because right now I connected my Portainer to a remote Docker daemon if I understand it correctly.
What I want to achieve is to connect Portainer to another Portainer Agent using API.
From documentation the EndpointCreationType is number 2 for Portainer API.
The request is mostly the same but URL has to be to https://<IP>:9001
If yes, how can I fix this error:
{
    "message": "Unable to get environment type",
    "details": "Get \"https://<IP_ADDRESS>:9001/ping\": x509: certificate is valid for <IP_ADDRESS>, not <IP_ADDRESS>"
}

@solairen
I was able to create an Agent Endpoint via the API:
http --verify=no --form POST https://<ip>:9443/api/endpoints \
    "X-API-Key:<my-api-key>" \
    Name="test-remote-agent" URL="tcp://<ip>:9001" EndpointCreationType=2 TLS=true TLSSkipVerify=true TLSSkipClientVerify=true

Thanks!

@tamarahenson
Okay, I have been using it wrong...
I've tried with TLS but I've set it to false.
Now it works!
Thanks!


Boot lag if no WAN

RancherOS Version: (ros os version)
1.1.0
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
docker-machine + VSphere
After installing an host via docker-machine on VSphere, and after install RancherOs on disk, I decided to block the WAN connection on firewall for the hosts on this LAN thinking there is no need for the WAN anymore because the open-vm-service file was previously downloaded from github.
But the fact is, if i block the WAN, RancherOS start to take ages to boot and something seems hanging because the WAN is shutdown
The relevant part of the log who show this delay is this one :
time="2017-09-21T16:20:53Z" level=debug msg="Starting container" c.name="cloudinit_cloud-init_1" container.ID=8a600e2e5924e61e117918f3e2447c672b5cd403bf9a570cf744118372a8c937
time="2017-09-21T16:20:53Z" level=debug msg="[0/1] [cloud-init]: Started container name=cloudinit_cloud-init_1"
time="2017-09-21T16:24:17Z" level=info msg="[1/1] [cloud-init]: Started "
time="2017-09-21T16:24:17Z" level=debug msg="[2/2] Done cloudinit Services"
time="2017-09-21T16:24:17Z" level=info msg="Project [cloudinit]: Project started "
time="2017-09-21T16:24:17Z" level=debug msg="[9/21] Done cloud-init"

If a unblock the WAN this part is pretty fast :
time="2017-09-21T18:42:09Z" level=debug msg="Starting container" c.name="cloudinit_cloud-init_1" container.ID=cce1c4185f88ec441edcdf9b789d321e936b4a415e00784c3eae4ad9f1d0800a
time="2017-09-21T18:42:09Z" level=debug msg="[0/1] [cloud-init]: Started container name=cloudinit_cloud-init_1"
time="2017-09-21T18:42:13Z" level=info msg="[1/1] [cloud-init]: Started "
time="2017-09-21T18:42:13Z" level=debug msg="[2/2] Done cloudinit Services"
time="2017-09-21T18:42:13Z" level=info msg="Project [cloudinit]: Project started "
time="2017-09-21T18:42:13Z" level=debug msg="[9/21] Done cloud-init"

And if re-block the WAN :
time="2017-09-21T18:44:46Z" level=debug msg="Starting container" c.name="cloudinit_cloud-init_1" container.ID=391f611e9be92469bf3b02cea85883464438a7e8acd7bc5fb9a7596dfe18a84e
time="2017-09-21T18:44:46Z" level=debug msg="[0/1] [cloud-init]: Started container name=cloudinit_cloud-init_1"
time="2017-09-21T18:48:12Z" level=info msg="[1/1] [cloud-init]: Started "
time="2017-09-21T18:48:12Z" level=info msg="Project [cloudinit]: Project started "
time="2017-09-21T18:48:12Z" level=debug msg="[2/2] Done cloudinit Services"
time="2017-09-21T18:48:12Z" level=debug msg="[9/21] Done cloud-init"
time="2017-09-21T18:48:12Z" level=info msg="[10/21] Starting read cfg and log files"

Did i missed something ?
Is there another services who require the WAN even if the host seems to boot normally after this hang period ?

yeah - its trying to get the open-vm-tools cfg because 1.1 detects that you're running vmware.
There are some improvements in the pipeline.

Duplicate of #2171
Close this one.

It is not a duplicate of the issue you mention and you can see than the problem is not the same at all...
In the issue you mention, the service config was downloaded.
This is not the case in this issue due to the fact than there is no WAN and the service config can't be downloaded from github, that's why the boot hang.

@rockandska
If it doesn't try to enable open-vm-tools service, there is no problem here.
Only running on vmware will enable it.
I consider one way to solve these two issues, but respect your thoughts, reopen this issue.

I only let this issue opened because i didn't tried yet to change the open-vm-tools service config file url via service options to see if i can download it from a private repository only accessible from the LAN instead of github


Prisma won't build consistently with the same Dockerfile specs

Bug description
Hi, when bundling an app with Prisma, it works on a node:20 and node:20-alpine locally.
When building with the same machines on Github Action ubuntu-latest runner they fail with the following error:
PrismaClientValidationError: Invalid client engine type, please use `library` or `binary`

For context, I'm trying to have both Accelerate (for Vercel) and Standalone (Long-running) together.
And here is the PR (including failed builds): https://github.com/dreampipcom/hypnos/pull/41
Only difference is the Docker Info (obviously) and my network settings (firewalls included).
Why is that so? Is there anything else I need to do?
How to reproduce
docker-compose up hypnos_local vs. Github Actions Dockerfile
Expected behavior
Consistent builds
Prisma information
https://github.com/dreampipcom/hypnos/pull/41
Environment & setup

OS: node:20 and node:20-alpine
Database: MongoDB
Node.js version: 20

Prisma Version
5.18.0

Solved: Missing NEXUS_STANDALONE environment variable in "base as build" Docker execution context.
Rationale: in my local machine, the environment variable was available at a higher reference point.


[ENH] _fit_transform method in BaseTransformer

Contents of https://github.com/sktime/sktime/pull/3847
What does this implement/fix? Explain your changes.
Implements a _fit_transform method in BaseTransformer. fit_transform now calls the private method, and contains a set of checks merged from fit and transform.
This PR also replaces the current fit_transform overrides in SupervisedIntervals and TSFreshRelevantFeatureExtractor with the private method.
Alongside allowing for easier implementation of efficient fit_transform methods, the change reduces the repetition of some checks when the method is called.
Any other comments?
Happy to make changes or add additional checks if people think they are necessary. As it is, this is essentially a merge of the current fit and transform methods with some repetition removed.

as a general comment: I do not like all the cloning of transformers to run a simple parallel for-loop (called vectorize).
The transformer should be fit once on all data. And then it should be stateless and without side-effects, i.e. it should be possible to apply transform to slices without cloning.

My suggestion: can we get rid of vectorize? I do not see any advantage of it. I.e. there is not even any parallelization.
This is the interface of sklearn:
https://github.com/scikit-learn/scikit-learn/blob/8c9c1f27b/sklearn/preprocessing/_function_transformer.py#L20
It is slim and simple.


After opening a file it should remain in working files even though its not edited

A lot of times you want to look at a file for reference e.g the generated js while you modify ts.
The working files tend only only keep files with modification. It should behave like sublime where it stays open

There are many ways to make a file durable, e.g. double click. With the work on tabs there are even more ways (e.g. double click on the title). Would you expect other ways?

I absolutely had no idea about double click.
May be a contextual tooltip?
Why not add the files to working files on a single click? Like sublime ?
On Monday, May 23, 2016, Benjamin Pasero<EMAIL_ADDRESS>wrote:

There are many ways to make a file durable, e.g. double click. With the
work on tabs there are even more ways (e.g. double click on the title).
Would you expect other ways?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/vscode/issues/6507#issuecomment-221042577

@nojvek sublime does not add files to working files on single click, you need a double click or edit.

Ahh you are right @bpasero
Although What I usually use is Ctrl+P to quickly search for file and open it. sublime adds it as a proper tab / working file, while vscode doesn't.
Is this an easy fix I could make?

Closing as duplicate of https://github.com/Microsoft/vscode/issues/5554 then.


Open already running instances of Yubico Authenticator instead of showing an error popup.

Yubico Authenticator version: 4.3.4
Operating system and version: Ubuntu 16.04 and Windows 10 (and probably happens the same under Mac, but cannot check)
YubiKey model and version: Doesn't matter
Bug description summary: In previous versions, yubico authenticator opened already running instances of the app when trying to open it again. Right now it shows a popup saying it's already open.

Steps to reproduce

Open Yubico Authenticator
With the app open (whether in background or foreground), try to open it again from the OS menu.

Expected result
To open the already running instance.
Actual results

Other info
This was the behavior until one of the latest versions. Can I ask why? I don't understand why it has been changed and I don't see any reference in the commits.

There is some motivation for this change available here. If this is an enough requested feature we'll consider re-adding it.

I'm sure that am not the only one which uses the keyboard to navigate through apps. In both windows and linux, using the windows key shows the system search, which is a very fast way to open apps (both already open and non open).
Having to move my hand from the keyboard to the mouse to go to the tray icon just to open the yubico authenticator is something I was able to do until that change, but not now :cry:

I use a batch file to open yubioath with a specific application that requires 2FA codes upon every launch - however sometimes, I already have yubioath which now produces this popup which interrupts an otherwise smooth workflow.
I would really appreciate if this could be looked into :heart:

I am having the same problem on Fedora 30. I selected for it to 'hide' when it starts and now it won't show up. When I click the icon I get the same popup window.
I have uninstalled and reinstalled it several time and get the same "Yubico Authenticator is already running".

I have the same issue on the new Yubico Authenticator 5.0.0 RC1. on Arch Linux. Every once in while I get "Yubico Authenticator is already running". I have to kill the process in order to get it started.

The reasoning given in the PR linked is "others do it", but it only shows that some applications do it, it doesn't show the proportion of application that show an error vs those that raise the already running instance, it doesn't tell whether the users prefer one behavior or the other.
Let's stop and think for a minute in terms of user experience, what are the pros and cons for an error vs raising the application?
Error:

pros:

?

cons:

interruption (I tried opening the application, it didn't open, I had to stop to understand why, I have to find where it is and open it again)
tells the user he did something wrong

Raising the application:

pros

user tried to open the application, the application opened

cons

it was reported that sometimes the application blinks in the tray rather than raising the window (it apparently doesn't happen for everyone, there is already a visual queue, do we need an error on top of that?)

Since there seems to be some interest for this, I've re-opened this PR https://github.com/Yubico/yubioath-desktop/pull/296 to test it on the various platforms.

What to do if I set "start minimized to tray" on Ubuntu 19.10 and there is no tray icon?

@powerriegel there's a config file at ~/.config/Yubico/Yubico\ Authenticator.conf in which you will find the settings for closeToTray and hideOnLaunch, change these or just delete this file if you're not worried about loosing window size/position, favorites and such.

as a small step i suggest listening to process signals and raising the window on receiving a signal like USR1. this way one could raise the window via pkill -USR1 yubioath-desktop
this would apply only to linux though?

This is a real bummer. There seems to be no instance running visually, but the window is popping up. Therefore the authenticator is rendered completely useless. This renders the yubikeys pretty much useless for many services as they need an authentication app.

As a person using i3 as their desktop environment, this has been bothering me for ages. I usually start programs by looking them up in dmenu by name, and very rarely look at the actual tray icons.
The reason given in the original PR is that Showing a "App is already running" message seems to be a common pattern, but I can't think of the last time I've seen an app behave this way. All other software that I use regularly in a fashion similar to Yubico Authenticator (e.g. KeePassXC) definitely just brings up the running instance.
Thanks to reading the comments in this issue I discovered the option to disable the tray icon entirely (which I somehow missed before), and that works for my use case - but it would be nice to see the original annoying behavior changed.

I am so fed up with this terrible application.  It doesn't work on Windows or Linux reliably and I honestly just want to export what's on the key and use a different authenticator app.  If you aren't going to fix this app, please make an export tool so we can get on with our lives.
Here's Windows...

Here's Linux...

THIS APP HAS A CATASTROPHIC BUG EVERY SINGLE DAY.  If you aren't going to fix the app, then at least work to deprecate it responsibly.

Yubico Authenticator 6.0 has now been released and uses a new codebase. As such, this issue has been marked with the legacy label, and will be automatically closed in 7 days. If this issue is still relevant to Yubico Authenticator 6, please comment on the issue saying so, and it will be kept open (or be re-opened). Sorry for the inconvenience!

And the solution was to remove the tray icon? lol

No, the solution was to use platform specific functionality to ensure single-window behavior.

Well, it is, since was the workaround I described to be able to use it (disable the tray icon).
But you're right, since the main issue described here was to be able to open already open instances, and that works now.

Just to document this here:
i am using an alternative software yubikey-oath-dmenu on linux that is perfect for my  keyboard-only usecase. pick and insert a totp-code in under 4 seconds ;-)
having the capability of the yubikey scriptable is the main feature in my mind


Metal support

This revision supports Metal.

This feature will be done in CubbyFlow-v0 and Jet Framework.


SocketCAN test

Test SocketCAN sample at samples/net/socket/can

Verified that SocketCAN is working between CAN0 and CAN1 transceivers

Verified that SocketCAN is working between FMUK66 and NavQPlus

2 SocketCAN interfaces can work in parallel with my changes in socketcan driver.
I tested it by switching CAN cable between 2 CAN slots on FMUK66 board on the fly.
Will check in he patch tomorrow.

Following commit should serve the purpose -
https://github.com/zephyrproject-rtos/zephyr/commit/3be103b246f500c36a12487b33ef8a0e8a46c78c

Looks to be completed.


Publish valid Maven POM for library with dependency on Maven BOM

To support publishing a POM for a javaLibrary that depends on a BOM:

Since Maven doesn't support a dependency on a BOM, we need a way to map this to an import scoped entry in
We can't do this automatically, since we don't resolve the dependencies before publishing, so we don't known when to map to 'import'.
Instead, we can provide a DSL for users to define an 'import' for their MavenPublication.pom.

Closing as a duplicate of #4543 which has a better description of the current issue(s)


Modify additional codes

Hi @mojvan, thank you for your proposition and time put in, but unfortunately modifying this example in this way do not add any functionality and makes the title prop redundant (index can be used instead). Also removing the extracted DATA and storing index as title reduce the clarity of the example.
If you are keen on helping with the website please focus on the reported and confirmed issues first.


Enable teachers to upgrade grades.

Pull Request Description:
This pull request introduces a minor enhancement to the README file, specifically focusing on improving user experience by providing information about viewing grades. Two additional lines have been added to the README file to facilitate this.
Changes Made:
Added instructions for users to easily locate and view their grades.
Incorporated information regarding the process of accessing grades within the README file.
Guidance for Reviewer:
To thoroughly review the changes introduced by this pull request, please ensure to:
Review README Modifications: Pay close attention to the modifications made to the README file. The two new lines added are crucial for enhancing user understanding of the grading process.
Verify Clarity: Check that the instructions provided for viewing grades are clear and concise, ensuring users can easily follow them without ambiguity.
Assess Impact: Evaluate the significance of these changes in enhancing user experience. Consider whether the added information adequately addresses potential user queries regarding grade accessibility.
Code Integrity: While the changes primarily focus on documentation, ensure there are no inadvertent impacts on the codebase integrity. Though unlikely, double-check for any potential conflicts or unintended consequences.
Thank you for your attention to detail in reviewing this pull request. If there are any questions or concerns, please don't hesitate to reach out for clarification.

"Subject: " and "Body: " in the commit message template are placeholders. In the future you should format commits like (using your commit as an example):
Add: new fourth line
{must have BLANK new line here}
Added a fourth new line after review and made changes to practice.
{you can potentially add steps/changes made in the commit}
* change 1
* change 2
* change 3

Issue #4


azorult-bomb

PR Template for new Detections

[x] windows_remote_services_allow_remote_assistance.yml
[x] windows_remote_services_rdp_enable.yml
[x] windows_service_stop_by_deletion.yml
[x] windows_modify_registry_disable_toast_notifications.yml
[x] windows_modify_registry_disable_windows_security_center_notifs.yml
[x] windows_remote_services_allow_rdp_in_firewall.yml
[x] windows_modify_registry_suppress_win_defender_notifs.yml

Details
*what does this PR have in it, screenshots are nice 😄
windows_remote_services_allow_remote_assistance.yml

windows_remote_services_rdp_enable.yml

windows_service_stop_by_deletion.yml

windows_modify_registry_disable_toast_notifications.yml

windows_modify_registry_disable_windows_security_center_notifs.yml

windows_remote_services_allow_rdp_in_firewall.yml

windows_modify_registry_suppress_win_defender_notifs.yml

Author Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description>
[ ] Make sure that CI/CD detection-testing and build-and-validate jobs passed ✔
[ ] Is there an Atomic Test? Is GUID on the test file under array: atomic_test_guid, example?

Review Checklist

[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

@tccontre : Awesome PR!


chore: drop python 3.8 and add python 3.13

Update CI to test with python 3.12 basically
Update CI to drop old Qt versions and add Qt 6 versions

Interesting, is this an unknown bug?
2024-09-28 04:11:34,233 - aqt.main - DEBUG - installer 2092 Load configuration from D:/a/aqtinstall/aqtinstall/ci/settings.ini
2024-09-28 04:11:34,233 - aqt.main - INFO - installer 2092 aqtinstall(aqt) v3.1.19.dev19 on Python 3.9.13 [CPython MSC v.1929 64 bit (AMD64)]
2024-09-28 04:11:34,233 - aqt.main - WARNING - installer 2092 The command 'install' is deprecated and marked for removal in a future version of aqt.
In the future, please use the command 'install-qt' instead.
2024-09-28 04:11:34,233 - aqt.helper - DEBUG - helper 2092 Attempt to download checksum at https://download.qt.io/online/qtsdkrepository/windows_x86/desktop/qt6_653/Updates.xml.sha256
2024-09-28 04:11:36,188 - aqt.main - ERROR - installer 2092 The packages ['qt_base'] were not found while parsing XML of package information!
==============================Suggested follow-up:==============================
* Please use 'aqt list-qt windows desktop --arch 6.5.3' to show architectures available.
Execute: ['python', '-m', 'aqt', 'install', '6.5.3', 'windows', 'desktop', 'win64_mingw81', '--archives', 'qtbase', 'icu', 'qt']

# aqt list-qt windows desktop --arch 6.5.3
win64_mingw win64_msvc2019_64 win64_msvc2019_arm64 wasm_singlethread wasm_multithread
# aqt list-qt windows desktop --arch 5.15.2
win32_mingw81 win32_msvc2019 win64_mingw81 win64_msvc2015_64 win64_msvc2019_64 wasm_32

.

Interesting, is this an unknown bug?

I've separated to a different PR as #822
It will be tackled there


quarto: 1.6.30 -> 1.6.33 and apply patch

This updates quarto to the latest pre-release version and applies a Deno 2 compatibility patch I generated from the commits on https://github.com/tjni/quarto-cli/tree/deno2-compatibility.
I'll work on getting this pushed upstream, but I'm hoping that this can unblock users in the meantime.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

sorry, late to the party, but this works really well, just tried to build a quarto website with it, and it runs! hopefully this can make it into upstream

Hi @tjni, thanks for creating this new approach! I look forward to using it on NixOS once it is available in unstable on NixPkgs.

Hi @tjni, thanks, I've tested this approach extensively and it always seems to work correctly.


sdl2 trouble on Windows

hello, thanks for sharing this.
sorry for asking this newbie question, but I don't know how to deal with sdl2...
I am getting this error when doing:
D:\_CODE\_C\EnTT-Pacman\build>cmake -DCMAKE_BUILD_TYPE=Release ..
-- Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.19045.
Finding SDL2
CMake Error at C:/Program Files/CMake/share/cmake-3.23/Modules/FindPackageHandleStandardArgs.cm
ake:230 (message):
  Could NOT find SDL2 (missing: SDL2_LIBRARY)
Call Stack (most recent call first):
  C:/Program Files/CMake/share/cmake-3.23/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPH
SA_FAILURE_MESSAGE)
  cmake/modules/FindSDL2.cmake:173 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:71 (find_package)

-- Configuring incomplete, errors occurred!
See also "D:/_CODE/_C/EnTT-Pacman/build/CMakeFiles/CMakeOutput.log".
See also "D:/_CODE/_C/EnTT-Pacman/build/CMakeFiles/CMakeError.log".

I installed sdl2 with
vcpkg install sdl2 but not sure where is it or if I need to add the path to the system environment...
Maybe I must add that path?
C:\src\vcpkg\buildtrees\sdl2\

You might need to add the toolchain path to the cmake invocation. So it might be something like this:
cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_TOOLCHAIN_FILE=C:\src\vcpkg\scripts\buildsystems\vcpkg.cmake ..

Another possibility is that the package could have been installed using the wrong triplet. On older versions of vcpkg, the default was x86-windows. For newer versions, it changed to the host triplet. You might need to explicitly name the right triplet. That might look like this (assuming you're on x86_64):
vcpkg install sdl2 --triplet x64-windows

If it's not one or both of those things then I'm not sure what it could be.

Hey, did you end up solving this or are you still facing this issue?

Hey sorry, not yet. But I'll do and post some feedback.

Thanks for your reply.


Ingredients Page: Make favorites icon different style when selected

currently changes on hover, but need to change in state to keep it red


provide lift api to manually start drag operations while mouse is down

attempts to provide a solution to #182

Closing this in a few days unless anyone has a good reason not to.

Any chances to have this API in the npm release?

Could we add this back in? We also need the functionality to trigger drags on other events


feat: add limit-req demo

Please answer these questions before submitting a pull request

Related issues (for new feature or improvement, one should submit an issue first before writing code): Fix #12

Codecov Report

Merging #16 (501782d) into master (82c36f2) will decrease coverage by 0.03%.
The diff coverage is 77.77%.

:exclamation: Current head 501782d differs from pull request most recent head 02519f8. Consider uploading reports for the commit 02519f8 to get more accurate results

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   78.85%   78.81%   -0.04%
==========================================
  Files           9       10       +1
  Lines         487      505      +18
==========================================
+ Hits          384      398      +14
- Misses         82       84       +2
- Partials       21       23       +2

Impacted Files
Coverage Δ

cmd/go-runner/plugins/limit_req.go
77.77% <77.77%> (ø)

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 82c36f2...02519f8. Read the comment docs.

Will add it later.


osal.c mdbx_panic error: #error FIXME

A compilation error #error FIXME was detected in "osal.c" function __cold void mdbx_panic(...) by MinGW GCC 7.1 on Windows 7 - x64.
I think it's because the relevant code fragment
if (IsDebuggerPresent()) {
    OutputDebugString("\r\n" FIXME "\r\n");
    FatalExit(ERROR_UNHANDLED_ERROR);
}

is defined only under #ifdef _MSC_VER, which is not true in this environment. But I'm not sure.

@zingchen , today I am on the road, so the problem will fix later.

@zingchen, I got home and assessed the volume of work.
It's more than I expected. Think I can do this not earlier in the end of weekend.

I see. There're several other FIXMEs that seem to be MinGW-relevant too.

I am sorry.

@zingchen, on the other hand - you could make this work and submit a pull-request. In this case please don't forget about testing (i.e. appveyor.yml). So, at least one more set of test cases must be implemented:

validation of cleanup on thread termination (and so on), with respect to glibc bugs #21031 and #21032.
validation of interoperability with libmdbx.dll and libmdbx.lib, which were built by MSVC 2013/2015/2017.

Related to https://github.com/leo-yuriev/libmdbx/issues/17


Gpx export track

Coverage decreased (-0.07%) to 84.701% when pulling ae6a4f611048bfc10489d7270fc500982da567d1 on gpx_export_track into 612b2c2ef0afd85b321ff61055b5571b0007548a on master.


Services returned by list differ from those when fetched directly

Describe the bug
The response when fetching services by list is different from that when directly fetching a service. Ideally they should both match and return the same data (including the key-value service data)
This does currently cause an issue in the UI when refreshing on service detail page (we fetch the specific service and it doesn't contain the required data to show the service info) - https://github.com/epinio/ui/issues/45. It also means the user cannot edit their service's key-value pairs (they're not present from the list https://github.com/epinio/ui/issues/46)
To Reproduce
Fetch service lists via /api/v1/namespaces/<ns>/services or /api/v1/services and service is
{"meta":{"name":"aaaaaa","namespace":"dev"},"boundapps":null}

vs /api/v1/namespaces/<ns>/services/<service name>
{"user":"my-epinio-user","details":{"a":"a"}}

Desktop/CLI (please complete the following information):

Epinio Version: 0.1.6

@richard-cox is this blocking Beta3 release ?

@kkaempf It should do, though that's a long way off. In terms of beta2 i don't think so, unless there's objections from @agracey ?
As it stands

Users can't edit 'data' for existing services
After navigating to a service in the ui and refreshing the page the user will see a forever loading indicator

@richard-cox Please add your thoughts about the spec below:

Current State Of The Service Responses
The relevant types are ServiceResponseList, ServiceResponse, and ServiceShowResponse.
The last is the data for an individual service.  The first is an array of the second, the data for a set of services.
In detail
type ServiceResponseList []ServiceResponse

type ServiceResponse struct {
	Meta      ServiceRef `json:"meta"`
	BoundApps []string   `json:"boundapps"`
}

type ServiceShowResponse struct {
	Username string            `json:"user"`
	Details  map[string]string `json:"details,omitempty"`
}

Proposed changes
First, extend the ServiceResponse with a Spec field, of type ServiceShowResponse. That should cover most of the things desired by the ticket.
type ServiceResponse struct {
	Meta      ServiceRef          `json:"meta"`
	BoundApps []string            `json:"boundapps"`
	Spec      ServiceShowResponse `json:"spec"`
}

The issue is the bound apps information ... Currently supplied for the list of services no such information is available when asking for a single service.
This can be fixed by moving that field into the ServiceShowResponse, i.e. the Spec.
This would look like:
type ServiceResponseList []ServiceResponse

type ServiceResponse struct {
	Meta      ServiceRef          `json:"meta"`
	Spec      ServiceShowResponse `json:"spec"`
}

type ServiceShowResponse struct {
	Username string            `json:"user"`
	Details  map[string]string `json:"details,omitempty"`
	BoundApps []string         `json:"boundapps"`
}

The disadvantage of doing this is that retrieving the data for a single service will become slower. Because that query would now have to query the kube for the service/app mapping. This currently involves asking for all apps, their services, and inverting that (*).
The two services endpoints asking for all services (in a namespace) are able to amortize that, as they ask for this mapping once, and then simply scatter the information to all the ServiceResponses.
If that change is made then asking for many services is very likely better done by asking for all and then filtering client side to the desired subset (matching names by glob pattern?), than asking individually.
Because the latter would incur the service/app overhead in each call, instead of once for all services.

(*) The knowledge of which apps and services are bound is currently stored only in the apps, implicitly, in the mountpoints to the service secrets.
The service resources have no information about which apps they are bound to.

Putting such information somewhere, to make the service -> apps query as fast as an app -> services query, runs the risk of desynching from the primary source I described 2 paragraphs above.


feat: min/max for integer flag

The tsconfig.json change in this issue was a breaking change for anyone still using Node v12, but the subsequent release was only a minor version bump (does this project not follow semver?). The use of optional chaining operator is no longer transpiled out by tsc, and Node 12 doesn't support that operator. Removing a supported version from engines should be a breaking change!


Change suit sensors on other players

About the PR
You can now change suit sensors on other players if they in crit/dead.
You can easily change your suit sensor or ownerless suit sensors without delay, but when you change suit sensors on another body you'll have 3sec doAfter
Why / Balance
Just QOL tweak for med and antagonists players.
Technical details
Some new conditions check on server's side + doafterevent in shared.
Media
https://github.com/space-wizards/space-station-14/assets/115770678/f266cf44-10d4-4e5e-950c-083145ffec02

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

tweak: You can now change suit sensors on crit/dead people without stripping them

The time on this should be faster or just instant.
It takes like 2-2.5 seconds to remove a jumpsuit on someone who's dead. So turning off their sensors takes longer.
This should also apply to people in stamcrit.

The time on this should be faster or just instant. It takes like 2-2.5 seconds to remove a jumpsuit on someone who's dead. So turning off their sensors takes longer.
This should also apply to people in stamcrit.

Lowered doafter time, but i really don't want to make it instant. And now you can change sensors in any moment when people can't interact

The time on this should be faster or just instant. It takes like 2-2.5 seconds to remove a jumpsuit on someone who's dead. So turning off their sensors takes longer.
This should also apply to people in stamcrit.

perhaps this just means taking someone's jumpsuit off should take longer
taking someone's jumpsuit off works to take off like four or five items at once despite it taking the same amount of time to remove like gloves, headsets, etc

The time on this should be faster or just instant. It takes like 2-2.5 seconds to remove a jumpsuit on someone who's dead. So turning off their sensors takes longer.
This should also apply to people in stamcrit.

perhaps this just means taking someone's jumpsuit off should take longer, since it's what people do for efficiency anyways
taking someone's jumpsuit off works to take off like four or five items at once despite it taking the same amount of time to remove like gloves, headsets, etc

It takes 7ish seconds if they're not in crit/dead and 3ish if they are. It definitely doesn't need to be longer.

7 seconds and 3 doesn't sound very good.  5 and 1.5-2 sounds much more reasonable

7 seconds and 3 doesn't sound very good. 5 and 1.5-2 sounds much more reasonable

1.5 sec to change suit sensors. You can't change it if person can interact (not crit/dead/sleep/cuffed etc)

bruh, me when test failed because of CargoTest 💀


Experimental Component - Structured List - Design

Design
Create final design specifications that are ready for production. Functionality and variations should match as is Carbon components.

[x] confirm Security design matches duo intent
[x] confirm Sketch file sizing/spacing specs
[x] screen grab sizing guidance and add to GitHub issue

Structured List
Sketch file: https://ibm.ent.box.com/file/332823096737

@IBM/carbon-developers ready for dev 👍
Dev issue: #1275


[2.0][rebased] Prevent segmentation fault in Reflection->getParameters()

This is the rebased version of #505, which is te porting of #504 to the 2.0 branch.

Will we do tests for this PR?

In the original PR we concluded that this was really hard to test: https://github.com/getsentry/sentry-php/pull/504#issuecomment-335767304


Shared app includes search results when search results were cleared before sharing

When the search is cleared before the app is shared, the search results still display in the shared version of the app.
Steps to repro:

Launch the app (e.g. http://<IP_ADDRESS>/ParkLocatorLGDec15) and click OK on the splash screen
Search for a park
Clear the search using the "X" on the search panel
Share the app
Open the shared version of the app.  Note the shared version includes the original search, despite the fact that it was cleared.

FYI @jessnoo

Verified with https://github.com/Esri/configurable-locator/commit/6a786ff75d0e3405f6293532d64e4ec119c0a9c7


setRouteLeaveHook doesn't work as expected with dynamic routes

Version
2.7.0
Test Case
setRouteLeaveHook doesn't seem to play well with getChildRoutes. After some debugging I could see that it is related to how getRouteID works internally.
Here is a test case:
http://jsbin.com/jizese/edit?js,output
Steps to reproduce
Click on "Page 1", then "Page 2", then "Home"
Expected Behavior
"Leaving page 1" should be shown in an alert window when leaving Page 1
"Leaving page 2" should be shown in an alert window when leaving Page 2
Actual Behavior
"Leaving page 1" is show when opening Page 1
Nothing is shown when leaving Page 2

This is unavoidable – if you change the route objects such that there's no more referential equality, how are we supposed to identify them?
If you're using dynamic routes and route leave hooks, make sure you memoize the route objects.

I understand. Could you elaborate on how to memoize the route objects for this to work? Thanks in advance!

BTW, each route has a path and is placed somewhere in the tree of nested routes. If you join paths of the routes from the root to a leaf, you should get a unique identifier for this leaf route. Couldn't this be used in getRouteID instead of the generated numbers?


feat: EnableJsonLayout for using NLog Layout for generating document

Now you can do this:
<extensions>
    <add assembly="NLog.Targets.ElasticSearch"/>
    <add assembly="Elastic.Apm.NLog"/>
    <add assembly="Elastic.CommonSchema.NLog"/>
</extensions>
<targets>
   <target xsi:type="ElasticSearch" enableJsonLayout="true">
       <layout xsi:type="EcsLayout" />
   </target>
</targets>

Resolving #122
Checklist

[x] My commits follow Conventional Commits
[ ] documentation is updated

Is there a pre-release where I can test this out?

@johnjaylward You can download the nuget-package here:
https://ci.appveyor.com/project/markmcdowell/nlog-targets-elasticsearch/builds/32860944/artifacts
Just place it in local-nuget-package-path, and then you can try it out. Would be nice with some feedback

Thanks @snakefoot , I tried it out, but I get an exception thrown from the Logger.
System.TypeInitializationException
  HResult=0x80131534
  Message=The type initializer for 'Program' threw an exception.
  Source=Program
  StackTrace:
   at Program.<Main>d__7.MoveNext() in ....\Program.cs:line 34

  This exception was originally thrown at this call stack:
    [External Code]

Inner Exception 1:
NLogConfigurationException: Exception when parsing ....\bin\Debug\netcoreapp3.1\NLog.config.

Inner Exception 2:
TargetInvocationException: Exception has been thrown by the target of an invocation.

Inner Exception 3:
ArgumentException: LayoutRenderer cannot be found: 'ElasticApmTraceId'

Config:
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      throwExceptions="true" >
<extensions>
  <add assembly="NLog.Targets.ElasticSearch"/>
  <add assembly="Elastic.CommonSchema.NLog"/>
</extensions>
  <targets>
    <target type="ColoredConsole"
            name="console"
            encoding="UTF-8"
            layout="${longdate} ${level:uppercase=true} Thread(${threadid}:${threadname}) ${callsite}:${callsite-linenumber} - ${message} ${exception:format=tostring}"
            useDefaultRowHighlightingRules="true"
            errorStream="false"
            detectConsoleAvailable="false" />
    <!-- Use a buffered wrapper, or other async target for any network pushed logs like Elastic or SQL -->
    <target type="ElasticSearch"
            name="elastic"
            uri="http://localhost:9200"
            requireAuth="false"
            username=""
            password=""
            index="appdevlogs-dev-${date:format=yyyy.MM.dd}"
            documentType="logevent"
            includeAllProperties="false"
            excludedProperties=""
            maxRecursionLimit="1"
            disableAutomaticProxyDetection="true"
            disablePing="false"
            EnableHttpCompression="false"
            DisableCertificateValidation="false"
            enableJsonLayout="true"
    >
      <layout type="EcsLayout" />
    </target>
  </targets>
  <rules>
    <logger name="*" minlevel="Trace" writeTo="console" />
    <logger name="*" minlevel="Trace" writeTo="elastic" />
  </rules>
</nlog>

Dependencies:
  <ItemGroup>
    <PackageReference Include="NLog" Version="4.7.2" />
    <PackageReference Include="NLog.Targets.ElasticSearch" Version="7.1.14-alpha-lgtulitr" />
    <PackageReference Include="Elastic.CommonSchema.NLog" Version="1.5.0" />
  </ItemGroup>

Is there another dependency I should have for the renderer that isn't found?

ok, I think I found it. Looks like I was missing Elastic.Apm.NLog . I had thought that Elastic.CommonSchema.NLog pulled it in, but it looks like it doesn't. Maybe something to note in the documentation when using ECS.

Updated example in original post to reflect your learned experiences.

Looking at the message it generated, I guess I have a few comments/questions:

Why is log.level a top level field instead of being under the log object
I don't see any use of ${callsite} or ${callsite-linenumber} in the output. Is that configurable? I'd prefer to see those under log.origin.function and log.origin.file.line respectively assuming they are available at runtime.
As part of the "process" information, I'd like to be able to output the assembly version information so I can track errors that may have happened due to an upgrade. Before I found this PR, I was planning on putting it under the existing package.version field since the "process" object didn't seem to have version info. Can that be configured?
For some reason the log.logger shows as "Elastic.CommonSchema.NLog" instead of my logger name (i.e. "my.namespace.MyClass"). This seems unexpected. I don't really care what the logging system was that sent the log to Elastic, I care about the system generating the log.

Maybe these comments would be better placed in the "Elastic.CommonSchema.NLog" project?

Maybe these comments would be better placed in the "Elastic.CommonSchema.NLog" project?

That would probably be best, since nothing can be done about them here. Small comments:

You can add your own custom attributes like this:

<layout type="EcsLayout">
    <metadata name="CallSite" layout="${callsite}" />
</layout>

Think that Process and service (Has Version) should be merged together. But it is not me making the rules.

This has been fixed with: https://github.com/elastic/ecs-dotnet/pull/67

But besides those comments, then data arrives as expected in ElasticSearch ?

@johnjaylward Note that you can also use the NLog JsonLayout and built your own favorite structure (Instead of EcsLayout):
https://github.com/nlog/nlog/wiki/JsonLayout

But besides those comments, then data arrives as expected in ElasticSearch ?

Yes, it looks like it worked well once I got that fixed. I think if the Elastic.CommonSchema.NLog group is going to use those fields though, they aught to include the Elastic.Apm.NLog package in their NuGet dependency list. I'll bring that up over there as well. If I'm not using it directly, then I shouldn't have to worry about it.

Thank you for the testing and fast feedback.


Creation of Fbo with_depthbuffer=True hangs UI thread on iOS

This issue occurs on iOS specifically - it does not reproduce on other platforms.
When running a 3D OpenGL example, the creation of the FBO with_depthbuffer=True hangs the UI thread.  If this setting is set to false, then the UI thread does not hang, but the OpenGL rendering is broken (rendered flat), as one would expect.
The simple example, attached, uses the kivy3 wrapper for rendering 3D objects.  The UI thread hangs exactly at the point where the FBO is created, inside kivy3/renderer.py:

Here is the self-contained example code:
ios_3d_test.zip
Other notes:

The app does not crash, the UI thread merely hangs.
There are no exceptions thrown in the log when the issue occurs
Other background threads continue to run.

Any other information we can provide to help resolve this one?

If it's iOS, do you have a different behavior with Xcode running or when the app is in release?
If you can on XCode, can you try to pause the app when it hang, and get the traceback for all the threads?

Thanks, Tito for the reply.
It makes no difference if we are seeing it in dev mode or with the released app - it does the same.
Not sure how to do your second suggestion (pause and get traceback on all threads), can you suggest how to do this?  Thanks!

If you run it via Xcode, while running, you can go in menu Debug > Pause; wait a little bit, then you'll see all the backtrace for all the threads on the left.
Expand all of them, select everything, copy & paste here in a code block.

Thanks. The problem for us is that it hangs immediately - as soon as the Fbo is created, not after some period of time. But I will get the trace as @tito suggested.


Update browser-support.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/3286

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Checking with PM Neeraj Jain

NOT merging, per PM. "it overly complicates our offerings."

Ok, but it works in this way. Is there some plan to change it?


Multi-triggering in parallel, and choosing the best option

Triggering at like 3 locations per frame, separated by fractions of a period. Then pick the location with the highest peak score.
get period.

+0
+min(2/3 period, 1/4 frame)
-min(2/3 period, 1/4 frame)

Multi-triggering enables "choose better of integer and parabolic pitch tracking".

bump? idk

https://en.wikipedia.org/wiki/Overlap–save_method
use a wide slice of the document, with a narrower correlation kernel. this results in somewhat slower performance, but deterministic results.


fix quantization for onnxruntime v1.16.0

LGTM! Maybe we'll need to disable some tests or put ORT 1.15.1 in the CI for now

You're right! tests are disabled in https://github.com/huggingface/optimum/pull/1405/commits/b20ffa83677fe44d6a6be859577d383142a93876 and https://github.com/huggingface/optimum/pull/1405/commits/590f993ed63f6349620b68ee21857f9fcd33877e

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Bug: Dashboard flow runs select isn't informative on mobile

When on <md screens, tabs are correctly condensed to a mobile select but with the following options:

@zhen0  No I am not

Hmm!  Now you've replied I can assign you! ✅

Looks like the numbers in the drop down are also a bit off. We can add to a separate issue if easier for you Aman Sal.  Adding labels to the dropdown is the main scope of this issue.

Should now be resolved on smaller screens


Add default 'ORDER BY id ASC' to pagination if no other ordering is set

Because in relational databases there is no default order of records there is no guarantee that you pagination works without an explicit ordering.
See this post on Sequelize and specifically this post on Postgres.
Write a new function, possibly within a helper module, which does pre filtering of the three related find all arguments search, pagination, order. In case a pagination argument is supplied (!== undefined) and no order argument is supplied (=== undefined), set a default order argument ORDER BY id ASC.
Expected work time: 3h

Done. Commit #d96374


Android Basics: tip-calculator

https://developer.android.com/codelabs/basic-android-kotlin-training-tip-calculator?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-tip-calculator#6
In which task and step of the codelab can this issue be found?
It seems that the binding code line is incorrect: The id for the XML layout is cost_of_service_edit_text (it could be found on the same Calculate tip pathway) and the Binding section of the codelab writes the binding using: binding.costOfService.text. I think it has to be: binding.costOfServiceEditText.text. I had to edit it like that for my code run properly.
Thanks!

@RoloErre
Thank you so much for reporting this issue. This is fixed and will be published soon.
Here is the updated code:
fun calculateTip() {
    val stringInTextField = binding.costOfServiceEditText.text.toString()
    val cost = stringInTextField.toDouble()
    val selectedId = binding.tipOptions.checkedRadioButtonId
    val tipPercentage = when (selectedId) {
        R.id.option_twenty_percent -> 0.20
        R.id.option_eighteen_percent -> 0.18
        else -> 0.15
    }
    var tip = tipPercentage * cost
    val roundUp = binding.roundUpSwitch.isChecked
    if (roundUp) {
        tip = kotlin.math.ceil(tip)
    }
    val formattedTip = NumberFormat.getCurrencyInstance().format(tip)
    bin
```ding.tipResult.text = getString(R.string.tip_amount, formattedTip)
}


Add a keybinding to sync the list of known projects with search path

This scratched my itch as a custom keybinding, but it seemed generally useful.
The use case is that the contents of the projectile-project-search-path directory changes on disk (i.e. you add a new project your the terminal or file manager) and Emacs / projectile doesn't know about it until this command is run. I found it a common enough case to have a keybinding for. An alternative would be to have it run periodically in the background, but it's not clear how often, and typically once you add a directory, you wouldn't want to wait around for it to fire, just get working on the project.
Note, I considered 'pD' (for 'discover'), but that seems better suited to projectile-discover-projects-in-directory. I didn't add that (yet) though, as there is no room for it in the SPC-p panel, and I don't use it :-)

I've done it slightly differently in b72e00b, by writing a +default/discover-projects wrapper that will do projectile-discover-projects-in-search-path normally, but call projectile-discover-projects-in-directory if you supply the prefix arg. Then bound it to SPC p D, which should resolve this issue.
Thanks for bringing it to my attention!


Sync pse/halo2 0902

I think this can be merged.

closed for https://github.com/scroll-tech/halo2-snark-aggregator/pull/22


Repo sync for protected CLA branch

The pull request is created from main638048413079719949sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit 96fff39:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


[Bug]

the Bug
It Doesnt Works, I Opened the Installer as Administrator Outside The Downloads Carpets and it doesnt opens
To Reproduce
Open it as Administrator Outside or In The Download Carpet
Expected Behavior
Probably a New Discord Block?
Discord Version
Stable 59984
Additional Context
The Problem is in Windows Version
The Version launched at 16 Oct 2018 Can Be Launched but Doesnt Works on Discord

I don't understand the issue. You also seem to be citing a version from two years go...

The Issue Is... I Installed the last BetterDiscord Injector but it doesnt
Open, i tried launching it as administrator outside the Downloads Carpet,
but nothing Happened
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Libre
de virus. www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
El mar., 12 may. 2020 a las 17:38, Zack<EMAIL_ADDRESS>escribió:

I don't understand the issue. You also seem to be citing a version from
two years go...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rauenzi/BetterDiscordApp/issues/355#issuecomment-627612286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APR35YJ2EGR5CFOL6PCZ5ZTRRG6VLANCNFSM4M7GD6KA
.

I think there is a bit of a language barrier occurring. But I also think you might be using something very outdated. If you cannot provide versions, screenshots or any other definitive piece of information, then I cannot really help.

i Downloaded The Windows Installer of That Version, i opened it as administrator outside the downloads carpet, but the installer didnt opened, ill try the manual installation

the manual installation works!


Introduction to Application Performance Management (APM) and its significance

Proposed title of article
Introduction to Application Performance Management (APM) and its significance
Introduction paragraph (2-3 paragraphs):
The performance of computer application systems directly affects the output of businesses. For instance, a client getting irritated by slow application systems. Such clients are easy to lose when a company's service application system performs poorly. To prevent such financial and reputational losses, APM is used to ensure IT applications run smoothly and effectively. It is also used to maximize the utility of applications by, among other functions, detecting application issues that may cause poor performance or crushing of applications before it happens.
This article will enlighten application users with all there is to know about APM and its significance in financial production. It will focus on APM as a tool to measure performance and outline how the response time of applications is affected by peak load and how the applications use computer resources to get tasks completed. It describes the dimensions of APM in detail and outlines the challenges that may be experienced during its application.
Key takeaways:
The key take-homes of this article include but are not limited to;
•	Understanding of what Application Performance Management (APM) is and its purpose in IT applications
•	Knowledge of how performance metrics are used to measure the performance of applications
•	The ability to apply Application Performance Management (APM) in the identification of causes of delay in computer applications
•	Understanding of challenges that may be experienced in the implementation of Application Performance Management (APM), and knowledge of how to overcome them.
References:
n/a
Templates to use as guides
n/a

@Ernest715
seems like a helpful topic - lets please be sure it add value beyond what is in the official docs and that it does not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already) to avoid any potential article closure. - approved


Machine creates CA and certificate when just getting usage

$ docker-machine
INFO[0000] Creating CA: /Users/ben/.docker/machines/ca.pem
INFO[0000] Creating client certificate: /Users/ben/.docker/machines/.client/cert.pem
NAME:
   docker-machine - Create and manage machines running Docker.

USAGE:
   docker-machine [global options] command [command options] [arguments...]

...

This seemed a bit odd. I was thinking "what's it doing? I didn't ask it to do anything!"

Instead of tying this to a specific command, I set it as a "before" to the CLI app.  We could move it out but then would have to make sure to call it from each command in case the user specifies different keys.  Perhaps we just take off the info logging?

@ehazlett We won't need the CA and certificate until a machine actually gets created, right?  So why not tie it to Create?

Yeah we shouldn't.  I'll update.  Thanks.
On Thu, Jan 29, 2015 at 5:38 PM, Nathan LeClaire<EMAIL_ADDRESS>wrote:

@ehazlett https://github.com/ehazlett We won't need the CA and
certificate until a machine actually gets created, right? So why not tie it
to Create?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/432#issuecomment-72118982.

Closing - #460 merged


Compilation fixes for non x86 architectures

Two small fixes to allow OpenSFM to compile on non x86 architectures.
I'm trying to upstream all the fixes to allow OpenDroneMap run an ARM processors. For reference:
OpenDroneMap/OpenDroneMap/issues/484

Great @EmbeddedAndroid. I can't test on ARM so trusting you on that one. Thanks!

@paulinus thanks! compiled test on both ARM and x86!


Chocolatey package

FYI,
I maintain chocolatey package and it will keep all versions, with up to few hours latency after you release on Github:
https://chocolatey.org/packages/fzf
The update is automated and the report is available here: https://gist.github.com/majkinetor/a700c70b8847b29ebb1c918d47ee4eb1
Windows users can install fzf now with: cinst fzf or cinst fzf --version x.y.z.
The binaries are embedded in the package and will always work regarding the github status.
Cheers, and thanks for the great tool.

Thanks for letting me know, I'll add the link to the README page.


Modules should keep track of all their nodes et vice et versa

Each module should have an ObjectListField owned_nodes where all of its nodes are connected.
And each node should have a fake field module connected to its module message attribute.
I say "Fake field" because the created nodes are not instances of IcarusNode and therefore can't really have proper fields.

closed by 894e378edd07da4cd24d073663c1a778db4d9e17


Create test.txt

Fixes #
Proposed Changes

build now

build now

build now

build now

build now

Can one of the admins verify this patch?

build now


Remove async from metrics.

Discussion in #87 made me want to provide an option without any async processing.
I will leave that PR open in case it is decided that we would like async.

@pdewilde do you want to merge this or are you thinking of going in a different direction?


dependency broken?

compile 'it.enricocandino:androidmail:1.0.0-SNAPSHOT'
does not compile unable to resolve

Hi Daniel, which is the error that you got?

when i include " compile 'it.enricocandino:androidmail:1.0.0-SNAPSHOT' " in my app gradle under dependencies and i sync the gradle i get the error below
Error:(28, 13) Failed to resolve: it.enricocandino:androidmail:1.0.0-SNAPSHOT

Hm, did you add the mavenCentral repository to your build.gradle?
buildscript {
    repositories {
        jcenter()
        mavenCentral()
    }
    ...
}

allprojects {
    repositories {
        jcenter()
        mavenCentral()
        maven {
            url "https://oss.sonatype.org/content/groups/public"
        }
    }
}

I thought it was in the sample app but I was wrong.
It's better to specify this in the README.
Let me know if it was this the problem!

Yes that worked. thank you


Webview does not paste clipboard text correctly by pressing Shift+Insert

VSCode Version: 1.52.1 and earlier
OS Version: Linux Ubuntu 20.04
OS Version: Windows 10

The problem can be observed with any Webview panel.
Steps to Reproduce:

Checkout Webview sample: vscode-extension-samples/webview-sample
In file src/extension.ts add these two lines

@@ -199,6 +199,8 @@ class CatCodingPanel {

0

+                               Input:
+                               <input type="text" id="input-text">

Start Debugging of the extension and in the new window open command "Cat Coding: Start cat coding session"
Focus on the "Input" text entry, type something and copy the text into the clipboard. In my case it is "123".
Paste the text by "Ctrl+V". Observe that the inserted text is correct
Paste the text by "Shift+Insert". Observe that the text is inserted twice. Expected - the text is inserted once

Does this issue occur when all extensions are disabled?: Yes

I have submitted a PR #114520 that fixes this issue

I have submitted a PR #114520 that fixes this issue


Fix mask with from_awkward

Optimization so far failed when applying dak.mask when that mask that is applied was generated using something like dak.from_awkward.
The _get_column_reports method was not consistently dealing with the fact that AwkwardInputLayer.mock also returns a type tracer report.

I marked this as draft because there is another bug raising an error during optimization. The bug happens here https://github.com/dask-contrib/dask-awkward/blob/3ce8e4ef128f262509097d542a02061778528aa8/src/dask_awkward/lib/core.py#L488-L493
This sets the type of the last layer in the graph to the output type. The the case of mask and from_awkward, this will overwrite the type of the mask (which is supposed to be something like ## * bool) upon which awkward will complain that the mask needs to be bool or that it has the wrong structure.
I would also like to fix this but I cannot make sense of this code. What layer is supposed to have its _meta changed and why?

Hi @jrueb, thanks for the contribution! I'll dive in to looking at this at some point this week.

This looks like a hole in our interface with the way we've defined the mock method for layers (it has grown a bit organically as needed). We should probably make it such that any layer class that implements the mock method needs to return a two element tuple from that method.
@martindurant what do you think? Should AwkwardMaterializedLayer.mock return (new_materialized_layer, None) (instead of just new_materialized_layer)? I'm thinking we need to define mock's return signature to be:
class MyLayer:
    ...

    def mock(self) ->tuple[MyLayer, TypeTracerReport | None]:
        ...

And another question for @martindurant, do you happen to quickly recall the purpose of the code @jrueb quoted in his comment https://github.com/dask-contrib/dask-awkward/pull/272#issuecomment-1573299479? (I'm struggling to quickly remember, still thinking about it/tracking it down, but wanted to pick your brain just in case)

Here's what I have in mind
diff --git a/src/dask_awkward/layers/layers.py b/src/dask_awkward/layers/layers.py
index ad37585..e504a3a 100644
--- a/src/dask_awkward/layers/layers.py
+++ b/src/dask_awkward/layers/layers.py
@@ -28,7 +28,7 @@ class AwkwardBlockwiseLayer(Blockwise):
         nb = layer.numblocks
         layer.numblocks = {k: tuple(1 for _ in v) for k, v in nb.items()}
         layer.__dict__.pop("_dims", None)
-        return layer
+        return layer, None

     def __getstate__(self) -> dict:
         d = self.__dict__.copy()
@@ -211,7 +211,7 @@ class AwkwardMaterializedLayer(MaterializedLayer):
                 else v
                 for v in task
             )
-            return MaterializedLayer({(name, 0): task})
+            return MaterializedLayer({(name, 0): task}), None

         # failed to cull during column opt
-        return self
+        return self, None
diff --git a/src/dask_awkward/lib/optimize.py b/src/dask_awkward/lib/optimize.py
index bb8bfcd..dd0f83b 100644
--- a/src/dask_awkward/lib/optimize.py
+++ b/src/dask_awkward/lib/optimize.py
@@ -348,11 +348,7 @@ def _get_column_reports(dsk: HighLevelGraph) -> dict[str, Any]:
             layers[name], report = lay.mock()
             reports[name] = report
         elif hasattr(lay, "mock"):
-            mocking = lay.mock()
-            if isinstance(mocking, tuple):
-                layers[name], reports[name] = mocking
-            else:
-                layers[name] = mocking
+            layers[name], _ = lay.mock()

     for name in _ak_output_layer_names(dsk):
         layers[name] = _mock_output(layers[name])

Yes to the consistent interface, for sure.
Not certain about that code, but every layer we control should ideally have a _meta, and I think in this case it is being set for the first time. I don't follow why the DaskAwkwardArray's meta should disagree with the meta of its output layer.

I've just pushed a commit that fixes the interface; we still need to address the layer meta issue.

Codecov Report

Merging #272 (2bc187d) into main (b430e13) will not change coverage.
The diff coverage is 75.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #272   +/-   ##
=======================================
  Coverage   92.04%   92.04%
=======================================
  Files          21       21
  Lines        2400     2400
=======================================
  Hits         2209     2209
  Misses        191      191

Impacted Files
Coverage Δ

src/dask_awkward/layers/layers.py
95.34% <66.66%> (ø)

src/dask_awkward/lib/optimize.py
96.08% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I'd like to get this in because it fixes the mock interface. Regarding the failed columns optimization-- @jrueb would you mind opening a dedicated issue for that?


Update Aditya profile

@aditya-uw I think you also need to add a new page for yourself

This is closed by #33 !


Fix PropType Error in RN 0.48.xx (React 16.xx)

I migrated from React.PropTypes to PropTypes to avoid errors in React 16.x.x

Hi @xmarianox, thanks, but this issue is already being addressed in the 2.0 version: #165


Downloading Java JRE... Failed to download Java JRE. Reason: 0x38 "OpenSSL SSL_read: Connection was reset, errno 10054".

Downloading Java JRE...
Failed to download Java JRE. Reason: 0x38 "OpenSSL SSL_read: Connection was reset, errno 10054".

Are you able to access this URL? Looks like the problem with your internet connection
https://github.com/adoptium/temurin11-binaries/releases/download/jdk-11.0.12%2B7/OpenJDK11U-jre_x64_windows_hotspot_11.0.12_7.zip

it has been a 10 months and no other occurence of this. Should we close it ?

I agree on closing this issue


LibWeb: Implement transition property 🎉

We're still failing a lot of WPT tests, I think due to lack of events and some other OM things, but the basic functionality works pretty solidly.
There's a small hack in the included test. Without it, the reported left value after the transition from 0px to 200px completes, is 0px even though the div is visibly at 200px. I suspect this is a pre-existing bug but honestly I'm not sure. Our attempt at implementing the contradictory animation specs still confuses me. 😅
WPT Results Before
Ran 118 tests finished in 64.0 seconds.
• 14 ran as expected. 0 tests skipped.
• 2 tests crashed unexpectedly
• 2 tests had errors unexpectedly
• 16 tests timed out unexpectedly
• 97 tests had unexpected subtest results
WPT Results After
Ran 118 tests finished in 74.1 seconds.
• 18 ran as expected. 0 tests skipped.
• 4 tests crashed unexpectedly
• 3 tests had errors unexpectedly
• 16 tests timed out unexpectedly
• 90 tests had unexpected subtest results

CI spotted that I missed a visit(). I'll try and fix that up tomorrow.

Fixed the missing visit().

Amazing!! Thanks for all your hard work here, I'm very happy to see transitions finally ready to go 😄

Amazing!! Thanks for all your hard work here, I'm very happy to see transitions finally ready to go 😄

Thank you for doing so much if the work originally! This stuff is hard. 😅


YAML configuration output is not correctly indented

Proposal
I'm often terrorised by incorrect YAML indentation and I noticed what Prometheus outputs in /config isn't perfect strictly speaking.
Bug Report

What I did: Copy the configuration shown on /config

Expected: A valid YAML document

What happened instead: yamllint 1.12.1 strictly complain about the document

Prometheus version: 2.5

https://s.natalian.org/2018-11-09/yaml-indentation.mp4

Is it invalid YAML, or is the linter being over-sensitive?

Though not complying with the linter's rules, the YAML exposed by Prometheus is perfectly valid. There isn't much we can do here as we rely on an external library for YAML serialization/deserialization.


[credscan][file,queue] Fix warnings

Replace the secret-looking signed identifier ID with a UUID
Fixes issue #25479

API change check
API changes are not detected in this pull request.


Update wifi_odin_v4.json

Similar change to ESP config file - make it easier to distinguish between variable name and value

@ranshe no PR will be accepted to public repositories as we only deliver releases here. Please open a PR to the internal repository or discuss with us. Personally I would not do this change as this has been the common format since mbed-os-example-client.

We had two workshops with prospects a few weeks ago.
In both workshops we had prospects edit the wrong line, so I guess the fact that "this is how things always were" doesn't mean it's the right way to do it.
Anyway, where is that internal repo?

https://github.com/ARMmbed/mbed-cloud-client-example-internal


Adding: Ilan Gold

Onboarding checklist
Welcome @ilan-gold!

[ ] add to website
[ ] add to<EMAIL_ADDRESS>google group. Note: the email address used here will also get access to the shared google drive and calendars.
[x] add to zulip

[x] @core team
[x] core-team private channel

[x] Grant "Leader" trust level on discourse
[x] add to password manager
[x] add to github org and scverse/core github team
[ ] announce the new member on twitter

Duplicated now with #70


fix product not appearing in the search after removal

Description
Address bug where instead of updating the investigation index after removing the product, the product search index was removed.

 Remove product from a case not suppose to delete product from the db


Added a new developer guide on installing Python with Pyenv. Changed a few titles in other guides.

@bryanfriedman @BrianMMcClain Can we get this new Python guide out? I'm keen to fill out the Python page for our new readers :D


Update fake master job

I need to make this optional in the Jenkinsfile so the test job will be able to either test that fake-master can be rebased (merge-and-build.sh) or sync fake-master with enterprise-3.5 (update-fake-master.sh).
@stevekuznetsov @jupierce @tdawson @bbguimaraes

@kargakis: mostly minor things, overall looks good.

@bbguimaraes thanks, tested locally with the new changes, works fine.


Emoji Display Error

Hello, I loaded up my old Tweetdeck today. Everything else is working perfectly fine, but it seems that emojis their next peoples usernames are now going on a line underneath instead of being displayed next to them like usual.
This isn’t a major issue so you don’t have to work on it immediately

Can't reproduce, emojis work fine for me


Setting calendar to current month

Can you tell me how I can get the calendar to display in the current month on load?
Thanks

Sorted this. Edit the app.js file and change the beginning to
\This section generates todays date so that it can be used in the JSOn construct to change the hardcoded day value.
var currentDate = new Date();
var twoDigitMonth=((currentDate.getMonth()+1)>=10)? (currentDate.getMonth()+1) : '0' + (currentDate.getMonth()+1);
var twoDigitDate=((currentDate.getDate())>=10)? (currentDate.getDate()) : '0' + (currentDate.getDate());
var createdDateTo = currentDate.getFullYear() + "-" + twoDigitMonth + "-" + twoDigitDate;
var todaysdate = createdDateTo;

"use strict";

var options = {
	events_source: 'events.json.php',
	view: 'month',
	tmpl_path: 'tmpls/',
	tmpl_cache: false,
	day: todaysdate,


java-openliberty:  appsody build should explore more caching of maven dependencies

Is your feature request related to a problem? Please describe.
In appsody build for the java-openliberty stack, we are basically populating all of the image's .m2 repository for any app change at all.
Describe the solution you'd like
This could be as simple as copying the pom.xml into an image layer, then running some mvn command.    Or we could do more...?
Describe alternatives you've considered
Additional context
PR #485 has an improvement for a similar problem for the java-microprofile stack.    That (and the linked/referenced issues) might be useful to read through.

Forgot we had this issue open.  Closing as fixed by #626.


How to use Android to scan and log in?

How to use Android to scan and log in? please help me ，thanks！

same.. i dont know how to login in 1 device

Print QrCode and scan with your device.

You need to ask the API to give you the Base64 QR code, then pass it via socket to your html page using sket.io
    ```

go func() {
sessionID := params.SessionID.String()
handler, err := wa.Login(qr, params.ProxyURL, sessionID)
if err != nil {
errCh <- err
}
wa.Connections[sessionID] = handler
}()
    case qrText := <-qr:
            return profile.NewScanQrOK().WithPayload(&models.QRCode{
                    Base64: qrText,
            })

    }

thank you，i will try it，hope it works

Hello，I still don't understand, can you add a demo, such as login, login through Android phone code scanning, I am very urgent need, the above code do not know how to use, please help me, thank you  @beshoo @Rhymen   thank you very much

Hello，I still don't understand, can you add a demo, such as login, login through Android phone code scanning, I am very urgent need, the above code do not know how to use, please help me, thank you  @beshoo @Rhymen   thank you very much

You can use example code. Login and send Message.

thank you ，The demo can only be scanned with iPhone. I don't know how to scan and log in with Android phone. Would you like to help write a demo with Android phone scanning code to login and send messages，thanks very much！

thank you ，The demo can only be scanned with iPhone. I don't know how to scan and log in with Android phone. Would you like to help write a demo with Android phone scanning code to login and send messages，thanks very much！

NO WAY,You can scan the code via android and iPhone! it is not related to the device type!


Improve minor things in scripting-docs/jswinrt/handling-windows-runtime-events-in-javascript.md

This serie of tiny commits improves some minor things (syntax highlighting, clearer wording, …) in scripting-docs/jswinrt/handling-windows-runtime-events-in-javascript.md
Please see the individual commit messages for more information.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

scripting-docs/jswinrt/handling-windows-runtime-events-in-javascript.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@PaulCapron : Thanks for your contribution! The author, @mikejo5000, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

scripting-docs/jswinrt/handling-windows-runtime-events-in-javascript.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off


Proposal to rename master to main. Add dev branch.

[ ] rename master to main in settings
[ ] create dev branch from main
[ ] update CI settings

👍 on the rename.
Probably the same on update CI but not sure what it means.
As for a 'dev' branch: I suggest we do not create a long-lived "dev" branch. I know it's part of the popular Git Flow model and I can see the utility for a time when there is a mature workflow with releases that are coordinated and changed across teams; however, it can cause confusion about where features reside, and it adds overhead of a group decision and a manual merge to get that work to the main branch. I would say the cost isn't worth the extra effort at this stage.
I suggest people feature branches for each new set of work, similar to GitHub Flow. After releases, if there are smaller stability changes that diverge from main then the group can create a release branch. The fewer shared branches, the simpler the coordination.

@trentlarson thank you for the follow up. I'll make those changes


docs: update timestamping config

Add more details of TSA configuration.
Decouple new feature usage from the default configuration.
Fixes: https://github.com/ratify-project/ratify-web/issues/114

Nice work @junczhu @binbin-li . Next we should be able to generate a versioned doc for v1.3.0


DeepLinking to Apps which are not Installed yet?

Is it possible to DeepLink to Apps which are not Installed yet?

@confile  my app uses the scheme "CQ://" which is registered when installing the app.
if the app wasn't installed, iOS wouldn't know how to handle "CQ://"


Char and varchar can return Buffer

I don't see it documented but both CHAR and VARCHAR can return a Buffer if they have CHARACTER SET binary set. Is this intentional?
I opted for CHAR(60) CHARACTER SET binary instead of BINARY(60) specifically so that I would get strings instead. My use case is this: I'm storing the hash from bcrypt which is 60 characters ascii. I don't want the database to assume anything about the content and I don't want it doing case-insensitive matching, so therefor I choose CHARACTER SET binary.
Should I be rethinking this or should the library be updated?

Yes, this is intentional, unfortunately. You can always define your our typeCast to override this, but when I created the same type of column for my own stuff, I have always used the following: CHARACTER SET ascii COLLATE ascii_bin.

Thanks, I've updated my database to that. Do you think that it would be relevant to add this to the readme? Feel free to close the issue otherwise...

Yes, I agree docs should note this.

:+1: nice!