examples/mesos: Change ubuntu VB to be correct version

Summary of changes:
I tried to download the VM in getting-started but it failed because of the fact that Ubuntu was upgraded in January. This is basically the same request of this PR but a different file.
It's literally a one line change. Simply so when the VM gets downloaded the right version of Ubuntu is installed.
Lemme know if anything else in the file ought to be changed, etc. Thanks!

test-me-please

Thanks, I added a fixes tag to your PR description for #2620.


User documentation drafted

/sig multicluster
We should have a draft of user documentation for how to bootstrap/set up/use a cluster registry. This should probably include some examples as well.
From https://github.com/kubernetes/community/blob/master/contributors/design-proposals/multicluster/cluster-registry/project-design-and-plan.md#alpha-targeting-late-q4-17

Closing #11 in favor of this.

@perotinus I think the milestone for this was meant to be v0.1 (Alpha)


Fix extra-containers reference and some typos

I'm taking a look at the tutorials and am currently going through them to check for validity, so I'm going to be submitting PRs during the week, this is the first.
Note needs to be merged to refactor after successful merge into master.

Docs for 8e88c28940ee737feadb87c9038753e5218f591d are browesable at: https://ci.foundries.io/projects/fio-docs/builds/527/docs/artifacts/html/index.html

Merging this, as it is preventing me from rebasing refactor


2 redeferral-related fixes in Recycler

Do not try to take special mark actions in BackgroundScanStack, only in ScanStack, as memory may be unstable in the background case. 2. In marking, if MarkInternal returns true (indicating no further work is necessary), and if we're required to take special mark actions, detect finalizable object and take the action.

@dotnet-bot test Ubuntu ubuntu_linux_debug when you've got a chance

@curtisman I know you're watching
@digitalinfinity In case Curtis isn't watching

Other then the refactor comment.  LGTM.


Update log-replay-service-overview.md

Wrong header for mode.

Looks good; thanks, @neetusingh-ms


Stream color update for time reader

If we change the stream color, the stream color does not register the changes in front of time reader

Ideally it should automatically update the color for that part also but currently it requires a reload to register that change properly.

can I work on this issue
.

Yes, you can type @zulipbot claim to start working on this issue.

If you are not working

@zulipbot claim

@zulipbot claim

@zulipbot claim


What is the NBT branch?

I can't find on Google what the NBT branch of ProjectE that you talk about is, can you tell me how do I get ProjectE to calculate Mekanism's EMC values after putting the Mekanism.json file in the custom conversions folder please? Thanks in advance!

https://github.com/solidDoWant/ProjectE
You'll need to compile this and use it in place of your normal ProjectE. Currently only have a branch for 1.7.10.

Thankyou very much!

For anybody looking for the release binary, I've recompiled and uploaded it here: https://github.com/solidDoWant/ProjectE/releases/tag/1


Change to using the centrally shared copy of DevCheck (from eng\common)

/azp run

Need to update yml to use the .cmd instead

Need to update yml to use the .cmd instead

Good catch! But nah, anyone using the .ps1 can just change the path. Only reason for the cmd is end user interactive convenience, the yml specify the .ps1 they can (and should) continue doing so

/azp run

/azp run

/azp run

/azp run


logs in steps are not clear

Currently, our logs are quite messy, let's make more normalized

@rultor release, tag is 0.0.34

@rultor release, tag is 0.0.34

@h1alexbel OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.34

@h1alexbel Done! FYI, the full log is here (took me 20min).


Unsafe generic array cast

Issue details
I found unpleasant surprise using Array calss. This class store items as a generic array (T[]), but this field initialised as Object array:
public Array (boolean ordered, int capacity) {
    this.ordered = ordered;
    items = (T[])new Object[capacity];
}

In result we have Object array can't be cast to T array.
Next methods crahes with ClassCastException if returned value will be assignet to T[]:

Array.shrink()
Array.ensureCapacity()
Array.setSize()
assign of Array.items to T[] variable
SnapshotArray.begin()
SnapshotArray.setSize()

Reproduction steps/code
Array<MyObject> testArray = new Array<>();
testArray.add(new MyObject());
MyObject[] items = testArray.items;
// or
MyObject[] items = testArray.shrink();

