pack build fails on v0.2.0

Hello, today I tried to build my app and failed. When running pack build on a php application I get the following error:
pack build --trust-builder -v ${IMAGE_NAME}:${BITBUCKET_COMMIT} --builder heroku/builder:24 \
    -t ${IMAGE_NAME}:latest --publish --cache-image ${CACHE_IMAGE_NAME}  \
    --buildpack heroku/php --default-process web

[.....]

===> BUILDING
Timer: Builder started at 2024-06-26T16:29:08Z
target distro name/version labels not found, reading /etc/os-release file
Running build for buildpack<EMAIL_ADDRESS>Looking up buildpack
Finding plan
Creating plan directory
Preparing paths
Running build command
thread 'main' panicked at buildpacks/php/src/main.rs:65:14:
Internal error: could not determine Heroku stack name for OS/distro: "-ubuntu-24.04"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Timer: Builder ran for 7.244296ms and ended at 2024-06-26T16:29:08Z
ERROR: failed to build: exit status 101
ERROR: failed to build: executing lifecycle: failed with status code: 51

The same pack build command worked ok on v0.1.2.
I tried with heroku/builder:22 and heroku/builder:20, and removing --buildpack heroku/php parameter, same error.

Hi! I'm puzzled by this log line:
target distro name/version labels not found, reading /etc/os-release file

It should not be appearing, since:

