Problems build images for ARM8

When building an image on an ARM8/AARCH64 platform using
./gradlew jibDockerBuild --image ...
the image seems to build ok but attempting to run it then I get the error...
standard_init_linux.go:207: exec user process caused "exec format error"
On inspection (docker image inspect) I see that the architecture is "amd64" when it should be "arm64".
Is there any way to force it to create an arm64 architecture image?
I see there is an issue reported about using the base image architecture and that it claims to be fixed (and merged) but I'm not seeing it and I'm using the latest release at this time (1.0.2).

Hi @dementiacs,
#1547 is not in 1.0.2. We are going to make a 1.1.0 lease this week, so stay tuned. 1.1.0 will inherit the OS/Arch specified in the base image.

We will update here once 1.1.0 is released.

@dementiacs 1.1.0 has been released. Note that to use arch-specific Official Docker Hub Images, you will need to use an arch-specific organization like arm64v8/openjdk.
If you're still having problems, please describe the image that you're trying to use.

Note that to use arch-specific Official Docker Hub Images, you will need to use an arch-specific organization like arm64v8/openjdk.

This doesn't seem to really work, and you may need to use a specific digest: #2148

Note that to use arch-specific Official Docker Hub Images, you will need to use an arch-specific organization like arm64v8/openjdk.

This doesn't seem to really work, and you may need to use a specific digest: #2148

No need to use a specific digest. See https://github.com/GoogleContainerTools/jib/issues/2148#issuecomment-672129017


Add a page to thank contributors

