[QUESTION] Security Vulnerabilities detected by Binary Analysis Scan (Black Duck Binary Analysis)

Hi,
We have a Xamarin Forms v<IP_ADDRESS>7 project. We ran a binary analysis scan on the project by using the Black Duck Binary Analysis tool and it reports back security vulnerabilities from the libraries that SkiaSharp uses. We wanted to report these to you and also ask for a roadmap to update these libraries.
Below you can find the specific versions detected by the binary analysis scan and also the vulnerabilities that are detected for each library with their NATIONAL VULNERABILITY DATABASE URL.

Name
Version

expat
2.2.9

freetype
2.10.1

libjpeg-turbo
2.0.0

libwebp
v0.6.1-rc2

Vulnerability URLsexpat:http://nvd.nist.gov/vuln/detail/CVE-2013-0340http://nvd.nist.gov/vuln/detail/CVE-2021-45960http://nvd.nist.gov/vuln/detail/CVE-2021-46143http://nvd.nist.gov/vuln/detail/CVE-2022-22822http://nvd.nist.gov/vuln/detail/CVE-2022-22823http://nvd.nist.gov/vuln/detail/CVE-2022-22824http://nvd.nist.gov/vuln/detail/CVE-2022-22825http://nvd.nist.gov/vuln/detail/CVE-2022-22826http://nvd.nist.gov/vuln/detail/CVE-2022-22827http://nvd.nist.gov/vuln/detail/CVE-2022-23852http://nvd.nist.gov/vuln/detail/CVE-2022-23990http://nvd.nist.gov/vuln/detail/CVE-2022-25235http://nvd.nist.gov/vuln/detail/CVE-2022-25236http://nvd.nist.gov/vuln/detail/CVE-2022-25313http://nvd.nist.gov/vuln/detail/CVE-2022-25314http://nvd.nist.gov/vuln/detail/CVE-2022-25315freetypehttp://nvd.nist.gov/vuln/detail/CVE-2020-15999libjpeg-turbohttp://nvd.nist.gov/vuln/detail/CVE-2020-17541libwebphttp://nvd.nist.gov/vuln/detail/CVE-2018-25009http://nvd.nist.gov/vuln/detail/CVE-2018-25010http://nvd.nist.gov/vuln/detail/CVE-2018-25011http://nvd.nist.gov/vuln/detail/CVE-2018-25012http://nvd.nist.gov/vuln/detail/CVE-2018-25013http://nvd.nist.gov/vuln/detail/CVE-2018-25014http://nvd.nist.gov/vuln/detail/CVE-2020-36328http://nvd.nist.gov/vuln/detail/CVE-2020-36329http://nvd.nist.gov/vuln/detail/CVE-2020-36330http://nvd.nist.gov/vuln/detail/CVE-2020-36331http://nvd.nist.gov/vuln/detail/CVE-2020-36332

Kind regards.

Freetype should no longer have the vulnerability because it was bumped a long time ago in this commit: https://github.com/mono/skia/commit/541601602ee94b1fbf04d9b2fe0661951960f97c
The SkiaSharp package was first published a while back in v2.88.0-preview.61

libjpeg-turbo is also past this fix and is on 2.0.5. Also, the fix is first in v2.88.0-preview.61

libwebp is also far beyond the version that has the issues. It was on 1.1.0 but I am just updating just in case.

I have updated all the listed externals that needed updated and so I am closing this. As soon as the build is complete, it will be on the preview feed as version: 2.88.1-preview.96


keep alive websocket streaming code

`from binance.client import Client
from binance.websockets import BinanceSocketManager
client = Client("XXX", "XXX")
def process_message(msg):
print(msg)
bm = BinanceSocketManager(client)
conn_key = bm.start_symbol_ticker_socket('BTCUSDT', process_message)
bm.start()`
basic streaming code from document.
It runs well but after a few minutes It closes automatically
How can I keep alive this code?

#302


How to use airbrake with asp.net webform website project type?

We have a web application developed under .NET framework 4.6.1. It is not MVC. It's webform and it's website project type, not web application project type.
Can we integrate Airbrake with our application?

Yes, you can add HTTP Module
https://github.com/airbrake/sharpbrake#aspnet-http-module
or you can call directly from your try/catch block
https://github.com/airbrake/sharpbrake#basic-example

Thank you for your reply, rbalabukh. I will try. Any further question, I will post again.


[Wildcat]   Error executing 'mvn clean package'. Run Maven manually to get more info.

In the last days I started to have this error everytime I try to use WildcatRun. Nothing has changed in my environment.
There is a way to see a log from Wildcat?