Our images do have those labels set (https://github.com/heroku/base-images/blob/2323b38c808bb4eb555bf5965148f6882b987bd8/heroku-24/Dockerfile#L7-L8)
The upstream lifecycle bug that prevented them being picked up was fixed (https://github.com/buildpacks/lifecycle/issues/1324) and our images are using latest lifecycle that includes that fix

Could you paste the output of running pack report?

Sure, here it is:
Pack:
  Version:  0.27.0
  OS/Arch:  linux/amd64
Default Lifecycle Version:  0.14.1
Supported Platform APIs:  0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9
Config:
(no config file found at /root/.pack/config.toml)

I am running pack from a bitbucket runner container.

Parsing inputs...
Ensuring privileges...
Executing command...
===> ANALYZING
Timer: Analyzer started at 2024-07-03T19:19:26Z
Image with name "123456789012.dkr.ecr.eu-central-1.amazonaws.com/wp-myapp:6d91497a4a4653f8f87c0160192b1e91d6eca849" not found
Found image with identifier "index.docker.io/heroku/heroku@sha256:e0ed1f656a1e49460149ad73c615714ba7535b399a48982eca589b3530d5ee24"
Timer: Analyzer ran for 582.541759ms and ended at 2024-07-03T19:19:27Z
Run image info in analyzed metadata is:
{"Reference":"index.docker.io/heroku/heroku@sha256:e0ed1f656a1e49460149ad73c615714ba7535b399a48982eca589b3530d5ee24","Image":"","Extend":false}
===> DETECTING
Timer: Detector started at 2024-07-03T19:19:27Z
target distro name/version labels not found, reading /etc/os-release file
======== Results ========
pass<EMAIL_ADDRESS>Resolving plan... (try #1)
heroku/php 0.2.0
Timer: Detector ran for 7.934738ms and ended at 2024-07-03T19:19:27Z
===> RESTORING
Timer: Restorer started at 2024-07-03T19:19:27Z
Restoring Layer Metadata
Reading buildpack directory: /layers/heroku_php
Restoring metadata for "heroku/php:composer_cache" from cache
Writing layer metadata for "heroku/php:composer_cache"
Restoring metadata for "heroku/php:platform_cache" from cache
Writing layer metadata for "heroku/php:platform_cache"
Restoring metadata for "heroku/php:shim" from cache
Writing layer metadata for "heroku/php:shim"
Reading Buildpack Layers directory /layers
Reading buildpack directory: /layers/heroku_php
Reading buildpack directory item: composer_cache.toml
Reading buildpack directory item: platform_cache.toml
Reading buildpack directory item: shim.toml
Restoring data for "heroku/php:composer_cache" from cache
Restoring data for "heroku/php:platform_cache" from cache
Restoring data for "heroku/php:shim" from cache
Retrieving data for "sha256:9ef5011a080c7eb5aa190f752959e32f92335cdb80dde52199ed5a87ba9f6d9e"
Retrieving data for "sha256:6d3489c03b8be6a39f83075b075f60ca117a4e32d33348cc59952a42996c8365"
Retrieving data for "sha256:a08370ff5d1e1eb43b617b186599859f181ba1d93cdcecdef9d239cdb96aea7e"
Timer: Restorer ran for 5.001450303s and ended at 2024-07-03T19:19:32Z
===> BUILDING
Timer: Builder started at 2024-07-03T19:19:32Z
target distro name/version labels not found, reading /etc/os-release file
Running build for buildpack<EMAIL_ADDRESS>Looking up buildpack
Finding plan
Creating plan directory
Preparing paths
Running build command
thread 'main' panicked at buildpacks/php/src/main.rs:65:14:
Internal error: could not determine Heroku stack name for OS/distro: "-ubuntu-20.04"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Timer: Builder ran for 7.129431ms and ended at 2024-07-03T19:19:32Z
ERROR: failed to build: exit status 101
ERROR: failed to build: executing lifecycle: failed with status code: 51

Thank you Ed. I wrongly asumed that the Pack CLI was installed on each pipeline. But it was installed on the base build image and as it wasn't updated.
I updated Pack CLI manually and now deployment worked ok.
Thank you very much for your help.

No problem! Thank you for reporting the issue - it let me find an upstream bug which would be great to fix regardless :-)


Can we use redux toolkit / redux toolkit query with sdk ?

I just wanted to know if we can use RTK/RTK Query  with frappe react sdk instead of contexts in large scale applications ?

The SDK uses useSWR internally, which handles the caching and revalidation.
I haven't used RTK Query but I don't think you can use it with this SDK. If you want to build a library that uses RTK, you can do that by using Frappe-JS-SDK.


Fatal error

Hello,
im trying to use the spotify api, worked really good, but now i'm having some issues which i'm not being able to fix
/vendor/jwilsson/spotify-web-api-php/src/Request.php(239): SpotifyWebAPI\Request->handleResponseError(Object(stdClass), 403)
#1 /home/akaggdvp/public_html/test-flc/api/vendor/jwilsson/spotify-web-api-php/src/Request.php(129): SpotifyWebAPI\Request->send('GET', 'https://api.spo...', '', Array)
/vendor/jwilsson/spotify-web-api-php/src/SpotifyWebAPI.php(124): SpotifyWebAPI\Request->api('GET', '/v1/me', Array, Array)
/vendor/jwilsson/spotify-web-api-php/src/SpotifyWebAPI.php(1556): SpotifyWebAPI\SpotifyWebAPI->sendRequest('GET', '/v1/me')
vendor/index.php(16): SpotifyWebAPI\SpotifyWebAPI->me(Object(SpotifyWebAPI\Session))
#5 {main}
thrown in /vendor/jwilsson/spotify-web-api-php/src/Request.php on line 59
removed the /home/ dir, etc.
I'm not sure if the "array" means that the client id and secret aren't getting read, but, if there's any possible fix, let me know.

I just ran in the same issue. You need to add the username of the account you want to give access to the API. This is a recent change and not documented in the readme.
So next to edit settings, also set the users you'd like to give access.

I just ran in the same issue. You need to add the username of the account you want to give access to the API. This is a recent change and not documented in the readme. So next to edit settings, also set the users you'd like to give access.

Alright, thanks.
Also, is it possible to manage multiple users playlist via $api->addPlaylistTracks?
so for example, i have 2 access_tokens on my database, and i want to insert multiple tracks to both of those account at the same time, how is that possible?

I'm still getting the same error even after adding the account on dashboard at users and access
the
SpotifyWebAPI\Request->send('GET', 'https://api.spo...', '', Array)
SpotifyWebAPI\Request->api('GET', '/v1/me', Array, Array)
no idea what this is, this works, then after a while just doesn't. Idk what this is...

Hey!
Could you wrap your call to $api->me() in a try/catch and print the exception so we get the exact error?
Regarding your other question, there shouldn't be any problems keeping multiple access tokens in a database and then changing the access tokens on a SpotifyWebAPI instance on the fly. Quick and dirty example:
$accessTokens = [...]; // Fetched from your database

foreach ($accessTokens as $accessToken) {
  $api->setAccessToken($accessToken);

  $api->addPlaylistTracks(...);
}

And thanks @kasperkamperman, I'll add a note about it to the README.

Hello! I've tried using the Try { } then:
catch (Exception $e) {
$ex = $e->getMessage();
die('Caught exception: '.$ex.'');
}
the reponse i'm getting is "An unknown error occurred."
What do you think the error might be?
Thanks.

Hey!
Can you try again with this code, hopefully we'll get some more info:
try {
  // Anything else you might have...

  $api->me();
} catch (Exception $e) {
   $lastResponse = $api->getRequest()->getLastResponse();

  var_dump($lastResponse);
}

There's a really long message from the last response:
array(4) { ["body"]=> NULL ["headers"]=> array(14) { ["cache-control"]=> string(18) "private, max-age=0" ["access-control-allow-origin"]=> string(1) "*" ["access-control-allow-headers"]=> string(150) "Accept, App-Platform, Authorization, Content-Type, Origin, Retry-After, Spotify-App-Version, X-Cloud-Trace-Context, client-token, content-access-token" ["access-control-allow-methods"]=> string(38) "GET, POST, OPTIONS, PUT, DELETE, PATCH" ["access-control-allow-credentials"]=> string(4) "true" ["access-control-max-age"]=> string(6) "604800" ["content-encoding"]=> string(4) "gzip" ["content-length"]=> string(2) "71" ["strict-transport-security"]=> string(16) "max-age=31536000" ["x-content-type-options"]=> string(7) "nosniff" ["date"]=> string(29) "Sat, 02 Oct 2021 15:03:04 GMT" ["server"]=> string(5) "envoy" ["via"]=> string(28) "HTTP/2 edgeproxy, 1.1 google" ["alt-svc"]=> string(5) "clear" } ["status"]=> int(403) ["url"]=> string(29) "https://api.spotify.com/v1/me" }

Hmm, I did some further digging with a new Spotify app and a user that wasn't added to that app and I got the exact same error. But once I added the user to the app it started working (it took a few minutes and might require a relog).
So I just wanted as you to double check your app settings as mentioned above so see that the user is in fact added to the app.
If you're still getting errors could you add the following line to the Request.php file in the vendor directory in your project (should be located at vendor/jwilsson/spotify-web-api-php/src/Request.php) around line 223:
[$headers, $body] = $this->splitResponse($response);
var_dump($body); // Add this line
$body = json_decode($body, $this->options['return_assoc']);

And see if we're getting an even better error. I'll also look into making this process easier with a proper way to do it without having to hack the library code yourself 😅

For sure. How exactly do you mean to add the user to the app manually? as i want the application to be fully automated. By user are your refering to the API account or the logining one from the site?
And i'll try to do that and give you a response soon.

Yeah, while your app is in Development Mode I'm afraid you'll need to add users to your app manually (by using the email they used to sign up for Spotify with), but when you're feeling ready to publish it you can request Extended Quota Mode from Spotify which will allow users to be added automatically.
Spotify has some more info around it: https://developer.spotify.com/community/news/2021/05/27/improving-the-developer-and-user-experience-for-third-party-apps/

I've just pushed 5.0.1 which should give you a better exception message without having to hack the library files. Could you run it again wrapping the $api->me() call in a try/catch and printing the error?
In my case Spotify replied with "User not registered in the Developer Dashboard" which went away as soon as I added the user to the app.


(node/perfsonar1.ls.lsst.org) add routes for es.net pS nodes

As there have been no comments after 5 days, I'm going to go ahead and merge this.


WIP: Add a wasi-ephemeral crate to build with ephemeral WASI interfaces

Developers working on WASI interfaces can use the wasi-ephemeral crate
as a path dependency or git dependency, to build and run with the
ephemeral WASI interface rather than the latest snapshot.
(wasi-ephemeral uses publish=false and will not be uploaded to
crates.io.)
wasi-ephemeral builds with a build.rs script, so that it automatically
rebuilds on changes to the witx file.
wasi-ephemeral also supports building with an arbitrary witx file, using
the WASI_EPHEMERAL_WITX environment variable.
Note that wasi-ephemeral won't build with the current
wasi_ephemeral_preview.witx until generate-raw can render handles (and
no longer panics with "don't know how to render a handle"). However, I
can confirm that it works with WASI_EPHEMERAL_WITX pointed to either
wasi_snapshot_preview1 (in which case wasi-ephemeral becomes a copy
of wasi) or the old wasi_unstable.

The first two commits are https://github.com/bytecodealliance/wasi/pull/29 and https://github.com/bytecodealliance/wasi/pull/30, which can be merged independently of this.

Want to also add a README to the new crate directory explaining how to use it?

@alexcrichton Done! As soon as #33 goes in, this can go in.

Oh as one final comment, can this be built on CI? Otherwise r=me

@alexcrichton I'd love to do that; are we prepared to commit to supporting builds of any new version of the WASI submodule and their versions of wasi-ephemeral (that we update the submodule to)? That would mean we have to make changes like #33 before we can pull in new versions of the WASI submodule. I'm entirely in favor of that, I just want to confirm that others are as well before adding that to CI.

That's a good point, but I would prefer that we test building the crate while we have the crate, and then if a WASI submodule update was too difficult because it breaks the ephemeral crate, we could just comment out that CI config until it's fixed.

👍

Thanks!


App react works on simulator but not on IOS device

react: 16.0.0-alpha.12
react-native: "0.46.4
react-native-cli: "2.0.1
Xcode version: 9.2
Device IOS Version: 11.3
Device Simulator version: 11.2

[x] I have reviewed the documentation
[x] I have searched existing issues

I'm having trouble generating the application on the iphone 8 device. When I'm programming in the simulator, the application generates the bundle correctly, but when doing the same process for device, it returns me the following error:
'React/RCTUIManagerUtils.h' file not found

Someone has gone through this type of error and can help me?

any solutions ?, having same problem...


Fix build - Pester assertions not found

Fix  'MatchArrayOrdered' is not a valid Should operator in build: https://ci.appveyor.com/project/stuartleeks/posh-humpcompletion

After investigation, appveyor now includes Pester v4 which changes how assertions are registered.
Current status:

I can't find a way to revert the Pester version on appveyor: https://github.com/appveyor/ci/issues/1681
And Pester v4 doesn't currently expose the function to register custom assertions: https://github.com/stuartleeks/PesterMatchArray/issues/1

Waiting on either of those changing...

Fixed by #4


Lock down optparse-applicative to <0.17

There seems to be a breaking change in the API for optparse-applicative v0.17 and above, which causes the master cabal cache run to fail

Version 0.17.1 must be avoided, it was released between building the upgrade PR and merging it :)
https://github.com/pcapriotti/optparse-applicative/releases/tag/0.17.1
That version allows ansi-wl-pprint-1.0 which introduces deprecation warnings. I constrained the version to < 0.17.1 (consistent with stack resolver)


Click-and-drag does not cause search field to lose focus

Windows Terminal version
1.20.11781.0
Windows build number
10.0.19045.4529
Other Software
No response
Steps to reproduce
Settings:

"Automatically copy selection to clipboard" (copyOnSelect): false
"Automatically focus pane on mouse hover" (focusFollowMouse): false

Steps:

Echo some arbitrary text on the console
(not needed if for example PowerShell automatically printed its version; or the current working directory)
Open the search: Ctrl + Shift + F
Click-and-drag to select any of the output from step 1
ℹ Note: Do not perform a separate left click; instead press down the mouse button and directly start dragging your mouse
Try for example to copy the selection with Ctrl + C

Expected Behavior
Step 3: The search text field lost focus; the console output got focus
Step 4: You copied your selection to the clipboard
Actual Behavior
Step 3: A text selection is shown for the console output, but the search text field still has focus
Step 4: You did not copy anything because the search text field still has focus
Side note: If instead of click-and-drag you perform a separate click (press and release) first, then the console output properly has focus.

Good catch!


Split extra vars for Ansible playbooks

This PR is a rebase of https://github.com/ManageIQ/manageiq/pull/19864.
With #19698, we switched to using AwesomeSpawn to build the command lines, including ansible-playbook. So far, we used a single --extra-vars option with a JSON string, but the command line generated by AwesomeSpawn is broken, probably by too much escaping.
Given that extra_vars is a hash where each value is a string, this pull requests iterates over the hash entries to add them as independent --extra-vars option. This way, there no conflict between the generated JSON string and the escaping of AwesomeSpawn.


Hi, stop recording from notification banner

Hi, can we add stop screen record from notification banner. And can we make the custom text of the permission dialog.

Hi @RAIERIN
that will require a bit of config when setting up recording, particularly a listener / function to be called when the banner is pressed.
I'll look into it this weekend

Thanks @umbertoghio.

Hi @RAIERIN , I've made a new release, now if you add a androidBannerStopRecordingHandler callback function it will show and handle the future properly.
The permission dialog

is handled by Android, I haven't found a way to customize it
Feel free to send PRs in case you figure it out!


Automatic PR from RestAPI-PR708

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/708

Includes https://github.com/Azure/azure-sdk-for-python/pull/870


[fix] configurable concurrency

@edulop91 The listroles function takes a really long time here... I'm not sure why.


Custom API - Allow for use of ES6 import

Environment - Local/Nhost Dev
Machine - Apple M1 Macbook Pro
Issue
When trying to create a custom API, I installed the Twilio package and the nhost_api server crashed and logged the following
nhost_api | import net from 'net';

nhost_api | ^^^^^^

nhost_api |

nhost_api | SyntaxError: Cannot use import statement outside a module

nhost_api | at wrapSafe (internal/modules/cjs/loader.js:979:16)

nhost_api | at Module._compile (internal/modules/cjs/loader.js:1027:27)

nhost_api | at Module._extensions..js (internal/modules/cjs/loader.js:1092:10)

nhost_api | at Object.require.extensions.<computed> [as .ts] (/usr/src/app/node_modules/ts-node/src/index.ts:1286:43)

nhost_api | at Module.load (internal/modules/cjs/loader.js:928:32)

nhost_api | at Function.Module._load (internal/modules/cjs/loader.js:769:14)

nhost_api | at Module.require (internal/modules/cjs/loader.js:952:19)

nhost_api | at require (internal/modules/cjs/helpers.js:88:18)

nhost_api | at /usr/src/app/index.ts:17:26

nhost_api | at Array.forEach (<anonymous>)

Expectation
I should be able to import an npm package without issue

Closing this. It was an error on my behalf not realizing that the node modules needed to exist at the root api directory.


Add e2e test harness template file

What type of PR is this?
test
What this PR does / why we need it?

Update boilerplate

Add test-harness-template.yml file.
It's e2e-harness update and no functional changes are made.

Which Jira/Github issue(s) this PR fixes?
Fixes # https://issues.redhat.com/browse/SDCICD-1130
Special notes for your reviewer:
Pre-checks (if applicable):

[x] Tested latest changes against a cluster
[ ] Included documentation changes with PR

Codecov Report

Merging #156 (8b4951d) into master (da875bf) will increase coverage by 18.83%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##           master     #156       +/-   ##
===========================================
+ Coverage   61.39%   80.22%   +18.83%
===========================================
  Files           8        6        -2
  Lines         474      359      -115
===========================================
- Hits          291      288        -3
+ Misses        176       64      -112
  Partials        7        7

see 2 files with indirect coverage changes

/lgtm

/approve


Add OAuth2::Client#http_client

This patch let's OAuth2::Client use a configurable HTTP::Client instance.
A use case is already demonstrated in the second commit: Specs can inject an instance with a hard-wired handler, thus skipping the network stack.
This also allows more customization, for example when using custom certificates.
The default behaviour stays the same, i.e. every request creates a new HTTP::Client instance. The autocreated client is not cached by default because there's no proper connection management (see #6011) and making a single authentication request could keep up a long-running HTTP connection.

Resolves #4382


[release-2.8]Ensure policies remain when hosting cluster is a hub

The bug was by deleting the hcp cluster, we see that all the ACM policies on the ACM hub are also deleted.
Signed-off-by: Yi Rae Kim<EMAIL_ADDRESS>(cherry picked from commit c89a259ee16d48951964fe4e26242f0cc543188c)

/unhold

/cherry-pick release-2.7


Add http error identification for credentials and hostname errors

Internal ticket: https://jira.cloudera.com/browse/DBT-398
The error printed seems to be generic, so adding
code to identity the failure from Impala connection.
Test plan:

Move into a dbt impala project and install dbt-impala from this git repo
Add profile details in your home dbt file like below,
dbt_impala_demo:
outputs:
dev:
auth_type: ldap
host: {}
http_path: {}
password: {}
port: 443
schema: dbt_impala_demo
dbname: dbt_impala_demo
threads: 4
type: impala
use_http_transport: true
use_ssl: true
user: {}
target: dev
Update with wrong hostname or wrong user credentials
run > dbt debug, and you should see the type of error in the console
Add below snowplow configs to your .env if you face any snowplow error
SNOWPLOW_ENDPOINT=https://dcevents.cldrteam.datacoral.io/cldrteam/apievents
SNOWPLOW_TIMEOUT=10
SNOWPLOW_API_KEY=<your_key>
SNOWPLOW_ENNV=prod

Signed-off-by: Sanjeev kumar N<EMAIL_ADDRESS>

Note: For both Impala server connection and sql query error, it throws same "HiveServer2Error" exception class.


Update Custom Checkout types

Summary & motivation
r? @pololi-stripe
Update Custom Checkout types for custom_checkout_beta_2

Added lineItem.amountsSubtotal, amountDiscount, amountTaxInclusive, amountTaxExclusive
Removed lineItem.amount
Added lineItem.recurring.usageType, isProrated, intervalCount
Removed lineITem.recurring.interval_count
Added total.appliedBalance and balanceAppliedToNextInvoice
Added tax
Added status
Added recurring
Added lastPaymentError

Testing & documentation

r? @pololi-stripe
Updated in b475dd903dc708b92a58b29687ca69beb5e0bcc1 and f175d7e

Added line item adjustable quantity (forgot this earlier)
Updated type for ECE. To do this, I split all the ECE events into separate interfaces, then recombined them in the ECE type, and selected only the applicable events for Custom Checkout.

r? @pololi-stripe
I ditched the strategy of trying to reuse existing definitions. Instead, I just copied in the on/once/off methods for all events that we need, changing the return type in e80ef80c8738643fa5a9218b0df7a2f8695a4474


ActionMenuItemView title briefly visible when inflating new menu

[x] I have verified there are no duplicate active or recent bugs, questions, or requests

[x] I have verified that I am using the latest version of Aesthetic (see the README)

[x] I have given my issue description title

[x] I have read over the documentation (before asking questions on how to do something).

Aesthetic version: 0.4.7

Device OS version: 7.1.2

Device Manufacturer: Huawei

Device Name: Nexus 6P

Reproduction Steps
When switching between pages, for example in a FragmentViewPager, if the individual Fragments inflate their own Menus (via toolbar.setMenu()) the MenuItem's text can be seen, before the icon is shown. This is best described as a 'flicker' when changing menus. Most obvious or perhaps only replicated if the MenuItems actually differ between the two Fragments.
See below:

Note: The text displays for slightly longer than it seems in the gif, looks like a few frames were skipped.
Expected Result
The MenuItem text should not 'flash' on the screen before the icon is shown.
Additional
I've identified the source of this bug, and will provide a PR with the solution shortly.

I the same issue in my app


Add brils, mostly failing

This PR adds more bril programs as examples, including some that demonstrate current limitations/bugs.
cycle.bril fails because we always run the interpreter.
fib_recursive.bril fails because functions aren't supported yet.
fib.bril fails, see #71
duplicate_branch.bril succeeds.

Looks like you need to run cargo insta review
You should look at the results and check that they look reasonable for each kind of output- it's okay if they don't make sense you can accept them.


Icon path is not working in AppShell.razor

I'm using 0.6.69-preview of MBB
In the AppShell.razor the icon path is not recognized, I tried placing the png in wwwroot and also the project root folder as a content (build type).
<Shell BackgroundColor="XF.Color.Black">     <TabBar Title="Something">         <Tab Title="Articles" Icon="something.png">             <ShellContent>            ...             </ShellContent>         </Tab>

Sorry, I just referred the MMB Xaminals example and found that I need to use:
Icon="@(new FileImageSource { File="paw.png" })
Thank you so much for those thoughtful samples.

Just an additional suggestion.
Put that FileImageSource to the field instead of creating it razor directly - it will help to avoid visual glitches (we need to update that sample).

Thank you so much for the suggestion, @Dreamescaper


Fix console error in census page example

What is the context of this PR?
Fixes: #1021 and improves docs for navigation and mobile menu toggle button
How to review
Check page example no longer has a console error

 Issue #1021: Console error on Page Template Page


GetAndRemove-SFTPFile

Hi Darkoperator, Hi Internet
I’m not 100% shure if it’s the right place to posting it, so if it is wrong, please ignore it :-).
I have a small script witch does logon to some ssh-servers, list a directory there, download all files from the directory and then delete the files from the ssh-server.
A couple days ago, we moved the script to another server (with windows Server 2012 R2) and something bad has happened. Through a bug/error/mistake/ ever  in the Task Scheduler “new” Unified Background Process Manager (UBPM) environment variables are not 100% loaded to the PowerShell (https://support.microsoft.com/en-us/help/3133689/ubpm-doesn-t-set-environmental-variables-correctly-when-you-run-schedu) witch run’s in the Task Scheduler with a user how is not logged in on the server the script failed with the folling conditions:
Files not downloaded from remote ssh-server and file deleted from remote server.
Okay now are more way’s to ensure that this situation never happened again.
My Approach was to compare Remote and Local files and then start the remove (only if they marched), I did this in PowerShell, it worked very well, and it was easy to implant.
But I guess in a perfect world the “Get-SCPFile” should have an Parameter like compareAfterDownload and will then return a value or so. What do you think ?
PS: I’f darkoperator or some other expirenced devloper finde the idee fine, i will try to imeplent it Posh-SSH if that is okay.
Kind Regards
KD

With SCP it would not be possible, probably with SFTP, will add it to my backlog to take a look.
From: KonradDa
Sent: Monday, July 10, 2017 12:47 PM
To: darkoperator/Posh-SSH
Cc: Subscribed
Subject: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
Hi Darkoperator, Hi Internet
I’m not 100% shure if it’s the right place to posting it, so if it is wrong, please ignore it :-).
I have a small script witch does logon to some ssh-servers, list a directory there, download all files from the directory and then delete the files from the ssh-server.
A couple days ago, we moved the script to another server (with windows Server 2012 R2) and something bad has happened. Through a bug/error/mistake/ ever in the Task Scheduler “new” Unified Background Process Manager (UBPM) environment variables are not 100% loaded to the PowerShell (https://support.microsoft.com/en-us/help/3133689/ubpm-doesn-t-set-environmental-variables-correctly-when-you-run-schedu) witch run’s in the Task Scheduler with a user how is not logged in on the server the script failed with the folling conditions:
Files not downloaded from remote ssh-server and file deleted from remote server.
Okay now are more way’s to ensure that this situation never happened again.
My Approach was to compare Remote and Local files and then start the remove (only if they marched), I did this in PowerShell, it worked very well, and it was easy to implant.
But I guess in a perfect world the “Get-SCPFile” should have an Parameter like compareAfterDownload and will then return a value or so. What do you think ?
PS: I’f darkoperator or some other expirenced devloper finde the idee fine, i will try to imeplent it Posh-SSH if that is okay.
Kind Regards
KD
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Remove the ErrorAction to SilentlyContinue. You are telling the command with that to not raise an error, no error then nothing is catched
PS C:\Users\Carlos> try {Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalPath ./bashrc1} catch { write-host "catched"}
Catched
PS C:\Users\Carlos> Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalPath ./bashrc1
Get-SFTPFile : Cannot find path 'C:\Users\Carlos\bashrc1' because it does not exist.
At line:1 char:1

Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalP ...

  + CategoryInfo          : ObjectNotFound: (C:\Users\Carlos\bashrc1:String) [Get-SFTPFile], ItemNotFoundException
  + FullyQualifiedErrorId : PathNotFound,SSH.GetSftpFile

From: Paul Watson
Sent: Friday, May 4, 2018 2:44 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
I, too, am being asked to delete files on the remote host after get. How can I know if Get-SFTPFile failed? I am not able to catch the exception. In this case, I am specifying a filename that I know does not exist on the remote host.
try {
Get-SFTPFile -SFTPSession $Session         -NoProgress
-RemoteFile "/DATA/$LinuxPhaseDir/extracts/RX_340B/$Channel/abc.txt"         -LocalPath $LocalPath
-ErrorAction SilentlyContinue
}
catch {
"failed to retrieve $file.FullName"
}
[DBG]: PS M:\DW_Devl\FileTransfer\340b>>
Get-SFTPFile : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not exist.
At M:\DW_Devl\FileTransfer\340b\atest.ps1:80 char:13

        Get-SFTPFile -SFTPSession $Session `

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (SSH.SftpSession:SftpSession) [Get-SFTPFile], SftpPathNotFoundException
FullyQualifiedErrorId : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not exist.,SSH.GetSftpFile
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Carlos,
Thank your for your message. I removed the -ErrorAction and it fails in the
same way.
My $ErrorActionPreference is 'Continue'.
On Fri, May 4, 2018 at 3:43 PM, Carlos Perez<EMAIL_ADDRESS>wrote:

Remove the ErrorAction to SilentlyContinue. You are telling the command
with that to not raise an error, no error then nothing is catched
PS C:\Users\Carlos> try {Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1} catch { write-host "catched"}
Catched
PS C:\Users\Carlos> Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1
Get-SFTPFile : Cannot find path 'C:\Users\Carlos\bashrc1' because it does
not exist.
At line:1 char:1

Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalP ...

CategoryInfo : ObjectNotFound: (C:\Users\Carlos\bashrc1:String)
[Get-SFTPFile], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,SSH.GetSftpFile

From: Paul Watson
Sent: Friday, May 4, 2018 2:44 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
I, too, am being asked to delete files on the remote host after get. How
can I know if Get-SFTPFile failed? I am not able to catch the exception. In
this case, I am specifying a filename that I know does not exist on the
remote host.
try {
Get-SFTPFile -SFTPSession $Session -NoProgress
-RemoteFile "/DATA/$LinuxPhaseDir/extracts/RX_340B/$Channel/abc.txt" -LocalPath $LocalPath
-ErrorAction SilentlyContinue
}
catch {
"failed to retrieve $file.FullName"
}
[DBG]: PS M:\DW_Devl\FileTransfer\340b>>
Get-SFTPFile : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not
exist.
At M:\DW_Devl\FileTransfer\340b\atest.ps1:80 char:13

Get-SFTPFile -SFTPSession $Session `

CategoryInfo : InvalidOperation: (SSH.SftpSession:SftpSession)
[Get-SFTPFile], SftpPathNotFoundException
FullyQualifiedErrorId : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt
does not exist.,SSH.GetSftpFile
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/darkoperator/Posh-SSH/issues/156#issuecomment-386729413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACaejki9DQa-FgMw8yzkDZ4ZYkidat-wks5tvL2HgaJpZM4OTHQs
.

Again you are telling it to not have a terminating error and continue. Erroraction should be Stop or none specified to the try/catch statement to catch it
From: Paul Watson
Sent: Friday, May 4, 2018 4:50 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
Carlos,
Thank your for your message. I removed the -ErrorAction and it fails in the
same way.
My $ErrorActionPreference is 'Continue'.
On Fri, May 4, 2018 at 3:43 PM, Carlos Perez<EMAIL_ADDRESS>wrote:

Remove the ErrorAction to SilentlyContinue. You are telling the command
with that to not raise an error, no error then nothing is catched
PS C:\Users\Carlos> try {Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1} catch { write-host "catched"}
Catched
PS C:\Users\Carlos> Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1
Get-SFTPFile : Cannot find path 'C:\Users\Carlos\bashrc1' because it does
not exist.
At line:1 char:1

Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalP ...

CategoryInfo : ObjectNotFound: (C:\Users\Carlos\bashrc1:String)
[Get-SFTPFile], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,SSH.GetSftpFile

From: Paul Watson
Sent: Friday, May 4, 2018 2:44 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
I, too, am being asked to delete files on the remote host after get. How
can I know if Get-SFTPFile failed? I am not able to catch the exception. In
this case, I am specifying a filename that I know does not exist on the
remote host.
try {
Get-SFTPFile -SFTPSession $Session -NoProgress
-RemoteFile "/DATA/$LinuxPhaseDir/extracts/RX_340B/$Channel/abc.txt" -LocalPath $LocalPath
-ErrorAction SilentlyContinue
}
catch {
"failed to retrieve $file.FullName"
}
[DBG]: PS M:\DW_Devl\FileTransfer\340b>>
Get-SFTPFile : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not
exist.
At M:\DW_Devl\FileTransfer\340b\atest.ps1:80 char:13

Get-SFTPFile -SFTPSession $Session `

CategoryInfo : InvalidOperation: (SSH.SftpSession:SftpSession)
[Get-SFTPFile], SftpPathNotFoundException
FullyQualifiedErrorId : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt
does not exist.,SSH.GetSftpFile
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/darkoperator/Posh-SSH/issues/156#issuecomment-386729413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACaejki9DQa-FgMw8yzkDZ4ZYkidat-wks5tvL2HgaJpZM4OTHQs
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Carlos,
Perhaps I have it. Because Get-SFTPFile does not produce a terminating
error when the file is not found, I do need to use -ErrorAction Stop in
order to catch it.
Thanks for your guidance to this solution.
On Fri, May 4, 2018 at 3:58 PM, Carlos Perez<EMAIL_ADDRESS>wrote:

Again you are telling it to not have a terminating error and continue.
Erroraction should be Stop or none specified to the try/catch statement to
catch it
From: Paul Watson
Sent: Friday, May 4, 2018 4:50 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
Carlos,
Thank your for your message. I removed the -ErrorAction and it fails in the
same way.
My $ErrorActionPreference is 'Continue'.
On Fri, May 4, 2018 at 3:43 PM, Carlos Perez<EMAIL_ADDRESS>wrote:

Remove the ErrorAction to SilentlyContinue. You are telling the command
with that to not raise an error, no error then nothing is catched
PS C:\Users\Carlos> try {Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1} catch { write-host
"catched"}
Catched
PS C:\Users\Carlos> Get-SFTPFile -SessionId 0 -RemoteFile
"/home/carlos/.bashrc1" -LocalPath ./bashrc1
Get-SFTPFile : Cannot find path 'C:\Users\Carlos\bashrc1' because it does
not exist.
At line:1 char:1

Get-SFTPFile -SessionId 0 -RemoteFile "/home/carlos/.bashrc1" -LocalP
...

CategoryInfo : ObjectNotFound: (C:\Users\Carlos\bashrc1:String)
[Get-SFTPFile], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,SSH.GetSftpFile

From: Paul Watson
Sent: Friday, May 4, 2018 2:44 PM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Comment
Subject: Re: [darkoperator/Posh-SSH] GetAndRemove-SFTPFile (#156)
I, too, am being asked to delete files on the remote host after get. How
can I know if Get-SFTPFile failed? I am not able to catch the exception.
In
this case, I am specifying a filename that I know does not exist on the
remote host.
try {
Get-SFTPFile -SFTPSession $Session -NoProgress
-RemoteFile "/DATA/$LinuxPhaseDir/extracts/RX_340B/$Channel/abc.txt" -LocalPath $LocalPath
-ErrorAction SilentlyContinue
}
catch {
"failed to retrieve $file.FullName"
}
[DBG]: PS M:\DW_Devl\FileTransfer\340b>>
Get-SFTPFile : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt does not
exist.
At M:\DW_Devl\FileTransfer\340b\atest.ps1:80 char:13

Get-SFTPFile -SFTPSession $Session `

CategoryInfo : InvalidOperation: (SSH.SftpSession:SftpSession)
[Get-SFTPFile], SftpPathNotFoundException
FullyQualifiedErrorId : /DATA/PHS_UAT/extracts/RX_340B/Sentry/abc.txt
does not exist.,SSH.GetSftpFile
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/darkoperator/Posh-SSH/issues/
156#issuecomment-386729413>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACaejki9DQa-
FgMw8yzkDZ4ZYkidat-wks5tvL2HgaJpZM4OTHQs>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/darkoperator/Posh-SSH/issues/156#issuecomment-386732742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACaejsW-fGcboMpwlU6QJa-6TDmERkdFks5tvMD3gaJpZM4OTHQs
.


[EAGLE-526] aggregation framework

@wujinhu can you please add sufficient description here for people to understand what aggregation framework is used for?


Cherry-pick jdm's new tests on

Not ready for merge yet; it seems the two highfives' testing strategies have diverged a bit. Opening PR to track progress and remind myself to go through and standardize the test suite at some point.

r? @rust-highfive
(rust_highfive has picked a reviewer for you, use r? to override)


Slayer Plugin highlights Scorpia's offspring on Scorpia boss task

Describe the bug
If you get scorpia as a boss task, and have highlight targets enabled in the Slayer plugin, then Scorpia's offspring and Scorpia's guardians get highlighted as well, even though only the boss should be highlighted.
 Possibly the Scorpia's guardians should be highlighted, but the offspring definitely should not be highlighted.
Environment (please complete the following information):

OS: Windows 10
Latest Runelite version

Additional context
The offending line of code is referenced. Perhaps it is worthwhile to check for scorpia as a special case in this code block.
https://github.com/runelite/runelite/blob/71e548052470332a25c56d26e6f1d70b0ea9bac5/runelite-client/src/main/java/net/runelite/client/plugins/slayer/SlayerPlugin.java#L610-L629

Duplicate of #6285


[BUG] broken tests integration/consensus.test.js

yarn run test_integration_consensus -g 'can penalize malicious validators'
yarn run v1.22.10
$ BLOCKCHAIN_CONFIGS_DIR=blockchain-configs/3-nodes ./node_modules/mocha/bin/mocha --timeout 640000 test/integration/consensus.test.js -g 'can penalize malicious validators'

  Consensus
    Penalties
      1) can penalize malicious validators

  0 passing (2m)
  1 failing

  1) Consensus
       Penalties
         can penalize malicious validators :

      AssertionError: expected<PHONE_NUMBER>262 to deeply equal NaN
      + expected - actual

      -1643440067262
      +NaN

      at Context.<anonymous> (test/integration/consensus.test.js:575:14)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

