Message edit UI: Improve accessibility of message control button handlers

Following 0dbc5df42dfa526c477c9eac9ede3f8fa4ca9985, the message-edit UI is somewhat accessible.    We should change the "Preview" handling to leave the focus on the "go back to editing" button for easy toggling.  (Probably the go-back-to-editing).
And we should audit the other buttons for similar issues; I think the "markdown help" one has a similar issue with focus not returning somewhere useful when accessed from the message editing UI.

@zulipbot claim

@edith007 are you working on it?
If you are not please reply I would like to fix this issue.

@m-e-l-u-h-a-n I have started working on it..

@zulipbot  claim

Pull request for the issue is in progress here https://github.com/zulip/zulip/pull/17345 .
I am removing my name so that i could work on other issues.
@zulipbot  abandon

We should take a look at this again after the compose box redesign has been implemented.


DescribeTag action is missing

Hi,
I'd like to use this cookbook to describe (query) AWS tags, but there's no "describe" action.

Actions:
add - Add tags to a resource.
update - Add or modify existing tags on a resource -- this is the default action.
remove - Remove tags from a resource, but only if the specified values match the existing ones.
force_remove - Remove tags from a resource, regardless of their values.

You can get that by doing the same query at the bottom of providers/resource_tag.rb:
https://github.com/opscode-cookbooks/aws/blob/master/providers/resource_tag.rb#L89-L91
Should be trivial to make a method for that, and submit a Pull request ;)


Set up local resource broker configuration

https://github.com/confluentinc/vscode/pull/396 shows the user an input box for setting the number of brokers/containers to start for the local Kafka cluster.
Instead of prompting at every invocation of the command, we could expose a setting in package.json (under contributes.configurations) that either accepts null or a number (1-4, inclusive) that would be used for starting Kafka locally.
One question is how we want to expose those kinds of local configurations; do they show up as a ⚙ on the "Local" item within the Resources view? Do we link to them from the progress notification after a user has started the flow if they want to open the settings and cancel the existing workflow?

Shelving this until it's requested; only shows up the first time local resources are launched and containers are being created. Any other startup (with existing containers) will bypass this input entirely.


xDai Report several issues

Safes used for testing:
https://safe-team-xdai.staging.gnosisdev.com/app/#/safes/0x8B22bAf910E83a66FacF8539AFC125cBc026059b/balances
https://safe-team-xdai.staging.gnosisdev.com/app/#/safes/0xB8d760a90a5ed54D3c2b3EFC231277e99188642A/balances
1 - Incoming tx of tokens different than xDai were not showing, or taking too long to arrive
2 - Balance of tokens that are not xDai are not showing

3 - Opending a incomming tx would crash the page and show an error in console

4 - The 0xB8d760a90a5ed54D3c2b3EFC231277e99188642A safe got stuck rejecting a tx. It was approved, but is still in "awaiting confirmations". When I tried that I had an error of "You have reached your event rate limit for today." for the first time. After that I keeps seeing the error from time to time
I repeated the proccess in a different safe and I was able to reject the tx, so the reject itself is working fine

5 - An error of gas estimation pop's up everytime you try to sign a tx of any kind:

6 - An error that doesn't happen in the dev env shows in the console when I cancel the tx in the MM prompt:

Note: The "Safe creation" tx disappears sometimes from the tx table, but there are issues with that tx already reported in another ticket, so is not only about the xDai: #1488

There is an rpc.xdaichain.com error that also shows up in the console from time to time, but it doesn't seems to be caused by any user interaction itself.

Trying the tx builder I took the address of a nifty token in xDai.
The tx builder cannot load the ABI, but the Contract interaction form can do it, so it could be that is the Tx builder the one giving troubles.
The contract: 0xCF964c89f509a8c0Ac36391c5460dF94B91daba5
Loom video:
https://www.loom.com/share/27f3fd38d1a645fab98cd8d6567d7f22

Tried the WallectConnect App on the site Nifty.ink that works in xDai and has WC connection type
I was able to connect to it, but trying to use the app it would tell me to switch networks, like if the safe app is not in the xdai network

1-3 are critical for me
4 is also a issue on mainnet afaik. https://github.com/gnosis/safe-react/issues/1471
The others don't seem critical for me, so no must-fixes for a first version

I think issue 1-3 were related to a backend bug, fixed by Uxio this morning. It's already on staging
Number 4 was related not only to issue https://github.com/gnosis/safe-react/issues/1471 but also to the one Mikhail was already fixing for Synthetix https://github.com/gnosis/safe-react/issues/1384
So now the fix is merged to xDai integration branch the transaction status seems correct

@francovenica TX-Builder wasn't updated in the environment used for testing xDai. Now it should be fixed

I used the WalletConnect app on xdai with https://dex.staging.gnosisdev.com/wallet without any issues today!

To complete a little more what @dasanra commented, 1 - 3 were related to the oracles not present on xDai network and the service trying to estimate the ether price of the tokens. It has been fixed ignoring the usd conversion for the tokens as there's no reliable way to get them and using DAIUSD for "ether" usd price.

1 - Fixed
2 - Fixed
3 - Fixed
4 - Tx builder not recognizing addresses from xdai smart contracts was fixed
The safe was able to recognize the network. Tried with https://dex.staging.gnosisdev.com/wallet

The rest of the issues are in this new issue #1488

If I understand the comments correctly, this one can be closed, right?
Please reopen if necessary @francovenica


Removing Tabs from Overflow Drop-down Does Not Work Properly in Some Cases

Describe the bug
I have a horizontal group of clr-tabs, some of which are in an overflow drop-down menu. Every button in the drop-down, when clicked, should move from the drop-down to the list of non-overflowing tabs, when setting clrTabLinkInOverflow = false. This does not work when there two or more tabs in the overflow menu and instead, the tabs just disappear, except for the last one left in the overflow menu.
Steps to reproduce

Create a new component and add a horizontal tab group of clr-tabs, where there are any number of clr-tab, each containing a single button that can contain any text;
Add @ViewChildren allTabs : ClrTabLink to the component class for access to the tabs' buttons;
Add a click event to every tab's button which calls a function which changes this tab's inOverflow property;
Create the function changeOverflow(event), which takes the tab's id from the event object, finds the ClrTabLink object with the same id from allTabs and switches its inOverflow property (tab.InOverflow = !tab.inOverflow). Add this function to the click events;
Compile;
Open the app in the browser and click a button. It is successfully added to the overflow drop-down menu. Click the same button in the drop-down and it is successfully removed from the overflow menu and added back to the other tabs;
Now click at least two button and add them to the overflow menu;
Click all buttons in the overflow menu until one remains there. You can see that they simply disappear and tiny white space is inserted where the clicked button's initial position was, instead of the button itself. This does not apply to the last remaining button in the menu which successfully goes back to its place among the non-overflow tabs.

Please note that there is probably a simpler way to reproduce the bug, e.g. manually manipulate the inOverflow property of only two tab buttons, but the above steps demonstrate the problem better.
Expected Behavior
A tab button which is in overflow menu and its inOverflow property is set to false, is to properly display on its original position among the tabs list and to be removed from the overflow drop-down.
If I have an event which on click, sets all overflow buttons' inOverflow property to false at the same time, then it works fine and they are all returned to their original positions in tabs list.
Another strange behavior is when I have several buttons with inOverflow=true and there is another tab button which is not in overflow but has a click event which switches one of the other buttons' inOverflow, then it works fine - the in-overflow buttons are properly moved to the tabs list and then can be added to the overflow drop-down again.
Versions
Software used for this use case:
Clarity version: 4.0.4
Angular CLI: 8.3.25
Node: 10.19.0
OS: linux x64
Angular: 8.2.14
Browsers: tested on the latest Google Chrome and Firefox
Additional Notes
I also tested another use case, where I dynamically add and remove tabs from the overflow drop-down depending on the browser window width. This scenario works fine. However, this and the previous use case do not work properly (the tabs disappear when inOverflow set from true to false and the UI breaks even more) when I use the following software versions:
Clarity version: 2.3.9
Angular CLI: 8.3.25
Node: 10.19.0
OS: linux x64
Angular: 8.2.14
Code Extracts
Here is the click event I used in the tabs component class:
@ViewChildren(ClrTabLink) tabs : QueryList<ClrTabLink>;
 changeOverflow(event){
    let currentTab = this.tabs.find((tab) => tab.tabLinkId === event.srcElement.id);
    currentTab.inOverflow = !currentTab.inOverflow;
  }

Code for dynamically adding and removing tabs to overflow drop-down depending on the browser window width:
@ViewChildren(ClrTabLink) tabs : QueryList<ClrTabLink>;
  @HostListener('window:resize', ['$event'])
  onWindowResize(event){

    let windowWidth = event.target.innerWidth;
    let latestRightPosition = 0;
    let tabWidth = 120;
    let dropdownButtonOffset = 50;

    this.tabs.forEach((tab) => {
      let rightPosition = tab.el.nativeElement.getBoundingClientRect().right;

      if(tab.inOverflow){

        if(latestRightPosition + tabWidth + dropdownButtonOffset < windowWidth){
          tab.inOverflow = false;
          latestRightPosition += tabWidth;
        }

      }
      else{
        if(rightPosition + dropdownButtonOffset > windowWidth){
          tab.inOverflow = true;
        }
        else{
          latestRightPosition = rightPosition;
        }
      }
    });
  }

Hello, to help us triage can you please make an isolated reproduction for us to view? https://stackblitz.com/@clr-team has some starters. Also do you have Ivy disabled?

I am sorry for the delay. Here is the demo that demonstrates the bug. Just click on the last two buttons in the overflow dropdown menu. They will disappear instead of being added to the other buttons not in the overflow menu.
I am using the latest version of Angular in the example so Ivy should be enabled.

Hello, Bojidar,
Thanks for the provided demo. But I can't reproduce the problem looking at it. The items move between the overflow and main menu as expected.

Hello, this is very weird as it still does not work for me when I click on the Settings or Images tab, they just disappear. Have you tried clicking those tabs?

I managed to fully reproduce in Firefox and partially (only key-focus handling) in Chrome. Will need to be investigated further.


Please consider supporting homebrew package management

Thank you for creating this tool! It looks like a perfect solution to our "documentation with the code vs documentation where everybody knows how to find it" debate.
Would you consider adding support for the excellent home-brew package manager?
The exact details are outline in a few different sections on the brew docs page but in short:

a small ruby file (called a formula) is added to this repo and that file points to the binary release files hosted here on github
a pointer to that ruby file is either included in the upstream list that is distributed by the homebrew team or can be used independently.

