"Query returned empty result" via mysql has gone away. should it be an error? retry?

It came through as an Application error for some reason :-( https://keboola.slack.com/archives/C09U3R1J4/p1527138787000014

https://keboola.slack.com/archives/C09U3R1J4/p1528157886000131 again

again:( https://keboola.slack.com/archives/C09U3R1J4/p1533248050000033

again https://keboola.slack.com/archives/C09U3R1J4/p1534371369000100

co to vlastne znamena? ze to db server nedal a odpojil sa alebo ta query skutocne vratila prazdne data? Inac pisal dalsi klient na zendesk takze by bolo dobre to nejak fixnut.

este v logu jobu vidim ze to nejaky retry skusilo

No jeste to muze byt treba timeoutem VPN, ale to by jim delalo porad po stejny dobe.

So what is happening here is that it hits \ErrorException on the old Empty row packet body. and then retries and gets mysql has gone away, which it doesn't retry, but should

another one https://papertrailapp.com/groups/23635/events?focus=971129224817680385&q=9eee5cf081dcb929f184ad56bd341600&selected=971129224817680385

here's one where it hits empty row packet body and retries and the extractor thinks everything is ok but the result is missing 1/3 of the expected output:
https://keboola.zendesk.com/agent/tickets/9114

Mám dojem, že se kvuli tomu nestahly všechny data a zjištuji jestli je chyba u nas v DB nebo neco na chvilku vypadlo u vas.
Našel jsem k tomu tohle MySQL nastaveni, že by mohlo být duvodem "net_read_timeout"

For now i'd skip retry for empty row packet body before it gets fixed properly.

It's weird that the retry didn't downloaded all the data :/ Could be maybe something with MySQL internal cursor behaviour..
Looking into code. What about the @ suppressors that were they before? They're no longer needed?
https://github.com/keboola/db-extractor-common/commit/41007d32a941d0af393e51c956b685b591625825
Also, it would be great to confirm, that this issues are not caused by VPN timeout.

omg, how did the @ got there again?

We definitely removed all those.

Well, I don't know. It was there because the PDO doesn't only throw exception but also produced a warning or something like that? I'm not sure, that's why I'm asking.

Yes, it produces an ErrorException but those are being retried now.

if it's there it produces a warning and that gets hidden, so it continues without error. If it is not there the warning gets converted by error hander to exception and should be retried

i'll probably try the retry under debugger

i managed to do it once - the extractor wrote that it finished successfully and didn't download all the data. The problem is - i have no idea how - but it seems it's really possible :(

braindump: ok, got it - it's real hard on docker images, but not that difficult with RDS. When the connection is lost during fetch, the fetch is retried, but the $stmt cursor is no longer valid and so fetch returns false (no ex or warning) - i.e. for retry during fetch, the query has to be re-executed (and the csv file deleted).

omg, that makes sense.

Not sure about ETA, but current state is not acceptable - returning incomplete data and successful job. Can this throw a user exception before it gets fixed?

@ondrejhlavacek fix is here https://github.com/keboola/db-extractor-common/pull/71.  Will hopefully release today.

okay, thanks!


[release-v1.15] Add Konflux configurations

Add Konflux components and pipelines

hmm, it's the same? https://github.com/openshift-knative/net-istio/commit/270a56e94358e2fd78bc356cf6b93fd90a152257

hmm, it's the same? https://github.com/openshift-knative/net-istio/commit/270a56e94358e2fd78bc356cf6b93fd90a152257

yeah seems like a duplicate. let's close and see if it re-appears or if it was just timing.


Replace AG binary by tbd file

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2d795fb) 16.00% compared to head (5def421) 5.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #5       +/-   ##
==========================================
- Coverage   16.00%   5.00%   -11.00%
==========================================
  Files           7       5        -2
  Lines          25      20        -5
==========================================
- Hits            4       1        -3
+ Misses         21      19        -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Change master service internalType from LoadBalancer to ClusterIP

For security reasons we restrict the worker to master port 4200 to cluster internal by default with the option to override if necessary by the deployer.
Happy to discuss with you @stevelitras if this makes sense or not.

Brendan - I included this fix in the new helm stuff, but ended up doing it by hand - closing the PR, but the spirit made it in :)


[Filer] post add param:saveInside

What problem are we solving?
The s3 configuration file uploaded using the filer http port cannot be recognized
How are we solving the problem?
How is the PR tested?
Checks

[ ] I have added unit tests if possible.
[ yes] I will add related wiki document changes and link to this PR after merging.

The s3 configuration file uploaded using filerUi is also unrecognizable. Is it more appropriate to store all the files uploaded to the /etc directory in the metadata?


Prevent glob from accessing hidden files

Hidden files are often source control related (".git", ".repo", etc) or
editor related (vim's .*.swp), so are not guaranteed to exist, and may
be temporary. The build system shouldn't be using these files by
default.
If the glob pattern explicitly uses "." at the beginning of a path
component, allow returning hidden files for that component. Because of
this behavior, non-wildcard globs remain unchanged.
The one behavior that cannot be handled anymore is including hidden
files in recursive globs.

We already support GlobWithExcludes, why not have the primary builder provide the list of excludes it wants instead of enforcing .*?

For my usecase, this is required during the bootstrap phase with minibp, in order to use subdirs to read all the Android.bp files. So nothing from the primary builder is available.
In general, this behaves very similar to the normal shell commands (globbing, ls, etc) -- ignoring hidden files unless specifically asking for them.

LGTM


Support QueryInput when handling renamed files

Fixes #488. VS Code has lots of file rename handling logic but it was only applied to FileEditorInput objects. This PR updates it to also apply to QueryInput

@MattIrv I'll merge for today's insider build


Have these Stubby features been added to Unbound?

The README says

Unbound can be configured as a local forwarder using DNS-over-TLS to forward queries. However at the moment Unbound does not have all the TCP/TLC features that Stubby has for example, it cannot support 'Strict' mode, it cannot pad queries to hide query size and it opens a separate connection for every DNS query (Stubby will re-use connections)

I'm wondering how true this is today with the many updates that Unbound has had.

Unbound cannot support 'Strict' mode

AFAIK this describes actual checking of the certificate chain, and I'm pretty sure Unbound does this.

Unbound cannot pad queries

There are pad-* config options which appear to control this.

Unbound opens a separate connection for every DNS query

https://dnsprivacy.org/current_work/ says "Unbound version 1.8.3 now supports EDNS(0) Keepalive and TCP connection management". The edns-tcp-keepalive config option appeared in 1.8.0 which appears to do this.
Unbound also appears to support TLS session resumption with the tls-session-ticket-keys config option.
Is this all correct to your knowledge? Certainly Stubby does things differently and there may be reasons to use it in combination with Unbound outside the above. Thanks!

I have not researched the differences in a long time. It’s quite possible Unbound has some (all?) the features now. Being how this README info is dated, I may make it more generic.
The details should be in the Unbound change log. I’m curious if Stubby has updated the docs that speak to the differences. From a quick look, the info looks similar to what it did years ago.
https://dnsprivacy.org/dns_privacy_daemon_-_stubby/about_stubby/
I don’t maintain either project so I’m not extremely familiar with the finer points of the implementations. I only offer a Docker option for convenience.

On Dec 10, 2021, at 11:01 PM, Craig Younkins @.***> wrote:

The README says
Unbound can be configured as a local forwarder using DNS-over-TLS to forward queries. However at the moment Unbound does not have all the TCP/TLC features that Stubby has for example, it cannot support 'Strict' mode, it cannot pad queries to hide query size and it opens a separate connection for every DNS query (Stubby will re-use connections)
I'm wondering how true this is today with the many updates that Unbound has had.
Unbound cannot support 'Strict' mode
AFAIK this describes actual checking of the certificate chain, and I'm pretty sure Unbound does this.
Unbound cannot pad queries
There are pad-* config options which appear to control this.
Unbound opens a separate connection for every DNS query
https://dnsprivacy.org/current_work/ says "Unbound version 1.8.3 now supports EDNS(0) Keepalive and TCP connection management". The edns-tcp-keepalive config option appeared in 1.8.0 which appears to do this.
Unbound also appears to support TLS session resumption with the tls-session-ticket-keys config option.
Is this all correct to your knowledge? Certainly Stubby does things differently and there may be reasons to use it in combination with Unbound outside the above. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

I'd like to correct my 3rd point above.
Stream reuse was added to Unbound here, merged Nov 24, 2020.
The edns-tcp-keepalive option is for TCP/TLS clients connecting to Unbound and does not affect Unbound's connection to upstream. The same is true for TLS session resumption with tls-session-ticket-keys

Thanks for the update.
For 1, Strict mode in Stubby means "require a TLS connection and authentication of the upstream" and also requires at least one of ls_auth_name or tls_pubkey_pinset be set. See here.
You're right that Unbound can check the certificate chain (authenticate upstream). This requires configuring a tls-cert-bundle to load CA certs. When configuring forward-addr, you also need to append a '#' and a name before Unbound will check the tls authentication certificates with that name. The Actually secure DNS over TLS in Unbound article does a good job of explaining this.
checking of the certificate chain
If Opportunistic mode is desired,  remove the tls_authentication: GETDNS_AUTHENTICATION_REQUIRED field and add additional transports to the dns_transport_list (e.g. UDP, TCP) . In Opportunistic mode authentication of the nameserver is not required and fallback to clear text transports is permitted if they are in the dns_transport_list.
For 2, Unbound can now pad queries. Support for that was added in 1.13.1, released February 9, 2021.
For 3, the info you provided is accurate. The stream reuse feature now provides that functionality.
I see Stubby as more of a proving ground where newer DNS features are often implemented and available before Unbound receives them. Certainly, the implementation is different. Using them together still provides benefit, but at least for these particular features Unbound has caught up.


Trigger: QuickFix menu on a problem

Describe the feature
Hi!
I wrote this over the weekend because taking the time to copy an error message is just too much work for me:
https://marketplace.visualstudio.com/items?itemName=haselerdev.aiquickfix
But - even GPT-4 isn't quite good enough to be consistently able to fix problems every time. I'd rather have the tool as a way to prompt Genie then decide if I like it or need to iterate. You interested in me doing a PR to add this?

Oh, I see that this github doesn't actually have the source code - but still, happy to help if you want it, or feel free to grab relevant code from my repo if you'd rahter incorporate it yourself.

@cahaseler this is interesting but correct we use this repo as a docs-only repository. Did you developed this extension to be a companion to Genie?

Sure, will record a video a little later. I developed it to fill a gap and to be used alongside Genie, or just as a standalone simple tool without all the complex functionality.
All it does is add a button to the little lightbulb "QuickFix" menu that appears on "problems" identified by linters/typescript - when you click it, it grabs the error message and the block of code around the error, and passes them to GPT, then writes back the result over the initial problem. It mostly works, but there's also times when I wish it would just use that menu as a prompt for Genie instead, because even GPT-4 isn't perfect.
I developed it because I found I was copy-pasting error messages from my problem list into the "ad-hoc" prompt for Genie a lot. And I wanted to see how easy it was to develop a vscode extension with GPT's help as a weekend project. :)

@cahaseler thanks for your suggestion and your project. We implemented this in Genie now and credited your work in the Readme. Please check out v0.0.5 and let us know if it helps

Awesome!


fastfetch: 2.20.0 -> 2.21.0

Description of changes
Closes #332426
Release notes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

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

Since there's a large diff with nixfmt I think we should update this like normal and wait until the treewide format.
Making a separate commit for nixfmt and updating .git-blame-ignore-revs is also possible.

Since there's a large diff with nixfmt I think we should update this like normal and wait until the treewide format.
Making a separate commit for nixfmt and updating .git-blame-ignore-revs is also possible.

Yes, please split formatting commit from the actual change.

I see in the release notes mention about libnm -> libdbus. Does anyone use the wifi module to validate? Seems like we should be adding libdbus as a linux dependency.

Result of nixpkgs-review pr 332464 run on x86_64-linux 1

2 packages built:

fastfetch
fastfetch.man

Result of nixpkgs-review pr 332464 run on aarch64-darwin 1

2 packages built:

fastfetch
fastfetch.man

@drupol Done


feat(prettier): correctly format the NumberLiteral

Current dependencies on/for this PR:

main

PR #1283   👈

PR #1284

This stack of pull requests is managed by Graphite.

Merge activity

Nov 13, 8:27 PM: @Boshen started a stack merge that includes this pull request via Graphite.


feat(client): refactor universal nav missing types

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

ref #48101
While adding the styles to universal-nav component, I needed to add type safety back to the file.
You may be wondering, how is this related? shhh, just accept static analysis

Edit: I forgot about auto assignment, I wouldn't have said the line above if I remembered :rofl:
To be serious, while working on this, I just added types to make my life easier

Hey, I saw the conversation about ids and whatnot. It's possible to add all but one (the one being ref) of the allowed attributes to the link helper in a straightforward way:
interface LinkProps
  extends Omit<
    DetailedHTMLProps<
      React.AnchorHTMLAttributes<HTMLAnchorElement>,
      HTMLAnchorElement
    >,
    'ref'
  > {

Reason being, GatsbyLink does eventually pass all attributes to the underlying a element.  That way you can treat it as what it is - an enhanced anchor element.

@ahmadabdolsaheb I haven't got strong opinions about how to use these props, it's just fair to say that they are there if you want them.

Either way works for me.


Handeling Footer in Testimonals Wall

What is the change?
The footer for testimonial-wall was not covering the full screen in desktop as well as mobile mode. Fixed it.
Related issue?
close: #948
How was it tested?
The code was tested in local environment at port 3000
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[X] Have you followed the Contribution Guidelines while contributing.
[X] Have you checked there aren't other open Pull Requests for the same update/change?
[X] Did you lint your code before making the Pull Request?
[X] Your submission doesn't break any existing feature.
[X] Have you tested the code before submission?

Screenshots or Video:
Add a screenshot or demo video if appropriate.

Hey @keshavgbpecdelhi can you post a before and after video of the issue? Since when I'm viewing it in my localhost, the footer is responsive.

What's the change in code?

@AnanyaNegi click on the pull request, and you can see the changes. Remember :
Green  color for editing new code.
Red color for removing previous.
Github allows everyone to review and watch the code edits.
And

Hey @keshavgbpecdelhi can you post a before and after video of the issue? Since when I'm viewing it in my localhost, the footer is responsive.

Yes, let me show it by video if you ask for that, but I don't think it's that hard to find, if you read the issue again and compare the screenshot with the live site testimonials-wall you can see the changes.
Thankyou

@AnanyaNegi ok I edited the before and after screenshot both in the PR description. I think I made it very much clear now. Still doubt you can ask. Happy to solve :)

PR is almost ready to merge. A few changes.

Commit message is absolutely not acceptable, please update it. You can use the --amend flag while committing to update the commit msg and then force push.
There are changes to the package-lock file which are not required, it related to lockfile version. Please read the README file to understand it better.
There is a binary file added in views folder by the name of .testimonial-wall.ejs.swp  which I don't get if its of any use. Comment if disagree

@Pranav016 Thanks, yes I agree with you.  Did the required changes. Let me know if something else needed to be done.


[TEVA-2011] Statuscake Terraform provider version bump

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-2011
Changes in this PR:
The Statuscake provider has moved. This PR

changes the name of the provider in use
bumps the version to 1.0.1

Next steps:
As per https://www.terraform.io/upgrade-guides/0-13.html, rename the provider name in the production state file with
terraform state replace-provider 'registry.terraform.io/hashicorp/statuscake' 'terraform.example.com/StatusCakeDev/statuscake'

Review app deployed to https://teaching-vacancies-review-pr-2882.london.cloudapps.digital


system/uorb: bug fix and support flush/get_events api and comiple in aosp

Summary

orb_flush: After you call orb_flush(), you can determine whether the flush is completed by listening to the POLLPRI event
of fd and getting the event in orb_get_events.
remove non-posix implement, include api and header file, let uorb compile in aosp.
bug fix

Impact
new api: orb_flush, orb_get_events
Testing
Vela

[Experimental Bot, please feedback here]
The PR summary is lacking some details. Here's a breakdown and how it can be improved:
What's Missing in the Summary:

Why:  The summary mentions what changes are made (orb_flush, removing non-POSIX, bug fix), but not why these changes are necessary.

Is orb_flush a new feature to address a specific need?
Why remove the non-POSIX implementation? Compatibility, simplification, etc.?
What bug was fixed and what problems did it cause?

How: A brief explanation of how orb_flush works would be helpful.
Scope:  What functional part of the code is affected? Networking, inter-process communication, something else?

Improved Summary (Example):
This PR introduces a new orb_flush API to improve synchronization in [briefly describe the relevant functional area, e.g., inter-process communication using ORB].  This addresses the issue of [briefly describe the problem orb_flush solves]. The non-POSIX implementation of [mention what] is removed to improve code maintainability and align with POSIX standards.  Additionally, a bug fix is included that resolves [briefly describe the bug and its impact].
Impact Section:

New feature: The impact section correctly identifies the new orb_flush API.
User impact:  You need to elaborate on how the new API affects users.  Will they need to change their code? Are there any compatibility concerns?
Build impact: Be specific if there are any changes to the build process, required dependencies, etc.
Hardware impact:  Does this affect any specific hardware architectures or boards?
Documentation: The impact section should state whether documentation updates are included in the PR or if a separate PR is needed.
Security:  If the bug fix addressed a security vulnerability, it needs to be explicitly mentioned here.
Compatibility:  Specify if there are any backward or forward compatibility concerns.

Testing Section:

Insufficient Detail: "Vela" alone doesn't provide enough information.
Specifics Needed:

Build Hosts: List the operating systems, CPU architectures, and compilers used for building and testing.
Targets:  Provide details about the target architectures (e.g., simulated, RISC-V, ARM) and the specific boards and configurations used.

Logs:  While full logs might be excessive, include relevant snippets that demonstrate the problem before the change and the successful outcome after the change.

In Conclusion: The PR needs more detail and clarity to meet NuttX requirements. Be specific about the changes, their impact, and provide comprehensive testing information.


Not working with the latest react version

Cannot install the extension. It's peerDependency ia only react versions up to 17.x.x
npm i react-social-login-buttons
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.0.0 || ^17.x" from<EMAIL_ADDRESS>npm ERR! node_modules/react-social-login-buttons
npm ERR!   react-social-login-buttons@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

@bobthebuilder1997 please try version 3.6.1


Logging: Default Entry "type" to "global"

The other client libraries (notably python) default the entry type to global. Our documentation does not do a great job of explaining what this should be (all examples log to gcs_bucket).
# Current
$entry = $logger->entry($text, [
    'type' => 'global'
]);
$logger->write($entry);

# Ideal
$entry = $logger->entry();

# Even Better
$logger->logText($text);

Originally I chose to require the monitored resource because every other client library except python/go did the same - thinking about it now this does seem like a pretty sane default that would help simplify the code.
Count me on board for this change. @jdpedrie WDYT? If it sounds good to you as well I'll get started on it.

:+1:


docs: describe CU VFS inode semantics

CompilationUnit does not directly represent soft/hardlink information for its inputs. Currently a reasonable assumption is that files under different paths are unrelated, even if their content is the same.
For most languages this doesn't matter, but for the C preprocessor it does:

the common #pragma once extension will allow both files to be parsed
obscure cases of current-file-relative #include resolution

The "obvious" interpretation gives incorrect behavior in practice, and the C++ indexer treats same-content FileInputs as the same file to avoid this (see IndexVFS::IndexVFS in kythe/cxx/indexer/cxx/KytheVFS.cc).
This patch codifies this interpretation. I don't expect any indexers to actually change, as most languages don't care and C++ already does the right thing. But this provides important guidance to non-Kythe consumers of Kythe data.

@kadircet hit subtle correctness issues with a C++ analysis based on KCD data.
After some discussion we landed on asking clients to use this model rather than complicating KCD's data structures to represent links directly. Google's internal build infra forces this interpretation for real compilations, and it seems workable.
Kadir, does this spell out the contract in a way that makes sense?

thanks, yes this LGTM!

https://github.com/kythe/kythe/pull/5927 specifically used those semantics to resolve issues around pragma once


Are you trying to mount a directory onto a file (or vice-versa)?

after upgrade to the latest version I suddenly get:
Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/data/appdata/haven/app" to rootfs at "/app/haven": mount /data/appdata/haven/app:/app/haven (via /proc/self/fd/6), flags: 0x5400: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

docker-compose

services:
  haven:
    container_name: haven
    build:
      context: ./haven
      dockerfile: Dockerfile
    env_file:
      - .env.haven
    volumes:
      - "./appdata/haven/db:/app/db"
      - "./appdata/haven/app:/app/haven"
    ports:
      - 3355:3355
    user: "${DOCKER_UID:-1000}:${DOCKER_GID:-1000}"
    networks:
      net:
    restart: unless-stopped

taking a look. i hate docker.

Should be fixed now in #20


Documentation, A good readme file and a good description needs to be written

The readme file was updated. More details were added. A sprint should be created for the next week, describing some very small, but running demo of the application.


[BUG][Storage] There's no check on case and resource existence when add existing virtual networks to a storage account

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/storage/resource-manager/Microsoft.Storage/stable/2023-01-01/storage.json#L2987
API Spec version
2023-01-01
Describe the bug
original issue: https://github.com/Azure/azure-sdk-for-python/issues/36071
It's now able to add a duplicated subnet regardless of the case sensitive as well as a non-existing subnet to a storage account through API which should not be allowed.
Expected behavior
For same subnet with different cases, should have a check or dont duplicate added it .
For a non-existing subscription's subnet, should have a check for the given Vnet ID.
Actual behavior
Same subnet with different cases and non-existing subnets can be added to the storage account without any block.
Reproduction Steps
Send request to: https://management.azure.com/subscriptions/{subscriptions}/resourceGroups/{resourceGroups}/providers/Microsoft.Storage/storageAccounts/{storage} with body:
{
    "properties": {
        "networkAcls": {
            "virtualNetworkRules": [{"id": {subnet_resource_id}, "action": "Allow"}]
        }
    }
}

Whether with a non-existing subnet_resource_id or a subnet name with different cases. the subnet can always be added to the storage account successfully.

Environment
No response

@ChenxiJiang333
The check should be done in server side, client side might don't have permission to check, and adding the check on server side can benefit all client side like Portal/Powershell/CLI/SDK.
For server ask, would you please raise a help ticket on Azurite portal to get help from related team: https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request#go-to-help--support-from-the-global-header.

@ChenxiJiang333
The check should be done in server side, client side might don't have permission to check, and adding the check on server side can benefit all client side like Portal/Powershell/CLI/SDK. For server ask, would you please raise a help ticket on Azurite portal to get help from related team: https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request#go-to-help--support-from-the-global-header.

Hi @gengyujie, I have no such option under my tenant, could you follow this direction to raise a help ticket?

Hi @yifanz7, could storage team have a look at this issue?

@ChenxiJiang333
The check should be done in server side, client side might don't have permission to check, and adding the check on server side can benefit all client side like Portal/Powershell/CLI/SDK. For server ask, would you please raise a help ticket on Azurite portal to get help from related team: https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request#go-to-help--support-from-the-global-header.

Has created the support ticket, thanks.

closed for already send feedback to service team.


Update teamcity diff checker

Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.

Putting this in main branch so I can test it with manual trigger


Exception with message: range scan failled... bug?

Hi,
This is pretty easy to produce: Modify the BtreeIndexTest.testBtreeIndex test. Increase the repeat in the invokeTest method (multiply by 10)
final int repeat = 10000000;
=>
java.lang.IllegalStateException: range scan failed... bug?
at btree4j.BTree.scanRange(BTree.java:365)
at btree4j.BTree.search(BTree.java:333)
at btree4j.BTreeIndex.search(BTreeIndex.java:136)
at btree4j.BTreeIndexTest.invokeTest(BTreeIndexTest.java:181)
at btree4j.BTreeIndexTest.testBTreeIndex(BTreeIndexTest.java:147)
...

By the way, thanks for the great work 👍.

Apparently multiplying the DEFAULT_IN_MEMORY_NODES by 8 and the BTREE_NODECACHE_PURGE_UNIT by 8 as well solves the problem. This seems like a strange side effect from what I understood to be a cache. But maybe my assumptions are wrong?

Thank you for reporting. Reproduced.
That's strange. Will check it in this weekend.

@tanneess Some dirty pages were not properly flushed.
fixed in #13 and release v0.9.1 on maven central. Please test it by v0.9.1 or master branch.


Don't show path completions

Currently, cmp-cmdline shows paths in the list of possible completions. As demonstrated in hrsh7th/cmp-path#40, however, it can cause a lot of unexpected behavior by adding lots of effectively duplicate completions to the completion list (see screenshots in mentioned issue).
A possible solution to this would be to do a similar thing to cmp-path, and show the trailing / in the completion list (to make it clear which things are directories) but don't make it complete when the user selects it. Then, when the user types a / to complete within the directory. Just like using cmp-path.
Another possible solution would be to try and remove directories from cmp-cmdline's completion output entirely. I do not know the details of cmp-cmdline's implementation, so I cannot say for sure, but I am guessing this would be the more difficuly option to find a proper implementation for. However, I believe it would solve the problem better than the first suggested solution does.
I like the first option from a functionality standpoint because it means you don't have to always press . or / to start a path completion from the cmdline (a very common task). However, I also like the second option from a code organization/project purpose standpoint because it would probably be a good idea to not implement cmp-path within cmp-cmdline.

@hrsh7th any progress for this issue? I noticed treat_trailing_slash was added and enabled by default, which does mostly solve the problem, so thanks for that! Now the only issue is there being a ton of duplicate completions in the menu when cmp-path is showing things. (https://github.com/hrsh7th/cmp-path/issues/40 also, but that's less problematic now.)
It seems like this would be possible to work around if we could hide cmp-cmdline items when cmp-path items are visible. Is this possible? I could see this being useful in other cases too.
I suppose the simple solution for now is to not use cmp-path in the cmdline, but I would like to use cmp-path if possible because it is nicer to use.


[vpj] Fix regression in partial update + incremental push input schema detection

[vpj] Fix regression in partial update + incremental push input schema detection
Fix the regression in checking whether input schema is a update schema or not.
Previously it is using NO_OP_ON_FIELD.name(), while it should use .getName() instead to return "NoOp"
How was this PR tested?
New unit test and new integration test.
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

We need to have an integration test to test inc push + write compute with the new format and the existing format (AKA, using write compute schema).

We need to have an integration test to test inc push + write compute with the new format and the existing format (AKA, using write compute schema).

They are both here now.
This PR adds the integration test for old format, in the previous PR that adds new format support, I already added integration test for new input format.


What is the purpose of Threading.Channels?

Threading.Channels looked promising at first but after testing them in different scenarios - they behaved not much better, but sometimes worse than the simple BlockingCollection. And they block the thread on writes as well. I also tried to replace the Channels with a BlockingCollection in the AsyncFileWriter https://github.com/electricessence/AsyncFileWriter and with BlockingCollection the results were a bit (10%) better than with the Channels.
Also i created sample projects https://github.com/BBGONE/TestThreadAffinity
which compare Threading.Channels with the simple BlockingCollection : 	ThreadPoolTest (simple BlockingCollcetion) and  ThreadingChannels (replaced the BlockingCollection with the Channels).

Oops, it seems i figured it out. Although, the performance is the same in all usages, the channels have the benefit of nonblocking wait for when the channel can be written. This happens in the bounded scenarios. The BlockingCollection blocks the thread - not allowing to write, the Channels leave the thread to be used by others.

I checked it like this:
foreach (var num in partion)
{
                    ++cnt;
                    var enqueTask = svc.AddToQueue(CreateNewPayload(), Interlocked.Increment(ref SEQUENCE_NUM), token);
                    if (!enqueTask.IsCompleted)
                    {
                        Console.WriteLine($"Patrition #{partion.Key} On Count: {cnt} waiting async");
                    }
                    await enqueTask;
                    Interlocked.Increment(ref sentMessagesCount);
                }

Although it behaves as expected while waiting for the channel can be written One question remains open: Why the performance dropped 2,0- 2,5 if to the channel is written in 4 concurrent threads.
The performance drops with the BlockingCollection as well. But i think that with the channels it can be optimized.
If it can not be optimized, then close the issue please.
I will reopen it for checking this.

Why the performance drops 2,0- 2,5 if to the channel is written in 4 concurrent threads.

Because there's contention.  If you can optimize it, please submit a PR to do so.  I'm going to close this issue, though, as it's not actionable as is, and the right repo now for channels-related issues is corefx.  Thanks.

the channels have the benefit of nonblocking wait for when the channel can be written

And asynchronous reads when channels are empty.

Sorry. I converted it to CoreFX. It started to perform 2,5 times faster - with the same code and the same settings
I did not know that it had so big influence on the perfomance.


powershell scripts are not working

The script in section https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-modify-group-writeback#disable-automatic-writeback-of-all-microsoft-365-groups does not work. Also it is not in a correct format. In my opinion it does not work at all. It would be great to describe the scripts like this article: https://docs.microsoft.com/en-us/microsoft-365/solutions/manage-creation-of-groups?view=o365-worldwide#step-2-run-powershell-commands
Also the sync rule in https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-modify-group-writeback#delete-groups-when-disabled-for-writeback-or-soft-deleted does not work because of spaces behind the line break characters `. So please just remove the spaces that copy/paste works.
best regards

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

ID: 35863b41-f4b7-a772-b9d7-119b8eb08830
Version Independent ID: 2db4eafa-79cb-9401-baf6-6902a1593972
Content: Modify group writeback in Azure AD Connect - Microsoft Entra
Content Source: articles/active-directory/hybrid/how-to-connect-modify-group-writeback.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@slueders-bag
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thank you for your issue submission. Unfortunately, this appears to be a problem with the cmdlt and a support issue. Issues in this repo are for documentation issues. You can try one of the following links for additional help.
Microsoft Support: https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/
#please-close


Add kubeflow related sdk in notebook container.

Is your feature request related to a problem? Please describe.
We will need to include kubeflow-training sdk and few more sdk which might be present for other component like katib or kserve.
We have only included so far pipeline sdk.
Describe the solution you'd like
for e.g. pip install kubeflow-training
https://github.com/kubeflow/training-operator/tree/master/sdk/python
As a part of this issue, one should also look into other SDK which are present in kubeflow ecosystem.

We added training sdk but looks like we missed katib. Keeping this issue open

All kubeflow related sdks have been added!


handle indeterminate length proxy chains - fixes #449

@magiconair What are your thoughts on this as a solution for handling layered proxies?

I think this is the right approach.
The chain of ips you have to validate is client, proxy1, proxy2, ...., proxyN. The chain is composed of the xff header and the remote ip of your incoming connection which cannot be spoofed. Also, previous proxies only append the remote ip of their incoming connection to the xff header.
For this to work you have to trust a chain of proxies in front of you. The end of your trust chain contains the ip address of the inbound connection you care about and it cannot be spoofed. If someone sends a bogus xff header from an ip that is not in the list then this ip will be added to the xff header by your trusted proxy and the other trusted proxies in front of you will pass on that information to fabio which will see the ip and deny access.

With that in mind I wouldn't call this a work around. This is how this should work I think

I updated the commit description and PR title to reflect the discussion.

If someone sends a bogus xff header from an ip that is not in the list then this ip will be added to the xff header by your trusted proxy and the other trusted proxies in front of you will pass on that information to fabio which will see the ip and deny access.

Yes, exactly which is why I settled on validating all the elements contained within the xff header.  If some client want's to be clever and inject a bogus XFF header, including a header of an allowed netblock, they will still be denied.


Unable to include external libraries

Hi,
I try to include about informations of external libs I use, but I'm not able to get them shown. Here's the code snippet I used (I tried to visualize the lib "Otto" in several ways but nothing):
new LibsBuilder()
                    .withFields(R.string.class.getFields())
                    .withActivityStyle(Libs.ActivityStyle.LIGHT_DARK_TOOLBAR)
                    .withAboutIconShown(true)
                    .withVersionShown(true)
                    .withAboutDescription("Testbeschreibung")
                    .withAboutVersionShown(true)
                    .withLibraries("AboutIt", "Otto", "com.squareup.otto", "otto-1.3.8")
                    .withAboutAppName(getString(R.string.app_name))
                    .withLicenseShown(true)
                    .start(this);

What am I doing wrong?

@sparkitny just checked. There is a definition file for otto. and using .withLibraries("Otto") should work. Perhaps .withLibraries("otto") (if this works I have to check if everything is done case insensitiv)
Otherwhise. Do you see other libraries?

Ok,
it works on my android phone. With the emulator it doesn't work. Another lib I use is https://github.com/EverythingMe/easy-content-providers. Using providers-android it unfortunately doesn't show up.

@sparkitny it just doesn't show up on your emulator? ok that's weird. Shouldn't make a difference. Perhaps the built in caching of auto detected libraries gets active here.
Regarding easy-content-providers. There is not yet a definition file for this library.
You can easily create new definition files with this easy web tool:
http://def-builder.mikepenz.com/
All files are in here if you want to find one:
https://github.com/mikepenz/AboutLibraries/tree/develop/library/src/main/res/values

Hi,
now it works perfectly. I don't know what happened to the emulator, but after a complete restart of my pc it accepted the code changes and now shows the libraries.
Thanks for this lib,
Sebastian

@sparkitny perhaps instant run issue?

Perhaps,
Now it works perfectly
Thanks
Am 14.02.2016 12:33 nachm. schrieb "Mike Penz"<EMAIL_ADDRESS>
@sparkitny https://github.com/sparkitny perhaps instant run issue?
—
Reply to this email directly or view it on GitHub
https://github.com/mikepenz/AboutLibraries/issues/187#issuecomment-183876507
.


Upgrade to Bevy 0.14.0-rc.*

This PR allows transform-gizmo-bevy to be used with the newest Bevy 0.14 rc (currently at least rc.3). Closes #66.
Before we merge this PR, we should probably first wait until Bevy 0.14.0 is officially released, change the dependencies to point to that, and make sure this still works then.

Concretely, this implements the following migration steps:

Upgrade the bevy dependencies to 0.14.0-rc (which should always take the newest rc available)
Upgrade required rustc to 1.78.0 (from 1.77.0) as required by Bevy
https://github.com/bevyengine/bevy/pull/12313 : Add bytemuck & util as dependencies
https://github.com/bevyengine/bevy/pull/13489 : Rename camera.projection_matrix to .clip_from_view
https://github.com/bevyengine/bevy/pull/9202 : Match get_sub_app_mut as an Option
https://github.com/bevyengine/bevy/pull/12827 : Impl RenderAssets for GizmoBuffers instead of GizmoDrawData
https://github.com/bevyengine/bevy/pull/12889 : Rename Transparent3d dynamic_offset to extra_index
https://github.com/bevyengine/bevy/pull/12453 & https://github.com/bevyengine/bevy/pull/13277 : Use ViewSortedRenderPhases instead of RenderPhase

Looks like I forgot about the examples 🤔

Closing in favor of https://github.com/urholaukkarinen/transform-gizmo/pull/69.


Can I register the application using a different AAD tenant?

I can only see options to register the application in the current AAD tenant. Can I select a different tenant?

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

ID: 6f1c454d-73bc-5a73-5d3f-1b1ca7625044
Version Independent ID: 552b3f09-6650-e7f7-01f2-a702005b83e9
Content: Configure Azure Active Directory authentication - Azure App Service
Content Source: articles/app-service/configure-authentication-provider-aad.md
Service: app-service-mobile
GitHub Login: @mattchenderson
Microsoft Alias: mahender

@MikeWasson, If I understand your question correctly, as mentioned in the same document if the Azure Active Directory tenant you wish to use is different from the tenant with which you sign into Azure then follow the document section 'Configure with advanced settings' for advanced management options. Kindly let us know if you have any specific questions or needs more clarification.

@MikeWasson, Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Experiment/abstract class impl

So this ended up working pretty well. It's the version currently used to power the Packt Book Bot.
To do:

Move code for abstract base class to its own Maven artifact and publish it separately.
Make this code depend on the new artifact.
Document in README that OW users are to add a dependency on the abstract base class artifact and extend it to make their function


Mark build_test builds as bringup.

They require an additional dimension to be passed as part of the .ci.yaml configuration.
Bug: https://github.com/flutter/flutter/issues/125897
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Instead of marking them as flaky, can we simply add the os dimension to green the tree?


use of ldd and best practices improvements

Came across this neat project. I am the author of a similar but more comprehensive tool that can generate a full sbom.
Noticed a few improvement opportunities in this project.
 Below is an extract from the man pages for ldd.
 Security
       Be aware that in some circumstances (e.g., where the program specifies an ELF interpreter other than ld-linux.so), some versions of ldd may attempt to obtain the dependency information by attempting
       to directly execute the program, which may lead to the execution of whatever code is defined in the program’s ELF interpreter, and perhaps to execution of the program itself.   (Before  glibc  2.27,
       the upstream ldd implementation did this for example, although most distributions provided a modified version that did not.)

       Thus, you should never employ ldd on an untrusted executable, since this may result in the execution of arbitrary code.  A safer alternative when dealing with untrusted executables is:

           $ objdump -p /path/to/program | grep NEEDED

concatenated strings passed to os.popen
https://github.com/stolk/sotrace/blob/main/sotrace.py#L20
https://github.com/stolk/sotrace/blob/main/sotrace.py#L31
This coding style could lead to injection vulnerabilities and must be avoided. Construct the arguments manually with shlex split and use with subprocess methods that accept the command and args separately. There could be other better ways to implement this.
Good luck!

Thanks! I will try to fix these.
Owner/Director of Game Studio Abraham Stolk Inc.
Vancouver BC, Canada
@.***
On Mon, Apr 8, 2024 at 2:19 AM prabhu @.***> wrote:

Came across this neat project. I am the author of a similar but more
comprehensive tool https://github.com/owasp-dep-scan/blint that can
generate a full sbom.
Noticed a few improvement opportunities in this project.
 Below is an extract from the man pages for ldd.
Security
Be aware that in some circumstances (e.g., where the program specifies an ELF interpreter other than ld-linux.so), some versions of ldd may attempt to obtain the dependency information by attempting
to directly execute the program, which may lead to the execution of whatever code is defined in the program’s ELF interpreter, and perhaps to execution of the program itself.   (Before  glibc  2.27,
the upstream ldd implementation did this for example, although most distributions provided a modified version that did not.)
   Thus, you should never employ ldd on an untrusted executable, since this may result in the execution of arbitrary code.  A safer alternative when dealing with untrusted executables is:

       $ objdump -p /path/to/program | grep NEEDED

concatenated strings passed to os.popen
https://github.com/stolk/sotrace/blob/main/sotrace.py#L20
https://github.com/stolk/sotrace/blob/main/sotrace.py#L31
This coding style could lead to injection vulnerabilities and must be
avoided. Construct the arguments manually with shlex split and use with
subprocess methods that accept the command and args separately. There could
be other better ways to implement this.
Good luck!
—
Reply to this email directly, view it on GitHub
https://github.com/stolk/sotrace/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPE4ATJLK6V4LHAINJYM3Y4JOKPAVCNFSM6AAAAABF4KBPU2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZTANRYG43DQMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

So, the objdump command only shows the names of the dependencies, but does not resolve them, so I do not know where they are pointing to.


cage: 0.1.4 ->0.1.5

Description of changes
https://github.com/cage-kiosk/cage/releases/tag/v0.1.5
remove  CFLAGS = "-O0";  https://github.com/cage-kiosk/cage/issues/231

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test cage

Result of nixpkgs-review pr 245892 run on x86_64-linux 1

1 package built:

cage


[RHOAIENG-1574] - Add codeflare-sdk e2e tests

RHOAIENG-1574

Tests passed in jenkins build - https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/distributed-workloads/job/rhods-pipeline-dw/179/console

@jgarciao Could you please merge it ?


Lorax Launcher Fails with Unsupported Models Due to Adapter Loading Issue

System Info
Target: x86_64-unknown-linux-gnu
Cargo version: 1.70.0
Commit sha: N/A
Docker label: N/A
NVIDIA-SMI 545.23.06              Driver Version: 545.23.06    CUDA Version: 12.3
model_id = "bigscience/bloom-560m"
Information

[ ] Docker
[X] The CLI directly

Tasks

[X] An officially supported command
[x] My own modifications

Reproduction
run "lorax-launcher" without specifying model-id (it defaults to the "bigscience/bloom-560m") model or run any model not yet supported by lorax.
lorax-launcher
error message (client side):
{"error":"Request failed during generation: Server error: 'BLOOMSharded' object has no attribute 'load_adapter'","error_type":"generation"}
error message (server side):
2023-11-27T08:25:59.281666Z  INFO lorax_router::loader: router/src/loader.rs:146: adapter vineetsharma/qlora-adapter-Mistral-7B-Instruct-v0.1-gsm8k downloaded
2023-11-27T08:25:59.281719Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter vineetsharma/qlora-adapter-Mistral-7B-Instruct-v0.1-gsm8k status to Downloaded
2023-11-27T08:25:59.318818Z ERROR lorax_client: router/client/src/lib.rs:33: Server error: 'BLOOMSharded' object has no attribute 'load_adapter'
2023-11-27T08:25:59.318826Z  INFO lorax_router::loader: router/src/loader.rs:201: FAILED loading adapter vineetsharma/qlora-adapter-Mistral-7B-Instruct-v0.1-gsm8k
2023-11-27T08:25:59.318833Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter vineetsharma/qlora-adapter-Mistral-7B-Instruct-v0.1-gsm8k status to Errored
2023-11-27T08:25:59.318862Z  INFO lorax_router::loader: router/src/loader.rs:271: terminating adapter vineetsharma/qlora-adapter-Mistral-7B-Instruct-v0.1-gsm8k loader
Expected behavior
When an unsupported model is run, Lorax should revert to its original implementation, which does not attempt to load an adapter for models that are not supported. In cases where an unsupported model is run without specifying an adapter_id, the system should still attempt to generate a response. If an adapter_id is specified, Lorax should notify the user that the model is not supported, rather than crashing.

Thanks for reporting this, @malhajar17. This should be a quick fix we can have done in the next couple of days, if it's not urgent.

Added #76 which should address this issue.


⚠ DesiTorrents has degraded performance

In e4525b6, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1664 ms

Resolved: DesiTorrents performance has improved in 59ee17c after 6 minutes.


attach linux arm64 artifact

Update the build to attach the linux aarch_64 binary as an artifact to the build (so it will be published to Maven repository on deploy) and bump the version of protoc used to generate Java code to 3.21.9 (same version we're pulling it at runtime).

relates to #680


Skip to main content

Fixes #798
Changes: Added skip to main content link, a lot of changes have been committed due to the code formatter.  It has just fixed all the spacing and format of the code.
I have checked the functionality, everything's running fine.

@lmccart It doesn't seem to be working properly, but it works fine in my local build. I checked and believe that it has been merged properly too. Can't find the problem.

@lmccart I will try to push another pr later.

@all-contributors please add @pcgamer1 for code

@lmccart Please add me as a contributor for code.


[BUG]: Nested Admonitions

If you want to ask for support or request features, sponsor the project and contact me over email.

[x] I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
[x] I have read the documentation and cannot find an answer.

Describe the bug
A valid MDX with nested Admonitions is displayed correctly, but it can not be created over the toolbar - no option to insert a new admonition when the focus is already inside a admonition.
Reproduction
Start from https://codesandbox.io/p/sandbox/mdx-editor-base-q8s7zr?file=/src/App.tsx to reproduce the problem so that I can observe the issue on my side and make sure that the fix reliably addresses it.
To Reproduce
Steps to reproduce the behavior:

Go to https://mdxeditor.dev/editor/demo
Go to the diff' viewer
Insert a nested admonition, e.g. the one from here https://docusaurus.io/docs/markdown-features/admonitions#nested-admonitions
Switch back to the Richt Text Mode and see that it works
Focus inside an Admonition
Bug: the toolbar does not show the option to insert another admonition

Expected behavior
A clear and concise description of what you expected to happen.
Step 6: Would be wonderful to insert nested admonitions :)
Screenshots

Desktop (please complete the following information):

OS: Windows.
Browser Chrome, Edge, Firefox

Additional context
Add any other context about the problem here.

This is an intentional design of the toolbar item; a nested admonition UI can easily get messy. You can easily do a toolbar item that does what you describe if you want to, just follow the source of the current one.


As a GP clinic management staff, I can customise tags for patients

...so that I can have control over the organisation of my patients.

Completed in 1.3


[ContainerRegistry] switch to use official TS code generator

and re-generate code.
The only manual change is in swagger\README.md. Rest are from generated code.

Did we use the same swagger file as before or were there changes to the swagger file too? Am wondering if it would be easier to review the PR if we used the same swagger file with the new code generator. Then we can attribute all the changes to the code generator and not the swagger changes

Did we use the same swagger file as before or were there changes to the swagger file too? Am wondering if it would be easier to review the PR if we used the same swagger file with the new code generator. Then we can attribute all the changes to the code generator and not the swagger changes

Yes same swagger.  The only manual change I made is switching from using my local autorest.typescript to the released package.

Lets go!


Provide an option to not auto-update ~/.profile

The one-line install is nice, but I don't like the fact that it goes about adding stuff to ~/.profile. I'd like an optional argument or something to disable that feature.
The reason I'm specifically finding this a bit annoying is cause I keep all my dotfiles in Dropbox and Git, and I've got rake tasks to symlink things, and install homebrew, rbenv, pow, and virtualenv-burrito. My dotfiles are already setup to load virtualenv if found, so when I setup a new machine I keep having to undo the changes to ~/.profile manually.

Thanks for the suggestion. A couple of questions.

Did you change how the startup incantation? What did you change it to? By default, it looks like this:

# startup virtualenv-burrito
$VENVBURRITO_esc/startup.sh

The reason I ask is the script looks for that in your .profile and if it finds it, doesn't add it again.
Thanks!

I install virtualenv-burrito after I installed my dotfiles, since I sync them down with Dropbox. However I don't have a massive ~/.profile file, instead it loads in a number of different *.sh files, like aliases.sh, git.sh, and of interest to you, python.sh which contains the following:
if [ -f "$HOME/.venvburrito/startup.sh" ]; then
    source "$HOME/.venvburrito/startup.sh"
fi

I guess that's why the install script adds it to my .profile file. Thinking about it though, I'd almost like to suggest that the default behaviour should be to print a information message about what to add to your .profile rather than just adding it without asking. And maybe having an option to force add it if it's not already in the .profile file.
However, I don't mean to be picky :)... At the end of the day it'd be cool if there could be a one liner install command which doesn't auto-update the .profile file, so I can run that command from my install:virtualenv rake task in my dotfiles :)

Functionality added, you can pipe in the script like this:
curl -s https://raw.github.com/brainsik/virtualenv-burrito/master/virtualenv-burrito.sh | exclude_profile=1 bash

Or, if you are saving the script and running it from the command line, you can pass in an argument like:
bash virtualenv-burrito.sh --exclude-profile

They're equivalent.


[Elements] Update Color structure

Closes #88

Clicking "Details" here should navigate you to see the before/after changes


container: Do not restrict path access for privileged containers

Privileged containers should see and reach all host paths.
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

LGTM


Invalid "sensors must be cleaned warning"

Describe the bug
similar to https://github.com/Tasshack/dreame-vacuum/issues/311 i got a notification, but in the app this option is not available
To Reproduce
after a clean i got a notification
Expected behavior
this notification should not be visible
Screenshots

Additional Information (please complete the following information)

Model Name: dreame.vacuum.p2027
Firmware Version: 4.1.8_1094
Home Assistant Version: 2024.1.6
Configuration Type [With or without map support]: with map
Errors or warnings shown in the HA logs (if applicable):

Dreame Bot W10

found it, thanks

I also have the same, I was just using the service call to reset the sensor... and it resets it as expected. So this functionality is actually exposed in the firmware, but it's not in the app for some reason.

That's interesting to know, I like to know background info like that, very useful!! Thanks

Also all of the European models (called L series) are actually the Chinese models of previous year. I know this because there are no L series in the code, only S and X series are present and L series are actually the combination of S models with X docks and vice versa. So when you buy a European model from EU, it is much likely that it is already EOL or very close to it. Also every hardware they made actually has two models, one for EU/US and one for the CN market but they are exactly the same hardware/software with the exception of a region lock. Model numbers and names not related of those devices for make it harder for consumer to compare them in the market.

Manufacturers are crazy. I suppose that they are using their Chinese customers as beta testers first, and if all goes well then they release to other regions. We don't get the W10 in the UK, so I imported mine from Germany. It's a great robot, it does a decent job of cleaning three types of different floors with no change required from me. Pretty much the only "issue" I have with it now is unable to reset the sensors from their app... It's really cool of you to share your knowledge like that, did you consider putting it in the Wiki or readme if not there already? I'm a geek so I love reading stuff like this :)

Thank you man!!


Pop up an error when creating a new VM

OS: Windows
Repro Steps:

Select "Create Virtual Machine..." --> Complete the prompts according to the notifications.
Check the result.

Expect:
Succeed to create the new VM.
Actual:
Pop up an error when creating a new VM like below screenshot:

Could you please try this again?  I think I fixed this issue with this PR. https://github.com/microsoft/vscode-azurevirtualmachines/pull/24 I changed the way I look for the SSH key.

When did you fix this? I'm seeing it now
5:59:00 PM: Creating resource group "mh" in location "eastus"...
5:59:02 PM: Successfully created resource group "mh".
5:59:02 PM: Creating new public IP addresss "mh-ip"...
5:59:52 PM: Created new public IP addresss "mh-ip".
5:59:52 PM: Creating new virtual network "mh-vnet"...
6:00:27 PM: Created new virtual network "mh-vnet"...
6:00:27 PM: Creating new subnet "default"...
6:00:58 PM: Created new subnet "default"
6:00:58 PM: Creating new network interface "mh826"...
6:01:03 PM: Created new network interface "mh826".
6:01:03 PM: Creating new network security group "mh-nsg"...
6:01:38 PM: Created new network security group "mh-nsg".
6:01:38 PM: Error: Command "type C:\Users\fivei\.ssh\azure_mh_rsa.pub " failed with exit code "1":
The system cannot find the file specified.

We verified this issue just now on Windows. And the new VM can be created with East Asia like below screenshot:

Same, it's working now and safe to close this.

Got it. It seems that this issue has been fixed. And we will close it for now.


FR: Inserting inline metadata where the text cursor is

Sometimes during threading mode, I'll "leave" a note with the hotkey/command, but what happens is that the inline metadata gets inserted at the top of the file. I'm wondering if it's possible to insert the inline metadata wherever the text cursor is? For example, I like doing something like:
from:: [[page A]]

page B contents....

*text cursor here*to::[[page C]]

Hey @crazymedguy :)
I like this idea. I'm wondering though, when should it happen? I guess there can be a setting to say "Always place it under the cursor", or "Always put it at the top"?

@crazymedguy this is working in the latest release!
You need to enable the setting Commands > Threading > Thread Under Cursor for it to work


[FSK-16]: Configure Cypress

This PR adds cypress configuration.

@erfan-goodarzi @ASafaeirad
I've had to downgrade nodejs because of this and this

@erfan-goodarzi  do a rebase with dev before merging


aspect ratio not honored

The asp argument is not being honored in plotting function calls.
set.seed(2576)
X <- runif(33)
Y<-seq(0,2,length.out = 33)
gs <- gsplot() %>% points(X, Y,
                          asp=1,frame.plot=FALSE) %>% axis(side=c(3,4),labels=FALSE) %>% grid()
gs

Produces this:

asp=1 should produce 1:1 axes.

labels seems to be having the same problem.

@wdwatkins when you make examples, can you make sure they are reproducible for us? I think that just means you need to include X and Y in your code snippet, but that helps us diagnose.

not sure that labels is broken.   I added them.
To be more specific with labels: it works in the axis call like above, but not when its inside points(), the same way asp doesn't.
gs <- gsplot() %>% points(X, Y, labels=FALSE                           ) %>% axis(side=c(3,4),labels=FALSE)

I'm trying to suppress initial labels on the main axes, so I can relabel them with an axis() call that places them between tick marks for the time/date axis labeling.

You shouldn't need to suppress them in order to change the labels:
gs <- gsplot() %>%
    points(1:10,1:10) %>%
    axis(side=1,at=c(2,4,6,8,10),labels=TRUE)
gs

That also moves the tick marks though.  I'm trying to keep the original ticks, and only add labels.  To get something like this:
at=axTicks(1)+((axTicks(1)[2]-axTicks(1)[1])/2),tick=FALSE)

