Failure to list templates when using local CDN cache node

I configured a local CDN cache node and it seems to work great from the CLI to accelerated development. I was feeling uber nice about it. I even configured the agent.gcfg file to allow insecure CDN connections:
[CDN]
Allowinsecure = true
URL = cdn.subut.ai
SSLport = 8338
Kurjun =

Everything is dandy but I cannot see the list of templates in the Environment->Environments builder. I checked and there are several NPE exceptions occurring
karaflogs2017-11-27 07_13_39.txt
in the Karaf logs. I've attached them here:
Any idea what's going wrong from the log? Just in case here are the NGINX cache node access logs:
access.txt
The error logs are empty. I think this has something to do with the CDN self-signed keys.

Seems Kurjun supplies malformed JSON with missing names of templates.
I will add check against such cases

Ok thanks @Dilshat

Tested in dev and I still see the same problems with the template manager blowing NPEs:
karaflogs2017-11-29 05_57_18.txt
Here's my about screen and versions which do not look like dev at all. I think something is broken with devops. This looks like a production management:

This might explain why some of the version commands with CLI not working. Management must be trying to get them via the production subutai command and of course that does not exist but subutai-dev does:
subutai@subutai:~$ sudo subutai-dev list
CONT/TEMP
---------
management
openjre16
ubuntu16

Just in case I disabled my CDN node to directly hit devcdn:
subutai@subutai:~$ sudo subutai-dev destroy management
INFO[2017-11-29 00:08:11] management is destroyed
subutai@subutai:~$ ping devcdn.subut.ai
PING i-08c7895c5d5cc19a4.devcdn.subut.ai (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from ec2-35-158-125-126.eu-central-1.compute.amazonaws.com (<IP_ADDRESS>): icmp_seq=1 ttl=63 time=94.6 ms
^C
--- i-08c7895c5d5cc19a4.devcdn.subut.ai ping statistics ---
2 packets transmitted, 1 received, 50% packet loss, time 1001ms
rtt min/avg/max/mdev = 94.675/94.675/94.675/0.000 ms
subutai@subutai:~$ sudo subutai-dev import management
INFO[2017-11-29 00:09:07] Importing management
INFO[2017-11-29 00:09:08] Version: 6.2.3-dev
INFO[2017-11-29 00:09:09] Template's owner signature verified
INFO[2017-11-29 00:09:09] Downloading management
 24.20 MiB / 121.68 MiB [========>--------------------------------]  19.89% 1m3s

NOTE: The version INFO[2017-11-29 00:09:08] Version: 6.2.3-dev.
Conclusion, my local CDN is misconfigured most likely and is going to cdn.subut.ai instead of going to devcdn.subut.ai. Just to confirm here's the management about screen after the re-import:

@Dilshat this was my fault and I hope it did not make you change code unnecessarily. Is the change you made still valid or should we roll it out.
Please disregard my last message!

Ok here's where I am now. I think I fixed my CDN by adding the other two dev and master cdn server blocks. They were never there it seems. Now with it fixed I'm getting "better" results but something is still wrong. Here's the about screenshot for console and it's looking good minus a couple fields:

I still cannot see templates and the NPE is still continuing. It looks like I fix the CDN issues but you never know. Here's my local CDN configuration where I use separate self-signed keys for dev, master, and production with proper FQDN i.e. devcdn.subut.ai:
default.txt
Here's the Karaf log:
karaflogs2017-11-29 06_49_11.txt

One last thing. I just tried master now too. The problem looks exactly the same.

@Dilshat want me to test these changes?

@akarasulu yes please

@Dilshat I did not want to needlessly tire you out with this because I'm still concerned with the correctness of my local CDN cache node's NGINX configuration. @tilekasankulov @etursunbaev please privately provide the configurations I had asked for.


{networkfabric} HTML tags are rendering as empty parameter values in reference docs

Describe the bug
Problem to solve:
HTML tags in source code are causing build warnings in MicrosoftDocs/azure-docs-cli. More importantly, customer-facing reference docs are not instantiating properly -- parameter values formatted with `<>` are missing. Here's an example:

Requested solution:
These values should be escaped with a backtick. Here's an example from az webpubsub.

Related GitHub issues:
This is the same issue as these GitHub issues.
List of HTML tag instances that need update:

File name
Build error

latest/docs-ref-autogen/networkfabric/device.yml
HTML tag 'Fabric' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

latest/docs-ref-autogen/networkfabric/device.yml
HTML tag 'Rack' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

latest/docs-ref-autogen/networkfabric/device.yml
HTML tag 'Device' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

Related command
az networkfabric
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Azure CLI 2.62.0
Additional context
No response

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


Build Issue

Hi team,
I am building this project on Ubuntu-18.04. After "./build_and_install_llvm9.sh", where can I find "SanRazor-clang"? Thanks!
Best,
Dennis

Hi, it is in llvm/build/bin.

Hi team,
Thanks for replying! I have fixed that issue above. However, when I tried to run SanRazor with CPU2006, it always gives me build error. Here is the log file. Could you please take a look? Thank you so much.
Best,
Dennis
build_log.txt

Hi Dennis,
It seems that you did not install parallel, since the error said: cannot load such file -- parallel ... from /home/dennis/SanRazor/llvm/build/bin//SanRazor-clang:24:in '`. Can you check this?
Best wishes,
Jiang


Error: ssl_ca_certs is an invalid option in database URL

Bug description
When using a prisma client in a AWS context with Document DB (Essentially MongoDB), the connection url needs to have ssl=true&ssl_ca_certs=rds-combined-ca-bundle.pem in it. The connection requires a pem file to work. But when using ssl_ca_certs in a url string following error occurs:
ssl_ca_certs is an invalid option in database URL.
The full error is:
"Runtime.UnhandledPromiseRejection","errorMessage":"Error: The provided database string is invalid. MongoDB connection string error: ssl_ca_certs is an invalid option in database URL. Please refer to the documentation in https://www.prisma.io/docs/reference/database-reference/connection-urls for constructing a correct connection string. In some cases, certain characters must be escaped. Please check the string for any illegal characters."

The connection string is of the form: mongodb://user:pass@clusteName-cluseterARN.docdb.amazonaws.com:27017/dbname?ssl=true&ssl_ca_certs=certFileName.pem&replicaSet=rs0&readPreference=secondaryPreferred&retryWrites=false
This works in the context of other services we have using the node mongo client directly (not through Prisma).
The error occurs in the context of using featherjs and a feathers-prisma adapter in my use case. This error has also been reported in the feathers-prisma repository as it is manifest in the adapter for prisma. But. I don't believe this is a problem with feathersjs or the feathers-prisma adapter as the PrismaClient is passed to the adapter directly by feathers. I believe it could be a problem with the prisma client used in the context of AWS Document DB.
How to reproduce
Connect to a MongoDB database in AWS that requires a pem file and the ssl_ca_certs flag in the connection string.
Expected behavior
Prisma connects to the database when given a connection string with a ssl_ca_certs flag.
Prisma information
generator client {
  provider        = "prisma-client-js"
  output          = "../client"
  binaryTargets   = ["native", "darwin", "rhel-openssl-1.0.x", "rhel-openssl-1.1.x"]
}
datasource db {
  provider = "mongodb"
  url      = env("DATABASE_URL")
}
...

// Add your code using Prisma Client
import { PrismaClient } from "../client";
import { Application } from "./declarations";

export default function (app: Application): void {
  const connectionString = app.get("database");
  const prisma = new PrismaClient({
    datasources: {
      db: {
        url: connectionString
      }
    }
  });
  prisma.$connect();
  app.set("prisma", prisma);
}

Environment & setup

OS: AWS lambda
Database: MongoDB / AWS Document DB
Node.js version: 16

Prisma Version
Prisma 4.8.1

Our MongoDB support uses an official MongoDB Rust Client, which throws this error message. All connection string parameters on https://www.mongodb.com/docs/manual/reference/connection-string/ should be supported, and it seems ssl_ca_certs is not one of them.
I'll reach out to MongoDB to understand this situation better.

According to this StackOverflow response, maybe tlsCAFile is to be used here?
https://stackoverflow.com/a/73430442
Or sslCA?
https://stackoverflow.com/a/71923860

Confirmed, the correct parameter to use here is tlsCAFile which is also documented in the list of connection string parameters at https://www.mongodb.com/docs/manual/reference/connection-string/#mongodb-urioption-urioption.tlsCAFile So when you enable tls instead of ssl you should be able to use that with the provided .pem file.
Please let me know if you can confirm this works.

Sorry for the very late reply: using tlsCAFile instead of ssl_ca_certs works.

getting the below error while trying to establish a celery queue using document db , can anyone please help me with is
Cannot connect to mongodb://cb_docdb:**@myhost.us-east-1.docdb.amazonaws.com:27017/mydb: Field 'size' is currently not supported, full error: {'ok': 0.0, 'code': 303, 'errmsg': "Field 'size' is currently not supported", 'operationTime': Timestamp(1702035015, 1)}.


The Recover Document (Woah!) pop-up still occurs despite the following

It stilll happens, even if my storage is mostly empty as well as after undoing to recover.
It happens while I select a portion of a graphic.

This is a very annoying pop-up, and I want to obliterate it.
This dialog has nothing to do with storage quota. It's concerned with RAM, not disk space.
It shows any time the canvas (ignoring the selection) becomes entirely transparent, because this is the only symptom that occurs when the browser decides to rudely toss out the canvas data when low on memory. This usually occurs after you've spent a lot of time and effort on a document, creating many undo states (each taking up memory), so it's important to preserve the document in the case that this happens.
 Handling that one case would be fairly easy, but there are more complicated scenarios, like if you select all the opaque pixels manually with the Select or Free-Form Select tool. Simply preventing the dialog from showing during a tool action would leave it open to false negatives if the browser clears the canvas forcibly while making a selection.
For more background info, see https://github.com/1j01/jspaint/issues/24
It's very possible there's something simple that I haven't thought of, to remove the need for this dialog.

I tried to make this dialog a little clearer and more compact in https://github.com/1j01/jspaint/commit/a0abbb3e367b864c0b157c1eefe6e7e1b4112849. I still want to get rid of it though.
Before:

After:

Before (Expanded):

After (Expanded):

Chrome supposedly added new events for canvas context loss/restoration that apply to non-WebGL contexts: contextlost and contextrestored.

More information on WhatWG wiki...
Browser support information

That wiki page mentions another feature:

A new context creation parameter named 'storage' is to be added. Possible values are 'persistent' and 'discardable'. Usage: canvas.getContext('2d', {storage: 'discardable'});

This also sounds like the most useful, if I can just tell it to be persistent. However...

The default is 'persistent', which provides backwards compatible behavior for application that expect canvas backing store content to never be lost.

So I don't know if these things will actually help in practice, but in theory they seem intended to help with this scenario.

so maybe that's why?

@MesaBlack I'm not sure I understand the question, but, I was noting some possible avenues to eliminating the need for the dialog, not causes of the issue.
I'm not working on this right now, but I came across some mention of these events and I wanted to remember them in the future if I go to work on this, since they would be useful to explore. The events could help by directly detecting the scenario where the browser clears the canvas to save on memory usage.
(Further note to self: probably need to feature-detect these events before relying on them.)

I elaborated a bit


How to  1)configure multiple transmitters 2) how to apply different roughness maps to the different materials ?