Sorry, I should have specified that. Yeah, when I run mvn clean package from the command line for the same exact project, I get no compilation errors.
I tried both to use an anaconda environment (what I've been using since now) and system packages to see if something changes, but it doesn't.
I'm on Manjaro Linux, using Tomcat 8.5.87 and openjdk version 8.0.382 and maven version 3.9.5

The command :WildcatRun builds your project with "mvn clean package", copies the generated war into the server (Tomcat in your case) and starts the server.
To spot the error, please execute these phases separately:

Run "mvn clean package"
Execute the command :WildcatDeploy (this copies the generated war into the server)
Execute the command :WildcatInfo (to make sure Tomcat is set as default and the war has been copied)
Execute :WildcatUp to start the server

If any of these instructions fail, I will be easier to spot the problem


Initial support for Toyota Supra

Proposed change
This is a first attempt to add support for Toyota Supra, for which the Supra Connect service uses the same underlying APIs.
Type of change

[x] New feature (which adds functionality to this library)

Additional information
The API expects the value toyota in the request, however, it returns DRITTKUNDE as the response (German for "third party vendor"). To address this, the fallback parsing was updated to default to CarBrand.Toyota if DRITTKUNDE is returned. However, for all I know there may be other brands that also use DRITTKUNDE...(Rolls Royce infotainments are also iDrive based)
I have not yet had a chance to test this. Here is the fingerprint:
toyota_supra_fingerprint.zip
Checklist

[ ] The code change is tested and works locally.
[ ] Tests have been added to verify that the new code works.

Nice, thanks! Didn't know that there are even more cars and brands using NBT systems. Is it only the Supra? And only in the US? Or also globally/other cars?
Regarding the CarBrands issue: I would rather only have TOYOTA and DRITTKUNDE as valid options and no parsing from DRITTKUNDE to TOYOTA in the enum.
When initializing a vehicle, the brand could be added as additional parameter. Then we could check in MyBMWVehicle.brand property if the value in JSON is DRITTKUNDE and replace it with the init parameter?

For Toyota, yeah only the Supra. For other brands it's only Rolls Royce as far as I can tell (see the last paragraph in the intro here: https://en.wikipedia.org/wiki/BMW_iDrive). The Supra is available in at least Europe but no clue about China.

I would rather only have TOYOTA and DRITTKUNDE as valid options and no parsing from DRITTKUNDE to TOYOTA in the enum

The problem with this is that drittkunde is not a valid value in API requests. So e.g. if we're iterating through all CarBrands, where DRITTKUNDE has been included as a value, the API will return a 400 Bad Request.
One approach might still be to include DRITTKUNDE as an enum, but explicitly filter it out when iterating over all brands -- maybe a CarBrands.validBrands() helper?

Interesting!
I could imagine something like this:
    @classmethod
    @property
    def api_brands(cls) -> list[str]:
        """Return members for API calls."""
        return (cls._member_map_[name] for name in cls._member_names_ if name != "DRITTKUNDE")

IMO, I think that implementation would become a footgun, as it would break the normal/expected Enum iteration (i.e. you would have to know to do for brand in CarBrands.api_brands when making requests as opposed to for brand in CarBrands)

@rikroe Thoughts?

I see your point. But to be honest, I also have no other idea.
Could you please rebase so that test are green? Then we should be able to merge

Sorry for the delay @rikroe. Updated

With #591 merged, could you please have another look? The URLs and return values do differ for the vehicle list, and I'm especially curious if there is any change regarding the returned brands.
Also, I have updated the tests so it is hopefully easier to understand where to adjust them.

@rikroe @vanshg Hey everyone, I was looking for an API to interact with my Supra and stumbled upon this over some Home Assistant forums. For my use case I just want a local project to mess around and don't need all the bulk of HA. I installed this package and tried to use the CLI tool to login, there are no obvious errors but it returns vehicles as 0 so assume it can't find any cars on my profile. Any ideas?

The charging_profile has been fixed with #615.
I'll see if I find some time to update this PR and get it merged (as it is mostly issues in the tests).

I had a look into this and wanted to add some tests.
This lead to a question regarding your fingerprint @vanshg: Do you disable sending vehicle data automatically to you the API?
Your fingerprint does not show any information (like open windows or lock state of the doors) which should be possible according to Toyota's website.
If yes, could you please run a new fingerprint with this enabled?

I don't have anything disabled. When I check the Supra Connect app, I only have the ability to perform actions (lock, unlock, horn, lights, ventilation) -- Besides location, I don't see anything that indicates current vehicle state (door locks or windows)

Ah, so BMW gives you the same level of access as their pre-2015 cars...
As I've adjusted some things (however mainly for the tests), could you please test the latest version of this branch with your car to verify everything still works?

Yep, I can confirm this works. I am able to run various subcommands successfully (I tried status, vehiclefinder, and horn)


add Azure Maps as a source input

based on https://docs.microsoft.com/en-us/azure/azure-maps/azure-maps-event-grid-integration

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

Well that's an embarassing miss, thank you @nnmer :) #signoff


not working on large sitemap .net core 3.1

just test out SimpleMvcSitemap.
it work fine on sitemap size around 50 urls+ however
when i want to use it to generate 50k urls i found out that if sitemap has larger than 1000+ urls
will get xml error "error on line 1 at column 65697: Opening and ending tag mismatch: lastmourl line 0 and url"

To fix this, I had to stop using the new InProcess method for hosing on IIS.
In your project file, set the following:

OutOfProcess

It's a shame that we can't use InProcess, hopefully the author figure something out to get past whatever limitation it's running into.

Hi, This issue is serious! I have to do sitemap manually as a workaround. Please update and fix your lib.

Fixed! You can get the new version.


7plus.com.au "No video formats found" on some videos. June 2019

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.06.08
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE
F:\youtube-dl -v https://7plus.com.au/raffertys-rules?episode-id=RRL-005
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://7plus.com.au/raffertys-rules?episode-id=RRL-005']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.06.08
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-90069-gdd8351b118, ffprobe N-90069-gdd8351b118
[debug] Proxy map: {}
[7plus] RRL-005: Downloading JSON metadata
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug
 . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp3esl0iyj\build\youtube_dl\YoutubeDL.py", line 796, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp3esl0iyj\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp3esl0iyj\build\youtube_dl\extractor\sevenplus.py", line 61, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp3esl0iyj\build\youtube_dl\extractor\brightcove.py", line 680, in _parse_brightcove_metadata
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp3esl0iyj\build\youtube_dl\extractor\common.py", line 1327, in _sort_formats
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

F:\youtube-dl>

Description

WRITE DESCRIPTION HERE
Only some videos from 7plus.com.au give the above error. Others in the TV series download correctly. e.g. https://7plus.com.au/raffertys-rules?episode-id=RRL-004.
(Working episodes of Series 1 are  01-04 and 10 13 and 15. Others do not work.
This site does not require login but is possibly geo-locked to Australia.
The error "No video formats found" has been reported several times before for this site but I have not found any report with an explanation or workaround. Previous reports of this error for this site applied to previous versions of youtube-dl for Windows.
The same error occurs with the command line options -j and --get-url perhaps indicating an error to access or parse the JSON metadata. I have not been able to debug further.
Perhaps the error indicated from brightcove.py would show up on other sites but I don't have the knowledge to go further.

I think that some episodes are not properly configured at the 7plus end


Script doesn't work anymore

Hey everyone!
Few months ago the script stopped working for me and I have no idea how to fix it.
When I try to launch the script (tried both scripts>browse and scripts>export layers to files (fast) ) I get an empty error window.

THEN one of the two possible things happen:

Photoshop crashes without even giving me the usual "send crash report" window
The scrip interface pops up, all settings works fine, but when I click "run" photoshop crashes...

I've tried different versions of Photoshop and different versions of the script, same result all the time.  Tried a clean PS install too, doesn't help.
Is there a chance that it happens because I do not have admin rights on my windows laptop? It looks like the issue is on my side, but I can't figure out how to solve it.
Any help would me much appreciated!
Thanks!

Alright, apologies for disturbing you all.
It turned out to be my hardware issue \ admin rights thing.
It's a great script, you save millions of human-hours.
Thank you
Best
J

Hi!
I have the same issue. Can you explain how to fix it? I tried running with administrative rights but I still can't get the script to work. It was working amazingly and all of sudden it stopped.


added openssl and parameter for building nginx with ssl module

This adds support for the nginx SSL module

Thanks for the commit feel free to also add yourself to contribs


#67 [FEATURE] Set event name suggestion on Variable Changed event creation

https://youtu.be/_seSt3tMZdU
Currently the implementation suggests a name for a new Event or variable when created with the 'Create' button. If the parent of the property is a BaseAtom (Variable, valuelist, FSM etc) the naming structure is: "name of variable/valuelist/FSM" + name of property ("Changed" when you add a variable changed event) + type of Event/variable ("BoolEvent" for example). When you try to create a second default named event the system will add a number at the end just like it is with duplicating GameObjects. Everything can be seen in the video above.
There are some occasions where this default naming scheme can be a bit weird, as you can see with SetIntVariableValue as an example. However if you create custom names for a variable things get quite clear. If there are suggestions for a better naming scheme I will be happy too change things where needed.

Regarding 53686d8, you don't need to worry about supporting Unity 2019, feel free to use the 2020+ features.

I would like to suggest to target this PR to the V4.4.x release as I've just sent a PR for this feature targeted at V5. V5 required some additional changes to the AtomDrawer file overall.


Remove TileMapServiceImageryProvider

Deprecated in #3150.

Relabeling for 1.20, but the exact version will depend on when #3150 is merged.

This will be 1.20.

Removed in #3460, which replaced #3150.

Remember to delete TileMapServiceImageryProviderSpec as well

@hpinkos Shall I delete TileMapServiceImageryProviderSpec and TileMapServiceImageryProvider and send a PR?

@sanuj sure! We'll just leave the PR open until after the March release.

@hpinkos I'll remove it after the March release then other I would have to rebase the branch again.


Convey a useful message to queue_object_action

When creating the snapshots API I used a placeholder message to send
to this method. Unfortunately I never finished this because it's hard
to verify in an integration test.
@miq-bot add-label api, bug
@miq-bot assign @abellotti
Thanks @abellotti for noticing this!

👍  🎵


Shared crosshair

Most of the Grafana graphs can have a shared crosshair which I found useful when trying to figure out a detail time matching. Would it be possible to add that support in this plug-in?

Hm. Not sure if external plugins have access to the shared crosshair. Will do some research!


Using Lazy Loading in docs

This is related to the docs and the theme we use:
What is lazy loading?
Check it out here.
Why should we bother to use it?
Lazy loading help in masking less time to load if told simply and makes our docs load faster. This currently is not applicable because our docs are not yet filled with pictures and videos(This doesn't worry us if we are to use youtube videos ) but it could be in some time. See https://web.dev/lazy-loading/#why for more info on why we need it.
How to solve this issue?
One simple thing which can be done for now is using native-browser-lazy-loading, but yes, some browsers don't support it see here. But we can implement a JS-based lazy loading easily if we were to edit the theme and I recommend using this because sometimes I use IE for browsing(haha).
Seeing https://web.dev/lazy-loading/#implementing seems helpful. Also, I on browsing got this site http://jquery.eisbehr.de/lazy/, and yes found it the best.  Our theme currently uses jquery and yes this can be easily implemented.
So, I think we are gonna use Jsdelivr as CDN, yes, then we need to have a placeholder(a lightweight image may be blurred), and add it to use it. maybe I suggest we can have our logo(not yet ready :() as our placeholder and it can save some space and would look nice.

This sounds good. I appreciate you bringing this up and identifying a few alternatives.
In general, I'm against solving problems that we don't have yet. In my opinion, we should keep building documentation and once we hit loading problems, start actually implementing this. One problem in pre-emptively fixing this before it actually becomes a problem is the following: you say that the solution you found works with jquery, which our theme is already using. But we don't know if we will keep using the current theme. Maybe we'll find a problem tomorrow and we have to change. So we will have to change the lazy loading as well. So it's better to wait until things stabilize a bit (and we actually have this as an issue) before fixing it I think.
"Don't fix what is not broken" ;)

It will just appear as soon as we add some more pictures and videos lol.
Nevertheless, what you say is correct. What I think is that we can create a theme ourselves and maintain it instead of randomly changing the theme and also fix this problem(could create a huge impact on loading time). Also, nearly all the themes uses bootstrap along with jquery, so it will be very easy to implement this if we somehow find a way to add java script. Or more efficiently we can use webpack as done by read the docs theme, and create a single js file. All these are for future when we start with our own theme.

I had opened this issue just for the sake recording things as they may be useful for future reference. For now this is not a problem and I am closing this.

This really depends on the size of the videos -- and actually even more on the images. The largest video in our documentation currently (yes, the current status is not really a great reference point) has about 90kB (and for videos, loading at play time instead at page load can be enabled -- for images, this is another story).
I'd rather leave this open in order not to loose / forget about the approaches you looked into already.

Just added it to the Documentation board - we won't forget about this issue ;)


Error code remains zero when assertion failed

I've been asserting deep equality of React components, and noticed a test silently failing with return code 0 (NB: I'm generally quite new to JavaScript). This seems to relate to React objects containing Symbol.
Considering the following test:
const expect = require('chai').expect;
const a = Symbol('a');
console.log(`a = ${a.toString()} of type ${typeof a}`);

it("deep equal demo", () => {
  expect({ a, b: 'foo' }).to.deep.equal({ a, b: 'bar' });
});

I get a return code of 0 instead of 1, thus test is silently ignored in CI:
$ node --version; mocha --version; mocha demo.test.js; echo "Return code is $?"
v5.11.0
2.4.5
a = Symbol(a) of type symbol

  1) deep equal demo

  0 passing (12ms)
  1 failing

Return code is 0

If I comment out the it (leaving expect uncommented), I can see that the exception is thrown by chai and the error code is correct:
$ node --version; mocha --version; mocha demo.test.js; echo "Return code is $?"
v5.11.0
2.4.5
a = Symbol(a) of type symbol

XXX/node_modules/chai/lib/chai/assertion.js:107
      throw new AssertionError(msg, {
      ^
AssertionError: expected { a: {}, b: 'foo' } to deeply equal { a: {}, b: 'bar' }
    at Assertion.assertEqual (XXX/node_modules/chai/lib/chai/core/assertions.js:485:19)
    at Assertion.ctx.(anonymous function) [as equal] (XXX/node_modules/chai/lib/chai/utils/addMethod.js:41:25)
    at Object.<anonymous> (XXX/demo.test.js:6:35)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:20:19)
    at XXX/node_modules/mocha/lib/mocha.js:219:27
    at Array.forEach (native)
    at Mocha.loadFiles (XXX/node_modules/mocha/lib/mocha.js:216:14)
    at Mocha.run (XXX/node_modules/mocha/lib/mocha.js:468:10)
    at Object.<anonymous> (XXX/node_modules/mocha/bin/_mocha:403:18)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:148:18)
    at node.js:405:3
Return code is 1

In both cases, the test passes with return code 0 if I change bar to foo.
The problem seems specific to the ES2015 Symbol, although it seems to be properly recognized as the logs show.
It's not clear to me if mocha needs Babel in order to recognize Symbol (the log above makes me think not), but adding --compilers js:babel-register,js:babel-polyfill did not help.

@sedubois this was fixed in 872684f44f677774f544373abf753b5080af8655
npm install mochajs/mocha to grab it


Support multline details in quickpick items

Quickpick items can contain a label+description on one line and detail on a second, which overall still limits the amount of information to two lines. Can we support line breaks in the detail field to allow surfacing more information across multiple lines?

Marked this as a feature request to see what people think. Ultimately I think that having different sized Quick Pick Items makes it harder to scan and quickly pick... but I get the desire.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

You're right, this would not be suitable for long lists of items but rather to present a choice between a handful of options with detailed context. AFAICT there is no alternative UI primitive that would match this use case well -- modal dialogs with buttons don't give quite the same "picking" functionality, and webviews would be very heavy and a bit out of place.
I have been working with a modified VS Code app with this feature for a while, so I can provide a PR if this is accepted.

Our use case is to show a non-truncated version of a terminal command so the user can preview before running it in the shell. A requirement on my end is it would need to show a fully non-truncated version which would probably mean it needs a scroll bar if it's sufficiently long. If multi-line was supported and it was truncated eventually, allowing the overflow to be viewed in a custom hover would probably be sufficient.
cc @meganrogge


Add machine Profile Version support

Add a version field and allow machine groups to reference a profile and semantic version.

There is no longer any plan to implement this.


problem using bcc-dev edge ver pkg for alpine

Bug Report
After the latest kubearmor image update, the kubearmor pods were going into CrashLoopBackOff for following reason:
Error relocating /usr/lib/libbcc.so.0: btf__load_vmlinux_btf: symbol not found
Error code: 127

There was a problem using the latest update of bcc-dev (edge, 0.23.0-r0). This pkg was updated yesterday.
#490 affixed 0.20.0-r1 of bcc-dev from alpine branch v3.14 which seems to be working fine.
With this change Kubearmor won't be using the bcc-dev updates any more and hence the problem needs to be looked into.
General Information

Environment description (GKE, VM-Kubeadm, vagrant-dev-env, minikube, microk8s, ...)
All

Kernel version (run uname -a)
All

To Reproduce
Prepare kubearmor image using bcc-dev edge version and launch the pods. The pods will go into CrashLoopBackOff.
Expected behavior
The kubearmor pods need to be started normally.
CC: @daemon1024

For safety, let's add one more testing logic that GitHub action builds a test image before pushing the latest image to Github Hub. In this process, we need to check if the test image can be deployed in a k8s cluster and the KubeArmor daemonset runs without errors.

For safety, let's add one more testing logic that GitHub action builds a test image before pushing the latest image to Github Hub. In this process, we need to check if the test image can be deployed in a k8s cluster and the KubeArmor daemonset runs without errors.

Yeah! Good point. I believe the current tests are run directly in a self-managed k8s env with kubearmor running as a host process. The only change we may have to do is to use karmor cli to do the tests (which will automatically install kubearmor images). Is this right?

The issue here is that recently function "libbpf_find_kernel_btf" was renamed to "btf__load_vmlinux_btf" in the Linux kernel Commit link. These changes were then included in the bcc library which takes usage of this function, but not able to find the new function name in the older kernels, thus showing the above error.


Mingw warnings

There are some warnings when compiling for Mingw platform:
[57/236] Building C object CMakeFiles/re-objs.dir/src/main/init.c.obj
/home/runner/work/re/re/baresip-win32/re/src/main/init.c:44:13: warning: no previous prototype for ‘exception_handler’ [-Wmissing-prototypes]
   44 | LONG WINAPI exception_handler(EXCEPTION_POINTERS *ExceptionInfo)
      |             ^~~~~~~~~~~~~~~~~

[123/236] Building C object CMakeFiles/re-objs.dir/src/thread/thread.c.obj
/home/runner/work/re/re/baresip-win32/re/src/thread/thread.c: In function ‘handler’:
/home/runner/work/re/re/baresip-win32/re/src/thread/thread.c:95:27: warning: passing argument 1 of ‘pthread_setname_np’ makes integer from pointer without a cast [-Wint-conversion]
   95 |  (void)pthread_setname_np(*th.thr, th.name);
      |                           ^~~~~~~
      |                           |
      |                           thrd_t {aka void *}
In file included from /home/runner/work/re/re/baresip-win32/re/src/thread/thread.c:16:
/usr/share/mingw-w64/include/pthread.h:317:55: note: expected ‘pthread_t’ {aka ‘long long unsigned int’} but argument is of type ‘thrd_t’ {aka ‘void *’}
  317 | int       WINPTHREAD_API pthread_setname_np(pthread_t thread, const char *name);
      |                                             ~~~~~~~~~~^~~~~~

consider this PR to enable -Werror:
https://github.com/baresip/baresip-win32/pull/9

https://github.com/baresip/re/commit/853cee7613bff031a721573828059fb8670bbcba

https://github.com/baresip/re/pull/844

thanks, all fixed now :)