I'm using this method: http://stackoverflow.com/questions/7530933/placing-the-x-axis-labels-between-period-ticks.  Maybe there's a better option though.

@wdwatkins do you want:
set.seed(2576)
X <- runif(33)
Y<-seq(0,2,length.out = 33)
gs <- gsplot() %>% points(X, Y, axes=FALSE ) %>% axis(side=c(1:4), labels=FALSE)
gs

That would be my suggestion for what you are doing...unfortunately, that doesn't actually work as expected for me:

This seems like a bug, right @lindsaycarr ?

Yes, that does seem like a bug. Something with axes=FALSE because this works as expected:
``r
set.seed(2576)
X <- runif(33)
Y<-seq(0,2,length.out = 33)
gs <- gsplot() %>% points(X, Y) %>% axis(side=c(1:4), labels=FALSE)
gs

![image](https://cloud.githubusercontent.com/assets/13220910/16526459/fc41c22a-3f76-11e6-9403-88cad7dd8e47.png)

See #370

asp still not working. Is this at all related to #366?

Saving for later because it's complicated (asp tells us to ignore one of the axis' limits) and not useful for repgen. Would be nice to have for ease of meeting SPN compliance.


API integration not working

https://github.com/akveo/ng2-smart-table/blob/master/src/app/pages/examples/server/advanced-example-server.component.ts      i am trying fetch data from an API like how its shown in the above link but its not working. it still shows no data found in the smart table . but in console it shows something like     "Please check that data extracted from the server response by the key '' exists and is array.
Error: Data must be an array."  can any 1 help .this is the api whic i used 'https://address-book-demo.herokuapp.com/api/contacts'

