Random NTSTATUS when calling user32.dll!ConsoleControl under WoW64

Windows Terminal version
No response
Windows build number
10.0.19045.3086
Other Software
No response
Steps to reproduce
Build the following code for 32-bit target and run on 64-bit host
#include <iostream>
#include <windows.h>

typedef enum _CONSOLECONTROL {
    Reserved1,
    ConsoleNotifyConsoleApplication,
    Reserved2,
    ConsoleSetCaretInfo,
    Reserved3,
    ConsoleSetForeground,
    ConsoleSetWindowOwner,
    ConsoleEndTask,
} CONSOLECONTROL;

typedef struct _CONSOLEENDTASK {
    HANDLE ProcessId;
    HWND hwnd;
    ULONG ConsoleEventCode;
    ULONG ConsoleFlags;
} CONSOLEENDTASK, * PCONSOLEENDTASK;

typedef NTSTATUS(WINAPI* ConsoleControl)(
    _In_ CONSOLECONTROL Command,
    _In_reads_bytes_(ConsoleInformationLength) PVOID ConsoleInformation,
    _In_ DWORD ConsoleInformationLength
    );

int main()
{
    auto user32_dll = ::LoadLibraryExA("user32.dll", nullptr, LOAD_LIBRARY_SEARCH_SYSTEM32);
    auto ConsoleControl = (::ConsoleControl)::GetProcAddress(user32_dll, "ConsoleControl");
    auto count = DWORD{};
    auto rec = INPUT_RECORD{};
    auto task = CONSOLEENDTASK{ .ProcessId = (HANDLE)::GetCurrentProcessId(), .ConsoleEventCode = CTRL_C_EVENT };
    std::cout << "Press any key to generate event. Press Esc to exit.\n";
    while (true)
    {
        ::ReadConsoleInputW(::GetStdHandle(STD_INPUT_HANDLE), &rec, 1, &count);
        if (rec.EventType == KEY_EVENT && rec.Event.KeyEvent.bKeyDown)
        {
            if (rec.Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) return 0;
            auto status = ConsoleControl(ConsoleEndTask, &task, sizeof(task));
            std::cout << "status: 0x" << std::hex << status << "\n";
        }
    }
}

Expected Behavior
Output
Press any key to generate event. Press Esc to exit.
status: 0x0
status: 0x0
status: 0x0
status: 0x0

Actual Behavior
Output
Press any key to generate event. Press Esc to exit.
status: 0xb94a7
status: 0xb94ca
status: 0xb94db
status: 0xb94e6
status: 0xb955a
status: 0xb956b
status: 0xb9572
status: 0xb95fc
status: 0xb9603
status: 0xb962b
status: 0xb969b

Now that we've documented it, we have to reckon with all the weird corner cases! Alright, so, here goes.

ConsoleControl was never specified for use in a WoW64 context. I will update the documentation to make this clear.

These APIs were designed when the console host was moved out of csrss and was guaranteed to be locked to the architecture of the kernel. That requirement hasn't changed, unfortunately. Even for OpenConsole built from this repository.

ConsoleEndTask contains some authentication checks to ensure that the caller holds the console (condrv, really) Server handle for the given process. It will not work in isolation.

Now, the interesting thing for you is that ConsoleControl implements ConsoleEndTask in the client application by brokering over to csrss, and all other console controls by passing them along to win32k. It's very likely that the bitness problem is unsolveable, because ConsoleControl re-packs the csrss message locally before sending it along.
Right now, this is by design. The docs need to be updated, however. :smile:

I want to clarify two points

If we step down a level and use ntdll.dll!CsrClientCallServer everything works fine. We are using it now as a workaround for this issue.
Every time I observe the processor instruction pointer at random point in memory after exiting the 64-bit mode inside the ConsoleControl call on WoW.

ConsoleControl implements ConsoleEndTask in the client application by brokering over to csrss, and all other console controls by passing them along to win32k. It's very likely that the bitness problem is unsolveable, because ConsoleControl re-packs the csrss message locally before sending it along.

I don't see any reason why the ConsoleControl can't detect that it's running under WoW64 (::IsWow64Process2(::GetCurrentProcess()) and repackage the csrss message accordingly for the 64-bit architecture. Now we are forced to work around this using a low-level undocumented API.

Right now, this is by design.

Ok. Fine. This is a WT design. Windows fundamental design provides support for running 32-bit programs on 64-bit Windows. Moreover, 32-bit terminal emulators function perfectly under WoW64 seamlessly operating both 32-bit and 64-bit applications. The only misbehaving component is user32.dll!ConsoleControl, which is a high-level primitive wrapper that appears to have a buffer overflow and subsequent stack corruption that corrupts the return address.
FYI, this all works smoothly, regardless of whether it is a 32-bit or 64-bit executable, terminal, or application in any combination:

If I may - why are you using this API in the first place? To what end? I just wanna know a little more about how it's being used in the wild

Our terminal is hosting a console application and calling user32.dll!ConsoleControl is a way to tell the application about a Ctrl+Brake, Ctrl+C, Close, Logoff, Shutdown event. That is, asynchronously call their custom handler for these events. This is the same as SIGHUP/SIGINT/SIGTERM etc.

@zadjii-msft In short, @o-sdn-o has written a conhost clone and integrated it directly into vtm. That's why they need this API: Can't have a complete conhost without Ctrl+C functionality. 😅 vtm is obviously not feature-complete, but it works under the most popular situations already, as shown above, which is super impressive IMO. But a 32-Bit vtm doesn't work on a 64-Bit host, because conhost was always tied to the bitness of the host and so was any API it ever used.

But a 32-Bit vtm doesn't work on a 64-Bit host

32-bit vtm works fine on a 64-bit host, but for this we have to use undocumented APIs to workaround the bug in used32.dll!ConsoleControl.

Yep, that's what I wanted to convey with that sentence. 🙂
I should probably add that my previous comment is neither in favor of nor against fixing the API. Personally speaking, I'm mostly impartial about it. I share both, your concern about it being a public user32 API now, and Dustin's expression of it being an API that's simply tied to the host-bitness, because it always was.

I'm not really worried about using an undocumented API, but that it would be nice to be able to use the official API where possible.

Just to level-set and perhaps dispel some misinformation:

This is a WT design.

ConsoleControl was authored some time before Windows NT 3.51 was released. Almost nobody on the team was capable of high-level decision-making during the time at which this once-internal API was created, on account of many of us being children. It was not designed specifically for Windows Terminal, and it was not even designed for conhost (which did not exist at the time.) I promise you we're not just out here shipping private APIs to benefit our own first-party open-source terminal emulator!
Unfortunately, exposing the weird internals of our APIs and how they are generally broadly incompatible with the rest of the public API surface is part of the territory. That's something we could get better at!
I am not against fixing this API. We documented it publicly, and it should work for anyone who needs it. However, it is going to have to come in alongside "document the driver interface" and "make the driver interface WoW64-compatible".  Thanks!), and I appreciate you telling us about the problems you've encountered.
I am also not saying that it is hard to fix. The NT APIs don't typically use IsWow64Process and friends, opting instead for something lower-level, but it's definitely doable (and probably easily doable.)
However, if we do fix ConsoleControl: we don't have a business justification for servicing it to earlier versions of Windows, so we're probably only going to be able to fix it for whatever the next full OS swap version of Windows is. That's probably not going to help you over there on 10.0.19045. I'm sorry about that.

We've talked about it a bit more offline and agreed to put this on our backlog. 🙂
We'll almost assuredly not backport such a fix, because we simply can't (as Dustin explained above). But fixing it in Windows 12 is better than not fixing it at all.

Disclaimer: I've never used these APIs and am reading about them just now.  All comments are firmly in the category of "interested observer."
What would the fix be?
If somebody told me to use this API tomorrow, I'd query the bitness of the kernel, and pass either a 32 bit or 64 bit structure based on which kernel my program runs on.  But if I did that, a later kernel side change that changes the interpretation of the structure would break my program.  My program needs to know which structure the kernel is expecting.  Since the fix can't be atomically backported to all systems in existence, programs need to face this inconsistency.
I'm wondering if the fix requires defining new information classes for the new structure interpretation.  When run on an older kernel, these would always fail as opposed to being misinterpreted.


Added readme changes for subtitle work

📺 What
Update readme

Tickets: IPLAYERTVV1-11997

🛠 How
Update readme

wiki changes have been already - https://github.com/bbc/bigscreen-player/wiki


Devfile registry library doesn't support registries using self-signed certificates

Describe the bug
The devfile registry library doesn't seem to support registries that are using self-signed certificates. Using the branch of odo in https://github.com/openshift/odo/pull/4525, I tried to list devfiles in my own devfile registry, that's signed with self-signed certs:
johncollier@jcollier-mac registry % odo catalog list components
 ⚠  Registry TestRegistry is not set up properly with error: Get "https://devfile-registry-default.apps.ci-ln-s8jsgq2-002ac.ci.azure.devcluster.openshift.com/index": x509: certificate signed by unknown authority, please check the registry URL and credential (refer `odo registry update --help`)

There should be an option to skip TLS verification when retrieving the index.json and retrieving devfile stacks. We will also need this for the registry tests, as it will be setting up a registry with self-signed certs.

This could be a implementation of skipping TLS verification: https://stackoverflow.com/questions/12122159/how-to-do-a-https-request-with-bad-certificate


Session not saving

I am storing a session in a websocket event but then when I try to use it again in an HTTP request it is empty. I printed out the sessions in the websocket and the subsequent HTTP request. After filling the session and printing it, it is a dictionary, but in the http request, it is <FileSystemSession {}> I linked an image. I am using Windows. It seems to store with http requests but not websocket ones.

Do you have a minimum reproducable example I can try? When I use websockets it stores the session fine

Closing due to not reproduceable


Support deploying provision probe or mount probe only