Dear Christian Schöffmann,
Thank you very much for your wonderful work!  May I ask you two questions?
Q1.  How to  configure multiple transmitters
I want to set up three transmitting antennas, and let them be arranged in a triangle, with two times the wavelength at the bottom and 0.5 times the wavelength at the height (see Figure 1). May I consult you on how to configure Vira?

Q2.  How to apply different roughness maps to the different materials
May I have a suggestion on 1) how to apply different roughness maps to the different materials and also 2) how to adjust the level of reflectivity which yield different reflection due to the implemented Phong model?
My scene settings:
One wall:
use the predefined material 'woodTexture' from the materials folder, choose the standard shade option and choose transparent in the rendering mode tab
Detected object:
use the predefined material 'fabricTexture' from the materials folder, chose the shader to be: RadarSimulation/SimpleSurfaceMetallicAndRoughnessSaparate
What I tried was (Figure 2) from the material of the wall (e.g. WoodTexture): in  ''Main Maps'', adjusted values of ''Metallic'' or ''Smoothness''.
But I found two problems:

No matter whether there are obstacles or not
No matter what material the obstacle is (wood, plastic ...)
The final data generated by radar is always the same.

I am really sorry for bothering you again. Your answer will be of great help to me！Thank you in advance!
Best
Jialu

Hi Jialu,
Sorry for my late response.
Regarding Q1:
If you wan to simulate multiple transmitter antennas, you would need to place multiple cameras in the world space.
Currently we assume that we emit radio waves from the camera location, as this is also obtained by on-chip FMCW radars.
As an alternative, you can emulate the behavior of multiple transmitter antennas with the radiation pattern mask. Since the radiated energy in space will be different for the three antennas, you can sequentially run the simulation to obtain the receiver responds.
Regarding Q2:
Thanks for notification. I now did some small modification to the code. You can now directly attach the roughness texture to the radar sensor controller, in order to adjust the reflection of transparent and metallic objects. Please be aware that we are still currently evaluating the different reflection phenomena of non-conductive objects, so this is still experimental.
Best regards,
Christian

I will further work on this to make it more generic for different transparent materials. Stay tuned!


Job Handle config mail

Laravel Version: 5.8.23
PHP-versie: 7.1.14
Database Driver & Version: MSQL 5.5.62-0ubuntu

Description:
When i restart the queu and test it, it works... but then after a day/hours or so. it goes back to my main config and doesnt apply /Config settings.. so my from email is always the ENV, but in this reason I do not want the ENV but an other email.
Steps To Reproduce:
JOB:
`class SendAweberTegelexpertMail implements ShouldQueue
{
use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
public $****

public function __construct($****)
{
    $this->**** = $****;
}

public function handle()
{
    $subject = $this->subject;
    $email   = $this->email;
    $message = $this->mess;

    $config = [
        'driver'     => 'smtp',
        'host'       => '****',
        'port'       => ***,
        'username'   => '****',
        'password'   => '***',
        'encryption' => 'tls',
    ];

    \Config::set('mail', $config);

            \Mail::send('mails.plain', ["html"=>$message,"name"=>'****'], function($message) use($subject, $email) {
                $message->from('****', '****');
                $message->replyTo('****', '****');
                $message->subject($****);
                $message->to($****);
            });
}`

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!


Filter unknown file types from the nix build.

If you have a socket-type file in the source repo, the nix build will
choke on it with an error like:
error: file '.../node-server.sock' has an unsupported type

Webpack creates temporary socket files, so if you're doing frontend work
this is a huge pain.

LGTM, but put the note in a comment in the source.

Done, thanks.

I have no idea why this is failing to make progress...

I think there's a general hydra problem. :-(

Everything  else went fine, I really don't know what's up here. Maybe it really is blowing hydra up somehow? What about trying just adding .socket files to .gitignore, then the existing filter should get them, right?

That's actually a very good idea. Wish I'd thought of it. 😂


Use radius to get points within a range

❓ Questions & Help
The radius function in pytorch_geometric.nn can connect nodes within a distance to center point.
Is there a way to get points that the distance to the center points is between (r1, r2) rather than (0, r).

You can do this in two steps:
edge_index = radius(x, r=r2, max_num_neighbors=...)
row, col = edge_index
dist = torch.norm(x[row] - x[col], dim=-1)
edge_index = edge_index[:, dist > r1]

Thank you for your reply, it works.
By the way, I want to know if I choose neighbors that the distance is over r1 from the group(r2) that the distance is bwlow r2 and the group(r2) have few points such as 32, is it possible that all sampling points will be removed, or it is something like uniform sampling?
If not, will it require lots of sampling points if I divide r2 into more small areas?
The x and x0 for radius is in CUDA enviroment.

radius is like uniform sampling, it does not sort points according to their distance (at least not on GPU).

That means I can slice the area use radius only 1 time.
Thank you so much.


Result differs from yargs

Hey!
I've built a Vue.js terminal emulator and switched recently from yargs to getops. Now a user of my library comes up with this issue: https://github.com/ndabAP/vue-command/issues/176
I thought I just propagate it and see if you can say something to it.

@ndabAP Hey! Hmm, it could be a bug in getopts, but I can't reproduce any issues with white space:
#!/usr/bin/env node

const getopts = require("getopts")
console.log(getopts(process.argv.slice(2)))

./example --foo="a b c" --bar "a b c"

{ _: [], foo: 'a b c', bar: 'a b c' }

Can you clarify the problem?

@ndabAP Hmm, here, I found that stdin is:
reverse "a b c"

And stdin.split(' ')[1] will not give you the rest of the string. Instead we need to do this:
const argument = stdin.split(' ').slice(1).join(' ')