You should pass plain Json array without Contacts wrapper. Or specify array like this (include Total counter as well):
source: ServerDataSource;     constructor(http: Http) {     this.source = new ServerDataSource(http,          { endPoint: 'https://address-book-demo.herokuapp.com/api/contacts',          dataKey:'Contacts',         totalKey:'total'}); }

It's working fine . Thank you so much ...

how to do it work with HttpClient?

@sencis82  @MerkulovDev  can any one tell me how to use a pagination for server based data ...


Graphql fragments

I need to use fragment in graphql for reduce response data, but i can't find any doc or method for that.

@hadi-norouzi Hey man! Fragments don't need a special method, it's way to share graphql queries between each other.
You should create a item_fragment.graphql file, fill it with your fragment, generate graphql class using Artemis for example and then you can reuse your fragment.
Or simpler way just create a string. See more on fragments here https://graphql.org/learn/queries/#fragments. I would recommend you to close this issue.

I have this mutation string and fragment.
String searchOnMapMutation = r'''
  mutation map ($data: String){
    searchOnMap(data: $data){
      # searchLocationResultFragment
    }
  }
''';

String searchLocationResultFragment = r'''
  fragment locationRes on SearchOnMapRes {
    title
    region
    province
  }
''';

Can you help me for add this fragment to mutation string without using Artemis?