it seems the 3rd argument is missing for getLockupExtensionForNewOffenses().
could you take a look? @liayoo

Fixed, thanks!


Upgrade --all getting ignored / failing

Brief description of your issue
Winget sees that there are upgrades but..

ignores all but one upgrade when I run upgrade --all.
the one package it does attempt to upgrade errors out with Installer hash does not match.
Could be related to the recent Nvidia breach?

PS C:\Users\<username>\Desktop> winget upgrade --all --source winget
Name                Id                         Version     Available
----------------------------------------------------------------------
Brave               BraveSoftware.BraveBrowser <IP_ADDRESS> <IP_ADDRESS>
VeraCrypt           IDRIX.VeraCrypt            1.25.4      1.25.9
PowerToys (Preview) Microsoft.PowerToys        0.35.0      0.55.2
NVIDIA GeForce NOW  Nvidia.GeForceNow          <IP_ADDRESS>  <IP_ADDRESS>
4 upgrades available.
4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.

(1/1) Found NVIDIA GeForce NOW [Nvidia.GeForceNow] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.nvidia.com/gfnpc/GeForceNOW-release.exe
  ██████████████████████████████   140 MB /  140 MB
Installer hash does not match; to override this check use --force

4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.

Note: PowerToys isn't supposed to upgrade because they only support above 1809, and I'm on 1809, so that behavior is correct.
Steps to reproduce
Run winget upgrade --all --source winget.
Expected behavior
All packages are updated except for the unsupported ones by the system.
Actual behavior
Some packages are ignored, others fail.
Environment
Windows Package Manager (Preview) v1.3.431-preview
Windows: Windows.Desktop v10.0.17763.2628
Package: Microsoft.DesktopAppInstaller v1.18.431.0

