text:
"TypeScript for Visual Studio" installer not available for 5.1

Bug Report
There does not appear to be a "TypeScript 5.1 for Visual Studio" installer available on marketplace.visualstudio.com.
Previous releases of TypeScript have had installers made available, such as this one for 5.0.4.
Was it intended to stop publishing this for 5.1 onward?
🔎 Search Terms
Visual Studio 2019/2022, Windows SDK

https://github.com/microsoft/TypeScript/issues/54686 seems like it might be related

5.1.3 has now been published. Here is the link to it: https://marketplace.visualstudio.com/items?itemName=TypeScriptTeam.typescript-513
Apologies that it did not go up immediately. Future publishes should happen simultaneously with the NuGet releases.

Does this even work with the buildin node 12 version in VS2019 (or even node 10 in VS2017)?

@noshlag Simultaneous publishes don't seem to be working, since 5.2 is also missing from the marketplace.

Does this even work with the buildin node 12 version in VS2019 (or even node 10 in VS2017)?

Unfortunately no - there are ways of customizing VS2019 to use an alternative version of Node.js, but TypeScript won't work with the built-in Node.js from VS2019.

publishes don't seem to be working, since 5.2 is also missing from the marketplace.

Apologies, there was an oversight in the publishing process. That said, I would highly recommend switching over to using NuGet, or more preferably npm, to customize the version of TypeScript used in VS.

@cheit-epic It was on nuget.org but not anymore (because it was buggy):
https://github.com/microsoft/TypeScript/issues/57475#issuecomment-2189424566

authors: DanielRosenwasser HolgerJeromin cheit-epic fatcerberus noshlag wnayes


text:
React's ComponentProps type issues in TypeScript 5.6.2

🔎 Search Terms
react, forwardRef, component props, ComponentProps
🕗 Version & Regression Information

This changed between versions 5.5.4 and 5.6.2 - but in version 5.5.4 there are other related bugs 🤷♂

⏯ Playground Link
https://www.typescriptlang.org/play/?jsx=4&ts=5.6.2#code/JYWwDg9gTgLgBAbwLACg5xgTzAUzgUQBscQcA7GAFWxwBpV0tc4BhCcCM8mABSgjABnemgw1W7SFwp8BggOrAYACwgBXGACUcAMxGNxbDtKo19cHdADuAQygATbTrg3BcAEaucAMWt3HuuZMeL5Qtg5O2mT2OFDeamQAxjDAnEHiTiIAvhb8IHAA5FA4NskFANyoqDoJyalkFn4RugA8lLRwPAB8ABQMcImSnNwAXHA9YPxCYzwdxTpjTm1dAJRwALxdcNqlMAB0O8kAchAxIitjE1OCM3AAZIjzAPyLrZRdWWub2yXJB78wE4xRD9YowNRQBqeQQ+JoBHR9UToQbGbguNwJADWZAgVgarjgoXC8KiMTitRSnBaWJxeI6NNxZC65hWlRQWSqKGCElRFEUKiJ-icsiEbTgOAAHjByPY3EQSNxqLgtut+kYpNwRQolKoNEt3vcQUiBoRXIIjjZSC84IIYFBgGQAOZs9DoVzWyhwAA+cASMR0Dpw9hdcCybNQgzItp5Gr5OsFzWc60aYSFukR6DFkul0TlxFIFCVeGTBUEYBsZAKvX66EmcjG6uGcYFcOF12W5nQ81eOhaAAlKABZAAy8oLMGZ-S+W2QxrBEIaZDUhEIIay53DXPElBwtuWGzgnuzMrcCBRsZg1odOliMabMA5oied5M4qlJ5fipoLQrmC6NbgZ9GxMLUWmA7h-2NMYuAAN1ifpoJwOCoE3bltEEZcYAARgPcCZHbYIIGcPCYH5ZQE3hLpyjgAB6GiMGUPBigwwh4GANwrBwYAHGoqxlEwOAAAMAHkQCUH8yEwDoil0KtBKeVA0N3TCACYDx3PczyGEwxkI4jtO4MiKKcLIqNo+iVCY5TWLgdiBmgYpkjGc971LOBoGAR0HRsQg4DrIQ4AAajgeY-KmPZUCAA
💻 Code
I noticed that when using custom forwardRef function (I use it so my generic components are typed correctly), I get different results in these two situations:
import {
  type ElementType,
  type ComponentProps,
  type ComponentPropsWithoutRef,
  type ComponentType,
  forwardRef as baseForwardRef,
  type ForwardRefRenderFunction,
  type Ref,
} from 'react';

// setup code
function forwardRef<T, P>(
  component: (props: P, ref: Ref<T>) => React.ReactNode,
): (props: P & {ref?: Ref<T>}) => React.ReactNode {
  return baseForwardRef(
    component as unknown as ForwardRefRenderFunction<unknown, unknown>,
  );
}

type FooProps<T extends ElementType> =
  ComponentPropsWithoutRef<T> & {
    className?: string;
    as?: T | undefined;
  };

const Foo = forwardRef(
  <T extends ElementType = 'span'>(
    props: FooProps<T>,
    ref: Ref<HTMLElement>,
  ) => {
    return null;
  },
);

type Test<T> = T extends infer Component
  ? Component extends ComponentType<any>
    ? ComponentProps<Component>
    : never
  : never;

type Result1 = ComponentProps<typeof Foo>; // the result is weird: why `Omit<any, 'ref'>`?
type Result2 = Test<typeof Foo>; // the result is correct: Foo's original props + ref prop.
import {
  type ElementType,
  type ComponentProps,
  type ComponentPropsWithoutRef,
  type ComponentType,
  forwardRef as baseForwardRef,
  type ForwardRefRenderFunction,
  type Ref,
} from 'react';

function forwardRef<T, P>(
  component: (props: P, ref: Ref<T>) => React.ReactNode,
): (props: P & {ref?: Ref<T>}) => React.ReactNode {
  return baseForwardRef(
    component as unknown as ForwardRefRenderFunction<unknown, unknown>,
  );
}

type FooProps<T extends ElementType> =
  ComponentPropsWithoutRef<T> & {
    className?: string;
    as?: T | undefined;
  };

const Foo = forwardRef(
  <T extends ElementType = 'span'>(
    props: FooProps<T>,
    ref: Ref<HTMLElement>,
  ) => {
    return null;
  },
);

type Test<T> = T extends infer Component
  ? Component extends ComponentType<any>
    ? ComponentProps<Component>
    : never
  : never;

// ⚠ different results
type Result1 = ComponentProps<typeof Foo>; // the result is weird: why `Omit<any, 'ref'>`?
type Result2 = Test<typeof Foo>; // the result is correct: Foo's original props + ref prop.

🙁 Actual behavior
Type Result1 is wrong:
type Result1 = Omit<any, "ref"> & {
  className?: string;
  as?: ElementType | undefined;
} & {
  ref?: Ref<HTMLElement> | undefined;
}

and Result2 is correct:
type Result2 = PropsWithoutRef<ComponentProps<T>> & {
  className?: string;
  as?: T | undefined;
} & {
  ref?: Ref<HTMLElement> | undefined;
}

🙂 Expected behavior
Types Result1 and Result2 are same:
type Result = PropsWithoutRef<ComponentProps<T>> & {
  className?: string;
  as?: T | undefined;
} & {
  ref?: Ref<HTMLElement> | undefined;
}

Additional information about the issue
No response

Result2 is not correct. It leaks T and that shouldn't happen. It's the other result that is correct, you can double-check that on 5.4 by checking the types resolved based on the result of your custom forwardRef call and based on the results of "the same" inlined type (I copy-pasted its computed definition): TS playground
As we can see here, all 4 results are:
type Result = Omit<any, "ref"> & {
    className?: string | undefined;
    as?: ElementType | undefined;
} & {
    ref?: Ref<HTMLElement> | undefined;
}

If we now switch this playground to 5.6.2 then we'll see one of those going rogue (the one that you assumed is correct): TS playground

A reasonable self-isolated repro: TS playground

Huh, interesting. Thank you for the investigation.

`` ceb4951646b978c517240e7bf43517a53d969c25

We'd really need a much shorter repro to be able to prioritize this

additional repro:
function withP3<P>(p: P) {
  const m =
    <I,>(from: I) =>
    <I2,>(from2: I2) => ({ ...from, ...from2, ...p });
  return createTransform(m);
}

const addP3 = withP3({ a: 1 });
const addedSome3 = addP3({ b: '' });
const added3 = addedSome3({ c: true });

const addP3_other = withP3({ foo: 'bar' });
const addedSome3_other = addP3_other({ qwerty: 123 });
const added3_other = addedSome3_other({ bazinga: true });

We can see here { a: number } leaking from the first "chain" of instantiations into the second "chain".
I have a fix for both here: https://github.com/microsoft/TypeScript/pull/59972 :)

Great job as usual, thank you @Andarist!

- [ ]

Details

Vv

source: gharchive/issue

authors: Alavrgajesus Andarist RyanCavanaugh jakubmazanec

repo: microsoft/TypeScript

license_type: permissive

license_source: github-api


text:
Cannot set readonly property in IIFE

🔎 Search Terms
"readonly", "iife"
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
No response
💻 Code
class Foo {
	readonly bar: string = "";
	constructor() {
		(() => { this.bar = "test"; })();
	}
}

🙁 Actual behavior
Fails with Cannot assign to 'bar' because it is a read-only property. error
🙂 Expected behavior
Should work in exactly same way as:
class Foo {
	readonly bar: string = "";
	constructor() {
		this.bar = "test";
	}
}

Additional information about the issue
I assume that's expected for some reason, and I'd love to learn what are the logical grounds for that

Essentially another duplicate of #9998. #58729 would likely fix this.

@MartinJohns aren’t IIFEs already inlined for CFA? I’m guessing initialization of readonly fields must be handled separately.

You're right, my mistake. I swear I saw this issue before, but can't find it. :-/

authors: MartinJohns jcalz medikoo


text:
azure-sdk test: move typescript location

Fixes most of the azure-sdk failures by compiling with HEAD rather than 3.5.1.

✔ azure-sdk  [master ↓·1|✚ 36]$ rush add -p typescript.tgz --dev -m --all

Rush Multi-Project Build Tool 5.19.3 - https://rushjs.io
Node.js version is 12.13.1 (LTS)

Starting "rush add"

Determining new version for dependency: typescript.tgz
No version selector was specified, so the version will be determined automatically.

Trying to acquire lock for pnpm-4.6.0
Acquired lock for pnpm-4.6.0
Found pnpm version 4.6.0 in /home/nathansa/.rush/node-v12.13.1/pnpm-4.6.0

