NVDA does not read showInputBox content correctly

Repro Steps:

Download Live Share extension for VSCode
Go to the "share" icon in the status bar, click it, and sign in to start a collaboration session
Click the name icon in the status bar

Select "share server" in the quick pick menu that appears
Type any number in "Enter port to expose..."
Verify that NVDA reads the user input and the prompt, with no extra information

OR

Show any input box with vscode.window.showInputBox API with a non-null value provided for the prompt option
Enter some input in the input box
Verify that NVDA reads the user input and the prompt, with no extra information

Actual Result:
NVDA reads some background content before reading the input entered by the user, and NVDA does not read the prompt under the input box
Expected Result:
NVDA reads the input entered by the user and the prompt under the input box. NVDA does not read any extra information.
User Impact:
Screen Reader user will find difficulty in using the application if it does not read the contents and controls that are appearing on the screen properly
MAS Reference: https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={54f28d1f-a2d1-4dcd-84e1-5c9b87e8aba4}
OS: Windows 10 Enterprise 17134.228

Here is our input box ^

Possible dupe of https://github.com/Microsoft/vscode/issues/55186


Kaspersky Endpoint Security for Windows <IP_ADDRESS>6 thinks dbeaver-cli.exe is malware

Description
Hi, I open this issue bcoz in milestone 24.0.5 the problem not solved.
Kaspersky Endpoint Security for Windows <IP_ADDRESS>6 still thinks dbeaver-cli.exe is malware. Could you find a solution for this? (I use portable version)

DBeaver Version
24.1.1
Operating System
windows 10
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Thank you for report. We will try fixing it.

Hi, I started using 24.1.2 this morning. Interestingly, I didn't have any problems. For your information, the antivirus program didn't cause any problems

Today, Kaspersky deleted dbeaver-cli.exe on 24 1.2


buildHelmChart: namespace not actually optional

namespace is allowed to be null:
https://github.com/farcaller/nix-kube-generators/blob/6b7b51b332ff1eb023574c54981a67b6bb261b1a/lib/default.nix#L86
but is also used in the derivation's own name, where a null value results in:
error:
       … while calling the 'derivationStrict' builtin

         at //builtin/derivation.nix:9:12: (source not available)

       … while evaluating the derivation attribute 'name'

         at /nix/store/y295j2wgd271jxqmman497xbcyr99pw9-source/pkgs/stdenv/generic/make-derivation.nix:348:7:

          347|     // (optionalAttrs (attrs ? name || (attrs ? pname && attrs ? version)) {
          348|       name =
             |       ^
          349|         let

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: cannot coerce null to a string