This PR adds a page at /thanks that thanks all contributors. Right now the code needs a hard-coded list of Patreon supporters (@chrishtr I'll need you to supply this), a hard-coded list of git contributors (we don't expect too many, so hopefully this isn't too annoying) and an auto-generated list of contributors through the built-in feedback tools.

You can see it live here: http://browser.engineering/thanks

Uhh, @chrishtr, could you perhaps fill in the Patreon list?

https://github.com/browserengineering/book/pull/423


The property 'Support Tags' is 'Unknown' for one blob container under one SAS attached storage account without the 'Filter' permission

Storage Explorer Version: 1.16.0-dev
Build: 20201029.1
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Select one storage account which supports tags -> Attach it via SAS with full permissions except the 'Filter' permission.
Expand the attached storage account -> Blob Containers.
Select one blob container -> Check the value of the property 'Supports Tags'.

Expect Experience:
The value is 'True'.
Actual Experience:
The value is 'Unknown'.
More Info:
This issue doesn't reproduce for the storage account which does not support tags. Details as below:
Steps:

Select one storage account which do not support tags -> Attach it via SAS with full permissions except the 'Filter' permission.
Expand the attached storage account -> Blob Containers.
Select one blob container -> Check the value of the property 'Supports Tags'.

Actual:
The value of the property is 'False'.

The property "Supports Tags" is supposed to show whether the service is available. Our detection logic attempts to filter by tags to see if the service returns error like "invalid header". The reason why we leave it as unknown is that it is still possible that the user has "t" permission or the permission maybe changed later such as from roles.

@JasonYeMSFT ok, got it.


Review alternate candidate for that jimmy warting iframe..!

Review alternate candidate for that jimmy warting iframe..

create our own writable file stream mechanism on iati.cloud


Allow rectangle to be resized

At the moment there is no way to edit a rectangle's dimensions after it's been created.  It's a very common pattern in graphical editors to be able to resize elements, with the element gaining a 'bounding box' UI around it to indicate how it can be resized (see below):


How to use (make_service_fn) function (Help)

Sorry, I have a problem and I need help
I just started learning this library, this question has been confused for a whole week, I can't help but ask for your help
I built a class to handle the request from the client, but it has been unable to compile, the error message is as follows
Thank you all
error[E0495]: cannot infer an appropriate lifetime for lifetime parameter in function call due to conflicting requirements
  --> src\main.rs:69:21
   |
69 |                 tom.call(req)
   |                     ^^^^
   |
note: first, the lifetime cannot outlive the lifetime `'_` as defined on the body at 68:44...
  --> src\main.rs:68:44
   |
68 |             Ok::<_, Infallible>(service_fn(move |req| {
   |                                            ^^^^^^^^^^
note: ...so that closure can access `tom`
  --> src\main.rs:69:17
   |
69 |                 tom.call(req)
   |                 ^^^
note: but, the lifetime must be valid for the expression at 68:33...
  --> src\main.rs:68:33
   |
68 |             Ok::<_, Infallible>(service_fn(move |req| {
   |                                 ^^^^^^^^^^
note: ...so type `fn([closure@src\main.rs:68:44: 70:14 tom:std::sync::Arc<Host>]) -> hyper::service::util::ServiceFn<[closure@src\main.rs:68:44: 70:14 tom:std::sync::Arc<Host>], hyper::body::body::Body> {hyper::service::util::service_fn::<[closure@src\main.rs:68:44: 70:14 tom:std::sync::Arc<Host>], hyper::body::body::Body, impl std::future::Future>}` of expression is valid during the expression
  --> src\main.rs:68:33
   |
68 |             Ok::<_, Infallible>(service_fn(move |req| {
   |                                 ^^^^^^^^^^

error: aborting due to previous error

use std::error::Error;
use std::convert::Infallible;
use std::sync::{Arc};
use hyper::service::{make_service_fn, service_fn};
use futures_util::{future, TryStreamExt};
use hyper::{header, Body, Client, Method, Request, Response, Server, StatusCode};

struct Host {
    name:String,
    num:u64,
}

impl Host
{
    pub fn new(name: String, num: u64) -> Self
    {
        Self
        {
            name,
            num,
        }
    }
    pub async fn call(&self ,req: Request<Body>) -> Result<Response<Body>, hyper::Error>
    {
        let mut response = Response::new(Body::empty());
        match (req.method(), req.uri().path())
        {
            (&Method::POST, "/") =>
            {
                let (parts, body) = req.into_parts();
                let entire_body = body
                .try_fold(Vec::new(), |mut data, chunk| async move {
                    data.extend_from_slice(&chunk);
                    Ok(data)
                })
                .await?;
                let x = String::from_utf8(entire_body).unwrap();
                Ok(Response::new(Body::from(x)))
            }
            _ =>
            {
                let mut not_found = Response::default();
                *not_found.status_mut() = StatusCode::NOT_FOUND;
                Ok(not_found)
            }
        }
    }
}

#[tokio::main]
async fn main() {
    let addr = ([127, 0, 0, 1], 3000).into();
    let example = Arc::new(Host::new("tom".to_string(),123));
    let make_service = make_service_fn(move |_| {
        let tom = example.clone();
        async move {
            Ok::<_, Infallible>(service_fn(move |req| {
                tom.call(req)
            }))
        }
    });
    let server = Server::bind(&addr).serve(make_service);
    if let Err(e) = server.await {
        eprintln!("server error: {}", e);
    }
}

It's very easy to miss:

#[tokio::main]
async fn main() {
    let addr = ([127, 0, 0, 1], 3000).into();
    let example = Arc::new(Host::new("tom".to_string(),123));
    let make_service = make_service_fn(move |_| {
        let tom = example.clone();
        async move {
            let tom = tom.clone();
            Ok::<_, Infallible>(service_fn(move |req| {
                tom.call(req)
            }))
        }
    });
    let server = Server::bind(&addr).serve(make_service);
    if let Err(e) = server.await {
        eprintln!("server error: {}", e);
    }
}


Don't clear isCommitStateDirty flag in setReadOnly

When autoCommit is false, if setReadOnly is called on a connection after executing some queries, closing the connection returns the dirty connection (with transaction open) to the pool.
HikariDataSource ds = /* setup the pool */;
ds.setAutoCommit(false);
try (Connection conn = ds.getConnection()) {
  // ... execute some queries on `conn`
  conn.setReadOnly(false);
}
// connection returned to the pool with transaction open

In the above scenario, the following cleanup logic is not executed.
https://github.com/brettwooldridge/HikariCP/blob/ed2da5f1f4ef19f871fac12effc0b199706905dc/src/main/java/com/zaxxer/hikari/pool/ProxyConnection.java#L248-L251
Anyway I don't see why isCommitStateDirty should be set to false when setReadOnly is called. The line was added in 6ed3b185912bd835fb1a25da6ba80e0226fb7f2e without useful description.

Convince me this isn't user error. Why is setReadOnly() being called in the middle of an outstanding transaction?

Of course it is user error. But I would expect transactions to be rolled back when closing the connection in any case.
Also, some JDBC drivers does not throw when setReadOnly is called during transaction if the flag is same as before.

Well, I'm definitely not going to throw an exception, that's up the the driver. What concerns me about this fix is that it is a direct rollback of a a change submitted by another user many years ago (6ed3b185).
Granted this was a batch of fixes in a single commit called "small but important fixes" and he was just combing through the code looking for what he considered "correctness" fixes. The only concern I have is the potential change in behavior and whether it is likely to break existing code. I'm willing to put the change out there, but if breakage reports flood in, it will have to be reverted. You ok with that?

I don't need the fix now because I found it in my previous job. (did apply custom build with the patch there, and it worked without problem though)
If you think applying the change is risky, feel free to close the PR! 😄


wanna as seranrio

hi,
you have done great work
thank you.
I wanna Ask About the controlees on b1slayer batch transaction
what if i want that if i am sending 10 batches and if 1 fails then my other data posts and failed one gives me response .
how many ways i can control and pls give me some type of documentation if possible.
thank you.

Hi, @sandy5791.
I'm not sure I understand your question. Please refer to the official Service Layer documentation regarding batch operations.

@sandy5791, I'm closing this issue. Let me know if you still need assistance.


add DALLE-3 support

Currently the gem only supports dalle1. There is dalle3 available.
I believe this support is coming soon to them gem: https://github.com/alexrudall/ruby-openai/pull/418

Released in v6.4


How to check if a message is confirmable?

I'm setting up a CoAP UDP Server. The Handler interface does not provide the message Type.
How to get the UDP Message?

It depends what you register as Handler. If it is mux then it doesn't provide you such information. If you need handle ACK message you need to use WithHandlerFunc server option, where you get  udp request with Type. After that you can use mux similar as this code.

Okay thanks, so I can hook into handlerFuncToMux to handle deduplication and sending ACKs I guess. I will try that out.


Removed feeds from feeds_list aren't removing

Hello,
As the title says, the default feeds_list is persistent. I've rebooted thinking the feeds are cached in RAM, yet the feeds I've added aren't showing and the default feeds continue to show. I'm thinking I may not be using your tool correctly, although it seems pretty straightforward.
Any further information I can provide to help with this?
I'm a noob running arch linux, i3, dual polybars.
Thank you
p.s, really love your tool btw I'm so happy this exists!

Hi,
it is a bug, and I fix it in the incoming release. Meanwhile you can  use this command:
~/.config/polybar/scripts/news/news.sh select
to select what feeds to show. You can also  bind it to a keyboard shortcut adding the following line
bindsym Mod4 + Control + Shift + f exec --no-startup-id ~/.config/polybar/scripts/news/news.sh select
to ~/.config/i3/config.
Thank you for this problem report.

The release 3.0.3 should fix this bug.


feat: Introduce KeyPackageBuilder.

This PR introduces a KeyPackageBuilder. This simplifies KeyPackage creation and reduces the times when we have to touch the tests. Specifically, changing all instances of Vec<Extension> to Extensions would require to touch all KeyPackage::create instances. After this PR is merged, this is not needed anymore. Thus, in a way, I see this as preliminary work towards closing #720.

Sorry, @franziskuskiefer. I forgot that new pushes dismiss reviews. Can you approve again?


[Synapse] Deploy test resources step failing on nightly tests

Builds such as this are failing with:
Name 'sparkt499f98df8c4d4f46' is invalid. The name must be at
| most 15 characters long and must contain only alphanumeric
Discussions suggest there is a test-resources.json file that needs updating with a shorter name.

Following-up on the Teams conversation,  I'll add the pedantic issue stuff here.  :smile:
For work-item issues, we're asked to add an estimate, milestone, and (if possible) assign to an epic.     If this is work that isn't scheduled, adding to the "Backlog" milestone is a good place to park, and allows you to defer estimating.

This was discussed internally at one point and the suggestion was to use https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/template-functions-string?tabs=json#take to trim the random name down to the right number of characters for a spark job.


Fix incorrect file names

Checklist

[ ] Issue(s) linked

Not Required
Completed in https://github.com/turbot/steampipe-mod-aws-insights/pull/314


Compiling openraft with v2 API enabled results in some warnings

Following warnings are produced:
warning: unused import: `std::future::Future`
 --> external/openraft/openraft/src/testing/store_builder.rs:1:5
  |
1 | use std::future::Future;
  |     ^^^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: unused import: `crate::RaftStorage`
 --> external/openraft/openraft/src/testing/store_builder.rs:8:5
  |
8 | use crate::RaftStorage;
  |     ^^^^^^^^^^^^^^^^^^

These should be probably hidden by not importing the items in case v2 API is not configured.

Thank you man!
Meanwhile, storage-v2 is not yet stable or well tested. :(

Meanwhile, storage-v2 is not yet stable or well tested. :(

Yep, we are on the bleeding edge :-).


Missing PowerShell documentation for this process

Documentation for enabling the roles is present but the configuration of the CA with PowerShell is not. The use of PowerShell may be preferred by some customers.

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

ID: 2e75c5bf-aefe-154e-de20-a01ab982adf6
Version Independent ID: 92189653-36bb-7e01-aa84-20fbad5808c8
Content: Install the Certification Authority
Content Source: WindowsServerDocs/networking/core-network-guide/cncg/server-certs/Install-the-Certification-Authority.md
Product: windows-server-threshold
GitHub Login: @shortpatti
Microsoft Alias: pashort

@nromyn, Thank you for letting us know about the missing content.  We will work to get it incorporated.

Added a TIP pointing users to the PS cmdlets and optional parameters.


Incorrect exit code if ConnectionError raised by urllib3

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS:  Mojave
version: 0.12.17

Issue
During poetry publish, if a ConnectionError is raised, Poetry will exit with status code zero, indicating success, but it should be non-zero.
This can occur e.g. if remote repository is firewalled.
I am using AWS CodeBuild, which reasonably assumes that the command succeeded, when in fact it failed.
Other network problems are correctly handled.
This issue can be repro'd by configuring a non-existent repo', e.g.:
$ poetry config repositories.foo https://<IP_ADDRESS>/
$ poetry publish --repository foo
$ echo $?

[ConnectionError]
HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.Verified
HTTPSConnection object at 0x103495e10>: Failed to establish a new connection: [Errno 61] Connection refused'))

Exception trace:
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/cleo/application.py in run() at line 94
  status_code = self.do_run(input_, output_)
/Users/redacted/.poetry/lib/poetry/console/application.py in do_run() at line 88
  return super(Application, self).do_run(i, o)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/cleo/application.py in do_run() at line 197
  status_code = command.run(input_, output_)
/Users/redacted/.poetry/lib/poetry/console/commands/command.py in run() at line 77
  return super(BaseCommand, self).run(i, o)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/cleo/commands/base_command.py in run() at line 146
  status_code = self.execute(input_, output_)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/cleo/commands/command.py in execute() at line 107
  return self.handle()
/Users/redacted/.poetry/lib/poetry/console/commands/publish.py in handle() at line 54
  self.option("repository"), self.option("username"), self.option("password")
/Users/redacted/.poetry/lib/poetry/masonry/publishing/publisher.py in publish() at line 84
  return self._uploader.upload(url)
/Users/redacted/.poetry/lib/poetry/masonry/publishing/uploader.py in upload() at line 101
  self._upload(session, url)
/Users/redacted/.poetry/lib/poetry/masonry/publishing/uploader.py in _upload() at line 185
  self._do_upload(session, url)
/Users/redacted/.poetry/lib/poetry/masonry/publishing/uploader.py in _do_upload() at line 202
  resp = self._upload_file(session, url, file)
/Users/redacted/.poetry/lib/poetry/masonry/publishing/uploader.py in _upload_file() at line 237
  headers={"Content-Type": monitor.content_type},
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/requests/sessions.py in post() at line 581
  return self.request('POST', url, data=data, json=json, **kwargs)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/requests/sessions.py in request() at line 533
  resp = self.send(prep, **send_kwargs)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/requests/sessions.py in send() at line 646
  r = adapter.send(request, **kwargs)
/Users/redacted/.poetry/lib/poetry/_vendor/py3.7/requests/adapters.py in send() at line 516
  raise ConnectionError(e, request=request)

This has been fixed in the prereleases of the 1.0.0 version


spelling fixes

Description
fixed many spelling errors - most in comments, one in log.
Motivation and Context
cleaner repo.
Does this change the ABI/API?

[ ] This changes the ABI/API

What tests did you add/modify to account for these changes

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New module / feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I added a changeset to account for this change

Reviewer Checklist:

[ ] My review includes a symposis of the changes and potential issues
[ ] The code style is enforced
[ ] There are no risky / concerning changes / additions to the PR

Can you rebase? Then we can merge :).

Hi! Can you possibly rebase? If not I can rebase for you.

Closing until a rebase.


GStreamer RTSP Pipeline doesn't work on MediaMTX 1.8

Which version are you using?
v1.8.0
Which operating system are you using?

[x] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
Description
When running MediaMTX v1.5.1 using the same config, GStreamer pipeline starts on event runOnReady. When using MediaMTX v1.8.0 it fails with
ERROR: from element /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0: Could not read from resource.
Additional debug info:
../gst/rtsp-sink/gstrtspclientsink.c(3123): gst_rtsp_client_sink_send (): /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0:
Got error response: 400 (Bad Request).
Execution ended after 0:00:01.071807791
Setting pipeline to NULL ...

In attached files there is a config used with both MediaMTX versions.
mediamtx config.txt
Also, in attached files there are logs from both versions of MediaMTX
mtx1.5.log
mtx1.8.log
Describe how to replicate the issue

Use provided config
Run MediaMTX v1.8.0

Camera used in reproduction pipeline is AXIS 221
Did you attach the server logs?

yes
Did you attach a network dump?

no

Кривая синхронизация Даты и времени на самой камере.

Кривая синхронизация Даты и времени на самой камере.

That comment references another issue ( #2708  ). That one particular issue is not related to date/time

I'm having the same issue with the Docker image;
docker run --rm -it --network=host bluenviron/mediamtx:latest

I get the same error when running
gst-launch-1.0 filesrc location=video.mp4 ! qtdemux name=d d.video_0 ! rtspclientsink location=rtsp://<IP_ADDRESS>:8554/mystream

Output log
Setting pipeline to PAUSED ...
Progress: (open) Opening Stream
Pipeline is PREROLLED ...
Prerolled, waiting for progress to finish...
Progress: (connect) Connecting to rtsp://<IP_ADDRESS>:8554/mystream
ERROR: from element /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0: Could not open resource for reading and writing.
Additional debug info:
../subprojects/gst-rtsp-server/gst/rtsp-sink/gstrtspclientsink.c(3305): gst_rtsp_client_sink_connect_to_server (): /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0:
Failed to connect. (Generic error)
ERROR: pipeline doesn't want to preroll.
Setting pipeline to NULL ...
Freeing pipeline ...

I also can't see any listener on port 8554
lsof  -nP -i | grep 8554

If I run not in host mode and expose the ports I get slightly further, specifically getting a new error ERROR: from element /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0: Could not get/set settings from/on resource.. This way I do see the service listening on TCP 8554.
docker run --rm -it \
-e MTX_PROTOCOLS=tcp \
-e MTX_WEBRTCADDITIONALHOSTS=192.168.x.x \
-p 8554:8554 \
-p 1935:1935 \
-p 8888:8888 \
-p 8889:8889 \
-p 8890:8890/udp \
-p 8189:8189/udp \
bluenviron/mediamtx

Output log
Setting pipeline to PAUSED ...
Progress: (open) Opening Stream
Progress: (connect) Connecting to rtsp://<IP_ADDRESS>:8554/mystream
Pipeline is PREROLLED ...
Prerolled, waiting for progress to finish...
Progress: (open) Retrieving server options
Progress: (open) Opened Stream
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Progress: (request) Sending RECORD request
Progress: (record) Sending server stream info
Progress: (request) SETUP stream 0
Redistribute latency...
Redistribute latency...
ERROR: from element /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0: Could not get/set settings from/on resource.
Additional debug info:
../subprojects/gst-rtsp-server/gst/rtsp-sink/gstrtspclientsink.c(4307): gst_rtsp_client_sink_setup_streams (): /GstPipeline:pipeline0/GstRTSPClientSink:rtspclientsink0:
Could not setup transport.
Execution ended after 0:00:00.003751000
Setting pipeline to NULL ...
Freeing pipeline ...

Setting protocols=tcp in rtspclientsink as per https://github.com/bluenviron/mediamtx/issues/2257#issuecomment-1695850914 seems to fix it and the connection succeeds.
Reverting back to host networking, but forcing TCP protocol means we're back to the original issue
docker run --rm -it --network=host -e MTX_PROTOCOLS=tcp -e MTX_WEBRTCADDITIONALHOSTS=192.168.x.x bluen
viron/mediamtx:latest

@XDGFX your issue is unrelated from this one and is a duplicate of #2257

@lqbrqt the runOnReady command you are using needs to be edited, you need to change $MTX_PATH_DELAY into ${MTX_PATH}_delay:
runOnReady: >
      gst-launch-1.0 rtspsrc location=rtsp://<IP_ADDRESS>:8554/$MTX_PATH latency=0
      ! rtpmp4vdepay
      ! queue max-size-buffers=0 max-size-time=0 max-size-bytes=0 min-threshold-time=0
      ! rtspclientsink protocols=tcp location=rtsp://<IP_ADDRESS>:8554/${MTX_PATH}_delay

This is because since #3044 environment variables in commands are now replaced by using os.Expand() instead of strings.ReplaceAll().


feat: add powerbi query runner

What type of PR is this?

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[x] New Query Runner (Data Source)
[ ] New Alert Destination
[ ] Other

Description

How is this tested?

[x] Unit tests (pytest, jest)
[ ] E2E Tests (Cypress)
[ ] Manually
[ ] N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@gabriel-suela


Upgrade to semantic-release v23

New feature motivation
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^22.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/@semantic-release/wordpress
npm ERR!   dev<EMAIL_ADDRESS>from the root project
New feature description
Close #212
New feature implementation
No response

Done - and done


Fix typo in Connection

Fix a dangerous typo in Connection class. VERSION evaluates to gem version not HTTP protocol version.
With this correction HTTP11 protocol version is unable to write fixed length bodies. Instead, it defaults to writing chunked bodies (this is covered with a spec).
I don't know if this behavior is desired. If no then maybe we should remove this condition version != HTTP11.
Types of Changes

[x] Bug fix.
[ ] New feature.
[ ] Performance improvement.

Testing

[x] I added new tests for my changes.
[x] I ran all the tests locally.

I checked failing github actions. It seems spec setup is failing, not the specs.
I confirm again I ran the specs locally and they were all passing.

You are right, maybe this is better:
				elsif length = body.length and trailers.nil?
					write_connection_header(version)
					write_fixed_length_body(body, length, head)


Add docs to methods

Autocomplete is great, but it would be amazing if the documentation of the .NET methods could be added to the stubs as well.

agree. if there are more prompts of multiple constructors and descriptions that would be great. simply add more comments in each definition. This will help us A LOT to code faster and know Rhino Geometry better! @sbaer @BobMcNeel


feat(tutorial): complete step 5 - Cambridge

https://carbon-tutorial-johnjwood.mybluemix.net/#/

:+1: All good - congratulations on completing the Carbon Essentials course!
You can now shut down your Bluemix instance to avoid potential hosting charges.


Don't allow a user to use localhost as a connection address

Tons of issues with people using localhost and then being unable to connect to the SQL socket, require using an IP which avoids that issue.

this is only force localhost to <IP_ADDRESS> in assing config or block this parameter? (question for.. know?)

Technically if it is set by a plugin I don't think there is much you can do there, though making a huge warning stating you must use the external ip instead of localhost on the docs should make a difference.

This is for panel DB connections, not plugins.

For now I've put a warning on the environment setup for database connections, and the command will also test connections, so I think I'm going to close this as 8722571 addresses it for now.


boards: arm: Add Renesas Spider board

This pull request is introducing Generation 4 of Renesas R-Car SoCs with the S4 SoC as well as the Spider board into Zephyr.
Gen4 SoCs vs Gen3 (Zephyr POV) :
Cortex-R52 is now used as Real Time core
We are now able to use embedded ARM timer
Some IPs are the same and will use existing drivers, sometime with few improvements (UART, GPIO...)
Introduction of new IPs
S4 SoC that is equipping multiple Gen4 boards have some Cortex-A55s and a Cortex-R52 core.
Spider board is the reference board for S4 SoC, developpment oriented, this board is providing a lot of available IOs.
Operation of basic samples such as hello_world and blinky have been validated.

This PR is ready to be reviewed as th last CI failures is a false positive !
pinctrl_soc_pin_t has to be defined as documented here : https://docs.zephyrproject.org/latest/hardware/pinctrl/index.html#pin-control-drivers.

@gmarull @povergoing  can you take a look please?

Just updated the PR with the following changes:

Rebased on main
Removed NMI init in soc.c following #59869
Moved GIC version to DT following #58035
Updated clock driver following #57172 thanks @xakep-amatop for the help
Updated pinctrl implementation for Gen4 SoCs

Updated the PR moving pinctrl private function prototype to a new internal header.

Hi @aaillet
Do you plan to push this? Seems stalled...

Hi @aaillet Do you plan to push this? Seems stalled...

Yes I plan to.
Not stalled, waiting for reviews, I will tag it to be talked about in a dev meeting soon.

Rebased on main
Added gpio key field to board dts as introduced in #61115
Trying to fix CI

Rebased on main
Disabled L1 cache (introducing issues with reading registers values being cached)

Rebased on main

Note for the future: the Zephyr documentation recommends performing a push rebase and making your changes in separate updates Workflow suggestions that help reviewers.

* Disabled L1 cache (introducing issues with reading registers values being cached)

Can you share some details about where you performed disabling and which registers you encountered problems with?

Rebased on main

Note for the future: the Zephyr documentation recommends performing a push rebase and making your changes in separate updates Workflow suggestions that help reviewers.

Thanks for the link, I'll do it that way next time :+1:

* Disabled L1 cache (introducing issues with reading registers values being cached)

Can you share some details about where you performed disabling and which registers you encountered problems with?

Here is how we activated cache for gen4 at first (same way it it done for gen3 socs) : https://github.com/iotbzh/zephyr/commit/96a79f89ae831d56747c09f735a75b4daf26b6df
We encountered problem while working on IPC driver.
Zephyr was not able to read updated master status through libmetal (it was confirmed that ra.
Even if linux was updating the value of this shared register, Zephyr was still reading the old value.
It was fixed instantly by disabling cache.
I removed it of this PR to avoid contagion to other drivers we might not have seen.
We will soon investigate on this topic

Removed a comment in board dts (was not respecting compliance), was useless.

@lorc @gmarull @povergoing can you please take another look ?

I have a small comment regarding SCIF3/HSCIF0 used by Cortex A cluster. If I remember correctly, Renesas BSP moved Cortex A to HSCIF0. At least this is what we use in OP-TEE, U-Boot and Xen.

I think that we agree on this subject.
Physically SCIF3 and HSCIF0 are using the same lines and Cortex A's are using HSCIF0. (HSCIF1 and SCIF0 are on the same line as well)
Zephyr part is only supporting SCIF so I listed physically available lines in the device tree.
As typical use case is that Cortex A's are using HSCIF0, I selected SCIF0 as main Zephyr serial port.
For other use cases, Zephyr may use SCIF3 line.
Please keep in mind that HSCIF0 and SCIF0 are not on the same line, they can work at the same without any issues. (On different physical port)
I hope I answered your comments.

Addressed comments from @fabiobaltieri concerning board documentation and clock driver.

Addressed last comments from @fabiobaltieri concerning clock driver


IPFS for downloading images, using peer-to-peer

Description
It should be possible to upload the images to IPFS, and then use such an address to download them p2p:
ipfs://bafybeig5sch22ecfox7gq724rz7uivydwvnnpuqdcnjz72iwelgtrakzui/ubuntu-24.04-server-cloudimg-amd64.img
CLI: https://docs.ipfs.tech/how-to/kubo-basic-cli/#install-kubo
GUI: https://docs.ipfs.tech/how-to/desktop-app/#install-ipfs-desktop

Implementation itself should be similar to ORAS: #2405
Setting up a hosting network is a completely different story.

Best case scenario, where both images and archives are found in local diskstore:
? Creating an instance "ipfs" Proceed with the current configuration
INFO[0000] Attempting to download the image              arch=x86_64 digest="sha256:32a9d30d18803da72f5936cf2b7b9efcb4d0bb63c67933f17e3bdfd1751de3f3" location="https://cloud-images.ubuntu.com/releases/24.04/release-20240423/ubuntu-24.04-server-cloudimg-amd64.img"
Downloading the image (ubuntu-24.04-server-cloudimg-amd64.img)
453.00 MiB / 453.00 MiB [---------------------------------] 100.00% 340.48 MiB/s
INFO[0002] Used ipfs "bafybeig5sch22ecfox7gq724rz7uivydwvnnpuqdcnjz72iwelgtrakzui"
INFO[0002] Attempting to download the nerdctl archive    arch=x86_64 digest="sha256:2c841e097fcfb5a1760bd354b3778cb695b44cd01f9f271c17507dc4a0b25606" location="https://github.com/containerd/nerdctl/releases/download/v1.7.6/nerdctl-full-1.7.6-linux-amd64.tar.gz"
Downloading the nerdctl archive (nerdctl-full-1.7.6-linux-amd64.tar.gz)
226.46 MiB / 226.46 MiB [---------------------------------] 100.00% 360.66 MiB/s
INFO[0004] Used ipfs "bafybeieipdaxd3fzy3j7syzzxdaqxramk65j7ajzcqgmi6b5jyq4jgbwue"
INFO[0004] Run `limactl start ipfs` to start the instance.

When ipfs (kubo) is not available, or image not found in IPFS, it would use HTTP...

It is still possible to use ipfs:// as the primary, but now using a secondary cid:
images:
# Try to use release-yyyyMMdd image if available. Note that release-yyyyMMdd will be removed after several months.
- location: "https://cloud-images.ubuntu.com/releases/24.04/release-20240423/ubuntu-24.04-server-cloudimg-amd64.img"
  arch: "x86_64"
  digest: "sha256:32a9d30d18803da72f5936cf2b7b9efcb4d0bb63c67933f17e3bdfd1751de3f3"
  cid: bafybeig5sch22ecfox7gq724rz7uivydwvnnpuqdcnjz72iwelgtrakzui
- location: "https://cloud-images.ubuntu.com/releases/24.04/release-20240423/ubuntu-24.04-server-cloudimg-arm64.img"
  arch: "aarch64"
  digest: "sha256:c841bac00925d3e6892d979798103a867931f255f28fefd9d5e07e3e22d0ef22"
  cid: bafybeidyzm3zkljztw3xqqjjapcqzsr4odz4evnmk4kfhdmmgird6citbi
# Fallback to the latest release image.
# Hint: run `limactl prune` to invalidate the cache
- location: "https://cloud-images.ubuntu.com/releases/24.04/release/ubuntu-24.04-server-cloudimg-amd64.img"
  arch: "x86_64"
- location: "https://cloud-images.ubuntu.com/releases/24.04/release/ubuntu-24.04-server-cloudimg-arm64.img"
  arch: "aarch64"

archives:
- location: "https://github.com/containerd/nerdctl/releases/download/v1.7.6/nerdctl-full-1.7.6-linux-amd64.tar.gz"
  arch: "x86_64"
  digest: "sha256:2c841e097fcfb5a1760bd354b3778cb695b44cd01f9f271c17507dc4a0b25606"
  cid: bafybeieipdaxd3fzy3j7syzzxdaqxramk65j7ajzcqgmi6b5jyq4jgbwue
- location: "https://github.com/containerd/nerdctl/releases/download/v1.7.6/nerdctl-full-1.7.6-linux-arm64.tar.gz"
  arch: "aarch64"
  digest: "sha256:77c747f09853ee3d229d77e8de0dd3c85622537d82be57433dc1fca4493bab95"
  cid: bafybeibptqoynryoxytxffgyuqjnmwpuujwp6wlps4spygapuwphzkeieq
# No arm-v7
# No riscv64

That means that it can share the digest (and the cache) with the https:// downloads.

The secret command to calculate the "cid" (Content-ID) is:
 ipfs add --cid-version=1 --only-hash --quieter


K2, VC Lib 3.8

Updates to latest conventions and vclib as of 2024-06-12, which means

Kotlin 2.0
Kotest 5.9.1
Serialization 1.7.1-SNAPSHOT fork with COSE
KMP-Crypto 3.2.0

Needs a new vclib published first

should I just publish a new VC Lib snapshot?


Do we want to use commitlint to lint commits?

I'm observing that some of you are defining the scope in their commits, some not, and others are not even using "fix, feat, chore" etc.
Do we want to use something like commitlint or do you prefer to leave it be?

I'm not a fan of the tool, so I would say no unless there is an overwhelming majority of having it in place.


Add RelativeRect.fromDirectional factory

Added fromDirectional factory to RelativeRect.
When we need to consider both TextDirection.ltr and TextDirection.rtl, this will help us.
RelativeRect.fromDirectional(
    textDirection: textDirection,
    start: 10.0,
    top: 20.0,
    end: 30.0,
    bottom: 40.0,
);

List which issues are fixed by this PR. You must list at least one issue.
https://github.com/flutter/flutter/issues/107058
Pre-launch Checklist

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

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

Thanks for the contribution! :)


Exception not raised inside Linux/Docker container

======================================================================
FAIL: test_rename_nopermissions (tests.Direct.test_Direct_Errors.TestDirectErrors)
Traceback (most recent call last):
File "/tmp/normalize-filename/tests/Direct/test_Direct_Errors.py", line 37, in test_rename_nopermissions
self.invokeDirectly([filename])
AssertionError: Exception not raised

Probably need to not run as root within container.

Closed in 516d6e44.


Update srhub config for delux indicator settings

To match the rest of the available targets. This is confirmed to set the LEDs to green once it's on wifi.

Thanks - I missed that one!


New package: XDesktopSoft.XWidget version 1.0

Pull request has been created with Komac v1.8.2 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111882&drop=dogfoodAlpha

/AzurePipelines run


No module named 'nlp'

I got this when trying to import after installing from master pip install -e ".[dev]" I try to use it on a ipynb and got this:
import torch
from transformers import *
from fastai2.text.all import *

from blurr.data.all import *
from blurr.modeling.all import *

I get
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-906f6e0a84bd> in <module>
      3 from fastai2.text.all import *
      4
----> 5 from blurr.data.all import *
      6 from blurr.modeling.all import *

~/Documentos/github/blurr/blurr/data/all.py in <module>
      1 from ..utils import *
----> 2 from .core import *
      3 from .language_modeling import *
      4 from .question_answering import *
      5 from .token_classification import *

~/Documentos/github/blurr/blurr/data/core.py in <module>
      6 from functools import reduce
      7
----> 8 import torch, nlp
      9 from transformers import *
     10 from fastai2.text.all import *

ModuleNotFoundError: No module named 'nlp'

Well, it is solved with a pip install nlp there is no conda install nlp but I guess it should be listed somewhere?

Yah I'll add nlp to the requirements file.
Will have to try xlm-roberta-base ... weird that it's causing the notebook to restart itself.

Yes, the bad thing is that I dont see a traceback or something, only that, do you need more specific specs or something?

Give me as much info as you can ... GPU(s), running local or colab, etc....  I'll see if I can reproduce.

Running locally with conda with fastai2 and core installed locally from github repo https://gist.github.com/41b8d0413833fd758f499978ebff38a0
Driver Version: 435.21       CUDA Version: 10.1
GeForce RTX 2080 8Gb
3~18.04.1-Ubuntu SMP Thu Jun 4 14:58:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

The nlp not found issue is fixed in the latest master branch.  If you can, please open a separate issue re: the issues with xlm-roberta-base.  Thanks.


Add a systemctl script example to README.md

This script is basically a copy-paste from the one provided here: https://docs.mattermost.com/install/prod-debian.html#set-up-mattermost-server
I'm not a systemctl expert but some option might not be mandatory (like LimitNOFILE).

Coverage remained the same at 89.46% when pulling db922803c2e724aaab965d75d66ac6c7e9ce624f on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling db922803c2e724aaab965d75d66ac6c7e9ce624f on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling db922803c2e724aaab965d75d66ac6c7e9ce624f on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling e55f795d7012402506e9e1a17dc1bf84ed69e8a4 on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling dbe98fcc3104532f56c6d664b303ca174a0f80f7 on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling dbe98fcc3104532f56c6d664b303ca174a0f80f7 on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Coverage remained the same at 89.46% when pulling 5ab4b32d33794242af403184bf14f959e8e89d44 on MartinDelille:patch-1 into a840de8b1697a11cb9eac5db99809bfe7e3c7274 on NotSqrt:master.

Fixed!

Thanks a lot !


4.6.7 introduced node_modules/flatpickr/dist/esm/index.js which is coded in es6 syntax

This is causing syntax error in IE11, as it does not support es6 arrow functions, and there are a lot of them inside node_modules/flatpickr/dist/esm/index.js.
It works in <4.6.6, because it does not have the /esm folder with es6 .js files.
Your Environment

flatpickr version used: ^4.6.7
Browser name and version: Internet Explorer 11
OS and version: Windows 10

This is causing syntax error in IE11, as it does not support es6 arrow functions, and there are a lot of them inside node_modules/flatpickr/dist/esm/index.js.
It works in <4.6.6, because it does not have the /esm folder with es6 .js files.
Your Environment

flatpickr version used: ^4.6.7
Browser name and version: Internet Explorer 11
OS and version: Windows 10

Why are you trying to import from the esm module when using IE11?

You should probably do something like this instead in your index.html (Which I guess you are referring this from)
<script type="module" src="node_modules/flatpickr/dist/esm/index.js"></script>
<script nomodule src="node_modules/flatpickr/dist/flatpickr.js"></script>

All I have included in my project is flatpickr.js through webpack import inside the Vue component that requires flatpickr.
These are the webpack imports (works in <4.6.6)
import flatpickr from "flatpickr";
import { Norwegian } from "flatpickr/dist/l10n/no.js";

I think the problem is
import flatpickr from "flatpickr";
I guess this import includes the esm folder and therefore IE is complaining?

Webpack by default is going to include the "module" file specified in package.json. It uses this to bundle flatpickr into an app's final javascript bundle (main.[some-hash].js).
Webpack prefers the "module" entry in package.json over the "main" entry. It seems to want to use ES2015 modules over any other module format (commonjs, umd, etc.), probably due to the fact that every other module format is harder to tree shake unused code from.
The EcmaScript module format is actually not the issue here - the import/export statements are handled by webpack and all of the code for an app is placed into a single flat file at build time (main.[some-hash].js). The actual issue is the ES6 syntax for the rest of the code inside dist/esm/index.js, which breaks IE at runtime. Essentially, including flatpickr inside a webpack build will cause the entire page to crash in IE due to the unknown ES6 syntax.
I think the ideal would be to have an ESM js file (i.e. ES2015 module format), but have its actual code be ES5 for IE11 compatibility.
Is flatpickr accepting PRs? We're a library that bundles flatpickr inside of it, and it would be ideal for us to get this fixed so that our users don't need complicated webpack workarounds to get flatpickr to work in IE. We would be happy to contribute a fix to restore this automatic IE compatibility.

Nevermind on a PR, I see that this was already merged: https://github.com/flatpickr/flatpickr/pull/2469
Looks like we're just waiting on flatpickr to be released


C: wrong hiliting of string

1st str not hilited correctly:
#error "dddd \
    edddd \
    "
#warning This is a short warning
/* <- keyword.control.import */
#endif
 /* <- keyword.control.import */

👋 @Alexey-T. Thanks for filing these bug reports with the syntax highlighting.
All of our syntax highlighting on GitHub is controlled by upstream dependencies (see https://github.com/github/linguist/tree/master/vendor/grammars) so you'll need to report the problems on these repositories, not here.
When grammars are updated we automatically incorporate the updates in Linguist during our build process.

@Alexey-T For C, github.com uses textmate/c.tmbundle. You might want to check first that there isn't already an issue for that.


Better discoverability for Python application error troubleshooting

Startup errors are a very common issue when deploying to app service due to missing dependencies, incorrect configuration, syntax errors, etc.
This PR is a two-part change intended to increase the discoverability of this common issue that users will typically experience at some point when deploying a Python app on App Service:

Move WSGI_LOG instructions to an include so it can be shown on both the open-source FAQ for Web Apps (as it is today) as well as Python web app configuration tutorial
Include the error message displayed by App Service message in the documentation for better discoverability. Currently, searching for Google or Bing for python app service "The page cannot be displayed because an internal server error has occurred." returns no first-party docs.

@stewartadam : Thanks for your contribution! The author, @huguesv, has been notified to review your proposed change.

#sign-off

@cephalin: I'm sorry - only the author of this article, @huguesv, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@stewartadam Sorry for the delay. Can you please change the author metadata to me at the top of the source? cephalin. That way I can sign off. Thank you.

Hi @cephalin looks like this got merged, any action required from me still? Thanks!

@stewartadam No you're golden. Thanks for your contribution!


Allow edit survey and report settings for open surveys

Issue #: https://github.com/bcgov/epic-engage/issues/19
Description of changes:

Added "Edit Survey" and "Edit Settings" in the action drop-down for Open Surveys.
Changed the button on the survey builder from "Save & Continue" to "Report Settings".
Added a "Back" button to the report Settings page to return to the survey builder.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #37 (7a5284a) into main (a44f7f9) will increase coverage by 0.00%.
The diff coverage is 85.71%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   68.44%   68.44%
=======================================
  Files         472      472
  Lines       15624    15628    +4
  Branches     1194     1196    +2
=======================================
+ Hits        10694    10697    +3
- Misses       4721     4722    +1
  Partials      209      209

Flag
Coverage Δ

metweb
61.49% <85.71%> (+<0.01%)
:arrow_up:

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

Files
Coverage Δ

.../src/components/survey/listing/ActionsDropDown.tsx
64.28% <100.00%> (+0.64%)
:arrow_up:

...-web/src/components/survey/report/SettingsForm.tsx
85.71% <80.00%> (-2.29%)
:arrow_down:


No module named 'configs'

Hi, i have some problems when clone easy_ViTPose and pip install -e .
Please help or fix that.
Thanks regards

Hello, thanks! should work now, pull last commit. I introduced a discrepancy with relative and absolute import from previous changes.
lmk if it works for you

It works, thank you so much.


Fix saucelabs link

Fixes a typo on http://testarmada.io/#enableAnExecutor in the executor grid above that Enable an Executor heading.

Thanks for the PR @tedpennings


Support OS X and cover it in Continuous Integration

Support OS X and cover it in Continuous Integration: (1) adjust build infrastructure for OS X, and (2) enable no-fail mode in Travis-CI. Currently there are only tests using the Clang/LLVM compiler suite (clang++ and clang) along with the GNU Fortran compiler, but there is no OS X support.

OSX (and FreeBSD as a bonus) are now supported. For the Linux side this exercise unveiled a problem with LIBXSMM when LD_PRELOADed as a BLAS replacement (a resolution will arrive with v1.2). OSX support has been tested in a virtualized environment with the necessary CL tools installed, plus the CI tests are also passing with Travis (OSX Mavericks). The FreeBSD support has been tested with FreeBSD 10 which was lacking GNU Make (sudo pkg install gmake); a few exotic scripts in the "samples" folder may require additional porting (Bash -> Sh), others may need to run with "env MAKE=gmake ./script.sh".


Add aliases to plugin and templates subcommands for plurality

Having aliases for the singular and plural of plugin/s and template/s subcommands would help reduce user error.
See https://docs.rs/clap/latest/clap/struct.Arg.html#method.alias

Hi, can I work on this one?

@raymundovr Sure - thanks!

@itowlson have submitted a PR, let me know if that's a correct approach.

I think it depends.  Were you thinking of sending another PR to change the 'default' plurality @raymundovr?  If so, then there's little point raising a new issue for 'part 2' (unless that would be useful to you).  But if we're parking this as 'done for now' - which is 100% reasonable, as your merged contribution fulfils what was asked for - then we should close.

hi @itowlson, just wanted to corroborate after the PR comments, up to you.

Cool, I'll close this - thanks!


fix: keep search type param when removing filter

Description
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Sports-Helsinki branch is deployed to platta: https://liikunta-pr583.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr583.dev.hel.ninja :laughing::tada::tada::tada:


Issue regarding npm run dev

When i m trying to run the dapp,an error is being thrown.
I'm new to this solidity stuff,and trying to learn through this pet-shop-box.
Can anyone help me?
Here is the error-:
Browsersync] Serving files from: ./src
[Browsersync] Serving files from: ./build/contracts
[Browsersync] Watching files...
events.js:182
throw er; // Unhandled 'error' event
^
Error: watch /home/hrishabh/crypto-pet/migrations/1_initial_migration.js ENOSPC
at _errnoException (util.js:1026:11)
at FSWatcher.start (fs.js:1383:19)
at Object.fs.watch (fs.js:1409:11)
at createFsWatchInstance (/home/hrishabh/crypto-pet/node_modules/chokidar/lib/nodefs-handler.js:37:15)
at setFsWatchListener (/home/hrishabh/crypto-pet/node_modules/chokidar/lib/nodefs-handler.js:80:15)
at FSWatcher.NodeFsHandler._watchWithNodeFs (/home/hrishabh/crypto-pet/node_modules/chokidar/lib/nodefs-handler.js:232:14)
at FSWatcher.NodeFsHandler._handleFile (/home/hrishabh/crypto-pet/node_modules/chokidar/lib/nodefs-handler.js:259:21)
at FSWatcher. (/home/hrishabh/crypto-pet/node_modules/chokidar/lib/nodefs-handler.js:480:21)
at FSReqWrap.oncomplete (fs.js:154:5)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: lite-server
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/hrishabh/.npm/_logs/2018-12-27T17_56_55_600Z-debug.log
hrishabh@hrishabh-HP-Pavilion-15-Notebook-PC:~/crypto-pet$ npm run serve
npm ERR! missing script: serve
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/hrishabh/.npm/_logs/2018-12-27T17_57_30_778Z-debug.log

I'm closing this issue as it was not related to truffle suite,another program was trying to suppress all ports in my linux and now that issue is solved.


Return all ajax parameters by default

This commit resolves issue #3548

Just a note that I haven't forgotten this pull request. It's currently queued up to be merged, I just need to write some tests that cover the AJAX adapter (and this PR as a result).


BlobNotFound

Attempting to download any camera segment or log from Connect to PC results in this message. (after all files have been requested and uploaded to the servers)
'''
This XML file does not appear to have any style information associated with it. The document tree is shown below.
Error
Code>BlobNotFound</Code
<MessageThe specified blob does not exist. RequestId:6e412e96-401e-0011-2da9-ffa143000000 Time:2022-01-02T07:23:07.5523848Z

'''

can you post a route where this happens?

can you post a route where this happens?

59755a6574d9ac67|2021-12-31--13-49-02--0
https://cabana.comma.ai/?route=59755a6574d9ac67|2021-12-31--13-49-02&exp=1672878023&sig=7pfj4nDKMxDV%2BMoxJFTOtdB8%2FF1UgbZ1aHvyQ8JnpyM%3D&max=17&url=https%3A%2F%2Fchffrprivate.azureedge.net%2Fchffrprivate3%2Fv2%2F59755a6574d9ac67%2F9ddbaade21dd594a5acda80580ff125d_2021-12-31--13-49-02

This is a known bug where uploading to a preserved route doesn't work. Unpreserving and then preserving the route again should fix this, if the data is still available.


HSEARCH-2188 Fixed type checking in NumericFieldUtilsTest

isAssignableFrom in NumericFieldUtilsTest was used the wrong way. I turned it around and added a unit test, which did not pass with the previous implementation.

What's the difference between this and #1046 ? Please close one.

There's a patch for branch 5.3 and one for branch 5.4

Jenkins, retest this please

OK, so I pushed the test and the cleanup to master a few weeks ago and backported the original Davide's fix to 5.3 and 5.4 just now.
Closing. Thanks!


nushell: 0.99.1 -> 0.100.0

Changelog
I also updated the nushellPlugins.net updateScript because it uses tagged releases.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 355657

x86_64-linux

:white_check_mark: 6 packages built:

nushell
nushellPlugins.formats
nushellPlugins.gstat
nushellPlugins.polars
nushellPlugins.query
nushellPlugins.units


Fixed list_leverage_trades()

Fixed problem with list_leverage_trades() (it didn't return anything).
Before:
res = client.list_leverage_trades()
print(res)  # None
print(res(type))  # <class 'NoneType'>

After:
res = client.list_leverage_trades()
print(res)  # {'positions': []}
print(res(type))  # <class 'dict'>

Thanks for noticing. Merged

BTW, updated version at PyPI


Grohe have changed there endpoint

Old endpoint is using a revoked certificate:
https://idp-apigw.cloud.grohe.com/v3/iot/
New endpoint seems to be:
https://idp2-apigw.cloud.grohe.com/v3/iot/
Judging from info in: https://github.com/FlorianSW/grohe-ondus-api-java/blob/master/src/main/java/org/grohe/ondus/api/OndusService.java

BIG Thank you for pointing that out! Then I can reverse all the ugly code to set up a bespoke SSL context to trust Grohe's old certificate.

BIG Thank you for pointing that out! Then I can reverse all the ugly code to set up a bespoke SSL context to trust Grohe's old certificate.

No problem.
I’m just happy I didn’t screw anything up, this is my fist pull request :)
I’m working on getting my Grohe Blue Home chilled water tap working with your integration (that’s the reason I looked at the OpenHAB/Java code).
But I’m not a programmer and no nothing about Python so it’s a bit of a learning experience for me :)
I have implemented the basics in a shell script (because that what I know :)).
A request ${BASE_URL}/locations/${LOCATION_ID}/rooms/${ROOM_ID}/appliances/${APPLIANCE_ID} returns:
[
{
"appliance_id": "ec613805-7fea-4a3d-861e-7df400a42beb",
"installation_date": "2020-12-08T22:13:32.000+00:00",
"name": "MY U-SPOUT",
"serial_number": "305a31303123456789012345678901234567890123456789",
"type": 104,
"version": "01.00.Z10.0300.0104",
"tdt": "2021-01-19T13:35:30.000+01:00",
"timezone": 60,
"config": {
"co2_type": 1,
"hose_length": 110,
"co2_consumption_medium": 48,
"co2_consumption_carbonated": 65,
"guest_mode_active": true,
"auto_flush_active": true,
"flush_confirmed": false,
"f_parameter": 3,
"l_parameter": 0,
"flow_rate_still": 16,
"flow_rate_medium": 25,
"flow_rate_carbonated": 16
},
"role": "owner",
"registration_complete": true,
"presharedkey": “XXXXXXXXXXXXXXXXXXXXXXXXXXXX=",
"params": {
"water_hardness": 0,
"carbon_hardness": 20,
"filter_type": 1,
"variant": 3,
"auto_flush_reminder_notif": true,
"consumables_low_notif": true,
"product_information_notif": true
},
"error": {
"errors_1": false,
"errors_2": false,
"errors_3": false,
"errors_4": false,
"errors_5": false,
"errors_6": false,
"errors_7": false,
"errors_8": false,
"errors_9": false,
"errors_10": false,
"errors_11": false,
"errors_12": false,
"errors_13": false,
"errors_14": false,
"errors_15": false,
"errors_16": false,
"error1_counter": 0,
"error2_counter": 0,
"error3_counter": 0,
"error4_counter": 0,
"error5_counter": 0,
"error6_counter": 0,
"error7_counter": 0,
"error8_counter": 0,
"error9_counter": 0,
"error10_counter": 0,
"error11_counter": 0,
"error12_counter": 6656,
"error13_counter": 0,
"error14_counter": 0,
"error15_counter": 0,
"error16_counter": 0
},
"state": {
"start_time":<PHONE_NUMBER>,
"APPLIANCE_SUCCESSFUL_CONFIGURED": false,
"co2_empty": false,
"co2_20l_reached": false,
"filter_empty": false,
"filter_20l_reached": false,
"cleaning_mode_active": false,
"cleaning_needed": false,
"flush_confirmation_required": false,
"System_error_bitfield": 0
}
}
]
Does that look like the output from the Sense Guard? Or is it very different?
Any possibility you have an example from the Sense Guard? That would be a huge help to me :)
Med Venlig Hilsen
Morten Vinding
Sonnerupvej 69
2700 Brønshøj
+45 24 47 01 69

On 19 Jan 2021, at 13.49, Gunnar Kreitz<EMAIL_ADDRESS>wrote:
BIG Thank you for pointing that out! Then I can reverse all the ugly code to set up a bespoke SSL context to trust Grohe's old certificate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/gkreitz/homeassistant-grohe_sense/pull/11#issuecomment-762819763, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJCB62OUSOVVDLAK5MH77DS2V5TZANCNFSM4WIUYTGQ.

No problem.
I’m just happy I didn’t screw anything up, this is my fist pull request :)
I’m working on getting my Grohe Blue Home chilled water tap working with your integration (that’s the reason I looked at the OpenHAB/Java code).
But I’m not a programmer and no nothing about Python so it’s a bit of a learning experience for me :)
I have implemented the basics in a shell script (because that what I know :)).
A request ${BASE_URL}/locations/${LOCATION_ID}/rooms/${ROOM_ID}/appliances/${APPLIANCE_ID} returns:
[
{
"appliance_id": "ec613805-7fea-4a3d-861e-7df400a42beb",
"installation_date": "2020-12-08T22:13:32.000+00:00",
"name": "MY U-SPOUT",
"serial_number": "305a31303123456789012345678901234567890123456789",
"type": 104,
"version": "01.00.Z10.0300.0104",
"tdt": "2021-01-19T13:35:30.000+01:00",
"timezone": 60,
"config": {
"co2_type": 1,
"hose_length": 110,
"co2_consumption_medium": 48,
"co2_consumption_carbonated": 65,
"guest_mode_active": true,
"auto_flush_active": true,
"flush_confirmed": false,
"f_parameter": 3,
"l_parameter": 0,
"flow_rate_still": 16,
"flow_rate_medium": 25,
"flow_rate_carbonated": 16
},
"role": "owner",
"registration_complete": true,
"presharedkey": “XXXXXXXXXXXXXXXXXXXXXXXXXXXX=",
"params": {
"water_hardness": 0,
"carbon_hardness": 20,
"filter_type": 1,
"variant": 3,
"auto_flush_reminder_notif": true,
"consumables_low_notif": true,
"product_information_notif": true
},
"error": {
"errors_1": false,
"errors_2": false,
"errors_3": false,
"errors_4": false,
"errors_5": false,
"errors_6": false,
"errors_7": false,
"errors_8": false,
"errors_9": false,
"errors_10": false,
"errors_11": false,
"errors_12": false,
"errors_13": false,
"errors_14": false,
"errors_15": false,
"errors_16": false,
"error1_counter": 0,
"error2_counter": 0,
"error3_counter": 0,
"error4_counter": 0,
"error5_counter": 0,
"error6_counter": 0,
"error7_counter": 0,
"error8_counter": 0,
"error9_counter": 0,
"error10_counter": 0,
"error11_counter": 0,
"error12_counter": 6656,
"error13_counter": 0,
"error14_counter": 0,
"error15_counter": 0,
"error16_counter": 0
},
"state": {
"start_time":<PHONE_NUMBER>,
"APPLIANCE_SUCCESSFUL_CONFIGURED": false,
"co2_empty": false,
"co2_20l_reached": false,
"filter_empty": false,
"filter_20l_reached": false,
"cleaning_mode_active": false,
"cleaning_needed": false,
"flush_confirmation_required": false,
"System_error_bitfield": 0
}
}
]
Does that look like the output from the Sense Guard? Or is it very different?
Any possibility you have an example from the Sense Guard? That would be a huge help to me :)
Med Venlig Hilsen
Morten Vinding
Sonnerupvej 69
2700 Brønshøj
+45 24 47 01 69

On 19 Jan 2021, at 13.49, Gunnar Kreitz<EMAIL_ADDRESS>wrote:
BIG Thank you for pointing that out! Then I can reverse all the ugly code to set up a bespoke SSL context to trust Grohe's old certificate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/gkreitz/homeassistant-grohe_sense/pull/11#issuecomment-762819763, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJCB62OUSOVVDLAK5MH77DS2V5TZANCNFSM4WIUYTGQ.

No problem. I’m just happy I didn’t screw anything up, this is my fist pull request :) I’m working on getting my Grohe Blue Home chilled water tap working with your integration (that’s the reason I looked at the OpenHAB/Java code). But I’m not a programmer and no nothing about Python so it’s a bit of a learning experience for me :) I have implemented the basics in a shell script (because that what I know :)).

It looked like a perfectly fine pull request, but that first one sure can be scary! :)
The protocol example for the Blue you show looks relatively different compared to the protocol for the other devices (or, at least the parts of the protocol I've implemented here). On a positive note - the protocol you show looks pretty simple, at least if one simply wants to build sensors from that data.
I'm not sure if this is the easiest component to start with if you're new to programming. I've written this code to make use of python's async features, which may be a new stumbling block. That said, some rambling guidance for roughly how I would start if I were to try to implement support for Blue:
My code will scan for appliances, putting all of them in an array in hass.data[DOMAIN]['devices'], https://github.com/gkreitz/homeassistant-grohe_sense/blob/master/init.py#L60 . Note that the last request there is to ${BASE_URL}/locations/${LOCATION_ID}/rooms/${ROOM_ID}/appliances/ (which returns all the appliances in a room). You may want to check that the type returned there for a Blue is 104 (as in your reply), I saw some worrying indications in https://github.com/FlorianSW/grohe-ondus-api-java/issues/12 that they may put type as 0 in that array (seems like a bug on Grohe's end).
For the Blue device, you'll likely want to deviate a bit from the code structure I have in place. In this code, I do a pattern where I have one GroheSenseGuardReader class (fetching data periodically), and then classes for the various styles of sensors (the easiest being GroheSenseSensorEntity). This structure is to keep down polling (1 request/appliance, instead of 1 per sensor), but when you're prototyping it may be easier to try to write something similar to GroheSenseSensorEntity directly - making a request to the correct url in async_update and storing the measurement as a variable in the object, returning it in the state method.
On https://github.com/gkreitz/homeassistant-grohe_sense/blob/master/sensor.py#L68 , you'd then want an if statement to check if the device is a Blue device (using the type above) and have all my code as an else, creating an instance of your new sensor class for a Blue device.
Do you know what sort of functionality you'd want in home assistant for the Blue? Are there values in the response you showed that you want to have as sensors? Or are you more trying to build commands for something?

No problem. I’m just happy I didn’t screw anything up, this is my fist pull request :) I’m working on getting my Grohe Blue Home chilled water tap working with your integration (that’s the reason I looked at the OpenHAB/Java code). But I’m not a programmer and no nothing about Python so it’s a bit of a learning experience for me :) I have implemented the basics in a shell script (because that what I know :)).

