NodeParameterDefinition is unsupported

NodeParameterDefinition is unsupported

not all plugin with its parameters are implemented.
But i can have a look if all needed data are available
Which plugin is it?

Plugin name is Jenkins Control Plugin. I set Node for execution in jenkins
jobs using Node Parameter.
On Thu, 21 May 2020, 22:14 MCMicS<EMAIL_ADDRESS>wrote:

not all plugin with its parameters are implemented.
But i can have a look if all needed data are available
Which plugin is it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCMicS/jenkins-control-plugin/issues/213#issuecomment-632053159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVLXTEUFHHOMKQV6WLCPGDRSULKJANCNFSM4NGZBXRA
.

I mean the plugin which offers node as paramters on Jenkins
Do you have following: https://plugins.jenkins.io/nodelabelparameter/

Yes I have the same plugin.
On Thu, 21 May 2020, 22:23 MCMicS<EMAIL_ADDRESS>wrote:

I mean the plugin which offers node as paramters on Jenkins
Do you have following: https://plugins.jenkins.io/nodelabelparameter/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCMicS/jenkins-control-plugin/issues/213#issuecomment-632056628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVLXTHWZV3PV4OLFQOKGWLRSUMM7ANCNFSM4NGZBXRA
.

ok
I looked at the API of the plugin and I no get the available nodes for parameter because its not implement the API correct and is not maintained for last 4 years.

Is a custom Input Field correct for your needs or do you need a listbox with available nodes(which needs more effort)?

I would pefer listbox as it will give more effectivce solution for me.
On Thu, 21 May 2020, 22:37 MCMicS<EMAIL_ADDRESS>wrote:

ok
I looked at the API of the plugin and I no get the available nodes for
parameter because its not implement the API correct and is not maintained
for last 4 years.
[image: grafik]
https://user-images.githubusercontent.com/4604206/82559548-1fc70300-9b70-11ea-9ea8-4b5a2f77e46e.png
Is a custom Input Field correct for your needs or do you need a listbox
with available nodes(which needs more effort)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCMicS/jenkins-control-plugin/issues/213#issuecomment-632062434,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVLXTED66SQEBEHGN4OQDTRSUOBVANCNFSM4NGZBXRA
.

OK.
A list box with all nodes should possible. But withouzt any logic klike the jenkins Ui will offer.
I mean. You can define which nodes are avaialable for parameter but due the missing API of the Node Plugin in Jenkins I cannot get the information. So you will see always all nodes.
Is this right for you?

Yes that way I will get an option to select node before triggering jenkins
job.
On Thu, 21 May 2020, 22:49 MCMicS<EMAIL_ADDRESS>wrote:

OK.
A list box with all nodes should possible. But withouzt any logic klike
the jenkins Ui will offer.
I mean. You can define which nodes are avaialable for parameter but due
the missing API of the Node Plugin in Jenkins I cannot get the information.
So you will see always all nodes.
Is this right for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCMicS/jenkins-control-plugin/issues/213#issuecomment-632067299,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVLXTDGGIX57MXWQY6IQKLRSUPNDANCNFSM4NGZBXRA
.

OK
I implement this in one of the next releases

Thanks for your prompt reply and action for issue resolution.
On Thu, 21 May 2020, 22:55 MCMicS<EMAIL_ADDRESS>wrote:

OK
I implement this in one of the next releases
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCMicS/jenkins-control-plugin/issues/213#issuecomment-632069673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVLXTAVQ7NU4JOL3I63343RSUQDDANCNFSM4NGZBXRA
.

Any updates

Not yet. Will try to update this week

Means no solution for NodeParameterdefinition ?

Means no solution for NodeParameterdefinition ?
No it is fixed but not merged yet. I test it currently and with 0.13.3 it will released

So when you are planning to deploy it

Due date is sunday. Have one issue left in milestone.
Will fix the last issue today if time permits

If you want to pre test it I uploaded an EAP version (2020 only)
Link: https://plugins.jetbrains.com/plugin/6110-jenkins-control-plugin/update/87312

released with 0.13.3

Working perfectly fine.


Make CloudPath.is_valid_cloudpath a TypeGuard (#337)

This allows us to infer that CloudPaths are the same type of object as the class is_valid_cloudpath is called on

Local version of #337 for live tests

Could we release a patch for this + #336? I'd really like to consume them!


Refactor the VM to not do type-checking

refactor out the typechecker :3
fix some more tests
fix upvalues and add the Type::Ty
linking
fix the faulty tests

Also: cargo fmt!!

Also: cargo fmt!!

We can do it after this and the next merge is done....


Formatting numeric and monetary values according to user's locale

This PR adds support to format numbers using locales specified in the LC_NUMERIC and LC_MONETARY environment variables via x/text/message printer.
In order to properly format numbers according to their meaning, i.e. number (e.g. 1h_change) or currency (e.g. price) the Comma*f functions have been replaced with custom Numericf and Monetaryf functions.
To test run: LC_NUMERIC=bn LC_MONETARY=de-CH go run ., which will display currency in a Swiss German format and numbers in Bengali. This will allow people to view the cointop numeric and monetary values in their favourite format:
   ▲[r]ank  [n]ame            [s]ymbol     €[p]rice   [1]H%   [2]4H%    [7]D%       24H [v]olume       [m]arket cap   [a]vailable supply     [t]otal supply   last [u]pdated
  *     1   Bitcoin           BTC        467’864.00  -০.৮৭%   -৮.২৬%   -৮.৬৫%    814’660’896’107  8’748’772’383’916          ১,৮৬,৮৫,৬৭৫        ২,১০,০০,০০০  17:34:53 Apr 18
  *     2   Ethereum          ETH         18’098.44  -০.৩৭%   -৯.৩১%   -১.০৯%    484’690’957’067  2’092’856’791’162         ১১,৫৫,২০,৮৭৯       ১১,৫৫,২০,৮৭৯  17:35:12 Apr 18
        3   Binance Coin      BNB          3’946.54   ১.৬৭%   -৯.৪৪%   -২.০৫%     63’228’874’268    610’341’951’978         ১৫,৪৫,৩৩,৬৫১       ১৭,০৫,৩৩,৬৫১  17:34:44 Apr 18
        4   XRP               XRP             11.26   ০.৬৪%  -১৫.৮০%   -৩.৮৫%    152’030’449’415    515’275’538’063      ৪৫,৯৭,৩৩,১৮,০৫৬  ১,০০,০০,০০,০০,০০০  17:35:59 Apr 18
        5   Tether            USDT             8.46  -০.০২%    ০.৪৫%   -০.৯১%  1’831’827’779’717    407’461’927’782      ৪৮,১৮,৭১,৯১,৪৯৯    ৪৮,১৮,৭১,৯১,৪৯৯  17:06:52 Apr 18
        6   Dogecoin          DOGE             2.61  -১.৫০%   ১৮.৪৭%  ৩৭৯.৩৫%    200’643’707’551    340’692’607’772    ১,২৯,২৩,৭৯,৭১,৭১০  ১,২৯,২৩,৭৯,৭১,৭১০  17:35:04 Apr 18

This is awesome! works as expected.
Code changes look good. Will merge 👍

It would make sense to make this a config file option as well in near future.
Do you think it makes sense to also change the formatting based on user's currency selection? I'm not really familiar with international currency formatting so could any suggestions here : )

@miguelmota I think making it a configuration option would duplicate the functionality that exists with locale (1). I'd advise at documenting how to use LC_MONETARY and LC_NUMERIC with cointop.
For example should someone want to see cointop with numeric and monetary values with a different locale than the set one in their environment it is possible to invoke cointop via /usr/bin/env LC_MONETARY=bn cointop without needing to change the user's environment.
Where is the content for https://docs.cointop.sh/faq/ managed? The above might make a good addition the the FAQ.
In my mind it does not make sense to switch the formatting with the user's current currency selection in cointop. As a European I'm used to reading numbers with a dot . as a thousands separator and a comma as a decimal separator, e.g. ¤ 1.234,45, regardless of what the currency is format is in the currency's country of origin.
It also poses the problem of which formatting to use for EUR as there are multiple ones?

Locale
thousands separator
decimal separator

de_DE.UTF-8
.
,

de-CH
’
,

fr-FR.UTF-8
  (space)
,

@afh The FAQ markdown file is located under https://github.com/miguelmota/cointop/blob/master/docs/content/faq.md
Based on your response we'll leave it as is and add information about these environment variables and configuration to the documentation pages.  The table helped me understand better. Thanks!

Thanks, @miguelmota 👍
Yes, I think we should leave it as is and add documentation. I went ahead and put up #122 for review to address this.
Glad to hear the table proved helpful :)

Thanks!


new highlighter, pics, foldings

New theme to support CM, UTX, ETX and FTX files.
Basic snippets for CM files
Basic foldings for FTX files

Nice work! Thank you!