[BUG] splat.nlog can't output to file

Describe the bug
A .net 5 wpf application, with the same configuration, it can output log to file directly using NLog, but only output log to console  using splat.nlog
Steps To Reproduce

create .net 5 wpf application with reactiveui
add splat.nlog in nuget
add nlog.config
register splat.nlog
"implement" the IEnableLogger interface
output log

Expected behavior
log can output to file
Additional context
nlog.config
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      autoReload="true"
      throwExceptions="false"
      internalLogLevel="Off">

  <targets>
    <target name="logfile" xsi:type="File" fileName="${basedir}/Logs/${shortdate}.log" layout="${longdate}|${level:uppercase=true:padding=5}: ${message}"/>
  </targets>

  <rules>
    <logger name="*" minlevel="Debug" writeTo="logfile" />
  </rules>
</nlog>

This is my problem, because the log is output before registering NLog, so NLog is not enable at this time.


Add Yul smart contract language

Adding Yul --
"Yul is a programming language that is being used as an intermediate language in the compilation of Ethereum smart contracts written in in the Solidity language. However, Yul is designed to be more general than Solidity, and may find additional uses soon." Source: https://www.cs.utexas.edu/users/moore/acl2/manuals/current/manual/index-seo.php/YUL____YUL?path=3463/32691/2451/9091
Yul is becoming more and more important, here is next week's solidity summit agenda:
https://summit.soliditylang.org/agenda.html
There is quite a lot of .yul code on GitHub already.
Description

Checklist:

[] I am associating a language with a new file extension.

[ ] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3AFOOBAR+KEYWORDS+NOT+nothack

[] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[X] I am adding a new language.

[X] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Ayul+NOT+nothack
https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Ayulp+NOT+nothack

[X] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

[URL to each sample source, if applicable]
https://docs.soliditylang.org/en/v0.8.13/yul.html#specification-of-yul-object (Specification of Yul Object)
https://docs.soliditylang.org/en/v0.8.13/yul.html#complete-erc20-example (Complete ERC20 Example)

Sample license(s):
© Copyright 2016-2021, Ethereum. (visible at the bottom of docs page)

[X] I have included a syntax highlighting grammar: https://github.com/davidhq/SublimeEthereum
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

[ ] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[ ] I have included a change to the heuristics to distinguish my language from others using the same extension.

[ ] I am changing the source of a syntax highlighting grammar

Old: [URL to grammar repo]
New: [URL to grammar repo]

[ ] I am updating a grammar submodule

[ ] I am adding new or changing current functionality

[ ] I have added or updated the tests for the new or changed functionality.

[ ] I am changing the color associated with a language

[ ] I have obtained agreement from the wider language community on this color change.

[URL to public discussion]
[Optional: URL to official branding guidelines for the language]

Please remove the .yulp extension.

done

Please also update the OP to link to the direct source of the two samples and state their respective licenses.

Added direct links to samples, entire license is not present, all I could include is "© Copyright 2016-2021, Ethereum." visible at the bottom of the page.
If license for samples in longer form is needed I can write the team and ask if they can provide this? Please let me know if needed. Thank you

If license for samples in longer form is needed I can write the team and ask if they can provide this? Please let me know if needed. Thank you

This is fine, thanks. I've found from the source that the docs are in a repo covered by the GPL 3.0 so I've updated your OP with that.

If license for samples in longer form is needed I can write the team and ask if they can provide this? Please let me know if needed. Thank you

This is fine, thanks. I've found from the source that the docs are in a repo covered by the GPL 3.0 so I've updated your OP with that.

great, thank you!


Nesting of objects

I have been working on a PR that will make RealmSwift support nesting of Object classes.
Currently it works like this:
import RealmSwift

public class DemoObject: Object {

	dynamic var name: String = "DemoObjectName"

	public class NestedObject: Object {

		dynamic var name: String = "NestedObjectName"

	}
}

extension DemoObject {

	 public class NestedObject2: Object {
		dynamic var name: String = "NestedObject2Name"
	}

}