Version of LibGDX and/or relevant dependencies
1.9.8
Stacktrace
Caused by: java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [Lcom.test.Test$MyObject;

The Javadocs for items explains that the backing array is an Object[] if you don't pass a class type to the constructor. Not sure it would make sense to redundantly explain that in every method's docs rather than link to the items docs as it does now.

@cypherdare javadoc of SnapshotArray have example:
Object[] items = array.begin();
for (int i = 0, n = array.size; i < n; i++) {
    Object item = items[i];
    // ...
}
array.end();

this totally right, but why method signature is public T[] begin ()?
When I call method with return type T[] I expect to get T array (not Object[]).

The method signature is T[] because that is a lot more convenient to use
than Object[]. You just have to pass the class type into the constructor
for it to work.
On Sat, Feb 3, 2018 at 1:06 PM Sergey Bubenshchikov <
<EMAIL_ADDRESS>wrote:

@cypherdare https://github.com/cypherdare javadoc of SnapshotArray have
example:
Object[] items = array.begin();for (int i = 0, n = array.size; i < n; i++) {
Object item = items[i];
// ...
}
array.end();
this totally right, but why method signature is public T[] begin ()?
When I call method with return type T[] I expect to get T array (not
Object[]).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/libgdx/libgdx/issues/5076#issuecomment-362840798, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABwfu87tTvvio-lbFWeF0RVwqEz0gTe2ks5tRKAKgaJpZM4R4OC0
.

It's T[] for convenience, which was deemed worth the minor gotcha that it should be accessed as Object[] unless the type is passed to a constructor.


ansis:SoilSite - relatedProject 1..* to 1..1, remove the Y, def change

change def to the project which established the site

ontology - relatedProject is already 1..1
def change to relatedProject
"@id": "ansis:relatedProject",
"range@type": "proj:Project",
"title": "related project",
"description": "Link to project that supported this activity.",
"description": "Link to project that supported this activity, either establishing a Land Survey or Soil Site, or requiring a Site Visit."

ontology - I don't think a change in def to relatedProj is required - can keep broad as to any number of activities that may be carried out. @abhritchie ?

Actually, on second read and in planning to change the definition in #48 it won't hurt to change the def in the ontology to what you used in the schema - it covers the types of activities that can be linked to a project including making observations through the relatedActivity of a SiteVisit

ontology - ansis:relatedProject is exactly 1 on LandsurveySite so leave as is


Ease use of starter-kit for "hello world" type explorations

Suppose I'm playing around with CBMC and starter-kit for perhaps the first time, evaluating it.
Probably what I want to do is write and run a kind of "hello-world" type program, maybe, say, find the max of two numbers.
In this case I probably haven't got a whole directory structure of source set up. I just want to instantiate the makefile templates, then go in and edit the a source file in that directory and go. Maybe even put my hello-world function in the harness.
But, if I try to avoid giving a source file location for the function I want to prove I get this:
What is the function name? bar
Enter path to source file defining bar:
Traceback (most recent call last):
  File "/usr/local/bin/cbmc-starter-kit-setup-proof", line 8, in <module>
    sys.exit(main())
  File "/usr/local/Cellar/cbmc-starter-kit/2.4/libexec/lib/python3.8/site-packages/cbmc_starter_kit/setup_proof.py", line 67, in main
    source_file = util.ask_for_source_file(function)
  File "/usr/local/Cellar/cbmc-starter-kit/2.4/libexec/lib/python3.8/site-packages/cbmc_starter_kit/util.py", line 110, in ask_for_source_file
    raise UserWarning(f"Source file '{src}' does not exist")
UserWarning: Source file '/Users/jmgruj/tmp/cbmc/proofs' does not exist

I think I would want a different user experience for people just looking to try out CBMC.

I don't know what the model of the user to use here is.  Someone just trying out CBMC without any source code at all is probably going to want to invoke goto-cc and cbmc directly as in the tutorials on little examples.  Someone with preexisting source code is probably going to have a directory with source code.

Twice now I have settled down to write a collection of examples to explore CBMC performance issues starting from a blank slate. I've wanted the convenience of the starter-kit's make system and I've kind of had to hack my way past the initial setup expectations.


Issue in importing tensorflow graph def to NNVM

Hi,
Whenever I run the from_tensorflow.py in tutorial, I get the following error

Can you please shed some light on this issue. The dtype is uint8 in the example.

please use https://discuss.tvm.ai/ for general discussions, the repo is moved as part of https://github.com/dmlc/tvm/


newGeneratedCohortSet() remove cdm_reference

I'm not sure what this issue is about but I'm assuming it has been fixed. Feel free to repoen if not and provide some more details or a reproducible example.


observe resourceOwner once, at top

we transitioned backups from multi-user to single- user some time ago, but never went back to
remove the resource-owner identification from
every progress bar on display.  This ends up
creating a lot of duplication of the same owner
in a single run, when we could use that space
for more valuable information such as item
or folder details.

Does this PR need a docs update or release note?

[x] :no_entry: No

Type of change

[x] :sunflower: Feature

Issue(s)

#2871

Test Plan

[x] :muscle: Manual

previous output, with the owner duplicated on each entry
Connecting to M365:         done
Discovering items to backup:     20s done
∙ contacts -<EMAIL_ADDRESS>     1s done
∙ email -<EMAIL_ADDRESS>     8s done
∙ events -<EMAIL_ADDRESS>     8s done
Backing up data:      7s ∙∙●
events 117 -<EMAIL_ADDRESS>- itemid
events   2 -<EMAIL_ADDRESS>- itemid2
events 139 -<EMAIL_ADDRESS>- itemid3
# ...

Now having moved the owner to the top of the operation:
Connecting to repository:      4s done
Backing Up ∙<EMAIL_ADDRESS>Connecting to M365:         done
Discovering items to backup:     21s done
∙ contacts:      1s done
∙ email:      9s done
∙ events:      8s done
Backing up data:     11s done
Backing Up ∙<EMAIL_ADDRESS>Connecting to M365:         done
Discovering items to backup:     15s done
∙ contacts:      1s done
∙ email:      7s done
∙ events:      5s done
Backing up data:      1s ∙∙●
events 10 - AAMkAGYyMGNlMDYzLThlZDQtNGJmZC05NmNlLTUyMjg4MDRiMmZiMABGAAAAAAA0FEK-SIbZRqwDDCv8-vtyBwDpPghBpT2hRqz_ILX-d…
email ∙∙∙∙∙∙∙●∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙∙   4 - Sent Items
events  0 - AAMkAGYyMGNlMDYzLThlZDQtNGJmZC05NmNlLTUyMjg4MDRiMmZiMABGAAAAAAA0FEK-SIbZRqwDDCv8-vtyBwDpPghBpT2hRqz_ILX-d…
# ...

As far as change in output goes, LGTM


Using folder opacity level to alter what is inside the folders

Do you think that you can modify the Export Layers code to allow one to use folder opacity level to change the transparency levels of what is inside the folders? Since GIMP doesn't have multiple layer selection, I think this would be a good idea during the meantime.

It's certainly possible, but I feel like it falls outside the scope of the plug-in, which primarily deals with exporting layers, not batch-editing them.
It sounds like something that could be handled by BIMP, although I'm not sure if it supports batch-manipulating layers. If not, I'll think about this request, but it's going to take quite a while before I take a look at this.

Although, I'm still not sure if I understand completely:
a) You want to export an entire layer group as one image while preserving its transparency. This is already doable in the latest revision (not an official release yet, but it's stable) - simply select the desired layer group from the preview, check "Export only selected layers" and hit Export.
b) You want to control the transparency of the underlying layers in the group by adjusting the transparency of the group (instead of individual layers, because it's painful to do this one layer at a time because GIMP doesn't support multi-selection), and you need to export the individual layers with the transparency inherited from the layer group. This is not currently possible, but it's perfectly in the scope of the plug-in. If this is what you request, I think I'll implement this when I release 3.0.

The b part is exactly what I want at the moment, and I have tried BIMP just to find that it doesn't work, but the feature should be available in BIMP. And yes, I did started that thread.

Done. If you want to use the feature now, you can download the latest revision. Check "Show more settings" and then simply check "Inherit transparency from layer groups". Note that each layer inherits the transparency from all of its parents, not just the immediate parent.
Let me know if it works for you. :)

I don't see the "Show More Settings" in the Windows build of GIMP 2. The option isn't under Advanced settings. Sorry this took a while, but I"m studying Industrial Design.

Sorry I wasn't clear - you need to download the latest work-in-progress version of the plug-in (it should be stable as-is). Once you open the plug-in, check "Show More Settings" at the bottom of the plug-in dialog (not the GIMP window) and then check "Inherit transparency from layer groups".

It is working. Wow, it really does work. I like it. Feel free and respond and then close this as this has been solved.

Great! Closing this issue.


For issue#1976 - jobs-app is replaced with Google's online boutique app

Hi 👋!
This PR is ".
Title : to address "TestLabelsDemoApp failures
This patch fixes this by adding:
I have updated code reference of app - jobs-app to Google's Online Boutique in helm chart & labelsEventChecker which is lightweight and expected to run without failure.
Fixes: #1976

Please let me know if any follow up corrections needed for this PR, thanks

Hello @mtardy - Please let me know if any follow up corrections needed for this PR, thanks

Hey thanks for trying this! You can take a look at this job https://github.com/cilium/tetragon/actions/runs/8120867506?pr=2176 to see what failed, and you can run the e2e tests locally by doing make e2e-test. It looks like the app doesn't deploy properly already. I think you'll need to update the labels accordingly as well, not sure they are correct as of now. You can try until make e2e-test passes locally.

Unsuccessful in standing up working version of app in local, using Mac Intel core, Sonoma. Any pointers how to progress will be helpful @mtardy

Hi @rajesamp, I have followed the Google microservices-demo to use helm to deploy boutique app manually https://github.com/GoogleCloudPlatform/microservices-demo/tree/main/helm-chart and then use golang code to install it https://github.com/Trung-DV/tetragon/blob/c24e60f5c71e4dc2b7dc4a6fb6a92c639bf1f548/tests/e2e/tests/labels/labels_test.go#L44-L51, seems that #1976 is not simply replace jobs-app by online-boutique, we have to update labelsEventChecker to verify services of online-boutique app. I think we should write a new test labels test for Online Boutique app as well as the labelsEventChecker for it.

Dear @rajesamp, @mtardy , I have replaced jobs-app by onlinebuotique as well as update the labels checker. Please help me to review.
I rewrite labels checker by compare the current jobs-app charts (which found from https://github.com/usrbinkat/pulumi-kubernetes/blob/main/pulumi/src/jobs_app/charts/jobs-app/charts/elasticsearch/templates/statefulset.yaml) and compared to onlineboutique from https://github.com/GoogleCloudPlatform/microservices-demo/tree/90b81cca772eb61b8695a638e44b9b2cde70853f/helm-chart/templates, does this is a right approach?

Dear @rajesamp, @mtardy , I have replaced jobs-app by onlinebuotique as well as update the labels checker. Please help me to review this PR: rajesamp#1

Can you open a PR against the main repo from your fork?

Dear @rajesamp, @mtardy , I have replaced jobs-app by onlinebuotique as well as update the labels checker. Please help me to review this PR: rajesamp#1

Can you open a PR against the main repo from your fork?

Dear @mtardy,
I have opened PR #2345, can you give me some advises to enhance the labels for Online Boutique in case we need them?

This PR has been superseded by #2345, so I'm closing it. Thank you @rajesamp


Load Object from Input

We have a Snippet load_struct_from_input that reads a length-prepended string of BFieldElements and copies it to memory at the first free address. But with the new BFieldCodec, encodings are not length-prepended anymore. The ugly solution is to force length-prepending on every encoding that is passed as input; the proper solution is to extend TasmObject with a load_from_input code generator and derive it automatically for structs decorated with the [derive(TasmObject)] macro.
Note that this problem arises only for standard input. Secret input does not need to be read as memory can be initialized to the right value.

Not needed since we have non-deterministically initialized memory now.


Colour changes on OP screen - Black and Purple

We received this screenshot below from a user. After receiving his browser version info (IE and Chrome) We advised him to
a) hard refresh
b) check configuration settings (i.e. is it an accessibility setting)
c) contact his IT
The user advised it was working fine on all his other devices and his IT thought it was a local issue but they were not sure which. (I have sent a follow up)
Nothing immediate to be done but I think it happened before and @helen asked about it on slack since this report so guess there was another report to her personally.