or
const argument = stdin.split(' ').slice(1).join(' ').replace(/"/g, '')

(if you also want to get rid of "s.)
I tried it here and it works now! Please reopen if you need help with anything else. 👋

You are simply awesome!


[chip-test, clkmgr] chip_sw_clkmgr_external_clk_src_for_lc

Test point name
chip_sw_clkmgr_external_clk_src_for_lc
Host side component
Rust?
Opentitantool infrastructure implemented
Yes
Silicon Validation (SiVal)
Yes
Emulation targets

[ ] None
[ ] CW310
[ ] Hyperdebug + CW310

Contact person
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[ ] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[ ] Test added to dvsim nightly regression (and passing at time of checking)
[ ] For SiVal test cases, test is running relevant FPGA or silicon regression

This test is currently not part of the SiVal test plan:
https://github.com/lowRISC/opentitan/blob/aaec0de35ef5c43d8702dc69143c752de40a4b1b/hw/top_earlgrey/data/ip/chip_clkmgr_testplan.hjson#L141-L152


Replace useTypeScript by parser in options

In this PR, we change the option param 'useTypeScript' boolean by parser who is a enum and he can be flow or typescript

cc @Kerumen @oliviertassinari This is a PR for change useTypeScript by parser

cc @Kerumen

It's live.


implement cert_transfer (misnomer mTLS interface)

Issue
grafana-agent needs to be able to get and trust traefik's certificate.
Solution
Use the mutual_tls interface, which (besides what the - wrong - name suggests) helps transfer the Provider's certificate over relation data to the Requirers.
Possible TODOs

[ ] Ideally add an integration test with a Provider and test it (traefik will soon implement this in a parallel PR)

@lucabello what's the status on this work? The CI seems seriously stuck

This PR is dependent on and blocked by:

https://github.com/canonical/certificate-transfer-interface/issues/5
https://github.com/canonical/certificate-transfer-interface/issues/6

Otherwise, we can't save the certificate to file as we wouldn't know which relation the certificate comes from.


Update build.gradle

During the build of the apk file of my app, I always get this issue from pdfrx. Specifying this packaging option on the main project level does not solve the issue.
Execution failed for task ':pdfrx:mergeReleaseNativeLibs'.

A failure occurred while executing com.android.build.gradle.internal.tasks.MergeNativeLibsTask$MergeNativeLibsTaskWorkAction
2 files found with path 'lib/arm64-v8a/libpdfium.so' from inputs:
- /jenkins/workspace/head_amos_mobile_exec/amos_mobile_exec/build/pdfrx/intermediates/merged_jni_libs/release/out/arm64-v8a/libpdfium.so
- /jenkins/workspace/head_amos_mobile_exec/amos_mobile_exec/build/pdfrx/intermediates/cxx/RelWithDebInfo/6i5b2q3w/obj/arm64-v8a/libpdfium.so If you are using jniLibs and CMake IMPORTED targets, see https://developer.android.com/r/tools/jniLibs-vs-imported-targets

Yes, I knew the issue. This is not the first time someone suggests the change (like #8).
But it comes from the duplication of libpdfium.so with other plugins or such. So basically I think it's an app issue and we should write it on app's build.gradle.
Am I wrong?


Replaced Set objects internally to string[] (Set polyfills don't work well on all browsers)

Sorry for the delay on this. One question I have would be what browsers are aimed at here? The MDN Set page seems to indicate that it has pretty solid coverage across all browsers.

No problem for the delay. I saw that on MDN as well, but Safari and Firefox
had problems with at least the way we were using sets. Moving to lists
solved that. Generally shooting to work well on all but Edge/IE.
Octavian
On Mon, Jul 20, 2020 at 1:24 PM Matthew Peveler<EMAIL_ADDRESS>wrote:

Sorry for the delay on this. One question I have would be what browsers
are aimed at here? The MDN Set
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set
page seems to indicate that it has pretty solid coverage across all
browsers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cislrpi/ngx-storyline-viewer/pull/7#issuecomment-661215850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLHNPK6XQU5C5YT5Z57CV3R4R4WFANCNFSM4OEZPU7Q
.

--
Octavian Udrea

Merged, and released new version. I've also made you a contributor to this repository so you should be able to commit directly to master, cut new releases, etc.


Adds swift 3 compatibility

Thank you! 🚢

Guys, this code failed CI, is it really swift3 ready?

It failed CI because the CI runs on Xcode 7.x, which has no swift 3 support
Sent from my iPhone

On 21 Jun 2016, at 20:54, Tiago Ferreira<EMAIL_ADDRESS>wrote:
Guys, this code failed CI, is it really swift3 ready?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah cool, thanks!


Revert Style/ArgumentsForwarding Rubocop rule

@Rukomoynikov Looks like you fixed it! 👀 🎉

@Rukomoynikov Looks like you fixed it! 👀 🎉

Thank you, but I just slightly altered your solution.


Problem with live-edit deployment error pages

Beta & live-web use the static error page but live-edit doesn't. It uses the .net YSOD page instead. I believe this is an environment level issue.
A good way to test this is to fire up the website. The site throws a "no nodes" exception. The "no nodes" exception is default behaviour for Umbraco when no content has been authored.
This isn't really that important in the long run though, I just want to ensure that if we get a real exception on the site it will use the static error page ("error-page.html") rather than a .net YSOD and the "no nodes" exception is a good way to test we have parity between environments.

This is only an issue if the site is viewed from the hosting machine itself becasue the customErrors setting in web.config is set to remoteOnly. All servers display this same behaviour. It doesn't affect the end user experience and is useful for managment.


Exposing CLLocationManager.ShowsBackgroundLocationIndicator into ListenerSettings

Changes Proposed in this pull request:

Add a new parameter in ListenerSettings:

/// <summary>
 /// A Boolean indicating whether the status bar changes its appearance when an app uses location services in the background. (>= iOS 11). Default: false
/// </summary>
public bool ShowsBackgroundLocationIndicator { get; set; } = false;

Affect only iOS 11+ devices.
Official documentation by Apple

Lovely! Good one :)

Our users like the blue bar.  Thank you @nicolas-garcia


Explicitly set aurora version to 11.17

Trying to fix this error in deploy:
Error: Failed to modify RDS Cluster (tna-metadata-db-staging): InvalidParameterCombination: Cannot upgrade aurora-postgresql from 11.17 to 11.13
	status code: 400, request id: bd751f27-e503-4bbf-bf0a-7d677a311401

  with module.data.module.metadata-db.aws_rds_cluster.this[0],
  on .terraform/modules/data.metadata-db/main.tf line 50, in resource "aws_rds_cluster" "this":
  50: resource "aws_rds_cluster" "this" {

Upgrading to 13 gave the same error with "Cannot upgrade aurora-postgresql from 11.17 to 13.7"

This SO comment is concerning:
The AWS documentation is not as clear as it could be, but currently it is impossible to do a major version upgrade on Aurora e.g. you can upgrade from 9.6.9 to 9.6.12, but you cannot upgrade from 9.6.9 to 10.7 without dumping all of the data, creating a new 10.7 instance and restoring it there.

https://stackoverflow.com/questions/56425050/cannot-upgrade-aurora-postgresql-from-9-6-9-to-10-7

@anthonyhashemi this looks like a possibility -- can trigger an OS upgrade which might help. That documentation looks a little old, the situation might have improved.
https://github.com/terraform-aws-modules/terraform-aws-rds-aurora/issues/308#issuecomment-1257131131

Okay, let's see if the OS upgrade fixes things before going ahead with this PR?
Here are some extra discussions around this problem: https://stackoverflow.com/questions/69113494/can-not-upgrade-aurora-rds-postgres-version-from-12-to-13

Pressed the OS update button. Did nothing for a few minutes, then seems to have worked.


Indent is applied every time php-cs-fixer is run

Bug report
Each time that I run php-cs-fixer 3.9.1 3.9.2 or 3.9.3 it keeps adding more indent to an array structure like below.
rm .php-cs-fixer.cache
php -ee d zend.enable_gc=0 vendor-bin/owncloud-codestyle/vendor/bin/php-cs-fixer fix -vv --diff --allow-risky yes

Then commit the changes.
Then repeat again, the indent is increased, commit the changes.
Keep repeating - the indent keeps increasing.
php-cs-fixer itself is not happy with the indent fix that it just applied.
See demo PR https://github.com/owncloud/core/pull/40202 - in the 3rd, 4th and 5th commits the indent of a lot of places keeps increasing.
This makes it impossible to commit code that php-cs-fixer is happy with and will pass CI. When the CI pipeline runs code-style checks, it wants to always add another indent.
Code snippet that reproduces the problem
  public function deleteKey($app, $key) {
  	$this->emittingCall(function () use (&$app, &$key) {
  		$this->loadConfigValues();

  		$sql = $this->conn->getQueryBuilder();
  		$sql->delete('appconfig')
  			->where($sql->expr()->eq('appid', $sql->createParameter('app')))
  			->andWhere($sql->expr()->eq('configkey', $sql->createParameter('configkey')))
  			->setParameter('app', $app)
  			->setParameter('configkey', $key);
  		$sql->execute();

  		unset($this->cache[$app][$key]);
  		return true;
  		}, [
  						'before' => ['app' => $app, 'key' => $key],
  						'after' => ['app' => $app, 'key' => $key]
  					], 'appconfig', 'deletevalue');
  }

The lines down the end keep getting more indent added:
							'before' => ['app' => $app, 'key' => $key],
							'after' => ['app' => $app, 'key' => $key]
						], 'appconfig', 'deletevalue');

Thanks. This seems to be fixed by #6480, can you give it a try?

Thanks. This seems to be fixed by #6480, can you give it a try?

will do - it's late here so I will play with it tomorrow

There are still cases where the indent is constantly increased on every run of php-cs-fixer. See each commit in demo PR https://github.com/owncloud/core/pull/40211
https://github.com/owncloud/core/pull/40211/commits/8f4a7ae580d32f14f1eefe59d08144b743f789de
https://github.com/owncloud/core/pull/40211/commits/c914206663239ae28b8830f862ec7f944fa132e7
https://github.com/owncloud/core/pull/40211/commits/e2f2b1fe80508f16e67853fcfbdced4b72f4584e
https://github.com/owncloud/core/pull/40211/commits/a04b87cada26afd1e824feac383c7b84b0210a67

I can confirm the issue with PHPCSFixer 3.9.5 and the following config:
return (new PhpCsFixer\Config())
    ->setUsingCache(true)
    ->setCacheFile(__DIR__ . '/.php-cs-fixer.cache')
    ->setRules([
        '@PSR12' => true,
        '@Symfony' => true,
        '@DoctrineAnnotation' => true,
        '@PHP80Migration:risky' => true,
        '@PHP81Migration' => true,
        'align_multiline_comment' => ['comment_type' => 'phpdocs_like'],
        'concat_space' => ['spacing' => 'one'],
        'combine_nested_dirname' => true,
        'dir_constant' => true,
        'echo_tag_syntax' => ['format' => 'long'],
        'function_to_constant' => true,
        'heredoc_indentation' => ['indentation' => 'same_as_start'],
        'is_null' => true,
        'linebreak_after_opening_tag' => true,
        'no_alternative_syntax' => false,
        'no_superfluous_elseif' => true,
        'no_useless_else' => true,
        'no_useless_return' => true,
        'non_printable_character' => true,
        'nullable_type_declaration_for_default_null_value' => true,
        'phpdoc_align' => ['align' => 'left'],
        'phpdoc_order' => true,
        'phpdoc_separation' => false,
        'ternary_to_null_coalescing' => true,
    ])
    ->setRiskyAllowed(true)
    ->setFinder($finder);

Since this issue seems to only occur with files that mix PHP and HTML now, I'm going to close here and keep #6521 to track it.


How to create custom states apart from user and conversation states in botframework.

I want to store some model data to state apart from user and conversation states. Is it possible to create another state?

@samthrusha123 - Thanks for reporting your issue.
We will investigate this issue and get back to you.

@samthrusha123 -State management automates the reading and writing of your bot's state to the underlying storage layer.
By default, Bot Framework uses the Bot Framework State to store conversation data.

User state
Conversation state
Private conversation state

If you want directly store data without middleware, you can directly from storage instead of using a state manager.
You can read and write directly to your storage object without using middleware or context object. This can be appropriate for data your bot uses to preserve a conversation, or data that comes from a source outside your bot's conversation flow. In this data storage model, data is read in directly from storage instead of using a state manager.
Reference Document-https://learn.microsoft.com/en-us/azure/bot-service/bot-builder-howto-v4-storage?view=azure-bot-service-4.0&tabs=csharp#set-up-a-cosmos-db-resource

@samthrusha123 - Could you please confirm if your issue has resolved with above suggestion or still looking for any help?


Synchronous flows based on the Webhook source

I've been working on the various demo integration flows that involved the Synchronizer component; these flows are meant to return data in responses back to the client. Our Webhook source was chosen as the simple event gateway to pass the client requests into the system. After some tests, I've figured out that the source does not return any data, only statuses.
Since this component is placed on the border between the regular HTTP client systems and CloudEvents based platforms, can we consider extending its behavior a bit out of CE framework boundaries and making it handle data as well as statuses? Otherwise, having HTTP webhooks that are limited to status responses only is a strong limiting factor, isn't it?

@sebgoa and people who follow that issue: since we are currently discussing this topic at a larger scale, would it be fair to close this issue?


Support pluggable ingress resources

To more natively support cloud provider based ingress, e.g. AWS ALB or GCP L7 LB.

This is now done.


[Bug] Unable to launch json-server along with the app

Pulled the latest version from your repo and installed json-server using npm install json-server. My intention is to launch json-server along with the app. The relevant lines of code for it goes to src/desktop.ts src/main.browser.ts src/main.browser.aot.ts and are executed just before the main bootstraper:
const jsonServer = require('json-server');
const server = jsonServer.create();
const router = jsonServer.router('db.json');
const middlewares = jsonServer.defaults();

server.use(middlewares);
server.use(router);
server.listen(3001, function () {
  console.log('JSON Server is running');
});

The problem is that does not work.
In the webpack's output I am getting the following errors:
ERROR in ./~/json-server/lib/server/defaults.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\json-server\lib\server'
 @ ./~/json-server/lib/server/defaults.js 3:9-22
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/errorhandler/index.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\errorhandler'
 @ ./~/errorhandler/index.js 19:9-22
 @ ./~/json-server/lib/server/defaults.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/lodash-id/src/node.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\lodash-id\src'
 @ ./~/lodash-id/src/node.js 1:9-22
 @ ./~/json-server/lib/server/router/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/express/lib/request.js
Module not found:ERROR: Can't resolve 'net' in 'C:\Ekopoint\ng2-admin\node_modules\express\lib'
 @ ./~/express/lib/request.js 18:11-25
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/express/lib/view.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\express\lib'
 @ ./~/express/lib/view.js 18:9-22
 @ ./~/express/lib/application.js
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/send/index.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\send'
 @ ./~/send/index.js 24:9-22
 @ ./~/express/lib/response.js
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/graceful-fs/graceful-fs.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\graceful-fs'
 @ ./~/graceful-fs/graceful-fs.js 1:9-22
 @ ./~/steno/index.js
 @ ./~/lowdb/lib/storages/file-async.js
 @ ./~/json-server/lib/server/router/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/etag/index.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\etag'
 @ ./~/etag/index.js 22:12-25
 @ ./~/express/lib/utils.js
 @ ./~/express/lib/application.js
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/graceful-fs/fs.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\graceful-fs'
 @ ./~/graceful-fs/fs.js 3:9-22
 @ ./~/graceful-fs/graceful-fs.js
 @ ./~/steno/index.js
 @ ./~/lowdb/lib/storages/file-async.js
 @ ./~/json-server/lib/server/router/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/destroy/index.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\destroy'
 @ ./~/destroy/index.js 14:17-30
 @ ./~/send/index.js
 @ ./~/express/lib/response.js
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/mime/mime.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\mime'
 @ ./~/mime/mime.js 2:9-22
 @ ./~/send/index.js
 @ ./~/express/lib/response.js
 @ ./~/express/lib/express.js
 @ ./~/express/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

ERROR in ./~/jju/lib/utils.js
Module not found:ERROR: Can't resolve 'fs' in 'C:\Ekopoint\ng2-admin\node_modules\jju\lib'
 @ ./~/jju/lib/utils.js 1:10-23
 @ ./~/jju/index.js
 @ ./~/json-parse-helpfulerror/index.js
 @ ./~/lowdb/lib/storages/_json.js
 @ ./~/lowdb/lib/storages/file-async.js
 @ ./~/json-server/lib/server/router/index.js
 @ ./~/json-server/lib/server/index.js
 @ ./src/main.browser.ts
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts

and warnings:
WARNING in ./~/express/lib/view.js
80:29-41 Critical dependency: the request of a dependency is an expression

WARNING in ./~/jju/lib/utils.js
8:2-3 Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Targeting node in webpack config results in the absence of the errors above, but the warnings are still there and json-server still doesn't start.

Hey @Deilan, you need a separate backend app to launch the json-server.

@nnixaa Thanks for the response? Could you kindly explain why I am ultimately need a separate app and can't launch it along with this app?


rubocop-ast-0.4.0 has broken Layout/LineLength

With the latest version of the gem I receive the following error when executing rubocop:
An error occurred while Layout/LineLength cop was inspecting /home/jonny/Programming/myapp/app/workers/bot.rb:330:8.

confirming! Also see https://github.com/rubocop-hq/rubocop/issues/8713

@jonnynux I had to update rubocop to 0.90.0. That fixes this problem.

Yes, it works great.

Sorry for this. Yes, 0.4.0 is compatible with 0.89.0+ only.
Let's keep this issue open, I should introduce a way for the main gem to know this and at least warn...


v7 default sort

Hello,
How to default sort on v7 ?

useTableState allows you to do this:
const tableState = useTableState({
  sortBy: [{
    id: 'firstName',
    desc: true,
  }]
})

const instance = useTable({
  state: tableState
})

That was fast ! Great thanks
On Fri, Aug 16, 2019 at 5:02 PM Tanner Linsley<EMAIL_ADDRESS>wrote:

useTableState allows you to do this:
const tableState = useTableState({
sortBy: [{
id: 'firstName',
desc: true,
}]
})
const instance = useTable({
state: tableState
})
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/1456?email_source=notifications&email_token=ABPAMXLA53H3FAUR3OCIXDTQE26QFA5CNFSM4IMKB6Y2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4O3N7Q#issuecomment-522041086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPAMXMDDIP7ZQ6REJJS6OLQE26QFANCNFSM4IMKB6YQ
.

--
Jean Chapelle

useTableState allows you to do this:
const tableState = useTableState({
  sortBy: [{
    id: 'firstName',
    desc: true,
  }]
})

const instance = useTable({
  state: tableState
})

Looks like useTableState is deprecated in newest API. How can I use sortBy function to do default sorting now? Thanks!

initialState.sortBy = [{id: “firtsName”}]
Tanner Linsley
On Oct 25, 2019, 10:03 AM +0200, Chun-Lin Wu<EMAIL_ADDRESS>wrote:

useTableState allows you to do this:
const tableState = useTableState({
sortBy: [{
id: 'firstName',
desc: true,
}]
})
const instance = useTable({
state: tableState
})
Looks like useTableState is deprecated in newest API. How can I use sortBy function to do default sorting now? Thanks!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

For anyone else that comes looking, here is a working example: https://codesandbox.io/s/tannerlinsleyreact-table-default-sorting-6i6xn
(Note that there is no use of a useTableState plugin; I could not find one, and it doesn't appear to be necessary anyway.)


Method not called on instance creation when registration configured via XML

In XML file method call is configured on type registration. But the method is not called on resolving an instance. Here are samples of configuration and code.
Tested on 5.11.6
  <unity xmlns="http://schemas.microsoft.com/practices/2010/unity">
    <reginster type="TestClass">
      <method name="TestMethod" />
    </reginster>
  </unity>

    class Program
    {
        static void Main(string[] args)
        {
            var unity = new UnityContainer();

            var t = unity.Resolve<TestClass>();

            Console.ReadLine();
        }
    }

    public class TestClass
    {
        public TestClass()
        {
            Console.WriteLine("constructor");
        }

        public void TestMethod()
        {
            Console.WriteLine("method");
        }
    }

All of these supposedly work. Did you configure the section?

Made mistakes in configuration. My bad. Thanks.


What are 'b' and 'pricing function' arguments for numerical greeks

What are 'b' and 'pricing function' arguments for numerical greeks and how do you pass them as arguments? My understanding is you should be using numerical_delta() instead of analytical delta function for faster, but slightly less accurate results.
from py_vollib.helpers.numerical_greeks import delta as numerical_delta
from py_vollib.black_scholes.greeks.analytical import delta

S = 49
K = 50
r = .05
t = 0.3846
sigma = 0.2
flag = 'c'

delta(flag, S, K, t, r, sigma)
numerical_delta(flag, S, K, t, r, sigma)

Results
>>> delta(flag, S, K, t, r, sigma)
0.5216016339715761
>>> numerical_delta(flag, S, K, t, r, sigma)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: delta() missing 2 required positional arguments: 'b' and 'pricing_function'

The py_vollib.helpers module is not intended to be called directly.  (Which is why it is undocumented.)
Instead use
from py_vollib.black_scholes.greeks.numerical import delta as numerical_delta and all will be fine.  The b argument is due to that inside helpers we are using a generalized form (borrowed from Espen Haug) that accepts black-scholes-merton, and other option pricing formulae.


Epic: User Validation for UX & UI Re-Design

Description
This epic aims to validate the implementation of the newly designed UX/UI changes by testing the actual product and gathering feedback from users. This will involve user acceptance testing and implementing changes based on user feedback.
Tasks

Develop a user acceptance testing plan that outlines the testing criteria, including who the testers will be, what scenarios will be tested, and how feedback will be collected.
Conduct user acceptance testing, including gathering feedback on the implemented interface and user flow.
Analyze user feedback and identify areas for improvement.
Make changes to the implementation based on user feedback, implementing design changes where necessary.
Conduct a second round of user acceptance testing to ensure the changes have improved the overall user experience.

Acceptance Criteria

User acceptance testing plan developed and testing conducted.
User feedback gathered and analyzed.
Implementation changes made based on user feedback.
Second round of user acceptance testing was conducted to validate changes.

Changed the title and some details based on your feedback of naming it more around exploration than UI/UX validation @abarciauskas-bgse .


feat: Add TLS connection support for outbound PG

Underneath it conditionally uses NoTls when sslmode=disable and
sets up tls connector otherwise.
NOTE:
By default, tokio_postgres is using sslmode=prefer, so to
avoid SSL setup overhead sslmode=disable should be provided
explicitly.
Refs: https://github.com/fermyon/spin/issues/936
Signed-off-by: Konstantin Shabanov<EMAIL_ADDRESS>

@lann for 👀 on the TLS side of things - I know this has been a thorn in our side elsewhere

Thanks @etehtsea. What scenario/example did you use to manually test this PR? Were you able to get it working with neodb?

I just tested locally; appears to work mostly as expected. The one hiccup is that sslmode=prefer seems to behave the same as sslmode=require

Thanks @etehtsea. What scenario/example did you use to manually test this PR? Were you able to get it working with neodb?

Yep, I've tested it against free neon instance


refactor(transformer_conformance): skip plugins we don't support yet

We don't need PLUGINS_NOT_SUPPORTED_YET. The PLUGINS are the plugins we supported.

#2988  👈
main

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

Join @Dunqing and the rest of your teammates on  Graphite

I think it's nicer to know which plugins we should work on next. I'll close this.


../configure fails

What operating system (Linux, Windows, …) and version?
MacOS 10.14.5
What did you do?
I cloned the repo using :
$ git clone https://github.com/Juniper/grpc-c.git
I installed required dependencies as described in the gRPC docs.
Then I did
$ cd grpc-c
$ autoreconf --install
$ git submodule update --init
$ ./builddeps.sh
then -
$mkdir build && cd build
$ ../configure
Error!
What did you expect to see?
Something beautiful!
I expected to see the grpc-c compile nicely.
What did you see instead?
Here, I get the following error :
checking for stdio.h... yes
checking for stdlib.h... (cached) yes
checking for string.h... (cached) yes
checking sys/param.h usability... yes
checking sys/param.h presence... yes
checking for sys/param.h... yes
checking for unistd.h... (cached) yes
checking grpc/grpc.h usability... no
checking grpc/grpc.h presence... no
checking for grpc/grpc.h... no
configure: error: Could not find grpc.h anywhere, install grpc
I have installed grpc.

Issue caused due to unused variable warning - error - line 57 in create_channel.cc
As a result grpc build failed, but didn't prevent protobufs build, so I didn't realize that it had failed for a while.
Commented out the two lines: line 56 and 57.
//  internal::GrpcLibrary
//      init_lib;  // We need to call init in case of a bad creds.


use url.URL in http and https plugins

The current http/https plugins use the global URL which comes from the dom lib. All instances of URL should be replaced with url.URL from the standard library.

🙋♂


Add module name override support

Add module name override support
Clean up tests
/// @mockable(module: name = Foo)
protocol Bar {
}
will now generate
class Foo.BarMock: Foo.Bar {
}

Thanks @elsh for putting it together, I will try to test it a bit more later today. One thing I've noticed, is that the module prefix is applied to both, the mock and the protocol type.
I might be wrong, but I think the module prefix cannot be applied to the type definition, also, having it only as the protocol prefix make it more flexible - the mock can be included in the test target, or pulled aside and stored in a separated module (I prefer to store them in a dedicated module to make them reusable, and leveraged test modules).

You're right; updated pr


hashes: remove version pin on schemars.

This pin was implemented incorrectly, and anyway appears to be completely unnecessary on 1.41.
Fixes #1687

Closing in favor of #1696. As usual, after 5 minutes poking at schemars, it's broken in ways I'm no longer interested in trying to fix.


inorder, preorder, postorder traversal

Preorder traversal visits the root node first, then recursively traverses the left subtree, followed by the right subtree.
Inorder traversal visits the left subtree first, then the root node, followed by the right subtree.
Postorder traversal visits the left subtree first, then the right subtree, followed by the root node.
I would like to solve all traversal in c++. Please assign this to me.

refer #177


Moving from 2.0.0-alpha.0 to 2.0.0 causes 2nd property of a component to be undefined

Hello! Just spotted this error and was wondering if you had any suggestions to fix this? (Or if its a bug)
Using the component like so:
<collaborate-sidebar-section customer-id={'customerId'} call-id={'callId'} />
would lead to this error "Warning: Failed prop type: The prop call-id is marked as required in StyledCollaborateSidebarSectionWrapper, but its value is undefined"
When swapping the order of the properties in the component like this:
<collaborate-sidebar-section call-id={'callId'} customer-id={'customerId'} />
customer-id would then be undefined.
For now I've reverted back to 2.0.0-alpha.0. This error occurred in chrome v114.0.5735.106 with a react web component create from react v17.0.2. If you need anything else please let me know!
Joe

I've just done a little testing and it seems to be 2.0.0-alpha.5 where I encounter the issue, 2.0.0-alpha.4 works as expected!

That is not expected behaviour, and I'm not sure what would be causing it. Do you also experience this with @r2wc/react-to-web-component@2?

@christopherjbaker I'm still having this issue personally and I'm using v2

Hello! Sorry for the late response! As we are using react 17 @r2wc/react-to-web-component v2 does not work with our app, but when I tried v1.0.0 I get the same issue. The changes I see between 2.0.0-alpha.4 -> 2.0.0-alpha.5 are here https://github.com/bitovi/react-to-web-component/commit/4b323ce04813773fdaf387ae3924b5b589f701dc
is transform?.parse not being defined?
We are planning on moving to react 18 so could see if this issue persists afterwards (this will be a little while away though)

Would you be able to put together a minimal use case of this? I'm not sure what would cause it and am unable to reproduce.

We have moved over to react 18 and it works as expected! So it looks like react 17 was the problem


Heart disease prediction

This is a project the predicts whether the person  have a risk of  heart disease or not .

Thanks for your contribution ❤


not working on IE11

the text labels not working in IE11

is there a solution for that ?

The issue here is that i use divs in foreign objects to hold the text, see:
https://github.com/RobinL/d3_orgchart_yammer/blob/master/website/js/main.js#L204
I do this because it enables text wrapping, which is quite hard to achieve in standard svg text elements.
If you want to modify, you just need to alter it so that rather than using divs, it uses text elements, and you'll just have to put up with no text wrapping, or implement this yourself.

ok i tried this to fix ... but it wont work ...
can u show me a code snippet how it works ...
i know the script append a foreign-object ... that will not work in IE ... but how can i replace it to a text ?


Properly unmarshal null DeepCachingType value as "NEVER"

If the client received DeliveryService JSON that contained a null value in deepCachingType, it was unable to unmarshal the value properly. In this case, null should be unmarshalled as the default "NEVER".

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1423/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1424/
Test FAILed.


Enhancement: Add the ability to set the Combo Camera to the Hud camera, and allow the ability to change the position where the combo is located, as well as the ability to add additional keybinds to arrows.

What is your suggestion, and why should it be implemented?
Basically, make an option to set the camera of the combo to the hud camera, because it would allow for the combo to be more easily seen, as well as allow for the ability to change the combo's location on the hud, because it would be helpful to change it's position if you don't want it in a certain place.

Also, just to clarify, I think it should be optional because some like to have it set to the game camera, because that's closer to how the og game worked.

In addition, I think the ability to add additional binds to arrows would be helpful. As an example, some people like to have multiple binds to certain arrows (such as have period and numpad2 binded to the up arrow.)


fix #24 - edit mumble endpoint

Describe your changes. Be sure to tag your assigned issue:

I worked on the edit mumble feature because there was no feature for making changes in the mumble.

Type of change :

[x] New feature

Preview (Screenshots) :

If it is possible, please link screenshots of your changes preview !

Checklist:

[x] I have read the Code Of Conduct document.
[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own
[x] I have added two reviewers to my PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] All new and existing tests passed.

@abhivemp Let me know if you saw any issue in there !


Allow override of server.basePath with environment variable

This is a much used setting. Since a lot of people (including myself) prefer to use the official docker from docker hub, I propose to make it an environment variable alike ELASTICSEARCH_URL.

Can this get merged? Thanks!

I would echo @yosifkit's comment from https://github.com/docker-library/kibana/pull/30#issuecomment-222842591:

I am usually -1 on adding more environment variables since that can lead to a slippery slope of adding more and more variables to be able to customize/generate a config file.
We kept ELASTICSEARCH_URL for backwards compatibility.  We now only do the sed when ELASTICSEARCH_URL is set, so it would be fairly easy to use your own config file by mounting it (-v /local/kibana/config/:/opt/kibana/config/:ro) with any configuration that your situation requires.

Closing feature requests, since this image is being deprecated.

This image is officially deprecated in favor of the kibana image provided by elastic.co which is available to pull via docker.elastic.co/kibana/kibana:[version] like 5.2.1. This image will receive no further updates after 2017-06-20 (June 20, 2017). Please adjust your usage accordingly.
Elastic provides open-source support for Kibana via the elastic/kibana GitHub repository and the Docker image via the elastic/kibana-docker GitHub repository, as well as community support via its forums.


Finalize UriHandler API

I've made the following changes as discussed in the API call:

Improve documentation
Rename to UriHandler
handleUri now returns ProviderResult<void> and we log errors, if any

Pushed updates.

Let's try to remove branding from the documentation.

test

another comment

what

@jrieken Removed product specific language from the docs. Up for final review.

Seems to have an accidental change but apart from that 🚢 it.

Good catch. I used the git merge ui for the first time. 🤦♂


Fix automatic releases / snapshot releases via travis

This is a blocker!
All the last builds on travis failed in the release stage with the script: ci/on-master.py ci/release.sh. The reason is a timeout error when communicating with Sonatype OSSRH (Maven Central). Although, it could be some permission issue which is hidden to users without these permission, chances are high, releasing takes simply too long.
[1235/1237] scalanativelib.worker[0.3].pom
ill.scalalib.PublishModule.publishAll
1 targets failed
mill.scalalib.PublishModule.publishAll java.net.SocketTimeoutException: Read timed out
    sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
    sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
    sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
    java.lang.reflect.Constructor.newInstance(Constructor.java:423)
    sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1944)
    sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1939)
    java.security.AccessController.doPrivileged(Native Method)
    sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1938)
    sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1508)
    sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
    java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
    sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:347)
    scalaj.http.HttpRequest.doConnection(Http.scala:367)
    scalaj.http.HttpRequest.exec(Http.scala:343)
    scalaj.http.HttpRequest.asString(Http.scala:491)
    mill.scalalib.publish.SonatypeHttpApi.withRetry(SonatypeHttpApi.scala:121)
    mill.scalalib.publish.SonatypeHttpApi.createStagingRepo(SonatypeHttpApi.scala:60)
    mill.scalalib.publish.SonatypePublisher.publishRelease(SonatypePublisher.scala:77)
    mill.scalalib.publish.SonatypePublisher.$anonfun$publishAll$11(SonatypePublisher.scala:55)
    mill.scalalib.publish.SonatypePublisher.$anonfun$publishAll$11$adapted(SonatypePublisher.scala:54)
    scala.collection.TraversableLike$WithFilter.$anonfun$foreach$1(TraversableLike.scala:789)
    scala.collection.immutable.Map$Map1.foreach(Map.scala:125)
    scala.collection.TraversableLike$WithFilter.foreach(TraversableLike.scala:788)
    mill.scalalib.publish.SonatypePublisher.publishAll(SonatypePublisher.scala:54)
    mill.scalalib.PublishModule$.$anonfun$publishAll$2(PublishModule.scala:118)
    scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