It looked like a perfectly fine pull request, but that first one sure can be scary! :)
The protocol example for the Blue you show looks relatively different compared to the protocol for the other devices (or, at least the parts of the protocol I've implemented here). On a positive note - the protocol you show looks pretty simple, at least if one simply wants to build sensors from that data.
I'm not sure if this is the easiest component to start with if you're new to programming. I've written this code to make use of python's async features, which may be a new stumbling block. That said, some rambling guidance for roughly how I would start if I were to try to implement support for Blue:
My code will scan for appliances, putting all of them in an array in hass.data[DOMAIN]['devices'], https://github.com/gkreitz/homeassistant-grohe_sense/blob/master/init.py#L60 . Note that the last request there is to ${BASE_URL}/locations/${LOCATION_ID}/rooms/${ROOM_ID}/appliances/ (which returns all the appliances in a room). You may want to check that the type returned there for a Blue is 104 (as in your reply), I saw some worrying indications in https://github.com/FlorianSW/grohe-ondus-api-java/issues/12 that they may put type as 0 in that array (seems like a bug on Grohe's end).
For the Blue device, you'll likely want to deviate a bit from the code structure I have in place. In this code, I do a pattern where I have one GroheSenseGuardReader class (fetching data periodically), and then classes for the various styles of sensors (the easiest being GroheSenseSensorEntity). This structure is to keep down polling (1 request/appliance, instead of 1 per sensor), but when you're prototyping it may be easier to try to write something similar to GroheSenseSensorEntity directly - making a request to the correct url in async_update and storing the measurement as a variable in the object, returning it in the state method.
On https://github.com/gkreitz/homeassistant-grohe_sense/blob/master/sensor.py#L68 , you'd then want an if statement to check if the device is a Blue device (using the type above) and have all my code as an else, creating an instance of your new sensor class for a Blue device.
Do you know what sort of functionality you'd want in home assistant for the Blue? Are there values in the response you showed that you want to have as sensors? Or are you more trying to build commands for something?

Thank you Gunnar,
My Blue is returning 104 as device id, so no problems there :)
Hmm I’m probably not gonna understand all of your code anytime soon anyway, so my plan is mostly to modify it to work reasonably with the Blue :)
I have implemented some of it already in the switch.py just copying the GroheSenseGuardValve class and adjusting it to GroheBlueHomeTap :)
I’m mostly after commands that’s why it is the switch.py I modified. The Blue has a “tap” command:
"{
  \"command\": {
    \"tap_type\": $1,
    \"tap_amount\": $2
  }
}"