Once a given instance of brew is aware of the formula for mark, a user only needs to type brew install mark to get the latest pre-compiled binary (or, built from source) installed into $PATH.
There is a small tool meant to assist in creating formula for golang programs called fgo: https://jkawamoto.github.io/fgo/

Hi! I'm a Linux user and don't really know/care much about Mac OS. I would be really glad to accept a PR with this brew's tap which probably can be copied from another project: https://github.com/reconquest/tubekit/blob/master/tubekit.rb since we don't need to use Go and we have releases to download.

@kovetskiy  No worries. I am primarily a *nix user too but for work we have a lot of mac users. There is a *nix port of homebrew as well.
You're correct, there is technically no need to include 'build it yourself' instructions in the formula if there is already a bre-built binary available so fgo could be overkill; making small changes to an existing file would be sufficient.
And as it turns out @mmiranda beat me to it with #137.

I think this issue can be closed since #137  is already merged.
I confirm mark is being installed on my computer with no issues.
brew tap kovetskiy/mark
brew install mark


Sp 3966

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/pdi-pur-plugin/build.xml clean-all resolve jacoco && ant -f plugins/pdi-pur-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/imp/Import.java
  engine/src/org/pentaho/di/imp/messages/messages_en_US.properties
  plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java
  plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/messages/messages_en_US.properties
  plugins/pdi-pur-plugin/test/org/pentaho/di/repository/pur/PurRepositoryUnitTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change: -7.2% :small_red_triangle_down:
Complexity Change: -7.7% :small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.3% :small_red_triangle_down:
Line Change: -3.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.loadfileinput.LoadFileInputField

Branch Change: -2.3% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -2% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.www.TransformationMap

Branch Change: -9.1% :small_red_triangle_down:
Complexity Change: -6.2% :small_red_triangle_down:
Instruction Change: -10% :small_red_triangle_down:
Line Change: -10% :small_red_triangle_down:

Note: The list of offending coverage changes extends beyond the presented results and was truncated to prevent reaching the comment size limit.

License header violations

:heavy_exclamation_mark: Couldn't find acceptable copyright on engine/src/org/pentaho/di/imp/Import.java

:heavy_exclamation_mark: Couldn't find acceptable copyright on plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java

:heavy_exclamation_mark: Couldn't find acceptable copyright on plugins/pdi-pur-plugin/test/org/pentaho/di/repository/pur/PurRepositoryUnitTest.java

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java

:heavy_exclamation_mark: Line has trailing spaces at Line: 3,018

@AliaksandrShuhayeu same here. +checkstyles

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/pdi-pur-plugin/build.xml clean-all resolve jacoco && ant -f plugins/pdi-pur-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/imp/Import.java
  engine/src/org/pentaho/di/imp/messages/messages_en_US.properties
  plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java
  plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/messages/messages_en_US.properties
  plugins/pdi-pur-plugin/test/org/pentaho/di/repository/pur/PurRepositoryUnitTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.trans.step.StepMeta

Branch Change: -0.7% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.8% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Couldn't find acceptable copyright on engine/src/org/pentaho/di/imp/Import.java

:heavy_exclamation_mark: Couldn't find acceptable copyright on plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java

:heavy_exclamation_mark: Couldn't find acceptable copyright on plugins/pdi-pur-plugin/test/org/pentaho/di/repository/pur/PurRepositoryUnitTest.java


Made tweaks to core architecture.

I've divided the core into different, more distinguished parts that has clearer areas of responsibility.
This will make it easier with things like adopting the new Roslyn bits.
IScriptHost

Contains the environment visible to scripts.

ICakeContext

Acts as a context for scripts aliases and extensions.

ICakeEngine

Responsible for execution planning, workflow and rules.

IExecutionStrategy

Handles task execution.

Cough...
https://visualstudiogallery.msdn.microsoft.com/a204e29b-1778-4dae-affd-209bea658a59

@patriksvensson only had some super minor formatting iisues, really like this PR, especially cleanup of CakeEngine and the introduction of  CakeContext, makes allot of sense.

@gep13 good one :+1:  could spot a mile away just by having visible white space in my editor (as all should, along with 4 spaces instead of tabs;))

@gep13 Haha, perfect! Will install that one! :smile:

Yip, it is one of my new favourites :smile_cat: makes all trailing whitespace really obvious, so no excuses to leave them in.


Login only from ip

Be able to make a user log-only with IP but still be able to login with password like have to have the same IP and know the password too
or if the IP is the same login auto

https://github.com/NikitaCartes/EasyAuth/wiki/Sessions

i should really stop opening issues without even testing the mod

But you can always like allow x use from a array of ips?

Nope, there's no such feature

ok then that can be posible?


Make hack/make.sh work on FreeBSD

On FreeBSD link command doesn't have -T flag, so we user -h instead.

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "freebsd-build" git@github.com:kvasdopil/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

LGTM

This change in itself looks good, but I don't think this should be merged already, given that FreeBSD support isn't there yet, and pretty much an early WIP
For that reason, I'm -1 (for now)
Also ping @tianon @jfrazelle for hack/make :)

I think this'd be simpler if we just finally abandon -T since it's not as well-supported as I thought. :disappointed:
Something like this should do the trick:
rm -f bundles/latest
ln -s "$VERSION" bundles/latest

given that FreeBSD support isn't there yet, and pretty much an early WIP

well, it won't go anywhere if it cannot build :wink:

given that FreeBSD support isn't there yet, and pretty much an early WIP
Actually i have working FreeBSD fork, so FreeBSD support is relatively easy to add. But i've decided to go in small steps and create a number of small PRs to check that nothing is broken and devs are ok with quality of code.

@kvasdopil I know you're working on support — and I'm looking forward to see it arrive — but before it is actually there, I don't see a reason to include this. If, for example, FreeBSD doesn't become a reality, there's code in the repository that doesn't do anything.
However, if the suggestion that @tianon gave above https://github.com/docker/docker/pull/13477#issuecomment-105627688 works, I'm +1 on using that, because that is a generic solution and not targeted specifically at FreeBSD.

@kvasdopil I personally agree 100% with the small incremental approach -- much easier to review :+1:
As for this specific change, I'm mostly interested in keeping our shell scripts as light as possible -- they're already big fat heavyweights, so I'd rather we not bulk them up any more than necessary. :+1:

@tianon @thaJeztah done

still LGMT, @tianon ?

LGTM :+1:

Thanks, @kvasdopil! Go FreeBSD! :)


UHF-8100: Remove cookie compliance patch

UHF-8100

What was done

Removed custom cookie compliance patch
Replicated the same functionality in helfi_eu_cookie_compliance module

How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-8100

Run make drush-updb drush-cr

How to test

[ ] Check that this feature works
[ ] Check that code follows our standards

Nevermind, this is not worth the effort.


Android Build Has Annotation Dependency Error

Describe the bug
With RN 0.60.3 the Android setup for Detox from the docs is causing an issue in the build (while running detox build -c android). Happened on a new React project.
To Reproduce

[x] I have tested this issue on the latest Detox release and it still reproduces

Environment (please complete the following information):

Detox: 13.1.1
React Native: 0.60.3

androidError.txt

As a temporary workaround add the following to your android/app/build.gradle
configurations.all {
    resolutionStrategy {
        // TODO: Line below required for Detox since version 12.5
        //   It should removed as soon as the project compiles without it
        force 'androidx.annotation:annotation:1.0.0'
    }
}

The issue is that Detox uses version 1.1.0 for android.annotation, while react native itself (or any of your dependencies) uses version 1.0.

Same issue. Workaround fixed the problem for me. Thanks, @elyalvarado

The issue still exists, and shouldn't be closed unless addressed, even if its just a documentation note to warn users having this error

The issue still exists, and shouldn't be closed unless addressed, even if its just a documentation note to warn users having this error

Closing in favor of #1689. Please refer to the versions support table for latest info on this.


Update<EMAIL_ADDRESS>

Update contains a fix for VirtualList, where it would not bubble scroll events: https://github.com/mesosphere/reactjs-components/pull/383

works like a charm!

Integration tests seem to be legit failures.

run integration tests

run integration tests

Merging as integration tests for 1.8 just don't work and this is for the 1.8 branch.


Can't add package for meteor version <IP_ADDRESS>

Hye, i had problem to add the package on my platform..
Any help?
I got error as below,

I have tried your solution proposed,
$ cd ~/workspace
$ git clone https://github.com/metstrike/meteor-oracle.git
$ export PACKAGE_DIRS="~/workspace"
$ cd ~/workspace/todos-oracle
$ meteor add metstrike:meteor-oracle
but it doesnt worked.
TQ.

Hello, were you able to resolve this issue? Based on the screenshot I assume you tried to install the package on windows, but we haven't yet built and published to Atmosphere the package for windows platform.

On May 18, 2016, at 2:11 AM, ctwardatulsakinah<EMAIL_ADDRESS>wrote:
Hye, i had problem to add the packing on my platform..
Any help?
I got error as below,
TQ.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Hi,
Sorry I didn't notice that you haven't yet publish it for Windows.. Thank you for replying.. But I can't resolve the issue, so I try to use the npm.
I hope you'll be able to publish it for Windows soon.. :'D
Because i really had difficulties to install and setup the oracle driver in Windows for my meteor project.


Error in corex test

After solving the error please make sure the test is running
Test Location
/sandbox/code/github/threefoldtech/jumpscaleX_libs/JumpscaleLibs/clients/corex/CoreXFactory.py
Error
JSX> j.clients.corex.test()
TEST NO AUTH
Wed 12 13:37:06 orex/CoreXFactory.py -  41 - test                               : EXCEPTION:
    TypeError('process_start() takes 2 positional arguments but 3 were given',)
--TRACEBACK------------------
<stdin> in <module>
    1
/sandbox/lib/jumpscale/JumpscaleLibs/clients/corex/CoreXFactory.py in test
    83    test(False)
/sandbox/lib/jumpscale/JumpscaleLibs/clients/corex/CoreXFactory.py in test
    41    r = cl.process_start("mc", "mc")
-----------------------------

process_start() takes 2 positional arguments but 3 were given

verified on branch unstable :
JSX> j.clients.corex.test()
TEST NO AUTH
open browser to:http://localhost:8002/attach/94115190009280
bin	       external_information  online_meetings  sys
blog	       faq		     opt	      team
boot	       home		     org	      technology
circles        how_to_buy	     presentations    test_dir
collaboration  itenv		     proc	      test_dir2
concepts       latest		     projects	      tf_farming
cooperatives   legal		     root	      threefold_app.md
data	       lib		     run	      tmp
definitions    lib32		     sandbox	      usr
dev	       lib64		     sbin	      var
docs	       media		     srv	      videos
docsites       mnt		     strategy	      web_resources
etc	       news		     support