java.net.SocketTimeoutException: Read timed out
    java.net.SocketInputStream.socketRead0(Native Method)
    java.net.SocketInputStream.socketRead(SocketInputStream.java:116)
    java.net.SocketInputStream.read(SocketInputStream.java:171)
    java.net.SocketInputStream.read(SocketInputStream.java:141)
    sun.security.ssl.InputRecord.readFully(InputRecord.java:465)
    sun.security.ssl.InputRecord.read(InputRecord.java:503)
    sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:983)
    sun.security.ssl.SSLSocketImpl.readDataRecord(SSLSocketImpl.java:940)
    sun.security.ssl.AppInputStream.read(AppInputStream.java:105)
    java.io.BufferedInputStream.fill(BufferedInputStream.java:246)
    java.io.BufferedInputStream.read1(BufferedInputStream.java:286)
    java.io.BufferedInputStream.read(BufferedInputStream.java:345)
    sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:735)
    sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:678)
    sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1587)
    sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
    sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:263)
    scalaj.http.HttpRequest.doConnection(Http.scala:365)
    scalaj.http.HttpRequest.exec(Http.scala:343)
    scalaj.http.HttpRequest.asString(Http.scala:491)
    mill.scalalib.publish.SonatypeHttpApi.withRetry(SonatypeHttpApi.scala:121)
    mill.scalalib.publish.SonatypeHttpApi.createStagingRepo(SonatypeHttpApi.scala:60)
    mill.scalalib.publish.SonatypePublisher.publishRelease(SonatypePublisher.scala:77)
    mill.scalalib.publish.SonatypePublisher.$anonfun$publishAll$11(SonatypePublisher.scala:55)
    mill.scalalib.publish.SonatypePublisher.$anonfun$publishAll$11$adapted(SonatypePublisher.scala:54)
    scala.collection.TraversableLike$WithFilter.$anonfun$foreach$1(TraversableLike.scala:789)
    scala.collection.immutable.Map$Map1.foreach(Map.scala:125)
    scala.collection.TraversableLike$WithFilter.foreach(TraversableLike.scala:788)
    mill.scalalib.publish.SonatypePublisher.publishAll(SonatypePublisher.scala:54)
    mill.scalalib.PublishModule$.$anonfun$publishAll$2(PublishModule.scala:118)
    scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