My first instinct was to send this patch:
diff --git a/lib/default.nix b/lib/default.nix
index daaf8c0..ec7cfff 100644
--- a/lib/default.nix
+++ b/lib/default.nix
@@ -90,7 +90,7 @@ rec {
     , apiVersions ? []
     }:
     pkgs.stdenv.mkDerivation {
-      name = "helm-${chart}-${namespace}-${name}";
+      name = if (!builtins.isNull namespace) then "helm-${chart}-${namespace}-${name}" else "helm-${chart}-${name}";

       passAsFile = [ "helmValues" ];
       helmValues = builtins.toJSON values;

but it seemed a bit ham-handed, and I'm not yet at the level of writing idiomatic Nix.

Good catch! I think the patch makes sense, too. You can’t use the or operator in here as we want less data if it's NULL, not more.
I guess extracting it into a let ... in would improve the readability, though. I’m happy to accept such a PR.

I didn't even think of a let-in -- thanks for the jump-start.

Fixed in bba2e70a98ea6693cf755da50c8bb3e3bf1730a0


Port forwarding does not work

I rebuilt opencloud-neutron-plugin to work on top of the ML2 neutron plugin. Instructions are in git.planet-lab.org: [opencloud-plugin.git] / ml2_plugin / README. Tested this on Washington.
Needs to be deployed to other Vicci headnodes and/or automated in install cloud.
Note the Juno supports "extension plugins" for ML2, and will yield a much cleaner approach.

I've added Scott's Neutron changes to the nova-cloud-controller Juju charm and deployed it on all the clusters.  Port forwarding should be working on all clusters now, and should come up automatically when we install a new cluster.


[Consensus] Normalise node IDs passed to Tendermint

Node IDs are addresses of the ed25519 node key - Tendermint now expects them to be lower case in seed, persistent peers, and for dialing. I see no reason not to normalise them (that is downcase the ID) before passing to Tendermint. For example this would help with:
https://github.com/hyperledger/burrow/issues/1050

I think I'm going to change my mind on this. The failure due to casing differences is annoying, but the fix may be worse than the cure. It's not clear where to hook the conversion and could lead to inconsistency with a node id working in one place and not another. If tendermint wants to treat them differently, so be it.


Tide example

Just stumbled across async-graphql. Looks like you've done a lot of great work in a short amount of time. Nice work.
I wanted to experiment with using async-graphql with tide in my project. It seems to work fine, as the attached example shows.

Thank you for your contribution!

here is the latest example.


Binding multiple values

When I want to store an UUIDv4 in my MySQL database I use UNHEX(REPLACE(?, "-", "")) to store it efficiently in an BINARY(16) column like this:
await client.execute(
  `INSERT INTO test.single (first) VALUES(UNHEX(REPLACE(?, "-", "")))`,
  [
    `b49450a5-5d16-4ff6-81f8-faf83f04f9a0`,
  ]
);

Which results in a single row being inserted, as expected! When I try to run the same query but insert into 2 BINARY(16) columns like this:
await client.execute(
  `INSERT INTO test.double (first, second) VALUES(UNHEX(REPLACE(?, "-", "")), UNHEX(REPLACE(?, "-", "")))`,
  [
    `b49450a5-5d16-4ff6-81f8-faf83f04f9a0`,
    `b49450a5-5d16-4ff6-81f8-faf83f04f9a0`
  ]
);

The driver throws an error:

error: Uncaught (in promise) Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '?, "-", "")))' at line 1
throw new Error(error.message);
^
at PoolConnection.nextPacket (https://deno.land/x/mysql@v2.9.0/src/connection.ts:185:13)
at async PoolConnection.execute (https://deno.land/x/mysql@v2.9.0/src/connection.ts:268:21)
at async https://deno.land/x/mysql@v2.9.0/src/client.ts:97:14
at async Client.useConnection (https://deno.land/x/mysql@v2.9.0/src/client.ts:107:14)
at async Client.execute (https://deno.land/x/mysql@v2.9.0/src/client.ts:96:12)
at async file:///usr/local/var/www/Deno/index.ts:28:1

Now I would assume there is something wrong with my query but when I run the same query but instead of binding the variables I just hard code them in the string like this:
await client.execute(
  `INSERT INTO test.double (first, second) VALUES(UNHEX(REPLACE("b49450a5-5d16-4ff6-81f8-faf83f04f9a0", "-", "")), UNHEX(REPLACE("b49450a5-5d16-4ff6-81f8-faf83f04f9a0", "-", "")))`,
);

It inserts just fine, what am I doing wrong? If need be I can provide credentials to my database but a friend of mine running a completely different database has the same issue so I think it isn't specific to my MySQL configuration.
These are the tables I'm running the queries on for debugging purposes:
CREATE TABLE test.double (
 	first binary(16) NOT NULL,
	second binary(16) NOT NULL,
	PRIMARY KEY (first)
)

CREATE TABLE test.single (
 	first binary(16) NOT NULL,
	PRIMARY KEY (first)
)

This is because of an open issue with a dependency: https://github.com/manyuanrong/sql-builder/issues/9.

The scope of the issue goes beyond "multiple values" -- any ? that is not at the end, will not bind reliably. (But yes, this is an issue with the dependency)


Allow overwriting mulcache size in native implementations

Currently, the poly(vec) mulcache has a fixed size that native implementations have to adhere to. This should be configurable, so that native implementation can (a) use a mulcache of a different size, or (b) not use one at all.

I don't think this is essential for the alpha-release. Moving to the next one.


Made IPC ZSTD-compressed consumable by pyarrow

I do not understand why the level must be zstd's default to work, but it works. This is either a bug on zstd's crate or on pyarrow's ZSTD's compressor :/

That magic number goes somewhere in metadata I assume? Now we aligned with pyarrows 'level'?

Yes, the thing is that the "level" is just a number that tradeoffs performance for compression, so the fact that only one level works is a bit worrisome imo - the ability to read and write ZSTD-compressed data should be independent of the level. This PR just matches the level, but why other numbers do not work is unclear to me.

Hmm.. a bit hard to test whats at fault here. As there are not many ipc implementations. xD

exactly! xD The evidence (prior to this PR) is:

pyarrow reads what pyarrow writes
pyarrow does not read what arrow2 writes
arrow2 reads what pyarrow writes
arrow2 reads what arrow2 writes


Error 3070003: Serialization Error Processing WASM, Error Details: Too many function defs

I wrote a contract and compiled it successfully using eosio-cpp version 1.6.2.
Then, when I tried to deploy it, I got the below error:
Error 3070003: Serialization Error Processing WASM
Error Details:
Too many function defs
This message is from
https://github.com/EOSIO/eos/blob/master/libraries/wasm-jit/Source/WASM/WASMSerialization.cpp#L683
which means that the number of function definitions are more than 1024.
https://github.com/EOSIO/eos/blob/master/libraries/chain/include/eosio/chain/wasm_eosio_constraints.hpp#L14
Actually, my code has about a dozen of functions (class methods).
If I remove some of part of a function from the contract code, deployment succeeds.
Also, if I divide the contract code into three parts, deployment of each contract succeeds.
original contract .wasm: 254.0 KiB
three-divided contract .wasms : 132.8 KiB, 171.2 KiB, 159.9 KiB
I have to modify each divided contract code accordingly, though.
Incidentally, eosio.token.wasm (from eosio.contracts v1.7.0): 103.8 KiB
My question is: Are there any other ways to cope with this issue?

Sorry, this was by my mistake.

@keisuke-kanao I have the same issue now. Can you explain what the mistake was?

Phillip Hamnett - Chintai, [10.09.20 09:01]
[In reply to Phillip Hamnett - Chintai]
@tbfleming @heifner this seems to be an issue with contracts being too large, but surely that would be very limiting when dealing with very complex applications like what Chintai is dealing with
Kevin Heifner, [10.09.20 09:10]
[In reply to Phillip Hamnett - Chintai]
Do you have more than 1024 functions?
Phillip Hamnett - Chintai, [10.09.20 09:11]
[In reply to Kevin Heifner]
Do you mean functions, or actions? I have about 80 actions, counting the functions now
Kevin Heifner, [10.09.20 09:11]
It is total functions, including actions and all functions
Phillip Hamnett - Chintai, [10.09.20 09:15]
[In reply to Kevin Heifner]
If it turns out I have reached the limit of the number of functions, what would you recommend as the best course of action?
Kevin Heifner, [10.09.20 09:16]
Inline some of them :(
Kevin Heifner, [10.09.20 09:17]
If this is a private net you could increase the limit. If public, you would need a consensus change.
Phillip Hamnett - Chintai, [10.09.20 09:19]
[In reply to Kevin Heifner]
Is there any obvious downside to increasing the limit?
Kevin Heifner, [10.09.20 09:19]
On a private chain, no.
Phillip Hamnett - Chintai, [10.09.20 09:20]
[In reply to Kevin Heifner]
And on a public chain? If I have to go to the BPs to ask them to do this, what potential downsides are there?
Kevin Heifner, [10.09.20 09:21]
It is not currently configurable without modifying the code. I doubt BPs would be willing.
Phillip Hamnett - Chintai, [10.09.20 09:25]
[In reply to Kevin Heifner]
Me too. But it seems that eventually any piece of code might go over 1024 functions. I guess I'm asking why the limit was put into the code in the first place
Matt Witherspoon, [10.09.20 09:30]
EOSIO 1.0 had a number of WASM protections in place to protect from abuse (such as a WASM declaring it had 1000000 functions that would cause a huge slow malloc blocking the main thread). All these protections are part of consensus. Unfortunately some of the limits were probably too strict, such as this one.

https://github.com/EOSIO/eos/pull/8360

Any additional updates on this? I'm getting this error when running unit tests for my contract. My contract is pretty large and complex. Plus I have some additional test actions that I need for unit testing. Does eosio-cpp remove unused code blocks and functions by default?


encode str in python3

this PR corresponds to #1

Hey! Thanks for the PL!
Is anyone still maintaining this repository?


While I'm not particularly familiar with the way in which the jolly advisor uses rails, pulling the base image from the official tool on docker hub is generally the way in which one would want to take care of this unless your saying you'd want there to be a completely custom made docker image that only includes the parts of rails necessary to build the advisor.
Another idea to set up the docker build in a much better way than it is right now would be to use docker-compose which is essentially a yaml file that would build the various images, run them as containers and connect them however you specified in the docker-compose.yml file.

I want the former. The official image installs nodejs, along with clients for mysql, postgres and sqlite. This is silly, even for an official image, since you will only use one client for your application.
I will take a look at docker-compose, though. Thanks for the suggestions!

You need nodejs or some other javascript runtime environment installed for rails to work in a container, but your best option given how things are done currently would probably involve just modifying the source for from rains:onbuild and modifying it so that it only installs what you want, push that to dockerhub and then reference it as needed.
Long term it would absolutely be worth it to use docker compose as you can configure it so that compose spins up the database as well.

docker-compose, AFAIK, is not entirely relevant to the original issue.  I am not entirely sure why the default rails dockerfile contains all this extra stuff, but it should be possible to write a custom one that only includes rails plus the basic requirements we have (I can work on that later if needed).
Regarding compose: it ads a (rather nice) layer of abstraction that, once you have containers for all your services, allows you to define how they should be launched with respect to each other (eg, container one depends on container two and three, with certain network connections, etc).
Two reasons it's not currently used in production: 1) it was super beta when I designed the current deploy system on docker.case.edu, and 2) its networking options are not quite flexible enough for what is needed in relation to Case's network.  1) has changed since then, but I don't think 2) has.
I won't go into full detail of how the deploy system works now, but the TL;DR is that it is a simple replacement for docker-compose written in bash with Case's specific network needs taken into account.

@ZJaffee can you explain (maybe offline?) why you need nodejs to run rails in a container? I have searched for it, and have not found an reason beyond "you just need it".
@matthewbentley I think we should look into using docker compose in prod, do you want to open another issue for it?

I don't know all of the details, but it's my understanding that because unless you want to remove all instances of this, https://github.com/hacsoc/the_jolly_advisor/search?utf8=✓&q=execjs, you need to have some type of javascript runtime environment installed for it to work. Given that you are using a linux server and container, nodejs is most likely your best choice of runtime environment. https://github.com/rails/execjs has a few other runtime environments you could choose from, but installing and configuring those is most likely going to be more complicated than apt-get upgrade \ apt-get install nodejs

But we already install therubyracer (https://github.com/hacsoc/the_jolly_advisor/search?utf8=✓&q=therubyracer&type=Code), so shouldn't that not matter?

It shouldn't matter then, in fact what you need is really simple, I'll just make a pull request because it will be easier to show you that way

Sounds good. I'll assign this to you then. Feel free to unassign if you don't feel like doing it anymore.

@matthewbentley this can probably be closed now, yes?

Probably fine.  We actually base off of ruby:2.3.0-onbuild now, so I'm not sure if the original issue is even a problem


Setting FA Icon on 5th UITabbarItem causes NSException

Great library - thanks! (new to Swift/iOS, though been using FA icons for many projects - glad to use them for latest one!)
Question is on setting the icon for tabs when you have more than 5.
Setting each was simple enough (the documentation states to use 'first', though we found we can do this for each one...)
`
tabBarController?.tabBar.items?[0].setFAIcon(icon:.FAUser)
    tabBarController?.tabBar.items?[1].setFAIcon(icon:.FAUser)

    tabBarController?.tabBar.items?[2].setFAIcon(icon:.FAUser)

    tabBarController?.tabBar.items?[3].setFAIcon(icon:.FAUser)

    tabBarController?.tabBar.items?[4].setFAIcon(icon:.FAUser)

`
Works great - IF you have only 5 tabs........
However, if you have a 6th tab, [4] no longer works....
When setting a 6th (or more) tab, iOS changes "#5" to "More" with a ... 'icon' and then the user can select which one they want (and also the user can arrange the tabs - not highly useful in our app, but a cool feature, I guess..)
So, this does NOT work..... (
        tabBarController?.tabBar.items?[X].setFAIcon(icon:.FAUser)
(where 'X' is 5 or more)
Doing so get an error because the array is only 5 long.....

*** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayI objectAtIndex:]: index 5 beyond bounds [0 .. 4]'

How to set the icons as FA Icons for these 'More' tabs?

@TCOA I just tried what you were trying to do. And the error you have mentioned above indicates that you haven't added the 6th view controller in the storyboard, that is why you cannot set the tab bar item for index 5.
Can you please try to add 6th view controller and link it with 6th tab bar item and see if this works? It will add last two items in "More" section.

well, it has been awhile (and tons of code since then!) since I did that, though I found a solution that works (not my idea - found it online somewhere - sorry, no reference to where...)
class MyViewController: FormViewController{ required init?(coder aDecoder: NSCoder) {         super.init(coder: aDecoder)         // Initialize Tab Bar Item         tabBarItem = UITabBarItem(title: "TAB TITLE", image: UIImage.init(icon: .FAMoney, size: CGSize(width: 35, height: 35)), tag: 1)     } ......... (finish the class)
Although you have to put this on every ViewController, it allows for setting the icon as well as the title for as many tabs as you have, and not running into the limitation of the array, so this is a much better overall solution (anyway, it worked great for me!)


Trigger preview site builds on PRs.

Have you read this:
https://securitylab.github.com/research/github-actions-preventing-pwn-requests
pull_request_target is dangerous since it will have access to secrets and write permission to the repo. But in this workflow you want to build and push a firebase deployment.

Have you read this:
https://securitylab.github.com/research/github-actions-preventing-pwn-requests
pull_request_target is dangerous since it will have access to secrets and write permission to the repo. But in this workflow you want to build and push a firebase deployment using the untrusted changes in the PR.

Yeah, +1 that `pull_request_target doesn't seem to be the right choice here


Integration with Mesos maintenance mode

On Mar 14, 2016, at 12:12 PM, Joseph Wu<EMAIL_ADDRESS>wrote:
Managing maintenance is currently up to the operator (you, presumably).  If you have something to contribute (code, docs, or examples), that would be greatly appreciated :)
We haven't prioritized other integration (like the CLI or web UI) since maintenance primitives themselves need to be supported by frameworks using the V1 HTTP API; and frameworks developers have not all started/finished migrating to the new API yet.  (Without framework support, frameworks will not react to your maintenance windows.)
Here's the tracking ticket for adding maintenance info to the web UI (no work has been done on this yet):
https://issues.apache.org/jira/browse/MESOS-2082

Is anybody working on this? It require switch from native mesoslib to HTTP API first.

Nobody is actively working on this at the moment. We have discussed the switch to using the HTTP API, but that work has not been started or been scheduled yet. Internally we use Singularity's Decommissioning and Freezing of slaves to provide maintenance windows for us.

We're running experimental Jenkins and Spark frameworks on our Mesos / Singularity clusters.  We use Singularity decommissioning to take hosts out for maintenance.
This means that our other schedulers can still place tasks.  This is surprising and disappointing from an operator's perspective who hasn't watched the history of this issue (that it predates maintenance mode entirely).

Any updates here? We also need this badly.

@yuriy-filatov it wasn't currently in our planned work. By 'need this badly' is the current decommission buggy or causing you issues, or is it just an inconvenience to also have to remember to set mesos maintenance mode manually? I don't have an operator api client in here yet, but if you'd like to PR this, the steps would be:

Add a client for the operator endpoints on the mesos master (or at least just that one and we can fill out more later)
Add a call to it on the slave decommission code path
Add a call to undo this on the reactivate slave code path

@ssalinas thanks for your quick answer.
Well, we're using Singularity with Mesos 1.6.1 in our Production env. It's using numerous spot fleets with hundreds of instances in AWS, so we usually upscale/downscale racks accodring to required capacity by moving unneeded slaves into maintenance mode in Mesos in order to fence them from scheduler and frameworks. After some timeout these slaves are terminated, but sometimes singularity still tries to operate with these slaves.
Would be great if mesos maintenance state will be synced with singularity in order to mark slaves as dead and drain spot clusters correctly.
Maybe there's another way - simply decomission unneeded slave in singularity, but that's another API call which might transform into some sort of attack in the future.

Ah, so you’re concerned with the opposite of what I was thinking, syncing mesos -> singularity instead of the other way around. That makes more sense in terms of urgency. I will have to check and see what data we are able to get from the mesos master for those events. As far as I knew from previous versions maintenance mode did not send a notification to schedulers.
As for the ‘attack’ a few things:

there is a config for the max number of concurrent decommissions you can use as a protection in Singularity
can’t lifecycle hooks hit an endpoint inside a vpc somit isn’t publicly exposed for an attack?
wouldn’t your mesos master already be hit this same way if that’s how you decommission currently ^?

Max number of concurrent decomissions within one tick sounds reasonable here.

Under 'attack' I've meant our own 'decomission' scripts. We may spin up/down up to thousands slaves at the moment and this number is constantly growing.

Mesos cluster is much more powerful than Singularity one at the moment, but we can upscale it. On the other hand - sending two requests for one operation sounds like an overkill, but easily doable.

Hi - any news on having Singularity pick up maintenance information from Mesos? I'm also running into the issue that when we use the Mesos API to put slaves into maintenance to prevent our other frameworks from starting new tasks, Singularity is still placing tasks. Maintenance information is attached to resource offers, so even if it's not nicely integrated with Singularity's system for maintenance, it would be nice if it would decline to use offers that are marked as currently unavailable.
To me it's strange that Mesos even makes such offers, but they seem to have made maintenance windows completely advisory rather than enforcing anything.


Support Intrinsic Size Attribute

https://github.com/ojanvafai/intrinsicsize-attribute
https://www.chromestatus.com/feature/4704436815396864
https://googlechrome.github.io/samples/intrinsic-size/index.html
https://css-tricks.com/aspect-ratio-media-elements-and-intrinsicsize/

The proposal is not being pursued:

https://twitter.com/tomayac/status/1165902659083280384
https://github.com/WICG/intrinsicsize-attribute/pull/18
https://github.com/WICG/intrinsicsize-attribute/issues/16#issuecomment-503245998


Fix go validation check

Fixes https://github.com/netobserv/flowlogs-pipeline/issues/461
The older PR got closed by mistake, referencing it here - https://github.com/netobserv/flowlogs-pipeline/pull/462

Closing this PR since go version validation is automatically done since go 1.21


FBT001 preview behavior may lead to false positives

The rule boolean-type-hint-positional-argument (FBT001) was extended in #7501 , under preview, to also flag positional arguments that have a type hint with any union of types that includes bool.
The intent of the rules is specifically to find places where a boolean is used as a (binary) flag, in a positional argument. If an argument has a type hint of bool or even bool | None, it makes sense to predict that the argument is being used as a flag. But the preview behavior allows any union, even, say bool | str | MyClass. This seems like a situation where it's a bit of a stretch to assume that the user is using the boolean as a flag.
Maybe the concern is unfounded and this doesn't actually happen in the real world, but I figured the community could weigh in a bit on the point before stabilizing this preview behavior.

I'm just curious. Do you have any examples from the ecosystem checks or from real world projects that raised this concern?

It's rare but it does happen - but maybe rare enough that noqa is the answer. Here is an example from pandas:
https://github.com/pandas-dev/pandas/blob/ee0902a832b7fa3e5821ada176566301791e09ec/pandas/io/formats/info.py#L350-L356
and a strange one from Pytorch:
https://github.com/pytorch/pytorch/blob/3473dfa69859f4049abe46075a91014922af198c/torch/onnx/_internal/exporter/_torchlib/ops/hop.py#L13-L19
It's relatively common for there to be a type alias for scalars that includes bool along with float etc., but the rule wouldn't flag that...
All that to say, it's probably rare enough that I'm being overly cautious here.


DM-34865: Harden JupyterLab lifecycle handling

This work improves how Noteburst workers handle JupyterLab pods, specifically when:

A worker starts up and a JupyterLab pod with the assigned identity already exists
A JupyterLab pod is culled out from under the Noteburst worker

Opened with the wrong branch name.


Feature Variant documentation: Improved confusing documentation and fixed odd grammar

Improved confusing documentation and fixed odd grammar in the Feature Variant documentation.

Thanks @rgoldberg


Cow: Watch okdigitalitfirm.com

Cow requests the watch of the watch_keyword okdigitalitfirm\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

okdigitalitfirm\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


HTTP 401 not captured on android

On iOS my authentication view is triggered normally just like in the example. However, on Android, the 401 unauthorised page is shown on the Turbolinks view without being captured by the error handler.
The handleError function is copied directly from the example app and hasn't been changed yet

Some further info: If I don't use startSingleScreenApp and just use visit the error handler works.

I faced this problem on my app too. It seems a problem with turbolinks-android. https://github.com/turbolinks/turbolinks-android/issues/91. It hapens because turbolinks needs javascript loaded on the page to trigger the error on native. I solved it responding with a 401 status  with a blank page with only turbolinks.js appended. Take a look on my wiki https://github.com/lazaronixon/react-native-turbolinks/wiki/Devise

Correcting myself.... I see this problem today on android 7.1.1 . I will investigate..

This is happening on both the simulator and the device.

The devise fix seems to have worked.
Instead of just sending the response header and basic html sending them to an error page with Turbolinks included is working.


What version of az cli does this task use?

It would be great of this document calls out what version of az cli the task actually uses.  Is it pinned to a specific version?  Does it always use the latest?  If not, when does the az cli version increment?  Only when the version of the task increments?  Of course I can run the task and call az --version but it would be great to document what the expected behavior for az cli version is with this task as customers can have dependencies on specific versions of az cli.

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

ID: 8de3304e-98f9-bac0-3ccc-c65f68a89e87
Version Independent ID: 8fb2f6a9-545d-dff1-ff86-974694a85054
Content: Azure CLI task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-cli.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @ushan
Microsoft Alias: UshaN

Hey @13daysaweek , member of the DevOps support team here. Came across this issue. This question is a big "it depends".
TL;DR: CLI version depends on a number of factors, but ultimately comes down to what is on the machine you're running the task on (whether it's ms-hosted agents or a private agent).
It depends on if you're using Microsoft hosted agents or private agents. If you're using Microsoft hosted, it depends on even more variables, such as what agent image you're using (Windows, Linux, mac), as well as the geographic region where your Azure DevOps organization is located. Azure DevOps does a rolling deployment, meaning some customers get updates before others. Will touch on this a bit more in detail below.
The actual task itself doesn't dictate the version of the Azure CLI. It all depends on what version of the CLI is on the image you're using. Let's say as an example you're using the Microsoft Hosted Windows 2019 image. Details on the software and tools installed on the image can be found here: https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md#cli-tools (this is the anchor to the CLI version, but you can scroll through and see the other tools). Currently the Azure CLI version on the Windows 2019 image is 2.25.0 (at time of writing). These images update roughly every week. Let's say a new version of Azure CLI was released tomorrow (hypothetical). It would be roughly a week or two, depending on a number of factors, until the hosted images were updated with the latest version of the Azure CLI. As I stated before, the MS hosted images roll out in a rolling deployment schedule. If the Azure CLI was upgraded  on the hosted images then began deploying, it may take two weeks or so on top of that before all customers see it.
If you're using a private agent on your own infrastructure, then the Azure CLI version only changes when you upgrade / downgrade that tooling.
Hope this provides some clarity for you!

@13daysaweek  Thank you for your feedback!
@MS-alsulli Thank you for the detailed response!


Implement missing methods action not working (Gopls)

Hi, first of all thanks for your plugin it has been so useful.
I found  a bug, this is the first time a code-action is not working for me, when i try to implement missing methods in golang i get the following error:

This code snippet should allow the reproduction:
type MyInter interface {
	Foo()
}

type Impl struct{}

var _ MyInter = (*Impl)(nil)

Simply place the cursor on the line var _ MyInter ..., trigger the code-action and select Declare missing methods  of MyInter.
At first i thought it would be an lsp related error but when I try with :lua vim.lsp.buf.code_action() and select the same action, it does work as intended.
Let me know if i can provide any more information.

Hello, thank you!
Unfortunately, I cannot seem to reproduce this issue.

How have you configured gopls?
What is your Neovim version?

Thanks again, it was indeed a change in the last nightly build.
It should now be fixed!

It does indeed work as expected, thanks for being so fast!


Can't See stack trace after uploading symbols with CLI for android

I'm using this command:
appcenter crashes upload-symbols -a Company_Name/APP-GO-1 --breakpad Tnt-Android-2249-symbols-PS.zip --debug
Altough im receiveing a correct response:
\ Uploading symbols...Response status code: 200
Body: {"symbol_upload_id":"...-5169-1840-8d13-...","app_id":"...-032b-4166-8862-...","user":{"email":"sfsfs.sdfsdf@globant.com","display_name":"sfsfs.sdfsdf"},"status":"committed","symbol_type":"Breakpad","symbols_uploaded":[],"origin":"User","file_name":"upload-2020-01-15T21-20-30Z.zip","file_size":0,"timestamp":"2020-01-15T21:21:10.337Z"}

I'm still unable to see the correct stack trace on appCenter.
I didn't had any problem by doing this for the iOS version of the same app and uploading symbols by hand also works.

Is there something we're doing wrong?

Is there a way we can check what symbols are already uploaded?

Every time we execute appcenter crashes upload-symbols ... it answers with "file_size":0, is this ok?

Thanks in Advance!

Hello, this forum is only used for issues with the documentation. For product support, please use the in-product support channel.


gz fn is inefficient

It uses wc -c which iterates over every single byte, when counting the size. I know wc is portable, but I suggest we use an alternative that does a syscall to fetch the size directly from the filesystem

Mind sending a patch?

Mind sending a patch?

Ok, I'll open a PR that uses stat. I have no idea about its portability, but it seems simple and fast enough.
BTW, I have this
alias sizeof="stat -c '%s'"

In my .aliases file. In know the fs function is more feature-rich, but I like to keep things simple (sometimes)


Placeholder in Input box on Compare Tool

There was a request to change the placeholder text in the URI box of the compare tool. (See below)

I tried directly setting the CSS placeholder attribute (see above). How is this input box being generated? I'm having trouble finding it in index.js.

Hmm, Eric split that menu out as a separate repo for reuse and it looks like that's for sure set here:
https://github.com/stamen/legend-layer-menu/blob/5e6427e00016812096a7f1c717b3f41025373e3e/src/legend-layer-menu.js#L40
which gets pulled into your code on build by this line:
https://github.com/stamen/ecoengine-compare/blob/a21e76ef46780d8cc2619e9628e666225610c4b6/bower.json#L31
We have an ecoengine branch of that, so we could change the copy here:
https://github.com/stamen/legend-layer-menu/blob/5e6427e00016812096a7f1c717b3f41025373e3e/src/legend-layer-menu.js#L40
But it might be better for you to fork the repo and change the bower.json to draw from your fork, so you can easily edit in the future.

I dont know if the above pathways that Dan suggests holds true... confirm?
But we talked about going from this:
GeoJSON API URL or Keyword
to:
e.g., 'Puma concolor' or API URL

I've tried to update this in the legend-layer-menu module, but it requires updating the bower package which is a process I'm unfamiliar with
https://github.com/stamen/legend-layer-menu

@shoeswat @bkg
I ended up merging legend-layer-menu into Compare. It's at the top of index.js right now, but could be moved to its own JavaScript file. The reasons:

Legend-layer-menu has Compare-specific code in it, like the placeholder text
The process of updating legend-layer-menu isn't well-documented (it's a Bower package)
Stamen abandoned work on legend-layer-menu as an independent component, so that repo won't evolve

So the legend-layer-menu repo and bower package will be deprecated. Once you pull the changes, you can go ahead and delete or archive Berkeley's version of the repo as well https://github.com/berkeley-gif/legend-layer-menu

O yea, I pulled this just to make a change in the text box lol. I think it
was requested at some point.
On Fri, Mar 18, 2016 at 7:02 PM, Kai<EMAIL_ADDRESS>wrote:

@shoeswat https://github.com/shoeswat @bkg https://github.com/bkg
I ended up merging legend-layer-menu into Compare. It's at the top of
index.js right now, but could be moved to its own JavaScript file. The
reasons:

Legend-layer-menu has Compare-specific code in it, like the
placeholder text
The process of updating legend-layer-menu isn't well-documented
(it's a Bower package)
Stamen abandoned work on legend-layer-menu as an independent
component, so that repo won't evolve

[image: screen shot 2016-03-18 at 4 00 38 pm]
https://cloud.githubusercontent.com/assets/156229/13894156/45d0ad40-ed22-11e5-82da-b2b4c6287190.png
So the legend-layer-menu repo and bower package will be deprecated. Once
you pull the changes, you can go ahead and delete or archive Berkeley's
version of the repo as well
https://github.com/berkeley-gif/legend-layer-menu
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/stamen/ecoengine/issues/163#issuecomment-198572733


make use of operatorNamespace for rookCephOperatorConfig configmap

The rookCephOperatorConfig configmap is consistently found in the
operator namespace. When interacting with an external cluster located
in a different namespace, it attempts to access it exclusively in that
particular namespace.
Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

/hold
waiting for rook changes to get merged we may get rid of the configmap.

waiting for rook changes to get merged we may get rid of the configmap.

@iamniting Which rook change are you waiting for?

waiting for rook changes to get merged we may get rid of the configmap.

@iamniting Which rook change are you waiting for?

I am waiting for Rakshit PR to get merged.

https://github.com/rook/rook/pull/13200 is merged now, we just need to resync downstream.

This PR is not required anymore we are resolving all the concerns using the PR #2294


{CI} Use azdev test for core and telemetry

Description
The current Unit Test for Core and Telemetry uses tox to run pytest, making that internal pytest uncontrollable.
This PR replaces tox with azdev test so that core and telemetry use the same test framework as other modules.

CI


Move everything from testify/assert to require

It’s time we leave assert behind and make a complete transition to require. I haven’t yet run into a situation where we explicitly want a test to proceed after an assertion failure.

apply gofmt rewriting rules for assert -> require
replace non-asserting assert.ObjectsAreEqual with alternative assertions
update doc to call out testify/require instead of testify/assert

Wow. This was as aweful reviewing as it looks like it would be. But thank you for doing this. Will actually make debugging failures easier as they won't be wrapped in output of other tests.


AWX requires proper dependencies

Add ansbile.utils to dependencies to fix "Could not load "ansible.utils.ipv4": 'Invalid plugin FQCN (ansible.utils.ipv4): unable to locate collection ansible.utils'" while running icinga2_agent role on AWX.

We also get this error when deploying VMs to Hetzner. Thank you!


Typo

On http://sisyphus-js.herokuapp.com/
It should be lose not loose.

Persist your form's data in a browser's Local Storage and never lose them on occasional tabs closing, browser crashes and other disasters!

"Data" should have been referred to as singular as well. Thanks @butchewing, fixed.


Instead of YOY stats, use a min / max instead

Sent over to Sophia for styling, but very similar to #412, #413 and #411.


fix(progress): moves min-width to apply only to inside measure variant

This moves the min-width of the progress indicator into the inside measure variant, so that the min-width (which provides space for the inside measure percentage) applies only when the percentage is shown inside the progress bar. In other cases, the progress indicator will now go down to 0, tracking the percent complete.

Deploy preview for pf-next ready!
Built with commit cb4b3584c6748eb4ea71c38558e69f00365ecac8
https://deploy-preview-940--pf-next.netlify.com


Some sort of "call to action" or "mission statement" describing wtf is actually going on here

Agreed. Need a mission statement.


Appstudio update test-component-pac-izij

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-izij-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
27 seconds
init

✅ Succeeded
26 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
23 seconds
sbom-json-check

✅ Succeeded
11 seconds
optional-label-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-izij-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
27 seconds
init

✅ Succeeded
26 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
23 seconds
sbom-json-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
11 seconds
optional-label-check

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-izij-on-pull-request has successfully validated your commit.


Un-document / deprecate old OO plans?

Before we had generators like count and scan, we had object-oriented implementations of plan logic, Count and Scan. In the spirit of self-consistency and paring down what there is to learn, it might be time to remove these. They add stateful list of subscriptions, flyers, and monitors, which are now better served by the preprocessors.
An OO-based plan is still sometimes a useful thing, so I think we should continue to ship some version of the base class Plan. But I don't think we need to ship ready-made OO counterparts for every plan. It's just one more thing to keep in sync.
Thoughts?

I got a :+1: from @tacaswell in an in-person conversation. Simplifying is good!


can't login, error: server is not running

I want to run yang-exploer on my docker toolbox, following these steps:

docker pull robertcsapo/yang-explorer
docker run -d --name yangexploer -p 8088:8088 robertcsapo/yang-explorer
Then i can access http://localhost:8088/static/YangExplorer.html, also can create new account.
But i can't login with new account, neither with guest/guest.
The error is : server is not running

docker@default:~$ docker ps
CONTAINER ID        IMAGE                                 COMMAND                  CREATED             STATUS              PORTS                    NAMES
256d6cc92c08        robertcsapo/yang-explorer             "/bin/sh -c 'bash sta"   3 seconds ago       Up 2 seconds        <IP_ADDRESS>:8088->8088/tcp   yangexploer

This is my docker info:
docker@default:~$ docker images
REPOSITORY                                              TAG                 IMAGE ID            CREATED             SIZE
robertcsapo/yang-explorer                               latest              241c5a9e8b12        6 weeks ago         2.257 GB

docker@default:~$ docker ps
CONTAINER ID        IMAGE                                 COMMAND                  CREATED             STATUS              PORTS                    NAMES
256d6cc92c08        robertcsapo/yang-explorer             "/bin/sh -c 'bash sta"   3 seconds ago       Up 2 seconds        <IP_ADDRESS>:8088->8088/tcp   yangexploer

could you provide a screenshot of "server is not running" and docker logs --tail 20 yangexploer from your docker host?

change YangExplorer.html and start.sh to run with ip-address, also has same problem:
docker@default:~$ docker logs --tail 30 yangexploer

Activating virtualenv ..
Starting YangExplorer server ..
Use http://8baf4e7cb69f:8088/static/YangExplorer.html

Activating virtualenv ..
Starting YangExplorer server ..
Use http://<IP_ADDRESS>:8088/static/YangExplorer.html

[06/Jun/2017 08:16:37]"GET /static/YangExplorer.html HTTP/1.1" 200 6704
docker@default:~$

Are you sure traffic is routed to your container? Because you should se more HTTP requests.
I've followed your steps and it works for me.

docker run -d --name yangexploer -p 8088:8088 robertcsapo/yang-explorer
Firefox -> http://localhost:8088/static/YangExplorer.html

Logged in as guest/guest

docker logs --tail 20 yangexploer

Output:
Activating virtualenv ..
Starting YangExplorer server ..
 Then it's not in the same Docker Host.
Try if you can reach the container by it's ip-address
docker inspect yangexploer | grep -i ipaddress

@NickZengXL based on your earlier responses, I am assuming that chrome is running natively but docker is running in virtual box, if so can you indicate how many nics do you have in vbox vm. Also assuming you have just 1 Nic and that is doing NAT, you would need to add a port forwarding rule in virtual box so traffic to port 8088 is delivered into the guest.

Hi, i have the same issue, i can load the yang-explorer ui on:
http://<IP_ADDRESS>:8088/static/YangExplorer.html
However when trying to login it says: "Server is not running"
I built my own docker container for this (docker for mac), and also pulled robertcsapo/yang-explorer but both containers give me the same error message.
On the yang-explorer server, the only messages i see are:
Quit the server with CONTROL-C.
[17/Oct/2017 14:13:52]"GET /static/YangExplorer.html HTTP/1.1" 304 0
[17/Oct/2017 14:14:11]"GET /static/YangExplorer.html HTTP/1.1" 304 0
[17/Oct/2017 14:14:34] WARNING [django.request] Not Found: /favicon.ico
WARNING:django.request:Not Found: /favicon.ico
[17/Oct/2017 14:14:34]"GET /favicon.ico HTTP/1.1" 404 2546
[17/Oct/2017 14:14:34] WARNING [django.request] Not Found: /favicon.ico
WARNING:django.request:Not Found: /favicon.ico
[17/Oct/2017 14:14:34]"GET /favicon.ico HTTP/1.1" 404 2546
[17/Oct/2017 14:14:34] WARNING [django.request] Not Found: /favicon.ico
WARNING:django.request:Not Found: /favicon.ico
Much appreciate your help!

@sandstonewater could you try http://localhost:8088 ?
If that doesn't work. Please post full debug
like this
Activating virtualenv ..
Starting YangExplorer server ..
Use http://b7f4fe8eba04:8088/static/YangExplorer.html

Performing system checks...

System check identified no issues (0 silenced).
October 17, 2017 - 06:38:16
Django version 1.8.3, using settings 'server.settings'
Starting development server at http://b7f4fe8eba04:8088/
Quit the server with CONTROL-C.

Hi, thanks for your quick reponse.
I tried  http://localhost:8088 but still doesnt work, same error message.
When the webui loads up, there is always a spinning clock, it never changes to a pointer.
Below is the full debug.
root@9f31311605c0:~/yang-explorer# ./start.sh
Starting YangExplorer server ..
Use http://<IP_ADDRESS>:8088/static/YangExplorer.html
Performing system checks...
System check identified no issues (0 silenced).
October 17, 2017 - 20:59:38
Django version 1.8.3, using settings 'server.settings'
Starting development server at http://<IP_ADDRESS>:8088/
Quit the server with CONTROL-C.
[17/Oct/2017 21:00:55]"GET /static/YangExplorer.html HTTP/1.1" 200 6704
[17/Oct/2017 21:00:55]"GET /static/history/history.css HTTP/1.1" 304 0
[17/Oct/2017 21:00:55]"GET /static/swfobject.js HTTP/1.1" 304 0
[17/Oct/2017 21:00:55]"GET /static/history/history.js HTTP/1.1" 304 0
[17/Oct/2017 21:00:55] WARNING [django.request] Not Found: /favicon.ico
WARNING:django.request:Not Found: /favicon.ico
[17/Oct/2017 21:00:55]"GET /favicon.ico HTTP/1.1" 404 2546
[17/Oct/2017 21:00:56]"GET /static/YangExplorer000800.swf HTTP/1.1" 304 0
[17/Oct/2017 21:00:56] WARNING [django.request] Not Found: /favicon.ico
WARNING:django.request:Not Found: /favicon.ico
[17/Oct/2017 21:00:56]"GET /favicon.ico HTTP/1.1" 404 2546

@sandstonewater As your receiving HTTP Code 304, could you verify with another browser or clear cache?


waitForRequest doesn't capture request quick enough

I use puppeteer for running click tests on a web application. A couple of test cases require me to find out whether or not  a network request is sent to fetch a file from server. To do this I use puppeteer's waitForRequest method.
However the requests aren't captured consistently. On multiple occasions the command times out which results in a failed test. The chrome dev tools on the other hand capture the request while the test is running.
In verbose mode I get this feedback in console
I wait for request "http://localhost:4200/assets/img/cursor/zoom-in.cur"
[1] Error | TimeoutError: Timeout exceeded while waiting for event
Emitted | step.failed (I wait for request "http://localhost:4200/assets/img/cursor/zoom-in.cur")
Emitted | step.finish (I wait for request "http://localhost:4200/assets/img/cursor/zoom-in.cur")
[1] Error | TimeoutError: Timeout exceeded while waiting for event

This is the test scenario
I.click('.zoom');
I.waitForRequest("http://localhost:4200/assets/img/cursor/zoom-in.cur")
I.seeElement(".zoom-menu");

Details

CodeceptJS version: 2.02
NodeJS Version: 9.11.2
Operating System: Windows 10 (1809)
Configuration file:

exports.config = {
  tests: './codecept-tests/*.js',
  timeout: 10000,
  output: './output',
  helpers:
  {
    Puppeteer:
    {
      url: 'http://localhost',
      show: true,
      windowSize: '1280x720',
      chrome:
      {
        defaultViewport: { width: 1280, height: 720 }
      }
    }
  },
  include: {},
  bootstrap: false,
  mocha: {},
  name: 'frontend',
  plugins: { screenshotOnFail: { enabled: true } }
}

In my case, the problem was that the Application was firing the Request before the waitForRequest took place. So it never catched the Request.
That is because Puppeteer waits 100ms (by Default) after actions like click, doubleClick or PressKey.
Puppeteer Config waitForAction
To solve the issue, I just had to set this waitForAction parameter to 0:
exports.config = {     tests: './integration/*_test.js',     output: './integration/output',     helpers: {         Puppeteer: {             url: 'http://localhost:3040',             restart: false,             show: true,             windowSize: '1280x960',             waitForAction: 0,             chrome: {                 defaultViewport: {                     width: 1280,                     height: 800,                 },             },         },         MockRequest: {},     },     include: {},     bootstrap: null,     mocha: {         reporterOptions: {             reportDir: './integration/output',         },     }, };

fwiw I do like:
await Promise.all([
  I.click('.zoom'),
  I.waitForRequest("http://localhost:4200/assets/img/cursor/zoom-in.cur"),
])

I have the same issue

if it still actual for you I solved this problem<EMAIL_ADDRESS>

It looks like your config for codeceptjs has a key waitForNavigation: 'networkidle0
exports.config = {
  tests: 'tests/*_test.js',
  output: './output',
  helpers: {
    Puppeteer: {
      url: process.env.CODECEPTJS_URL,
      show: false,
      waitForNavigation: 'networkidle0', // <-------- this adds waiting for network activity to end before executing, for example "click" action
      windowSize: '1200x900',
      chrome: {
        args: ['--no-sandbox'],
      },
    },
  },

and this blocks of execution for the following sumaltenous steps:
  await Promise.all([
    I.waitForRequest(some_url_here),
    I.click(refreshButton),
  ]);

because if we look at source code of click step for codeceptjs https://github.com/codeceptjs/CodeceptJS/blob/7df680264b6b0d079651d7cabb818926e53d494b/lib/helper/Puppeteer.js#L2208

then we will see that options.waitForNavigation will be put on the promise list.
If the step before clicking causes any network activity, then the click will wait for the network to end if you specified options.waitForNavigation
SOLUTION:

Don't use waitForNavigation option for Puppeteer helper. Only use waitForNavigation inside your scenario and your live will be better! :). Because global setting for waiting navigation will break some of your scenarios without any information about this fact.

Thanks a bunch! The solutions proposed by Meekohi and aelzhanova worked for my use case!
Sorry for the late reply


Having an issue with server-side rendering...

I'm using express, and I want to do some basic routing server-side.  At the moment I'm just trying to set up the router to do anything.  Right now it always returns a 404.
I believe I've followed the directions listed on https://github.com/rackt/react-router/blob/master/docs/guides/advanced/ServerRendering.md.
It's about as basic as you can get: an express server that routes every request to be handled by the code from the link listed above, and a ReactRouter that routes everything to the AppTemplate component.
It's definitely getting to the callback in routes.js, because it returns 'Not found' for every request.
I doubt it matters, but I'm running it in IIS via iisnode.  I've been having troubles debugging, which is one of the reasons I'm switching from express-react-views to routing via the <Router>.
Let me know what other information i can get you.
File structure:
server/
-- server.js // just calls babel-register and express-app.js
-- express-app.js
-- router.js

server/views/
-- app-template.js
-- routes.js

server/express-app.js
import Express from 'express';
import BodyParser from 'body-parser';
import CookieParser from 'cookie-parser';

let app = new Express();
export default app;

app.enable('trust proxy');

app.use('/public', Express.static('../dist'));

app.use(BodyParser.urlencoded({ extended: true }));
app.use(BodyParser.json());
app.use(CookieParser());

// some rest API code here, currently commented out

app.set('tokenSecret', process.env.tokenSecret);

require('./router');

server/router.js
// copied right from https://github.com/rackt/react-router/blob/master/docs/guides/advanced/ServerRendering.md.
import { renderToString } from 'react-dom/server';
import { match, RouterContext} from 'react-router';

import app from './express-app';
import Routes from './views/routes';

app.get('/*', (req, res) => {
    match({ Routes, location: req.originalUrl }, (error, redirectLocation, renderProps) => {
        if (error) {
            res.status(500).send(error.message);
        } else if (redirectLocation) {
            res.redirect(302, redirectLocation.pathname + redirectLocation.search);
        } else if (renderProps) {
            res.status(200).send(renderToString(<RouterContext {...renderProps} />));
        } else {
            res.status(404).send('Not found');
        }
    });
});

server/views/routes.js
import React from 'react';
import {Router, Route, IndexRoute} from 'react-router';

import AppTemplate from './app-template';

export default (
    <Router>
        <Route path='/' component={AppTemplate}>
        </Route>
    </Router>
);

server/views/app-template.js
import React from 'react';

export default class AppTemplate extends React.Component {
    constructor() {
        super();
    }

    render() {
        return (
            <html>
                <head>
                </head>
                <body>
                    <div id='app' />
                </body>
            </html>
        );
    }
};

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.

Reposted at http://stackoverflow.com/questions/35120863/server-side-rendering-with-react-router-is-not-working


Add support for extern variable declarations

The following declarations are not currently supported.
extern int forty_two;
extern enum { SYSTEM, USER } who;
extern const char version[];
extern struct {
    int major;
    int minor;
    int patch;
} semver;

#62 has related code

OK, I will review it tomorrow.

I assume you mean it has related code for the IR representation?

#62 adds VisitVarDecl method that also should be used here
So it will be more convenient to wait until the pr is merged

Partially fixed in #80 with limitations due to scala-native/scala-native#202


httpactioncontext identity name is null for msal 2.0 authentication for code flow

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description

Source

[ ] Internal (Microsoft)
[ ] Customer request

var isAuthorized = base.IsAuthorized(actionContext);
It always returns false while doing SPA authorization without implicit code flow (MSAL).
With adal library it was returning true.
It was unable to fetch the user in the principal.identity.

Is there some other configuration to set in AzureAD or frontend angular 11?
It is adding the access token in the header of API call but while proceeding to backend it fails the authentication as the var isAuthorized = base.IsAuthorized(actionContext) is false.

@RomaRathi This code looks like ASP.net backend, can you please give us more details about your use case? We would need:

Sample code
Expected scenario
Steps to reproduce

The description above is not enough for us to parse the problem.

Frontend code to append token.docx
@sameerag

Resolved.
By setting the correct audience.


keyboard short cut to move focus/editing to a new cell

Can we allow programmers to move the  focus/editing from one cell to another using keyboard shortcuts on top of mouse operations?
In Jupyter, it was easy, just quit Edit mode into Command mode and then use up and down arrow keys to move between cells. Maybe we should simply use up, down, left, and right arrows. But we need to first  enable Edit and Command modes (#88) .

@pionxzh

The hopping between cells via keyboard operations has been done in https://github.com/codepod-io/codepod/pull/327
But we still need the keyboard shortcut to switch between edit mode and command mode. https://github.com/codepod-io/codepod/issues/88

Please refer PR #398.


Dexter seems to fail when loading a customer application (UWM,   MuleSoft Application)

Describe the bug
While doing a SaaS controller migration planning exercise for United Wholesale Mortgage,   we took configuration and metadata dumps for a non-production and production controller.
Non-production DEXTER extracts worked without issue
Production repeatedly failed,  stopping in the middle of dumping an app in the middle of their applications at an app called "Mulesoft".
Dexter throws and error,  then stops processing,  and stops generating CSV files.
We have extracted the application configuration using ConfigExporter,  hoping that the problem will replicate in another controller after loading.   (JSON attached)
To Reproduce
Steps to reproduce the behavior:

Load this Mulesoft.JSON
Adapt the UnitedShoreJob file to fit your controller
Run the DEXTER job to attempt to load Mulesoft
See error

Expected behavior
DEXTER stops generating output files and cancels remaining job steps including report generation.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
.NET Framework Type (Full/Core)
.NET/.NET Core Framework Version

Logs
Add logs from the program execution https://github.com/Appdynamics/AppDynamics.DEXTER/wiki/Log-Files
Reports
Include any produced reports that you have issues with

MuleSoft.json.txt
and a fragment of a ConsoleLog showing the failure happening and stopping the run.

2020-11-04 11:21:01.8938|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|IndexAPMMetricsList(86): [46/108], https://appd.uwm.com MISMO duration 00:00:00.3813606 (381 ms)
2020-11-04 11:21:01.9084|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|IndexAPMMetricsList(86): [47/108], https://appd.uwm.com MuleSoft(37) [APM]
2020-11-04 11:21:01.9515|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Parse metrics
2020-11-04 11:21:01.9515|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix INFRA with depth 14
2020-11-04 11:21:01.9978|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix INFRA with depth 14 produced 7948 metrics
2020-11-04 11:21:01.9978|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix APP with depth 12
2020-11-04 11:21:01.9978|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix APP with depth 12 produced 1478 metrics
2020-11-04 11:21:01.9978|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix BACKEND with depth 6
2020-11-04 11:21:01.9978|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix BACKEND with depth 6 produced 3 metrics
2020-11-04 11:21:01.9978|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix BT with depth 22
2020-11-04 11:21:02.0760|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix BT with depth 22 produced 5548 metrics
2020-11-04 11:21:02.0760|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix SEP with depth 5
2020-11-04 11:21:02.0760|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix SEP with depth 5 produced 300 metrics
2020-11-04 11:21:02.0760|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix ERR with depth 7
2020-11-04 11:21:02.1026|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix ERR with depth 7 produced 1240 metrics
2020-11-04 11:21:02.1026|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix IP with depth 6
2020-11-04 11:21:02.1070|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix IP with depth 6 produced 0 metrics
2020-11-04 11:21:02.1070|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix WEB with depth 6
2020-11-04 11:21:02.1070|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix WEB with depth 6 produced 0 metrics
2020-11-04 11:21:02.1070|Info|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix MOBILE with depth 8
2020-11-04 11:21:02.1070|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Prefix MOBILE with depth 8 produced 0 metrics
2020-11-04 11:21:02.1244|Warn|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|System.ArgumentNullException: Value cannot be null. (Parameter 'key')
   at System.Collections.Generic.Dictionary`2.FindEntry(TKey key)
   at System.Collections.Generic.Dictionary`2.TryGetValue(TKey key, TValue& value)
   at AppDynamics.Dexter.ProcessingSteps.IndexAPMMetricsList.Execute(ProgramOptions programOptions, JobConfiguration jobConfiguration) in C:\Users\bhjel\RiderProjects\AppDynamics.DEXTER\ProcessingSteps\Index\IndexAPMMetricsList.cs:line 855
2020-11-04 11:21:02.1244|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|IndexAPMMetricsList(86): [47/108], https://appd.uwm.com MuleSoft duration 00:00:00.2201920 (220 ms)
2020-11-04 11:21:02.1443|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|IndexAPMMetricsList(86): total duration 00:03:15.5740044 (195574 ms)
2020-11-04 11:21:02.1581|Warn|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|If you need support, please review https://github.com/Appdynamics/AppDynamics.DEXTER/wiki#getting-support and send the logs
2020-11-04 11:21:02.1581|Trace|AppDynamics.Dexter|2964|1|AppDynamics.Dexter.Console|Application execution took 03:18:44.6211793 (11924621 ms)

In hindsight,  the customer application configuration may contain sensitive info.   Sending it straight to AppD (BradH) for debugging support.

I thought @bradhjelmar fixed that already. This is for the metrics lookup when something in the entity name is null.


You cant vote for vacants

In app.js the amount you can vote for i set on votesCount=Array.apply(null, Array(optionsarr.length)).map(function (x, i) { return 0; }); and should have vacoptions length too.

Resolved in https://github.com/cthit/VoteIT/pull/25


Illegal option -o pipefail

Hi,
When I run ./install.sh
I got this error:
./install.sh: 3: set: Illegal option -o pipefail

Should the #! point to bash? Or should the pipefail be removed?


CDI-550 expand the sentence little bit to be accurate.

Looks good.

ok for me. @tremes could you rebase on 2.0.EDR2 to have a clean Pr closing ? thx.

@antoinesd done


Fix show/hide of FAB when navigating to a different screen

The animation to show/hide FAB upon navigating to a different screen looks like it needs some work to make it look smooth.

Note: Will add more details soon.

Hey @waseefakhtar is this issue solved?
If not please assign this issue to me.

is help still needed for this issue?


Rewrite dns instructions reuslt in error

plugin/kubernetes: /etc/coredns/custom/ipa1.server:7 - Error during parsing: unknown property 'upstream'

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

ID: 2f95537e-770a-739c-0175-4447a995f8b2
Version Independent ID: 6c9a1132-05af-70f1-5a2b-8dad56de9cc3
Content: Customize CoreDNS for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/coredns-custom.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: jpalma

@jstewart612 -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

Apologies for the delay.
upstream in coredns configuration is deprecated and generates error in 1.18.7 coredns versions in AKS.
There is another issue reporting the same and the doc author is working on the related task to update the document.
CC: @zr-msft
Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Error with PSKeyword

I am trying to open this PDF file https://akademiasmaku.pl/przepis/pobierz-pdf/772
The code is:
content = requests.get('https://akademiasmaku.pl/przepis/pobierz-pdf/772', allow_redirects=True, verify=False, timeout=60).content
stream = BytesIO(content)
parser = PDFParser(stream)
document = PDFDocument(parser, '')
rsrcmgr = PDFResourceManager()
device = PDFDevice(rsrcmgr)
interpreter = PDFPageInterpreter(rsrcmgr, device)
for page in PDFPage.create_pages(document):
    interpreter.process_page(page)

In Python 2.7 I get this error:
Traceback (most recent call last):
  File "lpdfx/backends.py", line 125, in __init__
    interpreter.process_page(page)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfinterp.py", line 839, in process_page
    self.render_contents(page.resources, page.contents, ctm=ctm)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfinterp.py", line 852, in render_contents
    self.execute(list_value(streams))
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfinterp.py", line 877, in execute
    func(*args)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfinterp.py", line 760, in do_TJ
    self.device.render_string(self.textstate, seq)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfdevice.py", line 82, in render_string
    scaling, charspace, wordspace, rise, dxscale)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdfdevice.py", line 94, in render_string_horizontal
    for cid in font.decode(obj):
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/pdffont.py", line 714, in decode
    return self.cmap.decode(bytes)
  File ".venv2/local/lib/python2.7/site-packages/pdfminer/cmapdb.py", line 127, in decode
    if len(code) % 2 != 0:
TypeError: object of type 'PSKeyword' has no len()

I have tried to Google the error or look at pdfminer3k, but there are no fixes. How should the length of PSKeyword be calculated?

yeah same error!!  couldn't figure out the issue

Formatting issue with PDF!!

I am getting the same error, can anyone please help?


Example category title translation

I can't quite remember if we mention this before but the examples' category title because it seems to be parsed from the folder name, they have not been translated.

To make them translatable, can we either include them in the general UI translation file or under each category's folder have a file that provides the title translation (YAML or otherwise)?

They are already in the English UI Yaml and ready for translation.


Merge remote-tracking branch 'upstream/master' into product-scenarios-4.0.0 branch

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 8 committers have signed the CLA.:white_check_mark: CrowleyRajapakse:white_check_mark: YasasRangika:white_check_mark: ruks:white_check_mark: prasa7:white_check_mark: mushthaq33:white_check_mark: chamilaadhi:white_check_mark: tharindu1st:x: wso2-jenkins-botYou have signed the CLA already but the status is still pending? Let us recheck it.


Allow Multiple ports to be routed dynamically

Hi @tpbowden
Ingress router is awesome for HTTP and HTTPS traffic n standard ports, It will be even better if we can allow multiple non standard ports to be routed through this, instead of just 80 and 443
Multiple ports for single domain is addon feature :)
Hope you consider this for future

I'm not sure how this would work as for this to happen it would require the Dockerfile to be updated with extra ports under PUBLISH, the internal web server to listen on these ports and serve the correct protocol (which could be anything on non-standard ports but I assume you just mean HTTP) and also for these ports to be configured somewhere, probably via the CLI at startup.
It might be easier for you to just fork the project and make the required changes including the ports you want to use as I don't think its possible to allow arbitrary ports to be configured. Something like Nginx or HAProxy is a more suitable tool for this I think. Have you looked at the jwilder/nginx project?
I am open to suggestions if you have an idea of how it could work however!

Having thought about this it does not fit in with the way Docker works - going to stick with the default ports only being exposed for now.


verify_ssl handled wrongly in CLI mode

Due to the new parameter --no-check-certificate, verify_ssl is now False by default in CLI mode.
In contrast, if I pass the parameter, it becomes True, so exactly the opposite of what is intended.

Fixed in 3.7.1


Login with Linkedin is not available

Describe the bug
It's probably caused by this.
In the latest version of the code in NPM there's no Linkedin social provider. It can also be checked directly in NPM here.
Your Example Website or App
placeholder just to make the submission form happy
Steps to Reproduce the Bug or Issue
It's simply not there.
Expected behavior
I expect the source code in Github to be aligned with what is uploaded in NPM
Screenshots or Videos

Platform

OS: All
Browser: All
Version: 2.0.6

Additional context
No response

This is probably caused by both:

https://github.com/Lzok/remix-auth-linkedin/issues/3
https://github.com/Lzok/remix-auth-linkedin/issues/4

Any workaround to make LinkedIn auth work?

Hello there! I just published the version v2.0.1 for the remix-auth-linkedin package. Can you please confirm if that works well for you?
I will be paying atention to this.
Thank you!

Sorry for the delay on this!
V2.1.0 just released which tidies up the last of these missing links, so everything should be aligned now! 🎉


fix!: allow awaiting custom page navigation

Description
As documented by #43, we cannot currently await a navigate call to a DuckPage. This is an issue caused by the fact we do not provide onPopInvoked to the Page that DuckPage overrides. As a result, we do not call onPop in our delegate and never get here: https://github.com/JaspervanRiet/duck_router/blob/5390c2bbb9cdb27bcd5bbd1531ed2d874706e797/duck_router/lib/src/configuration.dart#L87
Solving this can use a number of approaches. My approach here has been very similar to the solution in #31: we want to hide most of this complexity. That means not exposing the popping interface to the user. We can do this by constructing a Page object builder in the exact same way that MaterialPage and CupertinoPage are created. This PR does that by no longer inheriting from Page in DuckPage. That provides a number of advantages:

We can use the builder pattern described above
It allows us to separate our data model from Page. #31 already hit a number of inflexibilities, now we can just construct our own model

The only real issue encountered is that we still need to provide the RouteSettings in createRoute to allow construction of certain routes. That means a small tweak to implementations. Thankfully it's not a functionality change, it's just syntax.
Related Issues
#43
Migration guide
Migration is trivial, since most of the complexity is still hidden:
BEFORE
class CustomPage<T> extends DuckPage<T> {
  // ...

  @override
  Route<T> createRoute(BuildContext context) {
    return MaterialPageRoute<T>(
      settings: this,
      builder: (context) => const CustomScreen(),
    );
  }
}

AFTER
class CustomPage<T> extends DuckPage<T> {
  // ...

  @override
  Route<T> createRoute(BuildContext context, RouteSettings? settings) {
    return MaterialPageRoute<T>(
      settings: settings,
      builder: (context) => const CustomScreen(),
    );
  }
}

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

We can actually do better! We can remove the name parameter altogether. You were right to call out the key, I took a look at the flutter source and we should use it. However, I'm not seeing the benefit of providing them all the way to the user. Have a look at my changes.
For arguments: the user never has access to the RouteSettings, so what would the value be? I might be missing something there.


Decide when and how to remove things that were deprecated in the SDK

A bunch of stuff I can find that is deprecated. In some cases the doc comments or annotations lie and say the element will be removed in Dart 2. Some of these have been deprecated for years.

@proxy
DeferredLibrary
MirrorsUsed
Provisional
NoSuchMethod default constructor
onError argument to parse (I think this one was intentionally left in for Dart 2)
kInvalidParams and similar in ServiceExtensionResponse

Most of this is likely unsafe to remove in Dart 2 without potentially breaking code. Should we update comments to say things will be removed in Dart 3? Do we have a place to track that they do get removed for Dart 3?
This is in additional to constants in dart:io which were called out as something worth discussing separately from other constants: #34218

Most of these are intentionally left in for Dart 2 (I think DeferredLibrary was the only one we believed we could remove, we just didn't want to take a chance late in the process).
I would want to remove them in Dart 3.0 (or when they have been deprecate for at least some amount of time, like six months, but Dart 3.0 probably won't happen before that).

Another thing we should consider deprecating soon and then deleting for Dart 3.0 is dart:web_sql

This was removed a little while back.

If possible it would be good if we could analyze pub.dev for usages of these apis before removing them, there is a thread on the dart-lang gitter because somebody got broken by a usage of MirrorsUsed in package:mustache. I filed an issue for them to drop the usage, but it would be better if we could do that prior to actually removing the APIs.


msi: add migration test

@kenhys We can test #533 by uncommenting https://github.com/fluent/fluent-package-builder/pull/537/files#diff-3c3d86e4105a9661b33356e55eded494b6e4bacc71a16a5e4ce2bb0e712668c9R42-R49

@kenhys We can test #533 by uncommenting https://github.com/fluent/fluent-package-builder/pull/537/files#diff-3c3d86e4105a9661b33356e55eded494b6e4bacc71a16a5e4ce2bb0e712668c9R42-R49

Oh, sorry, this may be wrong.
#533 will convert the baskslash to slash.
It has nothing to do with this test.
We should uncomment this test in another PR.

I made #542 instead of this.


basics for disputing test

https://app.clubhouse.io/augur/story/12678/create-basic-disputing-integration-test

I assume they work on your system, can you pull down my changes and confirm?
I get 2 tests that fail:
✕ should not be able to submit a dispute without REP (80012ms)
✕ should be able to dispute on all outcomes (2600ms)


Fix empty invite-only streams shown broken on subs page.

We now mark invite-only streams where the last user left as "deactivated" in the backend.
Now an invite-only stream that has no users left will be effectively deleted, and will not appear in the list of streams, even for an administrator (and thus one won't see the broken view as mentioned in the issue).
Fixes #3657

@khantaalaman thanks for keeping working on this!  After reading some code, I think I've identified what the actual fix for this should be, which is a bit different from what you add.  See the comment above and notes for how to add a second test that we need.
I think this is going to turn into one of those PRs with like 50-100 lines of test code and like 2 lines of changes :), but that sort of change is super important for ensuring we produce correct results.

@khantaalaman ping on fixing this; I'd love to have this issue resolved...

@khantaalaman Are you working on this?

@harshitongithub It would be great if you can take this. I am unable to complete my tasks due to some personal unforeseen reasons.

@khantaalaman okay no problem! I will try to finish it. :+1:

@HarshitOnGitHub I've added you as an assignee to the issue.

Closing due to inactivity; hopefully someone else will pick this up.


It never closes the files it opens

After using getFile few times I get:
EMFILE: too many open files, open '/etc/hosts'

It's not possible to debug this issue without additional information. If this is still an issue, please leave a comment or open a new issue. Also, feel free to submit a PR that fixes your issue and I'll take a look. Thanks.


Identify new records with previous update

Workflow defined in #16.

Currently this is only working for OXCGRT records. Closing here.


Refactor tests dst indentation

Fixes #1302 for tests in "pkg/server" and "internal"
Description

Now that openfga/language parser has better support for whitespaces, refactor to
have better readability in the test files.
References

Issue #1302
Review Checklist

[X] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[X] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Hi @adriantam / Team,
this PR remains in draft as I'm not sure how to fix the benchmark test (list_users.go) failing on line 186.
https://github.com/openfga/openfga/blob/c249beadaa519ca34fdb4b1ce81e1787c512c5a4/pkg/server/test/list_users.go#L180-L186
Reproduced locally using upstream/main branch, so I'm assuming it will be spot/fixed on main in the short term. As soon as I see a fix I will merge it and move this PR back to Review Ready state

Thanks for the PR @00chorch !
Someone from the team will be looking at this shortly


Improve current state of tests

Possible improvements:

do a review of all tests
get a better example model than the XXZ Hamiltonian (which is used for many tests, where a RBasis is needed); also the fact that the XXZ Hamiltonian has discrete magnetization plateaus and is very sensitive to the degenerate snapshot at $(\Delta, h/J) = (-1, 0)$ makes this model problematic at larger $L$
write tests for untested code parts
design an artificial parametrized eigenvalue problem, where we know the exact solution (non related to quantum spin systems)

In addition:

tests for resumed simulations
tests for the various truncation methods
consider the FullDiagonalization issues (see TODO)


importing models / repo name

The current repo name, gpkit-models is not a valid python package name. Is this an issue? How does one import a model or for that matter, install gpkit-models?

[ ] do we need an install script?
[ ] should we move all models into a models dir that is a python package with an __init__, or
[ ] should we rename the repo?

I think we'll want an install script if we use markdown files.
I do think the name should be gpmodels or gpkitmodels or gpkit_models.


Stuck on Building spark session...

Hello
I'm trying to build up a spark on the lambda framework on my lambda function, but I'm facing some problem, when I test local, it is always success, but when I upload and run it in the lambda function always stuck on **Building spark session...**and then finally always got a timeout issue, cos this don't have any error message, don't know how to solve it
I don't know how to do it and what's the config that I'm probably missing, please help to check ~~

@vivi609kfes Are you running the Iceberg Script? what is the size of the file ? Have you also tried increasing the time out in AWS Lambda just to get the error message?

@JohnChe88
I'm running a glue sample script(I don't use any glue library, just use lambda), and I just directly catch the event and then use AWS Lambda spark to clean it.
The weird thing is I can run it successfully locally (runtime less then 5 mins ), but then when I package to a container and upload to AWS Lambda went timeout (and no error message that I can try to check the issue)
thank you so much for such a quick reply !!
event_size : JSON file, about 200KB.
AWS Lambda : timeout = 5min
details about lambda config and timeout message:

@JohnChe88
Thank you so much for such a quick reply, I checked with my AWS Lambda resource again and increased the resource about my AWS Lambda, and then it worked!!!!
thanks again for your answer !!

Sure, @vivi609kfes  feel free to contribute and let us know if you are in Dev or PROD


Why store Token in Cookie and send it in Authorization Bearer Header?

Why store Tooken in Cookie, after that send token over Authorization Bearer Header?
It sends duplicate Token in Header: Authorization and Cookie when the domain of FE and BE is the same.

Yes, this is done just to be compatible with both ways, just a defensive programming


refactor(matcher): add error message for v3 catchall

Got an error Route "*" should be "/*". when I set path: '*'.
However it still doesn't work (I didn't notice new catchall rule).
So added new error message for catchall.

/* is valid?
I got error Route "*" should be "/*", so I changed '*' to /*. then got warn [Vue Router warn]: No match found for location with path. CatchAll doesn't work.

/* will match the * character, it no longer has a special meaning unless put after a param

Ahhhh, I finally understand. Thank you.
I'll fix it to show warning instead of error.

I'm going to take over this one because I want to release a new version 🙂
Thank you!

OK! Thank you!

Merged at 14e1eb96485f74669f582a87f522d3b13b567c9c
I couldn't push to the branch, maybe you unticked the option on the right that allows maintainers to edit 🤔 I still kept you as the author though!


Process entire stack subtree when syncing stack

When running av stack sync, sync the entire "stack" (starting from root) instead of just the subtree from the current branch.
Updated test to document new behavior.

Isn't this effectively behave same as always specifying --all as av doesn't support non-default (== branches other than refs/remotes/origin/HEAD) trunk branch?

It's different in my testing, since av stack sync will only sync the current stack from the root node - not from other root nodes going off the trunk branch.
Consider the following set up - I have two stacks I made off main:

main -> add-foo-txt-1 -> add-bar-txt-1
main -> add-foo-txt-2 -> add-bar-txt-2

Here's what av stack tree reports:
➜  test-stack git:(add-bar-txt-2) av stack tree
  * add-bar-txt-2 (HEAD)
  │ https://github.com/oleg-codaio/test-stack/pull/28
  │
  * add-foo-txt-2
  │ https://github.com/oleg-codaio/test-stack/pull/27
  │
  │ * add-bar-txt-1
  │ │ https://github.com/oleg-codaio/test-stack/pull/26
  │ │
  │ * add-foo-txt-1
  │ │ https://github.com/oleg-codaio/test-stack/pull/25
  ├─┘
  * main

If I ran av st sync:
➜  test-stack git:(add-bar-txt-2) av st sync
Synchronizing branch add-foo-txt-2...
  - fetching latest pull request information for add-foo-txt-2
  - branch is a stack root, nothing to do (run av stack sync --trunk to sync against the latest commit in main)
  - not pushing branch add-foo-txt-2 (upstream is already up-to-date)

Synchronizing branch add-bar-txt-2...
  - fetching latest pull request information for add-bar-txt-2
  - already up-to-date with parent add-foo-txt-2
  - not pushing branch add-bar-txt-2 (upstream is already up-to-date)

it only syncs the current stack. On the other hand, if I run av st sync --all, it syncs all my stacks:
➜  test-stack git:(add-bar-txt-2) av st sync --all
Synchronizing branch add-foo-txt-1...
  - fetching latest pull request information for add-foo-txt-1
  - branch is a stack root, nothing to do (run av stack sync --trunk to sync against the latest commit in main)
  - not pushing branch add-foo-txt-1 (upstream is already up-to-date)

Synchronizing branch add-bar-txt-1...
  - fetching latest pull request information for add-bar-txt-1
  - already up-to-date with parent add-foo-txt-1
  - not pushing branch add-bar-txt-1 (upstream is already up-to-date)

Synchronizing branch add-foo-txt-2...
  - fetching latest pull request information for add-foo-txt-2
  - branch is a stack root, nothing to do (run av stack sync --trunk to sync against the latest commit in main)
  - not pushing branch add-foo-txt-2 (upstream is already up-to-date)

Synchronizing branch add-bar-txt-2...
  - fetching latest pull request information for add-bar-txt-2
  - already up-to-date with parent add-foo-txt-2
  - not pushing branch add-bar-txt-2 (upstream is already up-to-date)

Oh. OK. I see. So, sync will target:

Current behavior: branches between trunk and current branch + current branch + children
--all behavior: all branches
New behavior: children of the stack root (the branch right next to trunk)

Yup, exactly. It will now include all the "cousins" of the current branch in its stack.


Especify custom executable for Edge.

Hi, could it be possible to especify a custom path for the Edge launcher? I would like to use a portable version of Edge.
Regards

@codepo8, what do you think about pulling this into our backlog? We could probably do something like add customEdgePath to settings json so we don't need to design new UI.

Sounds good to me.

@oxk4r01 this should now be possible using the launch.json flow for the extension. You can have the extension create a launch.json for you and customize it to add a runtimeExecutable entry with the path you want to use:

        {
            "type": "pwa-msedge",
            "name": "Launch Microsoft Edge in headless mode",
            "request": "launch",
+           "runtimeExecutable": "path/to/edge",
            "runtimeArgs": [
                "--headless",
                "--remote-debugging-port=9222"
            ],
            "url": "http://localhost:3002", // Provide your project's url to finish configuring
            "presentation": {
                "hidden": true
            }
        },


Not possible to print 64 bit decimal values with minimal libc

It is not possible to print 64-bit integer value using various print routines using either %llu or %lld in minimal libc.
The code in lib/libc/minimal/source/stdout/prf.c says that it only supports small subset of modifiers and "long long int" is not one of them.
I am asking this as in gPTP networking code there are lot of 64-bit values, and currently it is not possible to get those printed either in shell or in logs which is quite annoying when debugging things.

I am asking this as in gPTP networking code there are lot of 64-bit values, and currently it is not possible to get those printed either in shell or in logs which is quite annoying when debugging things.

Can you enable newlib as a workaround while you debug?

Sure, I have been doing just that. But it is not just debug prints that are affected but also net-shell prints status etc information using %llu and %lld. It would be nice to get this working also with minimal libc so that we do not add this kind of artifical dependency to newlib. Anyway, if we are deprecating minimal libc in the future, I think this can be considered low priority.

This was resolved for minimal libc when cbprintf became the standard formatter in Zephyr.


Adding support for fallback domains

Adds support for fallback domains: https://api.cloudflare.com/#devices-get-local-domain-fallback-list
closes #759
Has your change been tested?
Unit tests included. Will do manual testing with cloudflare-terraform
Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

thanks @kjhandy, this is awesome 👏


Updates to the HA document

Changes
Enhance enabling-ha.md doc

This document was missing the data.buckets configurable parameter for the leader election.
Remove references to data.resourceLock for HA config
While enhancing the HA document, we realized that the resourceLock field is not configurable in the Knative side, but rather hardcoded. You can see in https://github.com/knative/pkg/blob/master/leaderelection/config.go#L74-L87
, where the Config object is defined, that this field is not included. Also, the value for it seems to be hardcoded to "leases", as seen in https://github.com/knative/pkg/blob/master/leaderelection/config.go#L38

In addition, categorize the changes you're making using the "/kind" Prow command, example:
/kind documentation
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

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

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Update enabling-ha.md document, to highlight the usage of buckets for the HA solution in the pipeline controller.

@afrittoli fyi

/ok-to-test

@qu1queee it looks like you dropped the release part from the PR template, I added it back with no release notes, feel free to change it if you want to provide release notes

/approve

@afrittoli thanks, I did add a sentence for the release notes.

@jerop make sense, done

/lgtm


Updated raspberry.sh installer script to save apt-get log output to a file

Updated raspberry.sh installer script to save apt-get log output to a file that can be tailed
Fixed an issue where the raspberry.sh installer script can get stuck when there are packages that need to be upgraded

Please add the changelog entries to the 2.11 release. The 2.10 release is already published. Thanks!

why the separate log entry?  the  update=$(sudo apt get ... 2>&1)  captures all the stdout and error content,
and then the echo $update >>$logfile should save it to the existing log

i added the --assume-yes to the upgrade.. thanks

this fix (--update=yes) is in the scripts repo

Closed since we are moving this script.


Add parameter configuration functions

Now it should be possible to configure the inverter when the checksum function is known. See #4

The checksum function is different. See reopened #4


Make CacheLocation an enum

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Browser:

[ ] Chrome version XX
[ ] Firefox version XX
[ ] IE version XX
[ ] Edge version XX
[ ] Safari version XX

Library version

Library version: X.Y.Z

Current behavior

when setting cacheLocation with typescript the user becomes victim to type widening so something like this is required
  const config = {
    auth: {
      authority: (process.env.REACT_APP_AUTHORITY as string),
      clientId: (process.env.REACT_APP_AAD_APP_CLIENT_ID as string),
    },
    cache: {
      cacheLocation: ('localStorage' as CacheLocation),
      storeAuthStateInCookie: true,
    },
  };

Expected behavior

I think it would be a bit clearer if CacheLocation was an enum so that may be used directly without type widening
enum CacheStorage {
  LocalStorage = "localStorage",
  SessionStorage = "sessionStorage"
}
........
  const config = {
    auth: {
      authority: (process.env.REACT_APP_AUTHORITY as string),
      clientId: (process.env.REACT_APP_AAD_APP_CLIENT_ID as string),
    },
    cache: {
      cacheLocation: CacheStorage.LocalStorage,
      storeAuthStateInCookie: true,
    },
  };

See discussion here: https://github.com/syncweek-react-aad/react-aad/issues/95#issuecomment-513383239
Minimal reproduction of the problem with instructions

Sure. @jodonnell Are you open to submitting a PR for the same?

https://github.com/AzureAD/microsoft-authentication-library-for-js/pull/851
i was only sort of able to test this in context so i'm not 100% confident it works like i want it to :'(

Merged this to dev, @jodonnell Please let us know if you need anything else in this context.


ucontext_t is missing on i686-unknown-freebsd

wasmer-vm and wasmtime-runtime need ucontext_t + mcontext_t for PC and some other registers. Downstream (in ports/) this affects lapce,  replibyte veloren, zellij on i386. For example,
error[E0412]: cannot find type `ucontext_t` in crate `libc`
   --> cargo-crates/wasmer-vm-2.3.0/src/trap/traphandlers.rs:219:57
    |
219 |             let ucontext = &mut *(context as *mut libc::ucontext_t);
    |                                                         ^^^^^^^^^^ not found in `libc`

error[E0412]: cannot find type `ucontext_t` in crate `libc`
   --> cargo-crates/wasmer-vm-2.3.0/src/trap/traphandlers.rs:259:45
    |
259 |         unsafe fn get_pc_sp(context: &libc::ucontext_t) -> (usize, usize) {
    |                                             ^^^^^^^^^^ not found in `libc`

error[E0412]: cannot find type `ucontext_t` in crate `libc`
   --> cargo-crates/wasmer-vm-2.3.0/src/trap/traphandlers.rs:314:54
    |
314 |         unsafe fn update_context(context: &mut libc::ucontext_t, regs: TrapHandlerRegs) {
    |                                                      ^^^^^^^^^^ not found in `libc`

Sorry, I can't provide a synthetic example.

Note that it's usual that the crate doesn't have something, i.e. it's not a bug. Please use an API request template next time.