[Hub Generated] Review request for Microsoft.CostManagement to add version stable/2019-11-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[x] PowerShell
[x] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[x] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

/azp run

/azp run

@elabicha Remove/rename property on old stable version is not allowed. Please go through breaking change review https://msazure.visualstudio.com/One/_workitems/create/Scenario?templateId=d1c215ec-5d6a-4cb3-a193-95ddd0a1ba96&ownerId=85bb92fb-21fe-41ca-9309-47f2c5f57515

@elabicha Any update?

@elabicha Any update?

Any hints on why this breaking change is made?

Any hints on why this breaking change is made?


Add the possibility to keep the cards of some players

Another thing I don't understand is about test. I have tests that fail related to game._deck. Sometimes _deck is NoneType. But I don't understand why.

@LucasColas I believe the error is from before the prehand where there is no deck attribute

Yes,
We can use deck I think.

I don't understand. Whever I run poetry run black . all files are left unchanged.

Everything is good for pytest, pylint and black.
However we need to add a new test for codecov.

Everything is good for pytest, pylint and black.

Nice. I could see about adding a test when I get back from vacation next week. Feel free as well to take a crack at adding a new test case. Perhaps in the tests.game.conftest copy predicate (which I have right now running random games and each predicate is run before and after each hand state).


[ffmpeg/gstreamer] Skip the bitrate_check of the specified case

Hi @uartie , can you review this for me?

Hi @uartie  , can you review this for me?

superseded by #490


just a test branch for getting ios working with fork of substrate

current error is
➜  sunshine-identity-ui git:(master) ✗ cargo make ios
[cargo-make] INFO - cargo make 0.30.6
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: ios
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: workspace
[cargo-make][1] INFO - External file not found or is not a file, skipping.
[cargo-make][1] INFO - Project: identity-client
[cargo-make][1] INFO - Build File: Makefile.toml
[cargo-make][1] INFO - Task: ios
[cargo-make][1] INFO - Profile: development
[cargo-make][1] INFO - Running Task: ios
[cargo-make][1] INFO - Execute Command: "cargo" "lipo"
[INFO  cargo_lipo::meta] Will build universal library for ["identity-client"]
[INFO  cargo_lipo::lipo] Building "identity-client" for "aarch64-apple-ios"
    Updating git repository `https://github.com/4meta5/substrate/`
    Updating crates.io index
error: failed to select a version for `smallvec`.
    ... required by package `libp2p v0.19.1`
    ... which is depended on by `ipfs-embed v0.1.1`
    ... which is depended on by `sunshine-client v0.1.0 (https://github.com/sunshine-protocol/sunshine.git#0d27a44e)`
    ... which is depended on by `identity-client v0.1.0 (/Users/4meta5/flutter-bin/sunshine-identity-ui/native/identity-client)`
versions that meet the requirements `=1.4.0` are: 1.4.0

all possible versions conflict with previously selected packages.

  previously selected package `smallvec v1.4.1`
    ... which is depended on by `frame-support v2.0.0-rc4 (https://github.com/4meta5/substrate/?branch=ios-ignore-sysinfo#d2a91440)`
    ... which is depended on by `substrate-subxt v0.10.1`
    ... which is depended on by `sunshine-client v0.1.0 (https://github.com/sunshine-protocol/sunshine.git#0d27a44e)`
    ... which is depended on by `identity-client v0.1.0 (/Users/4meta5/flutter-bin/sunshine-identity-ui/native/identity-client)`

failed to select a version for `smallvec` which could resolve this conflict
[ERROR cargo_lipo] Failed to build "identity-client" for "aarch64-apple-ios": Executing "/Users/4meta5/.rustup/toolchains/nightly-x86_64-apple-darwin/bin/cargo" "--color" "auto" "build" "-p" "identity-client" "--target" "aarch64-apple-ios" "--lib" finished with error status: exit code: 101
[cargo-make][1] ERROR - Error while executing command, exit code: 1
[cargo-make][1] WARN - Build Failed.
[cargo-make] ERROR - Error while executing command, exit code: 1
[cargo-make] WARN - Build Failed.

pls help SOS

This branch isn't intended to be merged. It's intended to get compilation on ios using a fork of substrate so I can justifying PRing with the changes.

continued in #3


Adding filestore support

Add Filestore support to Jupyterhub solutions template.
The decision was to have Filestore be a shared storage solution within Jupyterhub, meaning that all users will share a Filestore instance.
Added flag enable_filestore_use for enabling Filestore use.

@saikat-royc adding you here since for some reason I can't seem to add you as reviewer.

overall looks good with minor comments

I just realized that we are no longer using kubectl with terraform and are instead using helm to install. Going to work on changing it to helm.

Closing old issues with merge conflicts. Please rebase and re-open if still relevant.


Added drum-kit project

Simple project using JS -> animations & sounds using JS
resolve #4

Hi @prnvtripathi  your website looks amazing.
Can you add your details in the README.md in the JS section?
Like This
| Drum kit | [Demo](Demo Link) | [Source Code](/projects/javascript/Drum-Set) | [prnvtripathi](https://github.com/prnvtripathi)|

Thank you for contributing! 🚀

Please check @Mridul2820  and see if I can do anything else

Add this in the main README.md file
| Drum kit | [Demo](https://prnvtripathi.github.io/drum-kit/) | [Source Code](/projects/javascript/Drum-Set) | by [Pranav Tripathi](https://github.com/prnvtripathi) |

And Remove it from the projects/javascript/README.md file

Sorry for that error, i think it's resolved now @Mridul2820

Thank you for contributing! 🚀


Suggestions

[Enter feedback here]

For this section:
const { TableServiceClient } = require("@azure/data-tables");

Mention odata function if using it
const { TableServiceClient, odata } = require("@azure/data-tables");

For this section:
let result = await tableClient.getEntity("hometasks", "1");

Mention that if the entity is not found it will throw and error. For async methods a user can handle this by adding catch function.
let result = await tableClient.getEntity("hometasks", "1").catch((e) => {[insert error handling lines]});

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

ID: 168890e7-484f-6d2f-9eec-3a97dcd44e6f
Version Independent ID: ffd475a2-052e-7ce0-851a-622453a14010
Content: Use Azure Table storage or Azure Cosmos DB for Table from Node.js
Content Source: articles/cosmos-db/table/how-to-use-nodejs.md
Service: cosmos-db
Sub-service: table
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

@jtvcodes Thanks for your feedback! We will look into it and update as appropriate.

Thanks @jtvcodes for the feedback, we are looking at updating this article this month based on the changes you have recommended. I will keep this issue open until the changes are ready.


Feature/darus integration

Enables the publishing of ROARs to the DaRus Data Repository.

Contributor: Michael Zimmermann, @zimmerml

[x] Ensure that you followed our toolchain guide.

[x] Branch name checked. That means, the branch name starts with thesis/, fix/, ...

[x] Ensure that the commit message is a good commit message

[ ] Ensure to use auto format in all files

[ ] Ensure that you appear in NOTICE at Copyright Holders

[ ] Tests created for changes

[ ] Documentation updated (if needed)

[ ] Screenshots added (for UI changes)

Follow up at https://github.com/eclipse/winery/pull/682


Integrate/subsume the Aspera/http-gateway-sdk-js

Once we feel it's a good time (the sooner the better in my opinion), we can bring in all the HTTP Gateway SDK functionality here.
We can then try to come up with a cohesive API and iterate on new functionalities

This may be slightly hard since the Desktop SDK followed and copied some from the Gateway SDK so name collision would exist.
Would be good to see if we can alter the behavior of Desktop and HTTP Gateway to be used more without the end user seeing anything.  Like same calls (drag/drop, start, progress...) all work without any dupe calls or splits by consumer.
@dwosk maybe good to have a working session to see what we can do for this?


Missing license

Hi @pjbgf,
I see this project has been recently introduced in the go-git codebase. However, it doesn't include any licensing information.
Would you consider changing that?
Best regards,
Paweł

@pawelaugustyn good shout, adding it now.


Request: group.GetEntities(ICollection buffer)

Is there any reason we cant have Group<TEntity>.GetEntities(ICollection<TEntity> buffer)?
Having ICollection as the input argument would allow HashSets and other collections to be used which can be useful in certain situations.

Thanks! Good idea! Will add :)

@FNGgames Just checked, I use List<T> because of AddRange. Very convenient. Will see if I can replace that


Support VS2017 in xtool

Relevant magic:
https://github.com/bazelbuild/bazel/commit/321aa540feb2cd0583b824bbb646c885fda17f0b

Now the only VS supported.


rgb_to_hex.py bugs

Description : there is an error with variable name r, it is not consistent and refrenced as ra. also the max number of 255 is inconsistent, under variable ra it is 265. there is also no colon after the declaration of the method
Steps to Reproduce
to reproduce this bug you would change the variable name r to ra where first declared. also you would change the max amount from 255 to 265 within the parentheses. you would remove the ":" after the def of the method
Expected Behavior: I would expect the code not to run because there will now be undeclared variables being used later in the code. as well as the code not recognizing it as a method because there is no :.The colors will come out not expected because of the changed 255 number.
Actual Behavior The program did not run due to the missing colon after the def.
Screenshots If applicable, add screenshots to help explain the issue/bug.
Additional Information
Operating System: OS
Browser: IDLE
Version: 3.11.2
Possible Solution: add a colon after the declaration of method before the indented lines. change 265 back to 255, and change ra to r.
Priority How critical is this issue/bug:
high, unable to run until fixed
Reproducibility How often can this issue/bug be reproduced: this can be easily reproduced. it is an issue from the coder.
Impact What is the impact of this issue/bug: high impact, the code is unable to run until fixed

These fixes are correct:
Added a colon after the declaration of method before the indented lines.
Changed 265 to 255.
Change ra to r.


Update .env.sample

Added missing env vars:

NOSTR_PRIVATE_KEY
SLACK_BOT_TOKEN
SLACK_CHANNEL_ID
LNWITH_URL

Moved empty IMGPROXY env vars out of "LEAVE AS IS" section

Renamed sections

Updated documentation

Also LIST_MONK_AUTH?
I have not seen the slack vars... neat!


Add reference to Erlang library

I've discovered your resources. They looks very interesting to me. I am studying aspects of category theory and functional programming in cereal as well. However, I am doing it with prism of Erlang and Scala.
I've added a reference to my library that implements a few patterns which simplifies a functional programming in Erlang. I hope it will be useful of persons who is looking into you documentation. BTW, It would be interesting to get your feedback on my activity as well!

It looks like you are not willing to take any external contribution to this resource?

Hi, it was not on my radar :) LGTM