TEST WITH AUTH
open browser to:http://localhost:8002/attach/94115190009280
bin	       external_information  online_meetings  sys
blog	       faq		     opt	      team
boot	       home		     org	      technology
circles        how_to_buy	     presentations    test_dir
collaboration  itenv		     proc	      test_dir2
concepts       latest		     projects	      tf_farming
cooperatives   legal		     root	      threefold_app.md
data	       lib		     run	      tmp
definitions    lib32		     sandbox	      usr
dev	       lib64		     sbin	      var
docs	       media		     srv	      videos
docsites       mnt		     strategy	      web_resources
etc	       news		     support

ALL TEST OK


how to add: position: fixed && opportunity to attach widow

https://codesandbox.io/s/ywv587r001
salute, I wonder whether it is possible through add css position && opportunity select text in textarea (via props/event) ?

You can totally use CSS to set the position fixed just like you did. Then you can use x and y props to set the position of the element, based on your requirements. I guess that if you want the component to have a fixed position then you don't want to make it draggable. Therefore use the draggable prop to disable this functionality. This will solve also the text selection problem.


fixed bug, correct gc_ref and chr_length added for new chromosomes, not 0.0

I dont know if she has an account yet, I can ask her. And yes she should definitly test this .


CMake Install for SDK and Toolchain and Github Action Build

This will add some CMake steps to download and extract the toolchain and the zigbee SDK.
Also Github will build this project and makes switch and light binaries available for download.

@doctor64 Is it possible for you to test the generated binaries from the github action?

Thank you very much for your contribution!
I'm not really github actions and cmake expert, so I have few questions:

As I understand, this will use windows based build system, correct?
Will this PR broke existing linux build? (I think yes)
Can you rework it to be cross-platform? Sorry, as I said i not expert in this.

I will check generated binaries from your repo ASAP, may be tomorrow

Yes, it is Windows only. The only reason for that is the fact, that I was only able to find the windows compiler. If you can point me to the Linux toolchain, I will implement this as well.

See docs/linux_build.md

Sorry I didn't see that. Will work on that.


Lexer generator

This is the first step toward adding support for raw blocks and arguments.
The compileLexer function is highly inspired by moo, but has support for matching groups which moo doesn't have.

Thanks for the comments, I fixed everything 😄 Also, the benchmark input is now generated instead of being stored, as discussed.


Allow to set default index name at service level

Motivation and Context (Why the change? What's the scenario?)
This PR addresses issues https://github.com/microsoft/kernel-memory/issues/349 and allows to specify the default index name at service level.
High level description (Approach, Design)
This PR introduces a new property in the KernelMemoryConfig class to set the default index name, that will be always used if no one is specified, regardless the memory and content storage used.
The configuration can be done using appsettings.json:
"KernelMemory": {
  "Service": {
    //
  },
  "ContentStorageType": "SimpleFileStorage",
  // "AzureOpenAIText", "OpenAI" or "LlamaSharp"
  "TextGeneratorType": "",
  // Name of the default KM index/Qdrant collection, when none is specified
  "DefaultIndex": "default",
  // Data ingestion pipelines configuration.
  "DataIngestion": {
    // ...
    }
  }

Or directly in code:
var kernelMemoryBuilder = new KernelMemoryBuilder()
    .With(new KernelMemoryConfig { DefaultIndex = "my_index" });

If the DefaultIndex property isn't set, the value default will be used.

Thanks for sending it. There are some changes I would avoid, e.g. the pipeline definition.
I created a similar PR here https://github.com/microsoft/kernel-memory/pull/354 if you want to test it out.

Proceeding with #354


Add support for window transparency

[ ] Patch chromium to support window transparency

[x] Mac
[ ] Windows
[ ] Linux

[ ] Implement on atom-shell side

[x] Mac
[x] Windows
[x] Linux

[ ] Window shadow on Mac
[ ] Click through transparent parts of window
[ ] Resizable transparent window
[ ] Set window shape

Patches in libchromiumcontent:
https://github.com/brightray/libchromiumcontent/blob/atom/patches/use_transparent_window.patch
https://github.com/brightray/libchromiumcontent/blob/atom/patches/render_widget_host_view_mac.patch
Fixes #381.

Clipping a window on Mac into a custom shape seems quite difficult in Chromium, traditional ways only work if we have GPU acceleration turned off, and I'm not familiar with how GPU acceleration works in Chromium. In upstream there is an issue to implement window shape API on Mac, I think we should wait for it instead of spending too much time hacking on it.

@zcbenz Can you keep this PR open while we wait? We're using this version at the moment and since we don't care too much about clickthrough it works for us quite well

@paulcbetts Sure, feedbacks are welcomed!

@zcbenz So far, everything is :gem:

So happy about this!

@paulcbetts I'm going to merge this, are you cool with it?

@zcbenz I think it's still broken on certain Win32 platforms when DWM is disabled / you've severely fiddled with your settings but we should address that in a separate PR because it's an edge case

:heart_eyes:

@zcbenz Awesome feature, I just checked on several devices on Linux, it works fine on a device with an nvidia, but transparency does not work on Intel HD devices, even with args --enable-transparent-visuals --disable-gpu.
Here is an extract of lspci -v
00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) (prog-if 00 [VGA controller])
	Subsystem: Intel Corporation Device 204f
	Flags: bus master, fast devsel, latency 0, IRQ 47
	Memory at f7800000 (64-bit, non-prefetchable) [size=4M]
	Memory at e0000000 (64-bit, prefetchable) [size=256M]
	I/O ports at f000 [size=64]
	Expansion ROM at <unassigned> [disabled]
	Capabilities: [90] MSI: Enable+ Count=1/1 Maskable- 64bit-
	Capabilities: [d0] Power Management version 2
	Capabilities: [a4] PCI Advanced Features
	Kernel driver in use: i915

Do you have a clue ?

@lionep I suggest testing it with Chrome extension's alphaEnabled API and then report the bug to Chromium, there is little we can do in atom-shell.

Should we create an issue for being able to click through transparent pixels? I noticed that in the docs you have said this is blocked by an upstream bug. The node-webkit project has support for this, but i'm not sure how different their implementation is:
https://github.com/nwjs/nw.js/wiki/Transparency#click-through-transparency

wow nice , can we use it on atom editor now?

Is there anything one could do to help?


version 1.47.1 debug prelaunchtask does not get called on restart

VSCode Version: 1.47.1
OS Version:mac os 10.15.5

Steps to Reproduce:
1.install vscode 1.47.1
2.open a nodejs project which works fine before this version (like in version 1.46.0)
3.start debug => the status bar shows now is building and then start the project
4.press restart in debugger => some times it calls the prelaunchtask some times it doesnot call
5.reinstall version of 1.46.0 => it works fine

Does this issue occur when all extensions are disabled?: Yes/No

Thanks for the issue, see here for a solution https://github.com/microsoft/vscode/issues/102296#issuecomment-658889772
/duplicate


fix: Publisher $restrictions regex to FCPATH

Description

fix regex

Related: #5792
Checklist:

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

Add |woff2?|. Im not the best on regex, can someone validate it?

@haugli92 Thanks!
Added.

@haugli92 Thanks!
Added.

I missed this one but I think it is now unduly restricting .htm files (that was the original intent of htm?. Do we need htm?? for some reason?

Also woff2 should've been covered by woff?, right?

Ah, nope - I'm a dummy, never mind me. 🤦♂


option to hide buttons

Is there an option to just show the KPS?


passing html atribute to widget

Hi;
Thanks for this great plugin. I am using it and I like it very much.
My problem as follows:
I could not pass html attribute to widget.  To pass attribute I have to create new custom widget. Can you do some thing to do that?
Thanks.

Hi.
I'm glad that you like the plugin. We're now a bit overwhelmed with work, so probably we won't release new features soon, but we keep this as a feature request.
We accept pull requests also ;)


Chrome Warning about touchstart event

For a few months now, I've got this warning in my JS console when using ngx-longpress2.

Topic is related to this

hi @damiencuvillier thanks for reaching out!
do you have any idea how to fix that?

@damiencuvillier If you are still facing this issue please consider to reopen this ticket
thank you


Fix ASAN failures

Quick Look and it looks like a false positive when e.g. appending to arrays.

  Why not just run the ASAN tests on MacOS, disable the leak sanitizer, or… at least…remove the badge for that workflow so that it doesn't look like I need to worry about using this package when I look at the README?  You've got a lot of alarming looking badges!

Thanks @dabrahams should nice it to macOS. Just to clarify "a lot of alarming badges" - which other badges do you find alarming?

Swift and platforms badges are red.  Codecov too but that's a separate issue.

Oh, that's the color choice (orange) of swift package index which renders those - maybe another color would be better by default if it viewed as some error wdyt @finestructure ?

I believe it is chosen to match the swift logo though.

Yes, it's the "Swift orange". I vaguely remember that the colour was configurable via some parameter but I'm not 100% sure. Do you want to raise an issue and we'll take a look?

Will do, thanks!

https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/discussions/3410 for that.

https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/3411

@dabrahams
Fixed this by moving to macOS for running ASAN/LSAN as you suggested.
Above https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/3411 for making the badges not signal CI state.
The codecov badge will eventually be fixed as test code coverage improves.


Extend Failure Detector functionalities

Describe problem solved by this pull request
Main goal of this PR is to extend Failure Detector logic to handle ESC arming state.
Some types of ESC require that you send them an arm command before they can actually spin.
If they succeed transitioning to the armed state, they will send back an acknowledgment to PX4.
This PR extends Failure Detector logic to handle this kind of information:
after the vehicle is armed Failure Detector will wait 300_ms to get an answer from the ESCs.
If within this time window all the ESCs do not report to be armed Failure Detector will auto disarm the vehicle.
Checks are performed using esc_status.esc_armed_flags .
During the development of this PR i've also addressed the following points:

Added autodisarm if system is in lockdown. Having the vehicle in lockdown but still armed can be really misleading from a UX standpoint.

Avoid triggering flight termination if vehicle is already in a lockdown state

Introduced a parameter COM_LKDOWN_TKO to extend/disable the "lockdown window".
In some development cases it might be super useful having this logic disabled.

I rebased on top of master to re-run the CI checks

Would I be correct in thinking that this has no docs implication? - current failure detector docs are here: http://docs.px4.io/master/en/config/safety.html#failure_detector

@cmic0 Could you please address Beat's comments please?

Would I be correct in thinking that this has no docs implication? - current failure detector docs are here: http://docs.px4.io/master/en/config/safety.html#failure_detector