Can you run winget upgrade --all --verbose-logs and then attach the log files in your reply? I have a theory as to what the issue is, but the logs will help tremendously, as I'm not able to replicate this on my system.

Decided to try running the command again today:
PS C:\Users\username> winget upgrade --all --source winget
Name                Id                        Version    Available
-------------------------------------------------------------------
Mozilla Thunderbird Mozilla.Thunderbird       91.6.1     91.7.0
VeraCrypt           IDRIX.VeraCrypt           1.25.4     1.25.9
iTunes              Apple.iTunes              <IP_ADDRESS>  <IP_ADDRESS>
PowerToys (Preview) Microsoft.PowerToys       0.35.0     0.56.2
NVIDIA GeForce NOW  Nvidia.GeForceNow         <IP_ADDRESS> <IP_ADDRESS>
Tachidesk Server    Suwayomi.Tachidesk-Server 0.6.1      0.6.2
6 upgrades available.
4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.

(1/4) Found Mozilla Thunderbird [Mozilla.Thunderbird] Version 91.7.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download-installer.cdn.mozilla.net/pub/thunderbird/releases/91.7.0/win64/en-US/Thunderbird%20Setup%2091.7.0.exe
  ██████████████████████████████  53.6 MB / 53.6 MB
Successfully verified installer hash
Starting package install...
Successfully installed

