HDDS-10404. Ozone admin reconfig command fails with security enabled

What changes were proposed in this pull request?
Fix the issue that "ozone admin reconfig" CLI fails in a secure cluster.
Before the patch, reconfig will fail. Please go to the JIRA page to find the detail failure stack.
After the patch,  rconfig will succeed. Here is the command output
bash-4.2$ ozone admin reconfig --address=om:9862 --service=OM start
OM: Started reconfiguration task on node [om:9862].
bash-4.2$ ozone admin reconfig --address=scm:9860 --service=SCM start
SCM: Started reconfiguration task on node [scm:9860].
bash-4.2$ ozone admin reconfig --address=datanode:19864 --service=DATANODE start
DN: Started reconfiguration task on node [datanode:19864].

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10404
How was this patch tested?
new robot test

TestReconfigShell is failing, please take a look:
https://github.com/apache/ozone/actions/runs/8015289325/job/21897406860#step:6:149

Thanks @ChenSammi for updating the patch.


res.send cause process exit without a catchable exception

I have encountered process exit with res.send which will not throw any exception when the res has been terminated by the peer server.
this can be reproduce again and again.
if i comment res.send, then the node process won't exit anymore.

I am having this issue as well with node 5.3 and express 4.13.3
        return res.status(err.status || 500).send((['development'].indexOf(process.env.NODE_ENV) !== -1) ? err : {message: 'something went wrong'});

Process finished with exit code 139

Hi you guys. Unfortunately trying that code snippet does not cause the app to exit for me. In order for us to take a look, please provide the following information:

What exact version of express are you using?
What exact version of Node.js are you using?
A full app we can copy and paste into a file and run that reproduces the issue.
The request to make to the app to reproduce the issue.

Without us being able to reproduce the issue or you providing a stack trace of an error, we would have to wait for you to provide a pull request to fix the issue, if there is an issue in express.

I did some more digging and oddly, it only seems to do it for me when I run with --debug node flag.
This command does not exit:
PORT=3000 NODE_ENV=development NODE_PATH=. FORCE_COLOR=1 node server.js --color

While this one will exist after a res.status(500).send('whatever')
PORT=3000 NODE_ENV=development NODE_PATH=. FORCE_COLOR=1 node --debug server.js --color

Here's my middle ware:
   app.use(function (err, req, res, next) {
        // If the error object doesn't exists
        if (!err) {
            return next();
        }

        // Log it
        logger.error(err);
        if (err.stack) {
            logger.error(err.stack);
        }

        // Error page
        return res.status(err.status || 500).send((['development'].indexOf(process.env.NODE_ENV) !== -1) ? err : {message: 'something went wrong'});
    });

And here is a test API that crashes for me:
    app.route('/api/test').get(function(req, res, next) {
        return next({'message':'test error'});
    });

And here is my terminal output:
error:
{ message: 'test error' }
Segmentation fault: 11

Ah, a "Segmentation fault: 11". This is a Node.js bug, not an express bug. The way node.js works is that there is no way to cause this, so encountering one means there is something very wrong going on in the node.js core code. Can you open an issue against node.js, link here, and help them with reproducing?

Looks like it has already been reported: https://github.com/nodejs/node/issues/4382

Awesome!  Thanks for the help on this.

No problem! Sounds like it should be addressed in 5.3.1 or 5.4.0, which ever comes first.

@dougwilson I am using v4.2.3.
and it doesn't have stack trace when exit.

and debug is not enabled for me.


find

Method Name

find()

Other information

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/find

Available here now: https://github.com/jsPolyfill/Array.prototype.find


camera shake on small resolutions

https://github.com/Godlander/vpp/assets/68275908/a4e235e7-9991-46ad-b22b-1ab9bcb7523c
like this

i think it has to do with fps instead of resolution.
make sure your fps is somewhere between 30 and like 200

oh, yeap

would like to fix this issue but i don't think its possible currently


Info required for witch file

Appreciate the amazing example shared by the community and contributors
I need to ask a question about witch file which is used as lambda, but I am not able to get the purpose and why this file is required?
Also do you have a python alternate of this file?
Thanks

Hi Ammar, thanks for opening an issue.
The witch.js file defines a AWS CloudFormation Custom resource handler. This handler is executed when deploying the cloudformation template. The purpose of the custom resource is to upload the contents of the static site to the S3 Bucket.
The function traverses the /www directory and uploads each file to the S3 bucket specified. I don't have a python equivalent of this custom resource.

Thanks @ConnorKirk
Really appreciate your timely and helpful response
Great resource and very helpful contributions by you all

Why is it called witch? :mage_woman: Is it an acronym or something, or is it just for fun?

@stilvoid has the authoritative answer


Missing license

In the spirit of Open Source, I implore you to consider adding an appropriate license to the project. As it is, the project must be considered proprietary and can't be used without first asking for your permission.
I'd recommend an MIT license. It's what most Reason projects use.
See https://help.github.com/articles/licensing-a-repository/ for more information.

Done! Thanks for bringing this to my attention :)


Gifs ?

Seriously how can you possibly read the gifs ? Those gifs in the readme are not useful, please add text + images.
But, this is the best effort for lisp in vscode by far!

Since repl development is dynamic in nature, I actually think gifs better show what the capabilities are, and are therefore useful.
Also, he's doing this work unpaid.  Feel free to submit a PR with your preferences and make an argument for them in the PR.

You have a point, I have a point, I'm willing to pay for a good dev env for lisp. There are so many efforts for vscode plugins for lisp, none of them seem to work (this one does) wasted quite a bit of time going through them.

Same. This is the only one that works. It's the first one that's ever been good enough to get me off of emacs.

Yeah, documentation isn't my strong suit. I'll try to work on it when I can, but it's not going to be high priority.

my two cents: gifs are ok for a readme, but not for a manual (which this project doesn't have yet)

The gifs have been removed from the readme for the new version.


typo: machinelearningservices/resource-manager/Microsoft.MachineLearningServices

SSL capitalization
configugation -> configuration
creadentials -> credentials

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go


LOD0 does not appear in lod_terrain example when CLIP_BOX_RADIUS is less than 8

As it says in the title. It's fairly easy to confirm this by changing that constant and running the example.
No leads on the root cause yet.

I think this is actually a non-issue.
Basically, with small radiuses, the camera just doesn't get close enough to the chunks on the ground to trigger the higher detail. For example, with R=4, if I just make the camera fly closer to the ground, then I can see LOD0 (red).


build(deps): build tonic[-build] from 0.8.2 to 0.8.3

Bah, this falls foul of our MSRV check, so probably hold off for now.


[TEST] Add fp16 tensor pool test

TensorPool tests are added to verify TensorPool requests and the following operations from both FP16 and FP32 tensors.
Self evaluation:

Build test:   [X]Passed [ ]Failed [ ]Skipped
Run test:     [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2283. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


(PDB-3154) pdb_support_guide: add stockpile info

CLA signed by all contributors.

Test FAILed

Test FAILed

Test PASSed


implement select, gernerate plist

add config function select & preffer
add rule plist_fragment
add plist implementation

Thanks for your hard work on this!
One more suggestion about logging: Since the logic is increasingly complex and we may have a lot of Github Issues in the future. Do you think we should add a logging system like https://developer.apple.com/documentation/os/logging to this project?

add config function select & preffer
add rule plist_fragment
add plist implementation


Redirect URI

Hi, thanks a lot for sharing your code. According to your source, I implemented it, but get back a response from Keycloak:
We are sorry... Invalid parameter: redirect_uri
Could you guide me, how a redirect_uri could look like?
I tested like
http://traefik-forward-auth.default:4181
(yes, my namespace is default for now, traefik, keycloak and your container are all inside one kubernetes), but was not successfull.
I also tried an external access https://auth.mydomain with the same result. Really would appreciate, if you can guide me, how the redirect_uri should look like.
Thanks,
Wolfram

Hi @wopl . What's the redirect url you have configured within Keycloak for your Traefik client?
In my Keycloak deployment the Traefik client (that used for the forward auth) has a redirect url of https://auth.mydomain./_oauth. This matches the auth.mydomain.io hostname that is configured for the Ingress Route, see https://github.com/sleighzy/k3s-traefik-forward-auth-openid-connect/blob/master/005-ingressroute.yaml#L13. The /_oauth path comes from the default "Callback URL Path" setting for the thomseddon/traefik-forward-auth container. You can see this in his README file  (https://github.com/thomseddon/traefik-forward-auth/blob/master/README.md) for the --url-path argument or $URL_PATH environment variable.
I hadn't explicitly included in my configuration or README, but should do so to provide more information, I'll update my README for this.

Hi @wopl . What's the redirect url you have configured within Keycloak for your Traefik client?
In my Keycloak deployment the Traefik client (that used for the forward auth) has a redirect url of https://auth.mydomain./_oauth. This matches the auth.mydomain.io hostname that is configured for the Ingress Route, see https://github.com/sleighzy/k3s-traefik-forward-auth-openid-connect/blob/master/005-ingressroute.yaml#L13. The /_oauth path comes from the default "Callback URL Path" setting for the thomseddon/traefik-forward-auth container. You can see this in his README file  (https://github.com/thomseddon/traefik-forward-auth/blob/master/README.md) for the --url-path argument or $URL_PATH environment variable.
I hadn't explicitly included in my configuration or README, but should do so to provide more information, I'll update my README for this.

In my Keycloak deployment the Traefik client (that used for the forward auth) has a redirect url of https://auth.mydomain.io/_oauth.

The auth.mydomain.io isn't literally my hostname just to be clear, I do actually have a proper domain name that points to my services. You would replace that with your own actual DNS name.

In my Keycloak deployment the Traefik client (that used for the forward auth) has a redirect url of https://auth.mydomain.io/_oauth.

The auth.mydomain.io isn't literally my hostname just to be clear, I do actually have a proper domain name that points to my services. You would replace that with your own actual DNS name.

Just replace auth.mydomain.io with the hostname that is actually used in your ingress route by Traefik to direct that traffic to the traefik-forward-auth service.

Just replace auth.mydomain.io with the hostname that is actually used in your ingress route by Traefik to direct that traffic to the traefik-forward-auth service.

Hi @wopl , all sorted now? Ok if I close this issue?

Hi @wopl , all sorted now? Ok if I close this issue?


Return password incorrect error message for login

Currently the login view returns the general invalid_credentials error message if either the username or password is incorrect. It would be nice to know if they had an incorrect password only or if their username was wrong

Actually that could be a nice feature for improving UX. However we use [authenticate](https://github.com/django/django/blob/master/django/contrib/auth/__init__.py#L62) function from django.contrib.auth to authenticate user and it does not say which piece of credentials is wrong. I think that this behaviour is determined by security reasons.
What's more I'm not sure if this is the feature request to Djoser or to Django itself. Any idea how to do it?

I thought about this more and while it is doable by overriding the django authenticate function in djoser, I feel like this would be a security mistake as you've mentioned. I'll close this for now since it doesn't make sense for this functionality to be in djoser.
Sorry!


No credentials provided issue.

Any attempt for frameworks or even mesos cli to communicate with the cluster result into a message such as:
I0125 10:30:57.357239 27386 sched.cpp:272] No credentials provided. Attempting to register without authentication

this has been seen when try to utilize spark jobs, register kafka & cassandra even when tried to manually execute a cli i.e.
 mesos execute --command="echo 'hello mesos'" --name='HelloMesos' --master=$(mesos resolve zk://XXX.XXX.XXX.XXX:2181/mesos)

Could you please advise?

the logs from mesos execute
> mesos execute --command="echo 'hello mesos'" --name='HelloMesos' --master=$(mesos resolve zk://XXX.XXX.XXX.XXX:2181/mesos)
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@712: Client environment:zookeeper.version=zookeeper C client 3.4.5
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@716: Client environment:host.name=XXX
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@723: Client environment:os.name=Linux
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@724: Client environment:os.arch=3.10.0-229.el7.x86_64
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@725: Client environment:os.version=#1 SMP Fri Mar 6 11:36:42 UTC 2015
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@733: Client environment:user.name=centos
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@741: Client environment:user.home=/home/centos
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@log_env@753: Client environment:user.dir=XXX
2016-01-25 10:30:53,727:27367(0x7fda4bc07700):ZOO_INFO@zookeeper_init@786: Initiating client connection, host=XXX.XXX.XXX.XXX:2181 sessionTimeout=10000 watcher=0x7fda5389b160 sessionId=0 sessionPasswd=<null> context=0x7fda28001b60 flags=0
2016-01-25 10:30:53,772:27367(0x7fda46d7b700):ZOO_INFO@check_events@1703: initiated connection to server [<IP_ADDRESS>:2181]
2016-01-25 10:30:53,776:27367(0x7fda46d7b700):ZOO_INFO@check_events@1750: session establishment complete on server [<IP_ADDRESS>:2181], sessionId=0x1521d33150b0032, negotiated timeout=10000
WARNING: Logging before InitGoogleLogging() is written to STDERR
I0125 10:30:53.777542 27371 group.cpp:331] Group process (group(1)@<IP_ADDRESS>:58487) connected to ZooKeeper
I0125 10:30:53.777581 27371 group.cpp:805] Syncing group operations: queue size (joins, cancels, datas) = (0, 0, 0)
I0125 10:30:53.777588 27371 group.cpp:379] Authenticating with ZooKeeper using digest
2016-01-25 10:30:57,109:27367(0x7fda46d7b700):ZOO_INFO@auth_completion_func@1286: Authentication scheme digest succeeded
I0125 10:30:57.109812 27371 group.cpp:403] Trying to create path '/mesos' in ZooKeeper
I0125 10:30:57.111749 27371 detector.cpp:156] Detected a new leader: (id='5')
I0125 10:30:57.111847 27375 group.cpp:674] Trying to get '/mesos/info_0000000005' in ZooKeeper
W0125 10:30:57.112638 27375 detector.cpp:444] Leading master master@<IP_ADDRESS>:15050 is using a Protobuf binary format when registering with ZooKeeper (info): this will be deprecated as of Mesos 0.24 (see MESOS-2340)
I0125 10:30:57.113106 27375 detector.cpp:481] A new leading master (UPID=master@<IP_ADDRESS>:15050) is detected
I0125 10:30:57.354746 27379 sched.cpp:164] Version: 0.25.0
I0125 10:30:57.357053 27386 sched.cpp:262] New master detected at master@<IP_ADDRESS>:15050
I0125 10:30:57.357239 27386 sched.cpp:272] No credentials provided. Attempting to register without authentication
E0125 10:33:04.565322 27388 socket.hpp:174] Shutdown failed on fd=6: Transport endpoint is not connected [107]