So my plan is to make a switch panel to fill up a glass or a carafe :)
But so far all I have managed is to set a hardcoded value, so when I flip the switch in Home Assistant, the tap runs for that value.
Obesely I need to modify that to accept integers instead of a boolean :)
Still some way to go, but I’m learning :)
Taping 20ml of water (minimum setting just for test):
https://user-images.githubusercontent.com/1188091/105553633-06311900-5d06-11eb-9636-47444f89f10b.MOV
Thank you so much for your help :)
I will keep you posted on my progress

Thank you Gunnar,
My Blue is returning 104 as device id, so no problems there :)
Hmm I’m probably not gonna understand all of your code anytime soon anyway, so my plan is mostly to modify it to work reasonably with the Blue :)
I have implemented some of it already in the switch.py just copying the GroheSenseGuardValve class and adjusting it to GroheBlueHomeTap :)
I’m mostly after commands that’s why it is the switch.py I modified. The Blue has a “tap” command:
"{
  \"command\": {
    \"tap_type\": $1,
    \"tap_amount\": $2
  }
}"

So my plan is to make a switch panel to fill up a glass or a carafe :)
But so far all I have managed is to set a hardcoded value, so when I flip the switch in Home Assistant, the tap runs for that value.
Obesely I need to modify that to accept integers instead of a boolean :)
Still some way to go, but I’m learning :)
Taping 20ml of water (minimum setting just for test):
https://user-images.githubusercontent.com/1188091/105553633-06311900-5d06-11eb-9636-47444f89f10b.MOV
Thank you so much for your help :)
I will keep you posted on my progress