Just pushed to VSCode Marketplace (https://marketplace.visualstudio.com/items?itemName=fabi.fanuc-karel-syntax). I had to revert publisher name, because it is used by vsce publish tool. But you/I can put credits if you like.

No problem! thank you 👍

@andymtorres The commits from this and the last merge were made by "Molina Frank" and thus you are not listed as a Contributor. Maybe your git is configured to use another name/mail than GitHub expects?

Hi,
Thanks for telling me this. But I am Molina Frank. Molina Torres is my maiden name. ^^
Andrea
Obtener Outlook para Androidhttps://aka.ms/ghei36

From: iBaff<EMAIL_ADDRESS>Sent: Monday, April 15, 2019 2:14:28 PM
To: iBaff/vscode-fanuc-karel
Cc: andymtorres; Mention
Subject: Re: [iBaff/vscode-fanuc-karel] new highlighter, pics, foldings (#5)
@andymtorreshttps://github.com/andymtorres The commitshttps://github.com/iBaff/vscode-fanuc-karel/pull/5/commits from this and the last mergehttps://github.com/iBaff/vscode-fanuc-karel/pull/3/commits were made by "Molina Frank" and thus you are not listed as a Contributorhttps://github.com/iBaff/vscode-fanuc-karel/graphs/contributors. Maybe your git is configured to use another name/mail than GitHub expects?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/iBaff/vscode-fanuc-karel/pull/5#issuecomment-483226990, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfhbE__W122mmR6MZmplw3ZCWIvzA7M6ks5vhG0kgaJpZM4ce3gB.

@all-contributors please add andymtorres for code


[IDEA] add option for saving values for twitter + discord (in addition to website) in project metadata

done


fix upload usage in README

This was changed per below, but the example wasn't updated:
0.6.0 (2017-01-23)

Breaking change: in mapbox-upload TILESET is now the first positional
argument and source file the second, reversing the order set in 0.4.0 (#85).

Gracias @andrewharvey 👍


get /arlas/explore/collection/_search returns hits with md.id but without timestamp and geometries

In gitlab by @sylvaingaudan on Jul 25, 2017, 16:31
get /arlas/explore/collection/_search returns hits with only md.id. It should include timestamp and the geometries.
Currently, the result is:
{
   "hits":[
      {
         "md":{
            "id":"......"
         }
      }
   ]
}

But should be:
{
   "hits":[
      {
         "md":{
            "id":"......",
            "timestamp":"......",
            "centroid":"......",
            "geometry":"......"
         }
      }
   ]
}

In gitlab by @sfalquier on Jul 28, 2017, 10:28
added ~2377174 label

In gitlab by @sfalquier on Jul 28, 2017, 11:15
@sylvaingaudan not reproduced locally.
Could you provide the ARLAS-server output and the corresponding raw document in elasticsearch?

In gitlab by @sylvaingaudan on Jul 28, 2017, 12:02
changed the description

In gitlab by @sfalquier on Aug 4, 2017, 18:06
added ~2424199 and removed ~2424198 labels

In gitlab by @sfalquier on Aug 4, 2017, 18:06
Should be fixed with #78

In gitlab by @sylvaingaudan on Aug 10, 2017, 08:31
changed milestone to %10

In gitlab by @sfalquier on Aug 11, 2017, 15:44
assigned to @sfalquier

In gitlab by @sfalquier on Aug 11, 2017, 16:47
mentioned in merge request !79

In gitlab by @sylvaingaudan on Aug 11, 2017, 16:53
closed via commit ad698b69b2cb3230c2c78a4f502697acaeae2b74

In gitlab by @sylvaingaudan on Aug 11, 2017, 16:53
closed via merge request !79

In gitlab by @sylvaingaudan on Aug 25, 2017, 11:48
mentioned in commit ad698b69b2cb3230c2c78a4f502697acaeae2b74


Populate Dictionary values in user config?

Cookiecutter version: 1.6
Template project url: n/a
Python version: 2.7
Operating System: MacOs High Sierra

Description:
I know I can define dictionary values the cookiecutter.json file, but I'm not sure how I can do this in the userconfig.yml file — nor am I sure it's possible. I may just not know the format (there's nothing in the docs about this). default_context seems to just allow key/value pairs, and trying use yaml-based nesting fails.
So, is there a way I can pre-propulate dictionary values in the default_context: of a user config?
What I've run:
.json example:
{
    "collection": {
        "sub-value-1" : {
             "sub-key-1": "somevalue"
         },
         "sub-value-2": {
              "sub-key-2": "othervalue"
         }
    }
}

yml attempt:
default_context:
  collection:
    sub-value-1:
        sub-key-1: "new value"
        sub-key-additional: "something I added"

(I've also tried using curly braces in the yml context, which doesn't work at all)
Thanks for your help!

I am not sure if you are still seeking an answer to this question since you've asked about a year ago (but doesn't seem you got an answer).
I came across the same issue #1149 and had the same question. But since it was failing with an exception (rather than being handled properly), I assumed it was a bug (which I fixed in a fork), so that the case you are describing now works.
The second issue I face now is that when prompted to set context options, the code only iterates at the top level (#1153), leaving you with something like
...
collection [default]:
Error: Requires JSON dict.

The Error occurs if you attempt to specify anything which is from an exception trapped in the code. That's another story though but still related.
All and all I think this feature of nested dictionaries was added without necessarily thinking about the end to end impacts (only works with cookiecutter.json). Once I hear from the maintainers (as to whether they agree or not) I will submit a pull request.


Browser instances are always shown!

Hello Cgoldberg,
I'm playing around with this module... but the browser instances are always being shown, while the test run! The installation of xvfbwrapper is happened inside an venv, together with selenium... There are no error logs shown... Test runs with your examples only, on a MacOS-X 10.12.4
Any help would be very nice!
Thank you

It's a Mac OXS Problem, X11 is not anymore supported by the newer OSX! More to read from here: https://www.xquartz.org/Developer-Info.html


[MB-10466] REFINE 2: Same Service Items are being requested for payment-Prime bug

Jira ticket
Summary
This ticket is a refinement to the original PR.
Prime brought up an issue that payment service items were now showing up as duplicates in the API. This ticket is a resolution to fix the issue. It was determined that the Eager Preload in FetchMoveTaskOrder service was the culprit.
What was done in this PR:

Created a new service in services/move_task_order/move_task_order_fetcher.go called GetMoveForPaymentRequests. This service now uses Eager Associations instead of Eager Preload.
Updated rules.go to implement the new service.
Updated tests in move_task_order_test.go for both handlers/primeapiv2 and handlers/primeapi

Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Go to officelocal, click "Local Sign In", and log in as Prime simulator.
Select a move and click on "Create Payment Request" (blue button at the top, in the Moves section).
Scroll down to "Shipment Service Items", go to "Domestic origin 1st day SIT", click the checkbox "Add to payment request".
Scroll down and click "Submit Payment Request".
You should get routed back to the previous details page and see a green banner at the top saying it was successful.
Click on "Create Payment Request" again for the same move.
Select the same service item you selected previously.
This time, you should see a red error banner saying "Conflict error".

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Screenshots

Messages

:book:
:link: MB-10466

  Generated by :no_entry_sign: dangerJS against<PHONE_NUMBER>54da510dc9c1e807161036052f7cb1

Test coverage on server failed.
Last run: Wed 16 Aug 2023 09:30:12 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


What's the resonable way to display notifications labels?

I would like to add notifications count label to the menu. From the documentation that would mean that I'd have to add
  'label'       =>  4,
  'label_color' => 'success'

but how could I programatically change that 4 with something like Auth::user()->unreadNotifications->count()
Not even the AppServiceProvider knows what the Auth facade is at this point. That's in case we would go for the other proposed way of building the menus:
 $events->listen(BuildingMenu::class, function (BuildingMenu $event) {
            $event->menu->addAfter('dashboard', [
                'key'         => 'notifications',
                'text'        => 'Notifications',
                'url'         => 'notifications',
                'icon'        => 'fas fa-fw fa-bell',
                // 'label'       =>  4,
                // 'label_color' => 'success'
            ]);
        });

If the sidebar would be a simple blade partial this would all come down to adding something like
{{ Auth::user()->unreadNotifications->count() }}

So, am I missing something here?

You can create a event listener for this.

Create the listener (app/Listeners/SidebarNotifications.php)

<?php

namespace App\Listeners;

use Illuminate\Auth\Events\Authenticated;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Support\Facades\Auth;
use JeroenNoten\LaravelAdminLte\Events\BuildingMenu;

class SidebarNotifications
{
    /**
     * Create the event listener.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Handle the event.
     *
     * @param  Login  $event
     * @return void
     */
    public function handle(BuildingMenu $event)
    {
        $event->menu->addAfter('dashboard', [
            'key'         => 'notifications',
            'text'        => 'Notifications',
            'url'         => 'notifications',
            'icon'        => 'fas fa-fw fa-bell',
            'label'       =>  Auth::user()->unreadNotifications->count(),
            'label_color' => 'success',
        ]);
    }
}

Update app/Providers/EventServiceProvider.php

add use JeroenNoten\LaravelAdminLte\Events\BuildingMenu;
place inside $listen the code below

        BuildingMenu::class => [
            SidebarNotifications::class,
        ],

Enjoy your custom sidebar menu item

I see, it looks like the right way to approach the thing, but how about when I have no unread notifications (count is 0) ? How do I remove the the label from the menu item?

Then you need to modify the script a bit like this:
$unreadNotifications = Auth::user()->unreadNotifications->count();
$notifications = [
    'key'         => 'notifications',
    'text'        => 'Notifications',
    'url'         => 'notifications',
    'icon'        => 'fas fa-fw fa-bell',
];

if ($unreadNotifications != 0) {
    $notifications['label'] = $unreadNotifications;
    $notifications['label_color'] = 'success';
}

$event->menu->addAfter('dashboard', $notifications);

Nevermind, I see that having a null value on the label will stop it from displaying. Checked with:
'label'       =>  Auth::user()->unreadNotifications->count()?Auth::user()->unreadNotifications->count():'',

So, all good! Thanks for the help!


[Fix] Removed duplicated TextUtil

Se unifica la clase utilitaria para manejo de textos.

Codecov Report

Merging #764 into develop will increase coverage by 0.06%.
The diff coverage is 5.88%.

@@             Coverage Diff             @@
##           develop     #764      +/-   ##
===========================================
+ Coverage    24.78%   24.84%   +0.06%
===========================================
  Files          253      252       -1
  Lines        18144    18142       -2
  Branches      1539     1539
===========================================
+ Hits          4497     4508      +11
+ Misses       13137    13124      -13
  Partials       510      510

Impacted Files
Coverage Δ

...opago/uicontrollers/discounts/DiscountRowView.java
31.3% <ø> (ø)
:arrow_up:

...llers/identification/IdentificationTicketView.java
0% <ø> (ø)
:arrow_up:

...mercadopago/presenters/PaymentResultPresenter.java
82.6% <ø> (ø)
:arrow_up:

...om/mercadopago/preferences/CheckoutPreference.java
57.59% <ø> (ø)
:arrow_up:

...ontrollers/reviewandconfirm/ReviewProductView.java
0% <ø> (ø)
:arrow_up:

...dopago/providers/ReviewAndConfirmProviderImpl.java
3.96% <ø> (ø)
:arrow_up:

...va/com/mercadopago/core/MercadoPagoComponents.java
18.39% <ø> (ø)
:arrow_up:

...go/uicontrollers/reviewandconfirm/SummaryView.java
0% <ø> (ø)
:arrow_up:

...src/main/java/com/mercadopago/util/ReviewUtil.java
0% <0%> (ø)
:arrow_up:

...c/main/java/com/mercadopago/DiscountsActivity.java
0% <0%> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


Fix incorrect doc links

FYI: Intra-doc links have been stabilized in Rust 1.48: https://github.com/rust-lang/rust/blob/1.48.0/RELEASES.md#rustdoc
There is also a lint that warns about broken links: https://doc.rust-lang.org/rustc/lints/listing/warn-by-default.html#broken-intra-doc-links


False positive finding for USE_DATA_CLASS rule on @Stable annotated class

@Stable is an android/Jetpack Compose construct
The suspicious code snippet:
@Stable
class PermissionState {
    var showRationale by mutableStateOf(false)
}

I think it's another case #802. We should definitely support a configurable set of excludes for this rule.


docker-compose: Fix startup issues on SELinux

If you don't set the SELinux context of volumes in compose files, there
will be access errors all over the place. This change addresses that
issue in the kitchen-sink sample compose file.
I've tested these changes on a Fedora 35 (Workstation) installation.
Motivation
On Linux distributions with SELinux enabled, the current docker-compose/kitchen-sink example fails to start. This is because SELinux denies reading or executing any files from within the container that do not have the proper SELinux context specified.
Modifications
This pull request adds the :z modifier (which "indicates that the bind mount content is shared among multiple containers") to the used volumes.
See https://docs.docker.com/storage/bind-mounts/#configure-the-selinux-label for more information.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (don't know)

the added SELinux labels shouldn't affect non-SELinux systems

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
[x] no-need-doc

This pull request only changes example docker-compose files.

[ ] doc

/pulsarbot run-failure-checks

Codecov Report

Merging #13708 (b1677b7) into master (d30e86c) will decrease coverage by 10.88%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #13708       +/-   ##
=============================================
- Coverage     47.92%   37.03%   -10.89%
+ Complexity    10679     1968     -8711
=============================================
  Files           703      209      -494
  Lines         68831    14425    -54406
  Branches       7378     1574     -5804
=============================================
- Hits          32988     5343    -27645
+ Misses        32119     8495    -23624
+ Partials       3724      587     -3137

Flag
Coverage Δ

unittests
37.03% <ø> (-10.89%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...g/apache/pulsar/client/impl/ConnectionHandler.java
50.00% <0.00%> (-5.32%)
:arrow_down:

...che/pulsar/broker/BookKeeperClientFactoryImpl.java

...er/loadbalance/impl/PulsarResourceDescription.java

...sar/broker/stats/metrics/ManagedCursorMetrics.java

...e/pulsar/broker/service/EntryBatchIndexesAcks.java

...broker/intercept/BrokerInterceptorDefinitions.java

...broker/service/persistent/DispatchRateLimiter.java

...org/apache/pulsar/PulsarInitialNamespaceSetup.java

...er/mledger/intercept/ManagedLedgerInterceptor.java

...r/broker/service/schema/AlwaysSchemaValidator.java

... and 485 more


[QUESTION] Is it possible to restrict content by audience?

What is the context of your question?

[ ] Installation
[x] Administration
[ ] Custom Content
[ ] M365 learning pathways web part

What is your question?
Is it possible to restrict playlists and content based on user login and/or audiences based on O3665 or AD groups? Examples would be targeted content for IT, HR, Compliance, etc.
If not, will it ever be possible?
Thank you

No, that is not currently possible.  I suspect you're looking for the concept of audience targeting and in this case, I think the more appropriate way to handle that would be by the use of sites/pages customized with the learning pathways web part filtered to show content only for certain sub-categories/playlists and then curated to those audiences.

Thank you Julie!


'az login' hangs in linux server: msal_extensions.cache_lock: Process 505865 failed to create lock file

Related command

Describe the bug
'az login' command hangs after user authentication . This happens executing from user accounts except root user.

azure cli 2.38.0 is installed. I tried reinstalling the same version, but didn't help.
Not even working after clearing /home/(username)/.azure directory, where token information and cache saved.
I tried 'az login --debug' command. It is stuck after a message "msal_extensions.cache_lock: Process 505865 failed to create lock file"

Expected behavior
complete the authentication and return token information and login details.
Environment summary
CLI version (az --version : 2.38.0) / OS : RHEL 7 / Shell Type : bash
Additional context
The terminal remains stuck after authentication. But if I quit the session by clicking ctrl+z after some couple of minutes and try $az account show, it displays the token information.

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

I can't repro this issue with a clean CentOS 7 container:
docker run -it --rm centos:7

echo -e "[azure-cli]
name=Azure CLI
baseurl=https://packages.microsoft.com/yumrepos/azure-cli
enabled=1
gpgcheck=1
gpgkey=https://packages.microsoft.com/keys/microsoft.asc" | tee /etc/yum.repos.d/azure-cli.repo

yum install azure-cli

az login

In your output, it looks like msal_extensions reached these lines in CrossPlatLock._try_to_create_lock_file: https://github.com/AzureAD/microsoft-authentication-extensions-for-python/blob/69eef72b0e7a540ae00abb6a65e749cfce4d532a/msal_extensions/cache_lock.py#L49-L51
CrossPlatLock._try_to_create_lock_file is called by CrossPlatLock.__enter__: https://github.com/AzureAD/microsoft-authentication-extensions-for-python/blob/69eef72b0e7a540ae00abb6a65e749cfce4d532a/msal_extensions/cache_lock.py#L57
CrossPlatLock.__enter__ is only called once by PersistedTokenCache.modify: https://github.com/AzureAD/microsoft-authentication-extensions-for-python/blob/d0696aeb6f65168b1e0d405cd871b80bb101cd76/msal_extensions/token_cache.py#L63
Could you try to clear ~/.azure (/home/username/.azure), instead of /home/.azure?
If that doesn't work, any chance you are running multiple Azure CLI instances concurrently?
cc @rayluo

@jiasli
Tried cleaning /home/(username)/.azure directory but no changes.
could you be more specific about azure CLI instances? Do you mean containers?
FYI We are running azure CLI commands from different user accounts concurrently on a single server.

could you be more specific about azure CLI instances? Do you mean containers?

I mean process.
Do different users share the same /home/(username)/.azure folder?

This happens executing from user accounts except root user.

Could it be you accidentally (or even deliberately) sudo once so that some helper file was created with root permission, therefore you non-root process won't be able to access it?
I know you said you already clean up the /home/(username)/.azure, but if your setup has one az instance running as root, the situation described above would re-occur.

could you be more specific about azure CLI instances? Do you mean containers?

I mean process.
Do different users share the same /home/(username)/.azure folder?

No. Every users have different directories inside /home. And their only .azure directory stored.
I can figure out multiple processes are using azure cli.

This happens executing from user accounts except root user.

Could it be you accidentally (or even deliberately) sudo once so that some helper file was created with root permission, therefore you non-root process won't be able to access it?
I know you said you already clean up the /home/(username)/.azure, but if your setup has one az instance running as root, the situation described above would re-occur.

Most probably it may happened. Maybe done from other admins. So how can I fix this gracefully?

Could it be you accidentally (or even deliberately) sudo once so that some helper file was created with root permission, therefore you non-root process won't be able to access it?

Most probably it may happened. Maybe done from other admins. So how can I fix this gracefully?

Let's validate this assumption first. When the issue occurs, before you delete /home/(username)/.azure, what are the files owners and permissions in that folder? If they were owned by root, why did your another admin need "sudo az login ..."?

Do different users share the same /home/(username)/.azure folder?

No. Every users have different directories inside /home. And their only .azure directory stored.
I can figure out multiple processes are using azure cli.
The process falling to uninterrupted sleep state (D).

"az login ..." is meant to be interactive. What happened to those now-sleeping processes? Did it go unresponsive so that you tried another one in a different terminal, which could then run into racing conditions with the first one? We better triage the first unresponsive "az login --debug" first. Kill the existing processes and start afresh, report back when with the "az login --debug" of your first failing login attempt.

Could it be you accidentally (or even deliberately) sudo once so that some helper file was created with root permission, therefore you non-root process won't be able to access it?

Most probably it may happened. Maybe done from other admins. So how can I fix this gracefully?

Let's validate this assumption first. When the issue occurs, before you delete /home/(username)/.azure, what are the files owners and permissions in that folder? If they were owned by root, why did your another admin need "sudo az login ..."?

The files permissions and ownership is belongs to the particular user only. I am not sure if ‘sudo az login’ used or not. Most probably it wouldn’t.

Do different users share the same /home/(username)/.azure folder?

No. Every users have different directories inside /home. And their only .azure directory stored.
I can figure out multiple processes are using azure cli.
The process falling to uninterrupted sleep state (D).

"az login ..." is meant to be interactive. What happened to those now-sleeping processes? Did it go unresponsive so that you tried another one in a different terminal, which could then run into racing conditions with the first one? We better triage the first unresponsive "az login --debug" first. Kill the existing processes and start afresh, report back when with the "az login --debug" of your first failing login attempt.

Exactly, I tried multiple times. Not from different terminal. After a process stuck I use ctrl+z to stop it and did again.
The process state D cannot be killed. Server restart can vanish these processes. I did that. And again the ‘az login —debug’ also going to uninterruptible sleep.
And the best part is I did reinstalling Azure cli by ‘yum reinstall azure-cli’

"az login ..." is meant to be interactive. What happened to those now-sleeping processes? Did it go unresponsive so that you tried another one in a different terminal, which could then run into racing conditions with the first one? We better triage the first unresponsive "az login --debug" first. Kill the existing processes and start afresh, report back when with the "az login --debug" of your first failing login attempt.

Exactly, I tried multiple times. Not from different terminal. After a process stuck I use ctrl+z to stop it and did again.
The process state D cannot be killed. Server restart can vanish these processes. I did that. And again the ‘az login —debug’ also going to uninterruptible sleep.

Did you mean, after you killed all existing processes and started afresh, your very first "az login --debug" will hang? And it is reproducible every time?
Ideally, we want the debug logs of your first hanging process.  is meant to be interactive. What happened to those now-sleeping processes? Did it go unresponsive so that you tried another one in a different terminal, which could then run into racing conditions with the first one? We better triage the first unresponsive "az login --debug" first. Kill the existing processes and start afresh, report back when with the "az login --debug" of your first failing login attempt.

Exactly, I tried multiple times. Not from different terminal. After a process stuck I use ctrl+z to stop it and did again.
The process state D cannot be killed. Server restart can vanish these processes. I did that. And again the ‘az login —debug’ also going to uninterruptible sleep.

Did you mean, after you killed all existing processes and started afresh, your very first "az login --debug" will hang? And it is reproducible every time?

Yes exactly.

Ideally, we want the debug logs of your first hanging process. So, you better stick with the --debug during your investigation.

$ az login --use-device-code --debug
cli.knack.cli: Command arguments: ['login', '--use-device-code', '--debug']
cli.knack.cli: init debug log:
Enable color in terminal.
cli.knack.cli: Event: Cli.PreExecute []
cli.knack.cli: Event: CommandParser.OnGlobalArgumentsCreate [<function CLILogging.on_global_arguments at 0x7f78343a8048>, <function OutputProducer.on_global_arguments at 0x7f7833f07ea0>, <function CLIQuery.on_global_arguments at 0x7f7833c98400>]
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableCreate []
cli.azure.cli.core: Modules found from index for 'login': ['azure.cli.command_modules.profile']
cli.azure.cli.core: Loading command modules:
cli.azure.cli.core: Name                  Load Time    Groups  Commands
cli.azure.cli.core: profile                   0.002         2         9
cli.azure.cli.core: Total (1)                 0.002         2         9
cli.azure.cli.core: Loaded 2 groups, 9 commands.
cli.azure.cli.core: Found a match in the command table.
cli.azure.cli.core: Raw command  : login
cli.azure.cli.core: Command table: login
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x7f7832c4a598>]
cli.azure.cli.core.azlogging: metadata file logging enabled - writing logs to '/home/localad.tech//.azure/commands/2023-04-18.15-45-04.login.672821.log'.
az_command_data_logger: command args: login --use-device-code --debug
cli.knack.cli: Event: CommandInvoker.OnPreArgumentLoad [<function register_global_subscription_argument..add_subscription_parameter at 0x7f78327d3268>]
cli.knack.cli: Event: CommandInvoker.OnPostArgumentLoad []
cli.knack.cli: Event: CommandInvoker.OnPostCommandTableCreate [<function register_ids_argument..add_ids_arguments at 0x7f7832724840>, <function register_cache_arguments..add_cache_arguments at 0x7f7832724950>]
cli.knack.cli: Event: CommandInvoker.OnCommandTableLoaded []
cli.knack.cli: Event: CommandInvoker.OnPreParseArgs []
cli.knack.cli: Event: CommandInvoker.OnPostParseArgs [<function OutputProducer.handle_output_argument at 0x7f7833f07f28>, <function CLIQuery.handle_query_parameter at 0x7f7833c98488>, <function register_ids_argument..parse_ids_arguments at 0x7f78327248c8>]
cli.azure.cli.core.auth.persistence: build_persistence: location='/home/localad.tech//.azure/msal_token_cache.json', encrypt=False
cli.azure.cli.core.auth.binary_cache: load: /home/localad.tech/skrishnan010/.azure/msal_http_cache.bin
urllib3.util.retry: Converted retries value: 1 -> Retry(total=1, connect=None, read=None, redirect=None, status=None)
cli.azure.cli.core.auth.binary_cache: save: /home/localad.tech//.azure/msal_http_cache.bin
cli.azure.cli.core.auth.binary_cache: save: /home/localad.tech//.azure/msal_http_cache.bin
urllib3.connectionpool: Starting new HTTPS connection (1): login.microsoftonline.com:443
urllib3.connectionpool: https://login.microsoftonline.com:443 "GET /organizations/v2.0/.well-known/openid-configuration HTTP/1.1" 200 1589
cli.azure.cli.core.auth.binary_cache: save: /home/localad.tech//.azure/msal_http_cache.bin
cli.azure.cli.core.auth.binary_cache: save: /home/localad.tech//.azure/msal_http_cache.bin
msal.authority: openid_config = {'token_endpoint': 'https://login.microsoftonline.com/organizations/oauth2/v2.0/token', 'token_endpoint_auth_methods_supported': ['client_secret_post', 'private_key_jwt', 'client_secret_basic'], 'jwks_uri': 'https://login.microsoftonline.com/organizations/discovery/v2.0/keys', 'response_modes_supported': ['query', 'fragment', 'form_post'], 'subject_types_supported': ['pairwise'], 'id_token_signing_alg_values_supported': ['RS256'], 'response_types_supported': ['code', 'id_token', 'code id_token', 'id_token token'], 'scopes_supported': ['openid', 'profile', 'email', 'offline_access'], 'issuer': 'https://login.microsoftonline.com/{tenantid}/v2.0', 'request_uri_parameter_supported': False, 'userinfo_endpoint': 'https://graph.microsoft.com/oidc/userinfo', 'authorization_endpoint': 'https://login.microsoftonline.com/organizations/oauth2/v2.0/authorize', 'device_authorization_endpoint': 'https://login.microsoftonline.com/organizations/oauth2/v2.0/devicecode', 'http_logout_supported': True, 'frontchannel_logout_supported': True, 'end_session_endpoint': 'https://login.microsoftonline.com/organizations/oauth2/v2.0/logout', 'claims_supported': ['sub', 'iss', 'cloud_instance_name', 'cloud_instance_host_name', 'cloud_graph_host_name', 'msgraph_host', 'aud', 'exp', 'iat', 'auth_time', 'acr', 'nonce', 'preferred_username', 'name', 'tid', 'ver', 'at_hash', 'c_hash', 'email'], 'kerberos_endpoint': 'https://login.microsoftonline.com/organizations/kerberos', 'tenant_region_scope': None, 'cloud_instance_name': 'microsoftonline.com', 'cloud_graph_host_name': 'graph.windows.net', 'msgraph_host': 'graph.microsoft.com', 'rbac_url': 'https://pas.windows.net'}
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/devicecode HTTP/1.1" 200 473
cli.azure.cli.core.auth.identity: To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code FC23YSKMB to authenticate.
msal.telemetry: Generate or reuse correlation_id: 801eec5e-d0fb-4da7-9f4c-dd9a15f8870a
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
FC23YSKMBurllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510                                                                                urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 400 510
urllib3.connectionpool: https://login.microsoftonline.com:443 "POST /organizations/oauth2/v2.0/token HTTP/1.1" 200 5559
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 found existing lock file, will retry after 0.250000 second
msal_extensions.cache_lock: Process 672821 failed to create lock file

Environment summary
CLI version (az --version : 2.38.0) / OS : RHEL 7 / Shell Type : bash

I can't repro this issue with a clean CentOS 7 container:
docker run -it --rm centos:7

How can we reproduce this? @violascarlett, are you using a certain container image, or a specific SKU of Azure VM etc.?

@rayluo yeah we are using Azure VM, installed RHEL 7.9 OS. And I am not sure how to reproduce this issue🙁.
If nothing helps can I try removing Azure cli and install it again? But #yum reinstall azure-cli didn’t help.

@rayluo I tried installing azure cli version 2.29.0 and it is working. I think something wrong with the msal authentication.


Update to a ready for demoing state

The Show Tutorial button is only displayed when you aren't logged in, as soon as you log in the button disappears from the menu. It needs to be added to the other menu or as part of the array of navigation pages: https://github.com/driftyco/ionic-conference-app/blob/master/src/app/app.template.html#L37-L69

There is something weird going on with Deeplink. If I go to the following URL http://localhost:8100/#/about/about it loads the About page for about a second and then redirects home.

Hmmm, that almost sounds like a deeplink bug. Gonna try using the deeplinker with just a plain starter and see if i can repro.

Well changing this code:

    // Check if the user has already seen the tutorial
    this.userData.checkHasSeenTutorial().then((hasSeenTutorial) => {
      if (hasSeenTutorial === null) {
        // User has not seen tutorial
        this.rootPage = TutorialPage;
        this.storage.set('hasSeenTutorial', 'true');
      } else {
        // User has seen tutorial
        this.rootPage = TabsPage;
      }
    });

to just this:
    this.rootPage = TabsPage;

fixes it.

Regarding the deeplinking, I created an issue here: https://github.com/driftyco/ionic/issues/9074

Looks good 👍


peer_list API changed in V20 - Dependencies?

API change for v20
https://github.com/lbryio/lbry/pull/966
I have made the adjustment for lbry.go here
However, has anyone reviewed where else the peer_list API is used? The change in returned parameters could cause issues for applications using this API.

@tiger5226 I don't think we are using this anywhere else as it's been quite unreliable. We used to have an availability function in the app, but that was removed until the functionality is more consistent.
Going to close this for now...


[st-engine] Add engine client

Why dependency check error? The seatunnel-engine-common is self module.


Controller names should be singular, as per Laravel naming conventions

I'm transitioning my application to this package, up till now I had been using Laravel's naming conventions for my controllers, so PostController not PostsController.
Laravel naming conventions
I was a bit confused as to why this is not the case by default with this package, just means that in order to extend my controllers the names have to be explicitly defined, I see no reason to deviate from the conventions.
$api->resource('quotes')->controller('QuoteController')

Is there a way for me to configure this if it can't be made default for the package?

Ah it's because we're following the JSON API convention of pluralising the resource name.
I'd be open to making this a setting in the API's config file though, so that you can toggle it to the singular (the default setting would need to be to pluralise it, as that's the default at the moment).
Any chance you could do a PR?

Added a singularControllers() helper method. Will be in the next release: v1.3.0.
Usage is as follows:
JsonApi::register('default')
    ->withNamespace('Api')
    ->singularControllers()
    ->routes(function ($api, $router) {
        $api->resource('posts')->controller(); // uses PostController
    });


Include x-forwarded-client-certificate header

Please describe your use case / problem.
When client-certificate verification is used, the x-forwarded-client-certificate (XFCC) header is not being set (or any equivalent one), so the client app can't know the identity of the user making the request.
I think this is required for client certificate validation to be useful.
Describe the solution you'd like
In the TLSContext definition, add a property that enables setting the XFCC header.
  apiVersion: ambassador/v1
  kind: TLSContext
  name: myapp-tls-context
  hosts:
  - myapp.blabla.org
  secret: myapp-server-tls
  ca_secret: myapp-cacert
  cert_required: true / false
  xfcc: true / false  <<== for example

The configuration could go further, and allow mapping Envoy's "forward_client_cert_details" and "set_current_client_cert_details" properties.
https://www.envoyproxy.io/docs/envoy/latest/api-v2/config/filter/network/http_connection_manager/v2/http_connection_manager.proto.html?highlight=forward_client_cert_details#enum-config-filter-network-http-connection-manager-v2-httpconnectionmanager-forwardclientcertdetails
Describe alternatives you've considered
I couldn't find any.
Additional context
Nothing.

I have encountered this issue as well, and cannot use Ambassador without a solution for this issue, as it's a requirement.
However I would like the client certificate to be forwarded to the AuthService, as that's where we would expect the client identity to be verified (e.g. from the subject field in the certificate).

This issue should not be closed, because is not yet been resolved !!

Could you please give some priority to this?

This issue definitely needs more priority. It also currently prevents us from using Ambassador in production.
In a case when Ambassador does TLS termination it must be able to pass client certificate information somewhere, so that some service could extract identity from the certificate and perform necessary authentication steps. For example, see this documentation on Keycloak's client cert auth configuration behind a reverse proxy with TLS termination.
Passing client certificate information through is a trivial configuration for HAProxy, Apache, Nginx and Traefik, and I think is absolutely needed for Ambassador.

It looks like there is a possible workaround to add client certificate to headers.
I stumbled upon the following special field from Envoy:
https://github.com/envoyproxy/envoy/blob/92c085afaeeed138067f289fa9294d56afc91e25/source/common/router/header_formatter.cc#L261
that appear to work fine with Ambassador.
For example:
  add_request_headers:
    x-forwarded-client-cert: "%DOWNSTREAM_PEER_CERT%"

Un-stale

not stale

It looks like there is a possible workaround to add client certificate to headers.
I stumbled upon the following special field from Envoy:
https://github.com/envoyproxy/envoy/blob/92c085afaeeed138067f289fa9294d56afc91e25/source/common/router/header_formatter.cc#L261
that appear to work fine with Ambassador.
For example:
  add_request_headers:
    x-forwarded-client-cert: "%DOWNSTREAM_PEER_CERT%"

Unfortunately it doesn't help if I want to forward the certificate to the AuthService, as it seems there is currently no way to add headers to the forwarded request.
I sincerely hope that PR#2053 takes this into consideration as well.

@sagibel Unfortunately it will not. Requests to the AuthService go through a different routing mechanism in Ambassador than requests to upstreams and so the add_request_headers headers are not added to requests to the AuthService.
It appears like this could be possible to add headers in add_request_headers to the request to an http AuthService from taking a brief look at Envoy's docs. We may need to do something like the workaround  mentioned here to get it working with gRPC AuthServices.

not stale

Hi everyone,
Im struggling to forward x-forwarded-client-cert in the backend service how the full config should look like to enable it,
i tried to follow example at https://www.getambassador.io/docs/latest/topics/running/tls/mtls/
I'm not using authservice i just need to pass the cert chain downstream to keycloak
This setup doesnt pass that header at all, i tried both tls termination and tls origination
Do i have to somehow enable %DOWNSTREAM_PEER_CERT%  first? The header is just not present.
apiVersion: getambassador.io/v2
kind:  Mapping
metadata:
  name: iam-service-mapping-smartcard
spec:
  prefix: /iam-service-smartcard/
  rewrite: ""
  timeout_ms: 10000
  add_request_headers:
    x-forwarded-client-cert: "%DOWNSTREAM_PEER_CERT%"
  tls: iam-service-tls-context-smartcard
  service: https://iam-service-smartcar-http:8443

---

apiVersion: v1
data:
  tls.crt: ...
kind: Secret
metadata:
  name: iam-service-ambassador-client-certs
type: Opaque

---

apiVersion: getambassador.io/v2
kind: TLSContext
metadata:
  name: iam-service-tls-context-smartcard
spec:
  hosts: ["some.host.org"]
  secret: ambassador-certs
  ca_secret: iam-service-ambassador-client-certs
  cert_required: true

Added in #2053.

This issue has been addressed.


Vv 76 mssql rebase

What
Why
Screenshots
Notes

Miranda, will merge, i have checked the UI and all the scripts running on pg and mssql.


Describe the Pipelines Section

In the latest release, we added the Pipelines section to the KubeRocketCI portal, where users can track and manage all the pipelines run within the platform. This feature should be described in detail in a dedicated documentation page.

The Pipelines Overview page has been added to documentation. It provides a deep look into the pipelines section of the KubeRocketCI portal.


Feature: an option to not include the terminal titlebar

As I was thinking more about #10, I realized that I'd prefer to not spend any space
in the README on the titlebar of the terminal (and thus not have any title).
This is similar to how your own demo in https://github.com/FHPythonUtils/AnsiToImg/blob/master/README.md#example-output currently has titles removed.
Describe the solution you'd like
I would like the option to set title=None in the API and a --no-title option in the CLI
that removes the titlebar entirely.
Alternative
It's likely possible to crop the image like suggested in https://github.com/FHPythonUtils/AnsiToImg/issues/7#issuecomment-1445359924, but I'd like a solution that doesn't depend on knowing the exact image size.

Yeah I'm not sure how easy this will be to do unfortunately. Grepping the inner svg shouldn't be a problem but calculating the width and height might be a bit fiddly! I'll have a play around and see what I can do

I haven't looked into it, but it sounds like you aren't actually generating the SVG inside the tool? If rich does it, this might be more appropriate to implement in rich first.
In any case, I think this would be a nice feature, but there's absolutely no pressure or rush for you to work on it. If I feel like it, I might look into it as well, but probably not right now.
Thanks again for making a nice tool!


Support for Uptime Robot APIv2

According to https://uptimerobot.com/apiv1

Important: The APIv1 will be retired by 1 Jan 2017 and please switch to APIv2 before that date.

As it's well passed 1 Jan 2017, they could kill off APIv1 support any day so I've hacked this together and it's working with their APIv2.  I'm no Python guy so feel free to tidy anything up.

Pretty cool. Thanks.


Fix TODO: use sum insted of left fold

As I understand from rustlang docs on the sum trait, it is available in stable since 1.12.
Let me know if I am missing something and there is a reason this cannot be merged.

Thanks!


Exiting tui in gdb causes a segfault

Describe the bug
Exiting tui mode in gdb causes a segfault. This is not the case on other platforms.
To Reproduce
Run gdb -tui. Type tui disable and hit return.
jamie@machine:~$ gdb -tui focus/test
Aborted (core dumped)

[nix-shell:~]$ lz4cat /var/lib/systemd/coredump/core.gdb.1000.a7b8b2b782434a4d8fe6be07cb8cb0d9.4596.1607891435000000.lz4 > gdb.dump

[nix-shell:~]$ gdb gdb ./gdb.dump
GNU gdb (GDB) 9.2
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-unknown-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from gdb...
(No debugging symbols found in gdb)
[New LWP 4596]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/nix/store/9df65igwjmf2wbw0gbrrgair6piqjgmi-glibc-2.31/lib/libthread_db.so.1".
Core was generated by `gdb -tui focus/test'.
Program terminated with signal SIGABRT, Aborted.
#0  0x00007feff0fc808a in raise ()
   from /nix/store/9df65igwjmf2wbw0gbrrgair6piqjgmi-glibc-2.31/lib/libc.so.6
(gdb) bt
#0  0x00007feff0fc808a in raise ()
   from /nix/store/9df65igwjmf2wbw0gbrrgair6piqjgmi-glibc-2.31/lib/libc.so.6
#1  0x00007feff0fb2528 in abort ()
   from /nix/store/9df65igwjmf2wbw0gbrrgair6piqjgmi-glibc-2.31/lib/libc.so.6
#2  0x000000000041b64c in handle_sigsegv(int) [clone .cold] ()
#3  <signal handler called>
#4  0x00007feff1000c61 in fwrite ()
   from /nix/store/9df65igwjmf2wbw0gbrrgair6piqjgmi-glibc-2.31/lib/libc.so.6
#5  0x00007feff1ce6734 in _rl_output_some_chars ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#6  0x00007feff1cdf6d9 in update_line ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#7  0x00007feff1ce177f in rl_redisplay ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#8  0x00007feff1ccbc33 in readline_internal_setup ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#9  0x00007feff1ce5479 in _rl_callback_newline ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#10 0x00007feff1ce54c8 in rl_callback_handler_install ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#11 0x000000000059efd0 in gdb_rl_callback_handler_install(char const*) ()
#12 0x000000000059fbff in display_gdb_prompt(char const*) ()
#13 0x000000000059f9ec in gdb_rl_callback_handler(char*) ()
#14 0x00007feff1ce5a20 in rl_callback_read_char ()
   from /nix/store/bcjk6hwdn1aq7m45a042a9k1lqih74ja-readline-8.0p4/lib/libreadline.so.8
#15 0x000000000059ecfe in gdb_rl_callback_read_char_wrapper_noexcept() ()
#16 0x000000000059f8b1 in gdb_rl_callback_read_char_wrapper(void*) ()
#17 0x000000000059ec1c in stdin_event_handler(int, void*) ()
#18 0x000000000059d906 in gdb_wait_for_event(int) ()
#19 0x000000000059de6c in gdb_do_one_event() ()
#20 0x000000000059df25 in start_event_loop() ()
#21 0x000000000066eeae in captured_command_loop() ()
#22 0x0000000000670dd5 in gdb_main(captured_main_args*) ()
#23 0x000000000042e7ac in main ()

I also have an rr recording if needed.
Notify maintainers
@globin @lsix
Metadata
jamie@machine:~/nixpkgs$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.80, NixOS, 20.09.2076.ffb3aab257e (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.9`
 - channels(jamie): `""`
 - channels(root): `"nixos-20.09.2076.ffb3aab257e"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Yep, I can reproduce it, I’ll to look into that asap

I have opened a PR that fixes this by back-porting the fix that was already available upstream (https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=a350efd4fb368a35ada608f6bc26ccd3bed0ae6b).

Wow, that was fast. Thanks so much :)

I'm afraid I don't have access to a computer at the moment due to some border-related excitement. Should be able to confirm by the end of the week.

I'm afraid I don't have access to a computer at the moment due to some border-related excitement. Should be able to confirm by the end of the week.

No problem. Just let me know when you see fIt so I close the issue.

No problem. Just let me know when you see fIt so I close the issue.

Yep, this works for me on the latest unstable. Thanks @lsix

Yep, this works for me on the latest unstable. Thanks @lsix


Proposed Element: Award RFP Document

This effort at transparency in spending will ultimately tell us where our federal tax dollars are spent, when they were spent, to whom they were paid and very generally for what (NAICS).  Transparency gets cloudy when you try to delver deeper into "for what" because that data isn't readily available.  Perhaps there is one line in FPDS-NG.  Perhaps the source documents are on FBO.  A great number of times this information is not readily available to the public.  FOIA requests would have to be issued to obtain this information.  An example is agency's fondness for creating BPAs that most companies do not have access to compete on.  Often times the high level BPA or IDIQ contract is available - but not the tasks that people are actually bidding to perform.
Technology exists (natural language processing/big data/etc.) to easily delve deeper in what tax money was spent to procure with the data (documents) we have available today.  This information exists in statements of work or whatever documents were used to solicit bids. This could be made more precise through structure, but using the documents as a start is also good.  This is how structure may be divined.  Methods for storing and retrieving this data are trivial today.  The source documents (like Section C, SOW as an example standard) for an award should be uploaded and made publicly available for any award at the contract or task order level.
Similarly, contracts that result from a grant should also be uploaded.  The example of the grant to Florida that went to buy busses made in Michigan is an example that could be derived.
Linking the source documents to the awards is a simple step that would enable an array of analytics.

I feel like this is more related to making the source documents available in an automated way (perhaps by award ID) and less about post-award spending reporting. @nvembar are you aware of any plans to make this available via FBO?

You have to keep in mind that most of the source documents are NOT available via FBO.  I would guess that most acquisition information is not readily available to the public because they are on hidden mechanisms/vehicles constrained to a certain group.  For example, Task Orders on GSA IT70, GSA PSS, Alliant, CIO-SP3, etc.  That's where a lot of work happens - on vehicles and then even further on BPAs within vehicles.  You don't see these unless you are on the vehicles.  I think in some cases certain companies are directly invited to participate.
The core of this data request is the statement of work / performance work statement; however having the full documentation set to include Q&A and solicitation instructions may also provide insight.
In my view, I see a document as an element.  Maybe unstructured, but still an element of data.  I think one of the most important data points because that's the only way we really know what the government is purchasing, especially related to services.  We know a lot of 541512 is bought - but what is that?  Could be a $1 micropurchase or a $33B Big Data Cloud offering.

I totally agree on this @HerschelC. I think there are some more systemic changes that need to happen with the procurement vehicles to collect and publish this data in a way that's easily consumable. CC'ing @vzvenyach in case he has more info in this area.


Unable to setup Azure AD login

Rancher Server Setup

Rancher version: 2.6.2
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.21.6+rke2r1
Cluster Type (Local/Downstream): Local

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Describe the bug

To Reproduce

Trying to setup Azure AD login isn't working for me.
I'm following this docs: https://rancher.com/docs/rancher/v2.6/en/admin-settings/authentication/azure-ad/
Result
There is no way to assign the "Azure Active Directory Graph" permission to the app as that API seems to be deprecated.
Expected Result
The API used is not deprecated and selectable via the "API permissions" dialog in the azure portal.
Screenshots

Additional context

This is still a problem... There is no way to set up azure AD login with rancher at the moment

@WolfspiritM You can still assign create AAD Graph permissions using the az CLI:
export AppId=<Application ID>
export ApiId=00000002-0000-0000-c000-000000000000         # Azure AD Graph API
export PermissionId=5778995a-e1bf-45b8-affa-663a9f3f4d04  # Directory.Read.All

az ad app permission add --id $AppId --api $ApiId --api-permissions $PermissionId=Role
az ad app permission grant --id $AppId --api  $ApiId

List of permission GUIDs: https://github.com/Azure/azure-powershell/issues/16067#issue-1023553283

I created the following ps1 in which I looked up all the required permissions that are required in the docs
param (
    [String] $appId,
    [String] $apiId = "00000002-0000-0000-c000-000000000000"
)

Function Set-Permission {
    param (
        [string] $permissionId
    )
    Write-Host("Setting permission $permissionId");
    Write-Host("add");
    az ad app permission add --id $appId --api $apiId --api-permissions "$permissionId=Role"
    Write-Host("grant");
    az ad app permission grant --id $appId --api $apiId
}

$requiredPermissionIds = @('a42657d6-7f20-40e3-b6f0-cee03008a62a','5778995a-e1bf-45b8-affa-663a9f3f4d04','6234d376-f627-4f0f-90e0-dff25c5211a3','c582532d-9d9e-43bd-a97c-2667a28ce295','cba73afc-7f69-4d86-8450-4978e04ecd1a','311a71cc-e848-46a1-bdf8-97ff7156d8e6')
foreach ($id in $requiredPermissionIds) {
    Set-Permission($id)
}

All grant actions fail, however:
Operation failed with status: 'Bad Request'. Details: 400 Client Error: Bad Request for url: https://graph.windows.net/{my-tenant-id}/oauth2PermissionGrants?api-version=1.6

Has anyone a clue?
Would be very nice to know if there is some planning on switching to the new API's, since this API is apperently going to be depricated very soon (june 2022)


INTLY-2712 check cr for valid json

Check cr for valid JSON before importing dashboard.
Verification steps:

Install the operator from this branch
Create a dashboard with invalid json and try to import it
The operator should log a meaningful error and the dashboard should not get imported
Check the dashboard CR: a status message with the json error should have been appended
fix the json and make sure the dashboard is imported now

closes #18

@david-martin yes, at the moment we don't remove the error message. No i'm thinking, maybe it would be a good idea to do so?

@pb82 The reason I mention it is because it made me think there was still a problem with the dashboard. However, that wasn't the case as the Dashboard was added OK to Grafana when I checked the grafana-dashboards configmap & UI.

Tested changes locally, all looks good. CR with invalid JSON is being ignored and an error logged in status.
  messages:
    - message: 'invalid JSON, error: invalid character ''\n'' in string literal'
      timestamp: 'Friday, 19-Jul-19 10:49:11 IST'
    - message: dashboard 'ferwa/dashboard-with-plugins.json' imported
      timestamp: 'Friday, 19-Jul-19 10:49:39 IST'
    - message: installing plugin<EMAIL_ADDRESS>      timestamp: 'Friday, 19-Jul-19 10:49:44 IST'
    - message: installing plugin<EMAIL_ADDRESS>      timestamp: 'Friday, 19-Jul-19 10:49:44 IST'

Import message is being created after fixing the invalid JSON.
I can't approve as I'm the author of the PR but :+1:


[WIP] Tracing: traces now correctly form a tree

To support https://github.com/micro/platform/pull/14

Self merge when you are ready!


Organization Name
Recast
Your Name
Russell Pierce
Your current position
VP of Engineering
(Optional) Company logo
No response
(Optional) Use case
No response
Also using other Powertools for AWS Lambda languages?

[ ] Java
[ ] TypeScript
[ ] .NET

Hi @russellpierce! This is absolutely amazing! I'll send a pull request to add Recast as a public reference. Just confirming that the link is https://getrecast.com/, right?
We always love to hear about how customers are using Powertools for AWS Lambda and the ideas they have, the challenges they're facing, and how we can make it even better. We're completely open to scheduling a meeting and hearing from you about any needs or requirements you may have. If you'd like, please send an email to aws-powertools-maintainers at amazon.com, and we can arrange a meeting to discuss further.
We're eager to learn about your use cases and happy to assist you in any way we can.
Thanks

Closed via #4491


Cannot install voicebank

How to fix that

This is Windows preventing OpenUtau from modifying C:\Program Files. Consider moving OpenUtau to somewhere else.

oh, tnx for help


SP-PnP-Js module is not working in SPFx extension

Thank you for reporting an issue, suggesting an enhancement, or asking a question. We appreciate your feedback - to help the team understand your
needs please complete the below template to ensure we have the details to help. Thanks!
####Category
[X] Bug
Version
Please specify what version of the library you are using: [ 1.3.4 ]
sp-pnp-js - 2.0.6
Expected / Desired Behavior / Question
sp-pnp-js should be working fine with existing SPFx components.
####Observed Behavior
Failed to load sp-pnp-js in SPFx component. We have used sp-pnp-js module in SPFx extension that is currently deployed to production. Started getting an error today in SPFx Extension with an issue below. Screenshot also attached.
Uncaught (in promise) Error: ***Failed to load component "477d3c2c-af7d-4fb2-8bef-96e5b6271f20" (XXXXXCommandSet).
Original error: ***Failed to load path dependency "sp-pnp-js" from component "477d3c2c-af7d-4fb2-8bef-96e5b6271f20" (XXXXXXCommandSet).
Original error: ***Failed to create global variable "sp-pnp-js" from script "{}"
at t [as constructor] (listview-host-assembly_en-us_3d49d8367af105d9948165ebdab83af1.js:208)
at new t (listview-host-assembly_en-us_3d49d8367af105d9948165ebdab83af1.js:413)
at Function.e.buildErrorWithVerboseLog (listview-host-assembly_en-us_3d49d8367af105d9948165ebdab83af1.js:413)
at Function.e.buildLoadComponentError (listview-host-assembly_en-us_3d49d8367af105d9948165ebdab83af1.js:413)
at listview-host-assembly_en-us_3d49d8367af105d9948165ebdab83af1.js:413
at

####Steps to Reproduce
Create an SPFx extension and add the below to config.json
"externals": {
"sp-pnp-js": {
"path": "https://cdnjs.cloudflare.com/ajax/libs/sp-pnp-js/2.0.6/pnp.min.js",
"globalName": "sp-pnp-js"
}
}
Thanks!

sp-pnp-js is a module, so the correct way to reference it would be:
"sp-pnp-js": "https://cdnjs.cloudflare.com/ajax/libs/sp-pnp-js/2.0.6/pnp.min.js"

You can verify that using the Rencore Script Check @ https://rencore.com/sharepoint-framework/script-check/

Thanks @waldekmastykarz - for ref see also: https://github.com/SharePoint/PnP-JS-Core/issues/718

@waldekmastykarz, yeah that was done already. It just broke from yesterday.
Sorry for a duplicate issue thread. We can close it here and continue here SharePoint/PnP-JS-Core#718. Thanks.


Added e2e tests to the project

'web-tests' folder

Coverage remained the same at 97.753% when pulling b00e37ead2095ff352cc0b9f66785306ef9952ab on roberttapia:master into b1424e1ee53bf6863023a320467fe4c76dc07f01 on unosquare:master.


Azure Storage - Diagnostics Logs and Metrics not stored periodically

Hi,
We have an issue in retrieving diagnostics data from Azure Storage accounts for Azure SQL database.
We have configured diagnostics settings for an Azure SQL database to store the logs and metrics to a Storage account. As per configuration, the .json file containing the metrics is generated under a Blob service in the storage account. Each .json file contains data for 1 hour, one record for each minute (starting from 00:00 to 00:59).
As per the Microsoft blog(https://docs.microsoft.com/en-us/azure/monitoring-and-diagnostics/monitoring-archive-activity-log), we expect this .json file to be generated at an interval of 1 hour. But the issue is, the files are created randomly.

Example 1 : .json files for hour=0,1,2,3,4,5 are generated on hour=8
Example 2 : .json files for day=1,hour=22,23 are generated on day=2,hour=1
Expectation : .json file for h=5 should be generated once the 5th hour of the day is completed.
Note : The expected behavior also occurs sometimes.
Sample scenario : UTC time now is 9.00am. So we expect the last .json file to be generated for h=8,m=0 to 59. But the last generated file stands at h=4 containing data from 4.00 to 4.59

Kindly check the issue and let us know how this inconsistency with diagnostics metrics storage an be avoided and generate files at regular intervals, as we rely on accurate periodic diagnostics.
Thanks in advance !

This issue is not related to the SDK.  If this issue persists, please file a support ticket to track this.


Improve Search Help

Today the search help function searches through Classes, Methods, Signals and Properties separately.
AFAIU, you can't restrict the search to the SceneTree methods by typing SceneTree. into the search box, because that leaves you with just the SceneTree class and a bunch of other entries where "SceneTree" appears in the description. It is not possible to restrict entries to properties or signals by typing "Property" or "Signal" either.
It would be nice to implement some form of fuzzy search that would let us do this. The Search Help is a great tool to discover/navigate the API. I'd love to see it become even more powerful.

As the current "Search Help" dialog let's you filter what you want, I think this can be considered closed.


Add numpy version

Adding NumPy version for an educational resource for training/sales.
Summary of changes:

Added demo_numpy.py: BQM construction function uses NumPy; update README to discuss
Remove usage of AdjVectorBQM

@hhtong I addressed all of your feedback, let me know if there's any other changes you'd like to see.

@hhtong I addressed all of your feedback, let me know if there's any other changes you'd like to see.


.Net Core Implementation

We are moving to .Net Core and would like to adopt PACT testing.  The current library does not support targeting Core, would you be interested in accepting a PR that expanded support to .Net Core?

Just realized this is a dup of #78


Wrong hunter level on Varrock Hard Diary

Wiki says you need 66 Hunter for 2x Dashing Kebbit fur but you actually need 69 hunter.
https://oldschool.runescape.wiki/w/Dashing_kebbit

You need 66 to make the cape. Players can get the fur from the GE and then make the cape


Where are the dyno-queues?

We know conductor are relying on dyno-queues， but where are them？Should I deploy a dyno-queue cluster or it just inside the conductor server instances？

gothrough this link ..  https://netflixtechblog.com/distributed-delay-queues-based-on-dynomite-6b31eca37fbc
Dyno-Queue is on top of Dynomite. Dyno-queue uses dynomite java driver and gets all the benefits from dynomite, you need to have Dynomite Cluster setup..


Feature request: mounting GCE-Disk directories (or even files) into containers paths

Hi,
here we were talking about the possibility of having a way to mount GCE Disks paths, being files or directories, to internal containers paths.
For containers living in the same or other cluster and the same or other pod:

mount gceDisk1/gcePath1/ to containerA internal path
mount gceDisk1/gcePath2/ to containerB internal path

Some use examples of this approach could be:

In case of using the nginx image, mounting a different file for image path /etc/nginx/nginx.conf, altering its configuration without having to create a Dockerfile
In case of using mysql image,  mounting a different directory for image path /etc/mysql/conf.d, altering its configuration without having to create a Dockerfile

In my opinion it's a great advantage to change default images behaviors altering its paths by Kubernetes spec files instead of having to rebuild the image by Dockerfile, tagging and publishing..
Hernan

cc @thockin

It's been brought up before but left on the table.  It's not a bad idea,
but will need some careful thought around the Cleaner logic required to get
it right.
On Thu, Jul 23, 2015 at 11:07 AM, Vish Kannan<EMAIL_ADDRESS>wrote:

cc @thockin https://github.com/thockin
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11756#issuecomment-124189232
.


CORE-9483 Enabling sending of primitive types across sessions

This PR addresses CORE-9483.
Background:
Due to some errors observed in Corda 4 when serialising primitive types in flow sessions, the function requireBoxedType(type: Class<*>) { require(!type.isPrimitive) } was added to a number of classes to assert that a passed parameter was not a primitive type. Today, this function is causing failures when we try to pass integers through FlowMessaging as Kotlin autoboxing is converting the boxed java.lang.Integer/kotlin.Int to the primitive int which then fails the requireBoxedType() check.
Findings:
When we remove the requireBoxedType() check, the sending of primitive types fails for two reasons:
Autoboxing
It's important to note that, unlike Java, Kotlin has no primitive types; instead, all primitive types (e.g., boolean, float, int) are represented by boxed classes. When Kotlin code is compiled to JVM bytecode, all Kotlin variables are compiled to Java variables and it is up to the compiler to determine whether they become the boxed (class) or unboxed (primitive) equivalent.
To send data between flows, we use the send(), receive(), and sendAndReceive() functions on the FlowSession class (or equivalent functions in FlowMessaging). The send(payload: Any) function accepts a payload of Any type, serialises it using an AMQP serialiser, and sends the resulting bytes to the designated counterparty.
To receive data, we use the receive(receiveType: Class<R>) function, which takes a type token receiveType representing the expected class of the received data. The function waits for a send event from the counterparty flow and attempts to deserialise the received payload to the specified class type.
The issue here is that the Any parameter in send() will cause passed arguments to be implicit cast to their boxed representation at runtime, meaning that a passed int will be cast to a Java.lang.Integer before being serialised and sent. The generic type parameter Class<R> in send() does not do such casting, and so upon receiving the serialised Java.lang.Integer at runtime we attempt to deserialise it as a primitive int causing the following exception:
java.io.NotSerializableException: Internal deserialization failure: java.lang.ClassCastException: Cannot cast java.lang.Integer to int

java.io.NotSerializableException.txt
Kryo Serialization / Class Loading
When we call receive() on a flow, we persist a FlowIORequest into the flow fiber and suspend the fiber, waiting for the expected message to be received. This suspension was triggering a Kryo serialisation/deserialisation of the flow which was tripping up when trying to load the int 'class' from the system class loader as there is no class to load for an int primitive, resulting in the following:
com.esotericsoftware.kryo.KryoException: net.corda.sandbox.SandboxException: Class int was not from a bundle, and could not be found in the system classloader.

KryoException- net.corda.sandbox.SandboxException.txt
Solution:
Autoboxing
We can assume that any primitive value will be autoboxed before serialization, and deserialize it as such on receipt with the following check:
return if (receiveType.isPrimitive) {
    deserializeReceivedPayload(received, receiveType.kotlin.javaObjectType)
} else {
    deserializeReceivedPayload(received, receiveType)
}

(Note: We could call the deserialiser with receiveType.kotlin.javaObjectType in all cases – it would simply be a no-op on boxed classes – but I felt this conditional check made the intent more clear).

This check makes sure we pass the class type of the boxed representation to the deserialiser. The return type, R, will ensure that the returned value is implicit cast to whichever representation the receive() function was called with, either boxed or primitive.
We could also solve this by modifying the API of send() and sendAndReceive() to prevent autoboxing, but this solution doesn't require any modifications to the API.
Kryo Serialization / Class Loading
Flow class loading is delegated through the GroupSandboxImpl; I've modified this delegator so that when it is called with a primitive, e.g. int or float, it will directly return the Java Class wrapper for that primitive from a mapping:
private val primitiveTypes = mapOf(
    "boolean" to Boolean::class.javaPrimitiveType,
    "byte" to Byte::class.javaPrimitiveType,
    "short" to Short::class.javaPrimitiveType,
    ...
)

Miscellanious
Special handling was also added to DeserializationInput for char types, as they go through a widening conversion to an Integer type in-transit. More info on this can be found in comments below.
Testing
As of now, I've written unit tests for AMQP serialisation/deserialisation of primitive types, and I've performed manual E2E testing of flows with primitive types. I'll add automated E2E and integration testing of the new behaviour.

Derp, is someone suggesting that AMQP has been unable to handle primitive Java types for the past 7 years? :thinking:

It's important to note that, unlike Java, Kotlin has no primitive types; instead, all primitive types (e.g., boolean, float, int) are represented by boxed classes.

Not quite. It's actually like this:

Int -> int
Int? -> java.lang.Integer
Short -> short
Short? -> java.lang.Short
Byte -> byte
Byte? -> java.lang.Byte
Char -> char
Char? -> java.lang.Character
Float -> float
Float? -> java.lang.Float
Double -> double
Double? -> java.lang.Double

The point is that Java primitives can never be null.

AMQP serialization has always supported Java primitive types - see AMQPPrimitiveSerializer. Consequently, there should not be any need to touch DeserializationInput.
Seriously, what's going on?

@chrisr3
AMQP serialization has always supported Java primitive types (Not so much 'handle' as return essentially untouched); the purpose of this ticket is to enable our flow sessions to handle Java primitive types. However, this work uncovered the fact that our implementation of DeserializationInput could not handle char types, because they were being cast to integers by the AMQPPrimitiveSerializer (for valid reasons).
The functional changes to DeserializationInput simply add handling for this (HERE) and, while doing that, I split out the readObject() function so it wouldn't need @Suppress("NestedBlockDepth", "ComplexMethod") annotations to pass the linter.

this work uncovered the fact that our implementation of DeserializationInput could not handle char types, because they were being cast to integers by the AMQPPrimitiveSerializer (for valid reasons).

I am deeply sceptical of any changes to AMQP's handing of primitive types. Our SerializationOutputTests class may not explicitly test char atm, but that's easily fixed:
diff --git a/libs/serialization/serialization-amqp/src/test/kotlin/net/corda/internal/serialization/amqp/SerializationOutputTests.kt b/libs/serialization/serialization-amqp/src/test/kotlin/net/corda/internal/serialization/amqp/SerializationOutputTests.kt
index a442695e42..fc93e21cda 100644
--- a/libs/serialization/serialization-amqp/src/test/kotlin/net/corda/internal/serialization/amqp/SerializationOutputTests.kt
+++ b/libs/serialization/serialization-amqp/src/test/kotlin/net/corda/internal/serialization/amqp/SerializationOutputTests.kt
@@ -100,6 +100,9 @@ class SerializationOutputTests {
     @CordaSerializable
     data class TestBoolean(val b: Boolean)

+    @CordaSerializable
+    data class TestChar(val c: Char)
+
     @CordaSerializable
     interface FooInterface {
         val pub: Int
@@ -332,6 +335,12 @@ class SerializationOutputTests {
         serdes(obj)
     }

+    @Test
+    fun `test char`() {
+        val obj = TestChar('?')
+        serdes(obj)
+    }
+
     @Test
     fun `test foo implements`() {
         val obj = FooImplements("Hello World!", 123)

And it passes, which suggests to me that AMQPPrimitiveSerializer is working correctly.

The functional changes to DeserializationInput simply add handling (narrowing conversion back to char) for this (HERE)

Are you trying to serialize naked primitive types with AMQP? I don't believe we have ever supported doing that - @mnesbit?

The functional changes to DeserializationInput simply add handling (narrowing conversion back to char) for this (HERE)

Are you trying to serialize naked primitive types with AMQP? I don't believe we have ever supported doing that - @mnesbit?

Actually, you've uncovered a good point. So, honest-to-goodness primitive types are handled as of today HERE – we simply return them... Or so it may seem. In actuality, the fact that the obj parameter is of type Any will cast any primitive objects to their boxed equivalent, so this will never be hit.

@CordaSerializable
data class TestChar(val c: Char)

@Test
fun `test char`() {
   val obj = TestChar('?')
   serdes(obj)
}

Re: this test, I mentioned it offline (but it's worth noting here for future reference), by wrapping the char in a dataclass we're invoking the ComposableObjectSerializer rather than the AMQPPrimitiveSerializer. This, in turn, invokes the ComposableTypePropertySerializer on the char which delegates to AMQPCharProperty[Write/Read]Strategy HERE which do the same sort of casting as I've added for the failing case.


deprecate functions and refactor tests

This PR deprecates convenient functions that relies on hard-coded endpoint. It also updates test tools dependences and refactors tests.

Pull Request Test Coverage Report for Build 351

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 98.639%

Totals

Change from base Build 349:
-0.06%

Covered Lines:
145

Relevant Lines:
147

💛  - Coveralls


nginx message

[x] parsers and subparsers create mdict
[x] Message ctor: mdict
[x] adapt listener
[x] adapt repo
[x] adapt factory
[x] adapt message
[x] nginx parser
[x] ua parser

resolves #51


Update documentation

I believe the documentation is out of date. For example, SqliteConnection::open takes a Path, not a &str.

Hmm you're right. It's supposed to be built by Travis when it runs the tests. I'll kick it around and see if I can get it to refresh. Thanks.

Unrelated to the documentation issue, but it would be nice if these methods took a T: AsPath instead of a Path.

Er, that got changed to AsRef<Path>, not AsPath.

Good idea - opened #33 to track that.


ROS1 and ROS2 Bloom instructions contradict each other

The ROS 2 tutorial for Releasing a ROS 2 package with bloom just says to follow the ROS 1 Releasing a Package for the First Time. But at the same time, the step 2 of that tutorial says:

Note: This procedure will not work for ROS2 packages, since ROS2 uses a new build system. If using ROS2, you will have to create the changelog and release tags yourself. Please see the ROS 2 Bloom page for more information.

The problem is that the ROS 2 tutorial doesn't mention anything about changelog and release tags creation.
Is the ROS 2 tutorial missing those steps? Or is the note in the ROS 1 tutorial outdated?

The catkin_prepare_release command does not currently update the setup.py file of packages built with the ament_python build type. catkin_generate_changelog has been working for me in all of the ROS 2 packages I've used it with.
I think the ROS wiki documentation should be updated to note the specific limitation where ament_python packages are concerned or explicitly note that only catkin and ament_cmake packages are currently supported to remain future proof against other build systems.

Makes sense, I wrote this issue while I was reading the guide to use it on an ament_cmake package. In the end I was able to use it without any problem (it was an ament_cmake package).
I'll update the note in the ROS wiki saying that catkin and ament_cmake packages are currently supported.
Thanks!

I think the ROS wiki documentation should be updated to note the specific limitation where ament_python packages are concerned or explicitly note that only catkin and ament_cmake packages are currently supported to remain future proof against other build systems.

It would be good to follow up on the documentation side of this.
I updated the catkin-prepare-release tool to at least provide correct feedback - that is works for ament_cmake packages: ros-infrastructure/catkin_pkg#259


From the tower side, I do a tight check, verifying that the enemy is alive. Maybe you can put a variable that tells me if I can shoot it or not. False if can't and true if I can. Or you can simply set isAlive to false.

Fixed in #70


[DO NOT MERGE] background-image: image-set not prefixed

For some reason background-image: image-set is not being prefixed for Chrome with -webkit although inline-style-prefixer v4.0.0 able to prefix this value:
var Prefixer = require("inline-style-prefixer")

const style = {
  'background-image': 'image-set(url(test.jpg) 2x)'
}

const prefixer = new Prefixer()
prefixer.prefix(style)
// ->
{
  background-image: [
    "-webkit-image-set(url(test.jpg) 2x)",
    "image-set(url(test.jpg) 2x)"
  ]
}

Here is also codepen with the setup to reproduce the bug.

Can you move this to the stylis repo? That library handles the prefixing.
On Tue, May 8, 2018 at 10:36 AM Konstantin Azizov<EMAIL_ADDRESS>wrote:

For some reason background-image: image-set is not being prefixed for
Chrome with -webkit although inline-style-prefixer v4.0.0 able to prefix
this value:
var Prefixer = require("inline-style-prefixer")
const style = {
'background-image': 'image-set(url(test.jpg) 2x)'
}
const prefixer = new Prefixer()prefixer.prefix(style)/*Will return { background-image: ["-webkit-image-set(url(test.jpg) 2x)", "image-set(url(test.jpg) 2x)"] }
Here is also codepen https://codepen.io/anon/pen/yjpzQx with the setup
to reproduce the bug.
You can view, comment on, or merge this pull request online at:
https://github.com/styled-components/styled-components/pull/1729
Commit Summary

Add failing test-case for image-set value

File Changes

M src/test/css.test.js
https://github.com/styled-components/styled-components/pull/1729/files#diff-0
(10)
M test-results.json
https://github.com/styled-components/styled-components/pull/1729/files#diff-1
(2)

Patch Links:

https://github.com/styled-components/styled-components/pull/1729.patch

https://github.com/styled-components/styled-components/pull/1729.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/pull/1729, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAiy1mps7PkvWQef9POqjx9acbjRX-cVks5twa13gaJpZM4T2w52
.


Fix sizing of HTTP Response header fields

Fixes the widths of the HTTP Response node header fields.
Moving to flex layout to remove the need to dynamically resize.

Coverage increased (+0.03%) to 67.769% when pulling 0171ffac6ade3cb094a4edcd56a632d2087dcf6e on fix-http-response-layout into 061afb3a9488bc4c79a508496196fea2d14f5ba1 on dev.


feat: switch to fastify v5

Migrate plugin to fastify 5
BREAKING CHANGE: fastify 4 is not supported anymore and minumum required Node.js version is 20.
🚨 Proposed changes

Please review the guidelines for contributing to this repository.

[[Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.]]
⚙ Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Refactor

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature Request: Get creator subscribers

I dont know if its possible but for this would allow me to fetch a list of people subscribed to me so I can give people perks on other mediums. Now I have to do it manually.

I see "User Subscriptions" but I don't think this is the same as creator subscribers.

Checking it now

I don't have a creator's account using which I can reverse engineer :(


add missing files to fix docker build

Looks like commit eb7a20 broke the docker build (via theDockerfile). Previously only main.go was COPY'd in the docker file.
Fix is simple, just COPY the whole cmd/spiffe-helper directory rather than just the main.go.
Looks like the nightly builds have been failing. One suggestion might to add building the docker image to the pr_build tests. I can take a stab at this if desired, I'd mostly just cut and paste from the nightly build minus the pushing.
Thanks,
Geoff

One suggestion might to add building the docker image to the pr_build tests. I can take a stab at this if desired, I'd mostly just cut and paste from the nightly build minus the pushing.

That would be a great addition if you have the time for it.

One suggestion might to add building the docker image to the pr_build tests. I can take a stab at this if desired, I'd mostly just cut and paste from the nightly build minus the pushing.

That would be a great addition if you have the time for it.

Thanks @gflarity . Just need to fix up the DCO and then we can merge this.

Done!


Clarity about dependencies

In some Linux distros, copying the password to the clipboard does not work even after installing openssl, libxcb-shape and libxcb-xfixes. We need to find out exactly what are the relevant libraries that need to be present (beyond X itself).

Thanks to cargo-deb, the generated .deb package already shows the dependencies on Debian systems:
Depends: libxcb-shape0, libc6 (>= 2.18), libxcb-xfixes0, libxcb1 (>= 1.6), libxcb-render0, libgcc1 (>= 1:4.2), libssl1.1 (>= 1.1.0)

What is left before closing this issue is to document these dependencies in the README.


[ANCHOR-585] Copy over changes into old doc versions

Copies the changes from https://github.com/stellar/stellar-docs/pull/1088

Preview is available here:http://developers-pr1089.previews.kube001.services.stellar-ops.com


Double scroll bar issue if grid's parent container is a fixed height

Hi,
If grid is within a fixed height container it will take the parent container height for the rows div (id: centerRow) and the paging div (id: south) will add another 38px or so to the overall height resulting in the parent container having a vertical scrollbar and the row container having a vertical scrollbar.  The height of the rows div should be calculated taking into account the height of the pagination container to avoid double scrollbars.

It appears that in the function 'BorderLayout.prototype.layoutHeight' the height of the south container is always zero.  So your calculation to set the height of the centerRow container is correct it's just that the south container's  height isn't correct (maybe because it hasn't got any content at that point?)

i've just done an update, 10.1.1, now the grid checks it's size every 500ms. can you advice is that fixes it? if not, i fear you might be experiencing something that's only for a particular browser / platform / version.

@ceolter thanks for the quick response!  I'll give it a go :-)

can you advise, is this fixed?

Sorry for the delay in getting back to you.  Unfortunately this hasn't fixed the problem.

I'm going to do a reproducing Plunker so you have something to work with.

thanks

I cannot reproduce this now in a plunker so closing.


Upgrading '@equinor/eds-core-react' peer dependency to ^0.35

Describe the solution you'd like
Currently most (or all?) of the packages have a peer dependency declared on @equinor/eds-core-react version ^0.34.0. The current latest version is 0.35.1 (https://github.com/equinor/design-system/releases). Using Node 18 we are unable to use the 0.35.x version of @equinor/eds-core-react alongside the fusion react components unless we install dependencies using the --legacy-peer-deps npm installation flag.
Can you increase or add the ^0.35.0 version to the peer dependency allowed versions?
Describe alternatives you've considered
Alternative would be to not use the latest 0.35.x version of @equinor/eds-core-react.

Copied to https://github.com/equinor/fusion/issues/258


Front to back connection

This pull request is updating the front with a fonction that is abble to fetch based on the JWT token
it will fectch data, if the JWT token is not on date anymore, it will make the refresh request to get a new one, then  it will try it again.
Tokens are stored in cookies now, as local storage is apparently not safe enough.
This commit is bypassing the 2FA security as the JWT token send by 2FA are not working or error is occuring in the backend.
Finally this commit starts to connect the dots, making a request in the loadpage function, and having some fetch function prepared for the apis

In this PR, the logic for friend is kinda done as well for the profile,
there's still some redirection issues where the pages is not load automatically but the adress changes,
to test it you can create multiple accounts via the front end and test to add them, view their profile and delete them, block them, or whatever ( as i said sometimes to go to a profile you have to load the address multiple times)
also when creating an account the error message might be displayed but the account is still created

changed !