By default, mantl clusters are built with mesos authentication enabled. You can see the list of valid mesos credentials that are generated in your security.yml. Many frameworks allow you to pass in the credentials in some way - usually via cli args, a config file, or env variables. For example, the cassandra-mesos framework will read the 'DEFAULT_PRINCIPALandDEFAULT_SECRETenv vars. The kafka framework has--principaland--secretcli args (see https://github.com/mesos/kafka#scheduler-configuration). Unfortunately, there doesn't seem to be a standard so you have to consult the specific framework to verify if it supports auth and how to configure it. I don't see an obvious way to pass in credentials when usingmesos execute`.

@ipolyzos Are you still having this issue?

@siddharthist i don't have the issue anymore.
The issue risen due to that mesos framework authentication is not supported in Spark 1.4.0 (you can refer to SPARK-6284 for more). This has been addressed in spark-1.5.0 ( please refer to Spark 1.5.0 - Release notes).
To work around, you can either upgrade to Spark 1.5.0 or temporarily to disable authentication (i.e. set --authenticate flag to false, for more info pleae refer to mesos role readme.rs)


Issue with one-sided phase delay

Good

Bad

The same sort of issue occurs with the phase response.

The current release/0.0.x branch is fixed. Not sure which change did it...


Add zeroize

https://crates.io/crates/zeroize
EDIT:
Wait until new version is released as changes/extensions are on the way (i.e. https://github.com/RustCrypto/utils/issues/651 https://github.com/RustCrypto/utils/issues/652)'

No support for tinyvec
error[E0599]: the method `zeroize_or_on_drop` exists for mutable reference `&mut ArrayVec<[u8; 5]>`, but its trait bounds were not satisfied
   --> src/main.rs:6:46
    |
6   | #[derive(Debug, Default, PartialEq, Zeroize, ZeroizeOnDrop)]
    |                                              ^^^^^^^^^^^^^ method cannot be called on `&mut ArrayVec<[u8; 5]>` due to unsatisfied trait bounds
    |
   ::: /home/alwagner/.cargo/registry/src/github.com-1ecc6299db9ec823/tinyvec-1.5.1/src/arrayvec.rs:104:1
    |
104 | pub struct ArrayVec<A> {
    | ----------------------
    | |
    | doesn't satisfy `ArrayVec<[u8; 5]>: AssertZeroize`
    | doesn't satisfy `ArrayVec<[u8; 5]>: ZeroizeOnDrop`
    | doesn't satisfy `ArrayVec<[u8; 5]>: Zeroize`
    |
    = note: the following trait bounds were not satisfied:
            `ArrayVec<[u8; 5]>: Zeroize`
            which is required by `ArrayVec<[u8; 5]>: AssertZeroize`
            `&mut ArrayVec<[u8; 5]>: Zeroize`
            which is required by `&mut ArrayVec<[u8; 5]>: AssertZeroize`
            `ArrayVec<[u8; 5]>: ZeroizeOnDrop`
            which is required by `&mut ArrayVec<[u8; 5]>: AssertZeroizeOnDrop`
            `[u8]: Sized`
            which is required by `[u8]: AssertZeroize`
    = note: this error originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0599`.
error: could not compile `rustcrypto_utils_zeroize` due to previous error


pkg/api,pkg/file,pkg/netstore: add targets to query param on download path

This PR adds global pinning targets to the download path
Once the target is obtained from the query, then it's injected into the context for its use in netstore.get
Only the Get is being addressed for the time being.
The cmd join could also be added with a targets flag
Download with targets in curl/Browser
localhost:8081/files/e49bcef241dc2e177ba05719330db67b39eafd060da426efdfb708bd45c24e22?targets=0x2222

Thanks for the PR, Santi!
A couple of things which I noticed:

There is no test which validate that we can get the targets via chunk_test.go
added a test for this specific case
You explicitly assign the targets to the context in functions  bzzDownloadHandler and fileDownloadHandler, but you are not doing this in the chunkGetHandler. Is getting targets supported on the /chunks API endpoint?
I thought we were going to use the fileDownloadHandler, but this is a good point, added the endpoint for this as well.
If anything, I would rather push for validating the global pinning logic asap. To do so, only support for one endpoint is needed. I think the /chunks endpoint is most important. We can add support for the other endpoints easily after we have validated the logic on the /chunks endpoint.

the signature in the first place.
please amend, thanks.

amended binaries thanks for spotting this.
The ctx is being passed into JoinReadAll


HTMLToQPDF.Example is crash

click "Create pdf" is crash

You need to include "QuestPDF.Settings.License = LicenseType.Community; " into the file, when using the newest QPDF.
( e.g. in MainWindow.xaml.cs within the public MainWindow()-Function above InitializeComponent();
Also it throws an error stating "Compose Single doesn't include that Methode". Still doing some research on the origins of that error.

Edit: You also have to update QPDF within HTMLtoQPDF AND HTMLtoQPDF.Example.
Also you have to explicitely add SkiaSharp as nuget-Dependency, as this isn't included in QuestPDF anymore.
If you did all these three things, it should work.


Inconsistent margins when info block is followed by info-columns block

Expected Behaviour
If I add an info-columns block after an info block, I want the left margin of the info block to line up with the info-columns block. Additionally, the whitespace between the info block and the info-columns block seems too large.
I did try to add an info-columns block on it's own, but then it didn't have enough space between the info-columns block and the previous component.
Actual Behaviour
The info-columns block is wider than the info block, so the info-columns content extends beyond the margin. I think the different size containers also affects layout when you resize the browser window.

Sample Code that illustrates the problem
info.css
main div.info {
  **width: 90%;**
  margin: 0 auto;
  background: var(--spectrum-global-color-gray-100);
  justify-content: center;
  margin-bottom: 64px;
}

main div.info h2 {
  margin-top: var(--spectrum-global-dimension-size-800);
}

main div.info code {
  display: inline-block;
  min-width: auto;
  background-color: var(--spectrum-global-color-gray-100);
  padding: 0 var(--spectrum-global-dimension-size-50);
  margin: 0;
}

info-block.css

main div.info-columns-container {
  **width: 100%;**
  background: var(--spectrum-global-color-gray-100);
}

main div.info-columns {
  width: var(--spectrum-global-dimension-static-grid-fixed-max-width);
  box-sizing: border-box;
  margin: auto;
}

Request
Is it possible to make the info block and info-columns block have the same width-- I think the 90% width looks good.

closing this -- behaves as expected when published to Staging.


Add attribute support for certificate subject

Fixes #128
Related cert-manager/cert-manager#4502

/ok-to-test

Small update. I'll be out for a few weeks but am still tracking this. Looking forward to having more time in a few weeks.

This PR is ready, but I'm waiting for cert-manager to cut a new release so I don't have to duplicate the CVS functions that are available in pkg/util.

Update: https://github.com/cert-manager/csi-driver/pull/129#discussion_r1328087163

/retest


Hiding eViacam Interface

When integrating Precision Gaze Mouse with eViacam, is there a way to hide the eViacam click bar after setup?  I am planning to use voice clicking, and I would like to hide the bar so it does distract from or cover other things I'm working on.

Open up eViacam go to settings or config. Under Click make sure "dwell clicking" isn't selected that will make sure that the "click window" option won't be selectable. And that's all you can use whatever other method to click without unnecessary toolbars.

Great thanks for helping @Brotie. I'm closing this for now but if that doesn't help please reopen this issue @edmun777


document meaning of trivia

Really for @DanielRosenwasser : https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API
I suspect its very similar to that of Roslyn compiler, if so the following might be helpful beginning of copy paste:

Syntax trivia represent the parts of the source text that are largely insignificant for normal understanding of the code, such as whitespace, comments, and preprocessor directives.
Because trivia are not part of the normal language syntax and can appear anywhere between any two tokens, they are not included in the syntax tree as a child of a node. Yet, because they are important when implementing a feature like refactoring and to maintain full fidelity with the source text, they do exist as part of the syntax tree.
You can access trivia by inspecting a token’s LeadingTrivia or TrailingTrivia collections. When source text is parsed, sequences of trivia are associated with tokens. In general, a token owns any trivia after it on the same line up to the next token. Any trivia after that line is associated with the following token. The first token in the source file gets all the initial trivia, and the last sequence of trivia in the file is tacked onto the end-of-file token, which otherwise has zero width.
Unlike syntax nodes and tokens, syntax trivia do not have parents. Yet, because they are part of the tree and each is associated with a single token, you may access the token it is associated with using the Token property.
Like syntax tokens, trivia are value types. The single SyntaxTrivia type is used to describe all kinds of trivia.

Let me know what you think, pinging @mhegaz, @CyrusNajmabadi.
Terminology
Full Start/Token Start
Tokens themselves have what we call a "full start" and a "token start". The "token start" is the more natural version, which is the position in file where the text of a token begins. The "full start" is the point at which the scanner began scanning since the last significant token. When concerned with trivia, we are often more concerned with the full start.

Function
Description

ts.Node.getStart
Gets the position in text where the first token of a node started.

ts.Node.getFullStart
Gets the position of the "full start" of the first token owned by the node.

Trivia
Syntax trivia represent the parts of the source text that are largely insignificant for normal understanding of the code, such as whitespace, comments, and even conflict markers.
Because trivia are not part of the normal language syntax (barring ECMAScript ASI rules) and can appear anywhere between any two tokens, they are not included in the syntax tree. Yet, because they are important when implementing a feature like refactoring and to maintain full fidelity with the source text, they are still accessible through our APIs on demand.
In general, a token owns any trivia after it on the same line up to the next token. Any trivia after that line is associated with the following token. The first token in the source file gets all the initial trivia, and the last sequence of trivia in the file is tacked onto the end-of-file token, which otherwise has zero width.
For most basic uses, comments are the "interesting" trivia, which can be fetched through the following functions:

Function
Description

ts.getLeadingCommentRanges
Returns ranges of comments between the first line break following the given position and the token itself.

ts.getTrailingCommentRanges
Returns ranges of comments until the first line break following the given position.

In the event that you are concerned with richer information of the token stream, createScanner also has a skipTrivia flag which you can set to false, and use setText/setTextPos to scan at different points in a file.

Let's get content like this into a wiki page. We can add some more content under the Building Tools for Typescript header.

I've put it into the wiki page, still need some review on this content.

looks good. thanks @DanielRosenwasser. one thing that is different from the Roslyn notion of trivia, is we do not have trailing trivia. all triva in TypeScript is leading. so for instance
var a; // comment
b;

The trivia list [" ", "//comment", "\n"] the leading trivia for the expression statement "b;".

Thanks! https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API#full-starttoken-start closing


Set up feature branch and add nfts.getByWalletENS with new API

This removes all of the methods and updates to use the new graphql api, and adds nft.getAllByWalletENS back with the new query and makes a lot of changes in the architecture and patterns of the SDK:

Moves the SDK from packages/libs/apis/sdk to packages/libs/sdk. This is because the SDK may expand to encompass more than just API functionality.
Import either the top-level SDK object or the specific functionality you want. This is also to enable the addition of more classes of functionality in a separately initialized object
Adds mutliple chain support since the GQL API allows multiple chain support. This allows flexibility in multi-chain dapps, using the same sdk object for multiple chains

// Import whole top-level object
import QuickNode from '@quicknode/sdk'
// or just the functionality you want to use
import { API as QuickNodeApi } from '@quicknode/sdk'

// there is a subcategory that is initialized
// The user can pass in a default chain, which itself will default to ethereum
// .api will be for our indexed data
const qnApi = new QuickNode.API({ apiKey: "abc123", defaultChain: "polygon" })

// For .api, the user can override the default chain
const nfts = qnApi.nfts.getByWalletENS({ ensName: "vitalik.eth, chain: "ethereum" })
const tokens = qnApi.tokens.getBalancesByWalletAddress({ walletAddress: "0x123", chain: "ethereumSepolia" })
const transactions = qnApi.transactions.getByWalletAddress({ walletAddress: "0x123" })

A new folder structure, separating out the api and client functionality, and within the api folder, separating the controller logic from the graphql queries and logic.
Add a post-query formatter formatQueryResult, which will be used to transform the graphql response into a consistent format
Rename getNFTsByWalletEns to getByWalletEns. I'm using this naming so we can be consistent with the other items we retrieve, for example with .token, we can also say .getAllForX, and also the NFT` part is redundant IMHO.

The queries are written for each supported network, for example EthMainnetWalletNFTsByEns, so codegen can generate the types for each chain/networks graphql query. I put as much as I could in fragments, and the repeated the queries for each network, wrapped around the fragments. I tried a couple different approaches, but I found this was the best way to still be able to generate the graphql-codegen schema types. The only alternative that really stood out was writing a custom codegen plugin, but I didn't see a straightforward way to have a type that will support multiple network names generated.
I also added the typed-document-node plugin to codegen which generates typed queries that we can use when we call the query in the controller, I found this useful for typechecking

Gonna merge since this is to a feature branch and I am writing PRs on top of this - The build is failing but I'm going to figure out the build issues after writing the rest of the functions :shipit:


VisualStudio2022 Plugin not working

Reqnroll Version
v2.1.0
Which test runner are you using?
xUnit
Test Runner Version Number
2.6.6
.NET Implementation
.NET 8.0
Test Execution Method
Visual Studio Test Explorer
Content of reqnroll.json configuration file
{
"$schema": "https://schemas.reqnroll.net/reqnroll-config-latest.json",
"bindingAssemblies": [
]

}
Issue Description
Was just trying Reqnroll for the first time. Using latest of all as of today. However, after doing all the steps as per the instructions, the context menus are not working. I cannot navigate to step defs either. The options menu also doesn't show any Reqnroll options.
However, I'll like to emphasise that the tests run fine.

Steps to Reproduce
use latest VisualStudio 2022.
Install the plugin by following:
https://docs.reqnroll.net/latest/installation/setup-ide.html#setup-vs
After doing this, I was not able to see the project template for Reqnroll. Then I installed the project templates using this:
dotnet new install Reqnroll.Templates.DotNet
Then templates started working. But now the context menus, step def nav and options menu not working, as explained above.
However, I'll like to emphasise that the tests run fine.
Link to Repro Project
No response

Would it be possible to raise this issue in the below repository, as it relates to the Visual Studio 2022 extension:
[Reqnroll.VisualStudio](https://github.com/reqnroll/Reqnroll.VisualStudio)

Should now be tracking this issue here:
https://github.com/reqnroll/Reqnroll.VisualStudio/issues/37

thanks @UL-ChrisGlew.

amazing. I just tried it. All works absolutely fine now :) Thanks again @UL-ChrisGlew
You guys are doing afantastic job. Cant tell you how much relieved I'm by seeing this support. I was scared today, when after a couple of years since using Specflow last time, today when I saw it dead, I was like freaked out. Now I'm confident :)

A hotfix has been published to the marketplace as v2024.4.154.
I will keep this issue open for a day so that people can find it.


Create Integ test page

The integ test page will be a list and must have in each row:

plugin name
security enabled?
distribution
architecture
tests executed
tests passed
tests failed
start time
end time
test log


if a  slidebox is be cached and then open  a modal in a new state,the original  slidebox's width becames 0px

if a  slidebox is be cached and then open  a modal in a new state,the original  slidebox's width becames 0px
it was like this https://github.com/driftyco/ionic/issues/360
version Ionic, v1.0.0-beta.14
forget my poor English :(

I was running into the same issue. Set the cache to be disabled on the slider by adding cache: false to the state. Seemed to get around the problem for now.

@Chuckv01  Set cache false to the satae  is a way,but  i want use  the cache :(

This issue is a duplicate of #2817.

This should be fixed now: https://github.com/driftyco/ionic/commit/040dabf2c9a8a6c3fd154e4e27391299c5978939


To try with docker-compose

Hi tsubotti!
I realized that we can't use docker-compose.
It is not maintained because you are not using it, right?
But, it is very useful who is to try shawk and I want to use it.

Thanks, pyama!

It is not maintained because you are not using it, right?

That's right. Now, I'm developing this project on VirtualBox via vagrant, but the development on docker-compose is also useful.


Moved Cohere import

:rocket:
This description was created by Ellipsis for commit 1256b60d4ff6d966881e6512464ee386f459a343

Summary:
Moved NonStreamedChatResponse import inside parse_cohere_json_schema method and updated pyproject.toml to include cohere as an optional dependency.
Key points:

Moved NonStreamedChatResponse import inside parse_cohere_json_schema method in instructor/function_calls.py.
Updated pyproject.toml to include cohere as an optional dependency.

Generated with :heart: by ellipsis.dev

This removes the top level cohere import from the function_calls file, allowing users to use instructor without having to install cohere


Improvements/feedback

I have spent some time working with this framework in attempt to use it in the monitoring app. In the end, there were several issues preventing me from using this framework for a production ready app.This lead meyme to write my own framework SwiftyArk. I wanted to share with you some of the feedback I have.

The main issue I had with the app is the networking architecture. You are making asynchronous network calls to the ark node, but the method signatures are synchronous:

let allMainBlocks = ARK.main.allBlocks()

is a synchronous assignment with no error handling. However, behind the scenes this is an asynchronous call that has the potential for many errors, which are never handled. It is not documentation that this assignment will fail, or result in an empty array. The swift way to do this is with a completion handler and error handling:
ARK.main.allBlock() { (error, blocks) in
        if let aError = error {
              /// handle error
        }
        if let allBlocks = blocks {
             // all blocks are returned here and unwrapped
        }
 }

The difference between the two (besides error handling) is the second option returns data in an asynchronous manner. In your API, the assignment occurs  on the main thread and will block the entire app until it returns or errrors out.

The networking of this app boils down to using NSString.(contentsOf:URL:) to grab raw data and convert it to a UTF8 string. This api wasn't designed for being used for networking, but rather read files from inside your app. It is not documented anywhere the error handling and doesn't give you any access to HTTP data (headers, statusCodes etc). You should look into using URLSession as it is the primary function for networking in iOS/mac apps. This will also allow you to extend your API to allow for PUT and POST operations, something NSString.(contentsOf:URL:)

Your data types are not very swift like. You declare your internal properties as variables, rather than constants. There is nothing in the API to suggest the types should be modified, so why not make them immutable?

More on the data types, the initialization of them is very un-swift like and leads to several issues. Data types like Account do not have any initializers, but rather assignments in an protocol (which I will get to). This is not very safe as these are declared as ** non-optional options**:

public struct Account: Accountable {
    public var  address: String!
    /// ...

You are basically saying "I guarantee there will be an address for an Account, but you do not provide (or prevent) any initialization, so its not safe:
let account = Account()
let example = account.balance * 10
// CRASH!

I would recommend you get rid of the assignment inside of a protocol and move it directly to an initializer, make the types immutable constants/optionals and prevent any unintentional initialization. Swift 4 introduced the Decodable protocol to direct add an initializer from json. Using it in conjunction with URLSession is a winning combo.

This last one is more of a style comment, but I have noticed you use Swift types in a non-typical way. Your primary API manager is an enum when it should be a class/struct. You also create protocols for each data type (Accountable, Delegable etc) when they essentially do the same thing for each data type. You could create one data protocol that would apply to all datatypes which can be decoded from son (like Decodable)

I hope this helps you. I want this framework to be great. You are doing a lot of great things for the community and I would very much like to see all the features you hope to implement. If you want some inspiration, you can check out SwiftyArk or if you need any advice<EMAIL_ADDRESS>Andrew

Beautiful!
Exactly the direction it needs to go before it's even close to a 1.0 release.

The main issue I had with the app is the networking architecture. You are making asynchronous network calls to the ark node, but the method signatures are synchronous:

This is a priority over in the next releases for exactly the reasons you stated.

The networking of this app boils down to using NSString.(contentsOf:URL:) to grab raw data and convert it to a UTF8 string. This api wasn't designed for being used for networking, but rather read files from inside your app. It is not documented anywhere the error handling and doesn't give you any access to HTTP data (headers, statusCodes etc). You should look into using URLSession as it is the primary function for networking in iOS/mac apps. This will also allow you to extend your API to allow for PUT and POST operations, something NSString.(contentsOf:URL:)

The move to the URLSession is necessary and planned. more below.

Your data types are not very swift like. You declare your internal properties as variables, rather than constants. There is nothing in the API to suggest the types should be modified, so why not make them immutable?
More on the data types, the initialization of them is very un-swift like and leads to several issues. Data types like Account do not have any initializers, but rather assignments in an protocol (which I will get to). This is not very safe as these are declared as ** non-optional options**:

Not pretty, and certainly not documented;
I mostly wanted these to be expanded into things such as address-checking, currency-conversion, etc.
I hadn't gotten to playing with Decodable much yet, but that makes perfect sense as a solution!
Thank you!

This last one is more of a style comment, but I have noticed you use Swift types in a non-typical way. Your primary API manager is an enum when it should be a class/struct. You also create protocols for each data type (Accountable, Delegable etc) when they essentially do the same thing for each data type. You could create one data protocol that would apply to all datatypes which can be decoded from son (like Decodable)

This I know is not typical.
Basically, I wanted to play with name-spaced-constants, avoiding instantiation if all I need is a value;
and also wanted to experiment with making it highly modular.
ARKKit definitely has room for work, and your feedback is a great help that I appreciate immensely!
It's very kind of you to spend time checking it out and suggesting these solid improvements!
This project started as a quick and dirty way to get a custom Ark app running for the IoT projects I'm hyper-focused on.
There were also no Swift wrappers to be seen,
so I figured it'd be a great way to learn the Ark Ecosystem and help kick-start interest from the Swift community at the same time.
That said, there is definitely much that will change before a final release;
and I will certainly make good use of your fantastic suggestions!
Thank you again, @Awalz
SwiftyArk is lookin great !
Keep up the amazing work!


tests: drivers: watchdog: Add negative test cases for Watchdog

Extend test coverage by adding tests that check invalid use of the Watchdog API.

[rebased on top of main - https://github.com/zephyrproject-rtos/zephyr/commit/6a42ede43b53ceffddeaa814b4bd99284c25240d]

It works in CI
08:50:35  INFO    - 1/1 nrf54l15pdk/nrf54l15/cpuapp tests/drivers/watchdog/wdt_error_cases/drivers.watchdog.wdt_error_cases PASSED (device:<PHONE_NUMBER>, 6.900s)

I have this PR on my ToDo list, but didn't find the time, yet.
The tests look somewhat tricky and I would like to try out if they also work on some non-Nordic hardware (STM32).

I have this PR on my ToDo list, but didn't find the time, yet.
The tests look somewhat tricky and I would like to try out if they also work on some non-Nordic hardware (STM32).

That's fine. I've take the liberty of assigning this PR to you. Please change it as necessary.


Typescript type definitions.

i can do some experiments to see if there's a way to avoid both of those. Again, some of this may be a result of my inexperience.
Modeling Jobs as a type has to deal with the fact that Jobs contain a mix of properties from both the application and the library. The obvious way to deal with this is through inheritance by using Job as a base type, which the user would extend. The question then is whether to make Job a class or an interface.
Making Job an interface seems like the right solution on first glance, since interfaces are purely compile-time entities that are overlayed on existing JavaScript objects. However, the downside of using an interface is that the user's actual concrete class would have to re-declare all of the properties in Job, because the compiler wants to check that the class implements all of the properties in the interface.
The alternative is to make Job an abstract class (in fact I probably should have used the 'abstract' keyword). However, in order to inherit from a class, it has to be a real value of some kind, i.e. Queue.Job has to exist as an actual thing.
One might ask why not move Job to the global namespace? Well, the problem there (and again this might be my lack of experience) is that index.js exports queue using commonjs syntax rather than export default, which has you know has slightly different semantics; And because of that, the index.d.ts file uses "export = Queue" rather than "export default Queue" in order to tell the typescript compiler to use commonjs semantics, which in turn means that you needs to use "import * as Queue" rather than "import Queue" in user code. And it's hard to mix importing * with importing other things.
That being said, it seems to work sometimes without me having to do this, and I'm not yet sure why. That is, for most of the time I didn't have Queue.Job = Job, and then there was a brief period where it stopped working, and then later started working again. Further investigation is warranted.
I'll go ahead and remove the package.json dependency, it's only there to make my text editor's linter happy.

Ok, thanks @viridia
I was under the impression we just needed the .ts and config files. Not keen on changing functional code to support a superset language. The package.json still has the @types/node dependency.

Sorry, since I switched over to using yarn I sometimes forget how to use npm :)

Good stuff. Thanks @viridia


configuring custom language server broken

How to reproduce the problem from neovim startup
I use the YANG language server and, since it is not natively supported here, I have added custom configuration for it following the pattern in the README, like this:
local configs = require'lspconfig/configs'
if not lspconfig.yang_language_server then
  configs.yang_language_server = {
    default_config = {
      cmd = {'yang-language-server'};
      filetypes = {'yang'};
      root_dir = lspconfig.util.root_pattern("yang.settings", ".git");
      settings = {};
    };
  }
end
lspconfig.yang_language_server.setup{}

With configuration like that, open neovim.
Actual behaviour
Error detected while processing /home/dch/.vim/vimrc:
line  203:
E5108: Error executing lua cannot open /home/dch/.vim/plugged/nvim-lspconfig/lua/lspconfig/yang_language_server.lua: No such file or directory

Expected behaviour
Everything should be fine!
git bisect reckons that this was introduced at 04f223f3d08388eac74a571516145154bf5af64c.
Minimal init.vim or init.lua and code sample
https://github.com/neovim/nvim-lspconfig#example-custom-config
Health check

Checkhealth result
health#lspconfig#check
========================================================================
## Checking language server protocol configuration
  - ERROR: gopls: The given command "gopls" is not executable.
  - INFO: terraformls: configuration checked.
  - INFO: jedi_language_server: configuration checked.
  - INFO: rust_analyzer: configuration checked.
  - ERROR: ccls: The given command "ccls" is not executable.

LSP log
N/A

it should be an easy fix if you want to PR it, just try wrapping dofile in a protected call

indeed that seems to do it, PR made

I am not sure if wrapping the dofile in a pcall helps here as it breaks texlab. Essentially, in the latest commit, when the config for texlab is loaded, the new table for texlab is not created in configs.__newindex.
I replaced the dofile here with the old call to require. This works fine.
I am in the process of debugging why this happens, but would love some help.

Texlab works fine for me, can you reproduce with our minimal config? I haven't seen your config, but are you doing something like this? https://github.com/neovim/nvim-lspconfig/pull/861#issuecomment-825862325

Hey @mjlbach, my texlab config is here and the call to setup is here.

Sorry, I can't read through your personal config. I'd recommend matching the recommended method of adding custom language servers per our readme. Anything else is unsupported (if you want, you're free to override our internal use of dofile with require)

Turns out the issue was this line. Not sure why, but removing that (unnecessary) line solves my issue.


Generating additional routes (.xml files) and scenarios (.json files)

The leaderboard repo contains predefined routes (.xml files) and scenarios (.json files). Will the script for generating these .xml and .json files be provided?

Already answered on the carla forum so closing it here.


registration throttling

Hello, and thanks in advance for your attention on this!
I'm encountering throttling of registrations after only five successful requests (the sixth is throttled).
Is this the intended behavior?
Here is the test code I've used to reproduce:
for ($i = 6; $i > 0; $i--) {
  try {
    $userId =<EMAIL_ADDRESS>'test123!', null);
    echo 'We have signed up a new user with the ID ' . $userId . '
';
  } catch (\Delight\Auth\InvalidEmailException $e) {
    die('Invalid email address
');
  } catch (\Delight\Auth\InvalidPasswordException $e) {
    die('Invalid password
');
  } catch (\Delight\Auth\UserAlreadyExistsException $e) {
    die('User already exists
');
  } catch (\Delight\Auth\TooManyRequestsException $e) {
    die('Too many requests
');
  }
}

This yields:
We have signed up a new user with the ID 1
We have signed up a new user with the ID 2
We have signed up a new user with the ID 3
We have signed up a new user with the ID 4
We have signed up a new user with the ID 5
Too many requests

We have a user who needs to onboard batches of clients. Our registration form takes a minute or two to fill out, so unlike the above code there are probably five requests over the course of 5 minutes at the least, not a few seconds, but the throttling still engages.
I'm aware of #253 but the behavior we're seeing above doesn't seem to line up with the limits you documented there. Before we implement custom throttling for registrations I just want to verify that this isn't a bug.
We are still on version 8.2.2 in case it matters.
Thanks for your time!

Thank you!
The limits documented in the issue that you linked to are not for the registration feature. Each feature has different throttling, as the needs naturally vary as well.
Is your application behind a proxy, e.g. a load balancer, perhaps? If you don’t pass the real IP address of the client in the constructor then, multiple clients may be treated as one for throttling purposes (because they will appear to have the same IP address).
Or is your onboarding done from a single machine in a controlled environment? If you try to create more than 5 accounts from a single client machine within a short time, yes, that will trigger a limit. This is unlikely to happen in normal usage. And if you still want to do it, because you trust the client, there is no reason why you couldn’t just disable throttling for that one client (and for that one feature), right?
(Your library version is fine.)

Yes, looks like the best solution here is indeed managing your own allow list of trusted IP addresses and then setting the fourth argument of the Auth constructor (i.e. throttling on or off) based on whether the client’s current IP address is in the allow list.

It's not directly connected to the registration process but I'm interested in bypassing throttling in specific situations. Unfortunately, I'm not an experienced programmer so... Regarding the statement "managing your own allow list of trusted IP addresses and then setting the fourth argument of the Auth constructor (i.e. throttling on or off) based on whether the client’s current IP address is in the allow list.", could any kind soul point me in the direction of exactly how this is done? I mean, 1) how do I manage a list of trusted IP addresses, and 2) how do I tell the constructor to disable throttling based on the list of allowed IPs? An example snippet of code would be GREATLY appreciated. Thanks in advance.


inky rutledge memorial site theme update

added a background, some color adjustments, some small layout adjustments, and a favicon
night mode only for now
more to come
here is a preview of how it looks

Wow! Thank you very much for the theme!


Readme references old version of paper

The readme says:

Enable PaperSpigot server mode by adding a -e TYPE=PAPER -e VERSION=1.9.4 to your command-line.

1.9.4 is very old, and specifying is unnecessary.  Later on covers using a specific version, rather than the latest.
(just checking I understand these flags, I'll submit a PR for this)

Yes, that is simply an example and a very old example. So a PR to update or simply removing that would be great.

PR ready, let me know if it goes too far:
https://github.com/itzg/docker-minecraft-server/pull/541

This is off-topic, but I just wanted to say thanks for putting this image together.  I used it to host a birthday party for my daughter and 20 friends, with some fun mods I wrote with PaperMC.  Without the heavy lifting done here, it would have been much more difficult.
Thanks again.

That’s so cool and thanks for the help.


feat(carousel): use passive event listener to improve scrolling performance

Use passive event listener to improve scrolling performance
Description
Added { passive: true } to touchstart event listener
Motivation and Context
https://developers.google.com/web/tools/lighthouse/audits/passive-event-listeners
How Has This Been Tested?
Using the new build from this branch, lighthouse audit's result is clear from Does not use passive listeners to improve scrolling performance
Types of changes

[x] Refactor touchstart event listener, added passive: true options

Checklist:

[x] My code follows the code style of this project.

Coverage increased (+0.07%) to 73.913% when pulling df9f6dc78ba9b02979e5d12886e0186ef1435626 on nugrahawahyu:feat/passive-event-listener into 209143100c93a75a92463ebd7b45fa6a27b07c5c on SSENSE:master.

Also, with or without the added event utility we'll need some tests for this. Feel free to consult @ashleysimpson or I if you have any questions about that!


Ensure that Cloud Storage bucket is not anonymously or publicly accessible

Possible check to target the following resource

[ ] google_storage_bucket_iam_member

closing, tracking in spreadsheet


[AutoPR containerservice/resource-manager] Dev containerservice microsoft.container service 2019 02 01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5406

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4318


Study page in web-version hangs just after Reset all my data

Probably because no words are added, but the web-version doesn't tell that

Hi, if you have no lists to study from, the app should display a call to action to add a list. If you have a list in your queue but no words studied or added yet, it should add a few initial words. Which of these two cases are you in?

I think it was the second case, I just added a first list, but the queue was empty.
The web-version didn't show a prompt to add words, while the Android version did show a prompt, I clicked on "Add words" and then the web-version also started to work.

Ok, thanks for the report. Something to double check and make sure that it works correctly.


Add 'GTID' to button name

https://github.com/github/orchestrator/blob/113d3ba9238063a8c16a9e78a927b51cc6d68cbe/resources/templates/layout.tmpl#L216
When viewed in the GUI, this button just says "Enable" with no tooltip or other indication of what its purpose is. Not until you click the button are you presented with a box that asks if you want to enable GTID. If the button was labeled "Enable GTID", it would clear what it is for.
Same for when this button says 'Disable' and the other button that just says "Fix" (maybe "Fix Errant GTID"?)

I'm unable to reproduce this, and for me the button appears in the correct context. Could you screenshot the entire pop up dialog please?

addressed by https://github.com/github/orchestrator/pull/717

As per https://github.com/github/orchestrator/pull/717#issuecomment-439149247 button appears on bottom instead of on relevant row, when enabled.

This fixes the button not showing when GTID is not possible, but when possible to enable, the button still shows at the bottom and still only says the word "Enable". There is not any indication this "Enable" button refers to GTID.

@utdrmac this does not reproduce for me. Could you please Shift-Refresh to make sure the latest js files are loaded into browser's cache?

woot! Just reproduced, on a dead/unseen server.

Sweet. Sorry I couldn’t reply; been traveling for holiday.

On Nov 21, 2018, at 3:14 AM, Shlomi Noach<EMAIL_ADDRESS>wrote:
woot! Just reproduced, on a dead/unseen server.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[meta] 6.8 release

Leftover Work
Geo:

[x] Integrate Lucene's LatLonShape (BKD Backed GeoShapes) as default geo_shape indexing approach elastic/elasticsearch#35320 (issue: elastic/elasticsearch#32039) - https://github.com/elastic/elasticsearch-net/pull/3770

Cross-cluster replication

[x] Refactor AutoFollowCoordinator to track leader indices per remote cluster #36031 (issues: #33007, #35895) - no client changes required
[x] Refactor auto follow coordinator #35895 (issue: #33007) - no client changes required

The last two removed CCR_AUTO_FOLLOW_POLL_INTERVAL which i think is returned on the ccr stats API's (investigate)
CRUD

[x] Rename seq# powered optimistic concurrency control parameters to ifSeqNo/ifPrimaryTerm #36757 (issues: #10708, #36148) - no client changes needed, API generator picked up new parameters
[x] Expose Sequence Number based Optimistic Concurrency Control in the rest layer #36721 (issues: #10708, #36148) - no client changes needed, API generator picked up new parameters
[x] Add doc’s sequence number + primary term to GetResult and use it for updates #36680 (issues: #10708, #36148) - https://github.com/elastic/elasticsearch-net/pull/3767

Distributed

[x] Add global and index level blocks to IndexSettings #35695 (issues: #35332, #35597) - no client changes required

Machine Learning

[x] Enable the use of endpoints starting with _ml instead of _xpack/ml #36373 (issue: #36315) - no client changes required
[x] Add support for rollup indexes in datafeeds #34654 - no client changes required

(First is not actionable on our end in 6.x).
Rollup

[x] Add non-X-Pack centric rollup endpoints #36383 (issues: #35958, #35962) - no client changes required
[x] Add more diagnostic stats to job #35471 - https://github.com/elastic/elasticsearch-net/pull/3759
[x] Add default fields to job configs #34831 - no client changes required
[x] Add wait_for_completion option to StopRollupJob API #34811 (issue: #34574) - no client changes required, was picked up in an API regen

Search

[x] Warn when using use_dis_max in multi_match #36614 (issue: #36488) - no client changes required
[x] Extend field caps API to mark meta fields #36309 - original issue was closed

Security

[x] Make credentials mandatory when launching xpack/migrate #36197 (issues: #29847, #33972) - no client changes required
[x] Make credentials mandatory when launching x-pack/migrate #33972 (issue: #29847) - no client changes required

SQL

[x] Move requests' parameters to requests JSON body #36149 (issue: #35992) - no client changes needed
[x] XPack FeatureSet functionality #35725 (issue: #34821) - https://github.com/elastic/elasticsearch-net/pull/3764

Stats

[x] Handle OS pretty name on old OS without OS release #35453 (issue: #35440)
[x] Add more detailed OS name on Linux #35352 - https://github.com/elastic/elasticsearch-net/pull/3766

Bug fixes
Aggregations::

[x] Preserve format when aggregation contains unmapped date fields [#35254] (issue: [#31760]) - reminds me of a ticket we once raised (will follow up to figure if they are related) - no client changes needed, this is a server-side change

Cross-cluster replication::

[x] Resume follow Api should not require a request body [#37217] (issue: [#37022]) (_breaking change we need a patch for) - change was implemented on 10th Jan, but CCR was implement in client on 25th Jan, therefore no client changes needed as already incorporated.
[x] Report error if auto follower tries auto follow a leader index with soft deletes disabled [#36886] (issue: [#33007]) - no client changes needed
[x] Fix the names of CCR stats endpoints in usage API [#35438] - no client changes needed

Index lifecycle management::

[x] Remove indexing_complete when removing policy [#36620] - no client changes needed

Index APIs::

[x] Make XContentBuilder in AliasActions build is_write_index field [#35071] - no client changes needed
[x] Raise a 404 exception when document source is not found (#33384) [#34083] (issue: [#33384]) - no client changes needed

Ingest::

[x] Support default pipelines + bulk upserts [#36618] (issue: [#36219]) - no client changes needed
[x] Support default pipeline through an alias [#36231] (issue: [#35817]) - no client changes needed

Network::

[x] Do not resolve addresses in remote connection info [#36671] (issue: [#35658]) - no client changes needed
[x] Always compress based on the settings [#36522] (issue: [#36399]) - no client changes needed
[x] http.publish_host should contain CNAME [#32806] (issue: [#22029]) - no client changes needed

Scripting::

[x] Fix serialization bug in painless execute api request [#36075] (issue: [#36050]) - no client changes needed

Linked to: https://github.com/elastic/elasticsearch-net/pull/3770

Relates to: https://github.com/elastic/elasticsearch-net/pull/3774


Use package ID as the application ID when launched from XPK.

XPK file will be unzipped to a temporary directory when launch, the
app ID will change as temporary directory changes. Use package ID can
avoid this issue.

@pozdnyakov  thanks.

lgtm


issue while making transaction for customer

[x] issue while making transaction for first created customer when more than 10 customers are created in total

issue fixed !


Changing values?

Hello.
Thanks for this great plugin! Works perfectly. I just wanted to know if it's possible to change the values set by this plugin. If yes, how do I do it?
Basically, I need to change the width and height and the overflow: overlay property set by the plugin. I understand the overlay value should exist according to the comment you've written in the code, but in my setup, it's creating problems. So, I'd like to unset it.
Thank you for the help.

To provide some context, I'm using the plugin like this:
* {
  @apply scrollbar-thin scrollbar-thumb-yellow-400 scrollbar-track-gray-200 dark:scrollbar-track-gray-700
}

because I wanted to apply same scrollbar style to all elements wherever the scrollbars were visible.
But, because of the overlay property, I'm getting this:
,
where as without it, I was getting this:

If there's some other way without changing values, for example, a way to disable the plugin on one of the elements or something would do too.

Oh, a simple workaround was to select all elements with overflow set to auto or scroll, something like this:
overflow-auto, overflow-scroll, overflow-x-auto, overflow-y-auto, overflow-x-scroll, overflow-y-scroll {
  @apply focus:outline-none scrollbar-thin scrollbar-thumb-yellow-400 scrollbar-track-gray-200 dark:scrollbar-track-gray-700
}

Sorry for bothering.

About a way to change the values, I'm still open to ideas for that, but it's not so important. Thanks a lot.

Oh, a simple workaround was to select all elements with overflow set to auto or scroll, something like this:
overflow-auto, overflow-scroll, overflow-x-auto, overflow-y-auto, overflow-x-scroll, overflow-y-scroll {
  @apply scrollbar-thin scrollbar-thumb-yellow-400 scrollbar-track-gray-200 dark:scrollbar-track-gray-700
}

The only problem with this is that, I need to apply it for all variants like the responsive ones. So, this does make it a little convenient, but still usable.
Sorry for bothering.
I'm still open to ideas for regarding this, and also about changing the width and height if possible. Thanks a lot.


Autocomplete: Fix server side issue where Reset would reopen item Popover

Description

In some instances on Blazor Server, calling Reset on an autocomplete will cause the item Popover to open.

How Has This Been Tested?

All existing unit tests pass. This particular issue would be difficult to test for as it is server only (latency induced?).
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Ok, not sure what's up with this failing test, it's indicating the popover isn't opening at all when searching in the failed test, but if I switch back to the dev branch this test still fails but for a different reason.

Ok, the issue here is the _suppressPopup flag isn't getting set to false in the unit test (and probably wouldn't in normal WASM either).

@just-the-benno I am not sure about this, any thoughts

I think I'm barking up the wrong tree. The Popover only comes up again if you have modified the text manually, not if you simply select an option. So something is behaving wrongly but there's more to it than clicking Reset.

Ok, here's an update on what happens with the method calls that occur and some variables at the time of calling.
Server, coerceValue true, after typing a value that is coerced:
ResetValue _isCleared: False
Clear _isCleared: False
CoerceValueToText _isCleared: True
UpdateValuePropertyAsync _isCleared: True, updateText: False
UpdateTextPropertyAsync _isCleared: False, updateValue: False
UpdateTextPropertyAsync _isCleared: False, updateValue: False
OnSearchAsync _isCleared: False

Server, after selecting an item from the dropdown list (WASM has an identical output in either case):
ResetValue _isCleared: False
Clear _isCleared: False
CoerceValueToText _isCleared: True
UpdateValuePropertyAsync _isCleared: True, updateText: False
UpdateTextPropertyAsync _isCleared: False, updateValue: False
UpdateTextPropertyAsync _isCleared: False, updateValue: False
UpdateTextPropertyAsync _isCleared: True, updateValue: False

The difference is that last line. I should also note that this isn't a true stacktrace, I just put Console.WriteLine statements in the important methods to see what was going on.

In my experience, if there is a difference between BSS and WASM, that is mainly caused by threading issues. WASM doesn't have a threading model yet, so not awaiting calls doesn't do any harm, while for BSS, with a so advanced threading and rendering mechanism, you either create a blinking effect (work, don't work, and we blame it on "latency" or the component behavior change in a very subtle yet still "breaking" way). Unfortunately, we have this pattern a lot in the library (like .AndForget()).
I hope I can put some time aside to investigate this bug soon.

@Mr-Technician, can you provide a quick trymud? I know it will "work" because of WASM, but this would be a good starting point for me. As an alternative, you can include the example in your PR, and I can work from there.

@just-the-benno Here you go: https://try.mudblazor.com/snippet/wuwcEMbToLhMTXVC


Fix issue "URI is not hierarchical" when working with extensions jar's

Replace working with content of plugin jars with NIO.
new File(url.toURI()) -> FileSystems.newFileSystem(url.toURI() ...

Thanks!


fix(core): fix twitter meta tags

fix(core): fix twitter meta tags
I believe the Twitter meta tags are outdated and this PR fixes them.
Taking into account the current MEAN.js tags and the Twitter docs, the most similar feature I found
is the Twitter Summary Card (https://dev.twitter.com/cards/types/summary), which requires 2 tags that currently do not exist (twitter:card and twitter:site) and there's another one already implemented in MEAN.js which is not referenced in Twitter docs (twitter:url).

Thanks Pedro.
Could you by any chance also review the other social network logins? They might also need an update.

I took a look at Facebook docs and those meta tags seem to be fine.


Custom validators

As described in documents, the second argument of the custom validators would be the vm instance.
But it is not, in my case:

export function uniqueEmailValidator (value, context) {
  console.log(value)
  console.log(context)
}

export default {
    data () {
      return {
        credentials: {
          email: '',
          password: '',
          repeatPassword: ''
        },
        api: new APIClient(this, '/apiv1/members')
      }
    },
    validations: {
      credentials: {
        email: {
          required,
          uniqueEMail: uniqueEmailValidator
        },
        password: {
          required,
          minLength: minLength(6),
          maxLength: maxLength(16)
        },
        repeatPassword: {
          sameAsPassword: sameAs('password')
        }
      }
    },

The actual value of the context is:
credentials: {
  email: '',
  password: '',
  repeatPassword: ''
}

Hi!
Documentation is right, but I'm afraid i have to work a bit on cleaner wording and provide a few more examples.
Citing myself :P

make use of the function context (this) to access any value on your component or use provided parentVm to access sibling properties

The actual function context this gives you access to a thing i call "root VM", which is always your whole component. The third argument, parent VM, is always just the dataset which is  a parent to the validated property, effectively giving you a "shortcut" to it's siblings. That's the usual use case for forms, so that's what sameAs is using by default.
Try that
export function uniqueEmailValidator (value, parentVm) {
  console.log('rootVm', this)
  console.log('value'. value)
  console.log('parentVm', parentVm)
}

Try hooking that validator on different depth levels, you will see the difference immediately :)

Interesting, thanks a lot. i'll check it

Thanks a lot,the the function context: this is working well as well.
Sorry for taking your time, instead of the reading documentation carefully.


TraceBack errors:

When I try to download courses, terminal shows the message:
  File "F:\Udemy\udemy-dl\udemy\shared.py", line 137, in download
    retVal = FFMPeg(None, self.url, self.token, temp_filepath).download()
  File "F:\Udemy\udemy-dl\udemy\ffmpeg.py", line 174, in download
    with subprocess.Popen(
  File "D:\Anaconda\lib\subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "D:\Anaconda\lib\subprocess.py", line 1307, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] 系统找不到指定的文件。

hmm how can i reproduce? can you please explain a bit more and do follow issue reporting guideline?

Sorry, after I rebooted my laptop, similar problems did not recur.


Apple -> Fapple

Thank you for the submission, but unfortunately I cannot accept it. While we love to make fun of Apple, the word 'apple' is somewhat too ubiquitous to always replace it (e.g. an apple a day keeps the doctor away).
Thanks a lot anyway!
Best,
Kenny


Update to use tokio

use 'tokio' instead of 'tokio-core' and 'tokio-io', also update the examples and add a few doc comments. I would like to investigate the clippy build failing before this is merged.

I want to get this merged so i can begin working on xi-term tonight I'm going to ignore clippy for now and try and fix it later on next week


feat (cursor-pagination): move validate pagination to apollo-utils

Goal
move the list-api validate pagination logic to apollo-utils so we can share it between list api and user-list search
paired with @nina-py

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix TSLint warnings in msbot-list.ts

Fix the following warnings in the file msbot-list.ts:

typedef
no-any
interface-name
no-empty
prefer-const
no-consecutive-blank-lines
newline-before-return
one-line

Pull Request Test Coverage Report for Build 163

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.668%

Totals

Change from base Build 150:
0.0%

Covered Lines:
2592

Relevant Lines:
3521

💛  - Coveralls

Replaces with https://github.com/Microsoft/botbuilder-tools/pull/386


Add Healthcheck to Harvey Nginx Docker Containers

Today, the nginx docker containers that act as a proxy do not have a health check. Add one so we can tell Docker if the containers are healthy or not and users can do with that what they will.

Added in a218823


non closing write operation in XChartPanel - saveJPGWithQuality

When saving the Chart within an XChartPanel as JPEG-File the write operation won't close and the file is accessible from outside not until the Java Program with the XChartPanel is ended.
Maybe the FileImageOutputStream within the Method saveJPGWithQuality should be closed manually or maybe a writer.flush() (writer = javax.imageio.ImageWriter) can help.

I just tested this by running BarChart01 from the xchart-demo module and it works fine. I'm on OSX. What OS are you running? Can you please try BarChart01 or tell me which demo I can run that you've noticed the problem?

@Chiamh I added code to manually close the output stream. I'd appreciate if you could test it. Thanks.

I found the problem using a QuickChart and a normal Chart within a small program of mine under Windows 7 SP1. With your small update the problem is fixed and it works within my own programs as well as in the xchart-demo. Thanks for your quick help!

great! Thanks for reporting the problem!


[GEM Dataset] Added TurkCorpus, an evaluation dataset for sentence simplification.

We want to use TurkCorpus for validation and testing of the sentence simplification task.

Thank you for the feedback! I updated the code.

Thank you for the feedback! I updated the code.


Rename report methods

The report methods aren’t indicative of what they’re doing.  They should be named something like format.

fixed in 58955bd.


Dashboard simulator, calculator enhancements

which package/s did you face the problem with?
Dashboard
What happened?
Set the value of HDD disk to 0 by default

a warning message in the simulator that contains IMPORTANT: Farming stops once 1 Billion TFT reached (new TFT)

Issue update
Work done

Added an alert message in the SimulatorView component,
updated the initial value of the HDD disk to be 0 instead of 100.

The alert has been added.

But the HDD initial value do you mean this one, because it's still 1000?

Verified, Devnet c98400f.
It was on the calculator side, not the simulator.
And now it accepts zero as its initial value.

TC2604 - Pricing calculator input validation

Verified:

Devnet
c98400f


[fix] Apply updated stuff (and other page properties)  when get_navigation_result_from_branch called

fix: https://github.com/sveltejs/kit/issues/4314
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Thank you. I made a change so that it only updates the page store when something has actually changed, rather than on every update


Refactoring required for management and state service extensions

@anthonydahanne : FYI, a new provider is responsible to expose the cluster tier manager now.
So defaultServerResource is not anymore under PoolSettings capability but under ClusteredTierManagerSettings .
PoolSettings will only expose things about pools now.

@ljacomet : for the refactoring into multi entities, perhaps you might want to reuse what @albinsuresh did for his needs: he had to introduce a provider for the cluster manager, which is the right thing to do (and is extensible) and also is something that you'll need anyway i think to split the providers ?


Set Up SAP Mobile Cards

Tutorial URL: https://developers.sap.com/tutorials/cp-mobile-cards-setup.html
Please specify the step you are referring to
Step 4
Iam unable to launch the API it is giving the following error message.
"This site can’t be reachedmobilecards’s server IP address could not be found."

Hi @AHADI2,
Thank you for raising the issue.
Can you please provide a response to the following questions:

Which OS flavour are you running the mobile cards client on? (iOS | Android)
Is your mobile device connected to your laptop when doing the demo?
Have you tried to uninstall and re-install the app on your device?

Warm Regards
Sandeep T D S

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Two projects can have the same name

Change uniqueness check to allow two projects to have the same name as long as they have
a different namespace.

LGTM. Thanks, @robolson!


Add contextual help menu like jupyter lab

In jupyter lab you can add a tab next to your editor which shows the function signature of the function you're currently writing / the one you clicked on latest. Pycharm has similar feature as well. I would love to be able to use it in vscode.
Example:

Function signatures are already shown in VS Code as tooltip:

If you want to have a larger window for that like in your screenshot, I think that should be provided through an extension rather than by VS Code itself.
The extension api certainly is extensive enough to implement that quickly (given that you already have a way to get the contextual help text).


Make view item dg

Also fixed a bug with using listi after view

Codecov Report

Merging #75 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #75      +/-   ##
============================================
+ Coverage     67.97%   68.02%   +0.04%
- Complexity      662      663       +1
============================================
  Files           112      112
  Lines          2114     2117       +3
  Branches        230      230
============================================
+ Hits           1437     1440       +3
  Misses          591      591
  Partials         86       86

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

.../seedu/address/logic/commands/ListItemCommand.java
100.00% <100.00%> (ø)
3.00 <0.00> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
88.80% <100.00%> (+0.18%)
53.00 <1.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c987c61...bb81f6f. Read the comment docs.

needs update of developerguide.md as well

Maybe rename as viewDetailedItem or viewDetails (similar to the command name) so that it is easier for the reader to follow.


Add a definition for hart

Following the discussions in the sw-dev forum, could you please add a definition for hart, possibly with some explanations and history how this name was introduced?
Also a better definition for core would help, possibly with an explicit differentiator between core and hart, since, for single hart cores, people tend to mix their names (including the SiFive manuals), increasing confusion.
I guess the definitions should target both simple readers, for a clean logical level understanding, and silicon vendors, with the technical requirements for a compliant implementation.

Added text to hopefully make execution environments and hart definitions clearer as part of push for ISA ratification.


Error pulling image "public.ecr.aws/v2k0k1b1/yelb/yelb-ui:latest"

Might want to update the registry/image.
Failed to pull image "public.ecr.aws/v2k0k1b1/yelb/yelb-ui:latest": rpc error: code = NotFound desc = failed to pull and unpack image "public.ecr.aws/v2k0k1b1/yelb/yelb-ui:latest": failed to resolve reference "public.ecr.aws/v2k0k1b1/yelb/yelb-ui:latest": public.ecr.aws/v2k0k1b1/yelb/yelb-ui:latest: not found

+1

I am having similar issues - Error response from daemon: repository public.ecr.aws/v2k0k1b1/yelb/yelb-ui not found: name unknown: The repository with name 'yelb/yelb-ui' does not exist in the registry with id 'v2k0k1b1'

blog author here. I am no more with AWS, but I'll suggest to raise a ticket and someone from AWS team will look into issue. I believe container repository is disabled/deleted.

Do you have the dockerfiles so that we can build and publish ourselves?


[Bug]: Table | Unable to select a row with a click on the Image column

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://user-images.githubusercontent.com/91450662/185114771-42d1e407-f601-41bd-8d74-057866d29cdf.mov
Steps To Reproduce

D&D table widget, map some data with Image type column
Do not define the onClick property for the Image column/leave it as is
Observe : Clicking on the image column does not select the row
Expected : Clicking on any column should select the table row, irrespective!

Public Sample App
No response
Version
Prod/Release v1.7.14

Hello @carinanfonseca , can I work on this issue?

Hello @carinanfonseca , I have raised a PR for this issue. can you assign anyone to review this PR.
Thank you in advance.


Add .margin to Approx

Description
Add .margin to Approx.
I'm testing to porting our tests from catch to doctest. We use CHECK(var == Approx(1.123).margin(1e-3)) often but this is missing from doctest. Would it be possible to add it?
Many thanks
Extra information

doctest version: v2.3.7

Sorry for the late reply. It would be possible but the Approx class has lagged behind the implementation in Catch2 so I cannot promise when this will be done. One thing you could do for now is to extract the Approx class from Catch2 (it doesn't seem coupled with the rest of Catch2) and use it with doctest.

As discussed here https://github.com/doctest/doctest/issues/389 , this is indeed a good feature but should probably go into a new class.


Allow to pass additional options to .build or .create

Hi
I think that might be a good idea to allow .build or .create take additional options that are then accessible in the factory block.
For example, I want to specify if the sample attribute samples are fetched from a sequence or are taken randomly.
At the moment I see no way I can do it via arguments.
The usage may be something like:
# General form. Optional second hash is an extension.
FactoryGirl.build(:article, {...}, {...})

# Example of the actual call. `:custom` is the key for user options.
FactoryGirl.build(:article, {...}, {:custom => {:random_samples => true}})

FactoryGirl.define do
  factory :article do |f|
    p f.custom
    # {:random_samples => true}

    ...
  end
end

Transient attributes do this already, outlined here: https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md. You declare attributes as transient and they won't be assigned on the instance when created, but you'll have access to their values within callbacks and dynamic attributes.

Yes, that should be sufficient for my case, thanks for pointing out.


Pulsar Proxy No active broker is available

Describe the bug
I tested pulsar proxy. One proxy found 3 brokers through ZK service. My producer connecte to proxy.
Pulsar v2.5.2
To Reproduce
Steps to reproduce the behavior:

Stop 1 broker service, and the client can still produce normally

Stop 2 broker services, and the client can still produce normally

When 3 broker services are stopped, the client prompts that No active broker is available

Start a broker service, and the client prompts that No active broker is available

Start two broker services, and the client prompts that No active broker is available

Start three broker services, and the client prompts that No active broker is available

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]

Additional context
Add any other context about the problem here.

Closed as stale. Please open a new issue if it's still relevant to the maintained versions.


Added Python Basic Programs 33-60

# Fix to  Issue #
===============================
Added 33 to 60 Python Basic Programs.

===============================
Checklist

[yes] I read the contributing guidelines
[yes] I structured my project as stated in the contributing guidelines
[] My project uses dependencies
[yes] I starred the repo
[yes] I had fun contributing!

Project reviewed and merged.
Thank you @harshpreet931
Feel free to add more projects.


rack.rb:89: warning: URI.unescape is obsolete

Expected behavior and actual behavior
No logs on after the server is starting
== The Middleman is loading
== View your site at "http://localhost:4567", "http://<IP_ADDRESS>:4567"
== Inspect your site configuration at "http://localhost:4567/__middleman", "http://<IP_ADDRESS>:4567/__middleman"
/Users/johndoe/.rbenv/versions/2.7.0/lib/ruby/gems/2.7.0/gems/middleman-core-4.3.5/lib/middleman-core/rack.rb:89: warning: URI.unescape is obsolete
/Users/johndoe/.rbenv/versions/2.7.0/lib/ruby/gems/2.7.0/gems/middleman-core-4.3.5/lib/middleman-core/util/paths.rb:36: warning: URI.unescape is obsolete
Steps to reproduce the problem (from a clean middleman installation)
Init and serve a new project with ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-darwin19]
Additional information

Ruby version: 2.7.0p0
Middleman version: 4.2
OS version: Mac OS 10.15.3

Fixed here, but not yet released: https://github.com/middleman/middleman/pull/2303
You'll need to use the stable branch for now: https://github.com/middleman/middleman/tree/4.x
Or downgrade to Ruby 2.6 for now

Still not quite fixed in latest 4.x:
/.../ruby/2.7.0/bundler/gems/middleman-e8cf86f3e335/middleman-core/lib/middleman-core/util/paths.rb:36: warning: URI.unescape is obsolete

Yeah getting that when updating to ruby 2.7

Any news for the 2.7 switch ?

for anybody interested I have this ugly hack to ignore these:
bundle exec middleman build &> output.txt; cat output.txt | grep -v URI; rm output.txt

for anybody interested I have this ugly hack to ignore these

…or simply override the affected places:
# until this is fixed https://github.com/middleman/middleman/issues/2312
Middleman::Util.module_eval do
  module_function

  def normalize_path(path)
    return path unless path.is_a?(String)

    # The tr call works around a bug in Ruby's Unicode handling
    WEBrick::HTTPUtils.unescape(path).sub(%r{^/}, '').tr('', '')
  end
end

Middleman::Rack.class_eval do
  # https://github.com/middleman/middleman/blob/master/middleman-core/lib/middleman-core/rack.rb#L90
  def process_request(env, req, res)
    start_time = Time.now

    request_path = WEBrick::HTTPUtils.unescape(env['PATH_INFO'].dup)
    if request_path.respond_to? :force_encoding
      request_path.force_encoding('UTF-8')
    end
    request_path = ::Middleman::Util.full_path(request_path, @middleman)
    full_request_path = File.join(env['SCRIPT_NAME'], request_path) # Path including rack mount

    # Run before callbacks
    @middleman.execute_callbacks(:before)

    # Get the resource object for this path
    resource = @middleman.sitemap.find_resource_by_destination_path(request_path.gsub(' ', '%20'))

    # Return 404 if not in sitemap
    return not_found(res, full_request_path) unless resource && !resource.ignored?

    # If this path is a binary file, send it immediately
    return send_file(resource, env) if resource.binary?

    res['Content-Type'] = resource.content_type || 'text/plain'

    begin
      # Write out the contents of the page
      res.write resource.render({}, rack: { request: req })

      # Valid content is a 200 status
      res.status = 200
    rescue Middleman::TemplateRenderer::TemplateNotFound => e
      res.write "Error: #{e.message}"
      res.status = 500
    end

    # End the request
    logger.debug "== Finishing Request: #{resource.destination_path} (#{(Time.now - start_time).round(2)}s)"
    halt res.finish
  end
end

thanks. But I don't want to do that I'd rather wait for an actual fix than patch in all that code.

@tdreyno, I admit I don't know the gem build process, but it looks like the fix for this issue is in master as of dead1c3, but it's not in the 4.x branch. Do you build the gems from the 4.x branch? Since it was committed to master in Dec. 2019 but not to the 4.x branch, is that why there is still an issue when using 4.3.10 and Ruby 2.7.1?
/Users/bb/.gem/ruby/2.7.1/gems/middleman-core-4.3.10/lib/middleman-core/util/paths.rb:36: warning: URI.unescape is obsolete

# gem info middleman-core

*** LOCAL GEMS ***

middleman-core (4.3.10)
    Authors: Thomas Reynolds, Ben Hollis, Karl Freeman
    Homepage: http://middlemanapp.com
    License: MIT
    Installed at: /Users/bb/.gem/ruby/2.7.1

    Hand-crafted frontend development

@b1tb4ng You're right. Backported. v4.3.11

$ ruby --version
ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-darwin18]
$ middleman version
Middleman 4.3.11
$ bundle version
Bundler version 2.1.4 (2020-01-05 commit 32a4159325)

I'm still getting a bunch of these:

~/.rvm/gems/ruby-2.7.2/gems/middleman-core-4.3.11/lib/middleman-core/builder.rb:232: warning: URI.escape is obsolete

This still isn't fixed for me either. I get lots of this error with Ruby 2.7.2:
<homedir>/.asdf/installs/ruby/2.7.2/lib/ruby/gems/2.7.0/gems/middleman-core-4.3.11/lib/middleman-core/extensions/asset_hash.rb:90: warning: URI.escape is obsolete

@tdreyno Will you release 4.3.12 to include #2383 to resolve two other warnings @mandrean and @marcoms pointed out? (https://github.com/middleman/middleman/issues/2312#issuecomment-717282210 & https://github.com/middleman/middleman/issues/2312#issuecomment-737167356)


unable to run gizmo; panic: oto: NewContext can be called only once

Hi!
I was very excited to read about Gizmo when you posted about it on Reddit a while back! The graphics style reminds me of Liero, a game my friends and I loved to play when we were younger.
So, when I saw you released the code for the game, I felt very excited to take it out for a spin :)
However, after the initial go get -u github.com/Lallassu/gizmo/..., running gizmo (from within $GOPATH/src/github.com/Lallassu/gizmo results in the following error:
u@x1 ~/g/s/g/L/gizmo> gizmo
[23:40:43.204][main.go:31][main.run][DEBUG] W: Rect(0, 0, 800, 600)
[23:40:43.242][sound.go:23][main.(*sound).create][DEBUG] Loading sound jump.wav
[23:40:43.242][sound.go:23][main.(*sound).create][DEBUG] Loading sound shot.mp3
[23:40:43.314][sound.go:23][main.(*sound).create][DEBUG] Loading sound shot2.mp3
panic: oto: NewContext can be called only once

goroutine 6 [running]:
github.com/hajimehoshi/oto.NewContext(0xbb80, 0x2, 0x2, 0x31fc, 0x0, 0x0, 0x0)
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:60 +0x311
github.com/hajimehoshi/oto.NewPlayer(0xbb80, 0x2, 0x2, 0x31fc, 0xc0004121d8, 0xc000514000, 0xc000475ff0)
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:89 +0x4d
github.com/faiface/beep/speaker.Init(0xbb80, 0xc7f, 0x0, 0x0)
	/home/u/goget/src/github.com/faiface/beep/speaker/speaker.go:42 +0x154
main.(*sound).create(0x7e2aa8)
	/home/u/goget/src/github.com/Lallassu/gizmo/sound.go:32 +0x56a
main.setup()
	/home/u/goget/src/github.com/Lallassu/gizmo/main.go:66 +0x12f
main.run()
	/home/u/goget/src/github.com/Lallassu/gizmo/main.go:43 +0x313
github.com/faiface/mainthread.Run.func1(0x6622f8, 0xc00001a120)
	/home/u/goget/src/github.com/faiface/mainthread/mainthread.go:37 +0x27
created by github.com/faiface/mainthread.Run
	/home/u/goget/src/github.com/faiface/mainthread/mainthread.go:36 +0xb1

Cheers,
Robin

Hi, this issue is already posted and seems to be a bug in Windows. I've only developed and tested on OSX. I will mark this a duplicate.
A workaround would be to remove the loading of the sounds.

duplicate of #2

Hi, this issue is already posted and seems to be a bug in Windows
Sorry for the duplicate bug report.
I'm running on Linux, though, so it seems the issue is more pervasive.

Ok good to know. Though it seems to be an issue with the sound loader so I will check that out later. Sounds are not activated yet so it can be removed at the moment.

Hi, this also should be fixed with the recent Beep update here

I get further now (with the latest version of Beep), could start the game menu but not a new game it seems. Also, when I run the game a second and third time, I got the following error:
u@x1 ~/g/s/g/L/gizmo> gizmo
[22:12:34.022][main.go:31][main.run][DEBUG] W: Rect(0, 0, 800, 600)
[22:12:34.042][sound.go:23][main.(*sound).create][DEBUG] Loading sound jump.wav
[22:12:34.042][sound.go:23][main.(*sound).create][DEBUG] Loading sound shot.mp3
[22:12:34.117][sound.go:23][main.(*sound).create][DEBUG] Loading sound shot2.mp3
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x3c pc=0x7fe327797bfa]

runtime stack:
runtime.throw(0x65ec43, 0x2a)
	/home/u/go1.12/src/runtime/panic.go:617 +0x72
runtime.sigpanic()
	/home/u/go1.12/src/runtime/signal_unix.go:374 +0x4a9

goroutine 4 [syscall]:
runtime.cgocall(0x5b56a0, 0xc000044d88, 0xc000322000)
	/home/u/go1.12/src/runtime/cgocall.go:128 +0x5b fp=0xc000044d58 sp=0xc000044d20 pc=0x427c9b
github.com/hajimehoshi/oto._Cfunc_snd_pcm_writei(0x7fe300049170, 0xc000322000, 0x42a, 0x0)
	_cgo_gotypes.go:160 +0x4e fp=0xc000044d88 sp=0xc000044d58 pc=0x55b4ce
github.com/hajimehoshi/oto.(*driver).TryWrite.func1(0xc0004162c0, 0xc00051c000)
	/home/u/goget/src/github.com/hajimehoshi/oto/driver_linux.go:140 +0x123 fp=0xc000044de0 sp=0xc000044d88 pc=0x55c2d3
github.com/hajimehoshi/oto.(*driver).TryWrite(0xc0004162c0, 0xc00051c0a0, 0x60, 0x8000, 0x100, 0x0, 0x0)
	/home/u/goget/src/github.com/hajimehoshi/oto/driver_linux.go:140 +0x1e0 fp=0xc000044e78 sp=0xc000044de0 pc=0x55bc80
github.com/hajimehoshi/oto.(*driverWriter).Write(0xc00041a020, 0xc00051c000, 0x100, 0x8000, 0x0, 0x0, 0x0)
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:136 +0xf4 fp=0xc000044ef0 sp=0xc000044e78 pc=0x55a984
io.copyBuffer(0x68d440, 0xc00041a020, 0x68d460, 0xc000416300, 0xc00051c000, 0x8000, 0x8000, 0x0, 0x0, 0x0)
	/home/u/go1.12/src/io/io.go:404 +0x1fb fp=0xc000044f68 sp=0xc000044ef0 pc=0x4b239b
io.Copy(...)
	/home/u/go1.12/src/io/io.go:364
github.com/hajimehoshi/oto.NewContext.func1(0xc00041a040)
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:79 +0x6a fp=0xc000044fd8 sp=0xc000044f68 pc=0x55bf7a
runtime.goexit()
	/home/u/go1.12/src/runtime/asm_amd64.s:1337 +0x1 fp=0xc000044fe0 sp=0xc000044fd8 pc=0x47ae01
created by github.com/hajimehoshi/oto.NewContext
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:78 +0x229

goroutine 1 [select, locked to thread]:
github.com/faiface/mainthread.Run(0x6622f8)
	/home/u/goget/src/github.com/faiface/mainthread/mainthread.go:42 +0x13d
github.com/faiface/pixel/pixelgl.Run(0x6622f8)
	/home/u/goget/src/github.com/faiface/pixel/pixelgl/run.go:32 +0x57
main.main()
	/home/u/goget/src/github.com/Lallassu/gizmo/main.go:14 +0x2d

goroutine 20 [syscall]:
os/signal.signal_recv(0x0)
	/home/u/go1.12/src/runtime/sigqueue.go:139 +0x9c
os/signal.loop()
	/home/u/go1.12/src/os/signal/signal_unix.go:23 +0x22
created by os/signal.init.0
	/home/u/go1.12/src/os/signal/signal_unix.go:29 +0x41

goroutine 21 [chan receive]:
github.com/hajimehoshi/oto.(*Context).Close(0xc00041a040, 0x0, 0x0)
	/home/u/goget/src/github.com/hajimehoshi/oto/context.go:116 +0xd2
github.com/faiface/beep/speaker.Init(0xbb80, 0xc7f, 0x0, 0x0)
	/home/u/goget/src/github.com/faiface/beep/speaker/speaker.go:34 +0x2e8
main.(*sound).create(0x7e2ab0)
	/home/u/goget/src/github.com/Lallassu/gizmo/sound.go:32 +0x56a
main.setup()
	/home/u/goget/src/github.com/Lallassu/gizmo/main.go:66 +0x12f
main.run()
	/home/u/goget/src/github.com/Lallassu/gizmo/main.go:43 +0x313
github.com/faiface/mainthread.Run.func1(0x6622f8, 0xc0000760c0)
	/home/u/goget/src/github.com/faiface/mainthread/mainthread.go:37 +0x27
created by github.com/faiface/mainthread.Run
	/home/u/goget/src/github.com/faiface/mainthread/mainthread.go:36 +0xb1

Well, that's very interesting. I'll try the game on my system and let you know.


Add Yue Pan to Contributors list

Congratulations on your Pull Request! See you in the next class.


Use data-testid to select topLevelMenu

Use new selector data-testid  and fix this issue:

CI here: https://github.com/epinio/epinio-end-to-end-tests/runs/6974180653?check_suite_focus=true
The test still fails but that's another issue:


Add basic rules for Crossplane

Towards: https://github.com/giantswarm/giantswarm/issues/24737
This PR:

add basic rules for crossplane

Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Add Unit tests
[x] Request review from oncall area, as well as team (e.g: oncall-kaas-cloud GitHub group).
[x] Alerting rules must have a comment documenting why it needs to exist.
[x] Consider creating a dashboard (if it does not exist already) to help oncallers monitor the status of the issue.

LGTM, but why no UTs? I'd be happy to pair on them if you want.

Sure, let's do it! :)


Add webkit bug to filter css property

In Safari filter won't apply to child elements while they are animating. WebKit Bug.

Thanks!