Disable auto vendor prefixing for a JSS style?

I'm finding that I can't get vertical text to work in Edge, because JSS keeps prefixing writing-mode as -ms-writing-mode which does not work correctly in Edge presumably because the values for IE are different than the ones supported in Edge.
Including
writingMode: 'vertical-lr',
in the JSS generates CSS as
-ms-writing-mode: vertical-lr;
but that does not work in Edge. If I manually switch the CSS to writing-mode in the browser dev tools, it behaves properly.
Using
'writing-mode': 'vertical-lr',
in JSS to try and force it to not prefix does not work, it still applies the MS vendor prefix.
Is there something I can prefix on a JSS property to prevent it from being prefixed by JSS?
Filing this as a bug instead of a question because I don't see any functionality to explicitly disable or ignore auto-prefixing for a a single JSS element.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
A JSS element should not be auto prefixed by the prefixer if we specify that it should not auto-prefix it, somehow.
Current Behavior 😯
Auto-prefixing is applied to the JSS element, even when that prefix does not work correctly in the destination browser.
Steps to Reproduce 🕹
I just took the default template and turned the tip sideways. It looks bad, but it gets the point across.
Link: https://codesandbox.io/embed/create-react-app-b6iij

Try to use writingMode on a div, set to vertical-lr
The div should appear sideways. In Edge, it does not, because the CSS got prefixed with -ms and the -ms prefix does not work there.

Context 🔦

We have a panel that we want to collapse to the side, but right now that works in every browser (including IE11, where the -ms prefix actually works) except Edge.
I realize that EdgeHTML will be going away soon, but maybe there are other use cases where someone would want to selectively disable auto-prefixing?
Your Environment 🌎

Tech
Version

Material-UI
v4.1.3

For this particular issue with writeMode and edge please open a new issue in cssinjs/jss.
If you want to disable vendor prefixing completely see our docs on JSS plugins

Thanks! I disabled vendor prefixing and everything works as it did before we upgraded from MUI 3 to 4.

I know this issue is closed but I have been having a similar issue and I am not seeing in the documentation how to disable the plugins.

@Talkingbaby
If you check out the link he included to the documentation, they provide an example of how to add the jss-rtl plugin.
The key line there is
plugins: [...jssPreset().plugins, rtl()],
Which takes the default plugins and adds rtl. This of course does not tell you how to remove them, just how to add them. What you basically want is the result of jssPreset().plugins, without the auto prefix plugin. What I did was looked at the source code to jssPreset().plugins and just copy/pasted the list there and removed the vendor prefixer. This might not have been the best approach, since if changes are made to that list in the future I won't get them, but it works for now.
Does that help?

If prefixer doesn't generate a correct prefix (I haven't verified), please create an issue here  https://github.com/cssinjs/css-vendor

@Talkingbaby
If you check out the link he included to the documentation, they provide an example of how to add the jss-rtl plugin.
The key line there is
plugins: [...jssPreset().plugins, rtl()],
Which takes the default plugins and adds rtl. This of course does not tell you how to remove them, just how to add them. What you basically want is the result of jssPreset().plugins, without the auto prefix plugin. What I did was looked at the source code to jssPreset().plugins and just copy/pasted the list there and removed the vendor prefixer. This might not have been the best approach, since if changes are made to that list in the future I won't get them, but it works for now.
Does that help?

Thank you it did seem to help solve our problem.

If prefixer doesn't generate a correct prefix (I haven't verified), please create an issue here https://github.com/cssinjs/css-vendor

Thank you. I'll try to create an issue as soon as I can. Just a summary our issue is a webkit prefixer was being created but it was actually breaking the style in chrome.


Examples compile error

Compiling several examples on Arduino IDE 1.6.3 I get an error like this (serial_nav.ino):

Arduino: 1.6.3 (Windows 8.1), Board:"Arduino Nano, ATmega328"
In file included from CustomNumericMenuItem.h:13:0,
from CustomNumericMenuItem.cpp:1:
D:\Documents\Arduino\sketchbook\libraries\MenuSystem/MenuSystem.h:180:11: error: expected nested-name-specifier before 'SelectFnPtr'
using SelectFnPtr = void ()(MenuItem menu_item);
^
D:\Documents\Arduino\sketchbook\libraries\MenuSystem/MenuSystem.h:180:11: error: using-declaration for non-member at class scope
D:\Documents\Arduino\sketchbook\libraries\MenuSystem/MenuSystem.h:180:23: error: expected ';' before '=' token
using SelectFnPtr = void ()(MenuItem menu_item);
^
D:\Documents\Arduino\sketchbook\libraries\MenuSystem/MenuSystem.h:180:23: error: expected unqualified-id before '=' token
D:\Documents\Arduino\sketchbook\libraries\MenuSystem/MenuSystem.h:188:32: error: 'SelectFnPtr' has not been declared
MenuItem(const char* name, SelectFnPtr select_fn);
^
<<<<<
and more ....
Am I missing something?

The library uses features from c++11 which require compiling with -std=c++11 or -std=gnuc++11. This is the default in version 1.6.6 of the IDE, so if you upgrade it should work fine. Let me know if it doesn't.

Jon,
I installed IDE version 1.6.10 and it now compiles.
Didn't realize that such a major change is just denoted by a minor update
version number ...
Thanks for your help!
I now have to sort out why my Arduino Nano is not recognized by my laptop,
i.e. the sketch won't upload.
But that's another (driver) issue.
Again: thanks!
Regards,
Eric
2016-07-26 9:15 GMT+02:00 Jon Black<EMAIL_ADDRESS>
The library uses features from c++11 which require compiling with
-std=c++11 or -std=gnuc++11. This is the default in version 1.6.6 of the
IDE, so if you upgrade it should work fine. Let me know if it doesn't.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jonblack/arduino-menusystem/issues/40#issuecomment-235182098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATr8SVtZJjouLR4WpWSOR9gKeKw9gAZRks5qZbQAgaJpZM4JUJ3I
.