This results in the tables DemoObject, NestedObject and NestedObject2. But because you can now nest classes, this will cause clashes with other nested classes with the same name (mostly the reason to perform nesting is the naming clashes).
What would be a good naming convention for the tables? Are there already any thoughts about the way this should be done?
My proposal would be to use something like DemoObject.NestedObject and DemoObject.NestedObject2 as table names.
What are your thoughts about this?

I dropped my changes in my forked cocoa-realm repository. You can check them out here.
https://github.com/depl0y/realm-cocoa/commit/c6b02dbb49b131e70e33132e556ebfe642771bb7
I feel to n00b to actually submit a PR at this moment, till we probably have figured out some of the consequences of these changes (table naming, etc).

Ok, one of the things I am coming across at the moment (with these changes) is that you can't set up a relation correctly.
For example:
extension DemoObject {
	 public class NestedObject2: Object {
		dynamic var name: String = "NestedObject2Name"
		dynamic var nestObject: DemoObject.NestedObject = DemoObject.NestedObject()
	}
}

This throws an error in the RLMProperty.mm file at setTypeFromRawType where it checks the type of properties. the className variable in the following piece of code is actually NestedObject instead of DemoObject.NestedObject.
else {
	// for objects strip the quotes and @
	NSString *className = [_objcRawType substringWithRange:NSMakeRange(2, _objcRawType.length-3)];

	// verify type
	Class cls = [RLMSchema classForString:className];
	if (!RLMIsObjectSubclass(cls)) {
		@throw RLMException([NSString stringWithFormat:@"'%@' is not supported as an RLMObject property. All properties must be primitives, NSString, NSDate, NSData, RLMArray, or subclasses of RLMObject. See http://realm.io/docs/cocoa/api/Classes/RLMObject.html for more information.", className]);
	}

	_type = RLMPropertyTypeObject;
	_objectClassName = className;
}

It may be that I am approaching this the wrong way.

Keep in mind that Realm is a cross-platform database and that any table created from one binding should be accessible from another. So using dots in the table name may be a deal-breaker.
Can I ask why you're interested in supporting nested models? If you're interested in the stylistic effect, could you not use a nested typealias?
class A: Object {}

class B: Object {
    typealias NestedA = A
}

We're open to reviewing a PR adding support for nesting model classes, but it should add clear value and keep Realm's promise of cross-platform compatibility.

Of course you can ask that :). In my project I really miss namespaces.
I for example have a class called Light which is a light specifically connected to a HomeWizard device. Then I also have a Light class which is connected to a Hue Bridge device.
Because there are currently 4 different lights I have to support, which all have different properties and due to the lack of polymorphism in Realm. I want to have class names like HomeWizard.Light and Hue.Light to make my code more readable.
Ok, I understand my story is quite confusing. Let me try to create a small diagram:

My HomeWizard and Hue class both conform to the Controller protocol. The HomeWizard Light and the Hue Light have a relation to either the HomeWizard or the Hue class. The Light classes conform to a protocol called Device.
Not so sure this makes my story less confusing... sorry.

So why not do the following?
class BaseLight: Object {
  // some common Light properties
}
class HomeWizardLight: BaseLight {}

class HomeWizard: Controller {
    typealias Light = HomeWizardLight
}

You can now use HomeWizard.Light to refer to HomeWizardLight, and cross-platform compatibility is maintained.

Well, I haven't though of that... crap

I don't think the additional complexity, potential bugs, performance impact, maintenance, cross-platform issues are worth the feature. Especially since the workaround is so simple and complete, unless I'm missing something.

Probably not ;). Well at least it was a day well spent debugging through Realm. Thanks.


Fix tooltip & popup positioning

Description
This tries to ensure the tooltip & popups are created on the correct
monitor
Test steps

This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor

Split header tooltipIt shows up on the same screen as my cursor

UsercardIt shows up on the same screen as my cursor

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is

REFERENCE
✔
❌

TODO: Add proper test steps to ensure we get testing on all platforms
Tested on
Linux Wayland

UNTESTED
This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor

Split header tooltipIt shows up on the same screen as my cursor

UsercardIt shows up on the same screen as my cursor

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is

Linux X11

by pajlada on Arch Linux
This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor
✔
❌

Split header tooltipIt shows up on the same screen as my cursor
✔
❌

UsercardIt shows up on the same screen as my cursor
✔
❌

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is
❌
❌

macOS

UNTESTED
This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor

Split header tooltipIt shows up on the same screen as my cursor

UsercardIt shows up on the same screen as my cursor

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is

Windows 100% zoom

UNTESTED
This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor

Split header tooltipIt shows up on the same screen as my cursor

UsercardIt shows up on the same screen as my cursor

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is

Windows >100% zoom

UNTESTED
This PR
Main branch

Emote tooltipIt shows up on the same screen as my cursor

Split header tooltipIt shows up on the same screen as my cursor

UsercardIt shows up on the same screen as my cursor

Emote popupIt shows up at the same place every timeregardless of where my Chatterino or cursor is

Fixes #4733
Fixes #4572

Looking for testers with dual-monitor setups for the following platforms:
macOS
Linux Wayland
Windows 100% zoom
Windows >100% zoom
The test steps are outlined in the PR description, hopefully should be easy enough to follow along

The relevant platforms have been tested a bit, and with that I'm happy to merge this in.
This is not a perfect fix, but it will have fixed the most annoying things and not caused too many new annoying things.
The biggest change this does is: Don't move the widget multiple times upon showing


Adding Devcontainer 🎉

Fixes

Fixes https://github.com/WordPress/openverse/issues/1022 by @crazy4pi314
Description

Testing Instructions

Checklist

[x] My pull request has a descriptive title (not a vague title like
Update index.md).
[x] My pull request targets the default branch of the repository (main) or
a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible
errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@dhruvkb should we close this (wonderful, exciting) PR and wait until the monorepo to implement this?

Cool, lmk and I can help once the monorepo stuff is setup! 💖


add cs awards to readme

Type of PR

[ ] BUG
[ ] FEAT
[ ] MAINT
[ ] DOC

Is your Pull Request linked to an existing Issue or Pull Request?

Give a brief description for the solution you have provided

PR Checklist

[ ] Added documentation for changes
[ ] Added feature to example notebooks or tutorial (if appropriate)
[ ] Added tests (if appropriate)
[ ] Updated CHANGELOG.md (if appropriate)
[ ] Made changes based off the latest version of Splink
[ ] Run the linter

Should we also add this to the docs homepage?

Yes you are right @ADBond, we probably should add it. I will be doing a couple more docs things in the next week so it is on the list!


⚠ PrivateHD has degraded performance

In 78e94bf, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2955 ms

Resolved: PrivateHD performance has improved in ddca058.


 I can connect but every time I try to update the price I get Required parameter missing or invalid on Variant Put Request - Not sure what I am missing?
The code I am running is below. The variant ID is correct. Apologies in advance if this is the wrong place for this.
$api = new BasicShopifyAPI(true); // true sets it to private
$api->setVersion('2019-10'); // "YYYY-MM" or "unstable"
$api->setShop( $shopify_url );
$api->setApiKey( $api_key );
$api->setApiPassword( $api_pwd );
$response = $api->rest(
    'PUT',
    '/admin/variants/__VARIANT_ID__.json',
    ['id'=>__VARIANT_ID__,'price'=>'69.99'],
);

Ta in advance

Ive worked it out
$response = $api->rest(
    'PUT',
    '/admin/variants/__VARIANT_ID__.json',
    [ variant=> ['id'=>__VARIANT_ID__,'price'=>'69.99']],
);