It's pretty clearly the stylesheet not loading - possibly just the bootstrap one. If it's in Chrome and IE then it sounds like a network configuration issue, possibly.
The bootstrap stylesheet is loaded from https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/3.3.5/css/bootstrap.min.css, but I can't think how that would be related.
We could ask the user if they can load that page when they click on it.
Any thoughts @evansd?

User feedback

Just checked it on the same laptop but at home and it works like normal (no purple thing)?! Must be something to do with set-up at work.

Great - can we ask them to visit that link from work? viz https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/3.3.5/css/bootstrap.min.css
and report back what they see. it should be a bunch of code.

Update

The link is blocked by our firewall !!! MY head of dept has asked our IT dept to resolve the issue. She (rightly) thinks your site is too important to be restricted in anyway.  I will let you know if that sorts it out as I guess that is a solution of sorts even if it not something that can be resolved ‘your side’

I think we can count this as fixed!
I would be intrigued to know what the IT dept justification is for blocking files from that link (it's widely used across the internet and will break quite a few sites) but I doubt we'll get much of an answer.

I actually think we should self-host all our dependencies anyway. Relying on third-party hosted stuff just creates potential for mysterious problems (like this one) and prevents doing development offline. Given that we're using Cloudflare there's no performance advantage. (The other possible advantage -- that first time visitors might already have some of those dependencies in their browser cache -- is pretty negligible I think.)

I agree, TBH it's just sheer laziness on my part because we don't have nice tooling around self-hosting such resources yet

@brianmackenna I've pushed out a change which should hopefully fix the issue he was having and prevent weird things like this happening in future. Do you mind getting back to him and letting him know?

Response from user

Good work! Looks fine now. Thanks to the team


feat(authentication): support cp4d /v1/authorize operation

The main focus of this change is to enhance the CloudPakForDataAuthentictor
implementation to support the official "POST /v1/authorize" operation for
obtaining an access token.   This includes support for using either a password
or an apikey when obtaining the access token from the token service.
In addition, a new Builder class was introduced which provides a more user-friendly
way to construct instances of CloudPakForDataAuthenticator.  The "fromConfiguration"
method was also added to replace the (now deprecated) map-based ctor.
In order to keep the IamAuthenticator class in sync with CloudPakForDataAuthenticator,
the Builder class and fromConfiguration() method were also added to IamAuthenticator.
In the case of both authenticators, the existing set of constructors are still supported
but are now deprecated and will be removed in the next major release of the Java core.

This looks good! If the Java generator doesn't need any related changes to this now using the Builder class, then this should be good to go. And I assume the previous non-builder method still works, it's just deprecated now correct?

correct, we still support all the ctors we supported before

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


Create NOTICE file

Create NOTICE file to enable downstream users to avoid compliance risk.
There is an associated issue: https://github.com/microsoft/TypeScript/issues/52663. T/his has not been added to the Backlog milestone.
Code is up to date with the main branch.
Because the substance of the change is to add a text file I have not run tests.

Fixes #52663

The associated issue is closed so I'm going to close this PR as well.


Prints not starting

I'm currently using a raspberry pi 4b with octopront version 1.8.6, octopi version 0.18.0. It is connected to a prusa i3 mk3s+.
I've installed the plugin with the manager and via the url.
I have checked the time settings on the pi with sudo raspi-config.

No print starts when the scheduled time arrives. Any ideas?
Log file attached
octoprint.log

so both timezone and date/time I configured correctly? double-check that the items are getting added to config.yaml when you add them to the scheduler queue. nothing in log indicates there's an error.

I've double checked the timezone again and I have set it to my timezone.
below the print seems to be added to the config.yaml file but the print still doesn't start up

I forgot to request that you enable debug logging for the plugin in the logging section of OctoPrint. That will then write lines out to octoprint.log correctly to debug what's going on. Select octoprint.plugins.printscheduler from the list and click save.

octoprint (2).log

ah...are your file stored on SD card and not in local OctoPrint storage? I never tested that and is probably the reason I think this might be happening.

yes the files were on the sd. Switched to the octoprint storage and the prints start up fine. Thanks very much


Closes issue #456. Completed averageAcceleration method. Added sumPrimes method

Closes issue #456

[x] Running yarn lint does not trigger any linter errors
[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!
[x] You have written tests to accompany your skeleton method

Hey @BlakeGuilloud, I've removed Ganon as a dependency. Let me know if there's anything else to do, thanks!

Hi @ktilcu, I've made that change now. Thanks


amend: updated deps

:tada: This PR is included in version 1.10.128 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New container?

This probably isn't the place to ask? When will a new version be pushed? Looking forward to the logging changes.

Sorry for the delayed response.
The gcr.io/cloudsql-docker/gce-proxy:1.13 was released and is publicly available.


implement linux nodeSelector test

What this PR does / why we need it:
Add a test ensuring control plane components only get scheduled on linux nodes.
Release note:
NONE

Sending this as WIP as local tests don't work :-/

/retest

/retest

/retest

Where CDI does not yet have a nodeSelector, we should invert the test so we test everything except for what we know does not comply.

/retest


Mixin issue

It's easier for me to report it through here as curseforge doesn't allow me to do comments on my browser, anyways the latest version throws this error:
[main/FATAL]: Mixin prepare failed preparing FireworkItemMixin in automaticelytra.mixins.json: org.spongepowered.asm.mixin.transformer.throwables.InvalidMixinException The specified mixin 'com.loucaskreger.automaticelytra.mixin.FireworkItemMixin' was not found org.spongepowered.asm.mixin.transformer.throwables.InvalidMixinException: The specified mixin 'com.loucaskreger.automaticelytra.mixin.FireworkItemMixin' was not found
Using:

fabricloader 0.11.6
fabric-api-0.36.0+1.17

Had the same, full log if this helps.

I removed the mixin from the json and seems to work

This has been fixed, had an old mixin in the json.


Allow some component events bypass strictTemplate

What problem does this feature solve?
This is sort of like https://github.com/vuejs/language-tools/issues/4879 but even more granular.
For better or for worse props/emits compose in different ways in Vue. For example I can have component but not be sure what attributes it accepts, one example while keeping it type safe could like this:
interface Props {
	type: 'button'
	[k: `data-${string}`]: string
}
const { type, ...restProps } = defineProps<Props>()

If I want to compose props and only pass only state I need with v-bind and so on
<OtherComponent v-bind="restProps" />

The same cannot be said with vue events, while using defineEmits I need explicitely define all possible events or else when I enable strictTemplate things will break.
An actual example I am struggling with
// BaseButton.vue

defineEmits<{
	click: [e: MouseEvent]
	focusin: [e: FocusEvent]
	focusout: [e: FocusEvent]
	keyboard: [e: KeyboardEvent]
	mouse: [e: MouseEvent]
	touch: [e: TouchEvent]
	// ... and so on
}>()

<button
                 @click="$emit('click', $event)"
		@focusin="$emit('focusin', $event)"
		@focusout="$emit('focusout', $event)"
		@keydown="$emit('keyboard', $event)"
		@mousedown="$emit('mouse', $event)"
		@touchstart="$emit('touch', $event)"
>
  <slot />
</button>

I defined events I needed, but now actual event listeners are not lazy, they are actual event listeners even if parent component did not use any of them "mousemove" for example would still fire to the event listener vue registered.
Composability is another issue, lets say I have FancyButton which wraps BaseButton, to declare events I have to the whole:
<BaseButton
                 @click="$emit('click', $event)"
		@focusin="$emit('focusin', $event)"
		@focusout="$emit('focusout', $event)"
		@keydown="$emit('keyboard', $event)"
		@mousedown="$emit('mouse', $event)"
		@touchstart="$emit('touch', $event)"
>
  <slot />
</BaseButton>

Which quickly becomes hard to maintain.
Without enabling strictTemplate this issue is largely masked because component would accept any event, now not so much.
What does the proposed solution look like?
This is largely Vue limitation of how to compose events, but in mean time it would be great to allow disabling strict events on some components. I am not sure how it would look like maybe something with types, comment or tsconfig.json option. It would allow to retain most of benefits of strict templates, while still making it usable with design system components like buttons where you can't and shouldn't realistically write all of the possible events.

You can enable fallthroughAttributes.

fallthroughAttributes will enable it globally while I want that only for few specific components

If your button is not the single root node of the component (or if v-bind="$attrs" is not set on the button), this option will not take effect.

Perhaps we can have a comment syntax or defineCompilerOptions to configure vueCompilerOptions locally.

enabling fallthroughAttributes introduced bunch of unrelated errors

Can you explain this in detail? If it's indeed a bug, please create a minimal reproduction that allows us to investigate it.

If your button is not the single root node of the component (or if v-bind="$attrs" is not set on the button), this option will not take effect.

BaseButton is dynamic component which can be either a or button based on prop. Yeah I don't use $attrs, I explicitely define props I need and use interface extensions inside wrapper components and to v-bind=props to pass them from up top.
Whole structure looks something like this:
<MyPage>
  <FancyButton>
     <template>
       <BaseButton>
         <template>
           <Component :is="button | a"><slot /></Component>
         </template>
       </BaseButton>
     </template>
  </FancyButton>
</MyPage>

Can you explain this in detail? If it's indeed a bug, please create a minimal reproduction that allows us to investigate it.

Will try later to make a reproduction later, when I enable fallthroughAttributes  it seems to break interface merging. I have design system package, and apps can customize or add additional props to buttons like this:
export interface BaseButtonInitialProps {
	id?: string
	type?: 'button' | 'submit'
	as?: 'a' | 'button'
	disabled?: boolean
	href?: string
	target?: '_blank' | '_self'
	download?: boolean
}

// Interface is intentionally left empty to allow consumers of the component
// to extend and customize its properties through TypeScript's declaration merging.
export interface BaseButtonProps extends BaseButtonInitialProps {}

Perhaps we can have a comment syntax or defineCompilerOptions to configure vueCompilerOptions locally.

Would be great


Subsequent dial with multiaddr not in the previous dialTarget

Version: 0.29
Platform: *
Subsystem: Dialer

Type: Bug
Description:
When a dial to a given peer is performed, if a dial to the same peer is currently in progress, this dial will just wait on the first to finish and return its connection.
A tricky scenario can happen with this flow. If the second dial uses an address that was not included in the first one, it will not be used. For instance, by using libp2p.dial(ma1) ... libp2p.dial(ma2), or by getting the multiaddrs from AddressBook and then dial from a not known multiaddr of a peer, the second multiaddrs will not be used.
Steps to reproduce the error:

libp2p.dial(serverMultiaddr)

(exchanged identify addresses (that do not announce the serverMultiaddr))

on reload, autoDial kicks in from peerStore
libp2p.dial(serverMultiaddr)

Reference: https://github.com/ipfs/js-ipfs/issues/3400

On a sync conv with @jacobheun

this is where multiaddr confidence would be really nice. Maybe in the interim we could add on identify and prune known addresses when they fail a dial? This wouldn't be great for intermittent failures but we need a way to avoid storing hundreds of multiaddrs for a peer until we get confidence in place

Since https://github.com/libp2p/js-libp2p/pull/1498 when passed a multiaddr, only that multiaddr is dialed so this is no longer possible.
There's a possible edge case where if you dial a peer id, then add a new address to the address book, then dial the peer id again it may join on the first dial promise rather than adding the new address to the dial target but this needs further investigation.

Closing as we currently only dial one peer at a time so this is not possible


ProfilePhoto. File download has failed or was canceled

I'm working on android client. I'm trying to load profile photos for chats. To do that:

I get file info using TdApi.GetRemoteFile with remote file id that I got from chat object
I initiate file loading using TdApi.DownloadFile with file id that I got from file info

After that I receive Error 400: File download has failed or was canceled in my default handler and TdApi.FileUpdate with isDownloadingActive = false and isDownloadingCompleted = false in my update handler. What can I do to fix this?

See GetRemoteFile docuemntation:

Even the request succeeds, the file can be used only if it is still accessible to the user. For example, if the file is from a message, then the message must be not deleted and accessible to the user. If the file database is disabled, then the corresponding object with the file must be preloaded by the application

So, if you somehoh saved remote file ID between, instance restarts, then you must ensure that the corresponding chat is known to TDLib, i.e. received from the server after restart.

Okay, I removed TdApi.GetRemoteFile. Now I just download file using new TdApi.DownloadFile(id, 31, 0, 0, true) wtih local id that I got from the chat object (chat.photo.small.id) but result is still the same. I don't save chat objects between restarts and receive them on every app launch with TdApi.UpdateNewChat.

Ok, error has gone after app reloading. I guess TdApi.GetRemoteFile removal worked after all. Closing the issue. Thanks for your help.


Allow proxying to shell

One of Paul Hammant's original suggestions during the internal release of mountebank. His example was:
(adapting the proxyOnce example)
POST /imposters HTTP/1.1
Host: localhost:46167
Accept: application/json
Content-Type: application/json

{
  "port": 6566,
  "protocol": "http",
  "name": "proxyOnce",
  "stubs": [
    {
      "responses": [
        {
          "proxy": {
            "toShell": "groovy foobar.groovy",
            "mode": "proxyOnce",
            "predicateGenerators": [
              {
                "matches": {
                  "method": true,
                  "path": true,
                  "query": true
                }
              }
            ]
          }
        }
      ]
    }
  ]
}

I think this will be best represented as an "is" imposter in conjunction with #171


Uncaught ReferenceError: shape is not defined

While I'm only adding import LineChart from 'react-svg-line-chart' I receiving error message:

Uncaught ReferenceError: shape is not defined

"react-svg-line-chart": "^1.0.4",

Thank you for reporting this. I fixed it in v1.0.5 release


ConditionalRequirements related to Manager EthernetInterfaceCollection for skipping logical instance like usb0

Hi,
As per https://github.com/opencomputeproject/HWMgmt-OCP-Profiles/blob/master/HWMgmt/OCPBaselineHardwareManagement.v1_1_0.json
HostName": {
	"ReadRequirement": "Recommended",
	"ConditionalRequirements": [{
		"SubordinateToResource": ["Manager", "EthernetInterfaceCollection"],
		"ReadRequirement": "Mandatory",
		"WriteRequirement": "Mandatory"
	}]

Based on ConditionalRequirements, WriteRequirement is verified for eth0(physical instance), usb0(Logical instance for Host interface communication between BMC and Host) in Manager ethernetinterface instance. In case of usb0(Logical instance for Host interface communication between BMC and Host) it's not required to support changing HostName, FQDN, IPv4Addresses#0, NameServers since it's going to be same as physical instance(eth0) in Manager resource.
Chances for some Physical instance also to restrict patch. Is it possible to skip WriteRequirement for logical interface by using use case or anyof comparison to make sure to change one instance in Manager ethernetinterface collection?
@mraineri @jautor Please add your inputs based on schema for skipping WriteRequirement for logical ethernetinterface instance like usb0 or we can accept patch on usb0, reflect the values in eth0, usb0
Thanks,
Mani

Valid issue.  Need to decide how to solve.

Can we use URIs to handle this situation
"EthernetInterface": {
"ReadRequirement": "Mandatory",
"URIs": [
"/redfish/v1/Managers/{ManagerId}/EthernetInterfaces/{EthernetInterfaceId}"
],
Thanks,
Mani

The URI won't work because other NICs on the Manager also need to be tested.  Decision to use "Supported".
Will also modify the Profile spec and/or model to handle this case.


Watcher task crashes

I searched the issues and didn't see this and it could possibly be an issue on my end.
When I pull the latest version it looks like in gulp/utils.js line 51 my code says
    if (exitOnError) {         process.exit(1);     } else {         undefined.emit("end");     }
so it clearly fails in the else. however in your repo i see you changed it to this.emit.
I'm assuming that would show up in the package im getting as well?

Weird that you get undefined there :o
Indeed it should be this so that it just keeps the watcher alive...

I took that code from there: http://blog.ibangspacebar.com/handling-errors-with-gulp-watch-and-gulp-plumber/
One possibility is that the project's build breaks that code and I didn't notice

The issue might be cause by the arrow function...

if you install your package are you getting the same thing? I was feeling like it was just me since no one else has complained about it :(

I'm also seeing that error

I'm pretty sure it's caused by the arrow function, it can't work like this

I published 0.5.4, which should fix this: https://github.com/dsebastien/modernWebDevBuild/releases/tag/0.5.4

nice! thanks!

theres no instruction for upgrade

Just bump the dependency to 0.5.4, that should be enough I think: https://github.com/dsebastien/modernWebDevBuild/blob/master/UPGRADE.MD


OPM Status for Tidbyt

OPM Status for Tidbyt [repeat of app readme.md]
Displays the current OPM Status, e.g. the Office of Personnel Management status, which is used by federal employees to know if the normal working conditions have been changed by inclement weather, health hazards, or other alerts. Updates every 2 hours. No API key required.
Uses the OPM Status JSON feed at https://www.opm.gov/json/operatingstatus.json for updates.

I have read the CLA Document and I hereby sign the CLA

Is there anything I can do, not sure why it failed the lint step.

did you run make lint or I guess pixlet format on this file? if not that may be the issue.

I didn't see "pixlet format" requirement but will run that ASAP.
"make lint" failed but the obvious pathing issues mentioned in the docs didnt' fix it.

why did make lint fail?

It gave me that builidifer not found error mentioned in the docs but the fix there didn't work. Wondering if this is because I'm on windows, looks like most of the docs are for Mac or Linux.

You'd probably have to hunt down installing make or whatever on your end. but pixlet format should be equivalent. I would run pixlet lint too. But make sure you are on the most up to date version of pixlet.
I just copy and pasted your code into a file and ran pixlet format and it looks like that is whythe lint build and test is failing.
Also note, the version in schema.Schema should be 1, as it is not for our use. I think it is an internal parameter or something used for something, so will fail if it is not 1.
Finally, if you don't have any schema, then you donn't need to pass config into main or need the get_schema().

Okay, I'll dump the schema and the config pass-in and test it with pixel format.

Error: unknown command "format" for "pixlet" Run 'pixlet --help' for usage. unknown command "format" for "pixlet"
Is there a special build flag I need to get that to work?

If it doesn't have the format command, then you most likely need to update your version of pixlet.

Yea, that was the issue, I had two copies of the pixlet repo so that was causing problems.
Ran pixlet format, I think it's all good now.

How long does a review usually take?

It depends. They try to do them on Thursday's, but once it gets merged sometimes they do internal testing. Oauth apps can take longer. Think they've been busy improving pixlet and stuff to make dev stuff easier. May take a little bit to catch up after the holidays too.
But nothing for you to do on your end if all checks have passed.

Thanks for the PR. Merged!


New Spotify Web Player doesn't scrobble

Expected Behavior
Spotify recently changed their web player, removing last.fm support. At its first incarnation (some weeks ago), this extension was working and was the only working solution known to me. Then, the old player was back. Since some days, the new web player is online again, but this extension is not working anymore.
Current Behavior
scrobbling doesn't work. already tried to reauthorize the extension.

Duplicate of #1270.
This issue is already fixed. The new version will be available in Chrome Web Store soon.


Cannot install on Ubuntu 17.10, missing xproto

I get the following error on Ubuntu 17.10:

node-gyp configure build
Package xproto was not found in the pkg-config search path.
Perhaps you should add the directory containing `xproto.pc'
to the PKG_CONFIG_PATH environment variable
Package 'xproto', required by 'xau', not found
gyp: Call to 'pkg-config gtk+-3.0 --cflags-only-I | sed s/-I//g' returned exit status 0 while in binding.gyp. while trying to load binding.gyp

I have GTK:

$ pkg-config --modversion gtk+-3.0
3.22.25

My PKG_CONFIG_PATH was wrong:
export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/usr/lib/x86_64-linux-gnu/pkgconfig
export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/usr/share/pkgconfig

My PKG_CONFIG_PATH was wrong:
export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/usr/lib/x86_64-linux-gnu/pkgconfig
export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/usr/share/pkgconfig

This solve too many different problens that i have, like not found gtk+3 on cmake of opencv! But thats it :)


ACR self.cmd did not detect command failure

Describe the bug
ACR team uses the self.cmd command to confirm that our command successfully runs with no errors. However, we have noticed that our test using self.cmd still passes even though our acr agentpool delete command returns an error
https://github.com/Azure/azure-cli/blob/f92f7233659890b427f4f6f486db5610cdca1020/src/azure-cli/azure/cli/command_modules/acr/tests/latest/test_acr_agentpool_commands.py#L67
Related command
acr agentpool delete -n {agents2_name} -r {registry_name}
Errors
NA
Issue script & Debug output
NA
Expected behavior
Test fail when command returns an error message
Environment Summary
{ "auzre-cli": "2.67.0", "azure-cli-core": "2.67.0", "azure-cli-telemetry": "1.1.0", "extensions": { "acrtransfer": "1.1.0" } }
Additional context
No response

Thank you for opening this issue, we will look into it.


Improve schema import error propagation

imodel-native: https://github.com/iTwin/imodel-native/pull/749

Integration test failure:
  1) Schema synchronization
       import schema acquire schema lock when need to transform data:
     Error: Failed to throw error with message: "exclusive lock is already held"
      at assertThrowsAsync (src/integration/SchemaSync.test.ts:108:9)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)
      at Context.<anonymous> (src/integration/SchemaSync.test.ts:930:5)

Integration test failure:
  1) Schema synchronization
       import schema acquire schema lock when need to transform data:
     Error: Failed to throw error with message: "exclusive lock is already held"
      at assertThrowsAsync (src/integration/SchemaSync.test.ts:108:9)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)
      at Context.<anonymous> (src/integration/SchemaSync.test.ts:930:5)

This is due to the change in importSchemas and importXmlSchemas APIs. The integration test will pass again once the native changes are merged and the new @bentley/imodeljs-native package is released.

@tcobbs-bentley @toddsouthenbentley @wgoehrig @aruniverse @calebmshafer your review is required.

/azp run iTwin.js

/azp run iTwin.js Docs - YAML

/azp run iTwin.js Integration - GitHub

@DanRod1999 any theories as to why the builds did not trigger when the addon was merged?

I will take a closer look. But it might just be a fluke on GitHub. I've seen pipelines not trigger before, but will investigate to make sure it is not a problem in our side


Question: Why does Dockerfile support multiple FROM entries?

While reading the docs I've stumbled upon:

FROM can appear multiple times within a single Dockerfile in order to create multiple images. Simply make a note of the last image ID output by the commit before each new FROM command.
https://docs.docker.com/reference/builder/#from

I've tested this via:
FROM busybox:ubuntu-14.04
ENTRYPOINT ["echo", "from1"]

FROM debian:latest
ENTRYPOINT ["echo", "from2"]

$ docker run deaee2cc56ed
from1
$ docker run 93b41e538f02
from2

This example is particular interesting since the docs say:

Only the last ENTRYPOINT instruction in the Dockerfile will have an effect.
https://docs.docker.com/reference/builder/#entrypoint

So I am wandering why are multiple FROMs allowed? What are the use-cases?

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

@luebken please see: https://github.com/docker/docker/issues/13026
non-last FROMs do take effect but only the last one is related to the image (and name) being asked for on the 'docker build' command.

Thanks for the pointer.


dfx?

There is incomplete support for dfx in https://github.com/nomeata/dfx-in-nix, but your project is probably a better place for this. Opening this issue to track progress (or to have a place for people to read why dfx is not part of ic-nix).

Very neat! How exactly did you convince dfx to use the replica binary you built and put on the PATH, instead of the one it usually extracts from its embedded cache?

dfx actually will find a binary if its path is given in the environment DFX_$name_PATH. See https://github.com/ninegua/ic-nix/blob/main/dfx-env.nix#L74-L77

Ah, this must be new or I was blind when I tried to nixify dfx.


Solve the issue #2: Failed to login

By using 'Ghost' instead of spidermonkey, issue related to cookie has been solved.

Thank you for your big contribution but I can't accept your pull request because:

you should not add build directory or TAGS files in pull request
there is unnecessary print codes
you need to update setup.py to solve dependency problem

Please fix these issues and give me a new pull reuqest.

Thank you for spending you time and giving me the detail feedback.
I appreciate to give me those feedback since it was the first time that I tried to do pull request.

Installing Ghost is somewhat too heavy solution and #4 solved this problem by implementing login code only with Python.


AWS clusters: get allowVolumeExpansion: true by default in StorageClass, or document procedure to update it once

We want our StorageClass resource to have allowVolumeExpansion: true, as that makes us able to increase the PVC's requested storage. This is a requirement for the troubleshooting docs here for example.
I believe its not the default at least for AWS based clusters default StorageClass resource, but I think its the default for GCP based clusters.

A lot of our prometheuses were down, so I did this now.

Make a script:#!/bin/bash
echo "kubectl patch storageclass gp2 --patch '{\"allowVolumeExpansion\": true}'" | deployer use-cluster-credentials $1

Run this against all the clusters. This will just error out in GKE and Azure clusters, and that's fine:ls config/clusters | xargs -L1  ./wat.bash

After that, I re-ran https://github.com/2i2c-org/infrastructure/actions/runs/5781619537 again, which completely passed this time. However, dynamic resizing only takes effect when the pod itself is recreated, so am doing that as well now.

Given we've already manually done this for existing clusters, I think for new clusters, we should use https://github.com/2i2c-org/infrastructure/pull/2941 and manage storageclass explicitly. This alleviates r2r concerns that @sgibson91 raised in https://github.com/2i2c-org/infrastructure/pull/2718 as well as allows us to use the cheaper & better gp3 storageclass.


Dynamic leftActionMode

Why is the setLeftActionMode  in FloatingSearchView not exposed?
I have a single Activity with Fragments and would be nice if i can use "Home-up/ back arrow" as the left action mode for navigation back to the previous Fragment.

Try using:
app:floatingSearch_leftActionMode="showHome"
And then listen for it using mSearchView.setOnLeftMenuClickListener(...)

Are u suggesting have a new layout for my search fragment?
I know that as an option B, but would like a single floatingsearchview for the whole app.

I do not understand your problem. My previous solution will give you a left arrow in the search view to which you will be able to listen to and switch fragments as needed. To make the search bar show the back arrow, add: app:floatingSearch_leftActionMode="showHome" in xml

Thanks for your replies!
I am trying to use your search view with a single Activity and multiple Fragments, meaning i have to handle the back arrow. My main screen uses app:floatingSearch_leftActionMode="showSearch" and would like to change the search view to app:floatingSearch_leftActionMode="showHome" after search is complete, so that the user can navigate back to the previous screen.

In other words i would like to switch between showSearch and showHome dynamically .

Ok then, I will look into it later and will add it to next release. I don't know when the next release is going to happen, but you can always download the library, dig into the code, make that method public, adn setup the lib as a local lib in your project.

Would this be added in the next release @arimorty ?

I will try my best for this release, if not this than next one (2.2.0) the next release contains very important fixes so I want it to be out within a few days

Will be included in 2.2.0

hi @arimorty ,,, when will the 2.2.0 version be released? i need this feature,, i have no idea how to go back to the previous view if user want to cancel search.

Hi,
The mentioned release 2.2.0 is an error, this feature was actually included in 2.0.0 (look at the milestone)
You can set the leftActionMode dynamically using this and you can also listen to when the left arrow is clicked. To be clear, the left action mode is the left button that is visible when search is NOT focused. If you want to switch a view or otherwise react to when the search loses focus, you can set a listener for focus changes.
Hope it helps. Please refer to the detailed sample project if you have questions before submitting a new issue.
Thanks,


Update key-vault.md

Both "next action" buttons lead to Mgmt APIs. We think one should point to the client API. TechnicalContent Workitem # 1270124. Edit line 127 to change mgmt link to the client API link.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/key-vault.md
:white_check_mark:Succeeded

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


Add recording features

features mentioned in chapter 1.2.1 in the 1.9 API guide

Closed with 3468ef


[Ameba] Add examples/lighting-app

Problem

Add examples/lighting-app

Change overview

Add examples/lighting-app

Testing
Tested lighting-app with chip-tool

Dear @anush-apple, @balducci-apple, @bzbarsky-apple, @carol-apple, @cecille, @chrisdecenzo, @chulspro, @Damian-Nordic, @electrocucaracha, @franck-apple, @harimau-qirex, @hawk248, @jelderton, @jepenven-silabs, @jmartinez-silabs, @kghost, @LuDuda, @lzgrablic02, @mrjerryjohns, @msandstedt, @mspang, @pan-apple, @robszewczyk, @sagar-apple, @saurabhst, @selissia, @tecimovic, @turon, @vivien-apple, @wbschiller, @woody-apple and @yufengwangca,
Please kindly help to review this PR and leave your valuable suggestions, many thanks!

fast track: this has been open  for a while now and it is a platform-specific example.


Add SpreadOperator to exclude-rules in test-pattern.

I don't think the performance cost is really an issue when running tests and hence we should disable it by default.
Is there anything else where I would need to add this? I lost track of where things belong.

Are there any performance measures available? If the spread operator has no significant performance impact I am okay with that.

@schalkms here is the issue and a linked benchmark.
I think it is ok for test cases to exclude this rule on default so 👍 . @Mauin waiting for your approval :)


CI: Fix values from build matrix

Fixes the parsing of the OS and Python version values from the build matrix in the GitHub Actions CI.
Small fix after #47.

Awesome! Thanks for the quick fix.


Fix horizontal scrollbar in actions panel

Now @DamnClin can finally have peace of mind

can it fix https://github.com/jhipster/jhipster-lite/issues/3387 ?


Follow RFC to allow client_secret to be empty string

Hi thanks for the authorization server! I am seeing an edge case here: My app originally communicates with Keycloak, and now I am migrating to Spring authorization server. The app has configured an empty client_secret (I do not fill in a strong password, because any "secret" in an app can be reverse engineered so is not a real "secret"). It worked well when sending requests (e.g. login) to Keycloak, but gives errors in Spring Auth Server.
After debugging, it seems that, if I choose a non-empty client secret, it works. I checked https://datatracker.ietf.org/doc/html/rfc6749#section-2.3.1 (linked from ClientSecretPostAuthenticationConverter), as well as https://datatracker.ietf.org/doc/html/draft-ietf-oauth-v2-1-08#section-2.4.1 (oauth 2.1). They say it is "required" but not say it is "nonempty". Indeed, oauth 2.0 says, "The client MAY omit the parameter if the client secret is an empty string", so I guess the RFC does allow an empty string.

@fzyzcjy ClientSecretBasicAuthenticationConverter and ClientSecretPostAuthenticationConverter both require client_id and client_secret.
The relevant information in section 2.3.1 Client Password:

Clients in possession of a client password MAY use the HTTP Basic
authentication scheme as defined in [RFC2617] to authenticate with
the authorization server

Alternatively, the authorization server MAY support including the
client credentials in the request-body using the following
parameters:

Regarding your comment:

"The client MAY omit the parameter if the client secret is an empty string", so I guess the RFC does allow an empty string.

If the client secret is an empty string, then the client was not assigned client credentials and only a client identifier. In this instance, the spec states that the client_secret parameter must be omitted from the request.
I'm going to close this PR as the current implementation complies with the spec.

@jgrandja Hi thanks for your reply! However, I am still confused.

Clients in possession of a client password
including the client credentials in the request-body

The RFC does not say whether the password is an empty string or must be a non-empty string IMHO. Could you please provide a bit more details why having a password means having a non-empty password?

client_secret
REQUIRED.  The client secret.  The client MAY omit the
parameter if the client secret is an empty string.
If the client secret is an empty string, then the client was not assigned client credentials and only a client identifier. In this instance, the spec states that the client_secret parameter must be omitted from the request.

I am not sure why client_secret MUST be omitted, since the RFC says it MAY be omitted.

@fzyzcjy

Could you please provide a bit more details why having a password means having a non-empty password?

Client credentials qualifies as client_id and client_secret. An empty client_secret means the client was not assigned client credentials. There are no authentication systems out there that allow for empty passwords. If there are then they are insecure.
Furthermore, if an authorization server allowed a client to be configured with an empty client_secret and allowed the client to authenticate via HTTP Basic or the client credentials in the request-body, then client impersonation can easily be performed since the client_id parameter is exposed in the request and can easily be obtained by a malicious user.

My app originally communicates with Keycloak, and now I am migrating to Spring authorization server. The app has configured an empty client_secret (I do not fill in a strong password, because any "secret" in an app can be reverse engineered so is not a real "secret"). It worked well when sending requests (e.g. login) to Keycloak, but gives errors in Spring Auth Server.

If this worked in Keycloak, then the client configured must have been a Public Client and NOT a Confidential Client.
Again, the current implementations have been implemented to spec but if you still need to customize the hooks are available. See the reference doc for further details.

@jgrandja Thank you for the information! I understand the explanation.

An empty client_secret means the client was not assigned client credentials. There are no authentication systems out there that allow for empty passwords. If there are then they are insecure.

I am not an expert about the RFC, but it seems that I cannot find this sentence... At least 2.3 explicitly mentions an empty string as the secret.

Furthermore, if an authorization server allowed a client to be configured with an empty client_secret and allowed the client to authenticate via HTTP Basic or the client credentials in the request-body, then client impersonation can easily be performed since the client_id parameter is exposed in the request and can easily be obtained by a malicious user.

I agree it can be easily hacked. However, IMHO, if you set a weak password to the client, then it is still easily hackable even if it not empty. In addition, it seems that the RFC is the thing to follow.

If this worked in Keycloak, then the client configured must have been a Public Client and NOT a Confidential Client.

Maybe you are right :) Then I hope this great repo supports public client as well.

Again, the current implementations have been implemented to spec but if you still need to customize the hooks are available. See the reference doc for further details.

Yes I have already customized it, and I PR just because future people may need this IMHO.

@fzyzcjy

Maybe you are right :) Then I hope this great repo supports public client as well.

Yes, it supports Public Clients. See this How-to guide on how to configure.

@jgrandja Thank you! However IIRC I have tried it and it did not work, and I am still confused which part of the RFC forbids empty password.


container media query is wrong

Hello!
Thanks for this utility, is awesome!!!
I am testing and can see that the container media query no is orderer correctly

my monitor is 1920 x 1080 and take 768
and see this in css

when i reorder manually this is fixed

Thank you, https://github.com/voorjaar/windicss/commit/2a1e69a270ffcf1151b98ffe17bca1bd88c4ee8b fix this error.
Try npm install again.

Thank you, https://github.com/voorjaar/windicss/commit/2a1e69a270ffcf1151b98ffe17bca1bd88c4ee8b fix this error.
Try npm install again.

I just realized that other media queries still have sorting problems, so I added a new sorting algorithm.
https://github.com/voorjaar/windicss/commit/809e928b5a1f1a6401895823aa37494291e957df is the final solution.

I just realized that other media queries still have sorting problems, so I added a new sorting algorithm.
https://github.com/voorjaar/windicss/commit/809e928b5a1f1a6401895823aa37494291e957df is the final solution.

excellent, update the dependencies and I see that now it is ordered correctly.

But I just noticed something else, I did these tests in interpretation mode, but when I change it to compilation mode, the container class and its media queries are ignored.

excellent, update the dependencies and I see that now it is ordered correctly.

But I just noticed something else, I did these tests in interpretation mode, but when I change it to compilation mode, the container class and its media queries are ignored.

https://github.com/voorjaar/windicss/commit/13e21803b304c439ec1a0341ba665e94069cc92f fixed this.
Thanks again. If you find any new issues, please let me know. I will keep this issue open for now.

https://github.com/voorjaar/windicss/commit/13e21803b304c439ec1a0341ba665e94069cc92f fixed this.
Thanks again. If you find any new issues, please let me know. I will keep this issue open for now.


trackDependency for node

currently there's a trackDependency example for Java, C#, and "Javascript", but it's unclear whether the Javascript example is also for node.js. We should either add a Node.js example, or rename the Javascript example to Node.js (if it is indeed intended to be a node example)

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

ID: 401d2997-0fbd-7966-8996-edbaab1819ff
Version Independent ID: c3ee3318-4c69-8ded-a8d5-5059d699351b
Content: Application Insights API for custom events and metrics
Content Source: articles/azure-monitor/app/api-custom-events-metrics.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@morgangrobin Thank you for your feedback. I'll review it and get back to you shortly.

I've put in a pull request to update the heading for that sample.


consider using stencils and offsets in place of canvases for ui

using a canvas for each widget was a somewhat more complex choice than i realized at the time. it's worth seeing if the same degree of abstraction can be provided using stencils and transform offsets. this might also cause the game to use less resources

https://love2d.org/wiki/love.graphics.stencil

the main downside of this will probably be that it leaks abstraction if a widget needs to set transforms itself. should probably do our best to only use relative transforms within widgets

this would be mutually exclusive with making it so that widgets only redraw themselves when required, instead of every frame

this would be mutually exclusive with making it so that widgets only redraw themselves when required, instead of every frame

considering the downsides, i think it's better to drop this one

considering the downsides, i think it's better to drop this one


Update documentation of the Zustand library

Tasks
Enhance existing documentation of the Zustand library with:
[ ] Persist middleware
[ ] Persist's partialize method
Use case
No response
Related Pull request
No response

I think this is done :heart: Please reopen if I misunderstood something.


All appsettings.json environments deployed to AWS

Describe the bug
All appsettings.json files are deployed to AWS even when using the csproj file to control which files are copied into the bin folder. The proper files are copied into the bin folder per configuration selected. However, the temp folder that is used to bundle the app before sending to AWS has all of the projects appsettings.json files. This causes a problem as the production appsettings overrides the others.
To Reproduce

Create an appsettings.production.json and appsettings.development.json files
Update csproj file with <ItemGroup> conditions (included below)
Right click the project and select "Publish to AWS..."
Select existing target and then "Edit Settings" from the far right
Change "Dotnet Build Configuration" to Debug
Publish and wait for temp folder to be created
Navigate to folder and see additional files that were not in the bin folder for that configuration

Expected behavior
Appsettings.json files that are excluded in the csproj file should not show up in the temp folder which is then packaged and set to AWS.
Screenshots
Bin folder

Temp folder create by AWS Toolkit

Computer (please complete the following information):

Windows Version: Windows 10 21H1
Visual Studio Version: 2022 Version 17.3.0
AWS Toolkit for Visual Studio Version: <IP_ADDRESS>

Additional context
Conditional files in csproj
<Choose>
  <When Condition=" '$(Configuration)'=='Debug' ">
    <ItemGroup>
      <Content Update="appsettings.json">
        <CopyToOutputDirectory>Always</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Development.json">
        <CopyToOutputDirectory>Always</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Production.json">
        <CopyToOutputDirectory>Never</CopyToOutputDirectory>
      </Content>
    </ItemGroup>
  </When>
  <When Condition=" '$(Configuration)'=='Release' ">
    <ItemGroup>
      <Content Update="appsettings.json">
        <CopyToOutputDirectory>Always</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Development.json">
        <CopyToOutputDirectory>Never</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Production.json">
        <CopyToOutputDirectory>Always</CopyToOutputDirectory>
      </Content>
    </ItemGroup>
  </When>
  <Otherwise>
    <ItemGroup>
      <Content Update="appsettings.json">
        <CopyToOutputDirectory>Never</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Development.json">
        <CopyToOutputDirectory>Never</CopyToOutputDirectory>
      </Content>
      <Content Update="appsettings.Production.json">
        <CopyToOutputDirectory>Never</CopyToOutputDirectory>
      </Content>
    </ItemGroup>
  </Otherwise>
</Choose>

Hi @issaaccbb,
Good morning.
Thanks for reporting the issue. Upon reviewing further, looks like your .csproj is wrong and needs CopyToPublishDirectory attribute to control what files will be copied when running the dotnet publish command. Tested it and it worked as expected.
Regards,
Ashish


HTTP to HTTPS redirect listener

An attempt to solve #308. I apologize in advance, I'm very new to go. This pr listens on a separate port that will redirect to the host and scheme of the configured redirect-url. The thinking is the redirect-url will be the publicly accessible https URL when wanting to re-direct http -> https.
Thanks!

@jehiah this is a patch for https://github.com/bitly/oauth2_proxy/issues/308

Thanks @bshelton229. I've been looking for a solution for this for a while.
@jehiah I'm intersted as well. Any thoughts on if/when this would be merged and released?

+1

hey guys, any updates regarding this very useful feature? thank you

hey guys, any updates regarding this very useful feature? thank you
cc @jehiah

@idntfy We are trying to slowly work through the open PR's in the v2.3 milestone. I've queued this (along with letsencrypt support) into the v2.4 milestone. Sorry for the delay.


Windows 10 2004, application starts then closes

Tried reinstalling, deleting /Appdata/Roaming/extraterm, nothing helps

I am having the same issue, tried the same mitigations. Hapens with Binary install and choco install.

For me, the problem was i didnt have wsl distibution installed, after i have installed ubunu distribution, and got wsl command functioning in command prompt (opening linux subsystem prompt) everything started working.
Seems extraterm defaults to wsl on windows these days, which is really strange, should be cmd imo.


API support for Iterator

Hi,
I really appreciate that you wrote an Obj-C wrapper for Tesseract on macOS! I am finding very useful but I was wondering if you would be open to adding more features to this API, particularly adding an iterator. The inspiration is from the example API for Tesseract. It doesn't seem particularly hard to implement but we would probably need to discuss on how to implement it, e.g., being its own function or an optional result for the existing function recognize since the flow would be very similar.
I can submit a PR if you would like to get things started.

Sure, feel free to discuss that with me. Can you provide me with a use case scenario for this feature?

My particular use case is doing recognition per letter (the iterator has more options) but sometimes Tesseract's best answer is not good enough. However one of the other possible choices that Tesseract considers might be right. Tesseract doesn't normally expose the other choices because they have a lower confidence interval but I would like to expose those options through its API and manipulate so the user can do "manual" correction so to speak.

I see. That would be a useful feature that we might include.
If you feel confident in writing it up yourself, feel free to do that and make a PR.
Otherwise, I'll do it when I'm free. Thanks.

I will start on the PR!

Fixed in #2


i18n(de): update recipes/add-yaml-support.mdx

Description (required)

update recipes/add-yaml-support.mdx
(one of the few files that survived #9240)
change translation of Rezept (recipe) to Beispiel (example) because it fits better in German

Related issues & labels (optional)

#4666
#4805
#5305
#5846
#8167

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

de/recipes/add-yaml-support.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

OK @trueberryless ! V5 is out, so I'm taking off the block, but please do update your branch because the site has changed A LOT since this PR was made. 😅  I'm sorry if there might be conflicts to deal with!

Thank you very much and sry for causing some trouble shortly before release...


import.meta.env类型造成歧义

Describe the bug
尽管在 env.d.ts 中 ImportMetaEnv interface中 定义了环境变量 booblean 或者其他非 string 类型
在客户端使用import.meta.env仍得到一个string类型的值，但得到的 typescript 的类型提示确是自己定义过的
这可能会引起一个隐藏错误或歧义！
目前解决方案是所有 ImportMetaEnv 内属性均定义为 string  , 自己手动转换为对应类型。这样不会导致直接使用 import.meta.env 得到一个与值不对应类型的类型提示。
我觉得这点需要在文档中标注出来！
Reproduction

System Info
System:
    OS: Windows 10 10.0.18363
    CPU: (16) x64 Intel(R) Xeon(R) CPU E5-2665 0 @ 2.40GHz
    Memory: 8.24 GB / 15.92 GB
  Binaries:
    Node: 14.17.4 - E:\software\node\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.14 - E:\software\node\npm.CMD
  Browsers:
    Edge: Spartan (44.18362.449.0)
    Internet Explorer: 11.0.18362.1
  npmPackages:
    @vitejs/plugin-legacy: ^1.6.4 => 1.6.4
    @vitejs/plugin-vue: ^2.0.0 => 2.1.0
    @vitejs/plugin-vue-jsx: ^1.3.3 => 1.3.3
    vite: 2.7.13 => 2.7.13

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

https://github.com/vitejs/vite/discussions/6637

Closing as duplicate of #6637. Vite uses dotenv which parses .env files as string key-value pairs.

@bluwy would it make sense to keep this topic in an issue instead of a discussion? ImportMetaEnv interface is incorrect and should be tracked as an issue imo.

As far as I know, we can just obtain a boolean env var via define which is compatible with esbuild.


URI as Metric causes error in UI

If you record a new metric where the Metric Name contains /'s it will blow up the UI due to the API call that it makes.  These should probably be escaped by default or the /api/tagk/ handler should catch and use the entire path after it instead of just the first segment.

Duplicate of #1305


Balance Button does not remove joiners from teams

If a player is on NOAT and waiting to join a team while the Game Commander
clicks the 'Balance' button to distribute everyone to a team, the player's name
will remain blinking on the team list as if they are still trying to join. This
even occurs when the balance button assigns them to the opposite team, causing
weird UI quirks.
Imported from trac ticket #13,  created by Tigereye on 01-05-2010 at 09:58, last modified: 07-31-2010 at 15:15

'''juckto'''
It also occurs when a player is requesting to join a game in progress, and the
commander hits #autoaccept.
If the player cannot be accepted due to imbalance reasons then he is put back
on NOAT with a message to that effect, but everyone else still sees them
blinking.
Trac comment by fuzzylunkin1 on 01-05-2010 at 09:59

I may be able to do this.  I'll look into it.
Trac comment by Xynth on 07-28-2010 at 05:44

r871
Trac comment by Xynth on 07-31-2010 at 15:15

Moving the juckto additional issue to a new ticket.  Will take two separate changes.
Trac comment by Xynth on 07-31-2010 at 14:45