Symlinking "/home/nathansa/src/azure-sdk/common/temp/pnpm-local"
  --> "/home/nathansa/.rush/node-v12.13.1/pnpm-4.6.0"
Querying NPM registry for latest version of "typescript.tgz"...

ERROR: The command failed with exit code 1
npm ERR! code EINVALIDTAGNAME
npm ERR! Invalid tag name "typescript.tgz@latest": Tags may not have any characters that encodeURIComponent encodes.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/nathansa/.npm/_logs/2020-02-20T18_12_21_616Z-debug.log

Or maybe this is what you meant?
✘-1 azure-sdk  [master ↓·1|✚ 36]$ rush add -p typescript@typescript.tgz --dev -m --all

Rush Multi-Project Build Tool 5.19.3 - https://rushjs.io
Node.js version is 12.13.1 (LTS)

Starting "rush add"

ERROR: The SemVer specifier "typescript.tgz" is not valid.

The later. Looks like rush still refuses to recognize fs paths as valid versions. :(

source: gharchive/pull-request

authors: sandersn weswigham


text:
fix: add es2016.intl.d.ts

part of #29129

cc @orta

ping @orta :)

ping @DanielRosenwasser

Overall looks good, I think UnicodeBCP47LocaleIdentifier should probably live here and get removed from the 2020 version.

Hmm, is that a breaking change?

That's what I thought but I'm cargo culting and tests seem upset if I just modified src

@DanielRosenwasser what do you think? Should we go ahead and merge this? Seems like the only outstanding question is whether to check in the lib/ changes.

can I get a verdict here :) ?

OK, I've rebased this and set it up locally - as it is a project targeting es2020 would get duplicate identifiers for UnicodeBCP47LocaleIdentifier:
❯ node ../../typescript-compiler/built/local/tsc.js
../../typescript-compiler/built/local/lib.es2016.intl.d.ts:29:10 - error TS2300: Duplicate identifier 'UnicodeBCP47LocaleIdentifier'.

29     type UnicodeBCP47LocaleIdentifier = string;
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  ../../typescript-compiler/built/local/lib.es2020.intl.d.ts:29:10
    29     type UnicodeBCP47LocaleIdentifier = string;
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    'UnicodeBCP47LocaleIdentifier' was also declared here.

../../typescript-compiler/built/local/lib.es2020.intl.d.ts:29:10 - error TS2300: Duplicate identifier 'UnicodeBCP47LocaleIdentifier'.

29     type UnicodeBCP47LocaleIdentifier = string;
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  ../../typescript-compiler/built/local/lib.es2016.intl.d.ts:29:10
    29     type UnicodeBCP47LocaleIdentifier = string;
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    'UnicodeBCP47LocaleIdentifier' was also declared here.

Found 2 errors.

I think we'll need to remove the identifier from es2020 to have it be usable in es2016. The question of whether this is a breaking change is a good one. If you had a setup like:
"target": "es2019",
"lib": ["es2020.intl"]

and were referring to UnicodeBCP47LocaleIdentifier you would still have access to it because of the target. However, if your target was es2015, and you explicitly just referenced the es2020.intl in lib:
"target": "es2015",
"lib": ["es2020.intl"]

Then moving the type back is a breaking change. Which given the breadth of the TS userbase, someone probably has this setup.
Which gives three options:

Accept the potential break in the types for target es2015 + a direct reference in lib, and move the identifier to es2016
NOOP on this change
Switch the type in es2016 to be string instead of the named alias, keeping UnicodeBCP47LocaleIdentifier in es2020

Personally, I fall into the last bucket.