Currently, when a user deploys a PieProbe object, both of provision and
mount probes are deployed. However, under some circumstances, it is
enough to deploy either provision probes or mount probes only.
This commit supports deploying either provision probes or mount probes
only, by adding disableProvisionProbes and disableMountProbes fields to
PieProbe spec.
Signed-off-by: Ryotaro Banno<EMAIL_ADDRESS>

Thanks for your feedback. I fixed my code to use the names disableProvisionProbe and disableMountProbes only.


BigDecimal E notation (scientific) string parser bugs, (& proposed fixes)

Hi all, and thanks for all the amazing work on Crystal! This is my first foray into crystal so let me know if there is anything wrong/mis-understood.
I noticed BigDecimal#initialize parses E notation numbers from strings incorrectly in some cases. I believe the issues could be boiled down to the following types of problem:

e1 : raises Invalid BigInt: e1 (ArgumentError)
1e- :  raises Invalid UInt64:
1e. :  raises Index out of bounds (IndexError)

ie errors in parsing logic and some leaking of underlying implementation detail (more examples can be seen in the specs in the branch compares below).
Below are a couple of proposals for fixes.  Let me know if a PR of either works and I can finish up and open one up.
Fix proposal 1: fixes on existing implementation
One option is simply try to fix the existing implementation but the parse logic seems a little ad-hoc:
https://github.com/crystal-lang/crystal/compare/master...stevegeek:fix_e_notation_big_decimal
Fix proposal 2: more robust parser
Maybe a more robust parser implementation is worth the effort? Eg I started writing up this EBNF-ish grammar below:
<scientific> :== [<sign>] <number>
<number> :== <digits> [ <fractional-optionaldigits> ] [ <exponent> ]   |  <fractional> [ <exponent> ]
<fractional-optionaldigits> :== '.' [ <digits> ]
<fractional> :== '.' <digits>
<exponent> :== <exp> [ <sign> ] <digits>
<digits> :== <digit>+
<digit> :== '0' | '1' ... | '9'
<exp> :== 'E' | 'e'
<sign> :== '+' | '-'