String searchOnMapMutation = r'''
  mutation map ($data: String){
    searchOnMap(data: $data){
       ...searchLocationResultFragment
    }
  }

  fragment locationRes on SearchOnMapRes {
    title
    region
    province
  }
''';

Something like that. You can export to constant fragment.
@hadi-norouzi

it's static, i want add another fragment to mutation and ignore this fragment for new events.  it's possible?
@SanjiKir

If you want to use this fragment only once and then use other fragments with the same mutation you have to create another mutation. I am not sure it's possible to dynamically substitute fragment in the Query, you can try template literals with ternary operators, but I would seriously recommend you to ask why are you even doing such things.
However, your questions are graphql related and not flutter related, I do recommend you to read the documentation and maybe find some playground to understand graphql itself a bit better.

@SanjiKir Thanks

Is it possible to pass fragments as variable into string queries?
const String graphQLFragmentBody =
    r'''fragment AuthFields on UsersPermissionsLoginPayload {
    jwt
    user{
      username
      email
      confirmed
    }
}
''';

const String graphQLDocumentRegisterUser = '''
    $graphQLFragmentBody
    mutation RegisterUser(
        \$username: String!,
        \$email: String!,
        \$password: String!
    ){
  register(input: { username: \$username, email: \$email, password: \$password }) {
    ...UsersPermissionsLoginPayload
  }
}
''';