(2/4) Found iTunes [Apple.iTunes] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://secure-appldnld.apple.com/itunes12/002-35070-20220304-5521E72A-137B-4F09-9844-45BEBA5C3B40/iTunes64Setup.exe
  ██████████████████████████████   200 MB /  200 MB
Successfully verified installer hash
Starting package install...
Successfully installed

(3/4) Found NVIDIA GeForce NOW [Nvidia.GeForceNow] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.nvidia.com/gfnpc/GeForceNOW-release.exe
  ██████████████████████████████   140 MB /  140 MB
Successfully verified installer hash
Starting package install...
Successfully installed

(4/4) Found Tachidesk Server [Suwayomi.Tachidesk-Server] Version 0.6.2
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/Suwayomi/Tachidesk-Server/releases/download/v0.6.2/Tachidesk-Server-v0.6.2-r1074-windows-x64.msi
  ██████████████████████████████   217 MB /  217 MB
Successfully verified installer hash
Starting package install...
Successfully installed

4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.
PS C:\Users\username> winget upgrade --all --source winget
Name                Id                        Version Available
---------------------------------------------------------------
VeraCrypt           IDRIX.VeraCrypt           1.25.4  1.25.9
PowerToys (Preview) Microsoft.PowerToys       0.35.0  0.56.2
Tachidesk Server    Suwayomi.Tachidesk-Server 0.6.1   0.6.2
3 upgrades available.
4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.

(1/1) Found Tachidesk Server [Suwayomi.Tachidesk-Server] Version 0.6.2
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/Suwayomi/Tachidesk-Server/releases/download/v0.6.2/Tachidesk-Server-v0.6.2-r1074-windows-x64.msi
  ██████████████████████████████   217 MB /  217 MB
Successfully verified installer hash
Starting package install...
Successfully installed

4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.
PS C:\Users\username> winget upgrade --source winget
Name                Id                        Version Available
---------------------------------------------------------------
VeraCrypt           IDRIX.VeraCrypt           1.25.4  1.25.9
PowerToys (Preview) Microsoft.PowerToys       0.35.0  0.56.2
Tachidesk Server    Suwayomi.Tachidesk-Server 0.6.1   0.6.2
3 upgrades available.
4 packages have version numbers that cannot be determined. Use "--include-unknown" to see all results.

As can be see, one of the packages that refused to update before (Geforce Now) went through this time.
I ran the command multiple times before opening the issue, so something must've changed.
The other package that was giving me trouble (VeraCrypt) is still not updating.
Of note is a new troublesome package (Tachidesk) which shows as updated "successfully", but still shows up as outdated afterward.
What's up with that?
I do not like how inconsistent things are.

Looks like it gets installed side by side

Awesome community for the win!

As can be seen, one of the packages that previously refused to update (Geforce Now) went through this time.
I ran the command multiple times prior to opening the issue, so something must've changed.

I believe this may be an issue with GeForce Now reporting a successful install but not actually installing if the application is currently in use / running

The other package that was giving me trouble (VeraCrypt) is still not updating.

Your issue with VeraCrypt is that the installation currently on your machine was installed with the exe file, and the version in winget uses the MSI file. The easiest way to fix this is to uninstall your current version and re-install. I can replicate this using version 1.1 of the client. Using version 1.2 of the client I get this message -
PS> winget upgrade veracrypt --verbose-logs
A newer version was found, but the install technology is different from the current version installed. Please uninstall the package and install the newer version.

To answer your other questions:

For some reason WinGetbot cannot detect the hash missmatch (as it says on the first post) on Nvidia.GeForceNow so we have to manually update it (i went trough the Nvidia folder Yesterday)
IDRIX.VeraCrypt changed their installer from EXE to MSI and there is no 1.25.9 exe available. (A reinstall/install will resolve the issue)
Microsoft.PowerToys changed their installer from a weird bundled MSI to a EXE. (A reinstall/install will resolve the issue)

If you find any other unresolved package issues you can report them here https://github.com/microsoft/winget-pkgs/issues/new/choose

there is no 1.25.9 exe available

Slight correction - There is an exe available here . I just can't seem to find the secret combination of switches for a silent install

Ops, just looked up my comment previous comment on VeraCrypt

VeraCrypt: I can see that there is a exe install for this software, we don't have the exe version because its not Unattended.

Chocolatey uses AHK for the installation so there is probably a very low chance you'll find the magic switch

Edit: we could make a issue here? https://github.com/veracrypt/VeraCrypt/issues

https://github.com/veracrypt/VeraCrypt/issues/447

We need to be cautious about adding that switch to the manifest. It's better for them to either publish to the Microsoft Store or become a verified developer so we can include Agreements in the manifest to meet the "explicit" agreement requirements.

We need to be cautious about adding that switch to the manifest. It's better for them to either publish to the Microsoft Store or become a verified developer so we can include Agreements in the manifest to meet the "explicit" agreement requirements.

So far as I can tell, it is just the EULA which that switch suppresses, which often happens with any installer which incorporates the EULA agreement into the install process. Its just that many publishers allow the eula to be suppressed with the default silent switch rather than requiring an additional switch

Microsoft.PowerToys changed their installer from a weird bundled MSI to a EXE. (A reinstall/install will resolve the issue)

So PowerToys is not updating because of installer type change, not because it sees I'm on 1809 (an unsupported Windows build)?
Wonder what would happen if it actually went through if that is the case.
Also is there a reason an exe can't "gracefully" be updated into an msi and visa versa?
If there is one, might make sense to at least let the user know that that is the reason nothing is happening.

So PowerToys is not updating because of installer type change, not because it sees I'm on 1809 (an unsupported Windows build)?
Wonder what would happen if it actually went through if that is the case.

The InstallerType changed in version 0.53.1, however winget upgrade did also not work before that because of the scuffed MSI bundle (if you have 0.53.1+ winget upgrade will work as expected)
However i do not know if PowerToys dropped support for 1809 after version 0.35.0, you should probably try to run the installer manually

Also is there a reason an exe can't "gracefully" be updated into an msi and visa versa?

Most of the time the MSI and EXE can be installed side-by-side or work differently

If there is one, might make sense to at least let the user know that that is the reason nothing is happening.

if you do winget upgrade Microsoft.PowerToys you will get A newer version was found, but the install technology is different from the current version installed. Please uninstall the package and install the newer version.
In the future you will be asked if you want to force upgrade (will probably uninstall and install the new version)

if you do winget upgrade Microsoft.PowerToys you will get A newer version was found, but the install technology is different from the current version installed. Please uninstall the package and install the newer version.

I see.
I also get that message for VeraCrypt.
Wouldn't it also make sense to show it when running upgrade --all for the packages that have that problem?


Superset 4.0.2 : Dashboard Reports Appearance via Email

Description
Hello, I recently started using Superset 4.0.2. I have a few problems with it. For example;

When I share a dashboard via email with other people, they only can see first five or six plots in email, but I want all my plot to be seen at my email.
I can't control width and height of cell in Table chart.
If I want to select two of my plots in dashboard and want to share it via email, I can't do that. I only can share one plot as PNG format.
Most importantly, I have a dashboard contains ten table chart and I want to share all my charts as text via email. As I can see Superset has no sharing option for dashboards as text format.

I would be very glad if you could help me with these issues.

@dosu All the solutions you suggested to me include configurations made via Python, but I need to do this on Superset. Could you help me?


Add roundtrip and database etcd metrics

These etcd metrics are interesting and might help to detect some regressions.
Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Pass
0
00:11:13


ci: add dependabot.yml

Integrates with Dependabot to initiate auto-package updates.

Codecov Report

Merging #10 (63ad6e1) into main (548583a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          408       408
=========================================
  Hits           408       408


Pretty URL doesn't work, because REDIRECT_URL is being ignored

Hello,
I have a problem with the pretty URL.
My Development Environment:

Windows 7
XAMPP 1.8.3-4 (Apache 2.4.9, PHP 5.5.11)
Laravel 5.0.14

Productive System:

Ubuntu 14.04.2 LTS
Apache 2.4.7
PHP Version 5.5.9
Laravel 5.0.14

In both environments I can see the following problem.
Here is what I did:

I installed Laravel using composer.

I didn't modify: c:/xampp/laravel-rocks/public/.htaccess

I created c:/xampp/laravel-rocks/.htaccess as follows:
	<IfModule mod_rewrite.c>
		RewriteEngine On
		RewriteRule  ^$ public/    [L]
		RewriteRule  (.*) public/$1 [L]
	</IfModule>

I inserted the following route into app/Htpp/routes.php:

Route::get('/hello', function () {
    return 'Hello!';
});

Symptom:
http://localhost/laravel-rocks/public/hello works fine.
http://localhost/laravel-rocks/hello fires a NotFoundHttpException in RouteCollection.php line 145.
mod_rewrite works, because public/index.php is being called. REQUEST_URI and REDIRECT_URL have the following values:
	$_SERVER['REQUEST_URI'] = "/laravel-rocks/hello"
	$_SERVER['REDIRECT_URL'] = "/laravel-rocks/public/hello"

Unfortunately Laravel ignores REDIRECT_URL!
I've written the following lines in public/index.php as a workaround:
if (isset($_SERVER['REDIRECT_URL'])) {
    $_SERVER['REQUEST_URI'] = $_SERVER['REDIRECT_URL'];
}

Now everything is working as I had hoped. But I think, prepareRequestUri() should be modified to read REDIRECT_URL if it exists.
What you think? Is it right?

This is a duplicate,

Where is the duplicate entry? having the same problem.

Where is the duplicate please?


Transcript: It does not work because it uses BlUniverse

Try:
transcript := GtTranscript new.
transcript show: 'something'.
transcript

The transcript view will be empty.

Exemplifying transcript UI behavior:

This is beautiful!


Fix Autofill E2E test

Task/Issue URL: https://app.asana.com/0/414730916066338/1208406680151445/f
Description
Steps to test this PR
QA-optional
No UI changes

#5071  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lmac012 and the rest of your teammates on  Graphite


Disabling pooling in rethinkdbdash