@hamishwillee  I'd say we would need to add something to this sentence:
The failure detector is active in all vehicle types and modes, except for those where the vehicle is expected to do flips (i.e. Acro mode (MC), Acro mode (FW), Rattitude and Manual (FW)).
as now can be disabled with COM_LKDOWN_TKO

@bkueng addressed the final comments. Let me know if you want me to clean the commit history up or if this is already good to go


Automatically added header below to the header

The title is kind of vague. Do you have any more details @gopicg?


Fix brew runtime error on travis

The reason why my last PR failed https://github.com/sociomantic-tsunami/libdrizzle-redux/commit/938c93189b60ea891241e34c9d9aa188658168bf ...

Thanks, merged in commit 9c68b29.


bugfix: lazy load workload engine

Description

Fixes https://github.com/dapr/dapr/issues/7449
Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@famarting please fix the linter issues

I was wondering if we should close this PR as after this comment https://github.com/dapr/dapr/issues/7449#issuecomment-1910786156 it became clear that the strategy implemented here won't be helpful for the majority of the usecases

@famarting Did this PR fix the issue for you? i.e. if GetWorkItems is not called explicitly then is Engine autostarting when calling the Dapr Workflow APIs?
Looking into the code, seems this solution won't work. When Dapr APIs are called, currently we already wait for engine to be started here: https://github.com/dapr/dapr/blob/c53fc16211e265cf486701bba9c2e94f50ee000f/pkg/api/universal/workflow.go#L37
and so the component implementation won't be called itself.
If we want to start the Engine at every Dapr Workflow API call, it needs to be done at API level itself.
But IMO we shouldn't do this. The reason being if GetWorkItems is not called i.e. there is no stream listening for workItems, then the workflow won't be executed itself. The DTF Grpc executor will be anyway be blocked waiting for an application to connect (and send the work item). So isn't it better and more cleaner to wait for that at API itself, which is what we are doing rn. We can actually add a log line saying Waiting for engine to be ready.. something like that. Thoughts @cgillum ?

We can actually add a log line saying Waiting for engine to be ready

That log message won't mean anything to users and will hardly be discoverable.

But IMO we shouldn't do this. The reason being if GetWorkItems is not called i.e. there is no stream listening for workItems, then the workflow won't be executed itself. The DTF Grpc executor will be anyway be blocked waiting for an application to connect (and send the work item). So isn't it better and more cleaner to wait for that at API itself, which is what we are doing rn. We can actually add a log line saying Waiting for engine to be ready.. something like that. Thoughts cgillum ?

I think it's a valid point worth considering. The choice is to either block the API caller until a worker is ready or to accept the message and block its processing until the worker is ready. Either way, these APIs are designed to be asynchronous, which is why I didn't see any harm in accepting the message even without a ready worker. It's one less error condition for the developer to worry about. But at the same time, if we know that the worker is supposed to be running, I can see that it might make it easier to diagnose problems if the client call times out because a worker didn't come online within a reasonable amount of time. Without a timeout, the developer is left wondering why nothing is happening after they try to start a workflow.
I could go either way on this.

I'm closing this PR, as it has been mentioned this change does not address the problem I described.
For me my root problem was that the streamed connection to GetWorkItems was failing on my end for other reasons. So the changes in this PR do not bring any improvements


[Torch] add loadWidgets to work with Fastload Utils app

I would like to make a feature request: Show widgets only optional (configurable with settings?). Torch should be just white on my Bangle, without any widgets.

How about hiding them by default with require("widget_utils").swipeOn() ?

@halemmerich I get some problems navigating back and forth Slope Clock <-> Torch, using Quick Launch.

Reset watch with long press hardware button to get a "Loading..." message.
Slope Clock is loaded.
Swipe to trigger loading Torch via Quick Launch.
Torch is loaded with a completely white screen. SlideOn works as expected.
Go back to Slope clock with red back button.
Slope Clock is loaded, 24pix reserved for widget but they don't respect the theme and they are hidden - not shown as expected. They are shown by a swipe down.

Sometimes the Red back button will show permanently on Torch after going back and forth between it and Slope Clock.
If I exit Torch via hardware button click the widget field will be hidden and the clock take up all screen space. SlidOn seem to persist and widgets are shown on a swipe down in this case.
One solution to this is probably to add slideOn to Slope Clock as well - might try that. But I thought this should be interesting for you if there is something to consider in Fastload Utils in some way.

You probably need to restore the normal widget state during remove using show from widget_utils. If this is not done, slopeclock (or any clock for that matter) gets modified widgets it does not expect. Adding swipeOn to slopeclock would only cover up the problem.

Great, thanks - will try.

I think it makes sense to check if Fastload Utils is installed, because then we don't need the widgets at all. I changed to using read instead of readJSON.
Your tip regarding using widget_utils show in the remove-function helps, but one issue remains:

If exiting Torch when the swipeOn widgets field isn't visible Slope Clock loads with widgets behaving as expected.
But if exiting when the swipeOn widget field is visible it will persist and show a white background when it should now be black (I use dark theme), and the Analog Clock Widget still show when it should not. Doing g.dump() from Web IDE gives this:

But on the watch the white widget field is there as described insted.

I landed on just loading widgets and hiding them. I think this is ready for merging.

But if using a clock without widgets, e.g. AI Clock, some widgets, e.g. Light Switch Widget, will interfere with the clock (and this would be alleviated by keeping the .hideinstead...).

You shouldn't ever be using fast load with a clock without widgets? The loading code should actually check this and if there are no widgets it will load the clock normally.
Is it an issue you were experiencing or just that you thought might happen?

This looks good though - merging...

I guess AI Clock could be modified with a widget_utils .hide

Yes, it should ideally do that instead of for (let wd of WIDGETS) {wd.draw=()=>{};wd.area="";}, but the code that it there should work.

e.g. Light Switch Widget, will interfere with the clock (and this would be alleviated by keeping the .hideinstead...).

I would suspect that the widget calls it's drawing code directly from some handler/event/timeout instead of calling the draw method. This would cause this problem when .hideing the widgets, since that essentially only replaces the draw method of the widget. Changing those direct calls to drawing code to the widgets draw would help to catch those cases.

Changing those direct calls to drawing code to the widgets draw would help to catch those cases.

Yes, that'd do it. Just looked at lightswitch and it's doing all kinds of stuff - I've just made some changes (see above) and that should fix it


#897 breaks workspace with cartorgrapher_ros

I have a workspace consisting of:

ceres-solver
cartographer
cartographer_ros

I need to build this workspace using catkin build because it contains non-catkin-packages.
If I try to build this with a recent version of catkin, I get this error

CMake Error at /opt/ros/kinetic/share/catkin/cmake/test/gtest.cmake:381 (add_library):
add_library cannot create imported target "gmock" because another target
with the same name already exists.
Call Stack (most recent call first):
/opt/ros/kinetic/share/catkin/cmake/all.cmake:147 (include)
/opt/ros/kinetic/share/catkin/cmake/catkinConfig.cmake:20 (include)
CMakeLists.txt:44 (find_package)

This is caused by the changes in #897. ceres already defines the targets gmoc an gmoc_main.

Please include the information about your platform and what exact version you are having installed.

Added version information in the first post.

It seems that neither of the packages is released / registered with ROS atm. That is why we didn't catch this problem.
I don't know much about these packages but ceres seems to not use catkin directly so I should not be affected.
cartographer and cartographer_ros seems to use custom CMake function to find GMock before catkin. Maybe it would be sufficient to update those by changing the order of these two steps (or even remove the GMock lookup completely since catkin includes that functionality now).

Never mind, there is a fix for cartographer_ros for this...

For reference: googlecartographer/cartographer_ros#768


[BUG] Unable to install games via Hydra Launcher

Describe the bug
When attempting to install any game via Hydra Launcher, none of the games display any installation options. Instead, each game shows "No download available". Reinstalling the launcher did not resolve the issue.
Steps to Reproduce

Open Hydra Launcher.
Navigate to the game installation section.
Choose any available game to install.
Note the absence of any download option and the message "No download available".

Expected behavior
Expected that practically every game would have a download and installation option available through Hydra Launcher.
Screenshots

Operating System
Windows 10
Hydra Version
v2.0.3 "Leviticus" (latest)
Additional Information
The issue occurs consistently every time I attempt to install any game via the launcher. I noticed that this started happening after reinstalling Windows 10. Everything worked fine before the reinstall.
Reinstalling Hydra Launcher did not fix the issue.

1 go to settings
2 click in the download sources
3 click in button add source
4 Copy these links one at a time to gain access to all the download sources :
https://hydralinks.cloud/sources/xatab.json
https://hydralinks.cloud/sources/tinyrepacks.json
https://hydralinks.cloud/sources/onlinefix.json
https://hydralinks.cloud/sources/kaoskrew.json
https://hydralinks.cloud/sources/gog.json
https://hydralinks.cloud/sources/fitgirl.json
https://hydralinks.cloud/sources/empress.json
https://hydralinks.cloud/sources/dodi.json
https://hydralinks.cloud/sources/steam-amiga.json


Fail to dump FRU bin file by IPMIUTIL command

Steps to reproduce:
Run ipmiutil command to dump FRU
ipmiutil fru -d <fru.bin>
Expected result:
Successfully dump FRU file
Actual result:
Got segmentation fault when dump FRU
Analysis:
frubuf is only malloc from load_fru function but ipmiutil doesn't call this function or malloc frubuf before fwrite --> segmentation fault.

Sorry.  That got broken when ifru.c changed to malloc() the fru buffer rather than being statically declared.  This needs to change to leverage pfrubuf rather than frubuf.

07/05/2023 ARCress ipmiutil-3.1.9-2  changes
lib/lanplus/lanplus.c - GH13: hpm netfn2c/cmd34 error, bridge netfn06,07/cmd34
doc/ipmiutil.spec - GH17: do not abort if sensor command fails
util/isensor.c - GH17: if MALFORMED, not fatal, show message before correct
08/03/2023 ARCress ipmiutil-3.1.9-3
util/ifru.c - GH15: fails to dump FRU with -d (added load_fru for fdump)
util/ilan.c - GH14: GH14: user set password fails w 0xCC, upriv = rd3 & 0x0F
See https://ipmiutil.sourceforge.net/FILES/ipmiutil-3.1.9-3.tar.gz and
https://ipmiutil.sourceforge.net/FILES/ipmiutil-3.1.9-3.el7.x86_64.rpm  for a version that fixes this issue.
This will be applied to github soon.

The 'util/ifru.c - GH15' change was applied in ipmiutil-3.2.0 and in github 12/05/2024.


Add openssl support for s390x