format pretty toggles case of sql types (solves issue #136).

Including a new test case.

Coverage increased (+0.02%) to 99.479% when pulling 80a86b18710cb038ba92f2a1fde854b46610a4c1 on wwe_issue_136 into 25a56aa9279345de0aba504b6fce12f69f9477de on master.


cursor should show after the number instead of before the number

@Root-vb Please Help ME
-> When user taps on any otp box, cursor should show after the number instead of before the number.

@Root-vb please provide any solution for this.


Can i replace 3D convolution with 2D convolution ？ What will be the impact

We have an ablation on this in the updated arxiv paper.

你好，这是石晓宇的邮箱。我会尽快处理该邮件


Use-case for reconciling external state

(As requested in the FAQ)
One idea I've been toying with is implementing a PagerDuty operator for kubernetes
Things like a PagerDutyService CRD. And then automatically hooking up the generated service endpoint with the AlertManager resource from prometheus-operator.
The main reason to not create the services manually is for us replicating the same set of services across multiple environments, and we're trying to make all the ways of declaring our infrastructure kubernetes manifest shaped, so deploying an app is just applying a bunch of manifests.

Thanks for filing this! It's definitely a use case that I think is a good one for Metacontroller to tackle. People should not have to think in terms of reconciling control loops just to plug one thing into another thing. We should just ask them to provide a little glue.
The way I've been thinking about the general pattern is something like:

I want to create a Kubernetes-native, declarative API that's a wrapper of an existing, traditional, imperative API.

Once #60 is addressed, you'll be able to get most of the way there with CompositeController. You'll have the ability to reliably create, update, and delete things in the external system in response to Kubernetes resources. The biggest gap I see remaining after that will be pulling state changes that happen in the external system back into the Kubernetes API surface, so you can automate reactions to those changes.
You can approximate that by setting CompositeController to poll your sync hook, and having your hook in turn query the external system's API to check on the latest state there. However, I've been thinking this pattern might deserve its own Metacontroller API -- something like AdapterController. We could make this API better fit the use case of 2-way synchronization of state between Kubernetes objects and an external, imperative API.
As an example, perhaps AdapterController would have the ability to define inbound hooks, which you can wire up to something that knows when things change in the external state. These inbound hooks could look like plain webhooks, or perhaps bindings through something like Knative Eventing. Metacontroller would then know to sync your controller if state changes on either side -- in Kubernetes or in the external system.
As another example, AdapterController might have the ability to curate a continuously updated collection of top-level objects in a CRD (i.e. Metacontroller creates these objects; it's not just responding to objects the user creates) that represent external state. This would mean, for instance, you could expect to see PagerDuty services show up in kubectl get pagerdutyservices even if they weren't created via the PagerDutyService CRD (assuming you choose to establish such a connection).
Let me know if you have any other thoughts on how Metacontroller could help, or if something like AdapterController would fit your needs.

Thanks for the detailed response.
Yes, I can see how I can get to what I need with finalizers, thanks. (This will only automate part of our PagerDuty account, so there's always going to be unmanaged resources there created manually.) As long as I can clean up stuff I create I'm happy, I think.
Having something that can pull changes back is quite an interesting concept. I'm not actually sure inbound hooks would help in the case I'm thinking about - there's no event based api to watch for changes, so it would be polling anyway. I'm also struggling to think about how the reconciliation loop would work...

if the controller treats the same objects in the api as both a desired state and also reflecting the state of the underlying resource we have issues. e.g. if someone creates a service manually, it shows up in the api. If they delete it, the controller would then just keep recreating it, which I think would be very unexpected.
if the controller doesn't treat them as desired state then it doesn't help much - you'd need to have a separate object DesiredPagerDutyService, which then would have to create them directly in the pagerduty api, which then means they show up in Kubernetes API. It doesn't quite seem worth it.

I think I'm thinking of this more in terms of Pods in the kubelet, vs docker containers on disk. We make Pod objects, they end up making containers, but we never reflect in the kubernetes API any other containers that happen to exist (excepting mirror pods, but those also come from desired state, rather than containers managed outside of kubernetes)
...It's quite possible I just don't grok how you're anticipating that sort of reconciliation might work
tl;dr finalizers sound suitable for what I need, thanks for the pointer to #60

I'm also interested in this pattern, except for the inbound hooks. DecoratorController almost fits the bill (thanks to #60), the only missing bit is updating the status of the target resource. Is this a reasonable extension to the DecoratorController?

@lionelvillard The original reason for not letting DecoratorController update status is that it wouldn't work in most cases, with the assumption that DecoratorController would mostly be used to attach things to resources that already have their own controller (which would overwrite any status updates you made).
Are you thinking of using DecoratorController for a resource that doesn't already have another controller? If so, why not CompositeController?
Or are you thinking of using DecoratorCotnroller to update status of one of the rare resources that allows multiple controllers to collaborate on status (I think Pod recently added an alpha feature to allow this)?

I'm thinking of using DecoratorController for resources without controllers, which is the case for custom resources. CompositeController could be used (not sure) but seems overkill and counter-intuitive.

@enisoc I'm happy to contribute if you agree that's the right thing to do (adding a status field in the response)

@lionelvillard OK, I'm convinced that it makes sense to let users try to update status in DecoratorController, since it seems people are finding Decorator useful even for cases when there's no existing controller for a resource. If you're able to work on that, I'd be happy to review it.


Resource Lock - impossible to work with

Command Name
az resource lock list
Errors:
Missing required resource_name parameter.

Expected Behavior
Be able to list resource locks
Environment Summary
Windows-10-10.0.19044-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.39.0

Extensions:
account 0.2.3
managementpartner 0.1.3

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

Some tries:
❯ az resource lock list --resource /subscriptions/da0efdd1-xxx/resourceGroups/UC-xxx -g UC-xxx
--resource-type is required if the name, --resource, is present

az resource lock list --resource /subscriptions/da0efdd1xxx/resourceGroups/UC-xxx -g UC-xxx --resource-type Microsoft.Resources/subscriptions/resourceGroups
The command failed with an unexpected error. Here is the traceback:
No value for given attribute
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 663, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 710, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/core/paging.py", line 129, in __next__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/core/paging.py", line 76, in __next__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/mgmt/resource/locks/v2016_09_01/operations/_operations.py", line 1731, in get_next
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/mgmt/resource/locks/v2016_09_01/operations/_operations.py", line 1693, in prepare_request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/mgmt/resource/locks/v2016_09_01/operations/_operations.py", line 564, in build_management_locks_list_at_resource_level_request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msrest/serialization.py", line 652, in url
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msrest/serialization.py", line 760, in serialize_data
ValueError: No value for given attribute

❯ az resource lock list --resource /subscriptions/da0efdd1-xxx/resourceGroups/UC-xxx -g UC-xxx
--resource-type is required if the name, --resource, is present

❯ az resource lock list --resource "/subscriptions/da0efdd1-xxx/resourceGroups/UC-xxx"
'NoneType' object has no attribute 'strip'

@zhoxing-ms for awareness

@lkurzyniec-bcg Please note that the --resource parameter is either passed in a complete resource id (not the resource group id), such as

az resource lock list --resource /subscriptions/{subscription}/resourceGroups/zhoxing-test/providers/Microsoft.Storage/storageAccounts/zhoxing

Or the resource name is passed in and used together with the --resource-group and --resource-type parameters, such as:

az resource lock list -g MyResourceGroup --resource ResourceName --resource-type Microsoft.Storage/storageAccounts

@lkurzyniec-bcg,
Can you can try with az group lock delete --name lockName -g MyResourceGroup and confirm?
More information on resource group locks here.

@lkurzyniec-bcg,
I see you tried with az resource lock, not az group lock. Make sure you try with the suggested command above, and confirm.

@ynden Thanks for your kind reply!
@lkurzyniec-bcg Please note that there are two command groups with different scopes.
If you want to operate the lock of resource group level, please use az group lock
> (env) PS C:\Users\zhoxing.FAREAST\Desktop\project> az group lock -h

Group
    az group lock : Manage Azure resource group locks.

If you want to operate the lock of resource level, please use az resource lock
> (env) PS C:\Users\zhoxing.FAREAST\Desktop\project> az resource lock -h

Group
    az resource lock : Manage Azure resource level locks.


#195 Implement insertion/extraction operators for basic_node template class

For serialization and deserialization of YAML nodes, insertion/extraction operators for basic_node template class have been added.
They behave as wrapper functions of basic_node APIs for the features.
This is also aimed to minimize the necessity of implementing unnecessarily long codes to achieve them.
With the implementation of the new APIs, the test suite and the documentation has also been updated.

coverage: 100.0%. remained the same
when pulling 48657ad82fccd6234b09d1c27c7a47e9fc7bce19 on feature/195_basic_node_operators_with_iostreams
into e1551b94637204b37622c1209d2bcc79c9aa84d6 on develop.


Update contact-support.md

removed Roles that include the action microsoft.office365.supportTickets/allEntities/allTasks can submit a case. as only Service Support Administrator OR Helpdesk Administrator roles are able to open support cases
Fixes https://github.com/MicrosoftDocs/microsoft-365-docs/issues/13192

@JohanFreelancer9 Please copy review

@JohanFreelancer9

@mjcaparas Please merge this PR. Thanks


Bug on Login/Register

Hi,
I have this problem on Login or Register:
https://i.imgur.com/H17Ix8S.png

It seems that your text file is not loading, please check your scriptfiles folder it should contains the YSI folder like the original source code: https://github.com/adri1samp/vc-rp/tree/master/scriptfiles/


Enable sidekick extension to instrument generator page for easier project installation

For users who already have the sidekick extension installed, the generator page (/tools/sidekick/) could be instrumented to simplify adding (and removing) a project.

Extension not installed yet:

Extension installed, project not added yet:

Extension installed, project already added:

Reopening due to visual and fucntional issues


All the tenant management API links are incorrect

The current API links all point to very old Azure Stack links and not the correct API docs for customer tenants. E.g. this is the correct link for Create: https://learn.microsoft.com/en-us/rest/api/activedirectory/ciam-tenants/create?view=rest-activedirectory-2023-05-17-preview&tabs=HTTP

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

ID: 9b07430c-4ce2-114e-051b-a488c6596903
Version Independent ID: 54be2749-8822-f561-56e4-b597aedeed0a
Content: Tenant management with Azure REST API - Microsoft Entra External ID
Content Source: docs/external-id/customers/azure-rest-api-operations-tenant-management.md
Service: active-directory
Sub-service: ciam
GitHub Login: @garrodonnell
Microsoft Alias: godonnell

@kleggMS
Thanks for your feedback! We will investigate and update as appropriate.

@garrodonnell
Can you please check and add your comments on this doc update request as applicable.

@kleggMS
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly

All links have been update now in this PR https://github.com/MicrosoftDocs/entra-docs-pr/pull/2231
#please-close


Feature/android 6 permissions full support

ok, that's it from me. I think this will also need a rebase after you're done with the corrections. Also, travis fails because the file .travis.yml needs to be updated to use android-23 (it's still on android-22).

Thanks for the tip about travis... i will change


Tree Builder: Sort data in presenter not model.

This is part 1 of 3 for improving performance of UI Perf: /ems_infra/:id?display=ems_folders.
This should speed up any tree that displays storages in a tree.
Counts are displayed for the left hand navbar and the expanders in the tree.
Currently, all hosts, then storages are downloaded to determine the count.
This PR does not change the numbers
It simply removes the sort_by that was forcing the hosts and storages to be downloaded when doing a count(*).

After 3 of 3, the difference for a non-expanded tree:

ms
queries
query (ms)
rows
comments

555.9
40
62.5
683
master

204.2
39
26.6
161
after 3 of 3

54%
3%
57%
76%

Code looks good!
Euwe?

Updated:
Fix bad assumption I had around textual_link (I was thinking too hard and deleting changes fixed the build)

Euwe?

mixed on this one. hopefully we won't get hit by this one

Euwe Backport details:
$ git log -1
commit 1bc2bb3e56738e0fa0ca3c41aa82da608ddf3e60
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Wed Oct 19 21:12:52 2016 +0200

    Merge pull request #12002 from kbrock/relationship_sorting

    Tree Builder: Sort data in presenter not model.
    (cherry picked from commit 706cc7cb6bdcb99a83d89359bfe0891e000d9716)


Xcode is not accurate, it has built in support

You mention to use XVim for Xcode, which itself is deprecated, replaced by XVim 2, which broke on XCode 14. There is however built in support for a subset of Vim commands as of the latest version of XCode.

I believe you are correct. The PR #22 should have addressed this. Thanks for reporting!


Fix reference to old rpc::pipeline and Frame::Done in comments

r? @carllerche

Whoops. Thanks for the ping.
Merged: eb365eea217f65a9b795d65bdf70f845b9b783c4


ANC Logic - Diagnosis and Next Steps: Hypertension [12h]

Issue
Hypertension is a disease that is diagnosed through monitoring blood pressure. There are two types of hypertension:

Gestational = under 20 weeks EGA
Chronic = over 20 weeks EGA

The EGA is the only thing that affects whether it is gestational or chronic.
In this flow, the nurse will be able to check one of three options for treatment.
There are two flows for hypertension.
Return for Vital Check Flow

Initial Diagnosis
Next Steps
Case Mgmt

Vitals Recheck
Final Diagnosis
Next Steps

No Vital Check Needed

Diagnosis
Next Steps

Supporting Documents

The logic for ANC can be found here on the ANC - Nurse tab.
The designs can be found in Figma under HC - ANC under the purple logic heading.

Note:

The next steps can appear after the initial touch point with the nurse if able to diagnose without a vital recheck. Otherwise, they will appear after the vitals are rechecked.
In the case that the design in the issue varies from what is in Figma, ping Jeanette on slack and use the Figma design.
You may notice that in some cases we can diagnose hypertension right away but then the diagnosis would change to pre-clampsia based on lab results. We should still diagnose and prescribe the hypertension medication if they qualify. Then we will add on the preeclampsia diagnosis and send them to the hospital to get taken care of.

To Do:

Create next steps activities
Diagnosis Logic
Create necessary pop-ups and the logic to display them
Style per Design

@jeanette-pelizzon

(2) If the follow up is still within these ranges, perscribe:

Methyldopa 250mg 1 tablet by mouth three times a day.

This looks strange to me. Based on what nurse needs to decide if to perscribe 3 or 4?
If this is really what is required, I'd expect to see some kind of instructions on form.

@anvmn
Chronic Hypertension: I believe the nurse gets to decide to do 3x or 4x a day based on some clinical factors. I will let @leowea chime in here.
Gestational Hypertension: It is my understanding that the nurses have a medication table that they refer to. I believe we don't give specific dosages here because it may depend on weight or another factor that we don't ask for.
// @adamhstewart

@jeanette-pelizzon @anvmn @adamhstewart
Thanks for tagging me on this. I should have removed the 'see medication table'. All pregnant women who are going to be treated with medication for any form of high blood pressure will receive methyldopa 250mg at varying doses. The nurse should give methyldopa 250mg by mouth twice a day when first diagnosed with hypertension. If still high the next visit, the dose should go up by 250mg, and this continues with each visit to a maximum of 500mg twice a day. My preference would be to give the nurse options so that she can increase the dose on her own if she wants to.
Here would be the ideal- the below options are always provided. The recommended dose would appear in parentheses next to the dose, but the nurse could choose any option. For example- no medication previously prescribed, then (recommended) would be next to the first option. If 2 times a day was prescribed last visit and the blood pressure remains high this time, then (recommended) would be next to the second option, etc.
Methyldopa 250mg by mouth two times a day
Methyldopa 250mg by mouth three times a day
Methyldopa 250mg by mouth four times a day
@jeanette-pelizzon sorry this has shifted. The NCD protocol had information that wasn't in the ANC protocol. Please let me know if this doesn't make sense. Also- if the (recommended) part is too complicated, we can just have the 3 options and let the nurses decide what to give.

@anvmn I updated the logic in the document and the design. What are your thoughts on the technical difficulty of the (recommended) appearing next to the appropriate dose based on past prescriptions? I kept that out of the design for now so it doesn't hold anything up immediately.

@jeanette-pelizzon

What are your thoughts on the technical difficulty of the (recommended) appearing next to the appropriate dose based on past prescriptions

This will require 2-3h to implement.

@anvmn I spoke with Adam and we are going to move forward with implementing the recommended call out. Here is an example of what that should look like:

Do I need to make this a separate issue?

@jeanette-pelizzon
No, we can have this issue updated with requirements and estimation.

@anvmn I can't select the recommended checkbox:

The HTML looks like maybe it's out of order with the prechecked input falling outside the div.

@adamhstewart  ^^ is fixed at #194

@adamhstewart #194 is deployed, so ^^ can be retested

Rechecked and approved


hi, is this still need it?

hi.
Im using this "patch" for _with_structured.. do you is is still need it or Langchain.ollama has fixit or planning to fix?
Thanks for your work

You no longer need it if you are using the ChatOllama from the latest langchain-ollama package and your llm is supported for tool use by Ollama.
There are still a few models that are capable of tool use on Ollama but aren't supported by them yet. Until they are supported by, you can continue to use tool-calling-llm.


DCOS-12454: Implement Mount component and filters in service picker

This PR implements the Mount component in the new service modal's service picker, allows for the relevant classes to be altered by plugins, and exposes corresponding components to plugins.
@rcorral said I should definitely not pass around the router context in order to redirect to the Universe page, but I don't see any other way to do it. If anyone has a better idea, please let me know!

Rebased with integration test fixes

retest this please

retest this please

@mlunoe Thanks for the review! I believe I have addressed all of your comments.

Dude, this is such an improvement. Well done sir

Retest this please

Adding hold merge until I implement the grid calculation logic as requested in https://github.com/dcos/dcos-ui/pull/1782#discussion_r96471438

retest this please

@jfurrow I just merged #1799, would you mind rebasing once more so that the tests pass?

@philipnrmn Thanks! I just rebased

The integration tests appear to be stale and they're blocking the merge 😟
Let's :shipit: when they're fixed up, there may have to be another rebase though.

Run integration tests

@mlunoe I believe we're seeing legit failures here (although not related to this PR) - check #1793 for more.

@philipnrmn 😳 Thanks for catching that — it's fixed now.

run integration tests


Setup Search and indexing for Documentation site

@lakhanmandloi any update on this, is there an estimate when this will be done?

@kochhar - Hi, this point is already in https://github.com/project-sunbird/sunbird-commons/issues/73. So lets close it here.


chore(sync): rename senders stage

Rename SendersStage to SenderRecoveryStage to provide more clarity about its scope

Codecov Report

Merging #554 (cf9f9ab) into main (f3c79ff) will increase coverage by 0.08%.
The diff coverage is 53.33%.

@@            Coverage Diff             @@
##             main     #554      +/-   ##
==========================================
+ Coverage   72.90%   72.98%   +0.08%
==========================================
  Files         243      243
  Lines       24660    24660
==========================================
+ Hits        17978    17999      +21
+ Misses       6682     6661      -21

Flag
Coverage Δ

fuzz-tests
61.66% <ø> (?)

unit-tests
72.98% <53.33%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

bin/reth/src/config.rs
0.00% <ø> (ø)

bin/reth/src/node/mod.rs
0.00% <0.00%> (ø)

crates/stages/src/stages/sender_recovery.rs
94.22% <66.66%> (ø)

crates/net/network/src/manager.rs
47.07% <0.00%> (-0.26%)
:arrow_down:

crates/transaction-pool/src/pool/txpool.rs
58.73% <0.00%> (+0.52%)
:arrow_up:

crates/transaction-pool/src/test_util/mock.rs
60.37% <0.00%> (+6.66%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


inject span tags from correlation

Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] New feature provided

Related issues

New feature or improvement

inject tags from correlation for stress testing

@wu-sheng @kezhenxu94 Do you think the MD5 algorithm needs to be modified?

MD5 thing is not the thing we should discuss, please read above comments. That is the key.

This will make duplicated injection for every segment. I prefer we don't do this. You are making this similar to OpenTracing Baggage, which we didn't plan to support when we disigned the correlation context.
Also, every new feature should be discussed before changing im case to avoid argument after you have finished the prototype codes.

The above code is only

This will make duplicated injection for every segment. I prefer we don't do this. You are making this similar to OpenTracing Baggage, which we didn't plan to support when we disigned the correlation context.
Also, every new feature should be discussed before changing im case to avoid argument after you have finished the prototype codes.

Do you have any other ideas? This PR comes from support for online flow pressure measurement.

The key issue is, injecting this has high-security and memory/network/backend overload concerns. We should carefully reconsider how to do this. Personally, I still prefer an API or a pure optional plugin. The key point is, asking users to activate this manually, then we could set up documentation with it to tell users, note the security issue.

The key issue is, injecting this has high-security and memory/network/backend overload concerns. We should carefully reconsider how to do this. Personally, I still prefer an API or a pure optional plugin. The key point is, asking users to activate this manually, then we could set up documentation with it to tell users, note the security issue.

@wu-sheng  there are some ideas about supporting it in an optional way.
step 1:
refactor some  codes of the span behavior:

define a new span callback interface for service loader
define a span filter chain to process span behavior, such as extract for the carrier or inject to carrier
the order of the span filter chain triggered is ordered by the number of callback interfaces.

step 2:
supports correlation behavior by filter chain:

the default correlation behavior is only propagate
the optional behavior will add an injection behavior, the code way is to add a filter rather than override to ensure the stability of the main process.

@wu-sheng thx for review this version. I have extracted the dangerous behavior to the optional plugin. If this behavior is tolerated, I will modify the doc and add the unit test later.

Do you think the MD5 algorithm needs to be modified?

Back to this, MD5 could be replaced by BASE64, basically in the open-source.

I am a little confused about this plugin. Could you consider simply providing ContextManagerExtendService's extension? It could create an extension of TracingContext.
Then, could add new logic into TracingContext#extract w/o changing any core level code.
Make sense?

no problem


Remote types and containers

Say I have the following remote types:
mod remote {
    pub struct Seconds(u64);

    pub struct Duration {
        pub secs: Option<Seconds>,
    }
}

#[derive(Deserialize)]
#[serde(remote = "remote::Seconds")]
struct SecondsDef(u64);

#[derive(Deserialize)]
#[serde(remote = "remote::Duration")]
struct DurationDef {
    #[serde(with = "SecondsDef")]
    secs: Option<remote::Seconds>,
}

Is it possible to arrange the remote deserialize to deal with this? It gets worse with my use case, having multiple levels of nesting and things like Option<Vec<etc>>...

Not currently. We are tracking this in #723. I would love your thoughts on a design!

Ah thanks for the link! I was only searching for the "remote" keyword. I'd need to think about the approach but a short inline attribute like map!(SecondsDef) would certainly be convenient. I suppose with and its path concat slightly complicates that vs serialize_with?


[DH-881] Turn on USE_TZ

The main effective change is that datetimes returned by the API have a Z at the end to indicate they're in UTC, and Django and DRF uses datetimes time-zone-aware internally.
The main snag was that the freezegun context manager returns a naive datetime (breaking comparisons based on it) – simplest workaround was to get the frozen time using django.utils.timezone.now().

Codecov Report

Merging #594 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #594   +/-   ##
========================================
  Coverage    96.45%   96.45%
========================================
  Files          123      123
  Lines         3724     3724
  Branches       325      325
========================================
  Hits          3592     3592
  Misses          94       94
  Partials        38       38

Continue to review full report at Codecov.

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

Have put back to work in progress – just spotted some more warnings showing up in tests to look at

@marcofucci @elcct Have updated this now.

Do we need the Z in the freeze_time calls?
Shouldn't it always use tz_offset=0 if you don't pass anything?

@marcofucci I've made those more consistent, have a look now.

@marcofucci Could you also have a look at the logic around Order.paid_on (if you haven't already)?


Implement unaligned memory accesses: LWL, LWR, SWL, SWR

Good explanation: https://www.cs.duke.edu/courses/cps104/fall02/homework/lwswlr.html

Hi @AndrewSultan
I submitted a code review with updated infrastructure, which should support LWL instruction as well. I expect you to uncomment LWL instruction code in tt.core.s and check that my solution works. If not, please propose fixes.
Additionally, tt.core.s contains tests for ULW pseudo-instruction which is just LWL and LWR instructions in the same box.
Then, you may proceed by implementing SWL, SRW, and USW using the similar infrastructure.
I would highly appreciate if you create a Wiki page about those instructions, explaining them in a way you find most comfortable, however that is not mandatory.

@AndrewSultan
Additionally, it would be nice if you implemented unit tests for masked writes to RF.
See simulator/func_sim/rf/t/unit_tests.cpp for example and our Wiki manual

First of all, we should add an addition condition in if in mipt_instr.cpp:
if ( operation == OUT_I_LOAD || operation == OUT_I_LOADR || operation == OUT_I_LOADL)
Secondly, change is_loadlr function:
return (operation == OUT_I_LOADR || operation == OUT_I_LOADL) ? 1 : 0;
But the tests still do not pass.

0x402484: addiu $v0, $v0, 0xb9b4	 [ $v0 = 0x42b9b4 ]
0x402488: or $v1, $zero, $zero	 [ $v1 = 0x0 ]
num            = 0
val before lwl = 3020100
mask           = ffffffff
val = ( val & mask) | ( static_cast(get_value( num)) & ~mask);
val after  lwl = 3020100
0x40248c: lwl $v1, 0x0($v0)	 [ $v1 = 0x3020100]
0x402490: bne $v1, $zero, 289

if ( operation == OUT_I_LOAD || operation == OUT_I_LOADR || operation == OUT_I_LOADL

I performed it in a git branch. Please review the code in #397

LWL is fixed — please proceed with stores, documentation, and unit tests for masked writes.

I'm going to write wiki a little bit later.

What about stores? I have not seen them operating


[Feature]: Is there a Skeleton / Loading function?

What problem does this feature solve?
Skeleton / Loading function.
What does the proposed API look like?
No response

No, this is something you would have to implement yourself.

@biapar here is a really low-tech solution I use in my app with Tremor. You can add these three tailwind classes to anything with a height and width (in this case a Card in a Grid layout
<Card className='bg-gray-300 dark:bg-gray-700 animate-pulse'>

Thanks. I'll try it.


Cannot parse empty attribute

nav.templ:
    1 {% package templates %}
    2
    3 {% import "container/types" %}
    4
    5 {% templ NavTemplate(apps []types.AppConfig) %}
    6
    7
{%= "Go Feed Me" %}

    8

    9             {% for _, a := range apps %}
   10

   11                     {%= a.Name %}
   12

   13             {% endfor %}
   14

   15
   16 {% endtempl %}

nav_templ.go (it stops after the imports):
>>  1 // Code generated by templ DO NOT EDIT.
    2
    3 package templates
    4
>>  5 import "github.com/a-h/templ"
>>  6 import "context"
>>  7 import "io"
>>  8 import "container/types"

Strangely enough this also fails when I try 'data-turbo-permanent="true"' potentially todo with the dashes aswell?
Tried doing a "valueless" attribute without dashes like "hidden". That also resulted in the same outut.

Believe it not, it was just that I set the maximum length of an attribute name to 15 characters. I've updated this to 128.

Wouldn't mind your thoughts on: https://github.com/a-h/templ/pull/16
It would likely break your code above which uses a string:
{%= a.Url %}

This would have to be changed to:
{%= templ.URL(a.URL) %}

This would then ensure that a.URL is sanitised by templ, to remove potential attack vectors from the href attribute. To avoid the extra sanitisation, you'd use templ.SafeURL(a.URL) to mark the variable content as trusted.


HW1 [Part 3]: Added projects to ag4219.md

I think I did something I wasn't supposed to do. I saw that there was a conflict in my README from a previous commit, but it gave me the option to resolve it using the resolve tool on github, so i tried it.
But then after resolving it, ALL the .md files showed up in my hw1 branch! I'm not sure if I should close this PR and try again, or just leave it? @timkpaine

lgtm


WebDriverConfiguration should be immutable

In adding WebDriverConfiguration in v2.2.0. I decided to allow Object Initialisation through setters. I believe this was a mistake and should be removed in the upcoming v3.x. A configuration should be immutable.
To allow this I should release a further v2.3.0 release to

add a public constructor
Make public that Setters will be removed in v3.x (it is not presently possible to make a Setter only Obsolete on a property)
Properly and publically document intended configuration construction from v2.3.0+

With v2.2.0 only a few days old and downloaded 45 times it is doubtful that anyone other than myself is going to be disrupted by this change if made quickly.

v2.3.0 released with parameterised constructor and release note to avoid object initialisation of WebDriverConfiguration instances

Usage has been documented on the Wiki


Project editor - Missing padding between buttons

Missing padding between buttons in the activity name entry window.
Screenshots

Affected versions

v1.1 (build: 20230830)

Device:

Tablet 10''

Fixed in v1.2.


Add method for transit by entity class and id in Workflow Bundle

Please close this request. The continuation of this PR see in https://github.com/orocrm/platform/pull/284


Document how to style a SVG image for dark theme

Describe your use-case which is not covered by existing documentation.
The current documentation suggests using an SVG image:

Ideally you use an SVG as they can be made working nicely with the dark theme

However I'm not sure how to configure this either in the SVG itself or with a custom theme CSS file.
Looking at the generated HTML, the .svg file is included through an <img>. As I understand it this doesn't allow for embedding a style block in the .csv such as:
<style>
    svg path {
        fill: #151616;
    }
    @media (prefers-color-scheme: dark) {
    svg path {
        fill: #EAE9E9;
    }
</style>

And I was unable to get something like below working with a custom CSS file:
.custom-header__logo {
	fill: #151616;
}

@media (prefers-color-scheme: dark) {
	.custom-header__logo {
		fill: #EAE9E9;
	}
}

Any guidance on this would be appreciated.
Reference any relevant documentation, other materials or issues/pull requests that can be used for inspiration.
No response
Are you interested in contributing to the documentation?
No response

Did you use Image as logo? Because only with Image <img> is used. From my experience it is not possible to apply styling to svgs loaded via <img>.
When you use SVG Logo the svg is directly included as code and then styling works.
I've never tried to set the color via @media. In my svg I just use currentColor for stroke or fill (currentColor is also used in all the svg that are included in core and in the ionicons plugin

That worked great! Illustrator had left some markup in the .svg that was causing the SVG Logo option to fail for me originally.

That worked great! Illustrator had left some markup in the .svg that was causing the SVG Logo option to fail for me originally.


Log tags are not detailed to analyze

All logs of wizard services have same tag, and this shouldn't be like this because in this way logs are not easy to analyze. For each service, there should a tag. I think the solution is creating a logger instance with service's own tag and overwriting it to service's logger that inherited from broker.

Would you please create a new branch for this feature's development?
@giveerr

completed


Plugin support on docker image

I was trying to add some compiled plugins to the premade docker image you supply (thank you). However, since CGO is disabled in that build, and go plugins require dlopen() there's nothing i can do other than make my own docker container if i want plugin support. Would you consider providing a container build (potentially in addition to the one you already provide), that has this flag disabled, and allows dynamic linking?

I've made some improvements to the https://github.com/benthosdev/benthos-plugin-example repo but still haven't found a reasonable workflow for using real Go plugins. I'm giving up on this for now, so my general advice and support is going to be around maintaining a custom main function that uses Benthos as a library. It's more work overall but trying to document how to work with Go plugins is a real mess.


typo

master is in frozen state and will soon be updated with dev's code, we do not care about this, sorry.


HBASE-28191 Meta browser can happen NPE when the server or target server of region is null

master branch PR : #5508
branch-2 PR  #5510
branch-2.5 PR  #5512

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 56s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
3m 50s
branch-2.4 passed

+1 :green_heart:
spotless
1m  0s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 35s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
spotless
1m  1s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 14s
The patch does not generate ASF License warnings.

12m  8s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5511

Optional Tests
dupname asflicense javac spotless

uname
Linux beb17f821f68 5.4.0-163-generic #180-Ubuntu SMP Tue Sep 5 13:21:23 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 43a109098d

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
82 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 42s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 16s
branch-2.4 passed

+1 :green_heart:
javadoc
0m 24s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m  0s
the patch passed

+1 :green_heart:
javadoc
0m 23s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
180m  6s
hbase-server in the patch passed.

189m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5511

Optional Tests
javac javadoc unit

uname
Linux b5c751dc7b90 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 43a109098d

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/testReport/

Max. process+thread count
4287 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 47s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
3m 12s
branch-2.4 passed

+1 :green_heart:
javadoc
0m 33s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 59s
the patch passed

+1 :green_heart:
javadoc
0m 23s
the patch passed

_ Other Tests _

-1 :x:
unit
195m 29s
hbase-server in the patch failed.

207m 32s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5511

Optional Tests
javac javadoc unit

uname
Linux 00e5978b7f21 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 43a109098d

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/testReport/

Max. process+thread count
3884 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5511/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix more InternalImportsByDefault errors

SwiftPM doesn't build with all toolchains when the InternalImportsByDefault upcoming feature is properly enabled.
Motivation:
With the upcoming feature InternalImportsByDefault enabled, some versions of the Swift compiler diagnose this pattern:
import protocol _Concurrency.Actor

// error: package protocol cannot refine an internal protocol
package protocol Proto: Actor {
  // ...
}

Modifications:
Address these diagnostics by avoiding direct imports of _Concurrency since the module is implicitly imported publicly in every source file and selectively applying access levels to its imports does not make much sense.
Result:
SwiftPM builds with all relevant Swift compilers with InternalImportsByDefault enabled.

This is a follow on to https://github.com/swiftlang/swift-package-manager/pull/7895.

@swift-ci please test

@swift-ci please test

@swift-ci test windows

IIRC _Concurrency is not imported implicitly when building with CMake, that's why those imports were added in the first place

IIRC _Concurrency is not imported implicitly when building with CMake, that's why those imports were added in the first place

I see, I wonder if that is due to -disable-implicit-concurrency-module-import getting passed down in CMake builds for some reason? It's hard for me to imagine a reason why that flag would be needed by anything in SwiftPM, maybe it's a mistake?

@swift-ci please test

Yeah, the flag seems to be intentional:
> rg disable-implicit-concurrency-module-import
Utilities/bootstrap
671:        swiftpm_args += ["-Xswiftc", "-Xfrontend", "-Xswiftc", "-disable-implicit-concurrency-module-import"]

Sources/swift-bootstrap/main.swift
221:            "-Xfrontend", "-disable-implicit-concurrency-module-import",

Maybe that dates from the days before _Concurrency was available ubiquitously? In any case, I won't question it for now. I've put the imports back and tried to thread the needle differently.

Yeah, the flag seems to be intentional:

Can probably remove. It is indeed from before _Concurrency was always available. I recently removed it from the manifest build themselves, I just missed that bootstrap was also doing it.

@swift-ci please test Windows


Refactor some classes/methods

@tomturner I've deleted some code trying to keep repetition to minimum and reuse existing code. Let's see if it works at all.

@tomturner ping. Any update on this PR? Can I get a review please?


Invsee showing the command sender inventory.

When executing /invsee command the player that executed receives their own inventory.

Add FakeInventories support and Fix the issue.


Added Udacity - Intro to Javascript

Added Udacity's free 2-week Intro to Javascript Course

Thank you for helping


Custom Routes lack Realtime

Like said, custom routes defined per pb_hooks lack the functionality of realtime subscription.
Subscribing to the collection via pb.collection("xxx").subscribe() is not a solution since there are many use cases where access control or programmatically accessed data can't be recreated.
I am using routerAdd via pb_hooks.

Please don't open issues like this. Without any elaboration on the use case, it is not clear what you are trying to do or what actually is even the problem here.
If you have a general question how to use something, elaborate in a Q&A discussion as the threaded format is more suitable for this type of posts.


Figure out a migration strategy

Right now I assume that I can just YOLO migrations because I'm ~the only user. If people actually start using the project, I should probably have a way to run migrations safely.

Hi, I have a question that doesn't seem out of place here: where do you expect user to install the bash scripts?

@simonmysun it's just a bash script, so you can put it anywhere on your PATH
(PATH is an environment variable that tells your shell where to look for executables. You can do echo $PATH in your shell and it should print out a colon-separated list of directories; put this script in any of those.
I personally keep it in ~/.local/bin, but that might not be a directory in your PATH
if you're on windows, most of this is not quite right and this script probably wouldn't work for you)

Yes, I understand that you can add it anywhere. I'm just thinking about letting ell to read its configuration from XDG_CONFIG_HOME. I don't want to place the scripts into /usr if it is not managed by a package manager.
Thank you for the suggestion, I think ~/.local/bin is a good choice. I will guide user to download released version there.
This is however not in XDG Base Directory Specification but in FILE-HIERARCHY(7). I'm not sure if all the operating systems follows it.


Unused flag in slash command example

Possibly partially related to #909
https://github.com/bwmarrin/discordgo/blob/577e7dd4f6ccf1beb10acdb1871300c7638b84c4/examples/slash_commands/main.go#L18
This flag is set, but is never used in the example, meaning the commands are never removed.
I'll try to send in a PR if no one else gets to it. 🙂

Then when shutting down and re-running it adds them again.

@Treslex I'm not entirely sure, but that looks like you have both a guild and global command registered for that bot.
I'm not sure about the 404s, perhaps also related to the above? The commands were deleted when I tested the functionality, so I'm not sure how to reproduce right now.

Hmmm thats odd. I was just running the the example without any modifications other than the token and guild id

In this particular example I used guild commands, not global ones. Also yeah, thanks for finding this thing. I completely forgot about this flag when was trying to finish the PR.

Closed by #1103


IPAM driver documentation

Closes #635
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM

LGTM


python310Packages.python-openstackclient: 6.0.0 -> 6.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.python-openstackclient is: OpenStack Command-line Client
meta.homepage for python310Packages.python-openstackclient is: https://github.com/openstack/python-openstackclient
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.1.0 with grep in /nix/store/pg19kq78bd2h4nsv2n44qpq99wgcqf4p-python3.10-python-openstackclient-6.1.0
found 6.1.0 in filename of file in /nix/store/pg19kq78bd2h4nsv2n44qpq99wgcqf4p-python3.10-python-openstackclient-6.1.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
openstackclient
python310Packages.python-openstackclient
python311Packages.python-openstackclient

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pg19kq78bd2h4nsv2n44qpq99wgcqf4p-python3.10-python-openstackclient-6.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pg19kq78bd2h4nsv2n44qpq99wgcqf4p-python3.10-python-openstackclient-6.1.0
ls -la /nix/store/pg19kq78bd2h4nsv2n44qpq99wgcqf4p-python3.10-python-openstackclient-6.1.0/bin

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

1 package failed to build:

python311Packages.python-openstackclient

1 package built:

openstackclient (python310Packages.python-openstackclient)

Maintainer pings
cc @emilytrau @SuperSandro2000 for testing.

Result of nixpkgs-review pr 218029 run on aarch64-darwin 1

1 package failed to build:

python311Packages.python-openstackclient

1 package built:

openstackclient (python310Packages.python-openstackclient)


strings magic free style objects

In Este, I have almost 100% statically typed styles. The last problem is some rules, like  '@media (min-height: 200px)': {. Flow can't type that.
Flow can type ':hover' etc, but not strings with arbitrary values. This would help a lot:
pseudo: {
  hover: {
    textDecoration: 'underline',
  }
},
media: {
  '(min-width: 700px), handheld and (orientation: landscape)': {
    display: 'none',
  }
},
children: {
  '> *': {
    margin: 20,
  }
}

With such design, Flow can check even deeply nested values. But we need strings constants as keys. This would be good enough. Ideally, we should remove any weird strings in favour of real typable JS objects, but the man can only dream.

This https://github.com/este/este/blob/fb6930f15858aa7d7c0966bcab8c905bb4477da6/src/browser/app/themes/types.js#L138

I see the issue, but I really don't like such verbose syntax. I am pretty happy with the style object API as is. Yet there's no problem to build a simple plugin that transforms your syntax into "valid" style objects. If that's something that would work for you, I might quickly create it.

Agreed