Traceback (most recent call last):
  File "ci/on-master.py", line 10, in <module>
    subprocess.check_call(sys.argv[1:])
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['ci/release.sh']' returned non-zero exit status 1
The command "$CI_SCRIPT" exited with 1.

Sonatype OSSRH is known to be not very fast, and the count of mill release artifacts increased in the past due to new contrib packages. Currently mill tries to stage and release all packages in one go. A possible solution might be to split that step. An indicator, that that might be the right move is the fact, that the Sonatype/Nexus stage/release API can be and IS already used that way, e.g. by Maven (nexus-staging-maven-plugin) or sbt (sbt-sonatype).
This is related to #543.

@lefou @alexarchambault is seeing a similar error when trying to publish Ammonite:

https://github.com/lihaoyi/Ammonite/pull/946#issuecomment-477992302

Nothing changed on the Ammonite/Mill build - the Mill version didn't even change - so It could be that it's just that Sonoatype's servers are being sad.

Or maybe related to the recent Travis CI global outage https://www.reddit.com/r/programming/comments/b6l559/travisci_has_been_down_for_over_24_hours/

I would rather not blame Travis. The issue was there "long" (or at least some days) before Travis was acting slow. But (only) you could exclude that by running the same task locally. I assume, Nexus just need longer than the configured timeout.