This PR provides the openssl option for the KBS https server because rustls does not work on s390x.
For selection of rustls and openssl, #[cfg(not(target_arch = "s390x"))] and #[cfg(arget_arch = "s390x")] are used. One curious thing is that [target.'cfg(not(target_arch = "s390x"))'.dependencies] does not work in src/api_server/Cargo.toml. So, we still need to comment out the dependencies manually to build kbs on s390x.
I appreciate if anyone give an advice to fix the above issue.

Thank you for your comments. I updated the commit to reflect your suggestion. Is there is any further problems, please let me know.

I added rustls and openssl features instead of architecture comparison. Further, I updated the revision of attestation service so that my type error fix is included. Please review my code.
Thanks,

FWIW I'm ok with that PR, but we should follow up with a CI addition to build with openssl.

@sameo Thank you so much. It looks like there are some errors in CI. Let me fix them.

I fixed src/kbs/Cargo.toml and changed the workflow for openssl CI. If you agree with the change in the workflow, please approve it. If not, please let me know. I will revert the change. Thanks,

@jialez0 I fixed the CI issues. I hope that this is final. I realized that feature names are commonly used by src/api_server and src/kbs. So, we need to define the same features for them.


Added JBoss JMX Console exploit deployment module.

Nice rvrsh3ll, i'll get an instance setup over thanksgiving week and test it!


OpenVPN client certificate authentication (Individual Certificate Authentication)

This pull request implements Individual Certificate Authentication for OpenVPN protocol. This pull request fixes #55.
Client certificate's Common Name is used as username during authentication process. If auth-user-pass is enabled for OpenVPN client, the password is ignored by server, and username recieved from the client is used instead of the Common Name.
Note: only self-signed certificates work presently. CA-signed certificates will not pass verification because OpenSSL library needs to be told where to look for trusted CA certificates, and this feature was not implemented in this pull request. It can be easily added in future, though.

Example:

Connect to SoftEther Server via command line client.
Create a new user, let's name him steve:

VPN Server/DEFAULT>UserCreate
UserCreate command - Create User
User Name: steve
Assigned Group Name:
User Full Name:
User Description:
The command completed successfully.
VPN Server/DEFAULT>

Generate a self-signed certificate for Common Name = steve:

VPN Server/DEFAULT>MakeCert
MakeCert command - Create New X.509 Certificate and Private Key (1024 bit)
Name of Certificate to Create (CN): steve
Organization of Certificate to Create (O):
Organization Unit of Certificate to Create (OU):
Country of Certificate to Create (C):
State of Certificate to Create (ST):
Locale of Certificate to Create (L):
Serial Number of Certificate to Create (Hexadecimal):
Expiration Date of Certificate to Create (Days):
File Name to Save Certificate to Create: /tmp/steve.crt
File Name to Save Private Key to Create: /tmp/steve.key
The command completed successfully.
VPN Server/DEFAULT>

Switch Steve to Individual Certificate Authentication with newly-generated certificate:

VPN Server/DEFAULT>UserCertSet
UserCertSet command - Set Individual Certificate Authentication for User Auth Type and Set Certificate
User Name: steve
Read X.509 certificate from file name: /tmp/steve.crt
The command completed successfully.
VPN Server/DEFAULT>

Now Steve can use the following OpenVPN client config to connect to SoftEther server:

dev tun
proto udp
remote myserver.com 1194
cipher AES-128-CBC
auth SHA1
resolv-retry infinite
nobind
persist-key
persist-tun
client
verb 3
key steve.key
cert steve.crt

Server log will contain the following lines:
2017-05-26 20:47:38.834 OpenVPN Session 1 (<IP_ADDRESS>:59213 -> <IP_ADDRESS>:1194): A new session is created. Protocol: UDP
...
2017-05-26 20:47:38.864 OpenVPN Session 1 (<IP_ADDRESS>:59213 -> <IP_ADDRESS>:1194) Channel 0: Client certificate received (subject: CN="steve"), will use certificate authentication.
...
2017-05-26 20:47:40.153 [HUB "DEFAULT"] The connection "CID-3" (IP address: <IP_ADDRESS>, Host name: <IP_ADDRESS>, Port number: 59213, Client name: "OpenVPN Client", Version: 4.22, Build: 9634) is attempting to connect to the Virtual Hub. The auth type provided is "OpenVPN certificate authentication" and the user name is "steve".
2017-05-26 20:47:40.153 [HUB "DEFAULT"] The Virtual Hub's Security Account Manager has received the following certificate from the VPN Client and accepted its contents as the certificate for when user "steve" logs in: CN=steve, SERIAL="00" (Digest: MD5="84EAC0F1521A13BF80C51F357A44679B", SHA1="5BB0C832DFA0C4FE043F8390850769CDE261E8F3")
2017-05-26 20:47:40.153 [HUB "DEFAULT"] Connection "CID-3": Successfully authenticated as user "steve".

If Steve so wishes, he can use the same certificate and key with SoftEther Client instead of OpenVPN client.

Regarding Patch Acceptance Policy: I choose option 1.

What about ca.crt?
Options error: You must define CA file (--ca) or CA path (--capath)
Use --help for more information.

ca.crt can be obtained using the command-line client:
VPN Server>ServerCertGet
ServerCertGet command - Get SSL Certificate of VPN Server
Save X.509 certificate to file name: /tmp/ca.crt

The command completed successfully.

VPN Server>

I would remove commit 2569939a18e262741bd6b5cad3ef3087c5ed6913 from this pull request and propose it in a new one, as it's unrelated.

I would remove commit 2569939 from this pull request
Done

The new strings should be added to the Traditional Chinese table (strtable_tw.stb) as well.

Pushed the fixes in a new commit (068094f)

Is the PreverifyOk variable required?

PreverifyOk was indeed not used; removed it.

You are right. I verified this behavior by creating a second valid certificate and using the private key in my client configuration and SoftEther server successfully fails the authentication

You PR has just been merged into the master branch. As a token of gratitude, your name will be added to the authors lists. Many thanks for your hard work. It is really a great addition to SoftEther.

@a-kr Thank you very much for your contribution!
@moatazelmasry2 I would have squashed the commits, as the second one is a fixup to the first.

I did squash the commits

Oh, sorry.
I was looking at the commits list in the pull request, as I usually squash them in the source branch and then merge normally.

But if it is not my repo, then I can not edit it, or?

You can, if the "Allow edits from maintainers." option is enabled, which is by default.
https://help.github.com/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork

@a-kr Build fails with OpenSSL 1.1.0f:
src/Mayaqua/Network.c:5836:11: error: dereferencing pointer to incomplete type ‘X509_STORE_CTX {aka struct x509_store_ctx_st}’
    if (ctx->cert != NULL)
           ^~

seems to be the right time to add openssl-1.1.0 to travis-ci matrix ))
I will add, stay tuned...

Are you planning to use Docker? Or do you prefer to compile the latest version of OpenSSL on Ubuntu 14.04?

I have made a quick fix in #476. It compiles with OpenSSL 1.0.1 (the older one), and should compile with 1.1.0, but I have no access to 1.1.0 at the moment and cannot verify it myself right now. Could you guys please test it with 1.1.0?

Working!
Thank you very much for the fast fix.

@chipitsine  Please do that
@davidebeatrici  Thanks for the Info

@moatazelmasry2 You're welcome.

When it will be implemented to the stable version(SoftEtherVPN_Stable)?

@DRSDavidSoft It's not implemented in the stable repository yet.

@davidebeatrici any active plans to implement it in stable releases yet, or it needs further testing?
Also, is it available in any beta or alpha releases at the moment?
P. S. I appreciate all the work the SoftEther team is doing, the quality of each section of the codes, logic and workflow is quite astonishing.

@DRSDavidSoft The stable repository is managed by @dnobori, we currently don't know when he will update it.
You can download Windows releases compiled from this repository on AppVeyor.

@davidebeatrici Thanks for the AppVeyor link, I'll try it out until this PR makes it way into the stable branch.
Out of curiosity, usually what process does it take for a similar PR to be merged into the stable branch by @dnobori? Or is it a bit more irregular and can't be said for sure?


README: production notes, db(s) cleaning

Following up on: https://github.com/maxcountryman/tower-sessions/discussions/17

Thanks for this!


Title: Adding erasing option in Drawing Pad (23)

Which pre-existing JavaScript project will you be working on?
Drawing Pad (23)
Describe your enhancement/improvement
The current available option is the "Clear" option, which removes everything, thus its not a viable option for small changes. I want to add an eraser in the Drawing Pad (23). It will enhance the usability of the application, thus providing a better UX.
@swapnilsparsh
Record

[x] I agree to follow this project's Code of Conduct
[x] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@mondalsurojit All the best 👍🏻


Media Picker - receiving selected items path

Hi,i have used your library and loaded media picker to load media where i can selete multiple images or videos,how can i get the path of file selected if i select multiple items.On the activity result how can i get the values as a listpaths.
Finally thanks for creating an amazing library,great work.

Are you not getting those in the calback? onFileChosen() ?

Thanks for your time.Yes i dont recieve the file on Mediafilechosen().
Can you help me with some lines on whether how to receive it.

Can you post some log? Also, can you check if the sample app works for you?

okay sure i will look into it and get back to you.

hi i got the files onMediachosen() after getting a look into the sample thanks for the help.


User description of Aakanksha Jain added

#1 referenced!

Made requested changes! @MJ10

Thanks for contributing!


TypeScript @ts-ignore: estimateFeeValue

Checklist

[x] I have read the CONTRIBUTING guide
[x] Good naming, keep simple
[ ] Tested desktop/mobile
[ ] Tested bright/dark
[ ] Tested en/ru
[ ] Affects money; I checked the functionality once again
[x] I checked the PR once again

Original issue
#3562

Requesting 2 approvals for this PR


Update recipe:- Retrieve your Core Web Vitals

Change getTestResults to runTest
To help folks run their test from the start

https://github.com/WebPageTest/WebPageTest-API-Recipes/pull/22


Destructuring assignments generate unreachable code

The two functions in the typescript snippet below both generate an unreachable var declaration. Although this will work as expected due to hoisting, javascript minifiers might try to remove this dead code and unintentionally introduce global variables. So it would beneficial to move this declarations to the top of the function to eliminate the (valid) warnings.
I tested with the Closure and UglifyJS2 minifiers. Both warn about this declaration but leave it (correctly) in place:
>  uglifyjs c.js -c
WARN: Dropping unreachable code [c.js:5,4]
WARN: Declarations in unreachable code! [c.js:5,4]
WARN: Dropping unreachable code [c.js:15,4]
WARN: Declarations in unreachable code! [c.js:15,4]