RISC-V compilation?

I saw recently there were updates to the rust/build.rs for the Rust bindings to get this library to compile in an sgx target. It would be really useful to have this library compile to theriscv32 target, which would be really useful within the context of zkVMs. Is it possible to add logic in the build.rs to allow this library to compile to the riscv32 target? (It should be similar to wasm which I saw was supported).
In particular, in getting the secp256k1 library to compile, this was helpful to add to the build.rs
 if env::var("CARGO_CFG_TARGET_ARCH").unwrap() == "riscv32" {
    const DEFAULT_RISCV_GNU_TOOLCHAIN: &str = "/opt/riscv";
    println!("cargo:rerun-if-env-changed=RISCV_GNU_TOOLCHAIN");
    let riscv_gnu_toolchain_path = env::var("RISCV_GNU_TOOLCHAIN").unwrap_or_else(|_| {
            DEFAULT_RISCV_GNU_TOOLCHAIN.into()
        });
  base_config
              .compiler("clang")
              .no_default_flags(true)
              .flag(&format!("--sysroot={riscv_gnu_toolchain_path}/riscv32-unknown-elf"))
              .flag(&format!("--gcc-toolchain={riscv_gnu_toolchain_path}"))
              .flag("--target=riscv32-unknown-none-elf")
              .flag("-march=rv32im")
              .flag("-mabi=ilp32")
              .flag("-mcmodel=medany")
              .flag("-Os")
              .flag("-fdata-sections")
              .flag("-ffunction-sections")
              .flag("-flto")
              .target("riscv32-unknown-none-elf");
}

so maybe something like that in this case would be helpful here too?

Note that the compiler path depends on the Linux (or MacOS or Windows) distribution method, and it's likely better to assume that it's in the path rather than in /opt.

It should be noted that it's perfectly possible to compile blst without C headers with contemporary gcc or clang. This means that you don't have to bother with --sysroot or anything of the sort. All you need is to pass -ffreestanding to a compiler [recognized by cc-rs] that can generate RISC-V ELF object modules. And the best part is that blst's build.rs does it for you already (last adjusted in 2899b7f). So the only question left is if cc-rs can spot a suitable compiler, and if doesn't on your system, you'd be expected to direct it to one. Easiest is to set the CC environment variable to clang. This is because the majority of clang installations are capable of generating RISC-V object modules, Linux, homebrew, Windows, including one provided by Microsoft as a Visual Studio option... With this in mind the question is rather what is it that doesn't work (past 2899b7f) for you. I mean do tell.

And the best part is that blst's build.rs does it for you already (last adjusted in 2899b7f).

To clarify, it's assumed that the commit in question should cover your target. At least it does cover riscv32im-risc0-zkvm-elf.

For public reference, it was confirmed that the issue is resolved by the referred commit, and by extension in latest version, "out of band."


[Feature Request] Add in additional metric for Consumer Lag

While the offset of CGs is useful, add in an additional metric around the CG's lag would be extremely beneficial for gauging the health of the CG in conjunction with the Commit metric being published.

I'm part way though implementing a high-water mark metric, which will allow you to compute the lag. Not sure when I'll have a chance to finish it, but it is coming...


Enhancement: Native schema inference for Data Connectors

Goal-State/What/Result
Each Data Connector uses the source's native mechanism to retrieve the table schema.
Why/Purpose
The current approach most data connectors have taken is to do a SELECT * FROM {table} LIMIT 1 and infer the schema from the returned results. As #2061 has demonstrated, this cannot be relied upon.
By When
A target date for done-done completion of the entire enhancement.
Issue/Spec written and reviewed:
Done-Done: Oct 18
Done-Done

[x] First Principles Driven
[x] The Algorithm
[x] PM/Design Review
[x] UX Review
[x] Threat Model / Security Review
[x] Tests
[x] Documentation
[x] Samples/Quickstarts/Tutorials
[x] Release Notes

The Algorithm

[x] Every requirement questioned?
[x] Delete (Scope) any part you can.
[x] Simplify.
[x] Break down into smaller iterations/milestones.
[x] Opportunities for automation.

Spec
Basic specification of the enhancement.
How/MAP
A clear and concise plan of how this should be implemented.
Implementation (issues)

[x] #2460
[x] #2461
[x] #2462
[x] #2463
[x] #2464
[x] #2465

QA

Target EOW: Done-done

Target EOW: Done-Done

This is done!


Downloader Crash Downloading Specific Episode

The program crashes while downloading EP9 from MyDress-Up Darling. All other episodes downloaded without any problems.
Tried different settings, buit the program still crashes.

i will look into a fix for the next release - for now if you want to download the episode disable the chapters

Works for me. Thx for the info


When can I add a support for amqp function, not just redis?

l want to support amqp.
example : rabbitmq

currently, i have no plan for add support for amqp.
However, i found it in Celery 4.1.0, celery support dynamic add task now with a little trouble. Detail for docs： Periodic Tasks

No plan to support amqp.


Cross origin error for any request "Error: XMLHttpRequest error."

hello,
I try to build an web application with flutter web but I got Cross origin error on any request on every website like Google.com.
this is my code:
Future<bool> getInfo() async {
  final response = await http.get('https://www.google.com');
  if (response.statusCode == 200) {
    return true;
  } else {
    return false;
  }
}

and this is error:

Error: XMLHttpRequest error.
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 906:28
packages/http/src/browser_client.dart 84:2
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 1446:54
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 150:18
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 703:44
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 732:32
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 526:7
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/stream_pipe.dart 61:11
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/stream.dart 1302:7
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 324:14
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 329:39
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/html/dart2js/html_dart2js.dart 37307:58
at Object.createErrorWithStack (http://localhost:5296/dart_sdk.js:4375:12)
at Object._rethrow (http://localhost:5296/dart_sdk.js:38296:16)
at async._AsyncCallbackEntry.new.callback (http://localhost:5296/dart_sdk.js:38290:13)
at Object._microtaskLoop (http://localhost:5296/dart_sdk.js:38122:13)
at _startMicrotaskLoop (http://localhost:5296/dart_sdk.js:38128:13)
at http://localhost:5296/dart_sdk.js:33625:9

@Jamalianpour
Sounds like this is a CORS issue. See this and this possible solution and check if it works.

@Jamalianpour
Sounds like this is a CORS issue. See this and this possible solution and check if it works.

@darshankawar
I can not use them, because I try to use third party API and I don't have access to edit CORS policy. and they don't have any CORS problem.

@darshankawar
I can not use them, because I try to use third party API and I don't have access to edit CORS policy. and they don't have any CORS problem.

I am getting same error while trying to run above code sample, but not sure if this is related to Flutter. Hence, keeping this issue open for further analysis.
Error: XMLHttpRequest error.
    dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 906:28                get current
packages/http/src/browser_client.dart 84:22                                       <fn>
dart-sdk/lib/async/zone.dart 1612:54                                              runUnary
dart-sdk/lib/async/future_impl.dart 152:18                                        handleValue
dart-sdk/lib/async/future_impl.dart 704:44                                        handleValueCallback
dart-sdk/lib/async/future_impl.dart 733:13                                        _propagateToListeners
dart-sdk/lib/async/future_impl.dart 530:7                                         [_complete]
dart-sdk/lib/async/stream_pipe.dart 61:11                                         _cancelAndValue
dart-sdk/lib/async/stream.dart 1219:7                                             <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 324:14  _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 329:39  dcall
dart-sdk/lib/html/dart2js/html_dart2js.dart 37307:58                              <fn>

    at Object.createErrorWithStack (http://localhost:60916/dart_sdk.js:5325:12)
    at Object._rethrow (http://localhost:60916/dart_sdk.js:39283:16)
    at async._AsyncCallbackEntry.new.callback (http://localhost:60916/dart_sdk.js:39277:13)
    at Object._microtaskLoop (http://localhost:60916/dart_sdk.js:39109:13)
    at _startMicrotaskLoop (http://localhost:60916/dart_sdk.js:39115:13)
    at http://localhost:60916/dart_sdk.js:34651:9

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.402, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.26.0-2.0.pre.402 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 2a188eeca3 (10 hours ago), 2021-01-17 19:27:00 -0800
    • Engine revision 609036f2bf
    • Dart version 2.12.0 (build 2.12.0-236.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86    • Android 9
      (API 28) (emulator)
    • macOS (desktop)              • macos         • darwin-x64     • Mac OS X
      10.15.4 19E2269 darwin-x64
    • Chrome (web)                 • chrome        • web-javascript • Google
      Chrome 87.0.4280.141

! Doctor found issues in 1 category.

I am getting same error while trying to run above code sample, but not sure if this is related to Flutter. Hence, keeping this issue open for further analysis.
Error: XMLHttpRequest error.
    dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 906:28                get current
packages/http/src/browser_client.dart 84:22                                       <fn>
dart-sdk/lib/async/zone.dart 1612:54                                              runUnary
dart-sdk/lib/async/future_impl.dart 152:18                                        handleValue
dart-sdk/lib/async/future_impl.dart 704:44                                        handleValueCallback
dart-sdk/lib/async/future_impl.dart 733:13                                        _propagateToListeners
dart-sdk/lib/async/future_impl.dart 530:7                                         [_complete]
dart-sdk/lib/async/stream_pipe.dart 61:11                                         _cancelAndValue
dart-sdk/lib/async/stream.dart 1219:7                                             <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 324:14  _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 329:39  dcall
dart-sdk/lib/html/dart2js/html_dart2js.dart 37307:58                              <fn>

    at Object.createErrorWithStack (http://localhost:60916/dart_sdk.js:5325:12)
    at Object._rethrow (http://localhost:60916/dart_sdk.js:39283:16)
    at async._AsyncCallbackEntry.new.callback (http://localhost:60916/dart_sdk.js:39277:13)
    at Object._microtaskLoop (http://localhost:60916/dart_sdk.js:39109:13)
    at _startMicrotaskLoop (http://localhost:60916/dart_sdk.js:39115:13)
    at http://localhost:60916/dart_sdk.js:34651:9

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.402, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.26.0-2.0.pre.402 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 2a188eeca3 (10 hours ago), 2021-01-17 19:27:00 -0800
    • Engine revision 609036f2bf
    • Dart version 2.12.0 (build 2.12.0-236.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86    • Android 9
      (API 28) (emulator)
    • macOS (desktop)              • macos         • darwin-x64     • Mac OS X
      10.15.4 19E2269 darwin-x64
    • Chrome (web)                 • chrome        • web-javascript • Google
      Chrome 87.0.4280.141

! Doctor found issues in 1 category.

Leaving these for reference here:
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
https://github.com/flutter/flutter/issues/32892#issuecomment-543838238

Leaving these for reference here:
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
https://github.com/flutter/flutter/issues/32892#issuecomment-543838238

@darshankawar
if you not sure this is related to Flutter, test other URL like this APIs:
https://www.thesportsdb.com/api/v1/json/1/searchteams.php?t=Arsenal
http://ip-api.com/json/?fields=61439
http://apilayer.net/api/check
I hope this issue will be fix soon.

@darshankawar
if you not sure this is related to Flutter, test other URL like this APIs:
https://www.thesportsdb.com/api/v1/json/1/searchteams.php?t=Arsenal
http://ip-api.com/json/?fields=61439
http://apilayer.net/api/check
I hope this issue will be fix soon.

There's nothing we can do on the Flutter side. This is a fundamental limitation on the Web. When sending HTTP requests (XHR) the server you are communicating with needs to allow it via a CORS handshake.

There's nothing we can do on the Flutter side. This is a fundamental limitation on the Web. When sending HTTP requests (XHR) the server you are communicating with needs to allow it via a CORS handshake.

so in flutter web we can't use many of third party API.
and I should say I can use these APIs in react or razor and I don't understand why flutter can't do this.

so in flutter web we can't use many of third party API.
and I should say I can use these APIs in react or razor and I don't understand why flutter can't do this.


"Micula", "Bercu Nou" and "Micula Nouă" villages are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Type "Micula" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Micula, PRAHOVA (Micula)".

Step 2
Type "Bercu Nou" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Bercu Nou, PRAHOVA (Micula)".

Step 3
Type "Micula Nouă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Micula Nouă, PRAHOVA (Micula)".


[BUG] carbon-cache always returns 0 data points for cached metric, but writes the metric to disk regularly

Describe the bug
We have a graphite cluster with approximately 130 million metrics spread across 3 storage nodes running graphite-web and 16 instances of carbon-cache. For a small proportion of our metrics (I'm unsure about the exact percentage, but from my experience it seems like <1%), carbon-cache always returns 0 data points. We know for a fact that carbon-cache is receiving the metrics in question though, because the whisper file is regularly updated. The way this problem manifests is that for certain metrics, the latest data point can be quite far away from "now", periodically "catching up" when carbon-cache flushes its data points to the whisper file.
I have verified with tcpdump that carbon-c-relay is sending the metrics to the same carbon-cache instance that graphite-web tries to query for the cached data points.
I feel like this might have something to do with hashing, since the same metric name exhibits the same problem in both clusters that receive the metric independently, and that metrics one character off (e.g. same metric from host124 instead of host123) do not exhibit the same problem. We have at least 3 instances of the same metric on different host exhibiting the problem (out of around 500 hosts reporting the metric), which gives me further cause to believe the issue is not related to something other than carbon.
The logs we're seeing:
76502:2022-11-08,02:00:11.291 :: CarbonLink sending request for statsd.host123-primary.some_metric.count to ('<IP_ADDRESS>', '9')
76503:2022-11-08,02:00:11.292 :: CarbonLink finished receiving statsd.host123-primary.some_metric.count from ('<IP_ADDRESS>', '9')
76504:2022-11-08,02:00:11.292 :: CarbonLink cache-query request for statsd.host123-primary.some_metric.count returned 0 datapoints

Metric exhibiting problem:

Metric of same name from other host (path differing by one character):

Note that I have changed the exact metric name to not expose our metric publicly, but I'm happy to provide the metric name privately for debugging (assuming the issue may be related to the exact metric name).
To Reproduce
We are able to replicate the behavior by querying a metric of the same name for different hosts, e.g. statsd.host123-primary.some_metric.count. The metric returns data, but is lagging far behind. The graphite-web cache logs show that the metric always returns 0 data points. Querying for the same metric where the path differs by one character e.g. statsd.host124-primary.some_metric.count, statsd.host125-primary.some_metric.count, does not exhibit the problem.
Expected behavior
carbon-cache should return data points for the metrics in question, given that it appears to have the data in memory (evidenced by the fact that it's writing data to disk at regular intervals).
Environment (please complete the following information):

OS flavor: Ubuntu Focal
Graphite version: Tested on 1.1.5 and 1.1.10 (both exhibit the same problem)
Setup type: pip/git
Python version: 2.7.17

Thanks for your reply, I will provide both configurations.
A representative carbon-cache carbon.conf:
...
[cache:9]
STORAGE_DIR = /mnt/graphite/storage/
CONF_DIR = /opt/graphite/conf/
LOG_DIR = /var/log/carbon/
LOCAL_DATA_DIR = /mnt/ssd/graphite/storage/whisper/
USER = graphite
MAX_CACHE_SIZE = 200000000
MAX_UPDATES_PER_SECOND = 500
MAX_CREATES_PER_MINUTE = inf
LINE_RECEIVER_INTERFACE = <IP_ADDRESS>
LINE_RECEIVER_PORT = 2436
ENABLE_UDP_LISTENER = False
UDP_RECEIVER_INTERFACE = <IP_ADDRESS>
UDP_RECEIVER_PORT = 2437
PICKLE_RECEIVER_INTERFACE = <IP_ADDRESS>
PICKLE_RECEIVER_PORT = 2438
USE_INSECURE_UNPICKLER = False
CACHE_QUERY_INTERFACE = <IP_ADDRESS>
CACHE_QUERY_PORT = 2439
USE_FLOW_CONTROL = True
LOG_UPDATES = False
WHISPER_AUTOFLUSH = False
ENABLE_TAGS = False
...

graphite-web config:
SECRET_KEY = '...'
TIME_ZONE = 'America/Los_Angeles'
LOG_ROTATION = True
LOG_RENDERING_PERFORMANCE = True
LOG_CACHE_PERFORMANCE = True
import os
_pid = os.getpid()
LOG_FILE_INFO = "info.%d.log" % _pid
LOG_FILE_EXCEPTION = "exception.%d.log" % _pid
LOG_FILE_CACHE = "cache.%d.log" % _pid
LOG_FILE_RENDERING = "rendering.%d.log" % _pid
DEFAULT_XFILES_FACTOR = 0
FIND_TIMEOUT = 60.0  # Timeout for metric find requests
FETCH_TIMEOUT = 60.0  # Timeout to fetch series data
CONF_DIR = '/opt/graphite/conf'
STORAGE_DIR = '/mnt/ssd/graphite/storage'
STATIC_ROOT = '/opt/graphite/static'
LOG_DIR = '/opt/graphite/storage/log/webapp'
INDEX_FILE = '/opt/graphite/storage/index'     # Search index file
WHISPER_DIR = '/mnt/ssd/graphite/storage/whisper'
DATABASES = {
    'default': {
        'NAME': '/opt/graphite/storage/graphite.db',
        'ENGINE': 'django.db.backends.sqlite3',
        'USER': '',
        'PASSWORD': '',
        'HOST': '',
        'PORT': ''
    }
}
INTRACLUSTER_HTTPS = True
CARBONLINK_HOSTS = [
    '<IP_ADDRESS>:2403:0',
    '<IP_ADDRESS>:2407:1',
    '<IP_ADDRESS>:2411:2',
    '<IP_ADDRESS>:2415:3',
    '<IP_ADDRESS>:2419:4',
    '<IP_ADDRESS>:2423:5',
    '<IP_ADDRESS>:2427:6',
    '<IP_ADDRESS>:2431:7',
    '<IP_ADDRESS>:2435:8',
    '<IP_ADDRESS>:2439:9',
    '<IP_ADDRESS>:2443:10',
    '<IP_ADDRESS>:2447:11',
    '<IP_ADDRESS>:2451:12',
    '<IP_ADDRESS>:2455:13',
    '<IP_ADDRESS>:2459:14',
    '<IP_ADDRESS>:2463:15',
]
CARBONLINK_TIMEOUT = 60.0
FUNCTION_PLUGINS = []

I understand this would be hard to diagnose remotely. I'm happy to run any tests that would help. Thank you.

@percygrunwald : yes, config looks perfectly fine, indeed. Will think how better to debug that. Maybe we have some hashing issue and graphite-web checking different carbonlink instead for proper one for some metrics...
BTW, instead of multiple carbon caches you can consider to use go-carbon instead, but please take note that it has no aggregator, though.

@deniszh thank you for your feedback.

Maybe we have some hashing issue and graphite-web checking different carbonlink instead for proper one for some metrics...

In my testing, tcpdump output seems to show that graphite-web is reading from the same carbon-cache instance that carbon-c-relay is writing to.
15:23:29.925423 IP localhost.43704 > localhost.2439: Flags [P.], seq 270706700:270706830, ack<PHONE_NUMBER>, win 64, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>],
 length 130
E...x<EMAIL_ADDRESS>E{..Ey.....~..}q.(U.metricq.XN...statsd.host123-primary.some_metric.countq.U.typeq.U.cache-queryq.u.

This is only the write side, but it's the correct port for carbon-cache:9. On the read side, it appears as though graphite-web is querying carbon-cache:9 as well:
76502:2022-11-08,02:00:11.291 :: CarbonLink sending request for statsd.host123-primary.some_metric.count to ('<IP_ADDRESS>', '9')
76503:2022-11-08,02:00:11.292 :: CarbonLink finished receiving statsd.host123-primary.some_metric.count from ('<IP_ADDRESS>', '9')
76504:2022-11-08,02:00:11.292 :: CarbonLink cache-query request for statsd.host123-primary.some_metric.count returned 0 datapoints

Unless the log is wrong, it appears as though the write and read are happening to the same carbon-cache instance (9).

BTW, instead of multiple carbon caches you can consider to use go-carbon instead

Thank you for the suggestion. We make use of go-carbon in our new architecture and it has worked very well as a carbon replacement.

Looks like big really exists, but I still very confused with it.

Sorry, could you please elaborate what you mean?
ср, 19 июл. 2023 г., 18:35 itinneed2022 @.***>:

Hello Deniszh, I think this is the same issue I'm experiencing on my
setup. Is there anyway to check the carbon daemons directly to see what
data they're currently holding?
—
Reply to this email directly, view it on GitHub
https://github.com/graphite-project/carbon/issues/940#issuecomment-1642408343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJLTVX77VQWELPWUTBGMRTXRAEETANCNFSM6AAAAAAR23KCUU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

@deniszh I'm seeing the same error @percygrunwald is seeing.
2023-07-19,14:45:16.451 :: CarbonLink finished receiving test.density-full.47_Claremont.47_Claremont_B.fullness from ('<IP_ADDRESS>', 'b')
2023-07-19,14:45:16.451 :: Carbonlink results: {'datapoints': []}

I created a question on graphite-web last year.
In my cache logs I've noticed some metrics do return datapoints, others do not.
Is there anyway to query the carbon cache daemon's directly and see if the metrics do exist in the caches?

@deniszh I'm seeing the same error @percygrunwald is seeing.
2023-07-19,14:45:16.451 :: CarbonLink finished receiving test.density-full.47_Clare.47_C.fullness from ('<IP_ADDRESS>', 'b')
2023-07-19,14:45:16.451 :: Carbonlink results: {'datapoints': []}

I created a question on graphite-web last year.
In my cache logs I've noticed some metrics do return datapoints, others do not.
Is there anyway to query the carbon cache daemon's directly and see if the metrics do exist in the caches?

I added some lines to my carbonlink.py file to try and narrow down the issue. Here's a snippet:
2023-07-20,13:46:14.823 :: Conn is: <socket.socket fd=30, family=AddressFamily.AF_INET, type=2049, proto=6, laddr=('<IP_ADDRESS>', 54484), raddr=('<IP_ADDRESS>', 7002)>
2023-07-20,13:46:14.823 :: Metric requested is: test.density-full.Broad_Flat.Broad_Flat_14.client_count
2023-07-20,13:46:14.823 :: CarbonLink sending request for test.density-full.Broad_Flat.Broad_Flat_14.client_count to ('<IP_ADDRESS>', 'a')
2023-07-20,13:46:14.823 :: len_prefix in recv_response is: b'\x00\x00\x00-'
2023-07-20,13:46:14.823 :: body size in recv_response is: 45
2023-07-20,13:46:14.824 :: body in recv_response is: b'\x80\x02}q\x00X\n\x00\x00\x00datapointsq\x01]q\x02Jtq\xb9dG\x00\x00\x00\x00\x00\x00\x00\x00\x86q\x03as.'
2023-07-20,13:46:14.824 :: len_prefix in recv_response is: b'\x00\x00\x00-'
2023-07-20,13:46:14.824 :: body size in recv_response is: 45
2023-07-20,13:46:14.824 :: body in recv_response is: b'\x80\x02}q\x00X\n\x00\x00\x00datapointsq\x01]q\x02Jtq\xb9dG\x00\x00\x00\x00\x00\x00\x00\x00\x86q\x03as.'
2023-07-20,13:46:14.824 :: Made it inside the try, result is: {'datapoints': [(1689874804, 0.0)]}
2023-07-20,13:46:14.824 :: result2 is: {'datapoints': [(1689874804, 0.0)]}
2023-07-20,13:46:14.824 :: CarbonLink finished receiving test.density-full.Broad_Flat.Broad_Flat_14.client_count from ('<IP_ADDRESS>', 'a')
2023-07-20,13:46:14.824 :: Carbonlink results: {'datapoints': [(1689874804, 0.0)]}
2023-07-20,13:46:14.824 :: CarbonLink cache-query request for test.density-full.Broad_Flat.Broad_Flat_14.client_count returned 1 datapoints
2023-07-20,13:47:43.056 :: Request-Cache miss [5df10b47fc6a60a71d25ff3c8bae38b4]
2023-07-20,13:47:43.056 :: Data-Cache miss [d15e419cbcaa3638f54205c85c887466]
2023-07-20,13:47:49.075 :: Request-Cache miss [6fa36018ab95e559b1c4deb71d1f6dae]
2023-07-20,13:47:49.076 :: Data-Cache miss [f523d7ea713e59b4d97153c6f1a6659e]
2023-07-20,13:47:49.094 :: Host is: ('<IP_ADDRESS>', 'a')
2023-07-20,13:47:49.094 :: CarbonLink creating a new socket for ('<IP_ADDRESS>', 'a')
2023-07-20,13:47:49.095 :: Carbonlink connection successful
2023-07-20,13:47:49.095 :: Conn is: <socket.socket fd=36, family=AddressFamily.AF_INET, type=2049, proto=6, laddr=('<IP_ADDRESS>', 55548), raddr=('<IP_ADDRESS>', 7002)>
2023-07-20,13:47:49.095 :: Metric requested is: CI_unixsys.aaprddb02_company_com.cpu-0.percent-idle
2023-07-20,13:47:49.095 :: CarbonLink sending request for CI_unixsys.aaprddb02_company_com.cpu-0.percent-idle to ('<IP_ADDRESS>', 'a')
2023-07-20,13:47:49.097 :: len_prefix in recv_response is: b'\x00\x00\x00\x1b'
2023-07-20,13:47:49.097 :: body size in recv_response is: 27
2023-07-20,13:47:49.098 :: body in recv_response is: b'\x80\x02}q\x00X\n\x00\x00\x00datapointsq\x01]q\x02s.'
2023-07-20,13:47:49.098 :: len_prefix in recv_response is: b'\x00\x00\x00\x1b'
2023-07-20,13:47:49.098 :: body size in recv_response is: 27
2023-07-20,13:47:49.098 :: body in recv_response is: b'\x80\x02}q\x00X\n\x00\x00\x00datapointsq\x01]q\x02s.'
2023-07-20,13:47:49.098 :: Made it inside the try, result is: {'datapoints': []}
2023-07-20,13:47:49.098 :: result2 is: {'datapoints': []}
2023-07-20,13:47:49.098 :: CarbonLink finished receiving CI_unixsys.aaprddb02_company_com.cpu-0.percent-idle from ('<IP_ADDRESS>', 'a')
2023-07-20,13:47:49.098 :: Carbonlink results: {'datapoints': []}
2023-07-20,13:47:49.098 :: CarbonLink cache-query request for CI_unixsys.aaprddb02_company_com.cpu-0.percent-idle returned 0 datapoints
2023-07-20,13:47:49.969 :: Request-Cache miss [5df10b47fc6a60a71d25ff3c8bae38b4]
2023-07-20,13:47:49.969 :: Data-Cache miss [d15e419cbcaa3638f54205c85c887466]
2023-07-20,13:47:50.412 :: Request-Cache miss [d6f5aae245b305a127886bf5de73f3ce]
2023-07-20,13:47:50.412 :: Data-Cache miss [fe44d544439db17608cd6d177e3bbf8e]

Something I'm consistently noticing in the logs is that any datapoint that returns empty has a byte string that ends in:
datapointsq\x01]q\x02s.
Where are successfully retrieved bytes always look like:
00datapointsq\x01]q\x02Jtq\xb9dG\x00\x00\x00\x00\x00\x00\x00\x00\x86q\x03as.
I don't know how to decode the byte strings but I'm thinking they are not properly formed?


New AspNetCoreHealth Authorization Feature For Discussion

Add Authorization Feature to AspNetCoreHealth. Instead of implementing Authorization details in AspNetCoreHealth use dependency injection with IHealthAuthorizationFilter interface to allow the calling application to implement their own Authorization Mechanism.  The HttpContext context is passed into the IHealthAuthorizationFilter Authorized method because the most likely scenario is the caller will want to check for an Authorization header and secrets.
Thoughts?
Here is the actual feature implemented ready for contribution:
https://github.com/jvonruden/AspNetCoreHealth/commit/2400b4ce0c4b47f334ce2694d872d97c0eb620c4
Summary:
1) Created an Interface: IHealthAuthorizationFilter
public interface IHealthAuthorizationFilter
{
	bool Authorized(HttpContext context, CancellationToken token = default);
}

2) In HealthApplicationBuilderExtensions.cs use dependency injection to see if someone has provided IHealthAuthorizationFilter and use it instead AuthorizedHealthAuthorizationFilter (Always returns Authorized == true)
var healthAuthorizationFilter = app.ApplicationServices.GetService<IHealthAuthorizationFilter>() ?? new AuthorizedHealthAuthorizationFilter();
appBuilder.UseMiddleware<HealthCheckEndpointMiddleware>(responseWriter, endpointsOptionsAccessor.Value.Timeout, healthAuthorizationFilter);

3) In the HealthCheckEndpointMiddleware Invoke method just before the _healthCheckRunner is called check for the Authorization from the provided IHealthAuthorizationFilter.
if (!_healthAuthorizationFilter.Authorized(context))
{
_logger.MiddlewareAuthorizationInvalid();
context.Response.StatusCode = StatusCodes.Status401Unauthorized;
await context.Response.WriteAsync("Invalid authorization.", cancellationToken: cancellationTokenSource.Token);
}
else
{
// _healthCheckRunner stuff.
}
4) The Fact Middleware test is a working example using the IHealthAuthorizationFilter HealthCheckEndpointAuthorizationMiddleware.cs

@jvonruden I've just been using an IStartupFilter to restrice access to health and metrics, something like the following:
public Action<IApplicationBuilder> Configure(Action<IApplicationBuilder> next)
        {
            return RequireSecretToMetricsAndHealth;

            void RequireSecretToMetricsAndHealth(IApplicationBuilder app)
            {
                app.Use(async (context, next2) =>
                {
                    var key = ExtractToken(context);

                    if (_endpoints.Any(e => string.Compare(context.Request.Path.Value, e, StringComparison.InvariantCultureIgnoreCase) == 0))
                    {
                        if (key != _key)
                        {
                            context.Abort();
                            return;
                        }
                    }

                    await next2.Invoke();
                });

                next(app);
            }
        }

This makes sense. I wonder if this scenario/solution is captured in documentation since it is pretty common to want to lock down the health endpoints

If not I would be happy to contribute and update to the docs

Would be great if you could contribute to the docs. Thanks


Support for Terraform Cloud SSH key variable in the control_tower_account_factory module

Describe the outcome you'd like
Support to allow the user to pass Terraform Cloud SSH key as an input parameter in the control_tower_account_factory module
Is your feature request related to a problem you are currently experiencing? If so, please describe.
For AFT, we are using Terraform Cloud. To define global and account-specific customizations. I am referring to existing modules in the Terraform code. When the AFT global and account-specific customization pipeline gets triggered for the first time, it fails the run execution in the respective Terraform Cloud workspaces due to a missing SSH key. To make run success I need to manually add SSH key to each account's workspaces. So supporting the input parameter SSH key for Terraform Cloud workspaces in the AFT will be convenient for users who are not completely in the Terraform Cloud registry.
Additional context
As our organization is in the early stages of Terraform Cloud adoption, all of our existing Terraform modules are not in the Terraform Cloud Private registry. We have a plan to standardize existing modules before publishing them in the registry, but it is a big project and not happening soon.

Hi Seema, thanks for bring this up and for the detail here; I've created a backlog item for us to look into this.


closing scrum -> demo day

I'm taking a bit from the ALSF CCDL to try to make closing scrum more of an opportunity to show off a new result, new writing, or other accomplishments from the week.

I think this is a great idea and a motivation to come to the end of the week with a deliverable ready to show to others.
How are we planning to implement it? Everyone shares the figure/link/text in lab-meeting (Slack) so we avoid any screen sharing in Zoom?

Sounds fun! How do the CCDL people like the 3 minute format as opposed to having more time and going every other week?

I was imagining screen sharing in zoom, but we could do something different. The CCDL does it every two weeks for more time but they also have an every two week sprint schedule, so it's natural to do it that way there.

I was imagining screen sharing in zoom, but we could do something different [...]

Screen sharing should work, but since sometimes we have some trouble with that I think it would be good to have a plan B to keep the scrum meeting short without any of these technical problems.

Had fun with this first iteration of demo day. One question I had is, does it make sense to have this be a time to both share and get feedback from others? Perhaps maybe this can be handled offline where people can ask people to DM them for feedback after they've shared?

That's a good point, I didn't think about the fact that this would make the slides effectively public. I personally am fine with it but it would be good to hear how others in the lab feel.

I'm also fine with the slides being public. We might add a note in the onboard document mentioning this, so those who might be more concerned could, during the demo, say (instead of write) what they believe should not be effectively public.

I wouldn't mind if the slides were public. +1 to what Milton said, people can always choose to screenshare results instead of adding them to a slide


Changing the player who negs to someone on the same team makes it a no-penalty buzz

Repro:

Neg someone
Click on the same word, and choose a different player on the same team to neg
Expected:
This buzz should count as a neg
Actual:
This buzz is counted as a "no-penalty" buzz, likely because we think this is the second incorrect buzz on that word.

I should have this fixed in the next PR.


Update my-first-pr.md

After some struggle lets make it easier I say.

Yes, navigating is something I've been able to manage but may be difficult for others. The directory tree was at first incomprehensible. So I like your idea a lot. When I read the detailed breakdown I can't open the door to my brain. So separating the steps as icon clicks from the steps as git operations helps. Thus this new section followed by the existing.

Oh wow I'll put that method in. I had never known what that box at the bottom right did?

The change requested (navigate from readthedocs.io/en) was made. It is so easy for us now to PR but it used to be extreemly dificult for me. Removing technical terms may help. I did not remove the section with technical terms because I still don't know what it says.


Update user-data.ts

Method naming starts with has so it makes sense if this method returns boolean instead of true/null/undefined.

Thanks!


feat: FloatingList

@pierreburel @Marsony
The sortByDocumentPosition technique is found in other libs to enable this feature, and this one doesn't write or read any refs during render, so hopefully good for concurrent features.
Usage:
FloatingList is a context provider for children

elementsRef is useListNavigation's listRef
stringsRef is useTypeahead's listRef which uses the value passed into useListItem or falls back to the node's textContent

<FloatingList elementsRef={elementsRef} stringsRef={stringsRef}>
  ...
</FloatingList>

useListItem reads that context and returns an index and a ref:

const {activeIndex} = useSelectContext();
const {ref, index} = useListItem();
const isActive = activeIndex === index;

// No need to manually assign an index:

For controlled (pre-selected values) is there anything else we should do?  @pierreburel - your demo has listValuesRef but this is not included here

Nice job!
I really like the idea of passing an optional string for useTypeahead to useListItem, but i'm not sure about the value name tho? Maybe more content or label? In a Select component for example, value would more likely be an id or slug, so it may feels weird:
const Option = ({ value, label, description }) => {
  const { setSelectedIndex, setValue } = useSelectContext();

  const { ref, index } = useListItem({ value: label }); // 👈🏻 here

  const handleSelect = () => {
    setSelectedIndex(index);
    setValue(value);
  };

  return (

      {label}
      {description}
      <svg …/>

  );
};

For controlled (pre-selected values) is there anything else we should do? @pierreburel - your demo has listValuesRef but this is not included here

I guess for now we have to do that?
const Option = ({ value, label, description }) => {
  const { selectedIndex, setSelectedIndex, selectedValue, setValue } = useSelectContext();

  const { ref, index } = useListItem();

  const handleRef = (node) => {
    ref(node);
    if (selectedValue === value && selectedIndex !== index) {
      setSelectedIndex(index);
    }
  };

  const handleSelect = () => {
    setSelectedIndex(index);
    setValue(value);
  };

  return
{label}
;
};

Any chance we could find a way to simply the ref part? Maybe by adding a parameter to ref or a prop to useListItem?
And yeah the listValuesRef was something i forgot to remove when i did some tests.

I like label indeed. For selectedIndex it could be a prop like onMount, or you could just a layout effect instead:
useListItem({
  label: str,
  onMount() {
    if (selectedValue === value && selectedIndex !== index) {
      setSelectedIndex(index);
    }
  }
});

Yeah it works well with a layout effect, maybe there's no need to overcomplicate things (like, would onMount needs a onUnmount too?).
In any case, LGTM! Amazing work as always 👍

Last question here is what's the best way to allow typeahead to work before the list has ever been rendered/opened, while maintaining conditional rendering. e.g. focus the button then type anything to select automatically without opening the list — a feature of native <select>.
The following works, but isn't very perf-friendly (children are rendered into a hidden div to register themselves). You could also avoid conditional rendering altogether, although FloatingFocusManager would need to be conditionally rendered as a parent still.
{!open &&
{children}
}

In some components I've also seen a hidden select being rendered so it works with form autofills too


Floating icalendar dates are not correctly roundtripped

When parsing an iCalendar string that contains a local/floating time, the json switches the timezone to UTC. Example:
$ical = "BEGIN:VCALENDAR
BEGIN:VEVENT
UID:a03a61fc-ef74-413a-84b9-adecc317719b
SUMMARY:test
DTSTART:20150108T150000
DTEND:20150108T160000
END:VEVENT
END:VCALENDAR";

print $ical . "\n";
$x = \Sabre\VObject\Reader::read($ical);
print json_encode($x->jsonSerialize());

The result contains this:
["dtstart",{},"date-time","2015-01-08T15:00:00Z"],
["dtend",{},"date-time","2015-01-08T16:00:00Z"]

I would expect this:
["dtstart",{},"date-time","2015-01-08T15:00:00"],
["dtend",{},"date-time","2015-01-08T16:00:00"]

Hello :-),
Thank for the report, but why? Is it part of the spec'?

Check out http://tools.ietf.org/html/rfc5545#section-3.3.5, specifically FORM 1: DATE WITH LOCAL TIME

I will work on this bug tomorrow. @evert, because we have a different timezone, feel free to work on this bug for your today if you have time. I am not blocking this bug for me.

k =) will probably take a look today then.

@evert Only if you have time ;-).

Thanks for diving this so quickly. I've tracked it down to DateTimeParser setting UTC if $tz is null, which it is for floating times. Removing that check makes the example above work, but will likely break other things. Also, I believe PHP assumes the local timezone if $tz = null.

Another bit of information: when I remove the is_null check the example does seem to work, but it didn't work when retrieving an event via the sabredav server I am running. This ended up happening because the server uses date_default_timezone_set('UTC')


service endpoint creation automation required for VSTS Kubernetes release

Thanks for these example. I went through the walk through and got my own Kubernetes instance working and deploying from VSTS.
One bugbear though is the service endpoint needs to be created in VSTS manually just after the kubernetes cluster has been created.  This means there's a manual step in between our automated process.  So, we have split our release into two environments, with an approval step in between to allow for the pause and manual intervention.  A bit hacky.
How can we automatically create the kubernetes service endpoint in VSTS using a powershell step or similar just after the cluster has been created? In this way, our release can be automated end to end.

Hi Ossentoo,
Thanks for your feedback. We will check the feasibility to automatically create kubernetes service endpoint.
Thanks
Balaji

Hi Ossentoo,
We have updated AKS lab with new Kubernetes task in CD pipeline. Now you don't have to create AKS service endpoint. With the new task, you can select the AKS cluster from the drop-down. Please try our updated lab and let us know your feedback
Thanks
Balaji

works well. thanks


No information on valid values for domainValidationMethod proeprty

Trying to find information on creating ARM templates for service managed certificates, there is no indication about what valid values for this string value are.

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

ID: d46863db-c48c-b7e1-acae-b2bdf98ae880
Version Independent ID: 72b4791b-0735-3c98-bc95-ab5927df4d86
Content: Certificate.DomainValidationMethod Property (Microsoft.Azure.Management.WebSites.Models) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Management.WebSites.Models/Certificate.xml
Service: app-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Is this information available anywhere else?

Thank you for reaching out to us about the Azure SDK for .NET. Unfortunately, this issue is related to the service side and not the SDK side, so we are not able to help you with it. Therefore, we are closing this issue for now. If you need more assistance, please open an Azure support request. We apologize for any inconvenience this may cause you.
We value your feedback and we want to make sure that your problem is solved. If you think that we have misunderstood your issue or closed it incorrectly, please feel free to comment on this thread and reopen the issue. We will be happy to assist you further. Thank you for your understanding and cooperation.


VSCode won't launch on mac

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

VS Code Version: 1.75.1
OS Version: 13.2

Steps to Reproduce:

Click on "Visual Studio Code" app icon
Nothing happens, Vscode icon appears in the dock for a second and disappers.

This problem appears after the tried to fix neovim extension by following the instruction of https://github.com/microsoft/vscode/issues/86745 and deleted the files ~/.config/Code and ~/.vscode. After that I deleted the Vscode app and reinstalled it.
This issue is similar to #81970. But If I launch with "--verbose" I see:
code --verbose
open: unrecognized option '--verbose'

Also I have followed https://stackoverflow.com/a/47561948/11908232 to completely uninstall vscode

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.75.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I looked at issue #115306 and downloaded the nightly version which is working but the main version is still not working

Is this still an issue with latest stable ?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Let's include what have changed in the release notes

Hi there, I jumped from 1.4.3 to beta02 and in the release notes there is no indications that this
import moe.tlaster.precompose.PreComposeApplication

fun MainViewController() = PreComposeApplication((title = "iOS")) { App() }

Have changed to this
import moe.tlaster.precompose.PreComposeApplication

fun MainViewController() = PreComposeApplication() { App() }

For multiplatform projects , on the iOS side, the error log is still not that easy to debug, including release notes will help us migrate to recent updates more easily, thanks

Thanks a lot @Tlaster <3

hey @Tlaster this is a nice API table on kodein that you can follow for the Readme :)
https://github.com/kosi-libs/Kodein


Clear chat, edit previous messages

Hi, thanks for all your work on this project! There are cases where it'd be nice to be able to clear the entire chat history (without having to create a whole new chat from scratch) and/or edit/delete previous messages in a conversation.
Btw, there is that button at the top with a counter-clockwise circling arrow (and touching it gives a check mark), but not sure what it actually does if anything?
Edit: Or if there was a way to duplicate the current chat setup or some other way to easily have multiple threads going at once, that could help for chat management.

I thought it was the "clear context" button (i.e., starts the conversation over, but without clearing the prior chat history.
If that's not what it is, then I don't actually know what it is. 😂

I thought it was the "clear context" button (i.e., starts the conversation over, but without clearing the prior chat history.
If that's not what it is, then I don't actually know what it is. 😂

Oh that could be!


Add option to limit # of messages per stream in QPS tests

The console output shows, interestingly, that 1 message per-stream streaming underperforms unary (perhaps because of the explicit initial Request and the concluding WritesDone and Finish stages), 3 messages per-stream catches up to unary, and then the qps continues to increase with messages per stream until leveling off after reaching around 30 messages per stream. This could be done to other languages too, but is currently only implemented for C++.

That's expected right now. We end up needing to do a write for each stage
of a request, so 5 or 6 syscalls need to complete for each length one
stream. Once Yuxuan gets done with the new API we'll be able to integrate
that and see parity here.
On Tue, Feb 28, 2017, 12:15 AM Vijay Pai<EMAIL_ADDRESS>wrote:

The console output shows, interestingly, that 1 message per-stream
streaming underperforms unary (perhaps because of the explicit initial
Request and the concluding WritesDone and Finish stages), 3 messages
per-stream catches up to unary, and then the qps continues to increase with
messages per stream until leveling off after reaching around 30 messages
per stream. This could be done to other languages too, but is currently
only implemented for C++.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/9892#issuecomment-282972070, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJpudZXZHPxOEs8K2NuBNdlA3mOm_Ckdks5rg9ebgaJpZM4MNp20
.

Rebased to get latest master and updated to fix some bugs that will arise once shared-cq gets merged. I can haz review?

Actually, I accidentally broke everything so I don't want a review yet. Back to the drawing board on this one.

Rebased master and removed WIP label because I believe that this is now in good shape

Squashed since approved. Waiting for all tests.

It strikes me that this keeps getting TIMEOUT on tsan because I just added a ton of unconstrained tests, each of which ends up taking about 3 minutes on tsan. So I need to exclude TSAN runs on most of these.

No significant performance differences

No significant performance differences

Test failures: #10243 #9804


Are commands not supported anymore ? Code is commented out and functions aren't available.

Hi,
Are setpoint commands not supported anymore ? The lines to set them are commented out in command_reader_task :
                par = splits[1]
                print("Heat Setpoint [" + par + "]")
                # cmdres = await zone.setHeatSPF(par)
                # print("Command Result [" + str(cmdres) + "]")```

 Also, these functions aren't in the "s30api_async.py" file. Since it is mentioned that, wrong parameters can brick the thermostat, what is the right method to do a schedule override (the controller is in schedule mode by default) ?

Thanks!

Apologies, I have not been keeping that command line program up to date as I’ve been updating the API, and as you’ve noticed the docs are scarce.
Yes, you can do those operations.  Call the zone.perform_setpoint
https://github.com/PeteRager/lennoxs30api/blob/645a41606bd292bfaa57da9f2c112a6f66f3b265/lennoxs30api/s30api_async.py#L2416
This method will perform the override if the thermo is already following a schedule.  The override will expire at the start of the next period or there is a method to cancel the override.
Let me know how that works and If you have other question.
There are good examples of usage in the home assistant integration
https://github.com/PeteRager/lennoxs30


QrCodeGenerator: forceUtf8 results in invalid numeric codes

If you pass forceUtf8: true but the text only contains numeric data then an invalid QR code is generated.
public static void Main(string[] args)
{
	var generator = new QRCodeGenerator();

	var validCode = new QRCode(generator.CreateQrCode("123456", QRCodeGenerator.ECCLevel.Q, forceUtf8: false));
	validCode.GetGraphic(10).Save("valid.png", ImageFormat.Png);

	var invalidCode = new QRCode(generator.CreateQrCode("123456", QRCodeGenerator.ECCLevel.Q, forceUtf8: true));
	invalidCode.GetGraphic(10).Save("invalid.png", ImageFormat.Png);
}

I suspect this is because PlainTextToBinary looks first at EncodingMode (which is Numeric in this case?) and GetDataLength looks first at forceUtf8.

Hi pontusi,
thanks for the feedback and your analysis. You're right, when forceUtf8 was true, the QRCoder didn't check if the content is text content. Now byte-mode payload-encoding is forced, when forceUtf8 is set to true. https://github.com/codebude/QRCoder/commit/bd284c57552f177b961733611baf031aafb2f759
(Update of the NuGet package will follow in a couple of minutes.)


Webpack2 build more than twice when entry file changed once

Do you want to request a feature or report a bug?
bug
What is the current behavior?

Run webpack dev-server after change one entry file, it will build many times with the same hash.
And followed by this message:
webpack: bundle is now VALID.
webpack: bundle is now INVALID.

Then I switch to webpack1, it works and only build twice.(Only one bundle is now INVALID.)
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?

Change one entry file
Run webpack2 dev-server instantly

Please mention other relevant information such as the browser version, Node.js version, Operating System and programming language.
webpack: 2.1.0-beta.28
node: 6.9.1
npm: 3.10.8

Can you set up a standalone example so it's easier to debug?

Also do you exhibit this behavior in webpack 2.2.0-RC.1

having the same problem here https://github.com/devinehowest/generator-devine-boilerplate
... with 2.2.0-RC.1

@bebraw
Here is an example: https://github.com/guox191/react-webpack2-demo
run make dev to reproduce it

This is https://github.com/webpack/watchpack/issues/25 again

@Kovensky Thanks. Closing.


stm32flash: 0.5 -> 0.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/stm32flash/versions.
meta.description for stm32flash is: "Open source flash program for the STM32 ARM processors using the ST bootloader"
meta.homepage for stm32flash is: "https://sourceforge.net/projects/stm32flash/"
meta.changelog for stm32flash is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6/bin/stm32flash passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 0.6 with grep in /nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6

found 0.6 in filename of file in /nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6

directory tree listing: https://gist.github.com/0b608a16239faea0a014b0b593d9619a

du listing: https://gist.github.com/96e7db909e262dfe8d48a99dc94d9b47

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
stm32flash

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A stm32flash https://github.com/r-ryantm/nixpkgs/archive/6642509bac4ca9d85f9c6a9fe0f41379e2dbc24b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6
ls -la /nix/store/3rkmpsi6lpsmflixizrh35a4nx4iaiva-stm32flash-0.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

stm32flash

Maintainer pings
cc @elitak for testing.

Result of nixpkgs-review pr 135967 at 6642509b run on aarch64-linux 1

1 package built successfully:

stm32flash

2 suggestions:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/development/embedded/stm32/stm32flash/default.nix:23:5:
   |
23 |     license = lib.licenses.gpl2;
   |     ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/embedded/stm32/stm32flash/default.nix:14:3:
   |
14 |   installPhase = ''
   |   ^


Fix URLs in docs

Signed-off-by: Waleed Malik<EMAIL_ADDRESS>Fixes #122

/approved
/lgtm

@wurbanski
s/approved/approve :D

/approve


Adding --iree-vm-target-* flags to control index bit width and i64 support

This allows us to specify a consistent type converter for all lowering into VM and adds options to control the handling of index and i64.
Progress on #2574.

I think I'm hitting an MLIR issue here and have filed a bug upstream: https://bugs.llvm.org/show_bug.cgi?id=46944


feat(ref-pattern): add new 'ref-pattern' rule

PR summary
This commit introduces the new spectral-style
'ref-pattern' rule, which replaces the legacy
'incorrect_ref_pattern' rule.
PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[ ] Dependencies have been updated as needed
[ ] .secrets.baseline updated as needed?

Checklist for adding a new validation rule:

[x] Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
[x] If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
[x] Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
[x] Added tests for new rule (packages/ruleset/test/*.test.js)
[x] Added docs for new rule (docs/ibm-cloud-rules.md)

Checklist for removing an old validation rule:

[x] Removed old rule implementation (packages/validator/src/plugins/validation/*.js)
[x] Removed or adjusted testcases (packages/validator/test)
[x] Updated default configuration to deprecate old rule (packages/validator/src/.defaultsForValidator.js)
[x] Removed docs of old rule (docs/ibm-legacy-rules.md)

:tada: This PR is included in version 0.24.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


.git folder included in the npm distribution

The presence of a .git folder means that the npm install errors out when run for the second time of a project as below:
npm ERR! path ... /node_modules/websocket
npm ERR! code EISGIT
npm ERR! git ... /node_modules/websocket: Appears to be a git repo or submodule.
npm ERR! git ... /node_modules/websocket
npm ERR! git Refusing to remove it. Update manually,
npm ERR! git or move it out of the way first.

Manually deleting the .git folder fixes it.
Could you please remove the .git folder from the distribution?

Thanks, fixed.
@theturtle32 may you do a release please?

Thank you. That was stunningly rapid.

@theturtle32 just wanted to remind you that this change is still awaiting release. If you could find the time, it would be greatly appreciated!

Please, release the package with this fix...
It is breaking a lot of installs

Fixed.
On Tue, Jul 2, 2019 at 1:31 PM Carlos Landeras<EMAIL_ADDRESS>wrote:

Please, release the package with this fix...
It is breaking a lot of installs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theturtle32/WebSocket-Node/issues/353?email_source=notifications&email_token=AAAM7QC4LPZQNGJAS2657E3P5O3KVA5CNFSM4HIEQXK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZCOYQI#issuecomment-507833409,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAM7QCISCFFNLJPOU5AIEDP5O3KVANCNFSM4HIEQXKQ
.

Released

Thanks @theturtle32

Normally it's not necessary to ignore .git/. For those who ran into this bug and are puzzled as to what was happening, see: https://github.com/npm/npm-packlist/issues/35.
So an alternative fix would have been to change "main": "index", to "main": "index.js",.


string to integer

Hi, this is more of a request. How would you convert an integer string into an integer ... efficiently?
From a string I can extract a sub string containing only numbers but now need to efficiently convert that into an integer for further manipulation.
I read that q4 is faster but it lacks features, and c4 has many features, but speed or efficiency how does it compare?

Here is how I would take a string of numbers and turn it into a number in s3:
0( BTWI: n l h -- f ... Between-Inclusive )
:BTWI l+ s3 s2 s1 r1 r2 <~ r1 r3 >~ b& l-;
0( STOI: a--n ...  )
:STOI l+ s1 0 {10* r1 c@ '0- + i1 r1 c@ '0'9 BTWI} l-;
NB: this assumes you already know it's a string of digits.
Of course, if you want, you can remove all the spaces to improve performance, at the expense of making it harder to read.

As to performance, the VM that c4 compiles to is very similar to s3. Since this code is essentially what c4 would compile to (without the spaces of course, and the call to BTWI is direct, not hashed at runtime), so c4 is maybe a little more efficient, but not much. But c4 is more readable.

So I just added a way to execute a dynamically built string, using xE. For example:
2000#|"hi there!"74665|
2000 xE
This will print 'hi there!' and push 74665 onto the stack.
The only thing would be: don't create a function using xE.

Thanks --- STOI works well.


Middleman v4 support in master but not released

Hello!
Could you release a new version of this library please?
Thanks,
Louis

Version 2.1.0 has been released.

Thank you.


ERC20 Java chaincode implementation exmaple.

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>This is an example of ERC20 token standard Java chaincode implementation.  The functionality is the same as the existing go and nodejs ERC20 chaincode.

I really like this PR.  The code is clear, legible, has a good flow, and I enjoyed reading through it.  I'm not familiar with the golang token transfer reference, but I felt like I had learned something about Fabric and chaincode by reading through the commit.
Here are a few considerations, more for discussion points than for "must have" features on this PR.  (These are more about providing some consistency across the samples as a whole, rather than this specific contract.)

Objections to the use of Maven.    This example uses maven to build a shaded Jar.  It would be improved (?) by aligning with general practices and presented a standardized front through the use of Gradle to compile, package, and test the chaincode.  That said, there's nothing inherently wrong with the use of Maven in this example.  That said, building the jar file is just one step in the lifecycle for managing chaincode, and it would be nice if Fabric had some clear guidelines and examples of building Java based Docker images running with Chaincode-as-a-Service.   (This would make a nice follow-on, or additional PR once we have the story sorted out.)

Objections to the use of System.out.   It's hard to object to the practice of the routine emitting anything to the chaincode standard output.   I really like seeing slf4j and logback (not log4j) pulled into the Maven dependencies.  Instead of System.out, can we use this as an example to highlight proper logging techniques with slf4j in the routine?

Objections to the use of org.json :   Similar to the choice of "best logger" and "best build system" -- are we 100% on track with the recommendation of org.json for the JSON handling in the example?   (Likewise are we 100% on track with Gson in other examples?)  If we are going to standardize and recommend a JSON parser, why not use Jackson?

Objections to the Java Chaincode SDK :   I read through this sample and was reminded that we have a disconnect between ChaincodeException and Fabric's adherence to golang Error strings in other parts of the system.  It's even more pronounced in this example, where the code is clean enough that I can understand what's happening, but the context is lost through the use of ChaincodeException(String error).  For instance, how does a chaincode describe the difference between "not found" and "insufficient balance available" ?   In consuming applications, everything will need to dig in and parse the string error out of the exception.  It would be much better (IMHO) if the natural exception / response from a chaincode invocation were a struct, consisting of an application-specific error code coupled with an (optional) explanation.

{
  int code;
  String message;
}

ChaincodeException sounds to me like : "Network error reading from a DB", or "no space left on device" when in the middle of a chaincode invocation.   Can we improve this somehow?  (Again: not in this PR but it's what came to mind when reviewing the new sample.)
Last bit:

Objection to the hard-coded certificates in the test chaincode stub.   What's in these stub certificates?  Will they expire?

Overall, I really like the new sample.  Thank you for the great contribution.

ping: @mbwhite and @denyeart and @bestbeforetoday -   any objections to this PR?
I really like the addition and would like to merge it.  Also can we use this to align on any new-fangled and/or best practices coming out of Chaincode-as-a-Service, Gateway client dev, and Java samples currently in the wings?

@renjithpta May I asked why you closed this PR? @bestbeforetoday's review was quite positive.


add Custom Scroll Bar

Description
Added a custom Scroll bar to all the pages (i.e to the main css)
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Added it to another hosted site of mine.
Works just fine
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] Any dependent changes have been merged and published in downstream modules

-web-kit scrollbar is not a standard practice. Its not supported in mozilla firefox.
So we are discontinuing with webkit-scrollbar
You need to think, bit basics of divs and appending with custom JavaScript.
And also, we are not going to use JQuery scrollbars, as it would make site slower for slow networks
https://developer.mozilla.org/en-US/docs/Web/CSS/::-webkit-scrollbar


[Discussion] PRPL pattern with dynamic routes

Let's say we have a group of dynamic child routes following /invoices/:id that all have a shared app shell. Ideally, a pre-rendered shell would be displayed if a route matching this pattern was the entry point (and the :id specific stuff could loaded and displayed dynamically).
Something like configuring redirects on a static host to resolve/invoices/* to a pre-rendered invoiceShell.html should work (and then the rest of the routing could be handled dynamically).
To use this method with react-static, we could define the shell in a component, and then render a dummy route in the static.config.js that would never be intended for ultimate use on its own.
I imagine this is difficult (impossible) to handle with react-static. Does the above approach sound like the best option to you?

If you're shell is heavy enough or contains vital SEO then yes that would
work. But you would merely be increasing perceived performance no? This is
also something that would need to live on the server redirect level. So
it's just barely out of reach for react statics domain
On Thu, Feb 8, 2018 at 12:21 AM Matthew Sisley<EMAIL_ADDRESS>wrote:

Let's say we have a group of dynamic child routes following /invoices/:id
that all have a shared app shell. Ideally, a pre-rendered shell would be
displayed if a route matching this pattern was the entry point (and the :id
specific stuff could loaded and displayed dynamically).
Something like configuring redirects on a static host to resolve
/invoices/* to a pre-rendered invoiceShell.html should work (and then the
rest of the routing could be handled dynamically).
To use this method with react-static, we could define the shell in a
component, and then render a dummy route in the static.config.js that
would never be intended for ultimate use on its own.
I imagine this is difficult (impossible) to handle with react-static.
Does the above approach sound like the best option to you?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/360, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCeYths72fkMnPaOdd1m37KYlTwjeks5tSqBbgaJpZM4R951J
.

You could also use a query parameter. That would ensure the correct page
loads without a redirect.
On Thu, Feb 8, 2018 at 12:24 AM Tanner Linsley<EMAIL_ADDRESS>wrote:

If you're shell is heavy enough or contains vital SEO then yes that would
work. But you would merely be increasing perceived performance no? This is
also something that would need to live on the server redirect level. So
it's just barely out of reach for react statics domain
On Thu, Feb 8, 2018 at 12:21 AM Matthew Sisley<EMAIL_ADDRESS>wrote:

Let's say we have a group of dynamic child routes following /invoices/:id
that all have a shared app shell. Ideally, a pre-rendered shell would be
displayed if a route matching this pattern was the entry point (and the :id
specific stuff could loaded and displayed dynamically).
Something like configuring redirects on a static host to resolve
/invoices/* to a pre-rendered invoiceShell.html should work (and then
the rest of the routing could be handled dynamically).
To use this method with react-static, we could define the shell in a
component, and then render a dummy route in the static.config.js that
would never be intended for ultimate use on its own.
I imagine this is difficult (impossible) to handle with react-static.
Does the above approach sound like the best option to you?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/360, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCeYths72fkMnPaOdd1m37KYlTwjeks5tSqBbgaJpZM4R951J
.

Yeah, I believe the primary benefit of the app shell is perceived performance b/c less time to first paint (and like you point out, potential SEO impact). Yeah, I assumed it was outside of react-static's domain, but thought I would confirm. So query params or server redirects it is for this. Thanks!


Yet another (really minor) improvement to development process

This PR contains:

FIX for a prow issue introduced by #72
new make target called destroy that deletes the current porch deployment
better strategy to keep (kpt) state between porch deployments (instead of trying to preserve the resourcegroup.yaml file)
use server-side apply for porch deployments
do not build container images of locally running processes

WARNING: unfortunately the new porch deploy method conflicts with the old one, so you have to delete and recreate your kind cluster after this change is merged into your local setup.

/retest


(3.0/3.1) BIO_s_dgram: add documentation and hazard warnings

Backport of #19795.

Looking for a second approval on this.

This pull request is ready to merge

Merged to 3.1 and 3.0.


Fix generator spec generator

Context
Rails offer the possibility to create our own set of generators [1]. While executing the following command
bin/rails generate generator my_awesome_generator

the test framework is called
See hook_for :test_framework
RSpec must give the expected generator
Problem
The given command get this error
bin/rails generate generator foo
      create  lib/generators/foo
      create  lib/generators/foo/foo_generator.rb
      create  lib/generators/foo/USAGE
      create  lib/generators/foo/templates
       error  rspec [not found]

History
@yhirano55 provided an implementation https://github.com/rspec/rspec-rails/pull/1847 Jul 19, 2017, which I don't know why it wasn't integrated in the first place, @penelopezone?
@tragiclifestories opened an issue about it https://github.com/rspec/rspec-rails/issues/2084 on  Feb 26, 2019.
@ConSou implemented a version https://github.com/rspec/rspec-rails/pull/2085 the same day and it was merged a couple of days after by @JonRowe on Mar 1, 2019.
However, as @kdiogenes commented a couple of days ago here https://github.com/rspec/rspec-rails/pull/2085#discussion_r348795229 this doesn't work.
Environment

Rails 6.0.1
Rspec Rails (3.9.0)

[1] https://guides.rubyonrails.org/generators.html#customizing-your-workflow

This PR fix the issue.
bin/rails generate generator --help
Running via Spring preloader in process 26116
Usage:
  rails generate generator NAME [options]

Options:
      [--skip-namespace], [--no-skip-namespace]  # Skip namespace (affects only isolated applications)
      [--namespace], [--no-namespace]            # Namespace generator under lib/generators/name
                                                 # Default: true
  -t, [--test-framework=NAME]                    # Test framework to be invoked
                                                 # Default: rspec

Rspec options:
  [--generator-specs], [--no-generator-specs]  # Generate generator specs

Runtime options:
  -f, [--force]                    # Overwrite files that already exist
  -p, [--pretend], [--no-pretend]  # Run but do not make any changes
  -q, [--quiet], [--no-quiet]      # Suppress status output
  -s, [--skip], [--no-skip]        # Skip files that already exist

Description:
    Stubs out a new generator at lib/generators. Pass the generator name as an argument,
    either CamelCased or snake_cased.

Example:
    `rails generate generator Awesome`

    creates a standard awesome generator:
        lib/generators/awesome/
        lib/generators/awesome/awesome_generator.rb
        lib/generators/awesome/USAGE
        lib/generators/awesome/templates/
        test/lib/generators/awesome_generator_test.rb

Rspec options:
  [--generator-specs], [--no-generator-specs]  # Generate generator specs

bin/rails generate generator awesome
Running via Spring preloader in process 25980
       exist  lib/generators/awesome
      create  lib/generators/awesome/awesome_generator.rb
      create  lib/generators/awesome/USAGE
       exist  lib/generators/awesome/templates
      invoke  rspec
      create    spec/generator/awesomes_generator_spec.rb

Can you add a cucumber scenario demonstrating their use, thats one reason why this was merged broken in the first place :)

@JonRowe I have no clue how to write that, are there scenarios already for any generators I can use as inspirations?

@JonRowe I have no clue how to write that, are there scenarios already for any generators I can use as inspirations?

There's only a markdown file, but the cucumber scenarios pretty much have the affect of running a command line er command, and then introspecting the output, any of them should be enough to get going, you essentially want a when step that runs the command and some then steps that check the generated output.
The benefit is they are isolated from rspec so can use the rails commands and thus check the loading etc

Can you rebase this?

Can you rebase this?

Is that okay?

Absolutely.

@joel does this now work for you? How are you going with a cucumber scenario?

@joel Are you up to wrap this up? Does this work as expected now?

@pirj it works as expected since the beginning haha 😆 I just didn't find the time to add the cucumber scenario asked by @JonRowe, I am not a Cucumber guy and there are not cucumbers scenarios I can use as example for the generators.
I can rebase if needed

I can help if needed for Cucumber @joel. 🙇🏻♂

@benoittgt it would be awesome! Can you open a PR on top of mine?

I am facing this issue yet again with Rails <IP_ADDRESS> and RSpec 3.9. Is there anything else that needs to be done when using rspec?
λ rails -v
Rails <IP_ADDRESS>

λ rspec -v
RSpec 3.9                                                                                - rspec-core 3.9.1                                                                                - rspec-expectations 3.9.0
  - rspec-mocks 3.9.1
  - rspec-rails 3.9.0
  - rspec-support 3.9.2

λ rails generate generator --help
Usage:
  rails generate generator NAME [options]

Options:
      [--skip-namespace], [--no-skip-namespace]  # Skip namespace (affects only isolated applications)
      [--namespace], [--no-namespace]            # Namespace generator under lib/generators/name
                                                 # Default: true
  -t, [--test-framework=NAME]                    # Test framework to be invoked
                                                 # Default: rspec

Runtime options:
  -f, [--force]                    # Overwrite files that already exist
  -p, [--pretend], [--no-pretend]  # Run but do not make any changes
  -q, [--quiet], [--no-quiet]      # Suppress status output
  -s, [--skip], [--no-skip]        # Skip files that already exist

Description:
    Stubs out a new generator at lib/generators. Pass the generator name as an argument,
    either CamelCased or snake_cased.

Example:
    `rails generate generator Awesome`

    creates a standard awesome generator:
        lib/generators/awesome/
        lib/generators/awesome/awesome_generator.rb
        lib/generators/awesome/USAGE
        lib/generators/awesome/templates/
        test/lib/generators/awesome_generator_test.rb

For Rails 6.0 support you need rspec-rails > 4.0.0.
(Repeated despite edit to make it clear to future travellers).


MTL-2141

Summary and Scope

Fixes: MTL-2141
Relates to: HPCDC-7633 (LR1 work)

Issue Type

RFE Pull Request

Offers support for HTTP/HTTPS proxies for fetching artifacts.
get-sqfs.sh -P http://hpeproxy.its.hpecorp.net:443/ e3346d4-1683407921384
Assuming given ID is for Kubernetes and Storage-CEPH
/var/www/ephemeral/data/ceph ~
Downloading storage-ceph artifacts ...
e3346d4-1683407921384/index.html.tmp                                       [ <=>                                                                                ]   4.33K  --.-KB/s    in 0s
e3346d4-1683407921384/5.14.21-150400.24.60-default-e3346d4-16834079213 100%[==========================================================================================================================================================================>]  10.92M  8.62MB/s    in 1.3s
e3346d4-1683407921384/initrd.img-e3346d4-1683407921384-x86_64.xz       100%[==========================================================================================================================================================================>]  93.37M  10.9MB/s    in 8.6s
e3346d4-1683407921384/storage-ceph-e3346d4-1683407921384-x86_64.squash  80%[========================================================================================================================================>                                  ]   2.62G  11.0MB/s    eta 72s

Also adds:

support for other artifacts; pre-install-toolkit ISOs, and hypervisor images
ability to exclusively download Kubernetes or Storage-CEPH, or both

Prerequisites

[ ] I have included documentation in my PR (or it is not required)
[x] I tested this on internal system (if yes, please include results or a description of the test)
[ ] I tested this on a vshasta system (if yes, please include results or a description of the test)

Idempotency

Risks and Mitigations

Need to still test this in vshasta to verify it works when no http proxy is set.