[bug] Images in floating windows with boarders

Here is a rendered image with a window and buffer specified, and it has a position of x=0, y=1 (with 0 indexed rows). It displays fine in a window without borders, but the window relative positioning doesn't seem to account for borders, and it just draws the image over them.

I'm happy to submit a PR for this. I just want to check weather or not this is something you think image.nvim should handle, or if this is up to users to deal with. Personally I'd vote for respect window boarders as an on by default render option.

Hey, your suggestion sounds awesome!
If we're bound to a window and it has borders I think we should account for them.
The borders can only have a width of one cell, right?

Yup I see an error when trying to open borders with a width greater than 1

There's a potentially related issue with moving windows around that have borders. I have an output window with a border and an image in the window, and when I scroll the screen, the window moves, and the image jumps all over the place. I haven't been able to reproduce this with a window that doesn't have borders.
After recording this video, I realized that the windows are being redrawn when scrolling (semi rapidly) so it seemed like it might be related to #34, but this happens way more frequently, and only happens with bordered windows.
lots of flashing images in this video as always.
https://github.com/3rd/image.nvim/assets/56943754/515b71a9-c84b-4638-b90d-006b170400bf

Just noticed in the video that the extmarks seem to be cleared and re-created on render, will investigate.
Weird thing about the floating windows, but looks like it can help reproduce the deeper problem.
Thanks for all the effort!