Closure:
JSC_UNREACHABLE_CODE: unreachable code at line 5 character 4
    var _a; // unreachable!
    ^
JSC_UNREACHABLE_CODE: unreachable code at line 15 character 4
    var _a; // unreachable!
    ^

c.ts:
export function unreachableVar(a:any[]) {
    const res = this.a.splice(0, 0, ...a);
    return res;
}

export function anotherUnreachableVar(a,b) {
    let c,d;
    switch(a) {
      case true:
        [c,d] = someFunc(b);
    }
    return 3 + c;
}

function someFunc(x) {
    return [1,2]
}

transpiles to c.js:
"use strict";
function unreachableVar(a) {
    var res = (_a = this.a).splice.apply(_a, [0, 0].concat(a));
    return res;
    var _a; // unreachable!
}
exports.unreachableVar = unreachableVar;
function anotherUnreachableVar(a, b) {
    var c, d;
    switch (a) {
        case true:
            _a = someFunc(b), c = _a[0], d = _a[1];
    }
    return 3 + c;
    var _a; // unreachable!
}
exports.anotherUnreachableVar = anotherUnreachableVar;
function someFunc(x) {
    return [1, 2];
}

Tested in tsc 1.7.5 and 1.9.0 (@next)

"unreachable" applies to statements, i.e. things that will execute. this is a declaration. var declarations are hoisted to the top of their scope. This is similar to function declaration hoisting, e.g:
function foo () {
   inner();
   return;

   function inner() { // this is not unreachable. the declaration is hoisted to the top of the scope
   }
}

So I would say this is perfectly legal JavaScript, and the checks in uglify/closure are too strict.

In case it helps, I've been working around this by moving the declarations to the top using UJS API (I run the TS compiler's output through some fixup steps using UJS, and this is one of them):
// Move var statements at the end of blocks (generated by TS for rest parameters) to the start of the block.
// This is needed to prevent unreachable-code warnings from UJS
root.walk(new UglifyJS.TreeWalker(function (node, descend) {
	if (
		node instanceof UglifyJS.AST_Block &&
		node.body[node.body.length - 1] instanceof UglifyJS.AST_Var
	) {
		node.body.unshift(node.body.pop());
	}
}));

@mhegazy Thanks for the quick response! I am aware the generated code is technically correct. But I think it is nice to produce linter passing / best practice code nonetheless. Just to play nice with other tooling. Especially since typescript is usually combined with post processing tools in serious projects (like uglify, webpack etc etc). Actually I like it that typescript compiled code is a lot less alien than babel compiled code. So I would say keep up the high standard and move the declarations to the top :wink:
(By the way, it would be really cool if there was a typescript minifier, which could take into account private fields and such +:100:)
@Arnavion thanks for the work around

with https://github.com/Microsoft/TypeScript/issues/5595 in place, this should be a simple change. but before that, it requires two passes over the tree and a lot of synchronization between the naming logic and the emit logic. in short it will be hard to do and will make the compiler slower if done. #5595 is planned for 2.0, so until then, closing this issue.

Sounds great, thanks for looking in to this!

Should we reopen this now that #5595 landed?

PRs are welcomed.

Seems to be fixed in Typescript 2.9.2 (you can try it in the playground).


updated redis perf faq to add P5

updated redis perf faq to add P5

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

@deepakverma @wesmc7777 This pull request has been closed. Microsoft contributors need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged. Read about authors working in the public repo for more information. Thank you.


torch rescore empty tensor bug fix

when hyp num is zero , it will cause core dump , see below:

please refine the lint problem.


services.mysqlBackup: Backup job might break due to garbage collection

Issue description
Surprise email from cron daemon:
subject: Cron <root@HOSTNAME> mysql /nix/store/vsj6yazxlzrhwhcdvkzp2b1wakz96gpi-mariadb-10.1.25/bin/mysqldump DATABASENAME | /nix/store/c09pc52nm3ii38f6n7qx9k5g81d55w7l-gzip-1.8/bin/gzip -c > /var/backup/mysql/DATABASENAME.gz

execl: couldn't exec `/nix/store/c280fqkd65rrpp5mfarp6cc0v4ns6rlb-bash-4.4-p12/bin/bash'
execl: No such file or directory

The cron job created by the backup service is broken. Several referenced paths do no longer exist:
[root@HOSTNAME:~]# crontab -l
# DO NOT EDIT THIS FILE - edit the master and reinstall.
# (/etc/crontab installed on Tue Aug 22 18:01:15 2017)
# (Cron version V5.0 -- $Id: crontab.c,v 1.12 2004/01/23 18:56:42 vixie Exp $)
SHELL=/nix/store/c280fqkd65rrpp5mfarp6cc0v4ns6rlb-bash-4.4-p12/bin/bash
PATH=/nix/store/qladnhrhvlcxmzv2j2g7wybfj1mmzi7w-system-path/bin:/nix/store/qladnhrhvlcxmzv2j2g7wybfj1mmzi7w-system-path/sbin

NIX_CONF_DIR=/etc/nix
15 01 * * * mysql /nix/store/vsj6yazxlzrhwhcdvkzp2b1wakz96gpi-mariadb-10.1.25/bin/mysqldump  DATABASENAME | /nix/store/c09pc52nm3ii38f6n7qx9k5g81d55w7l-gzip-1.8/bin/gzip -c > /var/backup/mysql/DATABASENAME.gz

[root@HOSTNAME:~]# ls -al /nix/store/vsj6yazxlzrhwhcdvkzp2b1wakz96gpi-mariadb-10.1.25/bin/mysqldump
ls: cannot access '/nix/store/vsj6yazxlzrhwhcdvkzp2b1wakz96gpi-mariadb-10.1.25/bin/mysqldump': No such file or directory

[root@HOSTNAME:~]# ls -al /nix/store/c280fqkd65rrpp5mfarp6cc0v4ns6rlb-bash-4.4-p12/bin/bash
ls: cannot access '/nix/store/c280fqkd65rrpp5mfarp6cc0v4ns6rlb-bash-4.4-p12/bin/bash': No such file or directory

[root@HOSTNAME:~]# ls -al /nix/store/qladnhrhvlcxmzv2j2g7wybfj1mmzi7w-system-path/bin
ls: cannot access '/nix/store/qladnhrhvlcxmzv2j2g7wybfj1mmzi7w-system-path/bin': No such file or directory