Ammonite's build succeeded after a few more tries, on a weekend morning. Maybe we can try kicking off the travis publishing for Mill on the weekend?

I re-started the process on this saturday, and suddenly the Nexus deployment succeeded. The overall process failed though. Here's the error message:
 {"message":"Not Found","documentation_url":"https://developer.github.com/v3/repos/releases/#get-a-release-by-tag-name"}
1 targets failed
uploadToGithub java.util.NoSuchElementException: key not found: id
    scala.collection.MapLike.default(MapLike.scala:232)
    scala.collection.MapLike.default$(MapLike.scala:231)
    scala.collection.AbstractMap.default(Map.scala:59)
    scala.collection.MapLike.apply(MapLike.scala:141)
    scala.collection.MapLike.apply$(MapLike.scala:140)
    scala.collection.AbstractMap.apply(Map.scala:59)
    ujson.Js$Selector$StringSelector.apply(Js.scala:88)
    ujson.Js.apply(Js.scala:54)
    ujson.Js.apply$(Js.scala:54)
    ujson.Js$Obj.apply(Js.scala:94)
    ammonite.$file.ci.upload.apply(upload.sc:29)
    ammonite.$file.ci.upload.apply$(upload.sc:17)
    ammonite.$file.ci.upload$.apply(upload.sc:52)
    ammonite.$file.build.$anonfun$uploadToGithub$2(build.sc:590)
Traceback (most recent call last):
  File "ci/on-master.py", line 10, in <module>
    subprocess.check_call(sys.argv[1:])
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['ci/release.sh']' returned non-zero exit status 1
The command "$CI_SCRIPT" exited with 1.

Ok, I've found the reason for the failed GitHub upload. Travis-CI truncates the history to 50, when initially cloning the project. As we currently have more that 50 commits since the last tag, our logic to calculate the version number fails. Instead of something like: 0.3.6-68-42fcbe it gets 42fcbe7b2d5e51b1e7a351bdd91ce96c87fdfbb2-0-42fcbe.

Please note, that also the last 18 Maven Central releases got the wrong version number. E.g. com.lihaoyi:mill-main-api_2.12:42fcbe7b2d5e51b1e7a351bdd91ce96c87fdfbb2-0-42fcbe should be com.lihaoyi:mill-main-api_2.12:0.3.6-68-42fcbe

Fixed with commit 5d7c40ad91758d753a856abd2501cf0874ad6e96


Multi-cloud installation guide should be separate

Issue/Feature Description:
Currently all projects installation guide is combined. Which creates confusion.
Why this issue to fixed / feature is needed(give scenarios or use cases):
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

Hi, I would like to work on this.


Update ml4gw submodule commit

For some reason the ml4gw submodule was pointing to a stale commit. This points to the v0.4.2 tag.
For now, this also removes the stale unit testing workflow.

Do we need a submodule any longer? Specially if we use a tag

I think we should submodule, in case we need to iterate faster than ml4gw deployments

@deepchatterjeeligo Alright we're pointing at dev

sorry for the delay. thanks.

This is not what I was asking, but fine for now.


remove hardcoded paths

there are a few harcoded paths, move those to hiera to facilitate windows support

https://github.com/ripienaar/puppet-mcollective/pull/5


Better development/build scripts

📦 Pull Request
This updates our development & build scripts to  use TypeScript's new project references feature. This should lead to faster compilations and enables hot-reloading development.
🗜 Versioning
(Check one!)

[x] Patch: Bug Fix?
[ ] Minor: New Feature?
[ ] Major: Breaking Change?

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


[api-docs] Add another valid slash command nesting example

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/3daa6cb3e01de70e6e7599f3f94f4649c6074b51

This commit will not effect our lib's code. No changes required.


Fix user account switching issue

Fixes #136
In the future it might also be good to add more tests?

So it looks like the issue is still open.
To fix the permission error, there is more needed, as @fred-asimov says (and he is correct from a cursory SO glance)
Someone else suggested using dd to ✨ magically make it work