I can no longer reproduce this reliably


editorState.getCurrentContent() returns an empty entityMap in 0.10.5

Do you want to request a feature or report a bug?
bug (or an undocumented feature ?)
What is the current behavior?
editorState.getCurrentContent() contains an empty entityMap
- Expected
+ Received

@@ -36,24 +36,11 @@
       "key": "4sa5t",
       "text": "Bonjour",
       "type": "unstyled",
     },
   },
-  "entityMap": Object {
-    "__add": [Function __add],
-    "__create": [Function __create],
-    "__get": [Function __get],
-    "__getLastCreatedEntityKey": [Function __getLastCreatedEntityKey],
-    "__mergeData": [Function __mergeData],
-    "__replaceData": [Function __replaceData],
-    "add": [Function add],
-    "create": [Function create],
-    "get": [Function get],
-    "getLastCreatedEntityKey": [Function getLastCreatedEntityKey],
-    "mergeData": [Function mergeData],
-    "replaceData": [Function replaceData],
-  },
+  "entityMap": Object {},
     "selectionAfter": Object {
     "anchorKey": "4sa5t",
     "anchorOffset": 0,
     "focusKey": "4sa5t",
     "focusOffset": 0,

{"blockMap": {"4sa5t": {"characterList": [{"entity": null, "style": Array []}, {"entity": null, "style": Array []}, {"entity": null, "style": Array []}, {"entity": null, "style": Array []}, {"entity": null, "style": Array []}, {"entity": null, "style": Array []}, {"entity": null, "style": Array []}], "data": {}, "depth": 0, "key": "4sa5t", "text": "Bonjour", "type": "unstyled"}}, "entityMap": {}, "selectionAfter": {"anchorKey": "4sa5t", "anchorOffset": 0, "focusKey": "4sa5t", "focusOffset": 0, "hasFocus": false, "isBackward": false}, "selectionBefore": {"anchorKey": "4sa5t", "anchorOffset": 0, "focusKey": "4sa5t", "focusOffset": 0, "hasFocus": false, "isBackward": false}}

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Here's a Jest test extracted from our tests suite:
import React from 'react';
import {shallow} from 'enzyme';
import Draft, {convertFromHTML, ContentState} from 'draft-js';

import BlockText from '../BlockText';

const exampleText = '
Hello
';
const blocksFromHTML = convertFromHTML(exampleText);
const state = ContentState.createFromBlockArray(
  blocksFromHTML.contentBlocks,
  blocksFromHTML.entityMap
);

describe('BlockText', () => {
  it('should not be empty', () => {
    const wrapper = shallow(<BlockText text={Draft.convertToRaw(state)}/>);
    // works in 0.10.4 but fails in 0.10.5!
    expect(wrapper.find('DraftEditor').prop('editorState').getCurrentContent()).toEqual(state);
  });
});

import React from 'react';
import PropTypes from 'prop-types';
import Draft, {Editor, EditorState} from 'draft-js';

function BlockText(props) {
  if (props.text) {
    const editorState = EditorState.createWithContent(Draft.convertFromRaw(props.text));
    return (

            <Editor
              readOnly
              editorState={editorState}
            />

    );
  }
  return null;
}

BlockText.propTypes = {
  text: PropTypes.object
};

export default BlockText;

What is the expected behavior?
editorState.getCurrentContent() should contain the complete state including entityMap
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
It was working in 0.10.4 but it does not work anymore in 0.10.5.
I've read the changelog but, as far as I understand, nothing should have changed ? 😉

It's actually not.
The object is hiding the actual entity collection and only making it accessible through the accessor functions.
Tools that then rely on the standard JSON access patterns such as testing frameworks or Redux, then display an empty object, because they cannot access the internals.


inkscape: extract_dir update to 1.1.2

Closes #7931

[x] I have read the Contributing Guide.

Thanks ♥


🐛 BUG:

Which Cloudflare product(s) does this pertain to?
Pages
What version of Wrangler are you using?
github vs studio
What operating system are you using?
mac
Describe the Bug
[ERROR] Error: Pages only supports files up to 26.2 MB in size
Purged however not able to update, old logs going on and on, while the files is updated below<29mB
website cookie cleaned
Old log:
b/wwwroot.tar.gz is 60.4 MB in size

Unfortunately this is a limitation of the Cloudflare Pages service—files can only be up to 25MB in size. Consider using R2 for storing large files (which doesn't have this file size limitation).


About Label

Hi, I want to test your trained model, but can you tell me what's your label corresponding?  Fake image's label is 1, and real image's label is 0?

Hi.
That's correct. We labeled fakes as positives, i.e. fake: 1, real: 0.
A higher output score indicates a fake.


Environmental damage not cleared in The Lodge

With the "Environmental Damage Restoration" setting at "Limited Resting", sleeping or waiting in The Lodge will not clear environmental damage as it should.

Details
The Lodge does not have any LocType keywords, which means that PEO_ENV_CND_EnvironmentalDamageClearing_Setting_2 [CNDF:002FAA04] did not account for it. I have modified this condition form to also specifically include The Lodge's location.
Game Version
1.13.61
Language
English (EN)
New Game
Yes
Reproduction Steps
Gain environmental damage, set "Environmental Damage Restoration" to "Limited Resting", attempt to wait or sleep in The Lodge.

SFCP_1115.zip


Add documentation re setting up environments

Also delete the extremely crufty jetty-config.md and some build.gradle cruft
Closes #344

Is it possible to update this example at https://github.com/kbase/auth2/blob/833e4aed898acc512e1264a4652ca224cd8aa036/README.md#administration as well?

I'm not sure what example you're talking about - can you be more specific?

Is it possible to update this example at https://github.com/kbase/auth2/blob/833e4aed898acc512e1264a4652ca224cd8aa036/README.md#administration as well?

Updated per our OOB conversation


Fix receivers on ValidateBasic funcs.

Description
Fix the receivers on the ValidateBasic funcs.
closes: #412

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

I merged this to unblock @scirner22


Ignore interactive upgrade message

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
$ az webapp
Describe the bug
I often get this output below. Is there a way of ignore the update warnings, like non-interactive mode? This output stops scripting:

New Azure CLI version available. Running 'az upgrade' to update automatically.
This command is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
Your current Azure CLI version is 2.47.0. Latest version available is 2.48.1.
Please check the release notes first: https://docs.microsoft.com/cli/azure/release-notes-azure-cli
Do you want to continue? (Y/n): Y

The docs says something about output & none, but I don't want to surpress output. I just need to ignore any upgrade questions when running in scripts.

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

Answering my own question:
https://learn.microsoft.com/en-us/cli/azure/update-azure-cli#automatic-update
$ az config set auto-upgrade.prompt=no


Move dependencies key to future versions

I think that dependencies need more work.

Dependencies should now go in references, so this can be closed.


Supress clean files output

Hey great plugin!
When the esbuild-clean-plugin cleans its files it prints out a list of all the files cleaned. I don't really need this information it just spams the logs. Can we have an option to supress it?
I didn't see anything obvious in https://nx-plugins.netlify.app/derived/esbuild.html#clean
I feel like this should be off by default then switched on by a verbose mode or something? Just a suggestion

Sure, I created this plugin in a great hurry, thank you for your feedback, I will make adjustments immediately.

This is supported in<EMAIL_ADDRESS>you can also refer to the docs: https://nx-plugins.netlify.app/derived/esbuild.html#configuration-2

Thank you for the speedy reply! I will try it out early next week and report back.

Hey I didn't want to make a new issue for this but your copy plugin is also verbose by default. Are you able to switch that to be false by default?

Hey I didn't want to make a new issue for this but your copy plugin is also verbose by default. Are you able to switch that to be false by default?

Sure, I used to set verbose option to be true by default because this plugin was not stable yet, clear logs help solve problems better, and now I think it can indeed be turned off.
Fixed in 1.3.0


Pasting doesn't work in Firefox on Linux

In Firefox 43 on Linux, trying to paste either text or an image via ctrl+v results in the text area losing focus (and I believe the contentEditable area gaining focus, though it's hard to tell), and no pasteImage/pasteText event being fired.  Also, any further key presses of v (without ctrl) behave as if ctrl+v was pressed.
Original issue: Reviewable/Reviewable#235

@pkaminski it's really only on linux, I'll install a vm to find out the causing

I reported the reviewable bug. I can reproduce it in the $('.demo-textarea').pastableTextarea() box in http://micy.in/paste.js/ . Let me know if I can help test something.

hi @SimonSapin ,  which distro of linux and which desktop environment are you using?

ah, great @SimonSapin . I like archlinux!

Archlinux x86_64, running GNOME 3 with X11.

Ping @layerssss -- have you made any progress on this? Thanks.

hi @pkaminski I going to fix it today or to tomorrow, haven't investigated the cause yet, but it shouldn't be a problem.

Awesome, thanks in advance!  Otherwise I'd have to add user-agent sniffing to reviewable.io to disable paste.js on Linux.

Hi what I've discovered is if I do copy an image from GIMP, it works fine. But if I copy an image from another webpage (right-click the image -> copy image), this issue occurred, and that's because Firefox only exposed the url of that image in this case. (This turned out to be not Linux specific, but Firefox specific, it can also be reproduced by ff on OSX and Windows.) So when this happens, it will only be  possible to retrieve image from a "CORS-enabled" url.
So now I added a new event pasteImageError, incase that having detected an image been pasted, but unable to retrieve it's data from just a url. You can see the usage of that in changes in the demo.
For example, if you go to http://www.centos.org/ , copy the logo, and paste, you will get that error, but you can still get that url if that would be useful for you.
If you copy the big wide image from http://www.merrypapery.com/texture , and paste, you will be able to get everything like previously expected. Because those images are hosted on a CORS-enabled CDN.
Last case, but the least, image pasting behaviour been detected, but no valid url nor data could be retrieved. (e.g. Safari)

fixed by:

96681d755953bd4493430e67ae85cf0290dbebbc (fixed the "broken" state)
0a6871fef6a7a09e7bd76d888c22c95d8685041f (added pasteImageError, try to fetch image from CORS-enabled hosts)

hi @pkaminski , I pushed a release 0.0.11, so if you are using package manager like bower you can update to that.

Thanks, I'll integrate the new release later tonight.  But did you also test pasting text?  Because that had the same effect as pasting images (lost focus, ctrl "stuck") and doesn't seem related to CORS.

Yes, text pasting fixed in a separate commit, it got to do something with the keyevent hijacking logic.

Has http://micy.in/paste.js/ been updated for both changes? I now have no problem type the letter V, but CTRL+V still moves focus away and doesn’t paste for me. (I can paste a selection with middle-click, but that’s always been the case, and it’s a "separate clipboard" form CTRL+C / CTRL+V.)

@SimonSapin It has been updated already. What is the image source? Did you copied an image from another program like GIMP, or just copied an image from anther webpage's context menu?

@SimonSapin if it's an image copied from another webpage, you would expect a few seconds before an error alert being shown and the "focus" being recovered.

alert like this,
 , if it's not the case, please tell me what's your image source

I’m copying text, in this case “Paste” from
Paste.js
 on the test page, with double click to select + CTRL+C to copy. I’ve tried waiting a full minute just to be sure, nothing happened after focus went away.
The original issue is also about copying text. I didn’t know Reviewable supported copying images before filing this issue, and I don’t expect to use that feature. (I haven’t used any image in code review so far.)

I deployed the new paste.js to Reviewable, and AFAICT things are now working correctly.  @SimonSapin, can you give it a try?  I think maybe the problem on http://micy.in/paste.js/ is just #19, for which I have a workaround on Reviewable.

@pkaminski Same as on the test page, CTRL+V still moves focus and doesn’t paste on Reviewable, but now I can type the letter V afterwards.

@pkaminski , I will try to integrate your workaround to fix #19

hi, @SimonSapin I'm unable to reproduce it currently, but I push another fix could potentially solve your problem (losing focus when pasting text). Could you please try it out on the demo page? And be sure to refresh the cache.

Still the same CTRL+F5, with devtools showing fresh requests with 200 response status codes being made for everything. I can type the letter V, paste text with middle-click, or right-click + "Paste" in the context menu, but CTRL+V moves focus and doesn’t paste.
http://micy.in/paste.js/paste.js has Last-Modified: Sun, 24 Jan 2016 14:55:06 GMT
However, I could not reproduce the issue in an new Firefox profile. Back in my main profile though, uBlock shows nothing blocked, disabling HTTPSEverywhere doesn’t help, and none of the other Firefox extensions should affect content.
Looking at the debugger for a bit, I’m suspicious of this code as it doesn’t seem to do any pasting:
        if (ctlDown && ev.keyCode === 86) {
          return paste._container.focus();
        }

While I was able to reproduce the original issue in a Sauce Labs Linux VM (don't know which distro they use), I can no longer do so.  Could your Firefox profile be corrupted in some subtle way, @SimonSapin?
The code you found is fine.  It focus a special off-screen element to receive the keystroke, which the browser then translates into a paste event (even if the clipboard holds an image).  This then triggers paste.js internal processing and a focus back to the original element.  Or at least it should.

Well, yes, as I said the problem does not happen in a new profile of the same browser, but “corrupted in some subtle way” is just a nice way to say we don’t know what’s going on.

This then triggers paste.js internal processing and a focus back to the original element. Or at least it should.

It sounds like this is where things are going wrong. Which part of the code is doing that? I can try debugging some more.

I believe the paste event is handled on line 186, and the focus back to the textarea after a text paste is invoked on line 151.  This is all in v0.0.11 which is what's currently integrated into Reviewable, but I'll try to integrate the latest 0.0.13 later this afternoon, see if it helps.

I integrated paste.js 0.0.13 into Reviewable, in case it makes a difference.

hi, @SimonSapin , could you try on the demo page ( http://micy.in/paste.js/ ) to see if you can reproduce the problem?

hi, @SimonSapin , could you set some breakpoints and see if they were executed as expected, and the order of execution.

https://github.com/layerssss/paste.js/blob/master/paste.js#L189
https://github.com/layerssss/paste.js/blob/master/paste.js#L213
https://github.com/layerssss/paste.js/blob/master/paste.js#L307
https://github.com/layerssss/paste.js/blob/master/paste.js#L162
https://github.com/layerssss/paste.js/blob/master/paste.js#L151

When I hit CTRL+C after setting breakpoints on all of these lines in the browser profile with the problem, none of them are executed. The code I quoted above is executed and I can step through jQuery triggering a focus event, but then nothing interesting seems to happen before control goes back to the event loop.

Thanks @SimonSapin , is that you mean CTRL+V ?  That'll be strange.

Yes, CTRL+V

Hi @SimonSapin , could you try on that demo-page( http://micy.in/paste.js ), but on the third box (that [contenteditable]), to see if text-pasting and image-pasting are functioning normally?

The third box works ok, but if I remember correctly it always has.

This issue should be fixed in 0.0.16 now


💡 Feature Request - Update Resource Group Names From POC

Describe the solution you'd like
In some of our module docs we need to create Resource Groups to deploy resources into. Today some of the Resource Group Names have references to POC etc. in them which we should change to something that aligns more to what we'd normally expect to see in a normal "production" deployment.
Some examples of POC nomenclature and differences in modules:

https://github.com/Azure/ALZ-Bicep/tree/main/infra-as-code/bicep/modules/spokeNetworking#deployment
https://github.com/Azure/ALZ-Bicep/tree/main/infra-as-code/bicep/modules/hubNetworking#deployment
https://github.com/Azure/ALZ-Bicep/tree/main/infra-as-code/bicep/modules/logging#deployment

As you can see in the community create hubPeeredSpoke module there is a good example of the resource group name being changed from POC: https://github.com/Azure/ALZ-Bicep/blob/main/infra-as-code/bicep/orchestration/hubPeeredSpoke/hubPeeredSpoke.bicep#L28
Describe alternatives you've considered
N/A
Additional context

Try to follow CAF naming recommendations and standards: https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/resource-abbreviations
Update pipelines and other places old Resource Group Names are used as well as modules and their docs
Review limits on Resource Group naming here: https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/resource-name-rules#microsoftresources

Naming Pattern Suggestion
rg-${parTopLevelManagementGroupPrefix}-hub-networking-prod-001 - Hub Networking Resource Group
rg-${parTopLevelManagementGroupPrefix}-logging-auto-prod-001 - Logging Resource Group

This is linked to PR #312

#312 closes this


Improve repository documentation

Fixes #69 and #67.
This PR brings the lagging documentation for configurable options up-to-speed. It also adds a contributing guide and code of conduct that will be surfaced on the pkdgown site homepage.
@jgockley62, would appreciate if you could check if the documentation is clear and easy to understand.

@jgockley62 can you clarify what you mean with modes in biomart?
I investigated separating optional arguments in #98 but all arguments are highly recommended.

Oh yeah sorry I wasn't clear. I wouldn't separate them just denote optional or required at the end of the description.


Migration from google code wicked-charts-wicket7 to adesso wicked-charts-wicket8

Hi guys,
We use google code wicked-charts-wicket7 for long time.
Now we want to upgrade to latest, so found new version in adesso wicked-charts-wicket8
But I couldn't use this as maven dependency, it is not found there.
I tried adding jcenter repository in maven setting.xml, it didn't work
Finally I was trying to download jar file for this dependency from https://oss.jfrog.org/artifactory/jcenter/. It asks credentials
I tried with default credentials admin/password, it didn't work.
Could you help us to sort out the problem?
<repositories>
        <repository>
          <id>central</id>
          <url>http://central</url>
          <releases><enabled>true</enabled></releases>
        </repository>

        <repository>
          <snapshots><enabled>false</enabled></releases>
          <id>jcenter</id>
          <name>jcenter</name>
          <url>https://oss.jfrog.org/artifactory/jcenter/</url>
        </repository>
      </repositories>

Development environment:
Java
Maven
Spring
Wicket
Highcharts

Hello, which version of Wicked-Charts are you using? Currently only versions 3.1.0, 3.2.0 and 3.2.1 are available in jCenter.

This is currently we use.
 <dependency>
                <groupId>com.googlecode.wicked-charts</groupId>
                <artifactId>wicked-charts-wicket7</artifactId>
                <version>2.0.1</version>
            </dependency>

Now upgrading to
 <dependency>
                <groupId>de.adesso.wicked-charts</groupId>
                <artifactId>wicked-charts-wicket8</artifactId>
                <version>3.2.1</version>
            </dependency>

wicked-charts-wicket8 is not resolved in maven project event after configuring the repository in maven setting.xml

Yes, you are correct and jCenter asks for credentials. This apparently has to do with the shutdown of jCenter. According to jfrog however, the repository should be available as read-only indefinetly, so I don't know exactly what credentials it needs either. Until the library is migrated to maven central, the only way to download it is through the GitHub releases, which is quite inconvenient. I will attempt to find out if jCenter can still be used.


docs: markdown element improvements and React bump

I ported some changes from the new styletron docs into base ui docs:

bump to the latest React (so we can use/test hooks in our docs)
inline code is styled similarly to GitHub
blockquote is styled too, so feel free to use it to highlight / quote some information
enable cross-linking, it uses prefetched next/link when internal, target="_blank" when external
markdown links use now baseui/link
heading anchors are styled similarly to GitHub, they appear only if you hover over the heading
more robust function when creating the anchor slug, it can handle a variety of special characters and even React Components (so if you put a link or inlined code into the heading it recursively extracts the text)

Before

After

Thanks for improving these! One issue I found: seems like the Example's headings are gone

Thanks for improving these! One issue I found: seems like the Example's headings are gone

Ooops. Looking into that.
Also unit tests are failing, need to update Enzyme and some tests.