Hello MortenVinding can you post your final code to get running the Grohe Blue tab.
I would like to integrate my Grohe Blue as well. Thanks in advance for your help.


fail to add br1 with ovs-vsctl “Bridge does not contain a column whose name matches "p4"”

Hello, sorry to bother you, but I had a problem not finding a solution. I've had this problem after following demo.
ovs-vsctl add-br br1 -- set bridge br0 datapath_type=ubpf p4=true
ovs-vsctl: Bridge does not contain a column whose name matches "p4"

but i've already install the p4c, PI and p4-ovs,  is it because there is any configuration I did't do?

Hello, sorry for commenting here, but I'm trying to run p4-ovs demo and I'm not able to do it, when I run: ovs-vsctl add-br br1 -- set bridge br0 datapath_type=ubpf p4=true it stays waiting for something forever.
Can you explain how you intalled it?
Hundred thanks

I also meet the problem that "ovs-vsctl: Bridge does not contain a column whose name matches "p4" ' how to resolve it? thanks too much

actually, I haven't solved this problem, I doubt that this project has some issues that not be fixed. btw, I noticed that 3 years ago this project stopped updating, so maybe you can find some other way to achieve the p4 in OVS. From what I know is that some people program the P4-OVS using VPP.


Don't build in the repository for profile mode