Also it looks like @fred-asimov suggests:
-sudo pulumi gen-completion bash > /etc/bash_completion.d/pulumi
+pulumi gen-completion bash | sudo tee /etc/bash_completion.d/pulumi

@danielbraun89 We/I need some more testing things to somehow actually try to catch these things. I'm making lots of mistakes! And mistakes are good when you have someone/some-robot to catch them, but I personally have no wish to push out non-functional code to people like @fred-asimov and @DaneWeber like I have done so far 😢. It's not very good.
Any suggestions anyone to help catch these earlier? Something in the test/ folder maybe? More robust shell testing?

Any suggestions anyone to help catch these earlier? Something in the test/ folder maybe? More robust shell testing?

First off, don't sweat it, it's a hard-to-predict bug that's really difficult to test!
I think at a higher level, in terms of devcontainer fixtures, it would make sense to have an integration/smoke test (that runs on everything, not something that every feature has to maintain separately), which actually creates a devcontainer with the feature. I'm not entirely sure how to do that programmatically.
It might be sufficient to have a test that pulls a basic docker image, or even the base devcontainer image, and then checks out the feature and runs install.sh inside of it. That probably would have been enough to catch this. I can give that a try soon, and see if it's as easy as I'm claiming. 😆

Any suggestions anyone to help catch these earlier? Something in the test/ folder maybe? More robust shell testing?

I haven't looked into what's required, but I'd think supporting the following would Pretty Darn Good:

Run the tests as a non-root user (the REMOTE_USER such as vscode)
Provide a .devcontainer.json for each test suite. In particular, I'd like to run tests with defaults and another with specified values.
Besides looking for the check commands to run without error, it would be cool to run regex patterns against the output for higher-fidelity but resilient tests.

Some of these installs download a ton of data, so it might be important to cache the downloads somehow.


IE 11: When activationEvent is set to focus, on click event x-editable element blurs out.

In IE 11, when activationEvent is set to focus, on click event x-editable element blurs out.
Can we have multiple activationEvent to enable edit mode through events like focus and click. Both would enable edit mode.

Another Issue I found is that when I have editable-select element and any value is selected from drop down it loses its focus and therefore user navigating from keyboard has to tab through all the items again.

Do you have a jsfiddle or plunker showing this issue?

Running into the same issue...seems like not much activity here...

Do you have a plnkr/jsfiddle showing your issue?  This will help others investigate your issue.
I have a Mac, so not easy for me to test with IE11.

http://plnkr.co/edit/Mn4vnP9vYS9eFiWhaNYQ?p=preview
Works in Chrome. Doesn't work in IE11.

I couldn't get the plnkr site to load in IE11 ☹

Well it's pretty simple anyways. Basically loads up fine. Then when you click on the item, it ONLY remains in the editable state while actively holding down the mouse-click. As soon as you let up on the mouse-click, it reverts to text state.

This seems to be a problem in Chrome now too with a recent Chrome update...
http://jsfiddle.net/L4jnp28a/5/
Expected results: enter edit mode via 'click' or 'focus' (tabbing to element)
Actual results: When clicking, edit mode is active only as long as mouse is down. A single click opens then closes edit mode immediately.

I see it too in Chrome now. Hack solution has been to instruct users to
right-click first, then type in the field. But would obviously be better to
have a fix.
On Tue, Aug 20, 2019, 1:21 PM Allison Shaw<EMAIL_ADDRESS>wrote:

This seems to be a problem in Chrome now too with a recent Chrome update...
http://jsfiddle.net/L4jnp28a/5/
Expected results: enter edit mode via 'click' or 'focus' (tabbing to
element)
Actual results: When clicking, edit mode is active only as long as mouse
is down. A single click opens then closes edit mode immediately.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vitalets/angular-xeditable/issues/465?email_source=notifications&email_token=ADF3RFMJ7MPFGRTM3C4VSL3QFRG37A5CNFSM4CCCNQUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4XRH3A#issuecomment-523178988,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADF3RFM7GM27MWZEEAOVYF3QFRG37ANCNFSM4CCCNQUA
.


Render reST file contents in Object inspector window

From<EMAIL_ADDRESS>on 2013-11-10T14:29:27Z
Spyder Version:  2.3.0dev3 (2889: r08903744e979 )
Python Version:  2.7.6rc1
Qt Version:      4.8.2, PySide 1.1.2 on Windows
What steps will reproduce the problem?

Open some README.rst
Switch "Object inspector" source to "Editor" What is the expected output? What do you see instead? It would be nice if "Object inspector" window contained rendered HTML of active file if active file is text reST. This would be very useful for catching error before PyPI uploads.

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1638

From<EMAIL_ADDRESS>on 2013-11-10T11:30:08Z
Labels: -Type-Defect Type-Enh Cat-ObjectInspector Usability Component-UI


Fix witness display bug

When we introduce a custom Debug implementation for the Witness we introduced a bug that causes code to panic if the witness contains an empty instruction.
The bug can be verified by putting patch 2 first or by running cargo run --example sighash on master.

Looks good! I guess there are technically two bugs, since there are two - 1s that could overflow. But I'm okay with only having a unit test for one.

The sighash example tests the other one, that is how I found this. Its not in any way obvious though. The code is so simple I rekon one test is ok.

Would be nice to get this in for release, making P-high so @sanket1729 sees it.


Return routes in BaseRestHandler

Description
[Describe what this change achieves]
Issues Resolved
#694
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@cliu123 Can you add more information to this PR?  Thanks...

@cliu123 Can you add more information to this PR? Thanks...

Sure. This can be post-GA. I'll add more info later.

Can we add more details on what this PR intends to solve

@Bukhtawar The issue #694 describes what this PR intends to solve.

Assuming routes becomes private, how does one add a route?
How will existing implementers of BaseRestHandler need to be changed? I imagine many define routes in their own implementations?

I imagine many define routes in their own implementations?
That is what happens, for example.
@dblock If defining routes field as protected in BaseRestHandler, the implementers would be allowed to add routes? Any thoughts? The intention is not to force implementers to implement the method abstractroutes when they do not need the method.

From what I understand, we need to also ensure one of routes() or replacedRoutes() are implemented so that an implementer cannot get away without specifying any?

From what I understand, we need to also ensure one of routes() or replacedRoutes() are implemented so that an implementer cannot get away without specifying any?

@Bukhtawar Agree. The intention of this PR is not to force implementers to implement the abstract method routes when they do not need the method. Perhaps routes was designed as an abstract method on purpose to ensure the implementers specify it?

Why is it necessary to overwrite the default RestHandler implementation in the BaseRestHandler? Implementing it as abstract does not enforce anything. Yes, it needs to be implemented by all subclasses of BaseRestHandler, but there is no enforcement on the actual return and it can be null or an empty list. The proposed implementation is subject of NullPointerException and subclasses of RestHandler will still need to initialize routes to a list.

Why is it necessary to overwrite the default RestHandler implementation in the BaseRestHandler? Implementing it as abstract does not enforce anything. Yes, it needs to be implemented by all subclasses of BaseRestHandler, but there is no enforcement on the actual return and it can be null or an empty list. The proposed implementation is subject of NullPointerException and subclasses of RestHandler will still need to initialize routes to a list.

@vrozov

Why is it necessary to overwrite the default RestHandler implementation in the BaseRestHandler?

IMO, It is actually unnecessary.

Implementing it as abstract does not enforce anything.

Is our goal that the subclasses of BaseRestHandlerdo not have to implement the method routes unless the subclasses really need the method?

there is no enforcement on the actual return and it can be null or an empty list.

For this purpose, removing the routes() method from the BaseRestHandler will make that happen?

Is there a harm returning an empty list and let the contract stay

Is there a harm returning an empty list and let the contract stay

I do not think there ia a harm returning an empty list, but looks like BaseRestHandler does not have to override the method routes(), and the abstract class routes() in BaseRestHandler makes subclasses of BaseRestHandler have to implement the method routes() even if the subclasses do not need that method.

Is our goal that the subclasses of BaseRestHandlerdo not have to implement the method routes unless the subclasses really need the method? If not, the issue #694 will need more details to clarify the goal.

@cliu123 The reason RestHandler provides the default implementation of routes() (and other related methods) is to avoid unnecessary reimplementation of the method in the subclasses. By introducing abstract routes() method or introducing protected List<Route> routes  to BaseRestHandler, the goal of the default interface implementation is defeated.

I think the way to look at it is can a RestHandler exist without a route. If not then I think we should have a mechanism to enforce it.
The abstract class enforces that and this change will change the enforcement

I think the way to look at it is can a RestHandler exist without a route. If not then I think we should have a mechanism to enforce it.
The abstract class enforces that and this change will change the enforcement

@Bukhtawar Would you mind posting the concern on the issue #694 where the goal of this PR is discussed.

The abstract class enforces that and this change will change the enforcement

@Bukhtawar The abstract class does not enforce that subclasses return nonempty routes. All it (unnecessary) enforces is that subclasses have some implementation of routes(), but such implementation may return null or an empty list. Should there be a requirement for subclasses to provide an implementation of routes(), RestHandler should not provide a default implementation.

Interface is meant to enforce a contract not validate if the implementation is correct @vrozov

Interface is meant to enforce a contract not validate if the implementation is correct @vrozov

There is no difference in enforcing a (compile-time) contract between interface and abstract class. Abstract class enforces that the abstract method is implemented by all subclasses, interface enforces that all methods without default implementation are implemented. There is no enforcement on how they are implemented in both cases. Introducing abstract routes() method to BaseRestHandler does not add any value, IMO.

Does it make sense for any plugin to not provide that info in the first place?

Yes, it does. Please see my note on the #694. For example, some handlers may be disabled based on settings/run-time configuration.

Passing invalid or null explicitly is another thing.

My point is that abstract route() or implementing route() (the way it was implemented) on the BaseRestHandler does not help to avoid returning null in the subclasses.