Hi There,
I was trying to disable pooling. But it breaks saying "run was called without a connection and no pool has been created". Code is very simple-
var r = require('rethinkdbdash')({
    pool: false,
    cursor: true,
    host: 'xx.xx.xx.xx,
    port: 28015,
    db:'brief-prod'
});

r.table('users').run().then(function(cursor) {
  cursor.toArray().then(function(result) {
    console.log(result);
  });
});

Not sure what am I missing. Stack is as follows-
D:\POCs\NodePOCs\rethinkdbdash\node_modules\rethinkdbdash\lib\term.js:124
throw new Error.ReqlDriverError('run was called without a connection and no pool has been created', self._query);
^
ReqlDriverError: run was called without a connection and no pool has been created after:
r.table("users")
at Term.run (D:\POCs\NodePOCs\rethinkdbdash\node_modules\rethinkdbdash\lib\term.js:124:13)
at Object. (D:\POCs\NodePOCs\rethinkdbdash\server.js:9:18)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
at Timeout.Module.runMain [as _onTimeout] (module.js:575:10)
at tryOnTimeout (timers.js:228:11)
at Timer.listOnTimeout (timers.js:202:5)
Thanks
Anup

If you don't use a pool, you have to manually create connections with r.connect

Can you please direct me any sample for this. I could not find any example for the same.

https://github.com/neumino/rethinkdbdash/blob/master/test/accessing-reql.js
On Mon, Nov 21, 2016, 00:10 asteranup<EMAIL_ADDRESS>wrote:

Can you please direct me any sample for this. I could not find any example
for the same.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/neumino/rethinkdbdash/issues/314#issuecomment-261870262,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZOu7x1z3lYvFmZcDxp3DVduv1NCXTBks5rAVH9gaJpZM4K3xxU
.


Including partials dynamically