I personally would pick the 1st option. ECMA402 are not designed to introduce new APIs per release and have them work independently without previous version, e.g you should not be able specify es2020.intl without es2019.intl and expect it to properly work. Future versions add new options to existing set of limited APIs and normative changes are made to previous types.
UnicodeBCP47LocaleIdentifier is an example of that, it used to just be BCP47 which includes grandfathered locales but was removed in later versions (see https://github.com/tc39/proposal-intl-locale/issues/63 for discussion).
I think the real change (outside of this PR) is probably to make es2020.intl includes all es2015.intl through out es2019.intl.

@typescript-bot user test this

This PR was deprecated and merged into main via https://github.com/microsoft/TypeScript/pull/45647 - thanks @longlho !

This PR was deprecated and merged into main via https://github.com/microsoft/TypeScript/pull/45647

I don't see this code anywhere in https://github.com/microsoft/TypeScript/pull/45647 or anywhere else in TS (as mentioned elsewhere). Is that intentional?

authors: DanielRosenwasser longlho ls-andrew-borstein orta sandersn


text:
Fix emit for undefined SourceFile

I like to print out the AST when debugging to see what I'm doing, but when it's not a "real" AST from a source file, things crash, even though they shouldn't. Remove all of the ! from emit and actually handle a missing source file, allowing printNode and such to actually print a tree without a source file.
Arguably, I could change the signature of printNode and co to explicitly note that they can have an undefined SourceFile (matching writeNode and so on), but maybe that's not a good idea, or something that should be internal.

@typescript-bot perf test this

Looks like a few failed to emit, which is very interesting.

Hm, no, the baseline failed too, so it's not this PR.


text:
Adding an outExtension option to control the file extension of emitted files

Fixes #54573 by adding a new outExtension option to control the extension of the emitted files (except .tsx, .jsx and .json input files which won't be affected by the outExtension option):

"input" (default): Infers the extension of out files from the extension of the input file. This is the behaviour prior to this PR and I've added it to limit the intrusiveness of the fix.
"module": Infers the extension of out files from the module option, if "commonjs" the out files will use .cjs, if any "es6" through "esnext" the out files will use .mjs and .js will be used for any other case.
"cjs": Use .cjs for all out files.
"mjs": Use .mjs for all out files.
"js": Use .js for all out files.

Note: I'm drafting the PR until the associated issue in the Backlog milestone.

@microsoft-github-policy-service agree [company="MongoDB"]

@microsoft-github-policy-service agree company="MongoDB"


text:
wsl2 kali networkingMode=mirrored not functioning right

Windows Version
Microsoft Windows [Version 10.0.26100.2314]
WSL Version
2.3.26.0
Are you using WSL 1 or WSL 2?

[x] WSL 2
[ ] WSL 1

Kernel Version
5.15.167.4-1
Distro Version
Debian 2024.3 ( Kali-Linux)
Other Software
No response
Repro Steps
sorry if this is too long but first time posting an issue
so i have wsl2 running Kali Linux with network mode made into mirrored thru .wslconfig file, it has been working great
now when i come to solve on HTB CTF machines, i connect to platform using OpenVPN from my host machine (Windows 11)
now when i try to ping the challenge ( to check for connectivity ) I want to  solve on HTB using Windows host it works fine and i receive a response back
when i try to do the same on my kali wsl2 there is no response back, although it was working fine before
when i check ifconfig, the IP VPN TUN (eth) is there which means it got attached successfully
└─$ ifconfig eth0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500         inet 10.10.16.3  netmask 255.255.254.0  broadcast 10.10.17.255
from windows
` ping 10.10.11.25
Pinging 10.10.11.25 with 32 bytes of data:
Reply from 10.10.11.25: bytes=32 time=186ms TTL=63
Reply from 10.10.11.25: bytes=32 time=68ms TTL=63
Reply from 10.10.11.25: bytes=32 time=66ms TTL=63
Reply from 10.10.11.25: bytes=32 time=67ms TTL=63
Ping statistics for 10.10.11.25:
Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
Minimum = 66ms, Maximum = 186ms, Average = 96ms`
from WSL
└─$ ping 10.10.11.25 PING 10.10.11.25 (10.10.11.25) 56(84) bytes of data. //stuck
What I tried to debug:
I tired another platform (THM) which uses same way of reaching the machines through OpenVPN and same issue still present
I tried to disable and re-enable WSL from Windows Features
enabled port forwarding
add rule for firewall
disable firewall completely ( bad idea )
I have checked multiple issues before posting here, all point towards the same direction but no final solution
so here is a conclusion for the issue :

When I connect to the vpn from the OpenVPN client in Windows, I can ping the machines just fine, but when starting a listener inside WSL it seems to not be reachable from the VPN's Network.

I then tried to connect to the VPN using the OpenVPN client within WSL, which fixes the issue, but (of course) prevents me from accessing a box's http server from outside WSL (Windows Host).

Expected Behavior
to be able to ping the machines from WSL and receive a response back and be able to access the challenge
Actual Behavior
ICMP response is stuck and hold, no response back
Diagnostic Logs
WslLogs-2024-11-26_13-07-48.zip
WslNetworkingLogs-2024-11-26_13-04-01.zip

thanks for reporting the issue. unfortunately, OpenVPN is known to be incompatible with mirrored mode at the moment. This is tracked in https://github.com/microsoft/WSL/issues/10879

resolving this is a duplicate, we are currently using the other issue to track this problem
/dupe #10879

authors: CatalinFetoiu ymuuuu

repo: microsoft/WSL

license: MIT


text:
wsl --list in powershell script, many unexpected space characters

I need to wsl -l in my dotfiles, but it doesn't behave correctly, when I use this command with pipe or redirection, output looks like this:
W i n d o w s   S u b s y s t e m   f o r   L i n u x   D i s t r i b u t i o n s :

 U b u n t u   ( D e f a u l t )

 L e g a c y

 U b u n t u - t m p

I'm talking about these commands:
wsl -l | echo
wsl -l | sls ubuntu
wsl -l > file.txt
Does anybody have any idea how to workaround this problem?

but it doesn't behave correctly

Are you sure? Does your script parse the output if the string "(Default)" or an --import distribution name that is Kanji or Chinese text?
It's correct, just, well, unfortunate. You can probably get to where you want to be from Linux with:
$ wsl.exe -l | iconv -c -f utf16 -t utf8

From Powershell, I'm not qualified to repeat the Google hits.

I can leave without this wsl -l check in my script for now, but would be much helpful to mark this as a bug for later investigation, if not we can close this issue.
@therealkenc thank you for advice.

It is not a bug. It is powershell's by-design behavior. And unrelated to wsl.exe, which does not output ASCII because it's output is internationalized.

As mentioned in #4607, it is unexpected that WSL outputs multi-byte codepoints. I suspect this bug was closed too hastily. It's unclear if this even went through proper triage; it doesn't appear @therealkenc works at Microsoft...

It doesn't work correctly in any console, powershell, cmd or conemu, the same problem.

authors: riverar silverqx therealkenc


text:
Importing the distribution failed.

Microsoft Windows [Version 10.0.19041.84]
Input at powershell:
PS C:\WINDOWS\system32> wsl --set-version Ubuntu 2
Output:
Conversion in progress, this may take a few minutes... For information on key differences with WSL 2 please visit https://aka.ms/wsl2 Importing the distribution failed. ./var/lib/gems/2.5.0/doc/test-unit-3.3.5/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/js/jquery.js' does not exist. ./var/lib/gems/2.5.0/doc/test-unit-3.3.5/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Light.ttf' does not exist. ./var/lib/gems/2.5.0/doc/test-unit-3.3.5/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-LightItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/test-unit-3.3.5/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Regular.ttf' does not exist. ./var/lib/gems/2.5.0/doc/test-unit-3.3.5/rdoc/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-RegularItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/strscan-1.0.3/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/js/jquery.js' does not exist. ./var/lib/gems/2.5.0/doc/strscan-1.0.3/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Light.ttf' does not exist. ./var/lib/gems/2.5.0/doc/strscan-1.0.3/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-LightItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/strscan-1.0.3/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Regular.ttf' does not exist. ./var/lib/gems/2.5.0/doc/strscan-1.0.3/rdoc/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-RegularItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/stringio-0.1.0/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/js/jquery.js' does not exist. ./var/lib/gems/2.5.0/doc/stringio-0.1.0/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Light.ttf' does not exist. ./var/lib/gems/2.5.0/doc/stringio-0.1.0/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-LightItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/stringio-0.1.0/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Regular.ttf' does not exist. ./var/lib/gems/2.5.0/doc/stringio-0.1.0/rdoc/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-RegularItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rdoc-6.2.1/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/js/jquery.js' does not exist. ./var/lib/gems/2.5.0/doc/rdoc-6.2.1/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Light.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rdoc-6.2.1/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-LightItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rdoc-6.2.1/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Regular.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rdoc-6.2.1/rdoc/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-RegularItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rake-13.0.1/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/js/jquery.js' does not exist. ./var/lib/gems/2.5.0/doc/rake-13.0.1/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Light.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rake-13.0.1/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-LightItalic.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rake-13.0.1/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/webrick-1.6.0/rdoc/fonts/Lato-Regular.ttf' does not exist. ./var/lib/gems/2.5.0/doc/rake-13.0.1/rdoc/fonts/Lato-RegularItalic.ttf
Wondering how to get the log.

Going to be same /dupe #4457.

authors: ChihweiLHBird therealkenc


text:
windows clipboard not working on WSL 2 - Debian

Environment
Windows:
Win32NT             10.0.19041.0 Microsoft Windows NT 10.0.19041.0
WSL Debian:
PRETTY_NAME="Debian GNU/Linux 10 (buster)"
NAME="Debian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
WSL 2
Linux version 4.19.128-microsoft-standard (oe-user@oe-host) (gcc version 8.2.0 (GCC)) #1 SMP Tue Jun 23 12:58:10 UTC 2020
Steps to reproduce
Command:        cat ./main.cpp | clip.exe
Trying to copy the content of a file from wsl debian to my windows clipboard.
Expected behavior
Expected to copied the content on the clipboard
Actual behavior
Output:            -bash: clip.exe: command not found

PRETTY_NAME="Debian GNU/Linux 10 (buster)"
Output: -bash: clip.exe: command not found

/dupe #5779 (message)

authors: masnm therealkenc


text:
Question: API and Error exception data

Is there a link that shows all end points and what they do and what they are required to call said endpoints? Is there also a link where i can see all error codes and what they are?

Have a look at the below link which shows all the endpoints:
https://github.com/microsoft/WinAppDriver/blob/master/Docs/SupportedAPIs.md
Error codes they are not specific.

repo: microsoft/WinAppDriver


text:
ProximityDevice NFC events not triggered / NFC doesn't work at all

Describe the bug

was first posted as issue here: https://github.com/microsoft/microsoft-ui-xaml/issues/9358
then here: https://learn.microsoft.com/en-us/answers/questions/1604064/windows-app-proximitydevice-nfc-events-not-trigger
reported via feedback hub...
contacted our Microsoft consultant
He sent us back to this... this is really frustrating - nobody really cares

We have an app that we migrated from UWP to WinUI 3 a while ago.
Unfortunately, we discovered that with the WinUI 3 app the ProximityDevice NFC feature no longer triggers events / no longer works at all. We use simple code that can be found in many examples and have set
<DeviceCapability Name="proximity"/>
var proximityDevice = Windows.Networking.Proximity.ProximityDevice.GetDefault();
var messageSubscriptionId = proximityDevice.SubscribeForMessage("NDEF", (device, message) =>
{
    Console.WriteLine(message.Data.ToArray());
});

this works in UWP and doesn't work in WindowsAppSDK/WinUI3 App
See min Repo examples
Works: https://github.com/minze-it/BlankProximityUWP
Doesn't work: https://github.com/minze-it/BlankProximityMaui
Steps to reproduce the bug

Start min repo apps
Click Button
Try read a NDEF 213/215/216 Tag which contains a NDEF message

Expected behavior
SubscribeForMessage delegate handler is invoked
Screenshots
No response
NuGet package version
Windows App SDK 1.5.2: 1.5.240404000
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
What do I need to do to use NFC in a WindowsAppSDK app, or where should I report a bug for errors affecting Windows.Networking.Proximity?

I did - from my private notebook... Not even a reaction or comment within nealy 2 month. Nothing. Nobody at Microsoft cares, like the old 3K closed "stale" issues in the WinUI xaml repository.
Business notebook:

I'm going around in circles trying to find a contact person who is actually willing to deal with a reported problem.
And our customer waits and waits for the feature to work again.
...instead of someone simply taking the courage to pass it on internally to the appropriate department

I think this repository is the right place

I think this repository is the right place

If you look around, then you will find that this is part of the footer for this template. It is even present for CoreApplicationView. This is telling for two reasons.

The Windows App SDK doesn't support UWP applications.
CoreApplicationView isn't available in desktop applications. This is documented too.

But I guess that right now you would need someone from Microsoft to comment on this issue.

1. The Windows App SDK doesn't support UWP applications.

2. CoreApplicationView isn't available in desktop applications. This is [documented too](https://learn.microsoft.com/en-us/windows/apps/desktop/modernize/desktop-to-uwp-supported-api#core-unsupported-classes).

Could you give us a more detailed explanation on those two reasons? It's not about UWP supporting WinAppSDK or not, It's about the Proximity Device as part of the WinRT-API that is working as part of a UWP-App but simply is not when used inside a WinAppSDK-App. And regarding to the mentioned docs Windows.Networking.Proximity.ProximityDevice is not unsupport so everyone assumes it's supported.

Could you give us a more detailed explanation on those two reasons? It's not about UWP supporting WinAppSDK or not, It's about the Proximity Device as part of the WinRT-API that is working as part of a UWP-App but simply is not when used inside a WinAppSDK-App. And regarding to the mentioned docs Windows.Networking.Proximity.ProximityDevice is not unsupport so everyone assumes it's supported.

To put it bluntly:

CoreApplicationView, which is a UWP only runtime class has the exact same link for get product support. As shown, CoreApplicationView is not supported in desktop applications. The Windows App SDK only supports being used in desktop applications.
This means that a UWP only class is being directed to a support forum that does not officially support that runtime class. This means that there is a mistake in the template, and they probably mistook it for the non existent Windows SDK tag.

I've contacted owners of the Proximity APIs to ask for help with this issue.

I've contacted owners of the Proximity APIs to ask for help with this issue.

wanted to ask briefly if they have already commented on the problem. Thank you

I am encountering the same issue as described above. I would appreciate it if anyone could share any tips or solutions that might help in resolving this problem.

authors: DarranRowe baaaaif codendone itmsau01 minze-it

repo: microsoft/WindowsAppSDK


text:
feat(mvp-ado-extension-behavior): update version number to 1.0.0

Details

Update task.json and ado-extension.json to show v1.0.0
Motivation

For release
Context

Pull request checklist

[n/a] Addresses an existing issue: Fixes #0000
[n/a] Added relevant unit test for your changes. (yarn test)
[n/a] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[x] Ran precheckin (yarn precheckin)
[x] (after PR created) The Accessibility Checks (pull_request) check should fail. All other checks should pass.

Codecov Report

Merging #859 (d15c436) into main (f193f3c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head d15c436 differs from pull request most recent head 0ba1224. Consider uploading reports for the commit 0ba1224 to get more accurate results

@@           Coverage Diff           @@
##             main     #859   +/-   ##
=======================================
  Coverage   87.36%   87.36%
=======================================
  Files          39       39
  Lines         823      823
  Branches       97       97
=======================================
  Hits          719      719
  Misses        104      104

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 f193f3c...0ba1224. Read the comment docs.

repo: microsoft/accessibility-insights-action


text:
CI: Update pip and install wheel

Without wheel installed, installing a package that doesn't have the modern pyproject.toml yet to specify build dependencies, pip will build the package from the legacy setup.py. This can take significantly more time than installing from a pre-built wheel.

This reduced the time of the step that installs the Python dependencies from ~45 minutes to ~1 minute.

repo: microsoft/aiida-pyscf


text:
Consume appcenter-file-upload-client-node as a package

The original PR's description:

It was decided to treat mc-fus as a separate library and publish it to npm without creating a separate repo.
This PR makes it a proper library and moves to the root of the repo (for visibility).
Also did some minor renaming to differentiate it with the original mc-fus library for web.
This PR also adds an additional step to CI that builds the library.

The changes to the original PR:

[x] fixed vulnerabilities in the ac-fus-lib dev dependencies
[x] ac-fus-lib connected as an npm dependency in the main package.json, it should fix the linux issue
[x] ac-fus-lib types are imported from the npm dependency

Tested:

[x] manually
[x] with npm test and npm test:ci
[x] ci tests

Related work items:
AB#82467
AB#82466

Thanks for working on this. There is still a build error with integration tests (the ones in powershell).

@jp-andre it has not appeared on the 2nd run. Probably a flaky one, since we're talking to the real AppCenter API.

This overall looks great to me. I would have put the .gitignore stuff in a separate file in the sub folder but the effect is the same. JP and I were thinking of adding a package installation test just as a followup to the incident that caused the revert of the previous version of this change. It's not related to the new package, but it would be a nice hardening followup to what was caused by a previous release of this change.

It's included in this PR as I requested it, though this is only a simple test here, that wouldn't have caught the linux-only sudo issue.

authors: annakocheshkova jp-andre russelarms

repo: microsoft/appcenter-cli


text:
Distribution step in build fails with "Error: App specific password is mandatory for 2fa enabled account in service connection id"

What App Center service does this affect?
Distribution
Describe the bug
I have a build configured to distribute to TestFlight. However, it fails with the error in the title.
To Reproduce
Steps to reproduce the behavior:

Connect to App Store Connect Users (TestFlight) using an Apple ID with 2FA setup.
Setup an app-specific password following instructions at https://docs.microsoft.com/en-us/appcenter/distribution/stores/apple#adding-two-factor-authentication.
Configure a build using provisioning profile and p12 cert to publish to TF.

Expected behavior
Assuming the build passes, the IPA should get uploaded to TF.
Screenshots
None.
Desktop (please complete the following information):
NA
Smartphone (please complete the following information):
NA
Additional context
None.

@FunkyLambda sorry for the lack of response here. Is this still a problem? If it still is a problem, can I ask you use our support system (the blue chat icon) in the App Center portal?
The reason for this is that we will have to get URLs from your build where it fails.

repo: microsoft/appcenter


text:
Help listing repositories

I am trying to get a list of all repositories under a project.
I was able to get the desired project however I am stuck on the build.ListRepositories call
var name string = *projectRef.Name
var ProviderType string = "Git"

buildClient, err := build.NewClient(ctx, connection)
if err != nil {
    log.Fatal(err)
}

response, err := buildClient.ListRepositories(ctx, build.ListRepositoriesArgs{Project: projectRef.Name, ProviderName: &ProviderType })

Using ProviderType "Git" yields
Value cannot be null. Parameter name: serviceEndpointId
I browsed to https://dev.azure.com/{org}/{project}/_apis/serviceendpoint/endpoints?api-version=7.0
And found there are no endpoints
I browsed to https://dev.azure.com/{org}/{project}/_apis/sourceproviders?api-version=7.0
and found the following options:

TfsVersionControl
GitHubEnterprise
Svn
GitHub
BitBucket
Git
TFSGit

When trying TfsVersionControl or TfsGit the ListRepositories() call never resolves.
We are using Microsoft's hosted Azure Devops (dev.azure.com) for hosting.
I tried curling the repositories endpoint using my PAT base64 encoded as a bearer token.
curl -L -X GET \ -H "Authorization $PAT" \ "https://dev.azure.com/{org}/{project}/_apis/git/repositories?api-version=7.0"

When running this I get an error

Microsoft Internet Explorer's Enhanced Security Configuration is currently enabled on your environment. This enhanced level of security prevents our web integration experiences from displaying or performing correctly. To continue with your operation please disable this configuration or contact your administrator.

is it possible I am correctly using buildClient.ListRepositories() and it's not catching this error?
Do we have to create a service endpoint in order to list repositories?

I think that you should use different client (git instead of build):
var name string = *projectRef.Name

gitClient, err := git.NewClient(ctx, connection)
if err != nil {
    log.Fatal(err)
}

repositories, err := gitClient.GetRepositories(ctx, git.GetRepositoriesArgs{
	Project: &name,
})

authors: mathieuszek mikeymop

repo: microsoft/azure-devops-go-api


text:
[ArtifactEngine] Update version to 1.0.0

Introduce artifact-engine version "1.0.0"
Upgrade handlebars  to resolve vulnerabilities for dependent pipeline tasks
Upgrade azure-pipelines-task-lib to "^3.1.0" version

Hi @harshiniks, @narula0781, Could someone please take a look at this PR?

chandrasekharmudili@github.com is the person you can contact on merging and releasing.

Published latest version to https://www.npmjs.com/package/artifact-engine.

@chandrasekharmudili, @harshiniks Thanks!

repo: microsoft/azure-pipelines-extensions


text:
HelmDeploy@0 fails when installing nginx ingress with internal loadbalancer

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: HelmDeploy
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: Hosted Ubuntu 1604

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
The HelmDeploy task doesn't parse annotations properly when set as arguments regardless of them being quoted or not. The value for the service.beta.kubernetes.io/azure-load-balancer-internal annotation is parsed as a boolean when it's supposed to be a string. I've tried different methods of quoting it but without luck. I've also tried moving it to the valueFile but that fails with a different error, also here I've tried quoting it in different ways. Running the same command from the CLI works.
kubectl: 1.16.9
helm: 2.16.7
repo: https://helm.nginx.com/stable
- task: HelmDeploy@0
  displayName: Upgrade/install nginx-ingress
  inputs:
    azureSubscriptionEndpoint: 'myServiceConnection'
    azureResourceGroup: 'myRG'
    kubernetesCluster: 'myAKS'
    command: upgrade
    install: true
    chartType: Name
    chartName: nginx-stable/nginx-ingress
    releaseName: nginx
    namespace: nginx
    valueFile: nginx-affinity.yaml
    arguments: '--set replicaCount="2" --set loadBalancerIP="10.0.0.10" --set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-internal"=\"true\" --set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-internal-subnet"="mySubnet"'

Setting the annotation in the arguments (with the value unquoted) results in the following command:
helm upgrade --namespace nginx --install --values /.../nginx-affinity.yaml --wait --set replicaCount=2 --set loadBalancerIP=10.0.0.10 --set controller.service.annotations.service\.beta\.kubernetes\.io/azure-load-balancer-internal=true --set controller.service.annotations.service\.beta\.kubernetes\.io/azure-load-balancer-internal-subnet=mySubnet nginx nginx-stable/nginx-ingress

Which fails with the following error:
##[error]Error: release nginx failed: Service in version "v1" cannot be handled as a Service: v1.Service.ObjectMeta: v1.ObjectMeta.Annotations: ReadString: expects " or n, but found t, error found in #10 byte of ...|nternal":true,"servi|..., bigger context ...|beta.kubernetes.io/azure-load-balancer-internal":true,"service.beta.kubernetes.io/azure-load-balance|...

Setting the annotation in the arguments (with the value quoted) results in the following command where the azure-load-balancer-internal syntax is broken:
helm upgrade --namespace nginx--install --values /.../nginx-affinity.yaml --wait --set replicaCount=2 --set loadBalancerIP=10.0.0.10 --set controller.service.annotations.service\.beta\.kubernetes\.io/azure-load-balancer-internal=\true" --set controller.service.annotations.service\.beta\.kubernetes\.io/azure-load-balancer-internal-subnet=mySubnet nginx nginx-stable/nginx-ingress

Setting the annotation in the valueFile results in the following error:
##[error]panic: interface conversion: interface {} is []interface {}, not map[string]interface {}

Running this from the command line works:
helm upgrade --namespace nginx --install --set controller.service.loadBalancerIP="10.0.0.10"  --set controller.replicaCount="2" --set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-internal"=\"true\" --set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-internal-subnet"="mySubnet" -f nginx-affinity.yaml nginx nginx-stable/nginx-ingress

Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

@alert101 An override.yaml value file seems to be the best approach for key-values with double quotes. Can you please share your valueFile that you used?
Also, for single quotes

Here's the override.yaml with double quotes:
controller:
  replicaCount: 1
  service:
    loadBalancerIP: "10.0.0.10"
    annotations:
    - service.beta.kubernetes.io/azure-load-balancer-internal: "true"
    - service.beta.kubernetes.io/azure-load-balancer-internal-subnet: "mySubnet"

Error is:
Error: YAML parse error on nginx-ingress/templates/controller-service.yaml: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal array into Go struct field .metadata.annotations of type map[string]string
I also tried with single quotes:
...
    - service.beta.kubernetes.io/azure-load-balancer-internal: 'true'
    - service.beta.kubernetes.io/azure-load-balancer-internal-subnet: 'mySubnet'

Same error as above.
And with double single quotes:
...
- service.beta.kubernetes.io/azure-load-balancer-internal: ''true''
- service.beta.kubernetes.io/azure-load-balancer-internal-subnet: ''mySubnet''
Different error:
Error: failed to parse /.../override.yaml: error converting YAML to JSON: yaml: line 5: did not find expected key

Trying your workaround with double single quotes in the args results in this command:

helm upgrade --namespace nginx --install --values /.../override.yaml --wait --set replicaCount=1 --set loadBalancerIP=10.0.0.10 --set controller.service.annotations.'service.beta.kubernetes.io/azure-load-balancer-internal'='true' --set controller.service.annotations.'service.beta.kubernetes.io/azure-load-balancer-internal-subnet'=mySubnet nginx nginx-stable/nginx-ingress
Which results in this error:

##[error]Error: release nginx failed: Service "nginx-nginx-ingress" is invalid: [metadata.annotations: Invalid value: "'service.beta.kubernetes.io/azure-load-balancer-internal'": prefix part a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is 'a-z0-9?(.a-z0-9?)'), metadata.annotations: Invalid value: "'service.beta.kubernetes.io/azure-load-balancer-internal'": name part must consist of alphanumeric characters, '-', '' or '.', and must start and end with an alphanumeric character (e.g. 'MyName',  or 'my.name',  or '123-abc', regex used for validation is '([A-Za-z0-9][-A-Za-z0-9.])?[A-Za-z0-9]'), metadata.annotations: Invalid value: "'service.beta.kubernetes.io/azure-load-balancer-internal-subnet'": prefix part a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is 'a-z0-9?(.a-z0-9?)'), metadata.annotations: Invalid value: "'service.beta.kubernetes.io/azure-load-balancer-internal-subnet'": name part must consist of alphanumeric characters, '-', '' or '.', and must start and end with an alphanumeric character (e.g. 'MyName',  or 'my.name',  or '123-abc', regex used for validation is '([A-Za-z0-9][-A-Za-z0-9.])?[A-Za-z0-9]')]
I'm guessing single quotes isn't a valid syntax. I also tried different variations of the workaround but none worked, either the same error as this last one or the first error.

Can you try with this override file:
controller:
  replicaCount: 1
  service:
    loadBalancerIP: "10.0.0.10"
    annotations:
      "service.beta.kubernetes.io/azure-load-balancer-internal": "true"
      "service.beta.kubernetes.io/azure-load-balancer-internal-subnet": "mySubnet"

I'll be damned, that worked. I thought I had tried that syntax but apparently not.
Thank you very much!

repo: microsoft/azure-pipelines-tasks


text:
Cache File Not Found - But it's not a filename

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Cache
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number: This is sensitive information

Agent - Hosted or Private:

Hosted: ubuntu-18.04

Issue Description
The cache task thinks that the string "GeneralCache6.0.0.17" is a file whereas the documentation says double quotes should force it to be a string.
Task logs
2020-11-14T06:06:23.6022508Z ##[section]Starting: Check General Cache
2020-11-14T06:06:23.6029050Z ==============================================================================
2020-11-14T06:06:23.6029337Z Task         : Cache
2020-11-14T06:06:23.6029554Z Description  : Cache files between runs
2020-11-14T06:06:23.6029787Z Version      : 2.0.1
2020-11-14T06:06:23.6029999Z Author       : Microsoft Corporation
2020-11-14T06:06:23.6030270Z Help         : https://aka.ms/pipeline-caching-docs
2020-11-14T06:06:23.6030581Z ==============================================================================
2020-11-14T06:06:23.8989014Z Resolving key:
2020-11-14T06:06:23.9488147Z  - GeneralCache6.6.0.17 [file] (not found)
2020-11-14T06:06:23.9636220Z ##[error]One or more errors occurred. (File not found: GeneralCache6.6.0.17)
2020-11-14T06:06:23.9712426Z ##[section]Finishing: Check General Cache```

Pipeline:
    - task: Cache@2
      displayName: Check General Cache
      inputs:
        key: "GeneralCache$(SomeVersionNumber)"
        path: $(CachePath)
        cacheHitVar: GENERAL_CACHE_RESTORED

I am not sure if $(SomeVersionNumber) is causing the confusion.
Documentation:

To avoid a path-like string segment from being treated like a file path, wrap it with double quotes, for example: "my.key" | $(Agent.OS) | key.file

This is YAML fighting against you. It is “eating” the double quotes before the task gets it. Wrap the double quotes in single quotes or use the pipe/newline syntax so the YAML parser passes the double quotes to the task

So this:
- task: Cache@2
  displayName: Check General Cache
  inputs:
    key: '"GeneralCache$(SomeVersionNumber)"'
    path: $(CachePath)
    cacheHitVar: GENERAL_CACHE_RESTORED

OR
- task: Cache@2
  displayName: Check General Cache
  inputs:
    key: |
      "GeneralCache$(SomeVersionNumber)"
    path: $(CachePath)
    cacheHitVar: GENERAL_CACHE_RESTORED

authors: cawoodm johnterickson


text:
Validate CI

Task name:
Description:
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

/azp run


text:
./ not being handled correctly

My YAML file:
          - task: CopyFiles@2
            displayName: 'YEHAW Copy Files to: $(Build.ArtifactStagingDirectory)'
            inputs:
                sourceFolder: '$(Build.SourcesDirectory)'
                contents: './_buildOutput/**/*'
                targetFolder: '$(Build.ArtifactStagingDirectory)'

Azure pipeline output:
##[debug]pattern: './_buildOutput/**/*'
##[debug]rooted pattern: '/azp/agent/_work/1/s/./_buildOutput/**/*'

As you can see, it includes the ./ in the final path. It should resolve to  '/azp/agent/_work/1/s/_buildOutput/**/*'

Moved into Developer Community: https://developercommunity.visualstudio.com/content/problem/1007438/-not-being-handled-correctly.html
Please use "Follow" button on the right to track it

@basickarl: Please go to this link and answer Microsoft team: https://developercommunity.visualstudio.com/content/problem/1007438/-not-being-handled-correctly.html

authors: Saibamen basickarl

repo: microsoft/azure-pipelines-yaml


text:
use helm newest version can't access a repo

Scenario which you are working on
A link to the Jumpstart scenario you are working on.
I was trying to create a custom location to onboard a k8s cluster.
Describe the bug
A clear and concise description of what the bug is.
It's an issue with the helm version. I install helm 3.5.0 (with the binary), and try to access this repo, and got an error
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
helm version
version.BuildInfo{Version:"v3.5.0", GitCommit:"32c22239423b3b4ba6706d450bd044baffdcf9e6", GitTreeState:"clean", GoVersion:"go1.15.6"}
The problem is similar to this issue,
https://github.com/helm/helm/issues/8761#issuecomment-695099518
After I downgrade helm to 3.2.4 (replace the binary file), it worked.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
"azurearcfork8sdev" has been added to your repositories
In the website, we mentioned helm 3. Perhaps more specific version it should be for example 3.2.4 because latest 3.5.0 will have this issue
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@azurepeterli I am not following. Which scenario you are working against?

@azurepeterli I am not following. Which scenario you are working against?

Hi thanks for getting back to me. I can't find the exact web page which mentioned "Helm 3" somehow.
I was creating a K8s connected cluster and I need install helm for k8s package management according to the document.
But it didn't specify the exact version (or only helm 3), so I install helm with newest version 3.5.0, but it would run into an issue when trying to access this repo.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
After I downgrade helm from 3.5.0 to 3.2.4 (simply replace the binary file), I could access that repo.
This problem was with helm 3.5.0, and there was a similar issue reported here,
https://github.com/helm/helm/issues/8761
I suggest in our document, if we mentioned "helm" or "helm 3", we add a note that latest Helm 3.5.0 may be problematic when accessing some repos, if this happens,  try downgrade helm to an older version such as 3.2.4 (simply replace the binary file)

Hi thanks for getting back to me. I can't find the exact web page which mentioned "Helm 3" somehow.
I was creating a K8s connected cluster and I need install helm for k8s package management according to the document.
But it didn't specify the exact version (or only helm 3), so I install helm with newest version 3.5.0, but it would run into an issue when trying to access this repo.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
After I downgrade helm from 3.5.0 to 3.2.4 (simply replace the binary file), I could access that repo.
This problem was with helm 3.5.0, and there was a similar issue reported here,
https://github.com/helm/helm/issues/8761
I suggest in our document, if we mentioned "helm" or "helm 3", we add a note that latest Helm 3.5.0 may be problematic when accessing some repos, if this happens,  try downgrade helm to an older version such as 3.2.4 (simply replace the binary file)

My question is in what scenario you are trying to run since none of the k8s scenarios are using these helm repo. Can you please clarify? Do you have an issue with helm 3.5 and a scenario?
This does not look like an Azure Arc Jumpstart issue and it looks you are trying something that is not supported.

My question is in what scenario you are trying to run since none of the k8s scenarios are using these helm repo. Can you please clarify? Do you have an issue with helm 3.5 and a scenario?
This does not look like an Azure Arc Jumpstart issue and it looks you are trying something that is not supported.

I was trying to create a k8s connected cluster using AKS, and onboard it through Arc.
Here is the document mentioned helm v3,
https://docs.microsoft.com/en-us/azure/aks/quickstart-helm
You're right looks like it's an AKS document, not Arc Jumpstart issue.
I'll close the issue, thanks!

I was trying to create a k8s connected cluster using AKS, and onboard it through Arc.
Here is the document mentioned helm v3,
https://docs.microsoft.com/en-us/azure/aks/quickstart-helm
You're right looks like it's an AKS document, not Arc Jumpstart issue.
I'll close the issue, thanks!

authors: azurepeterli likamrat


text:
Incorrect Encoding (Cyrillic symbols) when create stored procedure from file with encoding (Windows 1251)

Azure Data Studio Version: 1.24.0

After drop and create stored procedure from file (.sql) with encoding (1251), sometimes ADS create procedure with wrong encoding and something with Cyrillic symbols get mess....

I need to check every time (and yes it's very often) everything that executing and if some get wrong I need to do some actions like - reload window, copy paste query text in new window, close and open ADS..... and this repeat until ADS decide to run query with proper encoding :(
It's crazy..

@tanyogeorgiev Can you please provide me with an example of a problematic SQL file? I am unable to reproduce the issue using a sample database on my server that has Cyrillic characters on it either with UTC-8 encoding or Windows 1251 encoding, or following the example here: https://stackoverflow.com/questions/18325680/sql-server-2012-express-do-not-understand-russian-letters
However, it seems that when creating a table containing Cyrillic characters, the data won't be stored properly unless you put a 'N' before the string, else it won't display properly.

@tanyogeorgiev Can you please provide me with an example of a problematic SQL file? I am unable to reproduce the issue using a sample database on my server that has Cyrillic characters on it either with UTC-8 encoding or Windows 1251 encoding, or following the example here: https://stackoverflow.com/questions/18325680/sql-server-2012-express-do-not-understand-russian-letters
However, it seems that when creating a table containing Cyrillic characters, the data won't be stored properly unless you put a 'N' before the string, else it won't display properly.

@smartguest This is the file (zipped) that I use for my screenshots now:
SP ttt.zip
1.Open file in ADS - everything is looking pretty good (Pastelin Theme ROCK 💯)

2.Execute (drop and create) SP

3. sp_helptext - horrifying :(

4. Execute the SP - much more horrifying  :(

@smartguest This is the file (zipped) that I use for my screenshots now:
SP ttt.zip
1.Open file in ADS - everything is looking pretty good (Pastelin Theme ROCK 💯)

2.Execute (drop and create) SP

3. sp_helptext - horrifying :(

4. Execute the SP - much more horrifying  :(

sp_helptext works with the latest build when testing, however exec still has issues regarding encoding, I will continue to investigate this issue and will let you know of any updates.

sp_helptext works with the latest build when testing, however exec still has issues regarding encoding, I will continue to investigate this issue and will let you know of any updates.

Backlog? Guys this mean that we should change the ADS with another IDE/Editor... Come on, is this is your goals?

authors: smartguest tanyogeorgiev

repo: microsoft/azuredatastudio


text:
Manually closed unsaved queries keep opening with Azure Data Studio

Issue Type: Bug
After closing (manually) unsaved queries, said queries open up again if Azure Data Studio is closed and opened again (even with shutdown of pc in between).
Steps to reproduce:

Close query windows
Close Azure Data Studio
Open Azure Data Studio

Azure Data Studio version: azuredatastudio 1.25.1 (e7eacc32c0d20bd4da843320c3bfac231b308c98, 2020-12-11T01:59:24.271Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7300U CPU @ 2.60GHz (4 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.88GB (2.07GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

azdata-sanddance
msr
3.1.0

poor-sql-formatter
WSR
0.1.0

Issue 8792 has a comment which suggests removing:
c:\Users[your username]\AppData\Roaming\azuredatastudio\Backups[random number]\untitled\
Removing these files stopped the reported behaviour.

this keeps on happening -  need to constantly remove the files from \Backups[random number]\untitled\

Still having this exact issue, would be great if there was a fix for this.

I did a quick test with latest build on both Win and Mac (Intel) and I was not able to repro the issue. I'll close the issue for now but feel free to re-open it if you are still seeing this issue and/or have more information for the repro.

authors: MoishyBL alogemann ayamPedas caohai jayden-macrae


text:
Switching notebook kernels doesn't disconnect code cells

Create a Notebook and add a code cell
Select SQL kernel
Write a T-SQL query and execute it
Switch to Kusto kernel
Run cells

The cells still run successfully. We should be disconnecting the connection and prompting the user for a new one either when we switch kernels or when they try to run the code cell again.

Tested on stable. Not a regression.

authors: Charles-Gagnon VasuBhog


text:
[SQL Migration] Missing SKU recommendation and assessment functionality

Azure Data Studio Version: 1.38.0
OS Version: Windows 11 21H2

Steps to Reproduce:

Launch SQL Migration wizard, proceed to assessment and SKU recommendation steps
Observe missing functionality (see below)

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

Note:
This bug is for tracking a port request to the release branch for sqltoolsservice / mssql in ADS. Discussed details internally with product team as part of ADS Shiproom.
Background:
SQL Migration is adding several new features as part of its next extension release, which will happen off-cycle and independently of the ADS product release. However, some of these new features require backend changes to sqltoolsservice, which we are trying to release as part of ADS 1.39. These changes include:

adding the ability to export assessment and SKU recommendation reports to a folder of the user's choosing
adding the elastic SKU recommendation model, which will be made available via a feature switch
fixing some misc bugs regarding logging and error handling in the backend migration service

Submitting this bug / port request now, as these changes did not get checked in on time for the code complete date.
Changes:
Specifically, this is accomplished by adding additional outputs to the migration/getskurecommendations and migration/getassessments requests. Tested and verified that  these changes are backwards compatible (tested by building STS locally, which includes these changes, but consumed from an older version of the SQL Migration extension). The PRs in question are the following:

https://github.com/microsoft/sqltoolsservice/pull/1613
https://github.com/microsoft/sqltoolsservice/pull/1602

PR merged

authors: alanrenmsft raymondtruong


text:
Test coverage improvement

[x] produce the test coverage result for extension unit tests - done by Charles

[ ] upload the results to the drop folder so that the area owner can easily check their code coverage rate (in progress)

[ ] merge the coverage results in to one to utilize the pipeline's code coverage results viewer
there is a limitation in the ADO pipeline, only one result file is accepted.
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/test/publish-code-coverage-results?view=azure-devops

[ ] leverage coveralls

[ ] enable test coverage for integration tests

Closing older engineering issues.  Please reactive if you're still using this to track this work.

authors: alanrenmsft kburtram


text:
Database Projects - Add existing file

This PR fixes #14383

In-line with Visual Studio behavior:

SQL files will be automatically added to build <Build Include="..." />
All other files will be added as <None Include="..." />

If the file has been previously excluded, the <Build/None Remove="..." /> will be removed before adding the Include (if necessary).
For SDK-style projects, if the SQL file falls under default glob, no <Build Include="..." /> will be added.

I'll add some steps to the test plans too after this is checked in

Pull Request Test Coverage Report for Build 1694200852

30 of 38 (78.95%)  changed or added relevant lines in 4 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 48.252%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/models/project.ts
24
25
96.0%

extensions/sql-database-projects/src/controllers/projectController.ts
4
11
36.36%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/controllers/projectController.ts
8
42.38%

Totals

Change from base Build 1694008950:
0.05%

Covered Lines:
14740

Relevant Lines:
29011

💛  - Coveralls

The "Should handle adding existing items to project" test is failing, so please make sure to fix it before checking in. Thanks!

@kisantia the test failure is fixed. It came from the test case I added for adding existing file from outside the project root. There was nothing wrong with the source code, just had to tweak the test code a bit regarding relative paths and Linux specifics.

@kisantia the test failure is fixed. It came from the test case I added for adding existing file from outside the project root. There was nothing wrong with the source code, just had to tweak the test code a bit regarding relative paths and Linux specifics.

Awesome, glad it just needed a small fix. Should be good to check in then after the PR validation finishes!

authors: coveralls kisantia zijchen


text:
Fix apostrophe escape issue in table designer

This PR will fix #25536
Before:

After:

Pull Request Test Coverage Report for Build 10428007515
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 41.725%

Totals

Change from base Build 10424606776:
0.001%

Covered Lines:
30729

Relevant Lines:
68919

💛  - Coveralls

authors: coveralls lewis-sanchez


text:
As an SRE - I would like to be able to deploy a second cluster instance to a shared vnet/keyvault

As a: SRE, I want to be able to deploy a second Azure single key vault instance into an existing VNET and share the existing Keyvault.
Describe the solution you'd like: Currently, the Vnet and Subnet are created within Azure Common Infra.  The subnet is cluster specific (not shared between clusters) so it should likely be moved down to the cluster creation level.
Acceptance Criteria:. I can deploy multiple clusters into the same Vnet.
Describe alternatives you've considered:. Manually create a subnet in the existing vnet.
Does this require updates to documentation?:. Yes.

@jmspring do you have some steps in mind for this? Especially using spk. If so please list

The templates are changing. It will primarily impact azure common infra. Subnet prefix and subnet name will move from common infra to single keyvault.  The impact on spk?  Where the values need to be updated in the definition.yaml

Fixed in PR #1255

authors: andrebriggs jmspring

repo: microsoft/bedrock


text:
[Adaptive] [DCR] Update CancelAllDialogs behavior

Filing this because this came up in the triage today.
JS behavior is to propagate all the way up for CancelAllDialog.
New behavior - stop when it hits a container boundary but use a flag to propagate up. flag defaults to true (propagate up)

I need more details please:

What name for the flag
Where is the flag?  On DialogContainer?  On call to CancelAllDialog()?
@Stevenic ??

authors: tomlm vishwacsena

repo: microsoft/botbuilder-dotnet


text:
Where is dialog.lastActivity? It's used in schema examples but doesn't appear to be real.

See the component schema for Microsoft.DeleteActivity:
https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Schemas/Actions/Microsoft.DeleteActivity.schema#L21
The scenarios for DeleteActivity and UpdateActivity require keeping track of certain activity id's, and dialog.lastActivity is a great example, however it doesn't appear to work.

It's turn.lastresult.id actually.
The turn.lastresult would record the previous one result. If the last action is SendActivity, the lastresult is the Activity.
So, the id of last acticity is turn.lastresult.id

@Danieladu  can you take the chance to update the schema to "turn.lastresult.id" and also please check if there is any test on it.

Here is the test:
https://github.com/microsoft/botbuilder-dotnet/blob/d5726a7d10ea9dc1b446ec23165353d6ed955f01/tests/Microsoft.Bot.Builder.Dialogs.Adaptive.Tests/Tests/ActionTests/Action_DeleteActivity.test.dialog#L17
So, the dialog.lastActivity is assigned manually.

authors: Danieladu boydc2014 cwhitten


text:
TurnContext is disposed when using TwilioAdapter on Proactive Message

Version
4.14.1
Describe the bug
Using TwilioAdapter, I'm unable to start a dialog in a proactive message, and save changes to the turn context. After the activity goes out, TurnContext is disposed and I can't persist the updates to the conversation.
To Reproduce
Steps to reproduce the behavior:

Clone this repo: https://github.com/miguelhughes/ProactiveTwilioDialog
Initialize appsettings with credentials for twilio and a valid bot registration
Navigate to http://localhost:3978/api/notify
Object disposed exception is thrown

Expected behavior
I would like TwilioAdapter to behave as the standard adapter and not dispose the context after sending an activity, or at least give me a way to save changes to the conversation state after a message is sent.
Alternatively, if I must use the standard adapter on the proactive message, I need to be able to specify the sms status callback url on that message.
Additional context
The repo is a trimmed down of my working code, showing the problem. I cobbled it together from different bot framework samples: MultiTurnPromptBot, TwilioAdapterBot and ProactiveBot.
I'm trying to get status callbacks from Twilio. I have extended TwilioAdapter and TwilioClientWrapper to include my StatusCallback url on all messages. When using the standard adapter on conversation start and twilio adapter on replies, status callbacks work fine. But I can't use the modified Twilio adapter on the first message, due to this disposing behavior.
More background of why I'm trying to do this: https://stackoverflow.com/questions/71034954/unable-to-get-twilio-sms-status-callbacks-when-sending-proactive-message

Hey @miguelhughes, I'm investigating this.

Thanks @ramfattah . Is there anything I can do to help?

Hi @miguelhughes,
Thank you for your patience. This is my first time working with Twilio. I followed this documentation Connect a bot to Twilio to get familiar.
I created a free trial account on Twilio to get a phone number along with account credentials. I cloned your repo and in step 3 when navigating to http://localhost:3978/api/notify I get the following error ArgumentException: HttpRequest is missing "x-twilio-signature"
The same issue ArgumentException: HttpRequest is missing "x-twilio-signature" occurs in /63.twilio-adapter/ sample and not just your repo.

My appsettings.json

Not sure if I'm missing a pre-requisite step. Have you faced this problem in development before?

I'm experiencing similar issues where ITurnContext gets disposed when I launch a new thread, I was not having this issue on the -pre versions I'm currently using 4.15.2

@miguelhughes @eallegretta
Adapters previously integrated with the SDK, like Twilio, have been transferred to the BotBuilder-Community-dotnet repo for continued support and ownership.
Closing this issue, you will want to file this as a new issue in that repo for additional support. Thanks.

I found a workaround for this problem, in case anyone runs into this. I posted it on the linked SO item

authors: eallegretta miguelhughes ramfattah


text:
Fix bot state so that the context service key is used consistently to access the cached state

Fixes #3643
A bot state class's "context service key" is used to access the cached bot state in the turn state collection. Because the context service key is usually the name of the type (e.g. "UserState") there were some cases where the SDK was just using the type name instead of the actual context service key. This fails when a bot state class has a context service key that's different from the name of the type, as my new tests show.
In the case of a bot state memory scope, I also fixed the code to not rely on reflection to access the cached bot state. BotStateMemoryScope<T>.GetMemory now accesses the bot state instance in the same way that BotStateMemoryScope<T>.LoadAsync and BotStateMemoryScope<T>.SaveChangesAsync do, and once it has the bot state instance it can simply call BotState.Get. Because the classes that call GetMemory are expecting a dictionary and not a JObject, I modified BotState.Get to return an IDictionary<string, object>. I believe this is an improvement because that's the form that the cached bot state is already in and there's no need to convert it to a JObject. While it may seem like a dangerous change, the only place that BotState.Get is used is in one InspectionMiddleware method, and if any customers are using it and expecting a JObject then the compiler will just tell them to convert it to a JObject on their end, so there won't be any surprising behavior changes. I'm confident that the code is better this way and I can't imagine this breaking anything because the tests are all passing.
I have discovered that the problems I was seeing in BotStateMemoryScope<T> were not about GetMemory returning a JObject instead of a dictionary, they were about BotState.Get returning a copy of the of the state, which means any modifications wouldn't persist. I have also discovered that my "best option" to fix BotStateMemoryScope<T> by having Get return the original dictionary instead of a JObject will not work because it violates the contract. I see four other options:

Create a new public Get method (with a different name) to return the actual dictionary instead of a copy
Create an internal Get method and have the Bot Builder assembly share its members with the dialogs library
Create a private Get method and have BotStateMemoryScope<T>.GetMemory access it using reflection
Don't create any new methods and have BotStateMemoryScope<T>.GetMemory use reflection twice

I went with option 4 because it's the most similar to the way things are currently set up. The reason we need to use reflection twice is because the CachedBotState class is private and the context service key we need to access it is also private.
EDIT: I've thought of a fifth option. Since bot state is meant to be accessed using bot state property accessors, we could try to rework the adaptive dialogs library to access bot state using bot state property accessors. This would probably represent significant work/changes so I would not attempt it without some kind of go-ahead from the SDK team.

OK, here's what I think we should do:

Make existing class CachedBotState public
Make existing method BotState.GetCachedState() method
Change BotStateMemoryScope to use it

           var botState = GetBotState(dialogContext);
           var cachedState = botState.GetCachedState();
           return cachedState.State;

This gets rid of all reflection, and should be correct.

@tomlm - I've implemented your solution, and all checks are passing

looks good, thanks for the changess

authors: tomlm v-kydela


text:
Change LogAction.Text from stringExpression to IActivityTemplate

Fixes #4988
Description
The schema definition of field Text is in conflict with the field type in LogAction.cs
Specific Changes
Updating the schema definition to IActivityTemplate.

@boydc2014, @chrimc62, could you please review this?

If the intent is to support the full activity template this makes sense to me.

In reply to: 743360674 [](ancestors = 743360674)


text:
replyToId not part of token response of oauthprompt

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Versions
   "botbuilder": "^4.21.0",
   "botbuilder-dialogs": "^4.21.0",

Describe the bug
I implemented this sample  in my NodeJS bot. The authentication works is expected. To optimize the userexperience I would like to replace the login button with something else or delete it. Just to show the user he was succesful in his task and make sure he or she does not click the button again (resulting in a confusing error message)
TO be able to do that, I need a replyToId and update or delete the card. For instance like this:
await context.deleteActivity(context.activity.replyToId);
or
const finalCard = CardFactory.adaptiveCard(FinalCard);
finalCard.id = context.activity.replyToId;
const finalMsg = MessageFactory.attachment(finalCard);
finalMsg.id = context.activity.replyToId;
await context.updateActivity(finalMsg);

To Reproduce
Install the sample I mentioned
After         if (tokenResponse) {In maindialog.js add
console.log(JSON.stringify(stepContext.context.activity));
You will notice that there is not repyToId field under activity
Expected behavior
A replyToActivity value to use

@HesselWellema,
Which channel are you using?
"A2092" mentions to allow bots to leave out replyToId for messages if the channel (ex: WebChat or DirectLine) doesn't use it.
https://github.com/microsoft/botbuilder-js/blob/8a3076884bdc688a0cb6dca706ce47d8106b9af0/libraries/botbuilder-core/src/turnContext.ts#L125-L134

Using Directline.

I see what you mean. Thanks for taking time. I will post on SO and ask for alternative ways to get what I need. Will probably  have to solve it in the web-chat code. @Stevkan always has good ideas in matters like these

@HesselWellema - Checked SO and didn't see a post from you (that I noticed) so thought I'd just respond to you here since the conversation is already going and I have a potential answer.
So, it looks like, from your comment above, that the client you are using is Web Chat. That is actually to your benefit in this situation. The OAuth prompt that is rendered in Web Chat uses (or is transformed into) an adaptive card. Because of this, you can use the method demonstrated in the 05.custom-components/l.disable-adaptive-cards sample of the BotFramework-WebChat repo to disable previously clicked on adaptive cards!
You'll notice that there isn't a readme.md file like in the other samples (there really should be), but the code is pretty self-explanatory. To be clear, out-of-the-box this code will disable all previously rendered and clicked on adaptive cards. If you don't mind the digital elbow grease, you could probably further isolate the OAuth prompt. But I'll leave that up to you.

Another option, albeit a less practical one, is you could update the text of the card. This could be done in conjunction with the first option or by itself. This wouldn't be relying on updateActivity or deleteActivity but instead monitoring for the OAuth prompt's containing activity as it is received by Web Chat. When the activity is received, but not fully rendered yet, we do a quick update to the adaptive card's button's text. This method does work and is sometimes necessary, but it's not perfectly clean and it also relies on making changes directly in the UI which isn't a best practice. Additionally, as the underlying code of Web Chat is subject to change in future releases, it could pose a breaking change in your deployment if some relied upon class, etc., in the rendered HTML should change.
Again, this is doable, but the first option is the better option. I implemented this for testing purposes a couple years ago in an instance of Web Chat I still use and I think I had to update the path to the button's text once when an update changed some of the parent element's class names. It's also worth noting that if the page hosting Web Chat is refreshed or reloaded and you are persisting the conversation, the updated text will be lost as it was in response to an activity previously received.
If this second option is of interest, let me know and I can demonstrate how to do this. It's also entirely possible I already demonstrated this on SO in response to someone's posted question, so you may want to check there, first.

Tx for reachting out @stevkan. Appreciate it.
Actually I did find this sample and made a first attempt today to implement it. Was just trying to figure out a way to get these:
Components: {
AdaptiveCardContent,
AnimationCardContent,
AudioCardContent,
HeroCardContent,
OAuthCardContent,
ReceiptCardContent,
SignInCardContent,
ThumbnailCardContent,
VideoCardContent
},
      In my react web-chat project. Not sure how to import or require them.

 The other parts seam pretty straightforward

@HesselWellema - Let take a peek at my test react web chat project and I'll get back to you shortly.

@HesselWellema - You should be able to import Components like below. From there, deconstruct the Components object to access the individual component objects:
import ReactWebChat, { Components } from 'botframework-webchat';
const { AdaptiveCardContent, AnimationCardContent, etc., etc.} from Components;

Hope of help!

tx. I ended up doint it like this:
import ReactWebChat, { Components, createDirectLine } from 'botframework-webchat';
const OAuthCardContent = Components.OAuthCardContent;
I can now do with the oAuth card whatever I want. Great.
However: After authentication, webchat gives a console warning: queueIncomingActivitySaga.js:73 botframework-webchat: Timed out while waiting for activity "4ff2fa83-0b93-4d24-8bc1-4553aeff9a53" which activity "DHbNq7XfNhmAlnAaT5n2wj-eu|0000003" is replying to.
The warning is allways on the first activity send from the bot to the webchat after authentication succeeds.
I will try and reproduce with a minimal set of samples (to eliminate the option that it relates to my own code) and create a bugreport.

Cycling back to you @rampaged. According to the webchat team, they need the replytoId for the oAuthcard implemtation. As a result of the missing id, there is a wait for approx 4 seconds and an error in the console. This problem is reproducable with sample bot. It this something you guys can address?

@stevkan , tx for cycling back. I spent a lot of time trying to solve this and eventually moved away from authentication via the oAuth prompt. Decided to implement authentication outside the webchat. In hindsight, a better approach for this usecase.

authors: HesselWellema rampaged stevkan

repo: microsoft/botbuilder-js


text:
[Localization] Serbian Latin (sr-Latn-RS) uses Cyrillic (sr-Cyrl-RS) currency names

Describe the bug
In the currency converter, the currency names are all written in the Cyrillic script, even though the Latin one is set.
While Serbian is written in two different scripts, and every speaker can read both, you can't mix them in one place like this.
Steps To Reproduce

open calculator
go to currency converter

Expected behavior
Currency names are in the Latin script, just like the rest of the application
Screenshots

Device and Application Information

OS Build: 22000.466
Architecture: x64
Application Version: 11.2110.4.0
Region: sr-latn-RS
Dev Version Installed: no

Additional context

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for filing this issue! Unfortunately, we cannot fix localization issues directly in this project. I am going to close this issue here, but route it to our internal team to investigate.

authors: MicrosoftIssueBot bzzrak grochocki

repo: microsoft/calculator


text:
Users are unable to edit numerical values inserted from memory or history

Describe the bug
Users are unable to edit numerical values inserted from memory or history.
Steps To Reproduce

Perform a calculation in Standard or Scientific mode so that a history entry is generated
Press "MS" button to store memory entry
Press ESC to clear input
Press the previously stored history item from the list
Press Backspace and notice that you cannot edit the numerical value
Press ESC to clear input
Press "MR" button to recall stored memory entry
Press Backspace and notice that you cannot edit the numerical value

Note: This is only kind of related to #519, but in this case, we are only ever restoring numerical values, and not operators.
Expected behavior
Numerical values are editable as if you typed them in manually.
Screenshots
N/A
Device and Application Information

OS Build: 10.0.18942.0
Architecture: X64
Application Version: 10.1906.25.0
Region: en-US
Dev Version Installed: True

Requested Assignment
I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

I have a question. After restoring from history, what should happen to the expression when someone types in some value or an operator?

Great question, @praveensvsrk! After reviewing this bug again, I think it might be a good idea to tease apart memory from history.
I think memory is much more straightforward, and this bug does a great job at capturing the intended behavior. That is, when you insert a value from memory, it should be treated the same as if you inputted a number via keyboard or copy-and-paste, and you should be able to edit the value.
With history, since the entire expression is brought in, you are really inserting the expression + the result of evaluating that expression. It is not as clear as with memory that you should be able to edit that result. #163 is intended to capture some of the nuance there.
I renamed the issue to more accurately capture this.

Thanks for the clarification! :)
I would like to work on this issue!

Awesome, thanks for taking a look!

Sorry for the delay, I got occupied in personal work.
I checked the code flow, we are setting m_bRecord property of CCalcEngine to false when the OpCode is between IDC_FE and IDC_MINUS. IDC_RECALL (Which is the opcode for MR button) falls in this range, hence editing is disabled. Not sure why this was done in the first place, maybe someone here can throw some light on this.
I will submit a fix after some testing.


text:
Improved emissions data averaging

Issue Number: 345
Summary
Implements an improved averaging calculation better suited to time-based data points.
Changes
Per the commit history:

Make AverageOverPeriod extension method public
Integrate AverageOverPeriod into SciScoreAggregator
Remove unused CarbonAwareAggregator averageCalc

Checklist

[x] Local Tests Passing?
[x] CICD and Pipeline Tests Passing?
[x] Added any new Tests?
[x] Documentation Updates Made?
[x] Are there any API Changes? If yes, please describe below.
[x] This is not a breaking change. If it is, please describe it below.

Are there API Changes? No
Is this a breaking change? Yes
Averaging now requires a continuous dataset to be calculated.  Prior to this change the MarginalCarbonIntensity endpoint would average whatever points it received, which does not throw errors, but provides incorrect results.

This makes sense to me, and I'm going to approve it. I have one question: Is this going to break existing integration testing, and does it require an update to the README?

repo: microsoft/carbon-aware-sdk


text:
Zero-fill padding in detach_abi(com_array)

A code analysis warning recently fired for a customer on detach_abi(com_array). This function returns a std::pair<uint32_t, some_pointer_type>, which will have, on 64-bit builds, 4 bytes of padding between the uint32 size and the pointer members. Currently, that padding is uninitialized.
The idea behind the code analysis warning is that information may be leaked via those unitialized bytes.
In practice, that's almost never going to be an issue for this function, because the std::pair is not an interesting object to pass around, and only exists as a convenience to return both the size and buffer of the com_array at the same time.
However, the fix removes a pain point for a customer, is simple, risk-free, and actually gets optimized away in the 99% use case (return value stored in a local variable, access only the first/second members, not the padding bytes).  Demo showing optimization: https://godbolt.org/z/T4vPhMKxn

/azp run

repo: microsoft/cppwinrt


text:
docs: update character screen reference

PR Summary
Commit c2673e439583090f68d8771dc07a8d801be09190 renamed the character screen source code file. This PR adjusts sources to changes.

Thanks!

authors: emmanuel-ferdman pelikhan

repo: microsoft/devicescript


text:
Switch from ExAcquireRundownProtection to ExAcquireRundownProtectionCacheAware

Windows kernel exposes a cache aware / more scalable rundown protection API for rundown protection.
eBPF for Windows should switch to the cache aware version.

Can you assign it to me? @Alan-Jowett

authors: Alan-Jowett amitschendel

repo: microsoft/ebpf-for-windows


text:
release patched tarball has differences with git submodule go checkout, git am patches

There are two differences between the git submodule checkout of go; git am apply all patches, versus the released patched tarball.
chmod +x appears to be lost on many files (bash and shell scripts, testdata files), and .github, .gitignore, .gitattributes are missing.
$ git diff --no-index --stat go/ tarball/go/
 go/.git => /dev/null                                                             |   1 -
 go/.gitattributes => /dev/null                                                   |  16 -------
 go/.github/CODE_OF_CONDUCT.md => /dev/null                                       |   3 --
 go/.github/ISSUE_TEMPLATE/00-bug.yml => /dev/null                                |  94 --------------------------------------
 go/.github/ISSUE_TEMPLATE/01-pkgsite.yml => /dev/null                            |  47 -------------------
 go/.github/ISSUE_TEMPLATE/02-pkgsite-removal.yml => /dev/null                    |  42 -----------------
 go/.github/ISSUE_TEMPLATE/03-gopls.yml => /dev/null                              |  56 -----------------------
 go/.github/ISSUE_TEMPLATE/04-vuln.yml => /dev/null                               |  52 ---------------------
 go/.github/ISSUE_TEMPLATE/10-proposal.yml => /dev/null                           |  15 -------
 go/.github/ISSUE_TEMPLATE/11-language-change.yml => /dev/null                    | 165 -------------------------------------------------------------------
 go/.github/ISSUE_TEMPLATE/12-telemetry.yml => /dev/null                          |  30 -------------
 go/.github/ISSUE_TEMPLATE/config.yml => /dev/null                                |   5 ---
 go/.github/PULL_REQUEST_TEMPLATE => /dev/null                                    |  25 -----------
 go/.github/SUPPORT.md => /dev/null                                               |  14 ------
 go/.gitignore => /dev/null                                                       |  52 ---------------------
 {go => tarball/go}/src/cmd/compile/profile.sh                                    |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/plan9/mkall.sh                |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/plan9/mkerrors.sh             |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/plan9/mksysnum_plan9.sh       |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/unix/mkall.sh                 |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/unix/mkerrors.sh              |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/windows/mkerrors.bash         |   0
 {go => tarball/go}/src/cmd/vendor/golang.org/x/sys/windows/mkknownfolderids.bash |   0
 {go => tarball/go}/src/cmp.bash                                                  |   0
 {go => tarball/go}/src/debug/dwarf/testdata/ranges.elf                           | Bin 10348 -> 10348 bytes
 {go => tarball/go}/src/debug/dwarf/testdata/rnglistx.elf                         | Bin 11024 -> 11024 bytes
 {go => tarball/go}/src/debug/dwarf/testdata/typedef.elf                          | Bin 12448 -> 12448 bytes
 {go => tarball/go}/src/runtime/pprof/testdata/test32                             | Bin 528 -> 528 bytes
 {go => tarball/go}/src/runtime/pprof/testdata/test32be                           | Bin 520 -> 520 bytes
 {go => tarball/go}/src/runtime/pprof/testdata/test64                             | Bin 760 -> 760 bytes
 {go => tarball/go}/src/runtime/pprof/testdata/test64be                           | Bin 856 -> 856 bytes

It would be ideal if released patched tarballs match git submodule checkout / git am / git export.

@dagood for triage.

It would be ideal if released patched tarballs match git submodule checkout / git am / git export.

Why is that? IMO it is better if they match what you would find on https://go.dev/dl/. We use upstream tooling to create the source tarball--not just for convenience, but in order to make sure it matches upstream.

It would be ideal if released patched tarballs match git submodule checkout / git am / git export.

Why is that? IMO it is better if they match what you would find on https://go.dev/dl/. We use upstream tooling to create the source tarball--not just for convenience, but in order to make sure it matches upstream.
(And the reason matching upstream is useful is that someone who already has a build system for upstream source tarballs can (usually) swap in our tarball without changing the build process or dealing with side effects from discrepancies.)

Ack.
I'll redirect my request there then!

authors: dagood karianna xnox

repo: microsoft/go


text:
This repo doesn't meet the "durable ownership minimums" for Microsoft compliance

I've been pinged as this repo doesn't meet the "durable ownership minimums" for Microsoft compliance.
Please add another owner to this repo to meet those requirements!
I'll check this in 2 weeks, if the requirements are not met, I'll add myself as an owner, and archive the repo.

As this repo isn't following our maintenance requirements, I'm archiving it.

repo: microsoft/hack-workshop-iot


text:
[PL-900] 챌린지 완료 인증

깃헙 ID

https://github.com/OreoFlavor

Microsoft Learn 프로필 링크

https://learn.microsoft.com/ko-kr/users/16480312/

클라우드 스킬 챌린지 토픽/코드

파워플랫폼 기초 (PL-900)

챌린지 완료 인증 예시

안녕하세요. 해커그라운드 조직 의원회입니다.
디스코드 참여 부탁드립니다.
https://discord.gg/bnpvGuE2

/ok

repo: microsoft/hackers-ground


text:
Make tar2ext4 deterministic with files without parent dir in tar

Problem to fix
./dmverity-vhd roothash is  not deterministic for some file docker images.
IMAGE=gcr.io/distroless/cc-debian11@sha256:d5a2169bc2282598f0cf886a3d301269d0ee5bf7f7392184198dd41d36b70548

docker pull $IMAGE > /dev/null

./dmverity-vhd -v -d roothash -i $IMAGE > bad-debian-1.txt
./dmverity-vhd -v -d roothash -i $IMAGE > bad-debian-2.txt

diff  bad-debian-1.txt bad-debian-2.txt

The above command shows a diff which suggest it's not deterministic.
This is causing a problem for az confcom tool that the generated security policy is not consistent for each run (./dmverity-vhd roothash  is called here).
Container deployment is blocked as well for C-ACI.
Cause of the problem
In the example image, there is a problematic tar file only  with a file etc/nsswitch.conf. For most of tar files there would be also etc/ directory as the parent of nsswitch.conf, but for this specific tar there isn't.
For the such files without parent, MakeParents() makes the parent directories for ext4, but it uses time.Now() and it's causing the non-deterministic behavior.
Fix
Change time.Now() to the child's value.
Test

diff  bad-debian-1.txt bad-debian-2.txt doesn't show any diff after the fix
go test ./ext4/tar2ext4/.... It includes new a unit test case.

Note
go test ./ext4/internal/compactext4/... is broken, but it seems to be the case even for main branch. Does anyone now the problem?
--- FAIL: TestBasic (0.01s)
    compact_test.go:221: e2fsck 1.45.5 (07-Jan-2020)
        Pass 1: Checking inodes, blocks, and sizes
        Pass 2: Checking directory structure
        Pass 3: Checking directory connectivity
        Pass 4: Checking reference counts
        Pass 5: Checking group summary information
        Padding at end of inode bitmap is not set. Fix? no

        testfs.img: ********** WARNING: Filesystem still has errors **********

                  26 inodes used (81.25%, out of 32)
                   0 non-contiguous files (0.0%)
                   0 non-contiguous directories (0.0%)
                     # of inodes with ind/dind/tind blocks: 0/0/0
                     Extent depth histogram: 9
                  16 blocks used (100.00%, out of 16)
                   0 bad blocks
                   0 large files

                   5 regular files
                   3 directories
                   1 character device file
                   1 block device file
                   1 fifo
                   1 link
                   5 symbolic links (2 fast symbolic links)
                   1 socket
        ------------
                  18 files
    compact_test.go:221: exit status 4

@anmaxvl  You might have already checked it, but as I mentioned in the PR description the tests under ./ext4/internal/compactext4/... seems to be broken both in this branch and main. Do you know anything about it?

repo: microsoft/hcsshim


text:
Demos on actual documents

Hello! Thank you for making this available - I was wondering if there are any demos that you could share where text/information is extracted from a PDF? I have seen that the current scripts uploaded are for fine-tuning, but it would be nice to have a script for just inference on new docs! Thank you

refer to this PR https://github.com/microsoft/i-Code/pull/36 for more details/demo before it is merged.

authors: thefirebanks zinengtang

repo: microsoft/i-Code


text:
MCR - D365 Supply Chain Management deployment

Documentation Issue
D365 Supply Chain Management (SCM) is a key piece for several retail scenarios which centre around logistics, transportation and warehousing.
This backlog item is to capture:

Considerations
Recommendations
Deployment steps
Integration with other building blocks such as D365 IOM

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@uday31in Could you please update this board columns as New, Backlog, In Progress, In Review, Done.? And cause it has Backlog label please choose priority & size and assign a team member or move this item to "new" status? Thank you

authors: amanjeetsingh aydinderya

repo: microsoft/industry


text:
make image the right scale

I added an image on this page  but it's very large. Please can you make it a more appropriate size? When you've done it I'll try to remember to look and see how you did it.
Thanks!

we don't have the source of this page. for other diagrams, we actually moved to using draw.io ( vscode extension is at https://marketplace.visualstudio.com/items?itemName=hediet.vscode-drawio ).
See this image: https://github.com/microsoft/jacdac-docs/blob/main/static/images/jdbus.drawio.svg

I don't have time to get up to speed with draw.io so unless there is a ppt->draw.io converter I'll stick with the image.

Where is the source of this image in case it needs to be updated?

authors: pelikhan sehodges

repo: microsoft/jacdac