Addresses #129

Done - needs testing/review (particularly on Windows)

Merging to test on dev-development


NL: the month starts with a capital letter, but it shouldn't.

Describe the bug
Month starts with a capital letter but it shouldn't.
To Reproduce
Steps to reproduce the behavior:

Set phone's language to Dutch
Refresh the weather
Look at date
Month starts with a capital letter but it shouldn't.

Expected behavior
"11 mei" instead of "11 Mei"
Screenshots

Smartphone (please complete the following information):

Device: Huawei ANE-LX1
OS Version: Android 9
App Version (that you can find inside the advanced settings) 2.0.8 (87)

Fixed


add suricata project

change lto to fast may affect exec speed for pcap-aware targets. @apach301  FYI.


Available Authorizations modal is empty with v5 OpenIdConnect configuration

Running .Net Core 2.2 and Swashbuckle.AspNetCore 5.0.0-rc2.
Trying to setup OIDC configuration, but the Available Authorizations modal is blank, so cannot proceed with authentication.
Configuration as follows:
services.AddSwaggerGen(c => {
...
c.AddSecurityDefinition("token", new OpenApiSecurityScheme {
    Type = SecuritySchemeType.OpenIdConnect,
    OpenIdConnectUrl = new Uri($"https://login.microsoftonline.com/{tenant}/v2.0/.well-known-openid-configuration")
});
c.AddSecurityRequirement(new OpenApiSecurityRequirement {
    {
        new OpenApiSecurityScheme {
            Reference = new OpenApiReference {Type = ReferenceType.SecurityScheme, Id = "token"
        }, new string[] { }
    }
});
...
});

services.UseSwaggerUI(c => {
...
c.OAuthClientId(_clientId);
...
});

@cjamesrohan I've managed to populate the Available Authorization pop up with the required action items. My Startup.cs file looks as follows:
ConfigureServices method
...
 services.AddSwaggerGen(c =>
            {
                c.AddSecurityDefinition("oauth2", new OpenApiSecurityScheme
                {
                    Name = "oauth2",
                    Type = SecuritySchemeType.OAuth2,
                    Scheme = IdentityServerAuthenticationDefaults.AuthenticationScheme,
                    Flows = new OpenApiOAuthFlows()
                    {
                        Implicit = new OpenApiOAuthFlow()
                        {
                            Scopes = new Dictionary<string, string>
                     {
                         { _scope, _scope }
                     },
                            AuthorizationUrl = new System.Uri($"{_oAuthAuthority}/connect/authorize"),
                            TokenUrl = new System.Uri($"{_oAuthAuthority }/connect/token")
                        }
                    }
                });

                c.OperationFilter<AssignOAuth2SecurityRequirements>();
                c.OperationFilter<SwaggerControllerFilter>();
                c.SwaggerDoc("v1", new OpenApiInfo { Title = $"API (v1)", Version = "v1" });
            });
...

where AssignOAuth2SecurityRequirements Operation filter class is:
public class AssignOAuth2SecurityRequirements : IOperationFilter
    {
        public void Apply(OpenApiOperation operation, OperationFilterContext context)
        {
            // Determine if the operation has the [Authorize] attribute
            var actionAttributes = context.ApiDescription.ActionDescriptor as ControllerActionDescriptor;
            var authorizeAttributes = context.ApiDescription.CustomAttributes()
            .Union(actionAttributes.MethodInfo.CustomAttributes)
            .OfType<AuthorizeAttribute>();

            if (!authorizeAttributes.Any())
                return;

            if (!operation.Responses.ContainsKey("401"))
            {
                // Indicate that their could be a 401 response
                operation.Responses.Add("401", new OpenApiResponse { Description = "Unauthorized" });
            }

            // Initialize the operation.security property if it hasn't already been
            operation.Security ??= new List<OpenApiSecurityRequirement>();

            var oAuthRequirements = new OpenApiSecurityRequirement
            {
                {
                    new OpenApiSecurityScheme
                    {
                        Reference = new OpenApiReference
                        {
                            Type = ReferenceType.SecurityScheme,
                            Id = "oauth2"
                        }
                    },
                    new[] { "api1" }
                }
            };

            operation.Security.Add(oAuthRequirements);
        }
    }

Note:

Reference object inside OpenApiSecurityRequirement should refer to the name of Security scheme you define inside services.AddSwaggerGen.AddSecurityDefinition method (as shown above)
I'm implementing OAuth2 implicit flow.

I hope this helps.

@dev-victory thank you so much for your reply. However, my issue is with SecuritySchemeType.OpenIdConnect and setting the OpenIdConnectUrl. I can get the popup to work fine with SecuritySchemeType.OAuth2 and ImplicitFlow, but this does not satisfy my requirements. We are running strictly on OIDC.

Have same issue.

We have the same issue.

We have the same issue.
.net core 3.0.0 with Swashbuckle.AspNetCore 5.0.0-rc4

Same issue here.
.net core 3.0.0 with Swashbuckle.AspNetCore 5.0.0-rc4

It looks like OpenIDConnect is not supported in the Swagger UI that Swashbuckle takes a dependency on: https://github.com/swagger-api/swagger-ui/issues/3517
And it does not look like it's on the roadmap either: https://github.com/swagger-api/swagger-ui/issues/5473

Any update on this?

I met the same issue before and resolved it.
Now the available Authorization header works fine.
Please check my latest sample using SwashBuckle v5.5.1 and netcore 3.1
https://github.com/capcom923/MySwashBuckleSwaggerWithJwtToken

@capcom923 We are hoping to get it to work with SecuritySchemeType.OpenIdConnect which your sample doesn't resolve at all.

From the Swagger docs

OIDC is currently not supported in Swagger Editor and Swagger UI. Please follow this issue for updates

And here's the related issue in the swagger-ui repo:
https://github.com/swagger-api/swagger-ui/issues/3517

From the Swagger docs

OIDC is currently not supported in Swagger Editor and Swagger UI. Please follow this issue for updates

And here's the related issue in the swagger-ui repo:
swagger-api/swagger-ui#3517

I also have an open ticket somewhere for swagger-ui. Thanks for your response @domaindrivendev! Whenever this gets implemented by them, is there a plan to finish implementation in Swashbuckle? Just curious.

We are blocked by the same issue, so every information on the plans of introducing the fix would be greatly appreciated,

From the Swagger docs

OIDC is currently not supported in Swagger Editor and Swagger UI. Please follow this issue for updates

And here's the related issue in the swagger-ui repo:
swagger-api/swagger-ui#3517

I also have an open ticket somewhere for swagger-ui. Thanks for your response @domaindrivendev! Whenever this gets implemented by them, is there a plan to finish implementation in Swashbuckle? Just curious.

please link the ticket

OpenID Connect Discovery is now supported in Swagger UI v. 3.38.0.

@cjamesrohan @hkosova FYI https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/1941

@cjamesrohan - the latest master code now pulls in v3.38.0 of the swagger-ui and it seems that version out-of-the-box support for OIDC. Don't have the official Swashbuckle release out yet but you can pull down the latest preview package from myget.org and try it out. Let me know if this resolves your issue.

@cjamesrohan - the latest master code now pulls in v3.38.0 of the swagger-ui and it seems that version out-of-the-box support for OIDC. Don't have the official Swashbuckle release out yet but you can pull down the latest preview package from myget.org and try it out. Let me know if this resolves your issue.

@domaindrivendev - this does seem to let me authenticate appropriately, although it appears to populate the token with the 'access_token', and I don't see where we can configure it to use 'id_token'. At the moment, id_token is what our internal api's are expecting. Outside of that, I believe the OIDC authentication is working!

@domaindrivendev - this does seem to let me authenticate appropriately, although it appears to populate the token with the 'access_token', and I don't see where we can configure it to use 'id_token'. At the moment, id_token is what our internal api's are expecting. Outside of that, I believe the OIDC authentication is working!

@cjamesrohan

I don't see where we can configure it to use 'id_token'.

The security scheme needs to include the x-tokenName: id_token extension. OpenAPI YAML example: https://github.com/swagger-api/swagger-ui/issues/4084

@cjamesrohan

I don't see where we can configure it to use 'id_token'.

The security scheme needs to include the x-tokenName: id_token extension. OpenAPI YAML example: https://github.com/swagger-api/swagger-ui/issues/4084

@hkosova - great callout, thanks! I've tried implementing that, but still have the same results. See the comment here:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/1235#issuecomment-671190428

@hkosova - great callout, thanks! I've tried implementing that, but still have the same results. See the comment here:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/1235#issuecomment-671190428

Is there an example I can follow on how to configure Swashbuckle to use OIDC?

Is there an example I can follow on how to configure Swashbuckle to use OIDC?

@Alex-Torres This isn't a working sample, but maybe somebody could point out if there is an issue with the following securityScheme, and then from there use that as a working example? It seems like the problem is with SwaggerUI , rather than Swashbuckle
"securitySchemes": {
      "oidc": {
        "type": "openIdConnect",
         // IdentityServer4
        "openIdConnectUrl": "https://localhost:5000/.well-known/openid-configuration",
      }
    }

The openApi.json file is being generated by the Swashbuckle.AspNetCore package, but the SwaggerUI project is running using the 3.43.0 docker image, in an attempt to isolate the SwaggerUI project with the latest OpenId connect support
docker run -it -p 80:8080 -e API_URL="http://localhost:5003/swagger/v1/swagger.json" swaggerapi/swagger-ui:v3.43.0

I had a similar issue where "Available authorizations" was appearing empty when using openIdConnect. After some frustrating debugging, I discovered it was a CORS issue. My auth server is on a different domain so when swagger-ui attempts to fetch the /.well-known/openid-configuration it is blocked by the CORS policy. Try loading swagger-ui with the developer console open and check for errors. Firefox seems better at reporting CORS issues than Chrome.

@cdowney, can you share the JSON spec you used? I have a similar issue to @FinHorsley. Nothing shows in the modal, and there are no JS/CORS errors reported. There are no requests made to my authorization server at all, so I'm thinking SwaggerUI is ignoring the scheme entirely!

How do I set up swaggerGen to use OpenID Connect Discovery?  I can't find any documentation on how to do so.  So far I have this:
services.AddSwaggerGen(c =>
{
             OpenApiInfo apiInfo = new(){ /* ... */ };

     c.SwaggerDoc("v1", apiInfo);

    c.AddSecurityDefinition("AccountsOpenID", new OpenApiSecurityScheme
    {
        Type = SecuritySchemeType.OpenIdConnect,
        OpenIdConnectUrl = new Uri($"{Configuration["AzureAd:Instance"]}{Configuration["AzureAd:TenantId"]}/.well-known/openid-configuration"),

    });

    c.OperationFilter<MethodNeedsAuthorizationFilter>(); // puts auth UI on the right actions

Where MethodNeedsAuthorizationFilter is...
public class MethodNeedsAuthorizationFilter : IOperationFilter
{
    public void Apply(OpenApiOperation operation, OperationFilterContext context)
    {
        if (operation is null) throw new ArgumentNullException(nameof(operation));
        if (context is null) throw new ArgumentNullException(nameof(context));

        object[] methodAttributes = context.MethodInfo.GetCustomAttributes(true);

        bool needsAuth =
            methodAttributes.OfType<AuthorizeAttribute>().Any()
            || (context.MethodInfo.DeclaringType != null
                && context.MethodInfo.DeclaringType.GetCustomAttributes(true).OfType<AuthorizeAttribute>().Any()
                && !methodAttributes.OfType<AllowAnonymousAttribute>().Any());

        if (needsAuth)
        {

            OpenApiSecurityScheme scheme = new()
            {
                Reference = new OpenApiReference { Type = ReferenceType.SecurityScheme, Id = "oauth2" },
                Scheme = "AccountsOpenID",
                Name = "AccountsOpenID",
                In = ParameterLocation.Header,
            };

            operation.Security = new List<OpenApiSecurityRequirement>()
            {
                new () { {scheme, new List<string>() } }
            };
        }
    }
}

Added a question here: https://stackoverflow.com/questions/72187874/how-do-i-set-up-swashbuckle-and-swagger-ui-to-authorize-using-open-id-connect-di


Reserved Compressed instructions in Decoder tests

Missing tests for verifying reserved compressed instructions in the tests for the decoder.

This test plugin generates assemble file(s) with the reserved compressed instructions.
Closing this issue.


Flakey test observed in devfile describing urls

/kind bug

What versions of software are you using?
 Can you please provide failure logs or links to failed runs?
This is not enough information.
/triage needs-information

This issue is based on the report https://gist.github.com/kadel/2570427b9113ea0bb546f45723b7a643. Even i was not sure about reason, so i tried it locally couple of times and none of the time i could reproduce it. For now the best way to get the failure log is to wait for CI failure in the same spec.

For now the best way to get the failure log is to wait for CI failure in the same spec.

no need to wait for it to happen just use https://search.svc.ci.openshift.org/ to search through logs

This issue has been fixed in this PR: https://github.com/openshift/odo/pull/2762
And the reason why this is happening is described here: https://github.com/openshift/odo/pull/2762#issuecomment-608642267
I've never seen this test failed after this PR has been merged.
Same for #2891  and #2892
We can close these three issues.

@yangcao77 thank you.
Closing this
/close


Improve Club Settings View

Overview
As a user, I should be able to  view the clubs in the settings view properly.
Currently the implementation for clubs in the settings page displays them all as one column.
Ideally this view should be a variable grid view that allows for greater visibility of the orgs

Closed by #203


fix: add missing Migration lang item

Description

add missing Migration lang item

$ php spark migrate

CodeIgniter v4.1.6 Command Line Tool - Server Time: 2022-01-08 20:30:40 UTC-06:00

すべての新しいマイグレーションを実行しています...
Done migrations.

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[] User guide updated
[x] Conforms to style guide

Small semantic suggestion.
@lonnieezell do we need to do anything to "request translators" when new strings get added, for the translations repo?

Paul implemented static analysis checks in the translation repo that check for missing, duplicated or copied strings.

Paul implemented static analysis checks in the translation repo that check for missing, duplicated or copied strings.

But translators do not know if they run checks.


(ui) fix rej issues for frontend

Which issue is resolved by this Pull Request:
Resolves #
Description of your changes:
Environment tested:

Python Version (use python --version):
Tekton Version (use tkn version):
Kubernetes Version (use kubectl version):
OS (e.g. from /etc/os-release):

@ckadner here is the frontend portion.
Thanks for doing the heavy lifting!

@ckadner here is the frontend portion.
Thanks for doing the heavy lifting!

Thanks @drewbutlerbb4
/lgtm

Thanks @drewbutlerbb4
/lgtm


Validate data for all time zones

Since the promise of this library is to resolve an abbreviation or name for all inputs, we need to test all time zones and fill in any gaps.  CLDR should be authoritative in most cases, but TZDB will be in others, and we may need some manual corrections as well.

See #6

There are now unit tests for this in v1.3.0


P416 spec: Inconsistent "architecture" terminology

From reading the P416 specification, it seems that at some point in the evolution of P416, the term "model" was used to describe what later became the P416 architecture. Is my inference correct?
For example, in section 7.13:

The declaration of new match_kinds can only occur within model description files; P4 programmers cannot declare new match kinds.

I think this a minor spec bug; it should read "... within architecture description files...". This is the term used in most of the spec.
Elsewhere in the spec, the terms “architecture definition”, “architecture description”, “architectural description”, “architecture description file” and “architecture model” all seem to be used loosely interchangeably. (And Figure 2 has "P4 architecture model".) A pedantic quibble, but it would be helpful if consistent terminology were used throughout!

Yes, this looks like an oversight.
I have been pedantic about ensuring consistent use of certain terms, but clearly not all.
We welcome PRs ;-)

In the interest of tidying up the set of active issues on the P4 specification repository, I'm marking this as "stalled" and closing it. Of course, we can always re-open it in the future if there is interest in resurrecting it.


Login page doesn't follow Google material design Guidelines

Summary:
Login page looks too simple than it should be as a potential grown App. The page doesn't follow any UI/UX roles. It's just hit and run. So i tried to add some sort of life to it.
*Screenshots:
To be added..

@ammaratef45/z please, pay attention to this issue

@bahaaib/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

Thanks for your contribution, @bahaaib/z! If you would be a member of the project, you would now earn +15 reputation points, as explained in §29. You can join and apply to it, see §2.

The job #18 assigned to @bahaaib/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; @bahaaib/z is not a member of this project yet, but they can request to join, as §1 explains; there will be no monetary reward for this job

It is strongly discouraged to assign jobs to their creators, see §19: -15 point(s) just awarded to @ammaratef45/z

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @ammaratef45/z

@Bahaaib Thaks :+1:

Job was finished in 0 hours, bonus for fast delivery is possible (see §36)

The job #18 is now out of scope

Order was finished: +35 point(s) just awarded to @bahaaib/z


[WIP] Use mmap in from_par_iter

This currently doesn't work because MerkleTree can be any T but MmapMut only supports u8,
191 |                 (*anonymous_mmap(size)).to_vec()
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected type parameter, found u8
    |
    = note: expected type `std::vec::Vec<T>`
               found type `std::vec::Vec<u8>`

/cc @dignifiedquire

Closing in favor of a more elaborate solution in https://github.com/filecoin-project/merkle_light/issues/8.

Reopening with transmute.


Update ngrok to 2.1.18,4VmDzA7iaHb

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Merged as 6966643fa2a402bc67dc14b1da96115c7f9eaee5.


[Snyk] Upgrade html-snapshots from 2.5.0 to 2.5.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.5.0 to 2.5.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 3 days ago, on 2023-07-18.

Release notes

Package name: html-snapshots

2.5.1 -<EMAIL_ADDRESS>

2.5.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

5d8a1b0 Merge pull request #414 from localnerve/rc-2.5.1
a8e1f81 Merge branch 'master' into rc-2.5.1
85962e0 @ 2.5.1 -<EMAIL_ADDRESS>c356da8 Merge pull request #413 from localnerve/snyk-upgrade-7dda85887de35aec9cee3f5a3de4972a
73428d4 fix: upgrade html-snapshots from 2.4.3 to 2.5.0
dde244a Merge pull request #412 from localnerve/snyk-upgrade-bbade9c1e1c575534fab75301c39dff9
7d3dab3 fix: upgrade html-snapshots from 2.4.2 to 2.5.0
5466d98 Merge pull request #411 from localnerve/snyk-upgrade-e9bdfc6e9b56d664f1adf94d8bf943a4
d0be91f fix: upgrade html-snapshots from 2.4.3 to 2.5.0
60803b8 Merge pull request #410 from localnerve/snyk-upgrade-1dd4d3a0f9e22ce4431805e2cd270415
14500b6 fix: upgrade html-snapshots from 2.4.3 to 2.5.0
3356330 Merge pull request #408 from localnerve/snyk-upgrade-8c71b3e0128f37ab70ea5256a522b311
1869b4c fix: upgrade html-snapshots from 2.4.3 to 2.5.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same when pulling a76a4a9abc9fbae601ae7910f3cbefce775043ba on snyk-upgrade-54bde039d6f6d76c4eee6aa4d8b91df4 into f776f077c23f6be470634a84f369c898c7f2169b on master.