Hi,
I have been looking for a simple way to dynamically render partials based on a value in an {{#each ...}} loop.
For example:
{{#each modules}}
    {{> @exphbs.partials[ this.name ] }}
{{/each}}

This would allow me to load partials based on data rather than putting a lot of logic in the template.

Dynamic partials are supported in Handlebars 3.0. See: https://github.com/wycats/handlebars.js/pull/941
So you would do:
{{#each modules}}
    {{> (this.name) }}
{{/each}}

@ericf I must be missing something then...
TypeError: undefined is not a function
   at Object.eval (eval at <anonymous> (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:209:23), <anonymous>:5:90)
   at prog (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/runtime.js:178:15)
   at execIteration (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/base.js:133:19)
   at Object.<anonymous> (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/base.js:142:11)
   at Object.eval (eval at <anonymous> (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:209:23), <anonymous>:5:34)
   at ret (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/runtime.js:144:30)
   at ret (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/node_modules/handlebars/dist/cjs/handlebars/compiler/compiler.js:462:21)
   at ExpressHandlebars._renderTemplate (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/lib/express-handlebars.js:247:12)
   at ExpressHandlebars.<anonymous> (/Users/justinledoux/Documents/node/qsPortal/node_modules/express-handlebars/lib/express-handlebars.js:173:21)

This is what I have in the code:
{{#each portal.modules}}
	{{> (this.url) }}
{{/each}}

If I print {{this.url}} I am getting the proper values (alerts, mapping, news).

So I didn't try to the code myself… let me know what you figure out.

Looking at the unit tests the subexpression needs to be a helper. Something like this:
var app = express();
app.set('view engine', 'hbs');
app.engine('hbs', exphbs({
    helpers: {
        partial: function (name) {
            return name;
        }
    }
}));

{{#each modules}}
    {{> (partial this.name) }}
{{/each}}

Hi Justin! I took a look at this, and it appears to be a quirk of how Handlebars processes subexpressions:
"Subexpressions do not resolve variables so whichPartial must be a function."
In your code, this.url is (presumably) a string, when it needs to be a function returning a string.
modules: [
  { url: '...' },
  { url: '...'}
]

needs to be:
modules: [
  { url: function() { return '...'; } },
  { url: function() { return '...'; } },
]

Kinda awkward. Alternatively, you could perform a lookup on the value, like so:
{{#each portal.modules}}
    {{> ( lookup . 'this.url') }}
{{/each}}

I've tested both these approaches in v.2.0.1 of express-handlebars, and both work without any custom helpers required.

@samuelfine how do pass variables to partials in your case? (i am not able to do so and please help)

Apparently your dynamic partials need to parsed as a function which returns a string.

I'm having trouble with dynamic partials as well. I thought I was doing it correctly but I keep receiving a partial not found error. Oddly, the error disappears if I mention the partial in a commented handlebars statement, as if that commented statement loaded the template, so now the dynamic loading works. Please see the details here:
http://stackoverflow.com/questions/36438434/handlebars-and-node-dynamic-partial-not-found
Can you anyone help?

If you're using the handlebar-helpers library you can use a standard string function such as lowercase. It might not do anything but it returns a function and fixes this niggle.
{{> (lowercase this.url)}}


Free Non Strict Options for a Command

Hi!
Please how could the flags be defined for the app level. Yet for the command level to let provide any flag without strict advance definition.
So that the application options would be available by name and the command options would be available as raw slice separated of command arguments.  Something like this?
&cli.App{
		Name:  "app",
		Flags: []cli.Flag{
			&cli.StringFlag{
				Name:    "appOption",
				Aliases: []string{"o"},
			},
		},
		Commands: []*cli.Command{
			{
				Name:      "task",
				Action:    func(cCtx *cli.Context) error {
                                    appOption := cCtx.String("appOption")
                                    comOptions := cCtx.RestOfOptions()
                                    comArgs := cCtx.Args().Slice()
                                    return nil
                                },
			},
		},
	}

app -o app-level-options task --what ever -bc task-arguments next

Thx.

@kudla You can try setting the Command.SkipFlagParsing to true https://github.com/urfave/cli/blob/v2-maint/command.go#L45 This will allow you to get all the rest of the options in cCtx.Args().

That's great . Thank you @dearchap
What about partial definition ? E.g. some specific options would adjust the behavior of application it self yet all the rest options would be fed as is to a downstream application.

@kudla There is no support for that in the urfave/cli library. Its either all or nothing. Anything else you have to implement yourself.

Thanks @dearchap. I wen exactly the same direction


Add unique option testID for test automation

Add unique option testID and testIDPrefix prop to support testID automation on options
Reference to issue #77
Signed-off-by: Merlier<EMAIL_ADDRESS>

Thanks @Merlier for contributing to this project.  I edited the Readme to clarify the testID behavior which could be a bit confusing.
I renamed testIDPrefix to optionsTestIDPrefix in order to clarify what it applies to.  There are other components that may have testID's, and I wanted to make sure users know that this prop only affects the selectable options' test ID's.

Good job 👍 and quickly merged!
Thanks @peacechen


Remove [AssemblyNeutral] attributes

Hi @bricelam, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Brice Lambson). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla2.msopentech.com.

TTYL, MSOTBOT;


config.json: Add topics & difficulty for poker

See #345 & #731

"Domain-specific languages" may be a bit of a stretch. I figure using the combination of card values as a string with a relative value is a small but very domain-specific language.


feat(css): build assets with the entry name when it is an entry point

REF: https://github.com/vitejs/vite/issues/4863#issuecomment-1367775712
How can we use the entry name to construct two files with the name L.css and X.css when specifying an entry like this:
export default {
    build : {
        rollupOptions : {
            input : {
                L : 'login/index.less',
                X : 'X/index.less'
            }
        }
    }
};

Inside the assetFileNames hook, we cannot distinguish two assets with the same name index.css.
In comparison with Webpack, it will use the entry name as the asset name when using the plugin, mini-css-extract-plugin.

Description
To resolve the problem mentioned above, I hope that Vite can use the entry name as the asset name.

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

We need this feature as we have multiple pages that act as independent apps. Perhaps @patak-dev could have a review/merge this please?

Thank you, we've decided to have two separate vite configs for now that extend a common one, seems to work fine for us for now

IIUC this will fix #9877 and close #9485.

I restored the test for #9353 by adding a dynamic import chunk. Also I found that the Windows test was failing and fixed that as well.

/ecosystem-ci run

histoire: faling with using yarn in packageManager field with pnpm (not related to this PR)
unocss: failing with type error not related to this PR
vite-plugin-react-pages: seems to be failing with HMR tests but given that this PR only changes renderChunk hook and that hook doesn't run in dev, I guess it's not related
vitepress: failing because of the change in https://github.com/vitejs/vite/pull/13646 (this PR's branch is a bit old)

@lubomirblazekcz could you create an issue with a minimal reproduction so we can better evaluate if this is expected or not?


Icons for Variables when Debugging

Greetings! I'm kindly suggesting adding icons in Variables View (when debugging) like it is now in Visual Studio and IntelliJ. Icon may identify variable type or just visually indicate that it is a variable and not text
VSCode (No Icons)

Visual Studio (With Icons)

IntelliJ (With Icons)

Is that helpful? In your examples all the varialbes have the same icon. So what's the point?


restore fixed starting position

use character/attribute of initialized text mode to reset head position to 0x720

Awesome! Thanks!


Add NewRelic Distributed Tracing support

This pr introduces client and server interceptors that can pass newrelic distributed tracing payload through newrelic header.

@splittingred Let me know if there is anything you want to refactor/change. Otherwise, Im not planning to add anything else to this PR and it is ready for review/merge

would you like to have frozen_string_literal in all files? I can add to this PR. Also, I would recommend adding rubocop check to CI pipeline.

I added frozen_string_literal everywhere (`rubocop -a'). As for integrating rubocop to CI pipeline, I would rather not include it in this PR so it stays small and on topic

@ocha This is great. I'm OOO the rest of the day, but can cut a release for this tomorrow. Thanks again for your contribution!

Thank you and my pleasure


Add ends_with as an alias for suffix

One-line PR to add ends_with as an alias for suffix, corresponding to this PR:
https://github.com/duckdb/duckdb/pull/8763
This is my first contribution, and I wasn't sure if I should run the scripts to generate functions.json/search_data.json as part of this PR, or if that will happen automatically.

The autogeneration scripts run every Monday, no need to do that yourself


Any syntactic sugar for CoffeeScript users?

Hi, our apps are in CoffeeScript and personally I also use CirruScript. Since in CoffeeScript eveything is an expression, conditions like if and switch are such easier.
Is there any plan to add sugar for CoffeeScript or any language other than JSX?
At this moment I guess such code might work. Not sure whether this would change or not.
div = deku.element.bind deko, 'div'

div className: 'demo',
  deku.component UserComp, data: {}

We're probably not going to add any syntax enhancements specifically for CS, but there is the possibility of the vdom module being pulled out of Deku, so you'd be able to write it however you wanted to.

Thanks.
I noticed there is "This may change in the future" in Component Specs. I thought most of them are stable.
But is there plan on changing APIs(which may cause the deku.element.bind null, 'div' solution fail)?

Nope, the element syntax won't change. We need that to stay that way so it continues to work with JSX. It's just the internals that might change.


Convert all Node.js utilities to async/await.

[ ] Add switch to root method to converted dots note recurse.
[ ] Convert dots node outdated.
[ ] Convert dots node upgrade.

Convert only those projects that currently depend on your Node.js legacy. Your dots node outdated and dots node upgrade are find for now as zsh. The goal is to remove the package.json and move your new binaries into libexec and out of bin.
To implement format again, you're going to need to update Programmatic to ES6. You need format to be a library function to convert upgrade to Node.js.

[ ] Convert newline wrapping in Markdown for GitHub issue and release notes.
[ ] Determine if dots node upgrade actually depends on Node.js.
[ ] Determine if dots node outdated actually depends on Node.js.


Document git branch/merge process

Create a draft document in Github
Related: #1127

Preliminary thoughts from Slack. Not yet finalized.
I envision that we will be getting rid of the master branch and instead have a production branch which would reflect what is deployed to the production environment, and a staging branch which would reflect what is deployed to the staging environment. The feature/* branches deploy to dev/qa environments.
I think after a PR is approved for a feature/* branch, we can have QA test the build, and if it passes QA then we can merge to the staging branch. So it might make sense to add a QA approval requirement to each feature/* PR to ensure that we aren’t merging untested code into staging . And then prior to each release to production, QA would do testing in staging to make sure there are no regressions. Once regression testing passes, then we can merge staging to production and deploy to production.

A thought: we could make the staging branch stable - so we don't merge in feature/* branch code directly into staging but rather into a rc/v* branch. The rc/v* branch will have its own review build (we'd need to add a branch matcher for this). Then when we are about to do a release to production, we first merge the rc/v* branch to staging to deploy to staging, and regression test in staging. Once regression tests pass, we merge/deploy the staging branch to production to deploy to production.
This would allow us to make critical hotfixes by branching off of staging (since it would mirror the code in production except when we are about to do a launch). So if we have a critical hotfix we need to deploy to production, we would branch off of staging to create a hotfix/* branch. Then we merge the hotfix/* branch into staging and then production.
Overall branch/merge process:

            feature/*
                ↓
feature/*  ➞  rc/v*  ➞  staging  ➞  production
                ↑          ↑
            feature/*   hotfix/*

Another option is to follow the standard git flow model, which is quite well known. The release/* branch would be created to freeze the code to be released to production. This would also deploy to the staging environment. Any bug fixes would be created in a bugfix/* branch and merged into the release/* branch. After the release/* branch is stable and passes QA, we merge it into both master and develop.
Any critical hotfixes would need to be branched off master, and then merged back into master, release/* (if there is one open), and develop (if the branches needs the fix right away).
            feature/*     bugfix/*
                ↓            ↓
feature/*  ➞  develop  ➞  release/*  ➞  master
                ↑            ↑              ↑
                 ----------------------- hotfix/*

This was resolved w #1269


Added support for the MongoDB 4.3.x driver.

Hi there,
This is a minor update to support the MongoDB 4.3.x drivers.
Kind regards
Erich

Would be nice to include 4.4.x now, too.


Sequence doesn't work as a required property.

Consider the following:
# self incrementing integer sequence
scalar type int_seq_t extending sequence;

type DefaultTest8 {
    required property number -> int_seq_t {
        # The number values are automatically
        # generated, and are not supposed to be
        # directly writable.
        readonly := true;
    }
}

Attempting to INSERT DefaultTest8 fails, because the number is required, but is missing (it works fine if the property is not required). This is counter-intuitive.

This example originally comes from the front-page schema (type Issue).

This goes into the exact semantics of required. The way I understand it, it's basically a guarantee that the property will be non-empty for all objects. Whether or not that value needs to be explicitly supplied with an INSERT depends on whether this property has a default, and a self-incrementing scalar always has an implicit default "next value".
Incidentally, this makes me wonder, do we currently allow computables to be classified as required? We should be able to validate that the computable expression is guaranteed to be non-empty and providing such a guarantee may be useful. Let's say that full_name is a computable and gets used everywhere in forms & webpages, but it may be very nice to be able to enforce that it is in fact never empty. This would mean that if one of the fields it relies upon stops being required, it would be a migration error.

@elprans, @1st1 what are your thoughts?

@vpetrovykh See #117 and #349

Re the semantics of required, you got that right.  It guarantees that the property or link will always be there when selected.  How it got into the database is not important.

I've added a couple more INSERT tests for the sequences. They have unexpected results (as far as I can tell), but additionally trying to replicate them in REPL gave me even stranger results.

@vpetrovykh Please close issues when you commit.  You should always add a Fixes: #NNN to let github close things automatically.


[Azure Bot Service] Got 500 Error code during Bot migration to GA Bot service

Bot Info

SDK Platform: Node.js
SDK Version: botbuilder 3.13.1, botbiolder-azure 3.0.2
Active Channels: Skype, Telegram, Facebook
Deployment Environment: Azure Bot Service

Issue Description
During migration I've got message with 500 error code. Seems it was in process of file copying to created instance.
Reproduction Steps

Press button "Migrate" in portal.azure.com or botframework site
after couple of minutes see Error popup.

Expected Behavior
Success migration
Actual Results
Migration is not completed.

@edanilenko34, thanks for reporting this issue we are currently investigating.

@edanilenko34, what is the botId?

The BotID is 4949893f-f14b-4e63-b274-05fc8854c728

hi @edanilenko34, we think we have found the issue, if you want the long explanation please check this related issue #4227.
The short version is:

Use FuncPack
Use an App Service name you never used before when you migrate (e.g. instead of vlgtrans-upgrade, use vlgtrans-upgrade1)

Let me know if you have any questions.
Andrea

One more important step I just discovered. Once you move to funcpack, delete the node_modules folder from the app service prior to migration. A function using funcpack does not need the node_modules anymore, and that folder might cause the timeout during migration.

hi @edanilenko34, were you able to try this? Let me know if you have any questions.

@edanilenko34 Closing the issue assuming it as resolved. Please open a new issue if you have any.

The bot has not been migrated yet, reopening this issue.
@edanilenko34, here's what you need to do in order to migrate succesfully:

move your code to funcpack https://github.com/Azure/azure-functions-pack
remove the node_modules folder from the app service (or add it to .gitignore)
wait until mid this week (ETA 5/17), we're deploying another fix that addresses a timeout issue during migration
migrate your bot

I'm available to get on a screen share session if needed.
Let me know if you have any questions, and thanks for your patience.
Andrea

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


AMQP Support

AMQP support in Azure::Core

[ ] Samples
[ ] Documention

Detailed work items remaining:

Redesign Azure::Core::Amqp::Message and Azure::Core::Amqp::Value to improve C++ usability - 5 days
Doxygen support - 3 days
Removed SetXxx APIs in favor of Options for Connection, Session, MessageSender and MessageReceiver. 3 days
Removed SaslPlain authentication - 1 days
Readme.md support - 2 days

Big ticket remaining items:

Transparent credential expiration support
Reconnect support

Outstanding questions:

Do we really need connection and session as externally visible constructs given that the MessageSender controls opening and closing the connection/session?


[Candidate 2v2 Oil Carnage ZH v1] Map uses houses with missing fire ports

Observed behavior
Map uses houses with missing fire ports. Soldiers shoot from 0,0,0.
Expected behavior
Map uses houses with good fire port setups.
Screenshots

Not blocker bug, because houses sit on flat terrain and appear to cause no detriment.

i dont even know how to fix that tbh

is it necessary ?

Is not necessary when there are no gameplay implications.

This should not be changed. It's the same bunker used in oil rampage - one of most popular maps of all time. It's iconic to this map and nobody ever noticed, all players have the same building. Please close this issue as it wont be fixed. All other issues TimberOwl will review.


Update to  Plant Environment Ontology on identifiers.org

Hello,
It was recently brought to our attention here at identifers.org that we have an issue with our entry for the plant environment ontology. Seems like the issue is due to the person that registered trying to answer to the EO and to the PEO prefixes with only one namespace. It seems like EO was replaced with PECO according to this, was this also done to the PEO prefix?
Any clarification would be greatly appreciated.

The Plant Environment Ontology (EO) was deprecated and replaced with the Plant Experimental Conditions Ontology (PECO).  We have never had an ontology with the PEO prefix, not can I locate one at OBO Foundry or EBI-OLS.
I am not familiar with your system at identifiers.org.  Could you please provide an example of where this is used?

Hi, I'm very sorry for forgetting this issue open. I will close it now.
We will mark EO as deprecated on our registry if this is not available anymore.
Our system provides a registry of data collections and a resolver to the data in them. Our URLs and Compact identifiers can be used as permanent identifiers when referencing data in the collections. The PEO prefix because our current regex pattern allows it.


Dockerfile.*: Fix "etcd is distributed" -> "etcd is a distributed"

Correcting a typo from 2f109647 (version: openshift-v4.0, 2018-11-29).
CC @hexfusion

rhel-images:
Pulling image openshift/golang-builder:1.10 ...
error: build error: failed to pull image: repository docke.../golang-builder not found: does not exist or no pull access

unit:
--- FAIL: TestServer_Unix_Secure_DelayTx (0.25s)
	server_test.go:109: took 186.982776ms with no latency
	server_test.go:124: took 62.396391ms with latency 50ms±5ms
	server_test.go:132: expected took1 186.982776ms < took2 62.396391ms (with latency)

and more.  I expect both of those are unrelated to this change ;).

/test unit

/lgtm

Unit tests need vetting I will look into this.

@wking thanks for the PR!


Can't change font

Hi.
I can't change the font on the desktop ver. of jupyterlab while the browser ver.  allows me to change the font.
my env: macOS Monterey
Thank you.

are you trying to change the font size? what are the steps to reproduce?

this should be fixed now with v3.5.1-1

Cannot change font size.

@donhuvy you are clicking on 'increase text editor font size" while being in a notebook.

Can you explain / description detail about what you comment. I don't understand. I found a solution for see bigger text now

There are different editors in JupyterLab: notebook editor, text editor, etc. The menu item you where clicking was changing font size in text editor not in notebook. There is an issue in the JupyterLab repository tracking improvements here but I could not find it right now.


In Azure Table Storage, there is both a Timestamp column and TimeStamp column

Natively, Azure Table Storage creates a "Timestamp" column with the date the row was created. The AzureLoggingEventEntity also has a TimeStamp column (but with a capital S). Confusion is easy between the two.

Merged - Thanks!


107 feature test submit for carla leaderboard

Description
Branch where our submission was created. We created a specific Dockerfile that contains all settings and a special safeguard that removes files that use the Carla API.
Fixes #107
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Does this PR introduce a breaking change?
no
Most important changes
New Dockerfile
New Documentation
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 (might be obsolete with CI later on)
[x] New and existing unit tests pass locally with my changes (might be obsolete with CI later on)

As the submission is not working currently the branch will not be merged in this phase. When we created a working file, we will create a new PR.


Add registry url validation

fix: https://github.com/portainer/portainer/issues/1182

You hardcoded a http request, but most registries are behind https
The problem is that we don't specify protocol when adding a repo
I don't kwow how docker request registry, but we have to do the same here

Ah yes, you are correct. I'll check how docker implement this

Other solution is to add a new field to Registry, like a endpoint API, it should be by default https://<registry_url>, but can be override if needed
You can have a registry located at myregistry.mydomain.com from tour docker host, by the same registry is available at internal-url.local from portainer

Sorry I don't really get what you mean @WTFKr0. But isn't make things more complicated?
I have checked how docker constructs the endpoints. Docker creates all possible combinations of the endpoint, prefer https over http and v2 over v1.
See: https://github.com/moby/moby/blob/master/registry/service.go#L296
Should we also do this? We can run the check each in goroutine and see if at least one of them is valid

We need this feature for the registry browsing WIP
I think we can check like you say :

Docker creates all possible combinations of the endpoint, prefer https over http and v2 over v1.

But we have to store that data in registry object, so that browse module can use it
WDYT ?
/cc @deviantony

A discussion about this was started in https://github.com/portainer/portainer/pull/1482/files#r158226153
Either we include the protocol in the registry URL at creation time, but that might break the behavior of the current registry implementation.
Or we determine the protocol at creation time as well, and store it inside a new field.

@asasmoyo
Are you yet on this PR ?
I need it for the remaining work on registry management
If you don't have time, i can take that too
Thanx

@WTFKr0 I can work on this. Should I go with this?

Docker creates all possible combinations of the endpoint, prefer https over http and v2 over v1.

Yes I think so, and we have to store these two informations (protocol and version) in Registry object
https://github.com/portainer/portainer/blob/develop/api/portainer.go#L149-L158
So that I can access it later
/cc @deviantony

@WTFKr0 please check my update

Docker creates all possible combinations of the endpoint, prefer https over http and v2 over v1.

Agree, I'll review it soon.

So looks like you remove the if !strings.HasSuffix(url, "/") { part but have you tested if it works? @asasmoyo

Yes it is working, I just checked it.
Regarding the auth check, do all docker registry support basic auth? If so, it would be simple to check the authentication as well

Regarding the auth check, do all docker registry support basic auth? If so, it would be simple to check the authentication as well

It's not so easy !
Docker registry can be configured with basic auth, but it can be configured to delegate auth to a third party service. In that case, the workflow is :

You call the registry api uri
Registry send you a 401 response with information in header on where and how to auth
Then you make request to auth server, giving you an access token
You use this token (by sending header) to registry

@deviantony do we really need to do auth check here? I can try working on it if you want

@asasmoyo yes I believe that the related issue (#1182) implies a connectivity check and not only a URL validation (so credentials as well).
Try to follow the workflow explained by @WTFKr0

I just tested that and i'm stuck
I think i have a self signed https registry
So we have to tell portainer that certificate is valid
How can we do that ?

@WTFKr0
Skipping the TLS verification part might do the job via:
    tr := &http.Transport{
        TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
    }
    client := &http.Client{Transport: tr}

Not sure if it's a good idea though, but we can check if this works first.

OK with tlsskipverify
The good way to do that is to find where to put certif on portainer container to allow https request to this URL

we can add an option when adding a registry to accept non valid certificate too

@deviantony @WTFKr0 should we do this?

@asasmoyo yes, please add a switch "TLS verification" when creating a registry that should toggle this.
@WTFKr0 does it affect the usage of the registry within Portainer tho? E.g. can you still pull images with an invalid cert?

The pull action is done by docker daemon, not by portainer
If the registry had a bad certificate, user should add registry url to --insecure-registries daemon option
So, this is a dockerd issue, not a portainer one

Please check @deviantony @WTFKr0

If the registry had a bad certificate, user should add registry url to --insecure-registries daemon option

I think we'll also need to add a warning about this somewhere. Even if tls verification is disabled, push/pull actions to said registry would fail if the url is not in insecure registries.

Ok for me i just add my gitlab authenticatd insecure registry !
Maybe we can name option "Insecure registry : true / false" instead of "TLS verification" to keep same logic as docker daemon ?
When activating TLS verification, i always got the "Registry url invalid" error, shouln't i get a "Unable to validate registry server certificate" error ?

When activating TLS verification, i always got the "Registry url invalid" error, shouln't i get a "Unable to validate registry server certificate" error ?

What kind of registry server did you use? I just tried it using official registry server v2 with auth enabled + self signed ssl, I got "Unable to validate registry server certificate" when I checked "TLS Verification"

I have a registry v2 configured to use gitlab as a auth server
I have a selfsign cert on the registry, and the gitlab is in clear i think
Maybe it's another error, i try to find more info soon

I think we'll also need to add a warning about this somewhere. Even if tls verification is disabled, push/pull actions to said registry would fail if the url is not in insecure registries.

We're already displaying information about insecure registries when selecting the "Custom registry" option:

Docker requires you to connect to a secure registry. You can find more information about how to connect to an insecure registry in the Docker documentation.

what should we do next for this? @WTFKr0 @deviantony

/cc @deviantony
Can we tackle this one, as I need it to finish work on Registry Management
Thanx

@asasmoyo @WTFKr0 sorry for the late update guys, we're still thinking about how we should tackle this issue. IMO we need to add the ability to specify TLS options as we do for endpoints (IDK if all the options exposed in the endpoint creation form are required here).
Also this work should be part of the registry management feature and thus should probably be rebased against the feat56-registry-management and merged into this branch instead of develop.

IMO we need to add the ability to specify TLS options as we do for endpoints (IDK if all the options exposed in the endpoint creation form are required here).

Agree, I think we should do this. If you are ok I can start implementing this

Also this work should be part of the registry management feature and thus should probably be rebased against the feat56-registry-management and merged into this branch instead of develop.

I'll do the rebase before my next push

@asasmoyo sure, you can start working on this as a part of this PR. Thanks !

@asasmoyo any update on this?

Also found that you can use this API endpoint to  validate credentials: https://docs.docker.com/engine/api/v1.30/#operation/SystemAuth
Might be good to add in here.

@deviantony Sorry for the delay, I'll finish this by the end of this week.
Sure, I'll check it out.

I'am wondering what should I do for uploading cert files. There is already file upload handler here:
https://github.com/portainer/portainer/blob/develop/api/http/handler/upload.go#L33
If I'm correct, endpoint already use it by passing its endpointID as folder. Should I use this with, for example registry-registryID as folder? Or should I create a new upload handler for handling cert files for registry?

I am thinking to do this similar with endpoint. Client sends post registry request then send upload cert files requests. But if the cert files come after post registry, then we will not have required certs to do connection check (for tls enabled with client certificate / self signed certificate).
Could you suggest me better approach? @deviantony @WTFKr0

What about the following flow:

Create registry
Upload certs
Test registry connection (that would require the introduction of a new API endpoint)
If ok, go further
If failing, remove registry

Test registry connection (that would require the introduction of a new API endpoint)

what action should trigger this? what about after client receives all responses for certs upload?

@asasmoyo exactly, that should be a chain of promise: create endpoint -> upload certs (via $q.all() for example) -optional- -> test registry connection.


gradle update

Summary
Replaced the hardcoded contains("release") in react.gradle with a new flag to make it possible to config hermes release mode for custom build variants.
Changelog
[Android] [Added] - A new flag hermesReleaseIn{targetName} to enable hermes customization for custom build variants
Test Plan

Add a custom build type

buildTypes {
  // ...
  staging {
    initWith release
    applicationIdSuffix ".staging"
    matchingFallbacks =  ['release']
  }
}

Enable hermes and set the new flag

project.ext.react = [
    enableHermes: true,
    bundleInStaging: true,
    devDisabledInStaging: true,
    hermesReleaseInStaging: true
]

Add dependency implemention

dependencies {
  // ...
  if (enableHermes) {
    // ...
    stagingImplementation files(hermesPath + "hermes-release.aar")
  }
}

Build and run the staging variant

Thanks for sending the PR. I'm closing this as the logic is completely rewritten now.


lazy require servers

PLEASE READ BEFORE CREATING THE PR :)
For new install scripts
You need to create one file, and edit two files. See this PR for a good example: https://github.com/kabouzeid/nvim-lspinstall/pull/106/files
Whenever possible, use local config = require'lspinstall/util'.extract_config('rust_analyzer') to extract the config for your language server from nvim-lspconfig.
If they don't have a config for your language server yet, please create a pull request there first.
Don't worry, they are usually quite fast with merging new configs.

Users should still be able to set custom installers like this: https://github.com/kabouzeid/nvim-lspinstall#custom-installer

No problem, we can just treat the server differently depending on whether it is a string or a table. If it's a string, treat it as a module name. If it is a table, treat it as a config.


Compile Help

Hi, i'm compiling you project and i get some error:
C++ exception handling is not supported with /kernel。
Can you tell me something about how to use stl eg. std::array when writing kernel driver。
I compile it using VS2015 & WDK10.0 。Thanks 。

Many thanks。But i have some confusion，does all stl libray work correctly?  Have you ever occasionally got some unsupported situation?

The use of STL is merely for faster prototyping, and I do not believe all part of STL work correctly (if it does, it should have been supported by Microsoft). Some might not compile and some might trigger unexpected runtime behaviour.
I personally have not seen any issues with the very limited use of STL in the kernel, but I guess exception would cause an issue (I do not remember what STL does when _HAS_EXCEPTIONS is zero).

i get it。thanks。