update Model.cs list with GPT-4o + update Unity LTS to last (2021.3.3…

Update Model.cs list with GPT-4o
Update Unity LTS to last (2021.3.38f1)
Tested API id

You can already use this model by specifying a it as string.


Update runbooks with upstream changes (automation execution result)

This PR is the result of running the automation but opening a PR manually with all the commits together. We can also use this PR to validate if some changes are needed in the automation.
After https://github.com/kubevirt/monitoring/pull/240 to sync some changes upstream, enabling the automation would create too many PRs, so this approach before enabling allows us to have a single PR and reduce the noise created by the automation.
The changes here are mostly formatting differences that were made manually, so it's normal to differ from what the automation now suggests.
Other than that, some changes were never done downstream and there are also some language differences. For this, if we want what is downstream, we should create PRs upstream with those updates. So that when we rerun the tool, those changes are reflected here.
/cc @sradco @ousleyp

@machadovilaca Hi, I think a lot of the changes might need to be reverted, so I'm glad we are testing the automation. :)
The main thing I have noticed so far is that the removal of the leading spaces causes the formatting to regress.
For example, the current copy of CDIDefaultStorageClassDegraded.md in openshift/runbooks renders like this (correct):  (file view)
with the spaces removed, it turns into:

[(file view)](https://github.com/openshift/runbooks/blob/1a2bf4646a8b91e2484db0c594ef7d63c14a0346/alerts/openshift-virtualization-operator/CDIDefaultStorageClassDegraded.md)

@ousleyp I think many of these need updating in upstream, will take a look at update them there

/lgtm
kubevirt_cdi_ovirt_progress_total metrics name and description would be updated in a follow-up PR.
This PR is urgent to unblock the metrics code refactoring linter PR.

https://github.com/kubevirt/monitoring/pull/251
@ousleyp

/approve

/lgtm

I don't think there is a need for the metric
'mtv_operator_destination'. Why not integrate it to the migration and plan metrics?


Auto corrected by following Lint Ruby Bundler/OrderedGems

Auto corrected by following Lint Ruby Bundler/OrderedGems
Click here to configure it on awesomecode.io

Coverage remained the same at 78.476% when pulling 95371b6a238f7c9403a3c291e54eedd24751a997 on awesomecode-lint-ruby-bundler-orderedgems-48441 into f4466d839cbd23d5753497d7d6b18d67c9c6f757 on master.


[core] Deprecate NATIVE_IMAGE in favor of BINARY

Precedent to https://github.com/jreleaser/jreleaser/issues/899

There are two differences between NATIVE_IMAGE and BINARY:

sdkman publication is not allowed for BINARY.
NATIVE_IMAGE requires validation of java section.

Item 1. may be allowed as the Sdkman team will likely prevent a non-Java based binary (read: not compiled with GraalVM Native Image) to be published. That was easier in the past as there was a key associated with each candidate. Nowadays there's a key per vendor which means one could get a key for a valid project and use it to post an invalid project. cc: @helpermethod.
Item 2. may be resolved by adding an extraProperty to the distribution indicating its origin is related to GraalVM Native Image.

Released in v1.4.0 -> https://github.com/jreleaser/jreleaser/releases/tag/v1.4.0


Added functionality to switch to an existing R terminal

What problem did you solve?
#337  - Though feel free to reject if you don't like the proposal :)
(If you have)Screenshot

@andycraig  & @renkun-ken
Added a commit to fix the typo you mentioned.
Also added a commit to try to select the most "recent" R terminal. (more details in #337 )

What happens if there are multiple open terminals but none is named as ['R', 'R Interactive']? It would be nice to show a message or create a new terminal in this case?

@renkun-ken  -  If there aren't any terminals called R then rTerm is left undefined undefined and gets picked up by the block here and thus attempts to create a new terminal (printing an error message if its unable to do so).

Thanks, just forgot the case has been handled previously.
LGTM.


Support removing keywords with NOT RUN status using --removekeywords

As a user I want to hide/remove NOT_RUN status keywords from log.html to avoid confussion for other teammates
Points Discussed in slack (#devel)

--removekeywords to support removing NOT_RUN status keywords similar to PASSED
IF/ ELSE/ ELSE IF should not be excluded while using --removekeywords

I don't consider this important enough to be added in already late RF 4.0. Can be included in RF 4.1 if there are concrete needs.

Being able to remove NOT RUN keywords using --removekeywords could be handy. Not super important but if someone would be willing to provide a pull request the functionality could certainly be included. I'll update the title accordingly. Possible other way to exclude these keywords and control structures can get their own issues.

For the reasoning why we have keywords with the NOT RUN status see #3842.

let me try to implement this


Add toggle for debugProfileWidgetBuilds

Description
In https://github.com/flutter/flutter/pull/21492, we added a service extension to toggle profiling of widget build times. This was never wired up through the tool, so it was essentially dead code.
Related Issues
Replace this paragraph with a list of issues related to this PR from our issue database. Indicate, which of these issues are resolved or fixed by this PR.
Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behaviour with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[ ] No, this is not a breaking change.

cc @fkorotkov we're getting some odd git errors in the cirrus jobs for this PR

cc @devoncarew

We did not expose this though IDEs, but it would be good to do, in order to see how useful it is.
cc @DanTup for VS Code; I'll open an issue for DevTools

@devoncarew @dnfield I've wired it up, but I don't know what it's supposed to do so I'm not sure how to verify it's working. I don't see any  difference in the DevTools timeline view, nor the Logging view (though I can see it being toggled on/off). Where does it show up?

This change is fine, but FWIW, we should be wary of adding things like this just for completeness. We don't necessarily want every last feature exposed in the UI. It will make it unusable.

We ended up using this to debug an internal issue, so its usefulness justified the cost IMO.


For discussion: disable link ink skipping.

Back in 2017 we enabled link skipping (#281 ). After some discussion (#313 ) we then disabled it (#316).
Links skipping has now been enabled by default in some browsers so I wonder if we should explicitly disable it.
I think the same reasons apply to making it default disabled as existed last year: while it may look prettier, it often makes a single link look like multiple links - particularly when near punctuation. The skip is often far larger than necessary. Until browser vendors release an improved implementation, we should disable.
By way of dropbox paper, we've also had this accessibility comment from the Digital Accessibility Center:

We have recently had a suggestion from the Digital Accessibility Centre that links, where the underline is broken by descending letters, are not good for users of assistive technology.  In particular, a ZoomText user viewing a link at high magnification would have trouble identifying where the link ends.

Could we consider raising this with browser vendors? As suggested by Robin here? https://github.com/alphagov/govuk_template/issues/313#issuecomment-311927325

#Current browser behavior on Mac:
Chrome:

Safari

Firefox

And with text-decoration-skip-ink: none; applied:
Chrome

Safari
Didn't seem to work.

@edwardhorsford safari may be using https://developer.mozilla.org/en-US/docs/Web/CSS/text-decoration-skip

Safari (or at least the current version on my 10.13 laptop) still needs -webkit-text-decoration-skip-ink: none.

Yeah I think disabling ink makes sense.
This is an org name i just saw in an integration test so thought I would share

Following the launch of the GOV.UK Design System, GOV.UK Template will now only get major bug fixes and security patches, so I'm going to close this.
We've captured this issue on GOV.UK Frontend here.


HashStore .jar file contains too many dependencies

When the HashStore shaded .jar file is generated, it contains too many dependencies.
Investigate the maven setup so that we only include the necessary libraries.

This has been completed via Feature-67: HashStore .jar File Update
To clarify, it is okay that two .jar files have been created. However, the one with dependencies should have the name shaded as part of the .jar file name.
Only two lines were required to have maven package the project as expected under the relevant plugin, below for quick reference:
...
    <configuration>
        <outputFile>${basedir}/target/${project.name}-${project.version}-shaded.jar</outputFile>
        <outputDirectory>${basedir}</outputDirectory>
        ...
    <configuration>
...


All P100s, P110, are not working anymore (after a firmware update?)

-->
Version of the integration
HACS Version:
TP-Link Tapo controller
V.2.4.0
HA:
Home Assistant 2023.8.1
Supervisor 2023.07.1
Operating System 10.4
Frontend 20230802.0 - latest
-->
Configuration
Tapo
P100
1.4.18 Build 20230711 Rel. 55651
P110
1.2.3 Build 230425 Rel. 142542
Add your logs here.

Describe the bug
after the update of the firmware in the Tapo app of the P1XX
everything stops working - error mentioned: configuration failed
since couple of days ago everything was working smoothly
Debug log
Logger: homeassistant.config_entries
Source: custom_components/tapo/tapo_device.py:31
Integration: TP-Link Tapo (documentation, issues)
First occurred: 7 agosto 2023 alle ore 19:01:02 (28 occurrences)
Last logged: 08:35:57
Error setting up entry frigorifero for tapo
Error setting up entry lavatrice for tapo
Error setting up entry piano cottura for tapo
Error setting up entry luce viale 1 for tapo
Error setting up entry lampada di sale grande for tapo
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 388, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tapo/init.py", line 39, in async_setup_entry
raise error
File "/config/custom_components/tapo/init.py", line 37, in async_setup_entry
return await device.initialize_device(hass)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tapo/tapo_device.py", line 31, in initialize_device
raise DeviceNotSupported(f"Device {host} not supported!")
custom_components.tapo.errors.DeviceNotSupported: Device <IP_ADDRESS> not supported!

Hi Frank777777,
I had the same problem and deleted the devices and reinstalled them.
After that the problem was solved.
Best regards,
Jan

thanks for the hint .
quick question, as I have 32 devices, with energy data memorized.   do you know if I'm going to lose all data?
thanks for your kind reply.
Frank

I also like to confirm, is not working. I can not add anymore a device,    seems on the add a new device on tapo HACS, that I'm not recognized ( invalid )   and I check the password twice.

I have no experience, sorry for that.

well i can't connect to my tapo devices anymore as well.... what is going on. i tried to remove the addon .... install it again... update .... check firmware updates.... no matter what i do when i try to add the devices again... i get a failed to connect

Same for me, all my tapo devices (L530) stopped working.
I tried to reinstall the integrations for the devices, but now it just gives an "Invalid Authentication".
My Username is my fully lowercase email adress, so this is no Uppercase issue.

I had the same problem with two P110 plugs. No longer recognized by HA and the Tapo application on Android.
Since I had other problems with my network and especially with Zigbee devices, probably due to interference between WIFI and Zigbee using both channel 11, I reconfigured Zigbee2MQTT to use channel 20.  Impossible to change the WIFI channel on Deco P9+M4... As TP-Link support told me: "the algorithm takes care of it"!
After that, I had no more problems with the Zigbee devices and I reinstalled the two P110 plugs via the Android app before reinstalling the HA integration.
Of course, it's impossible to say that changing the Zigbee channel is responsible for getting the P110 plugs working again, but as far as I'm concerned, it works perfectly now. id app.

I have 6 working P110 devices. IP is via DHCP, so I would like to track and assign them via their MAC address.
When I run arp in the terminal, everything is shown (I have several P110 plugs, all are firmware up to date).
But. If I go to the configuration and enable the checkbox for MAC support, the device is no longer maintained by the tapo integration according to h.a. and I can't see any data anymore.
Since I am making an overview of everything in my house, I need to eject and insert the plug every 7 days or so, until everything is tested. Currently I am forced to find the IP address manually, this is not what I expect of automation.
After running arp manually in the terminal several times, and after enabling debug, I now do can check the checkbox for tracking.
After disabling the debug log, I do see the MAC address in there, so I am now reconfiguring all of my switches (without debugging).
I found 3 switches that accept tracking by mac, and 3 do not (the serial numbers indicate I have 2 batches, and all three in the same batch either work or not).
Debug shows:
2023-09-05 11:37:58.849 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 3 oven' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
More complete debug log:
2023-09-05 10:50:41.347 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 3 oven' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
2023-09-05 10:50:45.954 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 2 t.v.' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
2023-09-05 10:50:51.567 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 1 heating' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
2023-09-05 10:51:56.672 ERROR (MainThread) [frontend.js.latest.202308021] http://homeassistant.local:8123/frontend_latest/15664-2R3AuA7BqSA.js:1:78472 TypeError: this.contentElement is null
2023-09-05 10:52:02.743 ERROR (MainThread) [frontend.js.latest.202308021] http://homeassistant.local:8123/frontend_latest/15664-2R3AuA7BqSA.js:1:78472 TypeError: this.contentElement is null
2023-09-05 10:52:10.048 ERROR (MainThread) [frontend.js.latest.202308021] http://homeassistant.local:8123/frontend_latest/15664-2R3AuA7BqSA.js:1:78472 TypeError: this.contentElement is null
2023-09-05 11:37:56.479 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.137 seconds (success: True)
2023-09-05 11:37:56.864 INFO (MainThread) [custom_components.tapo.config_flow] <homeassistant.config_entries.ConfigEntry object at 0x7f8e6c3780>
2023-09-05 11:37:58.849 INFO (MainThread) [custom_components.tapo] Unloaded entry for a207fca4d2838ac53aa13093befefbeb
2023-09-05 11:37:58.849 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of None, last known ip is <IP_ADDRESS>
2023-09-05 11:37:58.849 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 3 oven' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
2023-09-05 11:38:04.227 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of None, last known ip is <IP_ADDRESS>
2023-09-05 11:38:07.829 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of None, last known ip is <IP_ADDRESS>
2023-09-05 11:38:07.830 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Smart-plug 3 oven' for tapo integration not ready yet: 'NoneType' object has no attribute 'replace'; Retrying in background
2023-09-05 11:38:09.252 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.155 seconds (success: True)
2023-09-05 11:38:12.446 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.140 seconds (success: True)
2023-09-05 11:38:12.625 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.140 seconds (success: True)
2023-09-05 11:38:13.089 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of None, last known ip is <IP_ADDRESS>
2023-09-05 11:38:14.518 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.147 seconds (success: True)
2023-09-05 11:38:17.541 ERROR (MainThread) [frontend.js.latest.202308021] http://homeassistant.local:8123/frontend_latest/15664-2R3AuA7BqSA.js:1:78472 TypeError: this.contentElement is null
2023-09-05 11:38:17.592 INFO (MainThread) [custom_components.tapo.config_flow] <homeassistant.config_entries.ConfigEntry object at 0x7f8e6c3780>
2023-09-05 11:38:17.606 INFO (MainThread) [custom_components.tapo.config_flow] <homeassistant.config_entries.ConfigEntry object at 0x7f8e6c3780>
2023-09-05 11:38:23.228 DEBUG (MainThread) [custom_components.tapo.setup_helpers] Creating new API to create a coordinator for 8022FE1EFCCD3F739C3A78963DDCF5E1204AB557
2023-09-05 11:38:23.471 INFO (MainThread) [custom_components.tapo.coordinators] Detected model of <IP_ADDRESS>: p110
2023-09-05 11:38:23.610 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.138 seconds (success: True)
2023-09-05 11:38:23.777 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.153 seconds (success: True)
When I enable the mac tracking, H.A. shows 'Failed setup, will retry' and the mouse hoover text shows 'NoneType' object has no attribute 'replace'.
I waited some time, but it remains in error state.
As soon as I uncheck the MAC address tracking, h.a. is pleased and everything is working again.
I think the code which looks for the MAC address is getting an empty variable. I think this should ve tested in the code, so it does not go in 'error' mode, but still collects data via the IP. The MAC tracking should be secondary and not break functionality. imho.

i have the Track MAC unchecked when i try to add the plug. it doesn't work. this is getting out of hand. i can't be dealing with stupid things like that all the time. i have tapo P110 that opens my office door so that people don't have to get up and open the door 20 times per day. i am replacing the p110 with a shelly smart plug and move on.

v2.8.1 should fix issue related to mac address

thank you. it is working now. i managed to add the devices. i don't know what you fixed but it finally worked.

@petretiandrea I did look for "Migration to version xxx successful", sorry. I see no search entry in the H.A. log.
My current version: Integration version: | 1.32.1, Frontend version: |<PHONE_NUMBER>2053
Today I updated hacs, and I changed the [x] checkbox to [on] again, and now it works. All 6 of my P110 energy switches are accepting this.
Note: Based on the MAC I did as well tell my wifi-router to use the same IP address over and over again, but now it is working I can remove that and let DHCP do its thing as usual.

@petretiandrea Sorry .. .. A day later and 3 of the P110 disappeared. The mac tracking disables the switch.
Why do I have 3 P110 switches which can be mac tracked, and 3 which do not? They are a different batch, but I upgraded the firmware of all 6 when I used them (they are new, a few weeks old).

I deleted and added the P110 switches, they do definitely not work with mac tracking.
My WiFi router does do mac tracking, and I assigned fixed IP's (in the DHCP range) to the switches. Now I am able to do whatever I want with the switches (energy meters).

I had to basically remove, update, and reinstall everything to get it to work again. Here are the steps for anyone who needs the help:

In HA, go to HACS -> Integrations -> Tapo Controller
Click on the 3 dots in the top right and remove the integration
Update HA to the latest version
Go to HACS -> Integrations -> Explore & Download Repositories
Install Tapo Controller integration again
Setup everything (hostname, username, password) as before and it should work

Hi,
I get back on this issue just to report another problem regarding the firmware update of Tapo SP P105.
More than one week ago I noticed that some of this plugs doesn't been recognized by home assistant even if some else was perfectly working.
I have actually plug 8 P105 and 14 P110.
The P105s that works, during the night are out of power (is placed in cascade with a master P110 that switch off during the night).
Checking the p105 "not working" (in home assistant) inside the TAPO APP i'd noticed that they have onboard firwmare version 1.2.1 Build 230804 Rel. 190922.
The p105 that works have onboard firware 1.1.5 Build 230324 Rel. 143956.
That's because I'd enable the auto-update of the firmware during the night (03.00-05.00 AM) and the one's that works was switched off during that time.
Of course the updated one's is in grey in home assistant. I tried:

To delect the instance of the single plug over home assistant and re-create. I got "INVALID AUTHENTICATION" red warning, of course using TAPO-CONTROLLER Hacs integration in home assistant. I triple verify the ip, username and password (I have a DHCP RESERVATION on router for this plugs, bind with mac address). The mail address doesn't have CAPITALIZED letters.
I tried to delete the single plug from TAPO app, re-register and re-join. They come over with the updated firmware :(

Nothing else to do... Of course the invalid authentication seems to depend to the firmware update... the "old" one still works (I deactivated the auto-firmware-update on all the plugs).
Of course directly on tapo app and with alexa, everything works!
I know that this is an open source and custom integration, and I really appreciate the time spent and the effort dedicated. Accordingly with you time, could you please take a look?
I've some automation with this plug that is not working anymore :(
I would say thank in advance for your eventually answare, kind and consideration.
Please let me know if you need any additional information.
Following the HA VERSION:
Home assistant core 2022.11.2
Home assistant oS 10.3

Are you using latest version of this integration? If not, update it, otherwise open a new issue by reporting your comment

Hi, I get back on this issue just to report another problem regarding the firmware update of Tapo SP P105. More than one week ago I noticed that some of this plugs doesn't been recognized by home assistant even if some else was perfectly working. I have actually plug 8 P105 and 14 P110.
The P105s that works, during the night are out of power (is placed in cascade with a master P110 that switch off during the night).
Checking the p105 "not working" (in home assistant) inside the TAPO APP i'd noticed that they have onboard firwmare version 1.2.1 Build 230804 Rel. 190922.
The p105 that works have onboard firware 1.1.5 Build 230324 Rel. 143956. That's because I'd enable the auto-update of the firmware during the night (03.00-05.00 AM) and the one's that works was switched off during that time.
Of course the updated one's is in grey in home assistant. I tried:

To delect the instance of the single plug over home assistant and re-create. I got "INVALID AUTHENTICATION" red warning, of course using TAPO-CONTROLLER Hacs integration in home assistant. I triple verify the ip, username and password (I have a DHCP RESERVATION on router for this plugs, bind with mac address). The mail address doesn't have CAPITALIZED letters.
I tried to delete the single plug from TAPO app, re-register and re-join. They come over with the updated firmware :(

Nothing else to do... Of course the invalid authentication seems to depend to the firmware update... the "old" one still works (I deactivated the auto-firmware-update on all the plugs).
Of course directly on tapo app and with alexa, everything works!
I know that this is an open source and custom integration, and I really appreciate the time spent and the effort dedicated. Accordingly with you time, could you please take a look?
I've some automation with this plug that is not working anymore :(
I would say thank in advance for your eventually answer, kind and consideration.
Please let me know if you need any additional information.
Following the HA VERSION: Home assistant core 2022.11.2 Home assistant oS 10.3 Tapo controller ver.: Stable v2.9.1

Are you using latest version of this integration? If not, update it, otherwise open a new issue by reporting your comment

Hi and thank you for your kindly answer.
I strongly think I'm Using the Latest updated version. I do attach some screenshoot just to confirm:

So I can proceed to open a new issue?
Waiting your kind confirmation.
Thanks.


Readme cleanups; added SlackEngine

Formatting and wording cleanups
Added SlackEngine

See individual commits for details.

Awesome! :-) Thanks.


Fixed bug with wrong black bars margins

Previously, at some window sizes, black bars could obscure
1 pixel wide lines to the right and/or bottom.

Sorry. 🙇🏼♀
I did something weird again...


memory out of bounds write in scard_process_list_readers

version
https://github.com/neutrinolabs/xrdp/blob/1c4e14415d923666ac60cb77c7e753ca24e0268d/sesman/chansrv/smartcard_pcsc.c#L600

vuln
scard_process_list_readers first read 4 bytes to bytes_groups, then use g_malloc(bytes_groups + 1, 1) to alloc memory.
if  bytes_groups=0xffffffff , it could trigger Integer overflow (0xffffffff+1=0), it could alloc 0 size memory to groups.
Then it could use in_uint8a  to copy data to  groups. resulting in out-of-bounds write
int
scard_process_list_readers(struct trans *con, struct stream *in_s)
{
    ..............................
    ..............................
    in_uint32_le(in_s, hContext);
    in_uint32_le(in_s, bytes_groups);

    groups = (char *) g_malloc(bytes_groups + 1, 1); // 0xffffffff + 1 = 0
    in_uint8a(in_s, groups, bytes_groups);  // copy 0xffffffff data to groups

Thank you very much but why not create a pull request?


Add missing rights check

After using this library in my project I found out, that there is currently no way to detect if the user has not granted permissions to the camera. This results in only a black screen for the CodeScannerView.
This PR adds an error case for missing rights, so the developer can update the UI to inform the user about the missing right. This might not be the cleanest implementation as it will always return the missingRights error for every error that occurred during initialization of init of AVCaptureDeviceInput. It works well for my case so far and is a purely additive change.
Do you mind merging this and releasing a new version @twostraws ? It has been a while since 1.0.7 and quite some new features were added (like viewfinder support)...

Sorry for getting back to this so late (new 👶😉). I updated the PR to include the discussed changes!

Thank you!


implement a ".join(*args)" method

It would be nice to have a DOMElement.join() method that mimics the str.join() method in python.
Ideally, this method would accept a list of things and returns the passed list with the chosen TemPy tag instances between the list elements just like the str.join method:
Br.join(["pizza", "pasta", "lasagna"])
>>> ["pizza", Br(), "pasta", Br(), "lasagna"]

Hr.join(["pizza", "pasta", "lasagna"])
>>> ["pizza", Hr(), "pasta", Hr(), "lasagna"]

Br.join(["pizza"])
>>> ["pizza"]

I'd make it a classmethod, so both Br.join(["pizza", "pasta", "lasagna"]) and Br().join(["pizza", "pasta", "lasagna"]) sytntaxes are allowed.
In the latter syntax copies of Br() have to be created.

Hey I want to do this

Hi @Navneet78 thanks for taking this. I assigned you this issue.
If you have any question just ask on this issue.

Hello @Hrabal What I am working on this issue and I am creating a class method in DOMelement class. which takes a list of elements as input and return a list. And that list having consecutive previous elements and that class instance. Am I right like in above example Hr is instance of DOMelement class and it's coming between any two elements of given list. Am I right ?
And also  do I need to write unit-test for this in tests?

Hi @Navneet78

Am I right like in above example Hr is instance of DOMelement class and it's coming between any two elements of given list.

Yes, Hr is an instance of Hr class, which is a subclass of DOMelement. The method should return an iterator containing the input iterator with instances of the called class in between each two elements. So, for exampe, calling 'join' on Hr will make Hr's and calling 'join' on Div will make Div's instances.

And also do I need to write unit-test for this in tests?
Yes please

hey, I have little confusion in a class method we are not passing instance of class as parameter. in classmethod we are just passing cls parameter instead of self (which points to instance of object) parameter and cls parameter which points to class not instance of class. So in order to pass instance of object to classmethod we need to specify function like this

Hr.join(["pizza", "pasta", "lasagna"], Hr)
'>>>'["pizza", <main.Hr 55664848.>, "pasta", <main.Hr 55664848.>, "lasagna"]

without passing instance of class we will get it like this

Hr.join(["pizza", "pasta", "lasagna"], Hr)
'>>>'["pizza", <main.Hr .>, "pasta", <main.Hr .>, "lasagna"]

I hope you got the difference. Please help what should I do?

Hey,
when defining a classmethod the signature changes from "self" (which points to the instance of the class) to "cls" (usually), which is the class itself. So, to have instances we can just instantiate the cls parameter directly:
class A:
    @classmethod
    def c_method(cls, val):
        return cls(), val

print(A.c_method("foo"))
>>> (<__main__.A instance at 0x109e01998>, 'foo')
print(A().c_method("foo"))
>>> (<__main__.A instance at 0x109e01998>, 'foo')

Hello, I think by using 'cls()' first you are creating a new object of class and returning it. It's not same instance as that we used to call that method. For instantiating an object we need parameters of init method and unlike 'self', cls don't have access to instance argument.

class A:
    def __init__(self, arg):
         self.ins_arg=arg
    @classmethod
    def c_method(cls, val):
        return cls(), val

print(A('arg').c_method("foo"))
>>> TypeError: __init__() missing 1 required positional argument: 'arg'

so please help !

Hi,

I think by using 'cls()' first you are creating a new object of class and returning it.

Yes, this is ok to happen, we don't want the same instance in between the elements of the input, but several new instances of the same class.

For instantiating an object we need parameters of init method

it's ok to just init the class with no parameters ( cls() ).


Accessing the ConfigMap within a Suite

I've written a trait that extends WebBrowser to support Appium. However, I need to inject two dependencies: the web driver server and the application path, which are required to instantiate the web driver. The best solution seems to be defining the vals in a class extending the trait. However, I can't get at those during instantiation. What do you think would be the best way to inject these dependencies into a subtrait of WebBrowser?

Hey @bvenners, did this get implemented at all? Thanks!


Is it possible to configure Archive using Amazon S3? (OHIF Viewer)

Hi all,
Do you know if it's possible to configure the OHIF Viewer or the Standalone Viewer to use Amazon S3 as the Archive instead of Orthanc? does somebody already did it?
I was working with the Standalone Viewer and for the folder containing the studies I was able to create the script which parses it and create the corresponding JSON file, however... I'm not sure how to make Amazon S3 to provide me URLs for each file I've uploaded (so the JSON can have them inside for each file) ... probably should have to make them as public and that's not possible in my case... Should I have to use the OHIF Viewer and not the Standalone Viewer... and just configure my Amazon S3 server as the OHIF archive server? Anyway I'm not sure how my S3 will provide authorization to OHIF app to see the files...
Am I doing something wrong?
Any help would be really appreciated. best regards!

At the moment, I am trying to delegate the S3 handling to the PACS server (Orthanc or Dcm4chee), otherwise one would need to develop an s3 library with server-side request signing. I think it's simpler to implement the s3 interface PACS side and leave OHIF with a DICOMWeb interface.
I know both Orthanc and dcm4chee have s3 plugins. Haven't got either of those to work yet.

Yes, it is possible, but you must be careful about security of your DICOM files.
If you put your DICOM files in a private S3 bucket, you will need to get a presigned URL for each single file using the AWS API.  This URL allows you (and anybody who get access to the URL) to download the files via http/https and you can define an expiration time for the URL (so the link is no longer valid after X seconds).
One problem with this approach is that somebody could look at the developer console in the browser and get the links to download your files. But I think that other approaches (such as using tokens, passwords etc) would have the same security issue (somebody could just get the token and make requests to get the DICOM files from the dicom server). Is you use https in the Standalone Viewer, the links will be encrypted in transit.

One problem with this approach is that somebody could look at the developer console in the browser and get the links to download your files.

What would be the problem with this? The user could only get the links to images they were already allowed to view in OHIF.

One problem with this approach is that somebody could look at the developer console in the browser and get the links to download your files.

What would be the problem with this? The user could only get the links to images they were already allowed to view in OHIF.

Furthermore, this approach is safer than using OHIF default Viewer. Currently, if I give a link of a specific study to somebody, I am also giving them access to the study-list. If they can take a look at the developer console, they can use the provided wado proxy to do whatever queries they please, effectively using the PACS account that was configured at config.json
I wouldn't trust OHIF with security and access control. It's just a viewer.

Hi guys! thanks for your prompt responses. So... summarizing:

@diaztula Do you suggest to use the Standalone Viewer instead of the OHIF Viewer... and handle the JSON creation in a backend server (which could be AWS Lambda function to resolve security issues and forgot about the server). Am I right?
@TZubiri I'm concerned about using the OHIF Viewer and relay on DCM4CHEE as my backend server.... I'm not sure if I understand very well what the DCM4CHEE server is... but it seems to have its own DB and it's made in J2EE... Is the DCM4CHEE a PAC Server? do you know how it works? I'm a little bit confused about it but it seems that is Open Source...
If I create the JSON using the pre-signed URL from AWS API... then each time that somebody want to see his study will be consuming AWS requests and will be charged.. Is that efficient? Isn't it better to download the study in a backend server and create the JSON in such way that the URLs are not pointing to S3 directly and the front end just access the files from the backend (and avoid S3 charges, for example if the user plays a video and see the same study several times...)...
Another issue I have to create the JSON using pre-signed URLs from AWS S3 API is that it has of course a timeout so...the JSON file will change every time, so I have to generate it every time the user wants to see his study or the timeout occur, and read the files from S3 to parse and generate the JSON.. which will also consume S3 requests and would be expensive...

Any advise you could provide would be really appreciated! best regards to both!

Hi guys! thanks for your prompt responses. So... summarizing:

@diaztula Do you suggest to use the Standalone Viewer instead of the OHIF Viewer... and handle the JSON creation in a backend server (which could be AWS Lambda function to resolve security issues and forgot about the server). Am I right?
Yes, that's it.

If I create the JSON using the pre-signed URL from AWS API... then each time that somebody want to see his study will be consuming AWS requests and will be charged.. Is that efficient? Isn't it better to download the study in a backend server and create the JSON in such way that the URLs are not pointing to S3 directly and the front end just access the files from the backend (and avoid S3 charges, for example if the user plays a video and see the same study several times...)...

Yes, if you want to keep your files safer you would generate a presigned URL every time the study is opened.
For large studies (e.g. CTs with thousands of files) and/or many clients using the viewer at the same time, performance could be an issue, since for each file you'll need to call presign from AWS and there is not batch presign AFAIK.
If you download your files from S3 to a server outside AWS (or inside AWS but in a different region) then you will pay download transfer anyway. And you will have to serve those files from your server so the OHIF can read them.

Another issue I have to create the JSON using pre-signed URLs from AWS S3 API is that it has of course a timeout so...the JSON file will change every time, so I have to generate it every time the user wants to see his study or the timeout occur, and read the files from S3 to parse and generate the JSON.. which will also consume S3 requests and would be expensive...

You could create presigned links that last at most one year (https://github.com/aws/aws-sdk-net/issues/528), or make the files publicly available inside your bucket, but this would be a security issue.

Any advise you could provide would be really appreciated! best regards to both!

Well, depending on the volume of data you expect to handle, it may worth using S3 as a storage for your DICOM. It will depend on your needs.

2- DCM4CHEE or Orthanc are PACS servers, the Non standalone viewer requires a PACS server, this interaction is defined in the DICOM standard, both OHIF and PACS servers provide the necessary information about how to connect in their documentation. It is a cumbersome dependency, but you might reap the benefits and reuse it later, for example, when you need to receive images from medical equipment (which are designed to communicate with PACS servers.)
3- This is an interesting point, what you are describing is a cache. There are three nodes in the design, S3, the server and the client. You might choose to cache the images in the server or in the client if you wish to save some money. You will be left to implement this cache on your own. I haven't checked whether the s3 plugins for Orthanc or DCM4Chee provide caching. But it would be a nice place to put this functionality, in order to avoid having devs like us develop caches multiple times. As diaztula mentions, since S3 and the server will be in the same region, this isn't an issue. This would only make sense for applications that are distributed globally. I don't know your use case, but it's pretty rare in healthcare.
4- Shouldn't be a problem, the presigning of requests should be automated.

and there is not batch presign AFAIK.

thanks @diaztula, as you say everything depends on our needs and the volume of data. Currently the volume of data is huge so that's why we are looking for cloud storage archive (like S3) so forgot about space and costs related to generating backups, etc.
1- Thanks for your advise. I also think that using the standalone viewer is better (the only disadvantage is that the tool bar cannot be changed you have to use it as it is.. and the other one is that you have to handle the JSON creation and handle very well the backend server side, I've never used AWS Lambda functions yet, but I'll research about them). On the other hand, The OHIF Viewer approach relays the JSON creation (I guess) to the PACS server (i.e. DCM4CHEE has) and I'm not sure if I would

@diaztula

@diaztula Do you suggest to use the Standalone Viewer instead of the OHIF Viewer... and handle the JSON creation in a backend server (which could be AWS Lambda function to resolve security issues and forgot about the server). Am I right?
Yes, that's it.

I also think that using the Standalone Viewer is a better approach than using the OHIF Viewer. The only disadvantage is that in the Standalone you cannot change the tool bar (as far as I know) and you need to handle the JSON creation and be careful when coding the backend server for that (or use AWS Lambda functions which I've never used before but I'll research about them), maybe using a Lambda function in the same region as S3 has no charge when reading files to S3 to get the pre-signed URLs for each DICOM file...

Yes, if you want to keep your files safer you would generate a presigned URL every time the study is opened.

that's the idea.... to keep it safer

For large studies (e.g. CTs with thousands of files) and/or many clients using the viewer at the same time, performance could be an issue, since for each file you'll need to call presign from AWS and there is not batch presign AFAIK.

that's my problem :(

If you download your files from S3 to a server outside AWS (or inside AWS but in a different region) then you will pay download transfer anyway. And you will have to serve those files from your server so the OHIF can read them.

yes but I was thinking that I'll just pay once that is when you download the study and then you can read it multiple times with no charge from S3 (I'm still not sure if S3 charges for reading a file... instead of downloading it). Anyway this approach has also an issue: I'll have to implement a kind of CACHE (small filesystem) in my backend server and take care about it's security... and that could take a lot of time and I'm not an expert on that matter....
On the other hand, I was also thinking that maybe it's possible to  create the JSON just one time and keep it in S3 with the study... and then when an user needs to see his study then in some way parse it and just update the URLs inside with the new pre-signed URLs and then reduce the cost to read each file again in S3 to generate the JSON... Not sure if you understand the idea.

You could create presigned links that last at most one year (aws/aws-sdk-net#528), or make the files publicly available inside your bucket, but this would be a security issue.

Yes I know, but as you said before that would not be secure, the idea is to create them with an hour of timeout for example, and then if the user wants to see his study again the pre-signed URLs will be generated again, but again not sure for large studies how long could take or if could be a performance issue,..

Well, depending on the volume of data you expect to handle, it may worth using S3 as a storage for your DICOM. It will depend on your needs.

Yes, that's for sure completely right. The volume of data is huge (need to handle a lot of different medical images, not just CTs) and I think S3 is one of the best solutions (since they will pay for the space they consume and most important thing: will never loose the data). But since I'm pretty new working with DICOM images and PACS servers, etc... I was just looking for some advise from people who has more knowledge and can provide some guidance, or just say me "hey! you are misunderstanding this!... " :P LOL
Again, thanks so much for spending your time on replying back to me! I really appreciate any help from the experts.

@TZubiri 👍

2- DCM4CHEE or Orthanc are PACS servers, the Non standalone viewer requires a PACS server, this interaction is defined in the DICOM standard, both OHIF and PACS servers provide the necessary information about how to connect in their documentation. It is a cumbersome dependency, but you might reap the benefits and reuse it later, for example, when you need to receive images from medical equipment (which are designed to communicate with PACS servers.)

Yes that's what I understood. However the user already have a PACS server (but the host it locally... with K-PAC... as far as I was able to analyze... it's not a great solution and they have lost several times studies because they are storing them locally... crazy thing)
So, when looking at Orthanc (I'm really not sure if I could use it for my solution since it's not completely free... unless I understood wrong). DCM4CHEE seems to be Open Source but (like Orthanc) you need to relay on it's own Architecture (J2EE) it's own DB.. etc... and would be replacing the PACS Server that my user already has installed (K-PAC) and that could be a huge change for them.... Also, I'm not sure about which kind of AWS Server should I have to pick to host this kind of server... On the other hand it seems it's resolves the JSON creation and has the ONLINE and NEARLINE Storage... but not sure if the ONLINE is inside the same server (so it could serve as the CACHE I'd have to develop if I use the Standalone Viewer) and the NEARLINE is S3... or how it works,.. It seems that DCM4CHEE servers resolves a lot of things... So I'm not sure if it would save me a lot of time and head aches, but again I'll relay on their architecture... and currently my backend is Node.js express server... which is something completely different to J2EE,.... and not sure is a good design solution to mix them... or use two different server... I'll have to research deeper about what's the best design and solution.

3- This is an interesting point, what you are describing is a cache. There are three nodes in the design, S3, the server and the client. You might choose to cache the images in the server or in the client if you wish to save some money. You will be left to implement this cache on your own. I haven't checked whether the s3 plugins for Orthanc or DCM4Chee provide caching. But it would be a nice place to put this functionality, in order to avoid having devs like us develop caches multiple times. As diaztula mentions, since S3 and the server will be in the same region, this isn't an issue. This would only make sense for applications that are distributed globally. I don't know your use case, but it's pretty rare in healthcare

So, you guys say that if S3 and the backend server are both in the same AWS region then the Cache will not be necessary? I didn't get that. could you please clarify?

4- Shouldn't be a problem, the presigning of requests should be automated.

No the only problem I see is that for the Standalone Viewer the JSON which needs to be created would need to be generated every time the pre-signed URL expires and that would mead to go to S3, read the entire study again (not sure if S3 charges for that) and generate the JSON with the new S3 URL... As I was saying to @diaztula before,.. maybe I can just generate the JSON once... and then each time the users wants to see the study again, if the URL already expired... then just get the new URLs and in some way insert them in the JSON which already was created... and in that case I should not have to go to S3 again to read each DCM file to extract its data and generate the JSON... I mean the JSON once created from the Study folder (which contains all the DCM files) is always the same... the only thing that will be changing is the URL for each DCM instance... and that the only thing I'd need to update... I don't know... I think it could be another solution to reduce S3 Costs and also improve performance. The JSON can be created when uploading the Study for the first time in S3 and that's all you will not have to read and parse every DICOM file again to get it... just change it to update the URL...
Anyway... Thanks so much for your help and time on replying back to me! 👍

2- The clinic I am working on also has a local PACS instance, however I am setting up my own Orthanc PACS in parallel, to allow me to have control over images transfered from their existing PACS, or medical equipments. Orthanc was designed to be a complement to traditional PACS servers, so nothing crazy is going on here, I view it as an interoperability tool.
Again, in my project I am trying to delegate the S3 Handling to Orthanc, so Ohif only has to worry about Dicomweb interfaces. This seems like the most reusable solution; by having solutions go through a centralizing technology, I avoid the problem of needing all technologies to interface with each other, which would require a . I don't need an s3 to viewer interface, I need an s3 to PACS technology, and a viewer to PACS technology.  But these 2 technologies will be more popular (and therefore more robust).
P.S: The amount of interfaces needed for a group of X technologies to have interfaces between all of them would be C(X,2)

So, you guys say that if S3 and the backend server are both in the same AWS region then the Cache will not be necessary? I didn't get that. could you please clarify?
3- Correct.

4- http://wiki.c2.com/?PrematureOptimization I'd say don't worry about it yet, implement it first, then worry about it if it's an issue.

@TZubiri thanks so much for your advice. I'll try to research a little bit more until deciding what architecture design to choose, but what you say has sense... reuse OHIF Viewer and Orthanc will save a lot of time. BTW, just last concerns:

Is Orthanc totally free? There is no issue to use it for my application design that is also for a clinic?
Why did you choose Orthanc instead of DCM4CHEE? Is it better option?
Thanks!


Fixed #24873 -- Prevented Prefetch objects from being overwritten.

Ticket 24873
Also I don't know if this could be considered a release blocker or not and where to discuss about it. Prefetch was introduced in Django 1.7 and this bug is there since the beginning. It keeps Prefetch from being used in applications with lots of relations.

Updated pull request.

buildbot, add to whitelist.

merged in 74261bc593bcf9411e7e25a4731dd9471499819e, thanks!

@timgraham do you have a sense for when/if this will be merged into 1.7? We have an API that relies heavily on nested prefetches and noticed this error. Considering starting a fork until this gets into a stable version. Thanks!

No, I didn't plan to backport it as it's not a regression (if we had found the issue during the mainstream support period for 1.7, it would be more likely to be backported as a bug in a new feature).


Tooltip: visual issue on first use

What happened?
Whenever a tooltip is shown for the first time, it appears at the top left corner of the window before jumping to the correct position.
This can also be reproduced on the documentation page of the tooltips:
Open the example in a full window and hover over one of the buttons.
What type of frontend frameware are you seeing the problem on?
Angular
Which version of iX do you use?
2.0.1
Code to produce this issue.
// Can be observed on the documentation page. Happens on every tooltip implementation.

We observe the same behavior when using iX with React (iX ver. 1.6.3, React ver. 18.2.0).

Position was calculating after visible so I updated the order. please follow the PR.


Create 1.26

What would you like to be added:
Why is this needed:

Projects to update (at start):

[ ] aws-cloud-controller-manager
[ ] aws-iam-authenticator
[ ] cni plugins
[ ] coredns
[ ] etcd
[ ] external-attacher
[ ] external-provisioner
[ ] external-resizer
[ ] external-snapshotter
[ ] kubernetes
[ ] kubernetes-release
[ ] livenessprobe
[ ] metrics-server
[ ] node-driver-registrar
[ ] pause

done


Populate the Code Of Conduct Panel

Now that the base governance for Code of conduct reporting and handling has landed we need to populate the CoCP and/or modify the composition which specified to match what we believe the final form should be.
We agreed in https://github.com/openjs-foundation/cross-project-council/issues/432#issuecomment-585936270 that as an interim measure the members who had volunteered for the temporary Code of Conduct team in openjs-foundation/cross-project-council#432 could act as temporary CoCP members as we ramp up the members as specified in the governance. As such they should be include in<EMAIL_ADDRESS>along with @rginn as the Executive director of the foundation.
<EMAIL_ADDRESS>should already be going to all CPC members.
@brianwarner has agreed to send out/manage

call for volunteers/election for Board representative
call for volunteers/electrion for CPC representative

We'll need to start the same for representatives from the projects and marketing committee as well.

[x] The Foundation Executive director - Robin Ginn (@rginn)
[ ] 1 member from the Board
[ ] 1 member from the CPC
[ ] 1 member of the OpenJS marketing committee
[ ] each impact project may optionally provide 1 member
[ ] the growth, at-large and incubating projects may optionally provide 1 member
[ ] optionally 1 invited outside expert as agreed by the other CoCP members.

Members:
@rginn from Foundation
@nainar from AMP
@ljharb from Node.js

Thanks @jorydotcom.  I just wanted to note that I've configured the OpenJS Foundation directory to manage the membership for the list.

List of current recipients
Open PRs against this yaml file to modify list membership


When running the game, there are bugs and the sound has static noise

Answers checklist.

[X] I have read the documentation and the issue is not addressed there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
An error occurred while running the game：
E (78032) spi_master: check_trans_valid(796): trans tx_buffer should be NULL and SPI_TRANS_USE_TXDATA should be cleared to skip MOSI phase.
Couldn't queue trans: 258 'ESP_ERR_INVALID_ARG'
Additionally, it has been noticed that there is static noise while playing the background sound of the game.

@LHYhorion I've seen the transaction data bug before, though TBH I thought I'd fixed it.
Regarding sound - I have found that the NES sound emulation is not very good right now, though I think the GB/C sound should be better. Is the sound issue you're experiencing with the NES or the GB/C emulators?
Right now there are some bugs and crashes that can be triggered. As I get time I'm looking into what might be the cause of them as well as ways to fix them, though any help would be appreciated.

The sound of the NES emulator isn't very good, but I found that removing the 32-bit conversion and retaining the 16-bit sound makes it sound better. Regarding the error issue, I found that sometimes the 'length' is calculated as 0. Assigning a non-zero value when it's calculated as 0 will prevent the error.

Thanks for the tip about the sound conversion potentially being the root cause. I'll make the change and test if that improves the quality of the output sound on my side as well.
Regarding the length issue on the SPI transaction - in the transmit function i'm doing a length check on the data being sent so a length of 0 shouldn't be an issue.  Regarding your comment of assigning a non-zero value when its calculated as 0 - what part of the code are you talking about here? Within the box-emu's hal component, the nes component, or within the espp/display_drivers component which is the main place the length is calculated for the SPI transaction data.


Fix "preload" markup - HSTS rules

TLDR;
the last semicolon was obsolete.
PreStory
Checked my site with this tool and the submission failed because the Markup wasn't correct.

The Required Markup (according to RFC6769):
Strict-Transport-Security: max-age=63072000; includeSubDomains; preload

The provided Markup from this package:
Strict-Transport-Security: max-age=63072000; includeSubDomains; preload;

Conlusion: remove the last ;

Hey @BePsvPT,
is anything missing or do I need to add something?
Cheers,
Enaah

@Enaah thanks a lot.


Introduce define to change namespaces

This PR is connected with these changes: https://github.com/openvinotoolkit/openvino/pull/6920

@apankratovantonp Is it ok that I have the next issue for ubuntu20:
CMake Error at /usr/local/share/cmake-3.13/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
 Could NOT find Java (missing: Java_JAR_EXECUTABLE Java_JAVAC_EXECUTABLE
Java_JAVADOC_EXECUTABLE) (found version "11.0.10")

???

@apankratovantonp Is it ok that I have the next issue for ubuntu20:
CMake Error at /usr/local/share/cmake-3.13/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
 Could NOT find Java (missing: Java_JAR_EXECUTABLE Java_JAVAC_EXECUTABLE
Java_JAVADOC_EXECUTABLE) (found version "11.0.10")

???
Yes, it is ok. This platform dose not even build ARM plugin.

@apankratovantonp Could you have a look?


mimetype is always 'application/octet-stream'

The mimetype always appear to be defined as 'application/octet-stream'

Even when there are no uploads?

tbh I did not check without upload

We can reopen to investigate if you are still having an issue and are able to provide specific reproduction steps that highlight what happens, vs what is expected to happen 😌


preparations for isolated TS modules

isolatedModules isn't actually enabled in tsconfig right now, which is why we're catching issues with type re-exports late. We can't enable it yet, as we need to wait for the jest version to be bumped to 26, as there are some bad types in a dependency.
These fixes will at least allow you to use yarn tsc --isolatedModules to check for errors.

Right? who doesn't want export {}


feat: add helm provider

Scope:

[x]  create helm provider
[x]  create provider credential framework to pass k8s credentials to helm provider.

Upgraded golang to version 1.17 to address https://github.com/kubernetes-sigs/json/issues/8

Merge conflicts resolved @rohilsurana


New package: BokehBlink v0.4.0

Registering package: BokehBlink
Repository: https://github.com/cjdoris/Bokeh.jl
Created by: @cjdoris
Version: v0.4.0
Commit: 9d5ccbf9a091550ce7c4b859eca48776abbb5776
Git reference: HEAD
Description: Interactive plotting made easy

You can close this, there's now another PR for the same package.


Orchestrated shutdown of the whole cluster

The idea is to have a ceph-salt shutdown command to shutdown ceph cluster.
1) Find and admin node

2) Find the FSID

3) Execute `ceph osd ...` on that admin node:
# ceph osd set noout
# ceph osd set nobackfill
# ceph osd set norecover
# ceph osd set norebalance
# ceph osd set nodown
# ceph osd set pause
# ceph orch stop osd

4) Stop ceph service on all nodes:
# salt run \* 'systemctl stop ceph-<fsid>.target'

We may need a ceph-salt up to start the cluster again.
See https://ceph.io/planet/how-to-do-a-ceph-cluster-maintenance-shutdown/ for more information.

rethinking this: ceph orch stop osd might not work. The idea is to prevent any destructive calls in cephadm. and  ceph orch stop osd is exactly that: destructive.
Which means we'd need to build some bypass if we need to call this.

The blog post says to stop services in the following order:

gateways
osds
mons/mgrs

It seems reasonable to start them in the reverse order.

There's another aspect to this:
The process of taking a cluster down is often combined with powering down the hardware. As @sebastian-philipp 's comment implies, it's a sensitive operation. I wonder whether ceph-salt really needs to be in the business of shutting down clusters and bringing them up again?

This is indeed a very sensitive operation, and ceph-salt don't know where gateways/osds/mons/... are deployed to stop services on the correct order.
I believe it's not clear ATM if this should be implemented in ceph-salt or cephadm itself.
Due to the above reasons plus the risk of supporting a feature like this, I'm closing this issue until really needed.

I've now learned that part of the shutdown / startup process is implemented in Deepsea, so we will need a similar implementation for "feature parity":

https://github.com/SUSE/DeepSea/tree/99f5e23248cef9299d983a2fa2da4699aaa16eff/srv/salt/ceph/shutdown
https://github.com/SUSE/DeepSea/tree/99f5e23248cef9299d983a2fa2da4699aaa16eff/srv/salt/ceph/startup

... ceph-salt don't know where gateways/osds/mons/... are deployed to stop services ...

ceph-salt can use ceph orch service stop <service> to stop all daemons of a service


support Scala 2.13.0-M3 (and bump to sbt 1.1.1)

Seems like its just scalameter which is not published, I do however want to change to sbt-jmh at one point

Actually nvm about the sbt-jmh part, I just noticed that scalameter is better suited for benchmarking for this project

I sent a PR with a 2.13.0-M3 target.

@mdedetrich ping?

@SethTisue Do you want to merge this or what is missing?

Codecov Report

Merging #49 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   16.12%   16.12%
=======================================
  Files           5        5
  Lines         806      806
  Branches      241      241
=======================================
  Hits          130      130
  Misses        676      676

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 7f90008...fce5d6e. Read the comment docs.

oh, I still needed to update the ScalaMeter dependency. done
let's see if Travis-CI likes it now, if so, this will then be mergeable afaik. (I guess you'll address Kenji's comment separately.)

hmm
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.storm-enroute#scalameter_2.13.0-M3;0.10-SNAPSHOT: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::

ok, green now. had to add the sonatype snapshots resolver. (I hadn't needed it locally since I use a local Artifactory that has it.)


chore: release main

:robot: I have created a release beep boop
biblo: 0.12.1
0.12.1 (2023-02-06)
Bug Fixes

Don't show prop values of 0 as undefined in ReaderControls (5a508db)
Hide ReaderControlsToggle when all props are hidden (5d5750c)

react-native: 0.11.2
0.11.2 (2023-02-06)
Bug Fixes

Don't show prop values of 0 as undefined in ReaderControls (5a508db)
Hide ReaderControlsToggle when all props are hidden (5d5750c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/saxofonsolo/biblo/releases/tag/biblo-v0.12.1 :sunflower:

:robot: Release is at https://github.com/saxofonsolo/biblo/releases/tag/react-native-v0.11.2 :sunflower:


Iterator functionality for pubsub

Adds a way to iterate over items in pubsub. I'm not sure if I've done this correctly, please let me know of any corrections.

This makes sense, nice clean implementation. Was there a driving need for it?

This makes sense, nice clean implementation. Was there a driving need for it?

Internal.. Was seeing all this iterate code which could be abstracted out.. Hence added it..


added guest_applications

Adding this PR to keep the manifest in sync with https://github.com/RedHatInsights/topological_inventory-api/pull/267

@miq-bot add_label enhancement

@miq-bot add_label ivanchuk/yes


Save Character button doesn't work after first use

Describe your bug here.
I've been having a problem in the character editor where, after using Save Character once, clicking it again won't do anything. The only way to get it to work again is to leave and re-enter the editor; switching to another character doesn't work. Minor issue, but still very inconvenient.
Command Prompt/Terminal logs (if existing)
N/A

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

OK so this was an issue for 0.7.2h that's already been fixed in the update i didn't know was out lol


TimeoutError when running mnist.py

Bug Description
When running mnist.py after some time, suddenly an error occurred :
Traceback (most recent call last):
File "/home/jdiez/project/AutoKeras/Test_01.py", line 11, in
clf.fit(x_train, y_train, time_limit=20 * 60)
File "/home/jdiez/lib/anaconda3/envs/autokeras/lib/python3.6/site-packages/autokeras/image_supervised.py", line 238, in fit
run_searcher_once(train_data, test_data, self.path, int(time_remain))
File "/home/jdiez/lib/anaconda3/envs/autokeras/lib/python3.6/site-packages/autokeras/image_supervised.py", line 41, in run_searcher_once
searcher.search(train_data, test_data, timeout)
File "/home/jdiez/lib/anaconda3/envs/autokeras/lib/python3.6/site-packages/autokeras/search.py", line 175, in search
timeout)
File "/home/jdiez/lib/anaconda3/envs/autokeras/lib/python3.6/site-packages/autokeras/bayesian.py", line 272, in optimize_acq
raise TimeoutError
TimeoutError
Process finished with exit code 1
Reproducing Steps
Steps to reproduce the behavior:

Step 1: Follow the instruction steps for 0.2.4 install
Step 2: Run mnist.py

Expected Behavior
Run without error
Setup Details
Include the details about the versions of:

OS type and version: Ubuntu 16.04
Python: 3.6
autokeras: 0.2.4
scikit-learn: 0.19.1
numpy: 1.14.5
keras: 2.2.2
scipy: 1.1.0
tensorflow: 1.8.0
pytorch: 0.4.1 (py36_cuda9.2.148_cudnn7.1.4_1)

Additional context
Torch, Tensorflow and Keras are all working fine insisde this conda env.
The same error occurs when adapting the code for CIFAR10, Fashion-MNIST

@Jules-Diez Thank you for your report.
I have solved it in the current master branch.

Ok I will try it tonight and see if it happens again or not.

I think that is a normal behavior.
Thank you for your feedback! @Jules-Diez


refactor: Inverted Asker and Bidder

Refactor Request
Describe the Refactor Request
The dApp currently utilizes two interfaces, PopulatedSwapOfferCard and FormattedSwapOfferAssets, for managing swap offers. However, there is inconsistency in the usage of bidder and asker addresses throughout the codebase, leading to potential errors. The refactor request aims to correct this inconsistency by ensuring that the asker address corresponds to the allowed address, while the bidder address corresponds to the owner address.
Definition of done:

[x] Refactor all objects using the PopulatedSwapOfferCard and FormattedSwapOfferAssets interfaces to ensure correct assignment of asker and bidder addresses.
[x] Verify and rectify all instances where bidder and asker addresses are inverted.
[x] Update all code segments that interact with these interfaces to maintain consistency with the corrected object structure.

The dApp have 2 Interfaces:
export interface PopulatedSwapOfferCard {
  id: bigint;
  status: PonderFilter;
  expiryDate: bigint;
  bidderTokens: {
    address: EthereumAddress;
    tokens: Token[];
  };
  askerTokens: {
    address: EthereumAddress;
    tokens: Token[];
  };
}

export interface FormattedSwapOfferAssets {
  id: string;
  status: string;
  expiryDate: bigint;
  bidderAssets: {
    address: EthereumAddress;
    tokens: Asset[];
  };
  askerAssets: {
    address: EthereumAddress;
    tokens: Asset[];
  };
}

Describe Preferred Solution
Identify and rectify instances where bidder and asker addresses are inverted. Ensure that the asker address corresponds to the allowed address and the bidder address corresponds to the owner address.

the asker address should be the allowed address.
the bidder address should be the owner address.

Related Code

Investigate all parts of the code that reference bidder and asker addresses.
Review all code segments that utilize the PopulatedSwapOfferCard and FormattedSwapOfferAssets interfaces.

Closed by #368


mavlink_parameters: workaround for param mismatch

This could workaround a potential problem with E90 where the MAVLink parameter type does not match the xml type.
@byuarus please test. I ran out of time to test this against an E90 but I thought it was worth a shot.

@julianoes @JonasVautherin
Hey guys, I found the same issue with the E10T camera and made a PR https://github.com/mavlink/MAVSDK/pull/841
It will support mismatch uint8 <-> uint32


Uncaught ReferenceError: Popper is not defined

Hi
I get the same error reported here
https://stackoverflow.com/questions/46952775/bootstrap-4-uncaught-referenceerror-popper-is-not-defined
bootstrap's version is 4.0.0 beta 2
popper's version is 1.12.6
I use them on wordpress 4.8.2 with chromium 62.0.3202.75 on Arch Linux 64bit (I'm testing it on my local server on debian 9)

I report here another small problem that I see from its web toolkit
"invalid proporty value" if I select the "a" tag
-webkit-text-decoration-skip: objects;
a {
color: #007bff;
text-decoration: none;
background-color: transparent;
-webkit-text-decoration-skip: objects;
}
from _reboot.scss on line 199

thanks
bye

Popper.js should be included before bootstrap not after 😉
About the invalid that's because your browser don't need this CSS rule but they are users which will need that

Hi
thanks
changing the order now it is ok
bye

I am getting same issue.
Please check my link-
http://devstheme.net/
give solution.

ReferenceError: Popper is not defined
After changing order I am getting this error..
SyntaxError: export declarations may only appear at top level of a module
popper.min.js:4:17879
ReferenceError: Popper is not defined
[Learn More]
bootstrap.js:3849:1

That's because you didn't choose the UMD release of Popper.js @NazreAlam

Use<EMAIL_ADDRESS>

@shmakovpn so where do you paste the info in the file you provided?? I'm new a JS and attempting to create a basic app (my first one!), I'm very early into the project and I CANNOT seem to get my JS to work because of this undefined..popper bullshit. I have the script in the correct order. I downloaded it properly and am using local storage... yet I keep getting this error. Please help. Thanks!

@PCMS88 templates/base.html
You need it

before that

You can find popper.js and my base.html in github.com/shmakovpn/jobex


Passport Policy Suggestion

@tjwebb I've been digging around a lot of different implementations of Passport.js into sails, and I've found this really useful.  By updating policies/passport.js to this, it will allow socket passport authentication which I am seeing more and more of.
// policies/passport.js
var http = require('http');
var methods = ['login', 'logIn', 'logout', 'logOut', 'isAuthenticated', 'isUnauthenticated'];
module.exports = function (req, res, next) {
  // Initialize Passport
  passport.initialize()(req, res, function () {
    // Use the built-in sessions
    passport.session()(req, res, function () {
      // Make the user available throughout the frontend
      res.locals.user = req.user;
      // Make the request's passport methods available for socket
      if (req.isSocket) {
        for (var i = 0; i < methods.length; i++) {
          req[methods[i]] = http.IncomingMessage.prototype[methods[i]].bind(req);
        }
      }
      next();
    });
  });
};

I've just tested this using:
Sails: 0.11.0
sails-auth: 1.0.8

socket passport authentication which I am seeing more and more of.

I'd love to incorporate this; would you like to submit a PR with this feature, along with a basic test showing how it works? Then I can update the docs and publish a new version with your improvements.
Thanks!

Sure thing, will do.

I've submitted a PR I ran a Mocha test on it and it passed (but again, I'm not great at the testing yet and any feedback is much appreciated). Super easy to add to your workflow and is great for things like Modal Logins or Login Confirmations especially while working with single page apps.  Personally I use angularSails with it, but you can do it with stock sails as well:
It accepts 'login', 'logIn', 'logout', 'logOut', 'isAuthenticated', 'isUnauthenticated'
//Login Example
io.socket.post('/auth/local', {
          identifier<EMAIL_ADDRESS>          password: 'admin1234'
      }, function (resData) {
          //Returns either and error or JSON object of logged in User, which means the users subsequent AJAX requests will be authenticated. If you have CORS enabled you'll want to send a prerequisite socket request to get that first and then add it to your following socket requests.
      });

@tjwebb, this can be closed, it was fixed with the lastest merge.


Receive Notifications in Background on Windows

See Issue #93 for history
Basically, when a raw notification is received and the app is in the background then the on('notification') event hander should be called.

Could you please give an update on this issue? As far as I understand this is needed to trigger a tab on the notification. Actually push.on('notification', function(data) does not fire when a user tabs the notification. IMO this is a must feature and needed for various scenarios. Thanks a lot!

@rdesimone I'm waiting for @rakatyal to get some time to take a look.

@macdonst thanks!

@rdesimone: Unfortunately we are not working on this currently but we plan to come back to it probably in the next month.

@rakatyal For our app we managed to trigger the tap on notification by listening to the 'activated' event and reading the launch args which are included in the toast. For cold start we have actually a very ugly working hack by modifying cordova.js.
var activationHandler = function (e) {                 var args = e.detail.arguments;                 window.localStorage.setItem("launch-args", args); // ugly hack                 var actType = e.detail.type;                 platform.activationContext = { type: actType, args: args };                 cordova.fireDocumentEvent('activated', platform.activationContext, true); };
We write the launch args into local storage. As on cold start 'activated' fires before 'deviceready', we check after 'deviceready' if launch args are available. The local storage solution is a hack, but it was not possible to add a working event listener outside of onWinJSReady = function() {...}. Trying with:
Windows.UI.WebUI.WebUIApplication.addEventListener("activated", resumingHandler, false);
and
WinJS.Application.addEventListener("activated", resumingHandler, false);
It is possible to receive the event but the launch args are empty where ever these listener are placed in the code. Maybe these finding help also for implementing the background notifications. Do you have any idea for a more elegant solution? Thanks!

For cold start I found a nicer solution without a hack:
var launchArgs = cordova.require('cordova/platform').activationContext.args ;


Http.Sys server throws 'Access denied' exception when machine IP addresses are registered with UrlPrefixes in VS2019 but VS2017 is working fine

Description
I have Asp.Net Core Web app that is hosted with Http.Sys server and configured to listen to all IPv4 addresses from local machine. When I try to run the application with VS2019 it throws an Http.Sys exception 'Access denied' for ip addresses that are different from Loopback address. If it is only the loopback ip address than it works fine, so the problem is for other ip addresses of local machine.
But the strange thing is that this is not the case with VS2017, It works fine and doesn't throw an exception even if my local machine has more ip addreses. And after running the application with VS2017, later on when I go back and rerun the application with VS2019 then it works also fine, seems like VS2017 has ability to give some kind of permission but VS2019 has not.
Program.cs hosting part:
.........
.........
webHostBuilder.UseHttpSys(options =>
               {
                   options.Authentication.Schemes = AuthenticationSchemes.NTLM | AuthenticationSchemes.Negotiate;
                   options.Authentication.AllowAnonymous = true;

                  var appUri = new Uri("http://localhost:5003");

                   //Get all ip4 addresses for local computer.
                   var localIp4Addresses = Dns.GetHostAddresses(Dns.GetHostName()).Where(x => x.AddressFamily == AddressFamily.InterNetwork).ToList();
                   localIp4Addresses.Add(IPAddress.Loopback);

                   //Listen on all IPs using IPv4.
                   foreach (var ip4Address in localIp4Addresses)
                   {
                       options.UrlPrefixes.Add($"{appUri.Scheme}://{ip4Address}:{appUri.Port}");
                   }
               });

Further technical details
.NET CORE: 2.1
Microsoft.AspNetCore.Server.HttpSys: <IP_ADDRESS>
Operating system: Windows 10 x64

Http.Sys has three modes:

localhost is always allowed.
Running as administrator you can register any IP/host.
Otherwise you need to pre-register IPs/Hosts using netsh.exe.

There's no relation to VS permissions here. It sounds like you're running VS as an admin in some instances.
Note registering for all IPs is not necessary, registering using the machine's host name should be adequate.

I have set up my VS2019 shortcut to run always as administrator, I don't know how this happened but yes you are right, I checked it and it works when VS2019 is started as administrator.
Additionally I should add here that when the app is hosted with machine's host name only and when I try to access the application with any IP of this machine, it says Invalid host name with HTTP Error 400.
The application can be accessed only with the hostname, so this is why I am registering all the IP's.
Anyway, thanks a lot :)


ssh-agent plugin - issue loading identities

Seems like the plugin is adding an extra / for some reason, as I'm getting the following each time I open a new terminal:
ssh-keygen: /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work: No such file or directory ssh-keygen: /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github: No such file or directory /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work: No such file or directory /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github: No such file or directory
My .zshrc contains the following:
`plugins=(
zsh-completions
git
colored-man-pages
colorize
history
sbt
bundler
rake
rbenv
ssh-agent
)
zstyle :omz:plugins:ssh-agent agent-forwarding on
zstyle :omz:plugins:ssh-agent identities ~/.ssh/id_rsa_work ~/.ssh/id_rsa_work_github
source $ZSH/oh-my-zsh.sh`
Relevant part of debug logs:
`_add_identities
+/Users/bschmidt/.oh-my-zsh/plugins/ssh-agent/ssh-agent.plugin.zsh:78> _add_identities
+_add_identities:1> local id line sig lines
+_add_identities:2> local -a identities loaded_sigs loaded_ids not_loaded
+_add_identities:3> zstyle -a :omz:plugins:ssh-agent identities identities
+_add_identities:6> [[ ! -d /Users/bschmidt/.ssh ]]
+_add_identities:12> [[ 2 -eq 0 ]]
+_add_identities:21> lines=+_add_identities:21> ssh-add -l
+_add_identities:21> lines=$'4096 SHA256:ppIhNeqcn7s8dTFQ6uqILwbje/duuuKLcPHBYilVFUo /Users/bschmidt/.ssh/id_rsa_work (RSA)\n4096 SHA256:TdTOo+x7KWJ8ciahJLZ99GoAc5TgNWmyP/OpbDoaEV4 /Users/bschmidt/.ssh/id_rsa_work_github (RSA)'
+_add_identities:22> line=4096 SHA256:ppIhNeqcn7s8dTFQ6uqILwbje/duuuKLcPHBYilVFUo /Users/bschmidt/.ssh/id_rsa_work (RSA)
+_add_identities:23> loaded_sigs+=SHA256:ppIhNeqcn7s8dTFQ6uqILwbje/duuuKLcPHBYilVFUo
+_add_identities:24> loaded_ids+=/Users/bschmidt/.ssh/id_rsa_work
+_add_identities:22> line=4096 SHA256:TdTOo+x7KWJ8ciahJLZ99GoAc5TgNWmyP/OpbDoaEV4 /Users/bschmidt/.ssh/id_rsa_work_github (RSA)
+_add_identities:23> loaded_sigs+=SHA256:TdTOo+x7KWJ8ciahJLZ99GoAc5TgNWmyP/OpbDoaEV4
+_add_identities:24> loaded_ids+=/Users/bschmidt/.ssh/id_rsa_work_github
+_add_identities:29> id=/Users/bschmidt/.ssh/id_rsa_work
+_add_identities:31> [[ 0 -le 0 ]]
+_add_identities:32> sig=+_add_identities:32> ssh-keygen -lf /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work
+_add_identities:32> sig=+_add_identities:32> awk '{print $2}'
ssh-keygen: /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work: No such file or directory
+_add_identities:32> sig=''
+_add_identities:33> [[ 0 -le 0 ]]
+_add_identities:33> not_loaded+=/Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work
+_add_identities:29> id=/Users/bschmidt/.ssh/id_rsa_work_github
+_add_identities:31> [[ 0 -le 0 ]]
+_add_identities:32> sig=+_add_identities:32> ssh-keygen -lf /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github
+_add_identities:32> sig=+_add_identities:32> awk '{print $2}'
ssh-keygen: /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github: No such file or directory
+_add_identities:32> sig=''
+_add_identities:33> [[ 0 -le 0 ]]
+_add_identities:33> not_loaded+=/Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github
+_add_identities:37> [[ -n '/Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github' ]]
+_add_identities:37> ssh-add /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github
/Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work: No such file or directory
/Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work_github: No such file or directory

Look at the logged errors:
ssh-keygen: /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work: No such file or directory

The path it looks for is /Users/bschmidt/.ssh//Users/bschmidt/.ssh/id_rsa_work, and that's because you're not supposed to put the absolute path in the zstyle parameter setting, just the filenames.
zstyle :omz:plugins:ssh-agent identities id_rsa_work id_rsa_work_github

Oh duh, makes sense, thanks brother!


Issues with a broken screen and the authorization popup.

My phones screen is broken and I can only access my phone via the Pc based version of Samsung dex. Because I don't see the popup, I cannot interact with it easily.
Questions:
Does the phone need to be unlocked for the popup to show up?
and if not, is there a workaround to it?

See https://github.com/Genymobile/scrcpy/issues/4914#issuecomment-2105635639

Does the phone need to be unlocked for the popup to show up?

Yes.


vaultwarden: 1.23.0 -> 1.23.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/vaultwarden/versions.
meta.description for vaultwarden is: "Unofficial Bitwarden compatible server written in Rust"
meta.homepage for vaultwarden is: "https://github.com/dani-garcia/vaultwarden"
meta.changelog for vaultwarden is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
/nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1/bin/vaultwarden passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.23.1 with grep in /nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1
found 1.23.1 in filename of file in /nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1

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

12 total rebuild path(s)

7 package rebuild(s)

7 x86_64-linux rebuild(s)
5 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1
ls -la /nix/store/agshxy3nsn5mwkr7002c0h3159pg36b1-vaultwarden-1.23.1/bin

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

1 package failed to build:

nixos-install-tools

3 packages built:

vaultwarden
vaultwarden-mysql
vaultwarden-postgresql

Maintainer pings
cc @msteen @ivan for testing.

3 packages built:
vaultwarden vaultwarden-mysql vaultwarden-postgresql


Some fixed to behaviour

[x] Added nginx_sites_available_subfolder and nginx_sites_enabled_subfoldrr params to make change config subfolder available
[x] Fixed upload config to a root owner folder

@ivalkeen ?


Feature Request - Allow mapping API Gateway context values to HTTP headers

Context
API Gateway allows mapping api gateway context values to HTTP headers, it seems that some of these values may be exposed in the request.environ, however if converting from a server based application to a lambda based application, using request.environ would require a modification to the app, whereas if zappa had a way to replicate the context to HTTP header mapping, no code changes would need to be made.
More details on api gateway context: http://docs.aws.amazon.com/apigateway/latest/developerguide/api-gateway-mapping-template-reference.html
Possible Fix
I have this implemented here: https://github.com/brianrower/Zappa/tree/feature/context-to-http-header-mapping
Testing and README update still needed, which I can do after discussion.
I've added a new setting to zappa_settings similar to environment variables where you map an HTTP header to it's api gateway context value, at runtime, the headers are mapped from the api gateway context, which is inside the lambda event.

Update now that I've done more investigation:

Zappa includes the LAMBDA context in request.environ['lambda.context']
BUT Zappa does NOT include the API gateway context (which is different)
The API gateway context is sent to the lambda in event['requestContext']
The API Gateway context is in the lambda's event['requestContext'], but this is not set in the request.environ like the lambda context is, so there currently is no way to access these values in the wsgi app
One solution would be to pass the whole event into the wsgi environ, this could be done with a simple 1 liner like this "environ['event'] = event" in handler.py right under "environ['lambda.context'] = context" but that would still require a user switching from API Gateway + a traditional web service to zappa to make a small modifcation to their code
A more full featured alternative would be the pull request I've submitted which uses zappa_settings to configure HTTP header mappings for the api gateway context, just like you can do in API Gateway if you're mapping to an API gateway to an HTTP request instead of a lambda.

My use case is blocked without being able to get the api gateway context. I'm utilizing AWS cognito to do IAM authentication in the API gateway and I need to get the cognito identity id to know who the user is.
I believe if I check the checkbox in api gateway that says "Invoke with caller credentials" the cognito identity id in the lambda context would get set, BUT that would require me granting all my cognito users the same IAM permissions as my lambda execution role, which would NOT be a good idea.

Thanks for adding this in!  I just wanted to add a user case example in case anyone else was trying to figure this out.
After adding the context_header_mappings to your zappa_settings file, the values can be accessed by using the request.headers object in Flask.
For example,
If I add this to my zappa_settings.yml file:
  context_header_mappings:
    user_id: authorizer.user_id

In my Flask app, I can how access this header using code like this:
from flask import request

@route('/hello')
def hello_world:
   print(request.headers.get('user_id'))

Added this example to documentation, closing ticket as resolved. (Feel free to reopen if this isn't resolved to satisfaction yet.)


Lock down propagate_errno behavior.

Test that initial value of errno is always zero.
This is because:
a) Each ecall creates a new enclave thread and
thread-local variables (including errno) are initialized
to their default values.
b) Ecalls do not transfer the current host errno value
over to the enclave.
Test that only ocalls annotated with propagate_errno actually
propagate the errno.
Test that int _ocall_errno field is only added for ocalls with
propagate_errno annotation.

bors try

Maybe flaky since the try passed...
bors r+

Failure log indicates that we were not able to create the enclave.
Hopefully this will happen again so that we can figure out what is going on.
54: Test command: D:\Jenkins\workspace\Bors_staging\build\X64-Debug\tests\oeedger8r\host\edl_host.exe "D:/Jenkins/workspace/Bors_staging/build/X64-Debug/tests/oeedger8r/enc/edl_enc"

54: Test timeout computed to be: 10000000

54: D:/Jenkins/workspace/Bors_staging/build/X64-Debug/tests/oeedger8r/host/edl_host.exe: cannot create enclave: 21

54: 21:51:57:000000 tid(0x8f0) (H)[ERROR]InitializeEnclave failed (err=0x80) (oe_result_t=OE_PLATFORM_ERROR)[..\..\host\sgx\sgxload.c oe_sgx_initialize_enclave:929]

54: 21:51:57:000000 tid(0x8f0) (H)[ERROR]:OE_PLATFORM_ERROR[..\..\host\sgx\create.c oe_sgx_build_enclave:577]

54: 21:51:57:000000 tid(0x8f0) (H)[ERROR]:OE_PLATFORM_ERROR[..\..\host\sgx\create.c oe_create_enclave:678]

 54/103 Test  #54: tests/oeedger8r .......................................***Failed    0.26 sec


Fixed bug with same generated value for multiple program runs.

At program start we have identical seed, so in case of using regen.Generate function - we get identical values.
It is better to get time based seed or always use NewGenerator with user defined seed, or set default seed at programm start.

I would rather leave the default implementation deterministic, continue to require the caller to opt-in to a time-based seed, and just update the comment on RngSource to explain what a nil value actually means. Defaulting to a time-based seed seems too magic and opinionated – time isn't an ideal seed since values don't actually change that much over short periods of time.

Is it a good idea to used deterministic logic in random generator?
User types something like this
func main() {
	res, _ := regen.Generate("[a-z0-9]{1,64}")
	println(res)
}

and don't waits identical result values at different execution of this program.
Or comment to regen.Generate() may tell user to predefined seed in this function.


HAWQ-1230. Introduce macro __MAYBE_UNUSED_FUNC to surpress "unused fu…

…ntion" warnings.

+1, nice job!

+1

Merged into master. Closing.