then created the productions below (syntax below works in http://hackingoff.com/compilers/ll-1-parser-generator)
Scientific -> OptionalSign Number
OptionalSign -> s |
Number -> Digits OptionalFractional OptionalExponent | Fractional OptionalExponent
OptionalFractional -> '.' OptionalDigits
Fractional -> '.' Digits
OptionalExponent -> e OptionalSign Digits |
OptionalDigits -> Digits |
Digits -> Digit RepeatDigit
RepeatDigit -> Digit |
Digit -> n

Here is a proposal that implements a first pass at this (not stack based to avoid needing a stack, just a translation of the parse table into logic):
https://github.com/crystal-lang/crystal/compare/master...stevegeek:feature_e_notation_big_decimal_parser
Lastly: Questions
I would also like to ask (not sure if this issue is the correct place... apologies if not):

Is InvalidBigDecimalException necessary?

Would ArgumentError suffice? There doesn't seem to be equivalent exception subclasses for other Big types. In this I use an ArgumentError and a bit more helpful message and avoid the expensive to_s expansion.
If it is desirable to have an explicit exception subclass, should it not rather be named an InvalidBigDecimalError for consistency? I see most Exception subclasses in stdlib are named with that convention

the default string representation for very long numbers should I believe be in E notation... ie BigDecimal.new("1E100").to_s should output "1E100" rather than  "10000...000" (100 zeros) (partly related to #7808) which can become very expensive (in other langs, both Ruby and Python convert decimal to strings in E notation, as does JS for large numbers). At least this representation expansion should be documented if it happens by default?
I also note overflowing exponent values will raise Invalid UInt64 exceptions (e.g BigDecimal.new("1e18446744073709551616")). Maybe this is an implementation detail which could be more neatly wrapped in a ArgumentError/InvalidBigDecimalException on parsing?

@stevegeek Thank you for the bug report!
I suggest cleaning up the text in the issue to just mention the parsing bug, then creating separate issues for each of the things that you'd like to discuss. Otherwise it will be a mess of discussion, trying to tackle many things at once. For example the output of BigDecimal#to_s has nothing to do with the bug reported here. Improving the parsing using a grammar is also good, but in #9552 I could fix it without doing any of that so I would extract that as a "feature" or RFC.
Thank you!

Same goes for whether to use a specialized exception or not. In my opinion it should just be ArgumentError.

@asterite Ok, will do! The proposed fixes from https://github.com/crystal-lang/crystal/compare/master...stevegeek:fix_e_notation_big_decimal do tackle the changes without any immediate big changes, I will have a look at #9552 and compare! The RFC idea sounds good too, will create tickets accordingly soon

@stevegeek Oh, sorry, I missed those changes. Next time I recommend directly sending a PR with them, it's much easier to track, review, etc.

hey, just wanted to say I haven't abandoned this, will get on and sort out as soon as I can!


How to use these well trained model

I appreciate your work, but I have some small questions.
The well-trained model you gave on the homepage is the encoder or encoder plus Linear?
How should I use these models?

Hi. the checkpoints that we uploaded are just a copy of the model at the end of the training, so they contain all the parameters that are needed to resume training (i.e. the encoder, the projector and/or predictor, and the classifier). Note that the linear classifier contained in the checkpoints is trained online with the encoder. If you want to train the classifier offline, we have plenty of bash scripts for doing that in here.
The checkpoints can be loaded using PyTorch or even just tested using PyTorch Lightning. Here are some useful resources:

loading models with PyTorch
loading models with PyTorch Lightning

Note: if you are loading the weights manually, you can find the parameter groups of the model under the key state_dict.

Just to complement @DonkeyShot21, if you can check part of our code in main_linear.py that loads the encoder part:
https://github.com/vturrisi/solo-learn/blob/7abc7feb6120b02bb5a9ef372a474308f4a1b1c8/main_linear.py#L27-L51

I'll close, but if you have any questions, feel free to reopen the issue. Thanks for trying solo-learn :)


Is time absolute or relative to scheduler start? / When will a job run the first time?

Is time absolute or relative to scheduler start?
IOW, is the first time that an added job is run immediately when the scheduler starts, or the next absolute time?
E.g. if I add this job to run every 5 minutes, will it run the first time when the minute part of the UTC time is a multiple of 5 after this (e.g. if I add the job at 14:04, will it run at 14:05?), or will it run first 5 minutes from the time when I add the job to the scheduler, or will it run immediately and then every 5 minutes from when I added it?

The schedule runs at absolute times, just like cron. It doesn't care when you start it.
0 0/5 * * * * * = 14:00, 14:05, 14:10, etc.
You can also do offsets.
0 1/5 * * * * * = 14:01, 14:06, 14:11, etc.


Fix compatibility with rules_proto 7

Changes are visible to end-users: no
Test plan

Manual testing; please provide instructions so we can reproduce: Bump rules_proto version to 7.0.2 and run tests.

@alexeagle Could you cut a new release with this change? That would help make rules_go compatible with AEGs.

To github.com:aspect-build/toolchains_protoc.git
 * [new tag]         v0.3.4 -> v0.3.4


Gradient calculation is now working

Code generation for gradient calculation is now working.

Great stuff, new functionality and increased readability! Travis is also happy, so I think it's good to go.


feat: Use compact 5424 template by default

Use the compact 5424 template with "-" place holders for host and timestamp to conserve bytes by default

:tada: This PR is included in version 2.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Sunstone template instantiate needs CREATE permissions

Description
When you try to instantiate a template using Sunstone it asks for CREATE permissions, when it just needs USER permission.
To Reproduce
Try to instantiate a template, with the following ACL:
109       #3     V-NI-------S-R---  @110     umac     *
111       #3     V-NI--G----SvR---     *     -m-c     *
112       #3     -----T-----------     *     u---     *
Details

Affected Component: Sunstone
Hypervisor: KVM
Version: development

Problem
Sunstone sends the contents of the template to replace the existing one, the new template is send to include the input from the user.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

We have tried with the commented ACLs and if the VM is instantiated, in the recent versions (5.10), if you encounter this error again, please explain the steps to replicate it


feat(client): Various improvements to config operations

This includes 2 improvements to config operations.
Replaces #3798
Note that this version is not manually tested by me, as it is not compatible with my cluster. As the changes are in code that seemingly didn't change since 1.6.1 I don't expect any troubles, but you never know...

@nathansamson Are you able to make the change @bacongobbler requested? Thanks for taking the time to submit this!

@carmstrong done. Somehow I had missed the comment

Highlighting the config that actually changed is nice. (I added asterisks to indicate where bold green ANSI highlighting was applied.)
$ deis config:set FOO=bar
Creating config... done, v2

=== outlaw-aqualung
*FOO*    bar
$ deis config:set BOO=far
Creating config... done, v3

=== outlaw-aqualung
*BOO*    far
FOO      bar
$ deis config:set FOO=baz
Creating config... done, v4

=== outlaw-aqualung
BOO      far
*FOO*    baz
$ deis config:list
=== outlaw-aqualung Config
BOO      far
FOO      baz
$ deis config:set FOO=bax BAX=foo
Creating config... done, v5

=== outlaw-aqualung
*BAX*    foo
BOO      far
*FOO*    bax

If this PR could use the termcolor library as suggested I think it's ready to go. Something like:
self._logger.info("{} {}".format(colored(key, 'green'), v))

s/green/magenta. Deis needs more pink! (kidding)

The second commit should use a lower-case verb as per our code standards:
feat(client): highlight changed config

Hi,
I will close this pull request in the hope that @Joshua-Anderson will include something similar in #4015
Otherwise I might try a third attempt in the new codebase
Regards,
Nathan


Test global variables initialized when using an imported package

Description:
After building and pushing the package to ballerina central, the test.bal inside tests package also get pushed to central. Because of that, when importing the package and executing, the test.bal also gets initialized. If we have defined global variables inside tests, user will get execution errors.
Steps to reproduce:
Define a global endpoint inside tests/test.bal file and build the package and push it to central.
import ballerina/io;
import ballerina/http;
import ballerina/log;
import ballerina/test;
import ballerina/config;

string url = setConfParams(config:getAsString("ENDPOINT"));
string accessToken = setConfParams(config:getAsString("ACCESS_TOKEN"));
string clientId = setConfParams(config:getAsString("CLIENT_ID"));
string clientSecret = setConfParams(config:getAsString("CLIENT_SECRET"));
string refreshToken = setConfParams(config:getAsString("REFRESH_TOKEN"));
string refreshTokenEndpoint = setConfParams(config:getAsString("REFRESH_TOKEN_ENDPOINT"));
string refreshTokenPath = setConfParams(config:getAsString("REFRESH_TOKEN_PATH"));

endpoint Client gMailEP {
    oAuth2ClientConfig:{
        accessToken:accessToken,
        baseUrl:url,
        clientId:clientId,
        clientSecret:clientSecret,
        refreshToken:refreshToken,
        refreshTokenEP:refreshTokenEndpoint,
        refreshTokenPath:refreshTokenPath,
        clientConfig:{}
    }
};

function setConfParams(string|() confParam) returns string {
    match confParam {
        string param => {
            return param;
        }
        () => {
            log:printInfo("Empty value, found nil!!");
            return "";
        }
    }
}

Create a new project and import your package as wso2/package-name. Simply run a main function, and you will get execution errors due to global variables/endpoints defined inside test.bal in imported package.
import ballerina/io;
import wso2/gmail;

public function main(string[] args) {
    io:println("Hello, World!");
}

Affected Versions:
ballerina-tools-0.970.0-alpha3-SNAPSHOT
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Hi Dushani,
For now please don't push the test package. When you are building to push, please remove test package.
thanks,
Dimuthu

When building the package to be pushed to central using "ballerina build", we don't include the tests inside the package to  the generated balo in the latest master. Since this issue is fixed closing this issue.


Update Python support to reflect coming 3.7 EOL

Python 3.7 end of support is 2023-06-27.  Some projects case-utils includes in its dependencies, most recently NumPy, are ending their support for Python 3.7 ahead of this EOL.
Rather than hold case-utils back for another 6 months, this patch removes 3.7 from the test base, and will encourage downstream adopters to do the same, so updates for imported packages continue to be retrieved.

This should be merged once CI finishes.


Compatibility with ethers-v6

Library installs and works in a project that is using ethers-v6.
At the moment, installation gives the following error:
Exit code: 127
Command: typechain --target ethers-v5 './src/abis/*.json'
Arguments:

@diego-G I forked this repository and made many fixes for ethers-v6. It will get past the error you are encountering, but I highly doubt using it is safe - tests don't run much less pass. There is just too much code in this repo to fix which depends on ethers-v5 (tx.wait(), Bignumber.from(...), ContractReceipt, etc.) for me to fix right now. But I will probably get the parts I need working before the LINK team does.


fields: make Ptr a deref API instead of a field API

Alternative to https://github.com/sourcegraph/log/pull/14 which I think is better IMO, so that we can avoid zap.Any, keep the readability of the API to see what type something is, and avoid potential confusion where some field types are generic but others are not.
	logger.Info("ptr",
		log.String("str", log.Ptr(strPtr)),
		log.String("nilStr", log.Ptr(nilStrPtr)))

Codecov Report

Merging #15 (d523849) into main (9e50e76) will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   57.48%   57.79%   +0.31%
==========================================
  Files          14       14
  Lines         541      545       +4
==========================================
+ Hits          311      315       +4
  Misses        223      223
  Partials        7        7

Impacted Files
Coverage Δ

fields.go
53.33% <100.00%> (+16.96%)
:arrow_up:

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 9e50e76...d523849. Read the comment docs.

@jhchabran @sanderginn IMO in that case, we should just introduce the appropriate StringPtr etc and friends explicitly

Ack, let's circle back there then, my bad, I missed your point with the original suggestion :)

Thanks for the discussion everyone! ❤  I've opened up a revert in https://github.com/sourcegraph/log/pull/16 for now


Add promptType Conversation

@Samoed, it seems like we might accidentally have removed the encode_conversations without a replacement. This seems like a bug.
(Checking in with @vaibhavad here as well, ensuring that encode_conversations() is required - we now denote these using the promptType e.g. "query" and "corpus".

I tried to leave it as it, but in models folder there weren't models that implement this function to test it and look for implementation.
https://github.com/embeddings-benchmark/mteb/blob/main/mteb/evaluation/evaluators/RetrievalEvaluator.py#L346-L349

@vaibhavad do you have a model that implements an encode_conversations() method?


Update endpoint IP selection order based on address type found

Issue #, if available:
Description of changes:
Currently, we set the etcd endpoint based on the address set on machine status and give preference to InternalIP/DNS over externalIP/DNS. CAPV introduced a change in which they set the machine address of type InternalIP to be the vm name. This breaks the endpoint check logic we currently have as we would not have a DNS entry for the corresponding endpoint. In this change, we record all the address types found in the machine status and take a precedence based approach in which DNS entries take precedence over IPs and external type takes precedence over internal. We are excluding HostName address type as it would again fail in our connection check.
Testing:
Built the controller and tested manually to be able to create clusters with CAPV 1.9.3
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Disable all drawers in non mobile resolutions

I have a responsive app with different style in mobile resolution and desktop resolution minor than 768px, how I can disable all drawers (left and right) in resolution minor to 768px?.
I have a directive for check my resolution like this:
angular.module('msx').directive('mobileScreenSwitcher', function ($window, snapRemote) {
    return {
        restrict: 'E',
        link: function (scope) {

            $window.onresize = function () {
                checkResolution();
            };
            checkResolution();

            function checkResolution() {
                var screenWidth = $window.innerWidth;
                if (screenWidth >= 768) {
                    // I want disable all drawers in this line.
                    snapRemote.globalOptions.disable = 'left';
                }
            }
        }
    }
});

Take a look at #50.

+1...


nixos/hound: adopt, rework, cleanup

Description of changes

add me as maintainer
remove hardcoded user id
validate syntax of config file
remove superfluous option extraGroups
use mkPackageOptionMD

Tested on my deployment without config changes and everything continue to work.
I do not think this warrants a release not entry as existing systems should continue to work and if the config file was not valid json before, hound would just be crash looping.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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

[x] 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/)
23.11 Release Notes (or backporting 23.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.

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

Thanks!


Fix link to Stackoverflow tag and link.

@sjwaight : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; verified. Thanks for this fix @sjwaight.


bug/#24-Fix azure frontend deployment

Fixes the frontend deployment of Doto to Azure.

[ ] The pull request is complete according to the following criteria:

[ ] Acceptance criteria have been met
[ ] The documentation is kept up-to-date
[ ] Comprehensive tests (if applicable) have been generated and all pass.

[ ] The pull request describes the changes that have been made, and enough information is present in the description for any developer to understand what has changed
[ ] Commmits have been squashed
[ ] The pull request title is of the following format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
[ ] The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#4
[ ] At least two reviewers assigned. One of which must be the assigner of the issue.
[ ] If there are merge conflicts, run git rebase as opposed to git merge with master.

So I'm not 100 percent sure if this works I can't test it without azure trying to pick up the build


feat: Added Option to Share Event

Fixes #975
Changes:

User can share Event Details from About Event Fragment as in EventBrite.
Share Icon color changes according to toolbar's color.
An example of info that's shared -

Event Name
Starts: 23:01, Tue, 10 Apr 2018
Ends at: 03:01, Mon, 30 Apr 2018
Url: https://www.example.com
Screenshots for the change:

@iamareebjamal Please Review.

@iamareebjamal Done !!


Reintroduce QRCode.terminal() in 1.5

QRCode.terminal() will be removed in 1.4.0
Deprecate term() in 1.5 and use terminal() with the same behaviour as term()

Sorry to ask, but why remove it and reintroduce it, instead of fixing it?

Thanks for your comment.
Well, I did not want to change the behaviour of terminal() with the (minor) release 1.3.x so I introduced term() which works for Unix and WIndows platforms but does not return anything but writes directly to the tty.
IMO the name "terminal()" is better than "term()" so I thought, moving the "term()" behaviour to the (deprecated) terminal() method (an deprecating term()) would make sense.
What do you think?

I really would miss the terminal function. As I sometimes use it to send it to a thermal printer. And therefore printing it directly would not work for me.

That's still possible:
>>> import io
>>> buff = io.StringIO()
>>> qr.term(buff)   # or qr.terminal(buff) for PyQRCodeNG >= 1.5.0
>>> print(buff.getvalue())


Fixed scrollbar issue

This fixes the scrollbar issue reported.

Thank you for the request!
I actually have a fix already that handles both the scrollbar and takes into account the Y position of the mouse before triggering the popup (it will only show now when the mouse leaves from the top of the viewport). I'll be merging it today.


ROX-21127: Refactor cluster init bundles pages

Description
Prerequisite to add cluster init bundle wizard, although discussion continues about its contents.
By the way, there is only an endpoint to fetch bundles, not to fetch a bundle by id:
https://github.com/stackrox/stackrox/blob/master/proto/api/v1/cluster_init_service.proto#L82-L86
Residue

InitBundlePage: Investigate second argument of service function and add confirmation modal for Revoke bundle.
InitBundlesRoute: Replace replace resources with Admin role.
Write integration tests.

Checklist

[x] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed
In stackrox/ui folder before yarn deploy
export ROX_MOVE_INIT_BUNDLES_UI=true

Manual testing

Visit /main/clusters/init-bundles

Click link to /main/clusters/init-bundles/id

gke-ui-e2e-tests failure

Cannot get catalogsources in kube-system: error: the server doesn't have a resource type "catalogsources"
Cannot get subscriptions in kube-system: error: the server doesn't have a resource type "subscriptions"
Cannot get clusterserviceversions in kube-system: error: the server doesn't have a resource type "clusterserviceversions"


Bugfixes on Lattice ALU

No longer closes #292 #284
It seems like there was not actually a problem in the end 🤦
Adding some more tests of ALU features.
EDIT:
There's still something wrong here. See final comments.

To be clear: I was asking the DSP to do something it doesn't seem to be able to do (3-stage mult+add). It can do mult+add in 2 stages no problem.

This PR fixes some issues with the Lattice DSP, but there are still workloads which we'd expect would synthesize to a Lattice DSP and yet don't.


Embed application build information into tracy files

TracyAppInfo can be used to add arbitrary data into the trace. We could record build information in release builds here such that we can correlate saved traces with which revision they were captured at. We don't likely want to do this in dev builds so that we aren't relinking binaries all the time. There may be some cmake magic we could use to make this not so bad.

Yup. Embedding the SHA from which IREE is built would be quite helpful (at lease before we have version scheme and well-organized releasing cycles.) We used to do that with DXC and allow recording the Git SHA in the generated SPIR-V blob.

Closing as wontfix (unless someone gets motivated). Embedding build information is tricky and we don't really have users following this workflow (today).


Describe false positive handling for sechub client

Describe false positive commands in client documentation

markFalsePositives
unmarkFalsePositives
getFalsePositives

Albert Tregnaghi<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

@sven-dmlr - so we need only the interactive commands described in #285


Merge pull request #248 from penguin-statistics/dev

Make GG happy again!

Yum yum new release! (ง •̀o•́)ง


Steam Deck Support

I've tried using this mod with the Steam Deck and I have a few issues:

Virtual cursor - why can't I select buttons like with the Midnight Controls mod?
Rear buttons - these are not detected
Text input - opening the virtual keyboard causes the text box to become deselected with no way to reselect

It would be great to have these improvements as I would like to play FTB modpacks on my Steam Deck and this mod is easy to get added in.

Are you using Steam virtual gamepad?

No, I have it set to "gamepad with joystick trackpad" so it'll work with the Midnight Controls mod in my Fabric instances.

using any steam input mode that uses "A button, B button, etc." is a virtual xbox 360 controller. so you are using steam's virtual gamepad. an option for having the extra buttons work is mapping them to unused keyboard binds (Ex: L4 is numpad1, R4 is numpad2, L5 is numpad4, R5 is numpad5) but then you need to use the keyboard controls menu and bind them.

Following up to mention this is the method to use Controllable with Steam Deck, which has been supported for a while.
https://mrcrayfish.gitbook.io/controllable-documentation/guides/how-to-connect-a-gamepad/method-4-steam-virtual-gamepad


Minimal surface example [minimal-surf-dev-rewritten]

This PR adds a new meshing miniapp, which solves Plateau's problem: the Dirichlet problem for the minimal surface equation.
It adds the SDIM=3 kernels for diffusion (setup) and vecdiffusion (setup & apply), and supports two types of solvers (ByVDIM or ByNODES), on all devices and w/ or w/o AMR.
Two problems can be run:

problem 0 solves the minimal surface equation of parametric surfaces,
problem 1 solves  -div( q grad u) = 0, with q(u) = (1+|∇u|²)^{-1/2} for surfaces of the form z=f(x,y), using Picard iterations.

Here are the different parameterizations available so far:

Costa

Mobius

Catenoid

Helicoid

Enneper

Scherk

Hold

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1263
@camierjs
@tzanio
@mlstowell + @stefanozampini + @rcarson3
04/01/20
⌛due 04/17/20
⌛due 04/24/20

LGTM

LGTM

Awesome miniapp!

LGTM

@mlstowell, @stefanozampini and @rcarson3 -- the reivews of this PR are due soon

LGTM

I wanted to thank @pazner and @v-dobrev for their patience answering all of my questions: it was a big help!

Merged in next for testing...


Get tests passing

rustdoc was assuming that this code block was a doctest.

I don't believe there is support for anything but rust, although I could definitely be wrong!
playing around if I do '```python' for that block the test fails and I don't get syntax highlighting. 🤷

also fyi I don't have commit rights, this is under your personal


"Actual commit differs from requested commit" error

See http://build.servo.org/builders/mac-dev-unit/builds/4846 for example

The git step also says
fatal: Unable to create '/Users/servo/buildbot/slave/mac-rel-css/build/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

This is a new step I added as a safeguard because Buildbot frankly sucks at git fetch, and often silently continues with the wrong commit.
Looking at the build logs for failed builds linked above (e.g. http://build.servo.org/builders/mac-dev-unit/builds/4849/steps/git/logs/stdio), it seems all of these are failing on our Mac builders, specifically due to another git process running in the same directory? It seems likely to me that a git process crashed at some point and didn't clean up after itself, meaning further git invocations don't go through. (This occurs because Buildbot doesn't just rm -rf the whole directory, but uses git clean -f -f -d -x, which I believe keeps the .git directory.)
I'll take a look on the builders; I'm open to suggestions on the error message to make it more useful.

Could it also be a force retry that broke stuff?

I doubt a force retry is to blame.
I've cleaned up all the errant index.lock files I could find (after checking no builds were being on each builder):

arm32 on servo-linux-cross1
mac-rel-css, mac-rel-wpt1 on servo-mac2
mac-rel-css on servo-mac4
mac-dev-unit on servo-macpro1

I confirmed there's none left via
root@servo-master1$ salt '*' cmd.run 'ls -a ~buildbot/slave/*/build/.git/index.lock'

All builders reported no index.lock files
It's possible this is affecting Windows builder(s), I didn't look at/touch those.

Linked PRs are merged now, so looks like this is fixed (at least for now). Closing, re-open if it happens again.


basic usage documentation is mission

can you please explain how to use the bot...
how to get the bot to join the voice channel?
also how to trigger the bot using voiice? is there any hotword?

Once you're sure you have all necessary packages installed, join a discord voice channel, then go to a text channel and type "0follow". (Follow is the text command that makes the bot join your voice channel and listen to the users in it)
Then, the trigger word for the bot is "song" So try saying "song play nirvana".
A couple of other voice commands are: "song stop", "song skip", "song pause", "song volume 30"


Library import offload

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Resolve library-related import issues
Resolve library-related update issues

Changes

New: By default, all smo libraries will be copied to temp and imported from temp. This avoids locking module files and enables update from website on older systems.
Fix: dbatools library is no longer imported as part of the manifest. This enables dbatools to unblock itself, resolving "library or dependencies not found" exceptions on import
Updated: dbatools library is also copied to that temporary folder before importing, same as smo (used to be copied to temp straight)
Updated: Maintenance task that cleans up in temp now cleans up the folders with all libraries, instead of just dbatools library

Notes

The import behavior for the smo libraries also supports the dbatools configuration for strict security mode, enabling dbas to disable the copy

thanks for the update and approval!


[now-static-build] Improve Gatsby default routes

This PR does 3 things:

add default caching headers
redirect all missed requests to Gatsby's own 404.html page
add deprecation message to gatsby-plugin-now since we're auto injecting it and it's not needed anymore

Fixes #2859

Codecov Report

Merging #3212 into canary will decrease coverage by 0.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #3212      +/-   ##
==========================================
- Coverage   12.41%   11.46%   -0.96%
==========================================
  Files         268      268
  Lines       10206    10084     -122
  Branches     1285     1313      +28
==========================================
- Hits         1267     1156     -111
+ Misses       8849     8838      -11
  Partials       90       90

Impacted Files
Coverage Δ

src/util/output/create-output.ts
59.37% <0%> (-7.3%)
:arrow_down:

src/util/prefer-v2-deployment.ts
56.25% <0%> (-5.66%)
:arrow_down:

src/util/get-files.ts
87.77% <0%> (-3.89%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
25% <0%> (-3.58%)
:arrow_down:

src/util/dev/builder-cache.ts
81.96% <0%> (-2.7%)
:arrow_down:

src/util/metrics.ts
70.58% <0%> (+2.4%)
:arrow_up:

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 17cd17a...ddfc783. Read the comment docs.


Added cache flag to helpers

Can be used like req = helpers.get(self.url, referer=referer, cache=False)
Prevents reading from cache, useful for tokens unique to each session

You shouldn't create a new session every time. This will mean that the cookies from the consecutive requests will not be persisted.
Other than that, I like the change.

I see no feasible way of doing persistent cookies without hacks, also I'm pretty sure only storing the cookies is bound to mindfuck people later on using the feature.

You shouldn't create a new session every time

Having a separate session cache with an expire_after of something like 10s is possible, would that be good?

Having a separate session cache with an expire_after of something like 10s is possible, would that be good?

That's what i had in mind, yes

I see no feasible way of doing persistent cookies without hacks, also I'm pretty sure only storing the cookies is bound to mindfuck people later on using the feature.

I did not mean persistent cookies across runs. Just in a single run, the cookies shouldnt be invalidated. If you are creating a new session every time, then the cookies are not persisted.

That's what i had in mind, yes

Fixed in latest commit, should I rename the flag to something more appropriate?

Arigato for your work


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
Upgrading Microsoft.WindowsSDK fails
Steps to reproduce
Run winget upgrade --id Microsoft.WindowsSDK
Actual behavior
PS C:\> winget upgrade
Name                                             Id                                  Version      Available      Source
-----------------------------------------------------------------------------------------------------------------------
Microsoft Help Viewer 2.3                        Microsoft.SQLServerManagementStudio 2.3.28307    15.0.18420.0   winget
Microsoft .NET Framework 4.8 Targeting Pack      Microsoft.WindowsSDK                4.8.04161    10.0.22000.194 winget
Microsoft .NET Framework 4.6 Targeting Pack (EN… Microsoft.WindowsSDK                4.6.00127    10.0.22000.194 winget
Microsoft Build of OpenJDK mit Hotspot 11.0.12+…  Microsoft.WindowsSDK                4.6.01055    10.0.22000.194 winget
Microsoft SQL Server 2012 Native Client          Microsoft.SQLServerManagementStudio 11.4.7462.6  15.0.18420.0   winget
Microsoft Visual C++ 2013 Redistributable (x86)… Microsoft.SQLServerManagementStudio 12.0.40664.0 15.0.18420.0   winget
Microsoft .NET Framework 4.5.1 SDK (Deutsch)     Microsoft.WindowsSDK                4.5.51641    10.0.22000.194 winget
Microsoft .NET Framework 4.7 Targeting Pack      Microsoft.WindowsSDK                4.7.02046    10.0.22000.194 winget
Microsoft .NET Framework 4.7 SDK                 Microsoft.WindowsSDK                4.7.02046    10.0.22000.194 winget
Microsoft .NET Framework 4.8 SDK                 Microsoft.WindowsSDK                4.8.04161    10.0.22000.194 winget
20 upgrades available.
2 packages have version numbers that cannot be determined. Using "--include-unknown" may show more results.
PS C:\> winget upgrade --id Microsoft.WindowsSDK
No installed package found matching input criteria.
PS C:\>

Expected behavior
Windows SDK getting upgraded to the latest version
Environment
Windows Package Manager (Preview) v1.3.1391-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.25158.1000
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.1391.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Screenshots and Logs
WinGet-2022-07-16-09-41-44.698.log
2022-07-16 09:41:44.699 [CORE] WinGet, version [1.3.1391-preview], activity [{4FE2A8C6-824B-4815-A521-79B54B2A4439}]
2022-07-16 09:41:44.699 [CORE] OS: Windows.Desktop v10.0.25158.1000
2022-07-16 09:41:44.699 [CORE] Command line Args: "C:\Users\user\AppData\Local\Microsoft\WindowsApps\winget.exe" upgrade --id Microsoft.WindowsSDK
2022-07-16 09:41:44.700 [CORE] Package: Microsoft.DesktopAppInstaller v1.18.1391.0
2022-07-16 09:41:44.700 [CORE] IsCOMCall:0; Caller: winget-cli
2022-07-16 09:41:44.705 [CLI ] WinGet invoked with arguments: 'upgrade' '--id' 'Microsoft.WindowsSDK'
2022-07-16 09:41:44.733 [CLI ] Found subcommand: upgrade
2022-07-16 09:41:44.733 [CLI ] Leaf command to execute: root:upgrade
2022-07-16 09:41:44.740 [CLI ] Executing command: upgrade
2022-07-16 09:41:44.747 [REPO] GetCurrentSourceRefs: Source named 'microsoft.builtin.desktop.frameworks' from origin Default is hidden and is dropped.
2022-07-16 09:41:44.747 [REPO] Default source requested, multiple sources available, adding all to source references.
2022-07-16 09:41:44.747 [REPO] Adding to source references msstore
2022-07-16 09:41:44.747 [REPO] Adding to source references winget
2022-07-16 09:41:44.747 [REPO] Multiple sources available, creating aggregated source.
2022-07-16 09:41:44.747 [REPO] Adding to aggregated source: msstore
2022-07-16 09:41:44.747 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2022-07-16 09:41:45.009 [REPO] Response status: 200
2022-07-16 09:41:45.009 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2022-07-16 09:41:45.069 [REPO] Response status: 200
2022-07-16 09:41:45.069 [REPO] Adding to aggregated source: winget
2022-07-16 09:41:45.083 [CORE] Examining extension: PFN = Microsoft.Winget.MSStore.Source_8wekyb3d8bbwe, ID = IndexDB
2022-07-16 09:41:45.087 [CORE] Examining extension: PFN = Microsoft.Winget.Source_8wekyb3d8bbwe, ID = IndexDB
2022-07-16 09:41:45.087 [CORE] Found matching extension.
2022-07-16 09:41:45.117 [REPO] Opening SQLite Index for ImmutableRead at 'C:\Program Files\WindowsApps\Microsoft.Winget.Source_2022.715.2101.548_neutral__8wekyb3d8bbwe\Public\index.db'
2022-07-16 09:41:45.117 [SQL ] Opening SQLite connection: 'file:/C:/Program Files/WindowsApps/Microsoft.Winget.Source_2022.715.2101.548_neutral__8wekyb3d8bbwe/Public/index.db?immutable=1' [1, 40]
2022-07-16 09:41:45.118 [REPO] Opened SQLite Index with version [1.5], last write [2022-07-15 22:01:30.000]
2022-07-16 09:41:45.381 [REPO] Creating PredefinedInstalledSource with filter [None]
2022-07-16 09:41:45.381 [REPO] Creating new SQLite Index [4294967295.4294967295] at ':memory:'
2022-07-16 09:41:45.381 [SQL ] Opening SQLite connection: ':memory:' [6, 0]
2022-07-16 09:41:45.414 [REPO] Examining ARP entries for Machine | X64
2022-07-16 09:41:45.596 [REPO] Examining ARP entries for Machine | X86
2022-07-16 09:41:45.600 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(192)\WindowsPackageManager.dll!00007FFD76FCBF45: (caller: 00007FFD76FC7A01) Exception(1) tid(3b04) 800700B7 Cannot create a file when that file already exists.

2022-07-16 09:41:45.600 [REPO] Ignoring duplicate ARP entry Machine|X86|Audacity_is1 [Audacity 3.0.2]
2022-07-16 09:41:45.621 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(192)\WindowsPackageManager.dll!00007FFD76FCBF45: (caller: 00007FFD76FC7A01) Exception(2) tid(3b04) 800700B7 Cannot create a file when that file already exists.

2022-07-16 09:41:45.621 [REPO] Ignoring duplicate ARP entry Machine|X86|Mp3tag [Mp3tag v3.14]
2022-07-16 09:41:45.636 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(192)\WindowsPackageManager.dll!00007FFD76FCBF45: (caller: 00007FFD76FC7A01) Exception(3) tid(3b04) 800700B7 Cannot create a file when that file already exists.

2022-07-16 09:41:45.636 [REPO] Ignoring duplicate ARP entry Machine|X86|VLC media player [VLC media player]
2022-07-16 09:41:45.637 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(192)\WindowsPackageManager.dll!00007FFD76FCBF45: (caller: 00007FFD76FC7A01) Exception(4) tid(3b04) 800700B7 Cannot create a file when that file already exists.

2022-07-16 09:41:45.637 [REPO] Ignoring duplicate ARP entry Machine|X86|WinMerge_is1 [WinMerge <IP_ADDRESS>]
2022-07-16 09:41:45.765 [REPO] Examining ARP entries for User | X64
2022-07-16 09:41:45.768 [FAIL] D:\a\_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(185)\WindowsPackageManager.dll!00007FFD76FCBF2D: (caller: 00007FFD76FC7A01) Exception(5) tid(3b04) 800700B7 Cannot create a file when that file already exists.

2022-07-16 09:41:45.768 [REPO] Ignoring duplicate ARP entry User|X64|9806-1938-4586-6531 [Burp Suite Community Edition 2022.6.1]
2022-07-16 09:41:46.269 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\user\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db'
2022-07-16 09:41:46.269 [SQL ] Opening SQLite connection: 'C:\Users\user\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db' [2, 0]
2022-07-16 09:41:46.272 [REPO] Opened SQLite Index with version [1.3], last write [2021-12-24 22:08:06.000]
2022-07-16 09:41:46.309 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\user\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db'
2022-07-16 09:41:46.309 [SQL ] Opening SQLite connection: 'C:\Users\user\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db' [2, 0]
2022-07-16 09:41:46.311 [REPO] Opened SQLite Index with version [1.3], last write [2022-07-16 09:19:34.000]
2022-07-16 09:41:46.364 [REPO] Found multiple matches for available package [Microsoft.WindowsSDK] in source [Microsoft.Winget.Source_8wekyb3d8bbwe] when searching for [Query:[none] Include:ProductCode='{1b45a8b2-a411-45ca-b322-d15ee6904559}'[Exact] Include:ProductCode='{4591faf1-a2db-4a3d-bfda-aa5a4ebb1587}'[Exact] Include:NormalizedNameAndPublisher='applicationverifierexternalpackage'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='applicationverifierexternalpackage'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='kitsconfigurationinstaller'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='kitsconfigurationinstaller'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='microsoftnetframeworksdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='microsoftnetframeworksdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='microsoftnetframeworktargetingpack'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='microsoftnetframeworktargetingpack'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='msidevelopmenttools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='msidevelopmenttools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='sdkarmadditions'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='sdkarmadditions'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='sdkarmredistributables'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='sdkarmredistributables'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='sdkdebuggers'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='sdkdebuggers'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='universalcrtextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='universalcrtextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='universalcrtheaderslibrariesandsources'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='universalcrtheaderslibrariesandsources'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='universalcrtredistributable'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='universalcrtredistributable'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='universalcrttools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='universalcrttools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='universalgeneralmididlsextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='universalgeneralmididlsextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winappdeploy'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winappdeploy'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkit'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkit'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkitnativecomponents'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkitnativecomponents'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkitsupportedapilist'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsappcertificationkitsupportedapilist'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsdesktopextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsdesktopextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsdesktopextensionsdkcontracts'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsdesktopextensionsdkcontracts'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsiotextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsiotextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsiotextensionsdkcontracts'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsiotextensionsdkcontracts'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsipoverusb'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsipoverusb'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsmobileextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsmobileextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsmobileextensionsdkcontracts'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsmobileextensionsdkcontracts'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkaddon'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkaddon'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkarmdesktoptools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkarmdesktoptools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheaders'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheaders'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheadersarm'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheadersarm'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheadersarm64'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktopheadersarm64'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibs'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibs'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibsarm'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibsarm'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibsarm64'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoplibsarm64'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoptools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoptools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoptoolsarm64'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdesktoptoolsarm64'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkdirectxremote'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkdirectxremote'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkeula'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkeula'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkfacadewindowswinmdversioned'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkfacadewindowswinmdversioned'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreapps'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreapps'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappscontracts'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappscontracts'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsdirectxremote'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsdirectxremote'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsheaders'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsheaders'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappslibs'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappslibs'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsmetadata'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappsmetadata'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappstools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoreappstools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoremanagedappslibs'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkforwindowsstoremanagedappslibs'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkmodernnonversioneddevelopertools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkmodernnonversioneddevelopertools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkmodernversioneddevelopertools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkmodernversioneddevelopertools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdkredistributables'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdkredistributables'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssdksigningtools'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssdksigningtools'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkit'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkit'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkitwindows'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkitwindows'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsteamextensionsdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsteamextensionsdk'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='windowsteamextensionsdkcontracts'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowsteamextensionsdkcontracts'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensedesktop'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensedesktop'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensedesktopotherlanguages'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensedesktopotherlanguages'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseiot'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseiot'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseiototherlanguages'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseiototherlanguages'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensemobile'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisensemobile'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseppi'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseppi'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseppiotherlanguages'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseppiotherlanguages'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseuap'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseuap'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseuapotherlanguages'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='winrtintellisenseuapotherlanguages'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='wptredistributables'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='wptredistributables'+'microsoftcorporations'[Exact] Include:NormalizedNameAndPublisher='wptx64'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='wptx64'+'microsoftcorporations'[Exact]]
2022-07-16 09:41:46.364 [REPO]   Checking match with package id: {1b45a8b2-a411-45ca-b322-d15ee6904559}
2022-07-16 09:41:46.364 [REPO]   Checking match with package id: {4591faf1-a2db-4a3d-bfda-aa5a4ebb1587}
2022-07-16 09:41:46.364 [REPO]   Found multiple packages with strong match fields
2022-07-16 09:41:46.364 [REPO]   Appropriate installed package could not be determined
2022-07-16 09:41:46.365 [CLI ] No app found matching input criteria
2022-07-16 09:41:46.374 [CLI ] Terminating context: 0x8a150014 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\WorkflowBase.cpp:32b

@agowa338 - Is this issue still happening?

I've asked the team what would trigger exit code 2008 from the Windows SDK installer.

For me, it is "Successfully installed" but doesn't actually upgrade.
PS C:\> winget upgrade --all
Name                                                      Id                   Version          Available    Source
-------------------------------------------------------------------------------------------------------------------
Windows Software Development Kit - Windows 10.0.17763.132 Microsoft.WindowsSDK < 10.0.22000.832 10.0.22621.1 winget
1 upgrades available.
2 packages have version numbers that cannot be determined. Using "--include-unknown" may show more results.

(1/1) Found Windows Software Development Kit [Microsoft.WindowsSDK] Version 10.0.22621.1
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.microsoft.com/download/f/6/7/f673df4b-4df9-4e1c-b6ce-2e6b4236c802/windowssdk/winsdksetup.exe
  ██████████████████████████████  1.32 MB / 1.32 MB
Successfully verified installer hash
Starting package install...
Successfully installed

PS C:\> winget upgrade
Name                                                      Id                   Version          Available    Source
-------------------------------------------------------------------------------------------------------------------
Windows Software Development Kit - Windows 10.0.17763.132 Microsoft.WindowsSDK < 10.0.22000.832 10.0.22621.1 winget
1 upgrades available.
2 packages have version numbers that cannot be determined. Using "--include-unknown" may show more results.

PS C:\> winget list Microsoft.WindowsSDK
No installed package found matching input criteria.

Windows Package Manager v1.3.2091
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.1889
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.2091.0

The upgrade now kinda succeeds:
PS C:\> winget upgrade --id Microsoft.WindowsSDK
Found Windows Software Development Kit [Microsoft.WindowsSDK] Version 10.0.22621.1
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.microsoft.com/download/f/6/7/f673df4b-4df9-4e1c-b6ce-2e6b4236c802/windowssdk/winsdksetup.exe
  ██████████████████████████████  1.32 MB / 1.32 MB
Successfully verified installer hash
Starting package install...
Successfully installed

PS C:\> winget upgrade
Name                                        Id                                    Version          Available     Source
-----------------------------------------------------------------------------------------------------------------------
Windows Software Development Kit - Windows… Microsoft.WindowsSDK                  10.0.22000.832   10.0.22621.1  winget
...
43 updates available.
1 package(s) have version numbers that cannot be determined. Use "--include-unknown" to see all results.

PS C:\> winget upgrade --id Microsoft.WindowsSDK
Found Windows Software Development Kit [Microsoft.WindowsSDK] Version 10.0.22621.1
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.microsoft.com/download/f/6/7/f673df4b-4df9-4e1c-b6ce-2e6b4236c802/windowssdk/winsdksetup.exe
  ██████████████████████████████  1.32 MB / 1.32 MB
Successfully verified installer hash
Starting package install...
Successfully installed

It looks like the issue is solved but the setup has an incorrect version number.

The update is again not working:
2022-10-23 01:09:55.536 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/packageManifests/Microsoft.WindowsSDK?Market=DE
2022-10-23 01:09:55.550 [REPO] Response status: 200
2022-10-23 01:09:55.556 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\Klaus\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db'
2022-10-23 01:09:55.556 [SQL ] Opening SQLite connection #4: 'C:\Users\Klaus\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db' [2, 0]
2022-10-23 01:09:55.558 [REPO] Opened SQLite Index with version [1.3], last write [2022-10-23 00:50:45.000]
2022-10-23 01:09:55.596 [REPO] Found multiple matches for tracking package [Microsoft.WindowsSDK] in source [Microsoft.Winget.Source_8wekyb3d8bbwe] when searching for [Query:[none] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkit'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkitwindows'+'microsoft'[Exact]]
2022-10-23 01:09:55.596 [REPO]   Checking match with package id: {4591faf1-a2db-4a3d-bfda-aa5a4ebb1587}
2022-10-23 01:09:55.596 [REPO]   Checking match with package id: {d6a76ead-c762-4d93-9c24-1fa3efa1e12d}
2022-10-23 01:09:55.596 [REPO]   Checking match with package id: {fcb5e4cc-2739-4812-97fd-f3b7c5d151ae}
2022-10-23 01:09:55.596 [REPO]   Appropriate installed package could not be determined
2022-10-23 01:09:55.607 [REPO] Found multiple matches for available package [Microsoft.WindowsSDK] in source [Microsoft.Winget.Source_8wekyb3d8bbwe] when searching for [Query:[none] Include:ProductCode='{1b45a8b2-a411-45ca-b322-d15ee6904559}'[Exact] Include:ProductCode='{4591faf1-a2db-4a3d-bfda-aa5a4ebb1587}'[Exact] Include:ProductCode='{d6a76ead-c762-4d93-9c24-1fa3efa1e12d}'[Exact] Include:ProductCode='{fcb5e4cc-2739-4812-97fd-f3b7c5d151ae}'[Exact] Include:NormalizedNameAndPublisher='windowssdk'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkit'+'microsoft'[Exact] Include:NormalizedNameAndPublisher='windowssoftwaredevelopmentkitwindows'+'microsoft'[Exact]]
2022-10-23 01:09:55.607 [REPO]   Checking match with package id: {4591faf1-a2db-4a3d-bfda-aa5a4ebb1587}
2022-10-23 01:09:55.608 [REPO]   Checking match with package id: {d6a76ead-c762-4d93-9c24-1fa3efa1e12d}
2022-10-23 01:09:55.608 [REPO]   Found multiple packages with strong match fields
2022-10-23 01:09:55.608 [REPO]   Appropriate installed package could not be determined
2022-10-23 01:09:55.608 [CLI ] No app found matching input criteria
2022-10-23 01:09:55.624 [CLI ] Terminating context: 0x8a150014 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\WorkflowBase.cpp:35e


Add an image in the beginning of Agent description blog post

          Would you like to add an image in the beginning? For example, the plot of the performance change.

Originally posted by @sonichi in https://github.com/microsoft/autogen/pull/1092#pullrequestreview-1798532786

Do we have a license to any stock images?  sticker badge would work.

I think images generated using DALL-E is fine :D https://help.openai.com/en/articles/6425277-can-i-sell-images-i-create-with-dall-e


Feature: I wish that the integrated terminal could be themed separately

I wish that the integrated terminal could be themed separately from the rest of the UI.

VSCode Version: 1.9.1
OS Version: 10.11.6

Duplicate https://github.com/Microsoft/vscode/issues/3112


Send bad_record_mac instead of decryption_failed

The decryption failed alert was deprecated a long time ago. It can
provide an attacker too much information to be able to distinguish between
MAC failures and decryption failures and can lead to oracle attacks.
Instead we should always use the bad_record_mac alert for these issues.
This fixes one instance that still exists. It does not represent a
security issue in this case because it is only ever sent if the record is
publicly invalid, i.e. we have detected it is invalid without using any
secret material.

Pushed. Thanks.


Client: Height issues for Project Contributors comtainer

Hello @kunalkeshan, Here is one question about this project/issue.
I observed the files of this client branch for this issue.
So, my question is why you've made so many files for different-different sections on the landing/home page?
I mean, why don't you write all the code in one file? :sweat_smile:
My point is while fixing any issue, we don't need to look for separate files for each section. Within a single landing page, we can solve and fix that issue. That's why.
I want to hear from you, what is the reason.

Hey @SAD0XER.
You can keep all the sections in the single page, and it's not wrong either. I'd like you think of this from a solo perspective first, it's a project that only you'll be working on and therefore you will know where which part of the page does what.
Now imagine you're working in a team, with people whom you don't know (basically OS). If everyone works on the same page, there's bound to be confusion and lots of merge conflicts.
Splitting that one single page into sections and keeping it in different files help keep the separation of concerns. Naming the sections/components in the way they're being used helps split to the code into small chunks so people can work on that part which they know the issue is at or one they've been assigned.

Also, as an example think that in a restaurant's kitchen, you'll have different people working in different stations right? One person to chop all the vegetables, one to handle the stove, one to decorate the plate and so on.
You don't have all of them doing their work in one single place, they have it separated for their specific purpose. It's the same way with code, while it's possible to do it in a single place, it's bound to become messy and unreadable as the project grows and the complexity increases.

Wonderful Explanation. As always, It cleared my doubt as well :smile:
Talking about this Height Issue, how can I solve this?
I tried to fix it in inspect mode, but it didn't work.
When I fix Desktop UI, Mobile UI worsens and vice versa.
Would you happen to have any idea how to fix this? I am ready to fix this.

closed in #142


Can read Firmware version but can't read card

OS version: Win 10
Arduino IDE version: 1.8.2
MFRC522 Library version: Latest
Arduino device: Uno R3(original)
MFRC522 device: MRFC522

Affected file(s) or example(s):

Example : dumpinfo.ino

Steps to reproduce:

Normal opertation

Observed Results:

Shows(reads) firmware version, but can read cards. just nothing happens.

Expected Results:

Expected full dump.

Seems the "normal" hardware problems, see https://github.com/miguelbalboa/rfid#troubleshooting .

Yup done.
Thanks.
You can close the issue.
Also I have a query :
If I want to read  few cards one after another, can you give me proper steps, (examples are not clear). I mean when to halt and stoptcrypto1 etc. Need more details.

If I want to read few cards one after another, can you give me proper steps, (examples are not clear). I mean when to halt and stoptcrypto1 etc. Need more details.

I agree with you, that the examples are not perfect, but that is the only thing we have. The Wiki never started :( https://github.com/miguelbalboa/rfid/wiki .
May search some tutorials, there are lot.

I think, if we treat a PICC as an object it would be better. How making a class. I make make it, give a pull request, but I need to understand this library more. Need good documentation.

The documentation of the code is very good, just compare it with the official arduino libraries ;) .
If you have some suggestions to refactor until the next release, please write them to #311 .
Best

@Subhajitdas298
I have the very same issue. How did you solve it?

Rotzbua, already helped me.
"Seems the "normal" hardware problems, see https://github.com/miguelbalboa/rfid#troubleshooting ."
Basically  loose connection. For my experienceconnect the PCD directly to arduino and also use max power in code.


Add Performance/Size cop

This started as a cop specifically for Array, but I discovered that Hash has a count method even though it is not mentioned in the documentation.
HASH = Hash[*1..100]

def slow
  HASH.count
end

def fast
  HASH.size
end

Benchmark.ips do |x|
  x.report('count') { slow }
  x.report('size') { fast }
  x.compare!
end

Calculating -------------------------------------
                count    21.099k i/100ms
                size   141.896k i/100ms
-------------------------------------------------
                count    247.280k (± 5.1%) i/s -      1.245M
                size      7.631M (± 8.5%) i/s -     37.886M

Comparison:
                size:  7630884.6 i/s
                count:   247280.4 i/s - 30.86x slower

Coverage decreased (-0.22%) to 99.42% when pulling 73210e5499b06aedc38cc27fd235861624bfcde2 on rrosenblum:size into f52bc7e2ec4b5a765bd905884613030d1f984277 on bbatsov:master.

@rrosenblum Guess this is Enumerable#count?
Btw, this cop is of little use if it checks only literals. You might enhance it check if's invoked on the result of to_a, to_h, Array, etc.

@bbatsov, I did some digging, and yes it is Enumerable#count.
I had the same thought about effectiveness of this cop after submitting the pull request. There are a few other cops that I think have similar issues of scoped effectiveness. ReverseEach, Detect, and DeprecatedHashMethods come to mind. ReverseEach and Detect are safer because they rely on a sequence of method calls. DeprecatedHashMethods seems too open ended because it could fail on objects that are not a hash.
I can make the changes to also add checks for calls to count on to_a, to_h, and other methods. Even with the changes, this cop with still have very scoped effectiveness.
With the changes made, this would not be caught by RuboCop
foo = {a: 1, b: 2, c: 3}
foo.count

It would be nice to be able to identify calls that have been made to count on variables that have been assigned to array or hash. The closest thing that I can think of is the check for useless assignment. What do you think of trying to keep track of what a variable has been assigned to? That way we can provide even better warnings.

It would be nice to be able to identify calls that have been made to count on variables that have been assigned to array or hash. The closest thing that I can think of is the check for useless assignment. What do you think of trying to keep track of what a variable has been assigned to? That way we can provide even better warnings.

Yeah, that would be a huge improvement. //cc @jonas054 @yujinakayama

I had the same thought about effectiveness of this cop after submitting the pull request. There are a few other cops that I think have similar issues of scoped effectiveness. ReverseEach, Detect, and DeprecatedHashMethods come to mind. ReverseEach and Detect are safer because they rely on a sequence of method calls. DeprecatedHashMethods seems too open ended because it could fail on objects that are not a hash.
I can make the changes to also add checks for calls to count on to_a, to_h, and other methods. Even with the changes, this cop with still have very scoped effectiveness.

Yeah, finding the balance between usefulness and no false positives is pretty hard sometimes in a language like Ruby.

I think it's better to merge the new cop as it is for now, perhaps put a TODO comment in it, and try to add the advanced detection of variable types later.

I think it's better to merge the new cop as it is for now, perhaps put a TODO comment in it, and try to add the advanced detection of variable types later.

Generally I agree, but as it stands the scope of this is tiny. Adding handling for array/hash producing methods will definitely make it much more useful IMO (and won't add much complexity).

My plan is to at least add the checks on calls to array/hash producing methods (to_a, to_h, Array, ...).
I foresee this same issue/conversation coming up on a number of the performance cops since all of these cops will deal with method calls on objects.

The code has been rebased and updated to account for Array[], Hash[], to_a, and to_h.

@rrosenblum You'll have to rebase again. :(

@bbatsov, the branch has been rebased.

:+1:


No Events Firing

After calling createAudioPlayer with a working player (I can play and manipulate a queue) I can't get any events to fire. I've tried registering as follows:
var audioPlayer = spotify.createAudioPlayer(clientId);
audioPlayer.addEventListener(spotify.AudioPlayer.EVENT_LOGIN, function(message){
   console.log('logged into player ' + message);
 });
audioPlayer.addEventListener(spotify.AudioPlayer.EVENT_MESSAGE, function(message){
   console.log('Got spotify message ' + message);
 });
audioPlayer.addEventListener(spotify.AudioPlayer.EVENT_PLAYBACK_STATUS, function(status){
   console.log('got playback status ' + status);
 });
audioPlayer.addEventListener(spotify.AudioPlayer.EVENT_ERROR, function(error){
   console.error(error);
 });
audioPlayer.addEventListener('volumeChanged', function(error, value){
   console.log('volumeChanged')
 });
audioPlayer.addEventListener('skippedToNextTrack', function(error, value){
   console.log('skippedToNextTrack')
 });
 audioPlayer.addEventListener('skippedToPreviousTrack', function(error, value){
   console.log('skippedToPreviousTrack')
 });

None of these ever get fired but my music is playing after calling audioPlayer play on a playlist. Any suggestions as to what I'm doing wrong.

Oh and I'm calling login after I've added the event listeners.

I've been digging and it appears the events are making it to eventNotificationFromAudioPlayer they just are not getting thrown back up into onEventCallback. I'm wondering if the registration for the callbackID is valid.

I've added a branch on my repository at https://github.com/ambientBOX/cordova-spotify-plugin/tree/fix-event-callback that fixes this issue but I don't believe it's complete. Can you please review it, if it's good enough I'll submit as pull.

I'll have a look at this issue as soon as I have some time later this week. Great work on initiating the fix.

I believe I found a fix for this issue. I'll create a new release as soon as possible.


Update roadmap.md url in CONTRIBUTING.md

Changes
Update roadmap.md url in CONTRIBUTING.md
/kind documentation
Supported kinds are: bug, cleanup, design, documentation, failing-test, feature, flake, misc, question, tep
-->
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/ok-to-test

Hmmm, the CLA link doesn't work for me, it's showing all blank after I allow authorization.

Hmmm, the CLA link doesn't work for me, it's showing all blank after I allow authorization.

Yeah I notice the same. @vdemeester @afrittoli any idea?

@pritidesai looks like CLA back normal, can we proceed with the PR?

/lgtm


[Hanabi] Investigate if yellow clue is not properly displayed

Only one player experiences this problem so far. It would be good if that player double-checks for a while what other people know about their cards when they want to give them a yellow clue.

Unable to reproduce.


test_static.sh behavior on travis CI inconsistent with local behavior.

Extension name (the extension in question)
CI infrastructure.
Description of issue (in as much detail as possible)
Running test_static.sh locally produces warnings below.
However pylint does not report these errors on the CI.
Running pylint on extensions...
Using config file /Users/tosin/Repos/azure-cli-extensions/pylintrc
************* Module azext_mesh.custom
W:247,31: Use % formatting in logging functions and pass the % parameters as arguments (logging-format-interpolation)
W:252,23: Use % formatting in logging functions and pass the % parameters as arguments (logging-format-interpolation)

...

************* Module azext_webapp.tunnel
W:178,20: Using deprecated method warn() (deprecated-method)
W:178,20: Using deprecated method warn() (deprecated-method)
************* Module azext_imagecopy.custom
W: 23, 4: Using deprecated method warn() (deprecated-method)
W: 23, 4: Using deprecated method warn() (deprecated-method)
W: 31, 8: Using deprecated method warn() (deprecated-method)
W: 31, 8: Using deprecated method warn() (deprecated-method)
W: 71, 4: Using deprecated method warn() (deprecated-method)

...

W:158,12: Using deprecated method warn() (deprecated-method)
W:158,12: Using deprecated method warn() (deprecated-method)
C:  9, 0: third party import "from knack.util import CLIError" should be placed before "from azext_imagecopy.cli_utils import run_cli_command, prepare_cli_command" (wrong-import-order)
C: 11, 0: third party import "from knack.log import get_logger" should be placed before "from azext_imagecopy.cli_utils import run_cli_command, prepare_cli_command" (wrong-import-order)
************* Module azext_eventgrid.custom
W:528, 8: Specify string format arguments as logging function parameters (logging-not-lazy)
************* Module azext_find.custom
W: 26,21: Use % formatting in logging functions and pass the % parameters as arguments (logging-format-interpolation)
************* Module azext_dms.custom
C: 17, 0: third party import "from azure.cli.core.util import get_file_json, shell_safe_json_parse" should be placed before "from azext_dms.vendored_sdks.datamigration.models import Project, MySqlConnectionInfo, PostgreSqlConnectionInfo, MigrateSyncCompleteCommandInput, MigrateMySqlAzureDbForMySqlSyncTaskProperties, MigratePostgreSqlAzureDbForPostgreSqlSyncTaskProperties, MigrateSyncCompleteCommandProperties" (wrong-import-order)
C: 18, 0: third party import "from azure.cli.command_modules.dms.custom import create_or_update_project as core_create_or_update_project, create_task as core_create_task" should be placed before "from azext_dms.vendored_sdks.datamigration.models import Project, MySqlConnectionInfo, PostgreSqlConnectionInfo, MigrateSyncCompleteCommandInput, MigrateMySqlAzureDbForMySqlSyncTaskProperties, MigratePostgreSqlAzureDbForPostgreSqlSyncTaskProperties, MigrateSyncCompleteCommandProperties" (wrong-import-order)

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

This is due to different versions of python being used. (our CI uses 3.6.3).

Hi, I am currently working on it.
I encounter the same situation as you. I can repro those on my machine. And this is not relative to Pytrhon version.
Currently we are migrating from Travis to ADO. ADO's behavior alongs with my local machine.
I think this can be close once migration is done.

This job has been migrated from Travis to ADO.
https://github.com/Azure/azure-cli-extensions/pull/1133
https://github.com/Azure/azure-cli-extensions/pull/1156
Closed as no need.


5Dim Battlefield & Optera's Vehicle Grid

Optera's Vehicle Grid mod adds equipment grids to vehicles, to allow equipment to be inserted into them like modular armours.
For vanilla vehicles, when equipment is inserted into a vehicle's grid and the player picks up the vehicle, the equipment remains inside the grid when it is placed down again. However, all equipment in the MK2 tank is deleted when it is picked up. Optera says this occurs when the item prototype is set incorrectly.
This issue is easily reproducible. Simply place down an Mk2 tank, put any equipment inside, pick it up, put it back down and look inside the grid.
This problem does not occur with 5Dim's locomotives, but I have not tested other vehicles.
Also, tanks are supposed to have a 12x10 grid, but the Mk2 tank has a 8x8 grid as if it were a car.

that is because Optera mod do it for vehicles without grid, but 5dims vehicles has grid (i do it) so...

Oh, thanks. I didn't realize that.  Since I fixed the other problem I'll close this issue now.


UnsupportedOperationException on addMappingForServletNames following the Spring Boot 2 sample

Serverless Java Container version: 1.9
Implementations: Spring Boot 2
Framework version: SpringBoot 2.7.5.RELEASE
Frontend service: ALB
Deployment method: Console
Scenario
Following the sample for Spring Boot 2 with the exception of using the SAM CLI which isn't available for use within my company. I am building manually and am confident in the final artifact as the appropriate handler is executed.
Expected behavior

Follow Spring Boot 2 sample
Build with mvn package
Upload with aws lambda update-function-code ...
Browse to /
Content at / is returned to the browser

Actual behavior

Follow Spring Boot 2 sample
Build with mvn package
Upload with aws lambda update-function-code ...
Browse to /
Timeout due to buried UnsupportedOperationException

Steps to reproduce

Follow Spring Boot 2 sample
Build with mvn package
Upload with aws lambda update-function-code ...
Browse to /

Full log output
Unfortunately I'm doing this at work and my company doesn't allow pasting snippets to the internet, however the stack trace follows:
Error loading class <>.StreamLambdaHandler: java.lang.ExceptionInInitializerError
java.lang.ExceptionInInitializerError
...
Caused by: ApplicationContextException: Unable to start web server; nested exception is java.lang.UnsupportedOperationException
....
Caused by: UnsupportedOperationException
    at FilterHolder$Registration.addMappingForServletNames(FilterHolder.java:281)

It appears that addMappingForServletNames() simply throws UnsupportedOperationException(). Should that method even be getting called? If no, any clues as to where I've gone wrong from the sample?

Hi @lancekf,
thanks for reporting this issue. I'm unable to reproduce it so far.

How did you create the Lambda function and the API-Gateway? Can you please share all commands to reproduce it from scratch?
What happens if you make a request to /pets? Do you see the same issue there?
In addition could you please add a full stack trace to see where addMappingForServletNames is called (if it's not shown, you could add logging using Thread.currentThread().getStackTrace() before throw new UnsupportedOperationException();)?

Hi @deki, thanks for the reply.

How did you create the Lambda function and the API-Gateway? Can you please share all commands to reproduce it from scratch?

This is an ALB and Lambda created via Terraform modules supplied by my company. For testing, I've been sending ALB request events directly to the Lambda as modeled here: https://docs.aws.amazon.com/lambda/latest/dg/services-alb.html

What happens if you make a request to /pets? Do you see the same issue there?

Yes.

In addition could you please add a full stack trace to see where addMappingForServletNames is called (if it's not shown, you could add logging using Thread.currentThread().getStackTrace() before throw new UnsupportedOperationException();)?

Unfortunately I can't provide a full stack trace as I'm posting this from my personal machine. I did realize yesterday the calling method prior to addMappingForServletNames() would've been helpful but then got sidetracked and forgot to add it... that is:
Caused by: UnsupportedOperationException: null
    at FilterHolder$Registration.addMappingForServletNames(FilterHolder.java:281)
    at AbstractFilterRegistrationBean.configure(AbstractFilterRegistrationBean:242)
    at AbstractFilterRegistrationBean.configure(AbstractFilterRegistrationBean:46)

Some of the differences with my setup I should point out:

I'm not using the CognitoIdentifyFilter. I've commented out the handler.onStartup() registration.
I'm packaging this according to a standard maven layout within my company, however this layout still includes using the maven-shade-plugin to create a shaded jar. Would a packaging issue be causing it to blow up on the servlet configuration?
I'm deploying my jar to the Lambda using the aws lambda CLI instead of the SAM CLI. I don't think this is a problem.'

Thanks again!

FWIW, I tried adding in the assembly-zip profile from the sample, uploaded the created zip, and... same issue :confused:

Hmm can you find out which filter is triggering the registration?
I also did a deployment using AWS CLI this morning and it worked fine. Used API-Gateway and not ALB though, need to rerun my test.

Hmm can you find out which filter is triggering the registration?

I think you're on to something here... I'm not the best at deciphering spring output but I believe it's trying to load these filters:
corsFilter
excludeFontVaryHeaderFilter
mdcFilter
dispatcherPerfLoggingFilter servlets=[dispatcherServlet]
requestLoggingFilter servlets=[dispatcherServlet, jerseyServlet]
requestContextFilter

Some of these are no doubt from some auto configuration applied by the standard project layout from my company that I'm using... my guess is it's a problem with the filters that have servlets=[] associated with them and I need to figure out how to disable them? Very much appreciate your help!

Yep, I managed to disable those logging filters and I can now GET a 200 from /pets, however I get an empty response body:
{
    "statusCode": 200,
    "statusDescription": "200 OK",
    "multiValueHeaders": (),
    "body": "",
    "isBase64Encoded": false
}

So that's a new problem... for the original filter registration issue, could there be any improvements to aws-serverless-java-container to better handle that? Ex. somehow check to see if any filters are registered for a servlet and unregister the servlet or disable the filter?

Coming back to the new problem... I've disabled all autoconfiguration and I'm still only getting an empty response body at GET /pets. I don't think it's mapping the request to the PetsController at all... I do see this at startup:
servlet.HandlerMapping.Mappins:
PetsController:
{GET [/pets/{petId}]}: listPets()
{GET [/pets]}: listPets(Optional, Principal)
{POST [/pets]}: createPet(Pet)

... but I'm seeing no indication that the request is actually processed through that controller. Which part of code is responsible for scanning for any @RestController endpoints?

Okay, so in addition to disabling some autoconfiguration, I also used the assembly zip packaging from the sample, and that seems to work. I still have lots of things to figure out with this setup and I'm happy to contribute how I can but please feel free to close this as a non-issue. Thank you very much for your time!

Ok, let me know if there is something that we can improve to streamline the experience.


Update data-collection-syslog.md

The table listing Supported Facilities is incorrect. Please refer to the RFC3164 BSD Protocol documentation, page 8 and 9.(https://www.rfc-editor.org/rfc/rfc3164.html)
I also added an additional column listing the Data Source Name which is used on Data Collection Rule (DCR) creation, for extra clarification/recognizably.

@arjan87
Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Closing this PR pending CLA completion.


12 hr format instead of 24

Is there a way to select from a 12 hr time picker instead of 24?

did you find a solution to this question