Given enough time I will eventually reimplement the service using systemd-timers (as it was already discussed in #24728) but I might not get this done in time for the next release.
edit: It seems to me that services.cron.systemCronJobs it severly broken. Changes are sometimes not detected and cron runs for quite some time with an old version of the crontab (or no crontab at all!)
Steps to reproduce

Configure services.mysqlBackup
? (not sure what I did in between)
nix-collect-garbage -d
Get surprise email from cron telling you the backup failed

Technical details

System: 18.03.git.977073d (Impala)
Nix version: nix-env (Nix) 1.11.14
Nixpkgs version: "17.03.1247.05126bc850"

Did some very quick changes as I needed to fix the backup for one of my hosts, :construction: work in progress can be found at https://github.com/fadenb/nixpkgs/commits/mysqlBackup_systemd_timers
Open issues:

Broken authentication (#24728) (@florianjacob did you by any chance already start to implement the socket auth?)

@fadenb not yet. ☹ As shown in #24728, the MySQL commands needed to set up socket auth are trivial, and I'm running it the way I showed there with a permission-reduced user successfully for a month now - with manually setting this up, though. The problem is doing this automatically.
There are two ways to get this set up automatically: Either through doing some MySQL bash script hackery like the wordpress module is doing, as well as some other modules, or making that unnecessary by providing proper NixOS options infrastructure to set up MySQL users, and therefore fixing https://github.com/NixOS/nixpkgs/pull/6963 . Sadly, the current options of the MySQL module are useless for the backup script case as well.
The latter is the route I'd want to go, ditching the problem of password handling by making socket auth the default for all users created through some mysql.users option. But I'm unsure how to solve this problem brought up by copumpkin:

Is there some way to make it clear to users that it won't delete databases? I could see it being both > scary (you'll delete my data) and counterintuitive (why doesn't my database list match my configuration.nix?)

What do you think?

PR for declarative Unix socket authentication at #29387

Hi @fadenb, I'm a little confused because there is crontab -l in your example.
However this command would show the file /var/cron/tabs/root -- the root user crontab.
NixOS edits the file /etc/crontab.
I am testing with the following steps:

In a separate terminal, watch the output of journalctl -u cron.service -f
Edit services.mysqlBackup.databases setting in configuration.nix.
nixos-rebuild switch
Note the log output -- the text RELOAD (/etc/crontab) should appear.

Could it be possible that the root user crontab (this is separate from /etc/crontab) has accidentally gotten the above contents? I'm not sure how this might happen, but could you please clarify for me.

I actually did not even know cron was in use on that system before the mail with the error :p . So if that ended up in the wrong file it must have been due to something nix related.
I can no longer reproduce the issue as I already migrated that system to a systemd-timer based backup job.

@fadenb what's your plan on posing a PR for the systemd-timer change? Is there something you still want to change? I think it turned otu well, my only remark would be that I'd personally have set AccuracySec = "5m"; to something larger like 1h, to follow the systemd.timer man page “To optimize power consumption, make sure to set this value as high as possible and as low as necessary.” - but I guess how much Accuracy is necessary differs between use cases. 🙃
Together with https://github.com/NixOS/nixpkgs/pull/29703 we could get this fixed finally. 👍

@fadenb I have merged your systemd-timer config into #29703.
This still leaves the crontab issue, which is of general interest for the release.
If you add
services.cron.systemCronJobs = [ "* * * * * root true" ];

to your config, is /etc/crontab updated correctly, and what is the result of running crontab -l as root?

Fixed in #29703


fix bug of Etcd3Registry cancel watch #4357

What is the purpose of the change
XXXXX
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4358 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4358      +/-   ##
============================================
- Coverage     62.85%   62.83%   -0.03%
+ Complexity      449      448       -1
============================================
  Files           769      769
  Lines         32995    32998       +3
  Branches       5215     5215
============================================
- Hits          20740    20733       -7
- Misses         9849     9861      +12
+ Partials       2406     2404       -2

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/remoting/etcd/jetcd/JEtcdClient.java
7.92% <0%> (-0.15%)
0 <0> (ø)

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
0% <0%> (ø)

...che/dubbo/remoting/transport/mina/MinaChannel.java
43.42% <0%> (-10.53%)
16% <0%> (-1%)

.../exchange/support/header/HeaderExchangeServer.java
66.98% <0%> (-0.95%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
81.6% <0%> (+1.53%)
0% <0%> (ø)
:arrow_down:

.../dubbo/monitor/support/AbstractMonitorFactory.java
81.08% <0%> (+2.7%)
0% <0%> (ø)
:arrow_down:

...c/main/java/org/apache/dubbo/rpc/RpcException.java
86.2% <0%> (+3.44%)
0% <0%> (ø)
:arrow_down:

...ng/exchange/support/header/HeartbeatTimerTask.java
78.94% <0%> (+5.26%)
0% <0%> (ø)
:arrow_down:

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 60c9d9d...b0c0223. Read the comment docs.


CON-638 added elasticsearch 5 support

Added another dev VM for ES that can be manually enabled to test it.
Added changes to erchef chef_index to support es 5.  It doesn't seem to like filtered search queries.
@ssd, @sdelano, @marcparadise, @chef/chef-server-maintainers

A couple of things we'll need to verify for this:

Reporting uses a forked chef_index search interface. It does not currently support ES, but a sanity check to ensure that reporting pedant tests pass would be good - the path that uses search will be hit in those tests.
We're building this because we need it, but what do we want to do in terms of documenting the change for customers? This is a factor for cloud all-in-one installations , but also customers who are on the "backendless" path.

We added some unit tests to the library in the omnibus cookbook.  I figure that the pedant tests exercising functionality should cover most of the other scenarios we care about and ensure the ES2/5 work like we expect solr to work.  It isn't automated at all - but I have run pedant tests locally for both ES2 and ES5.  There's one test that involves rabbitmq that seems to be flaking out but that test behaves the same way in master as well - so I'm not worrying about it.  All the other tests past.
I'll run the reporting pedant tests and see if that still works on just solr.

The query changes look good to me.

Ran opscode-reporting-ctl test --all using solr on this branch!  Everything still passes.


if postgis  enables pgsync not working

PGSync version: 2.1.1
Postgres version: 13
Elasticsearch version: 7.14
Redis version:
Python version:
Problem Description:
If any of the tables extended postgis extension, other table data also not indexing(if add geo type column any of the table).
Error Message (if any):

I have used successfully with postgis extension enabled. Can you provide more details.
any errors during the sync ? alternately a sample db schem and db schema wouild help too


Home Assistant 2023.8.0 Beta

Not Loaded

Logger: homeassistant.setup
Source: setup.py:209
First occurred: 20:52:53 (1 occurrences)
Last logged: 20:52:53
Setup failed for custom integration qr_generator: Requirements for qr_generator not found: ['pillow==9.5.0'].

You have to make sure that you install the version 2.0.1 (Beta) of this integration when you have HA version 2023.8.0 or greater.

You have to make sure that you install the version 2.0.1 (Beta) of this integration when you have HA version 2023.8.0 or greater.

Working


Add Cosanta / DASH / PirateCash

Could you please add to your repository 3 projects:
https://www.coingecko.com/en/coins/cosanta
https://www.coingecko.com/en/coins/dash
https://www.coingecko.com/en/coins/piratecash

Thanks for your PR! I'll review it soon and merge it in. Don't worry about the failed checks, I think my CI is borked.


Display Video frame correctly from a stream source

Hi @twistedfall ,
I would like to display video frames from the data I'm receiving via an NDI(Newtek) source. The video frames are streamed continuously from said source. I am able to display it on the screen however it seems I'm doing something wrong.
The left picture is whatI'm getting and the right one is the correct output.

The code I tried ( I'm new to rust and opencv, please correct me if I'm doing things wrong):
data is the video frame data as bytes [u8]
video.width and video.height are 1920 X 1080
Other information about the video frame :
CCtype: UYVY
frame format type:  Progressive
frame_rate_d: 1
frame_rate_n: 30

 let video_frames = Mat::from_slice(&data).unwrap();
 let video_frames = video_frames.reshape_nd(4, &[video.height, video.width]).unwrap();
 highgui::imshow("window", &video_frames)?;

Also, if I have to resize the video output to say 512 X 256 on the window do I have to use resize function?

That sounds like your problem is with the OpenCV usage, not specifically with its Rust bindings. Just from the looks of it seems like you're interpreting the color order wrong, e.g. your data is RGB, but OpenCV internally expects to work with BGR. You might try imdecode if your frame is an actual encoded image in a format that OpenCV understands.
It's also strange that left image contains more information than the right one, from your description that doesn't seem like it should be a case.
For resizing, yes, imgproc::resize is the function you're looking for.
Generally I might be able to help more if you have a corresponding C++/Python code that works for you. I will then help you translate that to Rust.

Many thanks @twistedfall. It indeed was the colour format. The video frame format is uyvy and I had to convert it to BGR as you suggested.
Also, the right image went out of the screen so I couldn't capture all of it with the snip tool.
Below is the code that worked for me :
// data is the video frame data as bytes [u8]
let data_ptr: *mut c_void = data.as_ptr() as *mut c_void;
let video_frame = unsafe {
     Mat::new_rows_cols_with_data(video.height, video.width, CV_8UC2, data_ptr, Mat_AUTO_STEP)?
};
let mut frame = Mat::default();
imgproc::cvt_color(&video_frame, &mut frame, COLOR_YUV2BGR_UYVY, 0)?;


test: snmp, enricher custom task manager

Codecov Report

Merging #303 (1a7e326) into develop (689f817) will increase coverage by 1.49%.
The diff coverage is 25.00%.

@@             Coverage Diff             @@
##           develop     #303      +/-   ##
===========================================
+ Coverage    47.23%   48.73%   +1.49%
===========================================
  Files           18       18
  Lines         1177     1147      -30
===========================================
+ Hits           556      559       +3
+ Misses         621      588      -33

Impacted Files
Coverage Δ

splunk_connect_for_snmp/snmp/manager.py
20.00% <25.00%> (+2.14%)
:arrow_up:

splunk_connect_for_snmp/customtaskmanager.py
19.75% <0.00%> (+1.23%)
:arrow_up:

splunk_connect_for_snmp/common/hummanbool.py
100.00% <0.00%> (+18.18%)
:arrow_up:

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 689f817...1a7e326. Read the comment docs.

:tada: This PR is included in version 1.1.1-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot establish any tunnels?

Apologies for the vague title. I've not been able to identify any characteristic differences in my setup.
Anyway, here's the form.
Describe the goal
I'm trying to route a TCP audio stream through a wss tunnel. The stream originates from the application-server and is transmitted to the application-client. Application-client shall connect to 1234/tcp and application-server shall listen on 1235/tcp. Thus the wstunnel-client should bind hosta:1234 and the wstunnel-server should bind hostb:1235.
In a bid to try to make my test setup work, I've simplified everything as much as possible. Both hosts are local, but hosta is <IP_ADDRESS> and hostb is <IP_ADDRESS>.
Describe what does not work
All attempts to establish the tunnel stall or time out. If traffic is sent over the tunnel before the timeout, the wstunnel-client's request is reported as malformed (400) and thrown out by the wstunnel-server
Describe your wstunnel setup

client (trying wss):

# wstunnel client -L 'tcp://1235:localhost:1234?timeout_sec=0' wss://<IP_ADDRESS>:443 -c 1 --log-lvl=debug
2024-08-22T21:31:00.631176Z  INFO wstunnel::tcp: Opening TCP connection to <IP_ADDRESS>:443
2024-08-22T21:31:00.631219Z DEBUG wstunnel::tcp: Connecting to <IP_ADDRESS>:443
2024-08-22T21:31:00.631376Z DEBUG wstunnel::tcp: Connected to tcp endpoint <IP_ADDRESS>:443, aborted all other connection attempts
2024-08-22T21:31:00.631382Z  INFO wstunnel::tls: Doing TLS handshake using SNI IpAddress(V4(Ipv4Addr([127, 0, 0, 2]))) with the server <IP_ADDRESS>:443
2024-08-22T21:31:00.631398Z DEBUG rustls::client::hs: No cached session for IpAddress(V4(Ipv4Addr([127, 0, 0, 2])))
2024-08-22T21:31:00.631473Z DEBUG rustls::client::hs: Not resuming any session
2024-08-22T21:31:00.632494Z DEBUG rustls::client::hs: Using ciphersuite TLS13_AES_256_GCM_SHA384
2024-08-22T21:31:00.632524Z DEBUG rustls::client::tls13: Not resuming
2024-08-22T21:31:00.632595Z DEBUG rustls::client::tls13: TLS1.3 encrypted extensions: [Protocols([ProtocolName(687474702f312e31)])]
2024-08-22T21:31:00.632601Z DEBUG rustls::client::hs: ALPN protocol is Some(b"http/1.1")
2024-08-22T21:31:00.632650Z  INFO wstunnel::tcp: Starting TCP server listening cnx on <IP_ADDRESS>:1235

### Between these lines, in another terminal I run `nc -vz <IP_ADDRESS> 1235` ###
### If I do not do this, instead the lines prefixed with # will be printed every 30 seconds or so forever. ###

# 2024-08-22T21:31:00.631451Z  INFO wstunnel::tunnel::server::server: Accepting connection
# 2024-08-22T21:31:00.631552Z  INFO tunnel{peer="<IP_ADDRESS>:56200"}: wstunnel::tunnel::server::server: Doing TLS handshake
# 2024-08-22T21:31:00.631634Z DEBUG tunnel{peer="<IP_ADDRESS>:56200"}: rustls::server::hs: decided upon suite TLS13_AES_256_GCM_SHA384
# 2024-08-22T21:31:00.631788Z DEBUG tunnel{peer="<IP_ADDRESS>:56200"}: rustls::server::hs: Chosen ALPN protocol [104, 116, 116, 112, 47, 49, 46, 49]
# 2024-08-22T21:31:10.634165Z ERROR tunnel{peer="<IP_ADDRESS>:56200"}: wstunnel::tunnel::server::server: Error while upgrading cnx: hyper::Error(HeaderTimeout)

### Otherwise, these lines follow: ###

2024-08-22T21:31:27.469858Z DEBUG tunnel{id="01917c00-696d-7732-9c73-a339bdef3072" remote="localhost:1234"}: wstunnel::tunnel::transport::websocket: with HTTP upgrade request Request { method: GET, uri: /v1/events, version: HTTP/1.1, headers: {"host": "<IP_ADDRESS>", "upgrade": "websocket", "connection": "upgrade", "sec-websocket-key": "VBJ2CTicJ/WaM9fk6hFzsQ==", "sec-websocket-version": "13", "sec-websocket-protocol": "v1, authorization.bearer.eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpZCI6IjAxOTE3YzAwLTY5NmQtNzczMi05YzczLWEzMzliZGVmMzA3MiIsInAiOnsiVGNwIjp7InByb3h5X3Byb3RvY29sIjpmYWxzZX19LCJyIjoibG9jYWxob3N0IiwicnAiOjEyMzR9.5ByF3M3ycSMde-TZW7BfLImJNlZjwWqHJOfYWteTihI"}, body: Empty }
2024-08-22T21:31:27.469935Z  INFO wstunnel::tcp: Opening TCP connection to <IP_ADDRESS>:443
2024-08-22T21:31:27.469961Z DEBUG wstunnel::tcp: Connecting to <IP_ADDRESS>:443
2024-08-22T21:31:27.470061Z DEBUG wstunnel::tcp: Connected to tcp endpoint <IP_ADDRESS>:443, aborted all other connection attempts
2024-08-22T21:31:27.470082Z  INFO wstunnel::tls: Doing TLS handshake using SNI IpAddress(V4(Ipv4Addr([127, 0, 0, 2]))) with the server <IP_ADDRESS>:443
2024-08-22T21:31:27.470082Z ERROR tunnel{id="01917c00-696d-7732-9c73-a339bdef3072" remote="localhost:1234"}: wstunnel::tunnel::client: failed to do websocket handshake with the server wss://<IP_ADDRESS>:443

Caused by:
    0: operation was canceled
    1: connection error
    2: peer closed connection without sending TLS close_notify: https://docs.rs/rustls/latest/rustls/manual/_03_howto/index.html#unexpected-eof
2024-08-22T21:31:27.470190Z DEBUG rustls::client::hs: Resuming session
2024-08-22T21:31:27.470715Z DEBUG rustls::client::hs: Using ciphersuite TLS13_AES_256_GCM_SHA384
2024-08-22T21:31:27.470732Z DEBUG rustls::client::tls13: Resuming using PSK
2024-08-22T21:31:27.470844Z DEBUG rustls::client::tls13: TLS1.3 encrypted extensions: [Protocols([ProtocolName(687474702f312e31)])]
2024-08-22T21:31:27.470854Z DEBUG rustls::client::hs: ALPN protocol is Some(b"http/1.1")

server

# ./wstunnel server wss://<IP_ADDRESS>:443 --log-lvl=debug
2024-08-22T21:25:27.380712Z  INFO wstunnel::embedded_certificate: Loading embedded tls certificate
2024-08-22T21:25:27.380741Z  INFO wstunnel::embedded_certificate: Loading embedded tls private key
2024-08-22T21:25:27.381035Z  INFO wstunnel: Starting wstunnel server v10.0.1 with config WsServerConfig { socket_so_mark: None, bind: <IP_ADDRESS>:443, websocket_ping_frequency: Some(30s), timeout_connect: 10s, websocket_mask_frame: false, restriction_config: None, tls: true, mTLS: false }
2024-08-22T21:25:27.381044Z DEBUG wstunnel: Restriction rules: RestrictionsRules {
    restrictions: [
        RestrictionConfig {
            name: "Allow All",
            match: [
                Any,
            ],
            allow: [
                Tunnel(
                    AllowTunnelConfig {
                        protocol: [],
                        port: [],
                        host: Regex(
                            "^.*$",
                        ),
                        cidr: [
                            <IP_ADDRESS>/0,
                            ::/0,
                        ],
                    },
                ),
                ReverseTunnel(
                    AllowReverseTunnelConfig {
                        protocol: [],
                        port: [],
                        port_mapping: {},
                        cidr: [
                            <IP_ADDRESS>/0,
                            ::/0,
                        ],
                    },
                ),
            ],
        },
    ],
}
2024-08-22T21:25:27.381088Z  INFO wstunnel::tunnel::server::server: Starting wstunnel server listening on <IP_ADDRESS>:443
2024-08-22T21:25:30.427081Z  INFO wstunnel::tunnel::server::server: Accepting connection
2024-08-22T21:25:30.427179Z  INFO tunnel{peer="<IP_ADDRESS>:39556"}: wstunnel::tunnel::server::server: Doing TLS handshake
2024-08-22T21:25:30.427265Z DEBUG tunnel{peer="<IP_ADDRESS>:39556"}: rustls::server::hs: decided upon suite TLS13_AES_256_GCM_SHA384
2024-08-22T21:25:30.427419Z DEBUG tunnel{peer="<IP_ADDRESS>:39556"}: rustls::server::hs: Chosen ALPN protocol [104, 116, 116, 112, 47, 49, 46, 49]
2024-08-22T21:25:40.429466Z ERROR tunnel{peer="<IP_ADDRESS>:39556"}: wstunnel::tunnel::server::server: Error while upgrading cnx: hyper::Error(HeaderTimeout)

### If I perform the `nc` test I mentioned in the client log, then the following lines appear. ###
### The lines above always appear after some time. ###

2024-08-22T21:38:02.367069Z  INFO tunnel{peer="<IP_ADDRESS>:40094" id="01917c06-6ffe-7600-ae73-1a5776c6c0aa" remote="localhost:1234"}: wstunnel::tunnel::server::server: Tunnel accepted due to matched restriction: Allow All
2024-08-22T21:38:02.367093Z  INFO tunnel{peer="<IP_ADDRESS>:40094" id="01917c06-6ffe-7600-ae73-1a5776c6c0aa" remote="localhost:1234"}: wstunnel::protocols::tcp::server: Opening TCP connection to localhost:1234
2024-08-22T21:38:02.367168Z DEBUG wstunnel::protocols::tcp::server: Connecting to [::1]:1234
2024-08-22T21:38:02.367281Z  INFO wstunnel::tunnel::server::server: Accepting connection
2024-08-22T21:38:02.367326Z  INFO tunnel{peer="<IP_ADDRESS>:56058"}: wstunnel::tunnel::server::server: Doing TLS handshake
2024-08-22T21:38:02.367320Z DEBUG tunnel{peer="<IP_ADDRESS>:40094" id="01917c06-6ffe-7600-ae73-1a5776c6c0aa" remote="localhost:1234"}: wstunnel::protocols::tcp::server: Cannot connect to tcp endpoint [::1]:1234 reason Connection refused (os error 111)
2024-08-22T21:38:02.367376Z DEBUG tunnel{peer="<IP_ADDRESS>:56058"}: rustls::server::hs: decided upon suite TLS13_AES_256_GCM_SHA384
2024-08-22T21:38:02.367620Z DEBUG tunnel{peer="<IP_ADDRESS>:56058"}: rustls::server::hs: Chosen ALPN protocol [104, 116, 116, 112, 47, 49, 46, 49]
2024-08-22T21:38:02.618635Z DEBUG wstunnel::protocols::tcp::server: Connecting to <IP_ADDRESS>:1234
2024-08-22T21:38:02.618749Z DEBUG tunnel{peer="<IP_ADDRESS>:40094" id="01917c06-6ffe-7600-ae73-1a5776c6c0aa" remote="localhost:1234"}: wstunnel::protocols::tcp::server: Cannot connect to tcp endpoint <IP_ADDRESS>:1234 reason Connection refused (os error 111)
2024-08-22T21:38:02.618796Z  WARN tunnel{peer="<IP_ADDRESS>:40094" id="01917c06-6ffe-7600-ae73-1a5776c6c0aa" remote="localhost:1234"}: wstunnel::tunnel::server::server: Rejecting connection with bad upgrade request: Cannot connect to tcp endpoint localhost:1234 reason Some(Os { code: 111, kind: ConnectionRefused, message: "Connection refused" }) /v1/events
2024-08-22T21:38:12.368983Z ERROR tunnel{peer="<IP_ADDRESS>:56058"}: wstunnel::tunnel::server::server: Error while upgrading cnx: hyper::Error(HeaderTimeout)

What I've tried

I've inspected the wstunnel-client's request and it is OK
I've tried several releases and built the program myself, but the same issue occurs.
I do not use a firewall. No nonstandard iptables rules are active.
I've tried different transports and different protocols to tunnel.

Desktop (please complete the following information):

OS: Linux 6.6 x86_64 Void Linux
also tested and failed on Debian 12 and 6.1 Void Linux across 4 machines
Version: n/a

So far, I have not managed to establish a tunnel.
What am I missing? I hope I've given enough info here, but I'll provide any information possible.
Thanks for everything :)

Looks like localhost:1234 isn't reachable on your server.

SOLVED - 443 is a low port, so I had to either run the server as root or set the low port bind cap on wstunnel with setcap(8).
Applying for my dumbass certificate tomorrow :laughing:


Call of ENGINE_finish() with NULL is not permitted.

Hi,
While digging in the code, I notice that this method can't be be called with NULL, because it triggers an error.
I don't see the value added of this error,  as a NULL engine pointer means that no engine is active.
Allowing this, could remove many lines of code around the calls to ENGINE_finish.
Want a PR ?

Many lines of code?  You mean something like if (engine != NULL)?

Yes, or any other form of testing the arguments before calling it.
There is  around 20 test / branching that could be removed.

Yes please.

I am in favor of this, since it simplifies calling code.
See #743.  @levitte? :)

done.


Use Arrow Keys to Create New Topics within Diagram

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
The problem I'm having is I want to be able to quickly build diagrams using just the keyboard. I'd like to be able to create new topics in a diagram by using the arrow keys on the keyboard. For example, CTRL + up arrow creates a new topic above the existing one - connected by an arrow.
Describe the solution you'd like
In fact, it might make more sense that the default behavior for arrow keys is to create new topic boxes - rather than nudging topic boxes. Perhaps CTRL + arrow key (for example) should be used for nudging instead. The goal here is to be able to create a flowchart with minimal mouse usage. Start with a topic container, type in the label, hit enter, hit an arrow key in the desired direction, type in the label and repeat as needed.
Describe alternatives you've considered
I looked at keyboard shortcuts and cannot find any way to create new topic boxes via the keyboard. (i.e., without using the mouse to click on the blue arrow buttons that surround a topic box)
Additional context
I'm coming from the MindManager mind mapping software where it was possible to create new topics with a single key press. Quick and useful.

Hi, I've corrected the formatting of my post. Do I need to resubmit as a freshly posted issue?

Hi @ironlion37,
Try shift-alt-arrow.
I think it might be what you are looking for, for general diagrams.
However, it seems you might be discussing mindmaps more specifically. For that, there needs to be more logic, for it to work well, as shift-alt-arrow only works well once in each direction. It does not do any re-layouting to make place for new shapes in the diagram or mindmap, if that is what you are after (like for instance freemind/freeplane and other mindmap tools).
BR!

Great that you got what you needed!
BR = Best Regards :-)

Btw, you as the poster of an issue/topic can close this topic when you comment.