Just to clarify when I am referring to a Rest Handler I am intending a handler defined by the plugin and not the interface name as such

It does not need to be a plugin implementation. It can be a core OpenSearch handler as well. Any subclass of BaseRestHandler is the subject of the discussion.

Looks like we went back and forth on things. Describe what this PR actually does in the description of this PR.
We're removing a method that will no longer need to be implemented. Was it really never used in this codebase? A-OK.

Looks like we went back and forth on things. Describe what this PR actually does in the description of this PR.

What this PR does is described in issue #694. I don't have the permission to link this PR to that issue. Would you please help do that if you have the permission?

We're removing a method that will no longer need to be implemented. Was it really never used or called in this codebase? If yes, those need to be removed. Otherwise please confirm and good riddance then.

There're several subclasses of BaseRestHandler have to implement the method routes() that is never used, for example and a few other handlers in here.

We're removing a method that will no longer need to be implemented. Was it really never used or called in this codebase? If yes, those need to be removed. Otherwise please confirm and good riddance then.

There're several subclasses of BaseRestHandler having to implement the method routes() that is never used, for example and a few other handlers in here.

The method was used and called. It is not removed. The PR fixes a bug in BaseRestHandler that overrides the default implementation of routes() method in RestHandler. Please see https://github.com/opensearch-project/security/pull/1256#discussion_r658515557

@dblock

Looks like we went back and forth on things. Describe what this PR actually does in the description of this PR.

What this PR does is described in issue #694. I don't have the permission to link this PR to that issue. Would you please help do that if you have the permission?

You don't need permissions, when you paste a link it links. I was referring to this:

We're removing a method that will no longer need to be implemented. Was it really never used or called in this codebase? If yes, those need to be removed. Otherwise please confirm and good riddance then.

There're several subclasses of BaseRestHandler having to implement the method routes() that is never used, for example and a few other handlers in here.

I meant, anything in this repo?
Are there any objections to merging this as is from @vrozov and @Bukhtawar ?

I meant, anything in this repo?

Even in this repo there are methods that have to implement routes() only because routes() was declared abstract in BaseRestHandler (one case is already taken care of). IMO, all such occurrences needs to be removed in the PR.

LGTM except for PR title

LGTM except for PR title

@vrozov Changed the PR title

@dblock @Bukhtawar How do the changes look to you?

@dblock @Bukhtawar How do the changes look to you?

Now that the unnecessary implementations have been removed in tests, this makes a lot more sense! Good with me.
You should still update the description in the PR for anyone reading this for the first time.

start gradle check

You should still update the description in the PR for anyone reading this for the first time.

@dblock Done.

Looks like I missed the whole party of conversation.
I like where we ended up!

@dblock are we waiting for anything else to merge this ?

Merged, thanks @cliu123!

@dblock @saratvemulapalli Are we going to backport this to 1.0 or 1.x(for 1.1.0) branches?

Since this is not a security fix it definitely doesn't go to 1.0, but I don't see why we wouldn't backport to 1.1. Feel free to make that PR @afazel if you need it.


One repo

fixed: https://github.com/open-cluster-management/backlog/issues/14034

/retest

/retest

COMPONENT_VERSION

it can be replaced by OSCI_COMPONENT_NAME and OSCI_COMPONENT_VERSION. for example:
    name: branch-ci-open-cluster-management-rbac-query-proxy-release-2.4-publish
    run_if_changed: ^(proxy/.*|pkg/.*)$
    spec:
      containers:
      - args:
        - -f
        - /opt/build-harness/Makefile.build-harness-openshift-ci
        - osci/publish
        command:
        - make
        env:
        - name: OSCI_COMPONENT_NAME
          value: rbac-query-proxy
        - name: OSCI_COMPONENT_VERSION
          value: "2.4"
        - name: OSCI_IMAGE_REMOTE_REPO_SRC
          value: registry.ci.openshift.org/open-cluster-management
        - name: OSCI_PUBLISH_DELAY
          value: "20"
        - name: SELF
          value: make -f /opt/build-harness/Makefile.build-harness-openshift-ci


chore: Clean up and clarify some translator flavor logic

Cleans up Translator flavor: removes some redundant methods, replaces list style getters with one built up from sub-components, and clarifies the subtlety around getters that return everything except concatenated polys.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (64): 122 (+121%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,844
6,844
6,844

l1_rollup_execution_gas
587,325
587,325
587,325

l2_block_processing_time_in_ms
1,442 (-1%)
5,384
10,493 (+1%)

note_successful_decrypting_time_in_ms
185 (-5%)
378
720 (+11%)

note_trial_decrypting_time_in_ms
74.9 (-14%)
76.7 (+2%)
:warning: 122 (+121%)

l2_block_building_time_in_ms
18,669 (-2%)
73,209 (-1%)
145,706 (-1%)

l2_block_rollup_simulation_time_in_ms
18,455 (-2%)
72,449 (-1%)
144,222 (-1%)

l2_block_public_tx_process_time_in_ms
8,489 (-1%)
33,073 (-1%)
66,369 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,818
30,407

note_history_successful_decrypting_time_in_ms
877 (+1%)
1,712 (-1%)

note_history_trial_decrypting_time_in_ms
93.3 (+3%)
111 (+49%)

node_database_size_in_bytes
19,107,920
35,778,640 (-1%)

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
156 (-1%)
47,763
31,978

private-kernel-inner
176
80,794
31,978

private-kernel-ordering
136
57,732
50,043

base-parity
73.9 (+1%)
128
265

root-parity
123 (-4%)
1,244
265

base-rollup
12,266
110,926
957

root-rollup
49.6
4,551
821

public-kernel-app-logic
1,657 (-1%)
62,017
52,067

public-kernel-tail
4,218
168,566
7,491

merge-rollup
3,175 (+1%)
2,760
957

public-kernel-teardown
112 (-6%)
62,017
52,067

public-kernel-setup
119 (+1%)
62,017
52,067

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
11.1 (+1%)
17.7
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.649 (+1%)
0.546
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
51.4
80.2
258 (-1%)
504 (+1%)
977 (-1%)
1,930 (-1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.527
0.495
0.470 (-1%)
0.471 (+1%)
0.464 (-1%)
0.462 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.3
118 (-1%)
377
737
1,456 (-1%)
2,895 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.532
0.534
0.511
0.508
0.505 (-1%)
0.505 (-1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
67.3

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.587

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
51,411
543,524

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
897
1,145
1,345

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,235 (-1%)
859
3,518 (+1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
831
763 (+1%)
1,350
905 (+1%)
2,071
1,091 (+1%)


[jax2tf] Clean up and expand binary elementwise harnesses.

This last commit should unify things the way we want. Note that I did not change the implementation of lax_div_rem because it requires paying some special attention to the generation of the arguments.


Update about-keyboard-input.md

Add virtual key code list link

@Karl-Bridge-Microsoft looks like Scan Code Table on this page is missing "Note 1" etc descriptions after the table. Is that intended?


Allow SSL certificate verification to be disabled

Hi thanks for this. This is great. Apologies for the delay in getting it merged. I've been swamped in marriage preparations the last month.

Tagged a pushed as v0.4.5


Burpsuit Task

Issue: #215
Burpsuite installed and configured browser

Hey @shrut2606
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @shrut2606
Your PR has been merged 🥳🥳 and you have earned 20 points.
Thanks for contributing in OpenCode'24 ✨✨


Translated Minecraft world import strings and advanced model import strings

Translated:

Importer.Folder.Minecraft
Importer.Folder.Minecraft.Description
Importer.Model.Advanced.ForcePointFiltering
Importer.Model.Advanced.ForceNoMipMaps
Importer.Model.Advanced.ForceUncompressed

(this comment will be updated if more changes will be made before this pull request's merge/closing)

Thank you for the new translations!


NPT desktop initial MVP

Design a rough MVP with the core features and UX for NPT tunnel, so that it can be handed over to @CurtlyCritchlow and @Dariamarg for further iteration.

Raised to P1 and pulled back into PR 93


cannot find icons

On line 254 of the config.toml file, we are told to add the icons we need to the following file, but the repository link is broken (https://github.com/Jieiku/abridge/blob/master/sass/include/_icons.scss). I have also uncommented lines for the individual icons and enabled icons in the config. It says that it would negatively impact performance if we loaded the entire fontawesome css file so I wanted to know if there was a preferred built-in way to do this still.

most of the independent scss files were merged into a single abridge.scss file during the large refactor, it greatly simplified some of the advanced scss features that were added.
Abridge now uses exclusively svg icons, they are very lightweight, however any scss you want can be added to the extra file:
https://github.com/Jieiku/abridge/blob/master/COPY-TO-ROOT-SASS/_extra.scss
I believe fontawesome has a way of loading their icons via scss, but I honestly lost interest in using fontawesome myself and instead I add svg icons to the icons function here as needed:
https://github.com/Jieiku/abridge/blob/8bc88707ca39db73c4fd890fa030b91b20a8e1c8/sass/_functions.scss#L84
It might be possibly to extend that function for user content but it could also be difficult, encoding svg icons into scss is not something just anyone would be comfortable figuring out how to do.
Pull requests are welcome, or if you have an idea you would like to discuss.

I updated the config.toml to correct the reference to the no longer existing file, feel free to comment if there is anything else you would like to discuss. Thanks again!


Validate TLS certificates

Use the validate_x509_rsa_key_pair() function from Puppet stdlib to
verify the following for our TLS certificates:

the private key matches the certificate
the private key is correctly formatted, i.e. it is a private key and
is not malformed
the certificate is correctly formatted, i.e. it is a PEM-encoded
certificate and is not malformed

We've seen each of these cases happen in the past, which this change
will guard against. If validation fails, the Puppet run will abort with
an error.

Still not present in stdlib version 4.11.0.

Closing this for now until a new version of stdlib is released.