Does Flutter 3 not support https self-signed certificates?

From https://github.com/flutter/flutter/issues/112052 by @kenanwangyi
I can use self-signed certificates for network requests in the development environment of flutter 2.5.3, but after upgrading to flutter 3, it will report an error:
HandshakeException: Handshake error in client (OS Error:
UNSUPPORTED_PROTOCOL(handshake_client.cc:697))
Does Flutter3 not support https self-signed certificates? Or how to set it up?

Hello
I use HttpClient to make network requests, the code is as follows, our server uses a self-signed certificate https, which belongs to the internal network address and cannot be accessed from the external network temporarily.When I initiate a network request, it prompts  'HandshakeException: Handshake error in client (OS Error:
UNSUPPORTED_PROTOCOL(handshake_client.cc:697))' .
Does Flutter 3 not support https self-signed certificates? Or how to set it up?
try {
      HttpClient httpClient = new HttpClient();
      httpClient.badCertificateCallback =
          (X509Certificate cert, String host, int port) {
        return true;
      };
      var url = "https://api-dev.yeedev.com/oauth/token";
      HttpClientRequest request = await httpClient.postUrl(Uri.parse(url));
      HttpClientResponse response = await request.close();
      String responseBody = await response.transform(utf8.decoder).join();
      if (response.statusCode == HttpStatus.ok) {
        print('success');
        print(response.headers);
        print(responseBody);
      }
    } catch (e) {
      print("fail：$e");
    }
  }

flutter doctor -v

[!] Flutter (Channel unknown, 3.3.0, on macOS 12.6 21G115 darwin-x64, locale zh-Hans-CN)
    ! Flutter version 3.3.0 on channel unknown at /Users/fudong/fvm/versions/3.3.0
    ! Upstream repository unknown
    • Framework revision ffccd96b62 (3 weeks ago), 2022-08-29 17:28:57 -0700
    • Engine revision 5e9e0e0aa8
    • Dart version 2.18.0
    • DevTools version 2.15.0
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/fudong/Library/Android/sdk
    • Platform android-32, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F17a
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 93.0.4577.63

HTTP Host availability check is taking a long time...[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

flutter run -v (part of the content)

[ +228 ms] success
[        ] Application launched on the device. Waiting for observatory url.
[+3599 ms] [VERBOSE-2:FlutterObservatoryPublisher.mm(101)] Failed to register observatory port with mDNS with
error -65555.
[        ] [VERBOSE-2:FlutterObservatoryPublisher.mm(103)] On iOS 14+, local network broadcast in apps need to
be declared in the app's Info.plist. Debug and profile Flutter apps and modules host VM services on the local
network to support debugging features such as hot reload and DevTools. To make your Flutter app or module
attachable and debuggable, add a '_dartobservatory._tcp' value to the 'NSBonjourServices' key in your
Info.plist for the Debug/Profile configurations. For more information, see
https://flutter.dev/docs/development/add-to-app/ios/project-setup#local-network-privacy-permissions
[   +1 ms] Observatory URL on device: http://<IP_ADDRESS>:60045/
[   +8 ms] Attempting to forward device port 60045 to host port 62185
[        ] executing: /Users/fudong/fvm/versions/3.3.0/bin/cache/artifacts/usbmuxd/iproxy 62185:60045 --udid
1f3c8472cde01fe391647650af39cf2268bf37d5 --debug
[+1011 ms] Forwarded port ForwardedPort HOST:62185 to DEVICE:60045
[        ] Forwarded host port 62185 to device port 60045 for Observatory
[   +3 ms] Installing and launching... (completed in 16.8s)
[   +2 ms] Caching compiled dill
[  +76 ms] Connecting to service protocol: http://<IP_ADDRESS>:62185/
[ +214 ms] Launching a Dart Developer Service (DDS) instance at http://<IP_ADDRESS>:0, connecting to VM service
at http://<IP_ADDRESS>:62185/.
[ +107 ms] DDS is listening at http://<IP_ADDRESS>:62193/GtVvHuBLxVM=/.
[   +1 ms] [VERBOSE-2:FlutterObservatoryPublisher.mm(101)] Failed to register observatory port with mDNS with
error -65555.
[        ] [VERBOSE-2:FlutterObservatoryPublisher.mm(103)] On iOS 14+, local network broadcast in apps need to
be declared in the app's Info.plist. Debug and profile Flutter apps and modules host VM services on the local
network to support debugging features such as hot reload and DevTools. To make your Flutter app or module
attachable and debuggable, add a '_dartobservatory._tcp' value to the 'NSBonjourServices' key in your
Info.plist for the Debug/Profile configurations. For more information, see
https://flutter.dev/docs/development/add-to-app/ios/project-setup#local-network-privacy-permissions
[  +60 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:62185/
[  +28 ms] DevFS: Creating new filesystem on the device (null)
[  +21 ms] DevFS: Created new filesystem on the device
(file:///private/var/mobile/Containers/Data/Application/FE59A625-07BE-4907-82D2-BCB4C3F2B00C/tmp/flutter_appcL
fQ9l/flutter_app/)
[   +2 ms] Updating assets
[ +236 ms] Syncing files to device Fu iPhone...
[   +1 ms] Compiling dart to kernel with 0 updated files
[        ] Processing bundle.
[   +1 ms] <- recompile package:flutter_app/main.dart a4c4349d-2db2-4d1d-9238-4a14ce9b6d6e
[        ] <- a4c4349d-2db2-4d1d-9238-4a14ce9b6d6e
[   +2 ms] Bundle processing done.
[ +153 ms] Updating files.
[        ] DevFS: Sync finished
[   +1 ms] Syncing files to device Fu iPhone... (completed in 159ms)
[        ] Synced 0.0MB.
[   +1 ms] <- accept
[   +3 ms] Connected to _flutterView/0x11883b020.
[   +3 ms] Flutter run key commands.
[   +1 ms] r Hot reload. 🔥🔥🔥
[        ] R Hot restart.
[        ] h List all available interactive commands.
[        ] d Detach (terminate "flutter run" but leave application running).
[        ] c Clear the screen
[        ] q Quit (terminate the application on the device).
[        ] Running with unsound null safety
[        ] For more information see https://dart.dev/null-safety/unsound-null-safety
[        ] An Observatory debugger and profiler on Fu iPhone is available at:
http://<IP_ADDRESS>:62193/GtVvHuBLxVM=/
[        ] The Flutter DevTools debugger and profiler on Fu iPhone is available at:
           http://<IP_ADDRESS>:9103?uri=http://<IP_ADDRESS>:62193/GtVvHuBLxVM=/
[+8776 ms] flutter: 请求失败：HandshakeException: Handshake error in client (OS Error:
[        ] 	UNSUPPORTED_PROTOCOL(handshake_client.cc:697))

Any news?
I am having the same issue, getting websocket closed when sending data to wss websocket

The host used in the example (api-dev.yeedev.com) does not resolve for me. Do we have another example that we can use?


py_what_backend and py_repl_backend share a lot of duplicate code

Most of this duplicate code is to do with the reaction handles and eval exception handling, which I plan to deal with for 0.1.0. Leaving this issue open until then.

If you can provide suggestions or ways you'd like to do it, let me know, and I'll file a PR.


docs: Create README.md and add goreportcard badge to it

Create README.md and add goreportcard badge to it
Fixes https://github.com/drpaneas/romie/issues/4 and https://github.com/drpaneas/romie/issues/27

Whoever 2 people have some time to review, please do. Let's talk about the description and agree on a reasonable one. Don't forget that more specific links and description will be added/updated when we add go code


Change selection of inline picker. fix #927.

Also remove semicolon by eslint checking.
fixes #927

Codecov Report

Merging #960 into master will decrease coverage by 0.42%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
- Coverage   85.55%   85.12%   -0.43%
==========================================
  Files          12       12
  Lines         630      632       +2
  Branches      122      124       +2
==========================================
- Hits          539      538       -1
- Misses          6        7       +1
- Partials       85       87       +2

Impacted Files
Coverage Δ

src/datepicker.jsx
89.82% <85.71%> (-1.7%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi @martijnrusschen!
I kindly ask you to check this PR as soon as you can and release it because my project is very depending now on this feature.
I hope I've done check and updating state in proper place and it not - I hope you get the idea and can do it better.
Anyway thank you a lot for this awesome and very flexible picker.

Any chance we can add a regression test to avoid breaking this again?

I think this definitely needs a test, @m1n0s if you need to use this immediately you can build your forked repo and use that in your project.

@martijnrusschen @rafeememon
I've added test case for my changes. Please review.
Leo


Export single Event

Hey.. since it is possible to grab parsed informations of a single event out of an ICS-File with multiple events:
Maybe it is possible or easy to implement exporting single event as ICS file?
thx

do not know what do you mean
You can get list of events like this:
$cal = new IcalParser();
$results = $cal->parseFile('file.ics');

foreach ($cal->getSortedEvents() as $r) {
	echo sprintf('	<li>%s - %s</li>' . PHP_EOL, $r['DTSTART']->format('j.n.Y'), $r['SUMMARY']);
}

My fault, if i was not clear enough, sorry.
with your great parser, it is possible to parse a complete ICS file, regardles if there are more than one events inside.
but we also need single-event as ICS File out of an multiple-event-ICS-file. Something like splitting the ICS file into ICS-Files per event.
the basic idea/thought is: after parsing a single event out of a multiple-event-ICS-file, maybe it is possible to generate a new ICS File of this single event?

You can parse one file by one, that´s best solution

2020 v 11:05, demlak<EMAIL_ADDRESS>

My fault, if i was not clear enough, sorry.
with your great parser, it is possible to parse a complete ICS file, regardles if there are more than one events inside.
but we also need single-event as ICS File out of an multiple-event-ICS-file. Something like splitting the ICS file into ICS-Files per event.
the basic idea/thought is: after parsing a single event out of a multiple-event-ICS-file, maybe it is possible to generate a new ICS File of this single event?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

i'm sorry.. still thats not what i was trying to explain.
Works: single ICS File with multiple events -> parsing each event by iterating through complete ICS file
Needed: single event -> generating ICS file
so.. AFTER parsing multiple events, i want to generate from this output a NEW ICS-file for each event
(each event in a seperated ICS file)
something like this.. maybe i will use this script.. https://gist.github.com/jakebellacera/635416

yes, icalparser it's just parser


SIAlertview crashed in iOS9

SIAlertview crashed in iOS9

I have a same problem.
Your must add ' si_alert_background_window.rootViewController = [UIViewController new];' in showBackground function.

Also SIAlerview blocked the screen when clicked cancel button


Automated Package Upgrade

:tada: This PR is included in version 5.3.71 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Build from source isaac_ros_dope errors

Hi,
I am getting a couple of errors that weren't appearing previously -
some packages appear to already be in the core ROS workspace now.
also there is an error associated with git.
I've just done a fresh install of docker and image build and then using (https://nvidia-isaac-ros.github.io/repositories_and_packages/isaac_ros_pose_estimation/isaac_ros_dope/index.html) with build from source (on x86_64).
It is okay to ignore these errors for now?
Many thanks
cd ${ISAAC_ROS_WS} && \
   colcon build --symlink-install --packages-up-to isaac_ros_dope
[0.232s] WARNING:colcon.colcon_core.package_selection:Some selected packages are already built in one or more underlay workspaces:
        'isaac_ros_tensor_list_interfaces' is in: /opt/ros/humble
        'isaac_ros_test' is in: /opt/ros/humble
        'isaac_ros_common' is in: /opt/ros/humble
If a package in a merged underlay workspace is overridden and it installs headers, then all packages in the overlay must sort their include directories by workspace order. Failure to do so may result in build failures or undefined behavior at run time.
If the overridden package is used by another package in any underlay, then the overriding package in the overlay must be API and ABI compatible or undefined behavior at run time may occur.

If you understand the risks and want to override a package anyways, add the following to the command line:
        --allow-overriding isaac_ros_common isaac_ros_tensor_list_interfaces isaac_ros_test

This may be promoted to an error in a future release of colcon-override-check.
Starting >>> isaac_ros_common
Starting >>> isaac_ros_tensor_list_interfaces
Starting >>> isaac_ros_test
--- stderr: isaac_ros_test
ERROR setuptools_scm._file_finders.git listing git files failed - pretending there aren't any
---
Finished <<< isaac_ros_test [0.98s]
Finished <<< isaac_ros_common [3.13s]
Starting >>> gxf_isaac_dope
--- stderr: isaac_ros_tensor_list_interfaces
ERROR setuptools_scm._file_finders.git listing git files failed - pretending there aren't any
---
Finished <<< isaac_ros_tensor_list_interfaces [3.29s]
Finished <<< gxf_isaac_dope [4.95s]
Starting >>> isaac_ros_dope
--- stderr: isaac_ros_dope
ERROR setuptools_scm._file_finders.git listing git files failed - pretending there aren't any
---
Finished <<< isaac_ros_dope [17.9s]

Summary: 5 packages finished [26.2s]
  3 packages had stderr output: isaac_ros_dope isaac_ros_tensor_list_interfaces isaac_ros_test

follow the below steps to overcome this error:

pip3 uninstall setuptools-scm
pip3 install setuptools==58.2.0
after this build the model again and you might not face issues

Thanks, I'll give that a go


Add National College of Computer Science "Tudor Vianu"

fixed the pull request, thanks

@mat123abc First stage of verification passed, official domain is portal.lbi.ro as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: http://portal.lbi.ro/

@mat123abc Pull request merged. Thank you!


Unable to delete any worlds.

I am currently running spigot-BT45_1.8.8_10.21 and Multiverse Build # 710 from here: http://ci.onarandombox.com/job/Multiverse-Core/
MV Config: http://hastebin.com/pocanacija.vala
Worlds.yml: http://hastebin.com/urivawahig.vhdl
I recently noticed I had a few worlds built up, and decided to delete all of the unnecessary worlds that I had. The worlds I wanted to remove were titled: Road, plotme, plotmenewcurrent, Survival, Survival_the_End, and Survival_nether (It should be be noted that Survival was set to Default world since the inception of the server but more on that later).
When trying to delete a world, for example plotme, I would run the command /mv delete plotme, then /mv confirm. When trying to delete it would say world could NOT be deleted!
I then stopped the server, deleted all the worlds aside from the Survival ones, then removed the worlds from the worlds.yml block. When I started the server, the folders and world.yml regenerated.
 All worlds except for Survival, Survival_the_end, and Survival_nether remained. I tried to make the changes in my backup worlds.yml file, re-upload it then start, the worlds came back after starting the server.
Also, in regards to the default worlds not deleting. I went to my server.properties file then changed level-name=Survival to level-name=Spawn. However, upon starting the server it changes back to Survival. I even download the server.properties file right after I make the change to make sure it's properly saving.
Hope something can be done about this, thanks!

Hey Eric :)

If you delete the world folder and remote that world entirely from the worlds.yml file there is no reason Multiverse would load the world. It is impossible it would even know the world exists. It is possible, however, that some other plugin is creating the worlds and having multiverse load them.

I'm going to close this one because it sounds like the issue is resolved (via update) but also that I'm not really sure what's going on.
If you can come up with a well defined test case and set of steps, we can reopen and look at it but it may even be related to #1663 and thus not really an issue, but a communication problem.


Fix -Wunused-variable in GroupwiseRegistration.cpp

/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: In member function ‘void GroupwiseRegistration::initProperties(int, const char**, int)’:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:572:6: warning: unused variable ‘nFace’ [-Wunused-variable]
  int nFace = m_spharm[subj].sphere->nFace();
      ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: In member function ‘void GroupwiseRegistration::updateDeformation(int)’:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:953:20: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (int i = 0; i < m_spharm[subject].vertex.size() && !updated; i++)
                    ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:957:16: warning: unused variable ‘v0’ [-Wunused-variable]
   const float *v0 = v->fv();
                ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: In member function ‘void GroupwiseRegistration::updateLandmark()’:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:977:8: warning: unused variable ‘id’ [-Wunused-variable]
    int id = m_spharm[subj].landmark[i]->id;
        ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: In member function ‘void GroupwiseRegistration::updateLandmarkMedian()’:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:1019:8: warning: unused variable ‘id’ [-Wunused-variable]
    int id = m_spharm[subj].landmark[i]->id;
        ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: At global scope:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:1165:42: warning: unused parameter ‘coeff’ [-Wunused-parameter]
 float GroupwiseRegistration::cost(float *coeff, int statusStep)
                                          ^
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp: In member function ‘void GroupwiseRegistration::optimization()’:
/tmp/GROUPS/Modules/CLI/Groups/GroupwiseRegistration.cpp:1229:6: warning: unused variable ‘n2’ [-Wunused-variable]
  int n2 = m_csize * 2 - n1;
      ^

Fixed in https://github.com/NIRALUser/GROUPS/commit/2328d6ac89b3cd008aae86b5c2c6ad1a51a9f158


Improved Exception Handling

Description
Improved JSON exception handling - Expanded on PR #356 (Had to make a new PR for some reason)
Testing

If the PR is testable, please describe how to test it here

Outstanding Work

[ ] If there is work still outstanding, please include it here
[ ] if there is nothing outstanding, remove this section.

Notes

Include any extra notes here

Pre Pull Request Checklist:

[ ] Code has been scanned with SonarLint
[ ] Methods have been annotated with @ Nullable and @ Nonnull (Read more)
[ ] There are no errors present in the project
[ ] Code has been formatted and indented
[ ] Methods have appropriate Javadoc (How to write Javadoc)

Can one of the admins please verify this patch?


Header onLayout not execute setState

I have gone through these following points

[x]   Check latest documentation: https://docs.nativebase.io/
[x]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x]   Check examples from NativeBase KitchenSink

Issue Description
If I using a Header component and like to fetch the current height by onLayout the process is working well, apart that my this.setState is not transmitting the values to the new cycle. it seems that like setState is not executed. If I wrap around my Header with a View, this  work neither. But when I using the react-native View component and not this one from native-base import, all was working fine.
For this moment this is my workaround, but in the future, it will be nice when you can fixing this.
To catch the height variable is not an option, because the header has a variable height and I cannot use the default device config values.
In the previous version<EMAIL_ADDRESS>this was working. It seems that the problem was a bit more inside native-base because also other components like View has this problem.
node, npm, react-native, react and native-base version, expo version if used, xcode version
node - v10.11.0
npm - v6.10.1
react-native - v0.59.8
react - v16.8.3
native-base - v2.12.2
Expected behaviour
setState is transmitting the current values
Actual behaviour
height values are correct but setState is not executing
Steps to reproduce
<Header style={styles.header} onLayout={this.updateContentHeight}>         <Left><Text>Back</Text></Left>         <Body>           <Title>Title of the header</Title>         </Body>         <Right><Text>Save</Text></Right>       </Header>
  updateContentHeight = ({nativeEvent}) => {     const { layout } = nativeEvent;     const { height } = dimension;     this.setState({contentHeight: height - layout.height});   };

Hi @kitfall , please provide minimal code production for the same on snack for our easy understanding.
Thanks

Hi @kitfall , closing for now due to inactivity. Let us know if the issue still persists on latest Native-Base v2.13.14


Removed quotes around author credit in minified file

@jameshibbard the minified file is generate by "Makefile" located at project directory. I haven't inspected the reason behind the PR, but in any case the edit should be done at "Makefile".

I'm not sure it's anything to do with the Makefile — that seems to be correct.
The problem is that there are two quotes in awesomeplete.min.js around the author credit.
If you clone the repo and run make then these disappear.  However, their presence breaks the library for people including it from the repo directly (e.g. via rawgit). Wasn't sue of the best way to bring this to someone's attention.

I verified that when I run the make file, the quotes get removed. you are correct on that.
By inspecting the logs, these quotes were added in commit 6611606 by @marc-costello   .
Since that commit had the same version of the make file made by @vlazar , it's fair to assume that the make files behaves differeantly with these quotes on different systems.
This PR is a good hotfix, but this issue may reappear if isn't treated at the makefile.
@vlazar , please advise.

@pazams I think you are right about Makefile being the issue when it was used by @marc-costello to minify. Removing double quotes from banner in https://github.com/LeaVerou/awesomplete/blob/gh-pages/Makefile#L3 might resolve the issue. Unless spaces won't add another one :)
@marc-costello Can you please check, if suggested change fixes the issue on your system?
@LeaVerou Might be a good time to replace Makefile with node (npm scripts, gulp, ...), since it needs to be installed to run the tests anyway. Also, the development process will be simpler and more cross platform.
The Makefile was introduced before npm and karma test runner were merged. Time to retire Makefile?

Merged as it fixes the issue for now.
I’m all in for retiring makefile, as long as someone sends a PR. Ideally whatever replaces it should not require too much setup (node is ok as a dependency, no other dependencies plz), to facilitate contributions.

I can do it, since I created Makefile.
The only required setup for contributors in any case would be running npm install command.


How to use routes? In Android for example.

Issue description
I think this is third time I am setting up Headscale. Each previous attempts I gave up as I was unable to configure it the same way as with Wireguard directly. I have three networks at home. All three reachable on docker server. <IP_ADDRESS>/24, <IP_ADDRESS>/24 and <IP_ADDRESS>/24. My goal is to reach all three when I connect to Headscale server with any client. Client should be Windows or Android.
To Reproduce
I started the server using Docker. Then created namespace and connected two clients to it. One is another linux server, other is my Android phone. For linux server I added "--advertise-routes=<IP_ADDRESS>/24,<IP_ADDRESS>/24,<IP_ADDRESS>/24" parameter to tailscale command. Then I enabled all three routes on Headscale and this seems to work.
But for Android client I just can not find an option to add this routes client side. So when I am connected I for example can not ping <IP_ADDRESS>/24 devices.
Can I add them on Headscale server side?
Context info
Headscale version: 0.17.0-alpha2
Android client: 1.30.1
OS: Linux Ubuntu
root@docker-server:~# docker exec headscale headscale nodes list
ID | Hostname      | Name          | NodeKey | Namespace | IP addresses                    | Ephemeral | Last seen           | Online  | Expired
1  | Mi 10T Pro    | mi-10t-pro    | [4LfgH] | clients   | <IP_ADDRESS>, fd7a:115c:a1e0::1   | false     | 2022-09-14 10:39:52 | offline | no
2  | web-wg-server | web-wg-server | [goF/h] | clients   | <IP_ADDRESS>, fd7a:115c:a1e0::2 | false     | 2022-09-14 10:01:20 | offline | no
root@docker-server:~# docker exec headscale headscale machines routes list -i 1
Route | Enabled
root@docker-server:~# docker exec headscale headscale machines routes list -i 2
Route           | Enabled
<IP_ADDRESS>/24   | true
<IP_ADDRESS>/24   | true
<IP_ADDRESS>/24 | true

Can you please raise this in Discord?


quota set by the plugin cannot block Metalnx user uploading data

We're running iRODS 4.2.10 on CentOS 7.
Metalnx version is 2.4.0
Quotas set by the logical quotas for a collection can be over exceeded by using Metalnx. In other saying, the data upload process through Metalnx cannot be blocked by the logical quotas.
Whereas the maximum_number_of_data_objects operation can block exceeds, the logical_quotas::maximum_size_in_bytes operation cannot so. And this issue occurs for rodsusers.
AVUs defined for collection /u0137480/home/lt1_d_pilot_003:
attribute: irods::logical_quotas::maximum_size_in_bytes
value: 200
units:
----
attribute: irods::logical_quotas::total_number_of_data_objects
value: 10
units:
----
attribute: irods::logical_quotas::total_size_in_bytes
value: 159345
units:

If Metalnx is streaming (open, write, close) the bytes iRODS, then this behavior is explained.
See https://github.com/irods/irods_rule_engine_plugin_logical_quotas#stream-operations
My guess is that Metalnx is using the streaming operations.

Regarding with the streaming operation, If I am interpreting the quoted part below correctly :

These changes have the following effects:
The plugin allows stream-based writes to violate the maximum bytes quota once.
Subsequent stream-based creates and writes will be denied until the quotas are out of violation.

the following points are correct, right?

The first streaming based upload can exceed the quotas set even if there is still only one byte exists before being over quoted.
From the second upload on, the streaming based uploads should be rejected because the first streaming based upload already exceeded the quotas set.

If these are correct, then there seems the second point still not working properly. Because via Metalnx, we can continuously upload data.
Also, I couldnt test this for other clients using Jargon api because of the limit of our dev environment. So might it be related to the api that Metalnx use?

That is surprising. I'll take a look.

Looking at this with fresh eyes again, I want to confirm we're interpreting the documentation about streaming in the same way.
What that documentation is actually saying is ... a violation for streaming operations is only considered during a call to open. That means, when the REP sees the open PEPs fire, it will check to see if there are any violations. If there are violations, the REP blocks the operation. If there aren't any violations at the time of the open, the open succeeds and any number of write operations can be executed (which can result in violations).
@mstfdkmn Does that align with your understanding/expectations?
I've confirmed that Metalnx 2.6.1 honors those rules when run against iRODS 4.2.12 and what will be iRODS 4.3.2.
@mstfdkmn Are you still seeing an issue? If not, then I'm going to close this.

I guess there were different things in this thread. Apart from how to interpret the documentation, the other thing was about
what discussed in the following thread https://github.com/irods/python-irodsclient/issues/379#issuecomment-1344649766. Please see that also Daniel created another issue while debugging this.
Anyway, we are no longer using the Metalnx client nor the logical quota. So I am not be able to easily test this again. Sorry! And yes this can be closed on my end.

Please see that also Daniel created another issue while debugging this.

Dan created https://github.com/irods/irods_rule_engine_plugin_logical_quotas/issues/84 and it was fixed/merged in March 2023 - and then included in the <IP_ADDRESS> and <IP_ADDRESS> releases of this plugin.

All good.
I think things are working. Closing.


[CMake] Optionally support building TF as a shared lib

Currently, TF is already built as a shared library that is included in the Python package. However, that library:

Is implicitly linked against Python libs, and thus expects Python to be present on the machine wherever it is used. It is undesirable for scenarios not requiring Python (e.g. native application that need TensorFlow).
Does not include the TF C++ API.

This PR allows optionally building TF as a stand-alone DLL that does not have the above issues. I am also working on allowing CMake to link all C++ tests against that DLL, and will submit such changes in a separate PR.
As a bonus, this PR also fixes a build break for tf_tools.cmake when GPU is enabled.

Can one of the admins verify this patch?

It seems like you save my life ;) Thank you!

@vit-stepanovs I should run cmake with -DBUILD_SHARED_LIB=ON but then what build target do I run?

@tomjaguarpw, the actual option name is tensorflow_BUILD_SHARED_LIB, not BUILD_SHARED_LIB. If you use that, cmake generates a project tensorflow.vcxproj (not tensorflow.sln!) which you can use to build the DLL.

Nice!
LGTM, @mrry what do you think?

Jenkins, test this please.

Thanks!

It works great on both Linux and Windows for me. Why is this PR removed from release 1.1.0? (it was there in 1.1.0-rc2)

It's probably because it didn't make it before the cutoff point when we forked the branch.
/CC: @av8ramit

I don't believe this was in 1.1.0-rc2, but yes we decided to push it back to 1.2 since it was merged too late for 1.1 to test properly.

Is there some way to build tensorflow as static lib?

I tried this in conjunction with https://github.com/tensorflow/tensorflow/pull/9666 to try to build a debug DLL,, but the generated .def file had over 120K symbols, so it exceeded the 65535 limit.  Any way to filter more aggressively?

@girving another reason to restrict symbols #9525
On May 12, 2017 1:24 PM, "Andy Dennie"<EMAIL_ADDRESS>wrote:
I tried this in conjunction with #9666
https://github.com/tensorflow/tensorflow/pull/9666 to try to build a
debug DLL,, but the generated .def file had over 120K symbols, so it
exceeded the 65535 limit. Any way to filter more aggressively?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/9124#issuecomment-301176827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_SbaXsy-KifiQ0902rs8mKlt-4mHgSks5r5MAKgaJpZM4M5rj-
.

Nice.  Is there a thread I should notify once we make progress here?  This one seems merged.


fix: combine aliases on windows base dirs (ie: X:\) (fixes: #577)

Illustrating a valid fix for my issue, that is still happening ;-)
I had a (very quick) look at running the tests, but unfortunately I had a lot of tests failing on the pristine source prior to any change. So I figured I missed a lot of dependencies or information to get this running properly. I'm afraid that I couldn't check that I didn't break anything properly before submitting that code change.
If you think this code change is definitively worth it, I'll take the time to add tests to your convenience if required.

Codecov Report

Merging #1080 (e0198af) into master (37285cb) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1080      +/-   ##
==========================================
- Coverage   91.30%   91.27%   -0.03%
==========================================
  Files          90       90
  Lines       13358    13361       +3
  Branches     1493     1494       +1
==========================================
- Hits        12196    12195       -1
- Misses        951      954       +3
- Partials      211      212       +1

Impacted Files
Coverage Δ

coverage/files.py
84.11% <100.00%> (ø)

coverage/misc.py
90.17% <100.00%> (+0.17%)
:arrow_up:

tests/test_oddball.py
81.54% <0.00%> (-1.20%)
:arrow_down:

coverage/debug.py
58.51% <0.00%> (-0.88%)
:arrow_down:

Codecov Report

Merging #1080 (8f4e695) into master (37285cb) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1080      +/-   ##
==========================================
- Coverage   91.30%   91.24%   -0.06%
==========================================
  Files          90       90
  Lines       13358    13358
  Branches     1493     1493
==========================================
- Hits        12196    12188       -8
- Misses        951      959       +8
  Partials      211      211

Impacted Files
Coverage Δ

coverage/files.py
84.11% <100.00%> (ø)

tests/test_debug.py
90.82% <0.00%> (-7.34%)
:arrow_down:

@vaab Thanks for sticking with this!  I took a quick look at the change, and it seems straightforward enough.  The tests are passing here on GitHub, so it seems good.  If you could add a test of the specific problem, that would be great!

Codecov Report

Merging #1080 (8f4e695) into master (37285cb) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1080      +/-   ##
==========================================
- Coverage   91.30%   91.24%   -0.06%
==========================================
  Files          90       90
  Lines       13358    13358
  Branches     1493     1493
==========================================
- Hits        12196    12188       -8
- Misses        951      959       +8
  Partials      211      211

Impacted Files
Coverage Δ

coverage/files.py
84.11% <100.00%> (ø)

tests/test_debug.py
90.82% <0.00%> (-7.34%)
:arrow_down:

Added the test as required, kept it quite simple. Thanks for your feedback.

@vaab no worries about the codecov drop.  Ironically, I can't make heads or tails of those reports, and have turned them off.
Thanks!

@vaab no worries about the codecov drop.  Ironically, I can't make heads or tails of those reports, and have turned them off.
Thanks!

This is now released as part of coverage 5.4.

This is now released as part of coverage 5.4.


Constraining Sticker to image boundaries

Is there a way to constrain user in moving a sticker, so he couldn't be able to move it outside of the parent image view?
I am grateful for any help and advice.
Keep up the good work. :)

Currently,the sticker can move in any position in the view.If the
StickerView'width and height equal to ImageView' width and height.The
constrain of moving is image boundaries.
On Dec 6, 2016 3:48 AM, "Milos Smiljanic"<EMAIL_ADDRESS>wrote:

Is there a way to constrain user in moving a sticker, so he couldn't be
able to move it outside of the parent image view?
I am grateful for any help and advice.
Keep up the good work. :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/9, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_wlJjgYN1UIraZfF20tDU2RWZmzFxks5rFGp-gaJpZM4LEosH
.

there is one bug in the stickerView if we load sticker on button click then it is not showing on the stickerview. It is showing after touch the StickerView Please help me to get rid from this problem

ok,i will check and fix it. It will be quick.
On Dec 6, 2016 1:22 PM, "AnuragKataria"<EMAIL_ADDRESS>wrote:

there is one bug in the stickerView if we load sticker on button click
then it is not showing on the stickerview. It is showing after touch the
StickerView Please help me to get rid from this problem
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/9#issuecomment-265063703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_woWdOovoKFhngrxURFg1VbO7Arh6ks5rFPElgaJpZM4LEosH
.

I update the code.You can see it in GitHub.In my device.The replace func
works well.
On Dec 6, 2016 1:24 PM, "吴盼杰"<EMAIL_ADDRESS>wrote:

ok,i will check and fix it. It will be quick.
On Dec 6, 2016 1:22 PM, "AnuragKataria"<EMAIL_ADDRESS>wrote:

there is one bug in the stickerView if we load sticker on button click
then it is not showing on the stickerview. It is showing after touch the
StickerView Please help me to get rid from this problem
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/9#issuecomment-265063703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_woWdOovoKFhngrxURFg1VbO7Arh6ks5rFPElgaJpZM4LEosH
.

Cool.
I noticed one thing in this new version - when I add new sticker on some event, it's not drawn until StickerView is clicked. I believe invalidate() is missing at the end of addSticker method.
And about the issue from above, I would like to somehow restrict user from hiding the sticker on a image, or at least make it to be 50% visible. For example, that you can't move the center of the sticker across the image boundaries.

Oh,sorry.It's my negligence.And I understand your demand.But in my
area.It's too late at night.So I will fix it and add an alternative method
to constrain the sticker move area.
On Dec 7, 2016 1:43 AM, "Milos Smiljanic"<EMAIL_ADDRESS>wrote:

Cool.
I noticed one thing in this new version - when I add new sticker on some
event, it's not drawn until StickerView is clicked. I believe invalidate()
is missing at the end of addSticker method.
And about the issue from above, I would like to somehow restrict user from
hiding the sticker on a image, or at least make it to be 50% visible. For
example, that you can't move the center of the sticker across the image
boundaries.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/9#issuecomment-265218581,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_wq7xE2MTa72Fxrujo1HmFjaurwJCks5rFZ6qgaJpZM4LEosH
.

Thanks a ton! Really appreciate it.

您好，如何将StickerView兼容photoview下进行标注？

不好意思，目前还无法和PhotoView愉快的玩耍，要研究一下，Sorry
2016-12-12 18:48 GMT+08:00 luo15140128<EMAIL_ADDRESS>
您好，如何将StickerView兼容photoview下进行标注？
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/9#issuecomment-266397636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_wjs0lXO_SViHkdoMx1h7O6_AkaK7ks5rHSaHgaJpZM4LEosH
.

可以研究下下喔 呵呵

刚刚看了下photoview上添加水印标注，有如下问题： （1）图片水印移动的时候部分超出照片范围外，（2）放大缩小图片的时候标注未跟着移动。  最大的难点是如何标注之后图片放大缩小，标注也跟着移动变化呢

恩，我再研究研


Add time sensitive period splitting

@ivarsb can you fix travis? a) downgrade bundler or b) upgrade bundler in travis yml

@ivarsb looks like we need to update bundler in our gemspec ~>1.9 to >=1.9?


Build failed: ##[error]PowerShell exited with code '1'.

Build: https://dev.azure.com/monacotools/a6d41577-0fa3-498e-af22-257312ff0545/_build/results?buildId=284627
Changes: https://github.com/Microsoft/vscode/compare/89c068f...50c5521

Merging to https://github.com/microsoft/vscode/issues/192119

Issue still happens in recent builds. For example in https://dev.azure.com/monacotools/Monaco/_build/results?buildId=284827&view=results with error:
Extension host test runner exit code: 0
Asking native host service to exit with code 0.
[main 2024-07-24T04:22:49.126Z] Extension host with pid 7336 exited with code: 0, signal: unknown.
Exit code:<PHONE_NUMBER>
Failed
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Exec:
At D:\a\_work\1\s\build\azure-pipelines\win32\exec.ps1:22 char:3
+         throw ("Exec: " + $errorMessage)
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Exec: :String) [], RuntimeException
    + FullyQualifiedErrorId : Exec:

Thanks @aiday-mar, issue is tracked in https://github.com/microsoft/vscode/issues/192119


MEDIA_ROOT Retrieval

Branch: fileformatting
To Reproduce: try to upload a video, save it, and click new node to preview
Description: content storage system currently saves content to base directory. This should be changed

Has to do with django server file size limit

this seems a browser quirk. I can duplicate the issue with Chrome, but there is no issue on FireFox.
Maybe let's revisit this issue after kolibri gets its production server running.

Could you clarify what's going on here, and what it has to do with file size limits? Also, I can't think of how this could be influenced by browser type...

here's the page mentioned the issue https://code.djangoproject.com/ticket/4444
"According to many sources the 'Broken Pipe' is a normal browser quirk. For example, the browser reads from the socket and then decides that the image it's been reading apparently didn't change. The browser now this (forcefully) closes the connection because it does not need more data. The other end of this socket (the python runserver) now raises a socket exception telling the program that the client 'Broke the socket pipe'."

I still don't understand the relationship between where the files are being saved and a "broken pipe" error. As this comment notes, the "broken pipe" error isn't really an error, but happens "whenever your browser closes the connection while the dev server is still busy sending data. The best we could is to have a more explicit error message."
Let's sit down and take a look at this tomorrow.

Problem with development server, might not be a problem when deployed


Push Notifications documentation enhancement

Removing myself from the reviewers list on this PR as I can see that @marklewin has already provided comprehensive input. @ben-xD please feel free to land this PR once:

You've addressed Mark's comments
Mark's resolved those conversations to acknowledge that
Mark's approved the PR


iOS: Plays file from playlist but without a sound

Cannot get the plugin to work on a mobile device, the playlist appears to fetch the file and play but produces no sound. Please see for yourself:
https://soundprogr.am/sample-playlist/
Any ideas at all? The files are hosted and served via the DigitalOcean Spaces CDN.
Test with version<EMAIL_ADDRESS>and 4.1.0
Let me know if anyone has any leads, other than that absolutely love this library.
Thanks,
Art

Believe this to be more of an issue with the library used:
https://github.com/521dimensions/amplitudejs


legends and scales

(rebased #432)
build at https://observablehq.com/d/c324a9a466e66329
closes #23
closes #236
related: #397 #92 #425

Is there a way to configure the positioning of the legend with respect to the plot, e.g. above or below?

No, there is no such thing for the moment. We'll probably want some configurability but we don't want to invent a new layout engine :) What you can already do, is to create the legend separately (as shown in a few of the examples). Alternatively, move it inside the figure element with d3.select?.

I wanted to work on landing exposing scales first before tackling legends. Ref. https://github.com/observablehq/plot/pull/425#issuecomment-892270954

In the two other branches we had accumulated quite a bit of conflict with main, and I wanted to clear that. But feel free to remove anything from this PR (I think the legend part is well separated).
About the issue of exposing Plot.scale(options) or not, I think that there will be a way around in any case if we expose the d3 scale: something like Plot.plot({ color: options }).scales.color.scale.

My preference for scales would be to just expose the materialized options (type, domain, range, etc.) with the goal of being able to take a scale constructed from one chart and propagate it to another chart; I would prefer not to expose a D3 scale instance or a scale function. We removed most of the need for that in #280 by precomputing the scaled channels, though scales still have access to the scale functions… which is a little messy. In my ideal world the implementation of scales would be hidden so that we have more flexibility in changing the implementation.

I'm good with this plan (including not exposing the d3 scale); it still gives enough information to re-create the scale outside of Plot if necessary.

Superseded by #539.


Screen reader announcement is bit different while interacting with signposts

Clarity Version: 3.1.1
WCAG 2.1 Guideline: 4.1.2 Name, role, value.
Test Environment:

Test URL: https://clarity.design/documentation/signposts
OS: Windows 10
User Agents/Assistive Technologies: Firefox 76.0
Screen reader: NVDA 2020.1

Steps to reproduce:

Open the test URL.
Move focus onto any of the signposts and try to interact with it - expand/collapse, move focus in and out, etc. Note: Use arrow key as well as tab navigation.

Expected result:

When screen reader user moves focus onto the signpost, appropriate label, control type and state should be annnounced.
When screen user moves focus in and out of expanded content, announcement should be standard or usual without any verbosity or additional content announcement.

Actual result:

Toggle word is announced along with expanded state which is bit verbose. The expanded state conveys the semantics for toggle, so the word "toggle" can be omitted from the label.
When focus is moved to the expanded content, the word "clickable" is read along with the heading which can be confusing. When user navigates away navigates backwards towards the expanded content, the word "clickable" is read along with close button. So it can be said that, when user enters the expanded content area, the word clickable is tagged to whatever element is focused.

Additional observation: No announcement is made after closing the SignPosts popup by pressing close button. If escape key is pressed instead of close button announcement is made as "Collapsed".


Run our custom eslint rules using ts-node

Use ts-node to run our custom eslint rules.  This lets us delete the pre-compiled js. It also means you can don't have to compile the rules while editing them
As part of this change, I've also switched us to using an eslint plugin instead of a rulesDir. This is now the preferred way to ship custom rules

LGTM but I believe there is specific build performance reason to have these files as JS in the repo. Adding @joaomoreno and @alexdima to comment on that

If performance is a concern, we could also try enabling ts-node's transpileOnly option or use swc to switch to using that compiler. This would skip the expensive type checking process

Yeah, it was a performance choice. So, seeing runtime numbers here would help.

@joaomoreno Some quick performance tests using time yarn eslint

Main: yarn run eslint  101.58s user 5.47s system 119% cpu 1:29.77 total

With TS Node: yarn run eslint  108.45s user 5.33s system 124% cpu 1:31.41 total

With TS Node (transpile only): yarn run eslint  107.12s user 5.03s system 123% cpu 1:30.46 total

With TS Node (swc): yarn run eslint  101.08s user 4.93s system 124% cpu 1:25.26 total

There may be some small performance gains from switching from eslint's rulesDir like we do on main, to using a plugin like this branch does

Yeah transpileOnly seems like the best balance here. But we'll still be type checking these scripts in some CI job, right?

@joaomoreno Type checking is done by the npmCheckJs script under build. However that is currently not run as part of CI. If we think running this makes sense, I'll take a look at enabling it in a separate PR


LazyCorpusLoader for non-nltk distributed corpora should be commented out.

c.f. #1718, #1716 and #1579

Residual issues with the Jenkins errors after the nltk tests seems to be problems upstream at Python  typing... =(
From https://nltk.ci.cloudbees.com/job/pull_request_tests/255/TOXENV=py35-jenkins,jdk=jdk8latestOnlineInstall/console :
Traceback (most recent call last):
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 718, in dispatch
    impl = dispatch_cache[cls]
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/weakref.py", line 352, in __getitem__
    return self.data[ref(key)]
KeyError: <weakref at 0x7fd27b659688; to 'EnumMeta' at 0x2a7b2b8 (Context)>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 721, in dispatch
    impl = registry[cls]
KeyError: <enum 'Context'>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/bin/pylint", line 11, in <module>
    sys.exit(run_pylint())
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/__init__.py", line 13, in run_pylint
    Run(sys.argv[1:])
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/lint.py", line 1220, in __init__
    linter.load_default_plugins()
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/lint.py", line 453, in load_default_plugins
    checkers.initialize(self)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/checkers/__init__.py", line 114, in initialize
    register_plugins(linter, __path__[0])
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/utils.py", line 992, in register_plugins
    module = modutils.load_module_from_file(join(directory, filename))
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/astroid/modutils.py", line 272, in load_module_from_file
    return load_module_from_modpath(modpath, path, use_sys)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/astroid/modutils.py", line 233, in load_module_from_modpath
    module = imp.load_module(curname, mp_file, mp_filename, mp_desc)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/imp.py", line 234, in load_module
    return load_source(name, filename, file)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/imp.py", line 172, in load_source
    module = _load(spec)
  File "<frozen importlib._bootstrap>", line 693, in _load
  File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 662, in exec_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/checkers/python3.py", line 100, in <module>
    class Python3Checker(checkers.BaseChecker):
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/checkers/python3.py", line 501, in Python3Checker
    'sys.version_info < (3, 0)',
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/pylint/checkers/python3.py", line 496, in <listcomp>
    [astroid.extract_node(x).repr_tree() for x in [
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/astroid/node_classes.py", line 624, in repr_tree
    _repr_tree(self, result, set())
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 743, in wrapper
    return dispatch(args[0].__class__)(*args, **kw)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/astroid/node_classes.py", line 613, in _repr_node
    depth)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 743, in wrapper
    return dispatch(args[0].__class__)(*args, **kw)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/site-packages/astroid/node_classes.py", line 613, in _repr_node
    depth)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 743, in wrapper
    return dispatch(args[0].__class__)(*args, **kw)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 723, in dispatch
    impl = _find_impl(cls, registry)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 674, in _find_impl
    mro = _compose_mro(cls, registry.keys())
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 662, in _compose_mro
    return _c3_mro(cls, abcs=mro)
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 616, in _c3_mro
    other_c3_mros = [_c3_mro(base, abcs=abcs) for base in other_bases]
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 616, in <listcomp>
    other_c3_mros = [_c3_mro(base, abcs=abcs) for base in other_bases]
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 620, in _c3_mro
    [explicit_bases] + [abstract_bases] + [other_bases]
  File "/scratch/jenkins/workspace/pull_request_tests/TOXENV/py35-jenkins/jdk/jdk8latestOnlineInstall/.tox/py35-jenkins/lib/python3.5/functools.py", line 571, in _c3_merge
    raise RuntimeError("Inconsistent hierarchy")
RuntimeError: Inconsistent hierarchy

Thanks @alvations. I hope this solves the problem!


Only delete known Maester tests with Update-MaesterTests

Checks the known Maester test names from the installed module path and only deletes those child object names instead of everything in $Path. Provides a resolution for #173.

This is good Sam. Thanks. Do you have thoughts on how to handle scenarios where we delete/retire tests?

Yes, I do have some early thoughts on how to handle removed or retired tests. I was also thinking of adding a test update process that can be separate from the module update process (an -Online parameter for updating tests).
Handling removed tests could be handled in a few ways:

Maintain an inventory of active tests, either manually as they change or via a script that checks test files/folders during a build step.
Leave the test files, but append ".Retired" (or similar) to the .PS1 and .MD file extension.
Add status metadata in each of the test files themselves, with logic in the Invoke and Update functions to skip/remove retired tests.
Move them to a "deprecated" folder and exclude that folder when invoking the tests.
Probably other possibilities as well.

I believe any of these ideas could benefit from an optional online update method that doesn't depend on the module being updated, but I'm sure there are factors that need to be considered.
When considering module download size, there will be benefits to simply removing retired tests from new releases.

I'm thinking we can add a hashtable to update-maester with a list of all .tests.ps1 files that we've removed. Then it can do a targeted delete of just those.

Would you generate the hash table in some automatic fashion by comparing builds or releases, or just add the information by hand when necessary?

I'm thinking we can add a hashtable to update-maester with a list of all .tests.ps1 files that we've removed. Then it can do a targeted delete of just those.

A thought: using a hash table to selectively remove retired tests will be a change to the current wipe/replace approach. If it works, would you like to test and merge this PR, and then follow-up with a separate PR to rethink how updates/changes to tests can be handled? This PR includes:

Error handling for directory creation and file copy operations.
Relative path expansion and validation.
Enhanced verbose output during the new/copy operations.
Checks to ensure that the only things that get deleted will be files and folders that match the names present in the module's installed maester-tests directory.
Replaced exit with return statements.
Simplification of the function and removal of a redundant line of code.
Added comment-based help.

OK, thanks!


Disable threading for point cloud client inside X-ray creation.

This just led to an unnecessary high amount of threads created. We now just do threading outside, so outer (X-ray) and inner (point cloud client) threads don't fight for computation time...

Thanks for your review!


change maxlength from 128 to 256

What is the purpose of the pull request
For #1437.
change maxlength from 128 to 256.
Brief change log

Edit localParams.vue

Verify this pull request

Manually verified the change by testing locally.

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@12fd103). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             dev   #1451   +/-   ##
=====================================
  Coverage       ?   0.17%
=====================================
  Files          ?     271
  Lines          ?   13564
  Branches       ?    2152
=====================================
  Hits           ?      24
  Misses         ?   13534
  Partials       ?       6

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 12fd103...e0f8e82. Read the comment docs.


Feature Request: Adding custom route option in the client GUI.

The user should have the option of adding some custom routes.
For eg. The user wants intranet ips to not use vpn.
Something like this
route ADD <IP_ADDRESS> MASK <IP_ADDRESS> <IP_ADDRESS> METRIC 25 IF 16

If you are using OpenVPN, this is easily done on the client side. Or are you using SoftEther own Protocol?

Close due to inactivity


Fix Nuget Readme rendering

Describe the bug
As seen on https://www.nuget.org/packages/NCronJob#readme-body-tab, the html trickery doesn't play well with nuger.org markdown rendering engine.

Yeah we could have a separate README.md just for nuget - not sure why <h1> and stuff doesn't work.
The img src should probably use the github.com/NcronJob-Dev/... URL instead of a relative one.

Basically we are blocked by: https://github.com/NuGet/NuGetGallery/issues/8644


merge video and audio on player

fluid player can merge video(webm or mkv or mp4 only video) and audio(m4a or mp3 only audio) on player?
i have video with 2 audio and i want can change audio
please answer , thank u .

Fluid Player (or any other HTML5 player) can not merge audio and video on the fly. There is a way to encode video with multiple audio tracks, however, Fluid Player has no support for this feature.

Marking the ticket as closed since fluid player doesn't support this feature


Observation: for..in slower than Object.keys

See https://jsperf.com/object-keys-vs-for-in-with-closure/3.
If the for...in loops enumerate lots of properties, using Object.keys gives a huge performance boost in most cases (at the expense of some extra memory).  If you're walking 4 items, perhaps no need to bother.  Wouldn't be job #1 until Moon is long-term feature complete anyway.

This is because of the hasOwnProperty check happening in the for..in test. Without this, they are almost equal (for..in is a little slower), but I think that's a good enough trade for the reduced memory usage.
Let me know what you think.

What is the actual memory impact?  My rule of thumb on JS is that, given that we already know performance is an issue, that temporary memory spikes aren't that big of a deal.  Since JS only ever gives us arrays of references  (which are either 32- or 64-bit) it's not that much overhead, and it won't hang around long before the GC gets to it.
Functional programming principles never seem to include worrying about intermediate storage overhead, so what the heck, right?

Yes, but for..in is already as fast as Object.keys, the jsperf test is not a good representation of this, as it has a check using hasOwnProperty. This check calls a function, increasing the time it takes for each iteration.
Without this check they are essentially the same, and Moon uses it without the check. I'm closing this for now as it doesn't effect performance benchmarks.


Version 1 of the task is not available

I tried today to move from version 0 to version 1 but I was getting the following error (which means the version is not available):
An error occurred while loading the YAML build pipeline: wrong number of segments

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

ID: b03d67d0-8738-72e6-dbe7-324c1784d4fc
Version Independent ID: bbb4d774-cd98-3ecb-7ccf-5f49e6f2b425
Content: Download Pipeline Artifact task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/download-pipeline-artifact.md
Product: devops
GitHub Login: @steved0x
Microsoft Alias: midenn

Hi @landrus , you shouldn't be getting an error when trying to use V1 of that task, I've reached out to some people to see what's up!

+1, I have exactly the same issue.

Thanks for reporting the issue. Can you provide an example of the YAML that you are using? And if possible the org name (just want to check whether there is a deployment issue going on).

Organization Name is my username (danielsel), Project is 'saw'
I would give a link to an example, but since it fails trying to load the yaml it doesn't even reach the state where a build is started.
A build from the affected pipeline: https://dev.azure.com/danielsel/saw/_build/results?buildId=73
The offending Build job (before I changed to V0 of the DownloadPipelineArtifact Task):
- job: saw_openwrt
  displayName: 'Package SAW for OpenWrt (opkg)'
  dependsOn:
  - solc
  - saw_ts
  condition: succeeded()
  pool:
    vmImage: ubuntu-16.04
  steps:
  - task: DownloadPipelineArtifact@1
    inputs:
      buildType: current
      artifactName: saw-contract
      downloadPath: $(Build.SourcesDirectory)/router/openwrt_pkgs/saw/files/etc/saw

Thank you for the quick response and the effort! Very much appreciated :)

Can I ask where you got this value from?
buildType: current
That value comes from the V1 version of the task which is still rolling out (the V0 version of the task doesn't have that property) so that might be why it is blowing up.

OK - looks like we've found the issue - it appears that our docs have been updated prematurely:
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/download-pipeline-artifact?view=azure-devops
… references buildType:. You should buy a lottery ticket, you've observed a rare moment in history where the docs precede the code! We are rolling out the tasks in the next deployment train (probably tomorrow) but in the meantime if you remove buildType: current you should get unblocked.

Ah, so V1 is not generally available?
That explains it then. This whole issue is about @landrus and me attempting to use V1 of the task.
I got this from: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/download-pipeline-artifact?view=azure-devops
Seems like this document is a mix of documentation for V0 and V1. And it doesn't say that V1 is not yet available ;)

Ha, I was too slow.
Yeah, reverting to V0 already did the trick for me. I just expected V1 to work too, that's all.
Thanks for the quick clarification and have a nice day!

Yeah - the YAML docs are automatically generated from the task definition. So it looks like they got published earlier than the tasks rolled out. Also turns out that our task update missed the release train so we are trailing the rest of the updates.

What is unfortunate is that this is in this state for several weeks now...

Thanks @landrus - we are working to speed up our release process for things like this. Thankfully I can now see this hitting various scale units now so it might already be resolved for you.

works now, so this can be closed


Invalid Json File

Any file I input tells me that it is an invalid json file: TypeError: e.map.fitBounds is not a function

Closing as there is not enough information to act on here.  If you think this is a bug, please provide a geojson file so others can reproduce the issue.


Unable to use Example❓

Question
Hey, for some reason, running yarn bootstrap on the example app returns;
ryanwilliams@Ryans-Air react-native-vision-camera % yarn bootstrap
yarn run v1.22.17
warning ../../../package.json: No license field
$ yarn && cd example && yarn && yarn setup && yarn pods
warning ../../../package.json: No license field
[1/4] 🔍  Resolving packages...
success Already up-to-date.
warning ../../../../package.json: No license field
[1/4] 🔍  Resolving packages...
success Already up-to-date.
warning ../../../../package.json: No license field
$ cd ios && bundle install
Using rexml 3.2.5
Following files may not be writable, so sudo is needed:
  /Library/Ruby/Gems/2.6.0
  /Library/Ruby/Gems/2.6.0/build_info
  /Library/Ruby/Gems/2.6.0/cache
  /Library/Ruby/Gems/2.6.0/doc
  /Library/Ruby/Gems/2.6.0/extensions
  /Library/Ruby/Gems/2.6.0/gems
  /Library/Ruby/Gems/2.6.0/specifications
  /usr/local/bin
Using CFPropertyList 3.0.4
Using concurrent-ruby 1.1.9
Using i18n 1.8.10
Using minitest 5.14.4
Using thread_safe 0.3.6
Using tzinfo 1.2.9
Using activesupport 5.2.6
Using public_suffix 4.0.6
Using addressable 2.8.0
Using httpclient 2.8.3
Using json 2.5.1
Using algoliasearch 1.27.5
Using atomos 0.1.3
Using bundler 2.2.27
Using claide 1.0.3
Using fuzzy_match 2.0.4
Using nap 1.1.0
Using netrc 0.11.0
Using ffi 1.15.4
Using ethon 0.14.0
Using typhoeus 1.4.0
Using cocoapods-core 1.10.2
Using cocoapods-deintegrate 1.0.5
Using cocoapods-downloader 1.5.1
Using cocoapods-plugins 1.0.0
Using cocoapods-search 1.0.1
Using cocoapods-trunk 1.6.0
Using cocoapods-try 1.2.0
Using colored2 3.1.2
Using escape 0.0.4
Using fourflusher 2.3.1
Using gh_inspector 1.1.3
Using molinillo 0.6.6
Using ruby-macho 1.4.0
Using nanaimo 0.3.0
Using xcodeproj 1.21.0
Using cocoapods 1.10.2
Using cocoapods-check 1.1.0
Bundle complete! 2 Gemfile dependencies, 39 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.
warning ../../../../package.json: No license field
$ cd ios && bundle exec pod install
Could not find cocoapods-1.10.2, cocoapods-check-1.1.0, cocoapods-core-1.10.2, molinillo-0.6.6, ruby-macho-1.4.0, activesupport-5.2.6, minitest-5.14.4, tzinfo-1.2.9, ethon-0.14.0, thread_safe-0.3.6 in any of the sources
Run `bundle install` to install missing gems.
error Command failed with exit code 7.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 7.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Could anyone give me some pointers as to why it doesn't work? I'm out of ideas. (I'm new to RN).
Thanks in advance.
What I tried
yarn bootstrap
VisionCamera Version
2.12.0
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Hi!
The number 1 tip I give newbies is to read error messages carefully as if it was an employment contract.
Your error message says:

Run bundle install to install missing gems

Actually the bundle install command should've already been executed by the script... 🤔

Hi @mrousavy, I did that when I first read the error also, but have had no joy, when I do run that command, it just says;
ryanwilliams@Ryans-Air react-native-vision-camera % cd example/ios
ryanwilliams@Ryans-Air ios % bundle install
Using rexml 3.2.5
Following files may not be writable, so sudo is needed:
  /Library/Ruby/Gems/2.6.0
  /Library/Ruby/Gems/2.6.0/build_info
  /Library/Ruby/Gems/2.6.0/cache
  /Library/Ruby/Gems/2.6.0/doc
  /Library/Ruby/Gems/2.6.0/extensions
  /Library/Ruby/Gems/2.6.0/gems
  /Library/Ruby/Gems/2.6.0/specifications
  /usr/local/bin
Using CFPropertyList 3.0.4
Using concurrent-ruby 1.1.9
Using i18n 1.8.10
Using minitest 5.14.4
Using thread_safe 0.3.6
Using tzinfo 1.2.9
Using activesupport 5.2.6
Using public_suffix 4.0.6
Using addressable 2.8.0
Using httpclient 2.8.3
Using json 2.5.1
Using algoliasearch 1.27.5
Using atomos 0.1.3
Using bundler 2.2.27
Using claide 1.0.3
Using fuzzy_match 2.0.4
Using nap 1.1.0
Using netrc 0.11.0
Using ffi 1.15.4
Using ethon 0.14.0
Using typhoeus 1.4.0
Using cocoapods-core 1.10.2
Using cocoapods-deintegrate 1.0.5
Using cocoapods-downloader 1.5.1
Using cocoapods-plugins 1.0.0
Using cocoapods-search 1.0.1
Using cocoapods-trunk 1.6.0
Using cocoapods-try 1.2.0
Using colored2 3.1.2
Using escape 0.0.4
Using fourflusher 2.3.1
Using gh_inspector 1.1.3
Using molinillo 0.6.6
Using ruby-macho 1.4.0
Using nanaimo 0.3.0
Using xcodeproj 1.21.0
Using cocoapods 1.10.2
Using cocoapods-check 1.1.0
Bundle complete! 2 Gemfile dependencies, 39 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.

But then when I exit back in the root directory, and run yarn bootstrap, I'm back at the original issue 🤔

I'm honestly not sure what the issue is, but you can easily just ignore the error.
Running
yarn
cd example
yarn
cd ios
pod install

should set up the example project for you. Then, in the example/ folder you can just run yarn ios or yarn android to start it.

I also can't build example.
in package.json there is no react-native-vision-camera as a dependency...
is it as expected?
FAILURE: Build failed with an exception.

* Where:
Build file '/Users/mine/react-native-vision-camera/android/build.gradle' line: 17

* What went wrong:
A problem occurred evaluating project ':react-native-vision-camera'.
> VisionCamera: Failed to find node_modules/ path!

BUILD FAILED in 5s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081

FAILURE: Build failed with an exception.

* Where:
Build file '/Users/shuzo/react-native-vision-camera/android/build.gradle' line: 17

* What went wrong:
A problem occurred evaluating project ':react-native-vision-camera'.
> VisionCamera: Failed to find node_modules/ path!


2.31 is a breaking change with CSP headers

If you have specified a restrictive CSP then 2.31 is a breaking change.
VM105:1 Refused to connect to 'https://api-js.mixpanel.com/decide/?verbose=1&version=1&lib=web&token=redacted' because it violates the following Content Security Policy directive: "connect-src 'self' api.mixpanel.com".

Consider either releasing guidelines for using CSP with Mixpanel or re-release 2.31 as version 3.0.0.

@tdumitrescu please be more careful with releases, both 2.31 and 2.32 contain breaking changes and should have been a major version bump.

@DrPhil @shnere we're very sorry for any inconvenience this may have caused you. We're curious to know what breaking change affected you in 2.32.0.
By necessity we have to be extremely sparing with major version bumps, and we make an effort not to introduce any breaking changes in the 2.x line. I suspect the problem arising in this issue is around the definition of a breaking change. With regard to 2.31.0: we did change one of our default API hostnames, and while this would affect you if you had set up a CSP restricting requests to the old hostname, this hostname was also never considered part of the library's public API. We absolutely should have communicated that upcoming change proactively to users, and that's on us, but there are a lot of internal elements of the library that we need to be able to update without considering every behavior change a "breaking" change. This includes Mixpanel's HTTP API surface area (IP addresses and hostnames), as well as request-sending methods (i.e. the recent default change from GET to POST, which saw tracking data move from URL params to the request body; this library never made any guarantees about how it's going to send out requests, and you shouldn't be expecting that to remain unchanged - it's an implementation detail and not part of a public API). What we consider the public API is documented at https://developer.mixpanel.com/docs/javascript-full-api-reference.
You don't have to agree with this stance - there are a lot of ways to play the versioning game, and some projects are much more aggressive than others about jumping versions - but I want to make sure people are at least aware of our approach and to know that we're not cavalier about versioning. We make every effort to ensure that any new 2.x release will work seamlessly without any changes at all to existing site implementations. There are further updates in the works for early 2020 which will change elements of the library that are not considered public and shouldn't be relied on - e.g., the number of events sent out per request, and request-retry mechanisms. If you feel uncomfortable automatically upgrading when these types of library updates occur, depending on your integration type there are ways to pin the library to a specific version, and then you can choose exactly when to upgrade explicitly.

Oh and @DrPhil: great idea about publishing CSP guidelines for Mixpanel users. We'll get working on it.

@tdumitrescu thank you for your response. 2.31 broke us because we didn't have https://api-js.mixpanel.com in our CSP.
In the case of 2.32, it broke because we are proxying calls done to the mixpanel API through our backend by changing the api_host variable, which was assuming calls to the API were done using GET and we didn't have a POST handler on our side. This was solved in two separate ways, in one case we simply added a POST handler on our backend and in the other we switched back the api_method to use GET as documented in the release, since it wasn't straight forward to add a POST handler.
I understand this may not be the correct or even supported/expected way to use this library, just a series of unfortunate events that ended up breaking our mixpanel integration.
Is there a way to pin the library to a specific version using the CDN? //cdn.mxpnl.com/libs/mixpanel-2-latest.min.js
Thanks!

So sorry for the disturbance there. If you're using the script tag to load the library asynchronously, you can change the URL to https://cdn.mxpnl.com/libs/mixpanel-2.32.0.min.js to pin it to 2.32.0.
If you don't want to mess with the contents of our JS snippet in the script tag, you can also set the URL via a global var as long as it's set before the snippet runs:
window.MIXPANEL_CUSTOM_LIB_URL = 'https://cdn.mxpnl.com/libs/mixpanel-2.32.0.min.js';

Hi @tdumitrescu are Mixpanel's required CSP headers documented anywhere? I couldn't find them in the JS docs, security docs, or a google search. Thanks!

Quick comment.
I'm running v2.39
I do not configure api_host anywhere.
I have one environment I just noticed hitting this new URL.  My other enviroment running v2.39, is still hitting api.mixpanel.com.
That's weird.

By popular demand on this thread, adding a link to unofficial Mixpanel CSP configuration on RapidSec, that is manually curated from many connected site.
Hope this helps.

Thanks @shaialon, what's the mechanism for submitting corrections to that config? It looks like there's a bunch of cruft in there which can be removed for safety.

@tdumitrescu we intend to open source it soon enough.
Until then you can send any suggested changes to<EMAIL_ADDRESS>or otherwise comment in this thread.
For you convenience - this is the underlying data that will soon be open sourced:
import { Directive, CspPackage, CspPackageType, CspPackageCategory, CspPackageSourceType } from "../types";
import { getAddedRulesCount } from "./utils";

const policy = {
  [Directive.CONNECT_SRC]: ["*.mixpanel.com", "cdn.mxpnl.com", "cr-input.mxpnl.net"],
  [Directive.SCRIPT_SRC]: ["https://cdn.mxpnl.com", "https://cdn4.mxpnl.com", "https://cdn-mxpnl.com"],
};

export const mixpanel: CspPackage = {
  id: CspPackageType.Mixpanel,
  version: 1,
  name: `Mixpanel`,
  icon: `https://scdn.rapidsec.com/technologies/Mixpanel.png`,
  description: `Mixpanle is a leading product analytics tool with a JS SDK to collect client-side data.`,
  categories: [CspPackageCategory.Analytics, CspPackageCategory.Marketing],
  topDomain: ["mixpanel.com", "mxpnl.com", "mxpnl.net", "cdn-mxpnl.com"],
  subDomains: ["api-js.mixpanel.com", "cdn.mxpnl.com", "api.mixpanel.com", "cr-input.mxpnl.net", "cdn4.mxpnl.com"],
  topAssets: {
    [Directive.CONNECT_SRC]: [
      "https://api-js.mixpanel.com",
      "https://cdn.mxpnl.com/libs/mixpanel-2-latest.min.js",
      "https://api.mixpanel.com",
      "https://cr-input.mxpnl.net/data?_channel_id=&_partner_id=*&_sub_id=0000&_app_version=1.0.23&_app=cs-dca",
    ],
    [Directive.SCRIPT_SRC]: [
      "https://cdn.mxpnl.com/libs/mixpanel-2-latest.min.js",
      "https://cdn4.mxpnl.com/libs/mixpanel-2-latest.min.js",
      "https://cdn.mxpnl.com",
      "https://cdn4.mxpnl.com",
      "https://cdn-mxpnl.com/*.js",
    ],
    [Directive.BLOCK_ALL_MIXED_CONTENT]: ["https://cdn.mxpnl.com/libs/mixpanel-2-latest.min.js"],
  },

  sources: [
    {
      title: `Cannot load Mixpanel in Chrome Extension due to CSP`,
      link: `https://stackoverflow.com/questions/15837450/content-security-policy-cannot-load-mixpanel-in-chrome-extension`,
      type: CspPackageSourceType.StackOverflow,
    },
    {
      title: `Mixpanel - breaking change with CSP headers`,
      link: `https://github.com/mixpanel/mixpanel-js/issues/234`,
      type: CspPackageSourceType.Github,
    },
  ],
  dynamicVariables: [],
  dependencies: [],
  policy,
  count: getAddedRulesCount(policy),
};

Thanks for the info @shaialon, we'll do a closer review and get back to you, but before then I would strongly recommend removing all the entries for cdn-mxpnl.com and cr-input.mxpnl.net. These domains are NOT associated with Mixpanel and are likely to be phishing sites (the IP addresses for cdn-mxpnl.com are also associated with a long list of deceptive domains like "cdn-js.net").

Thanks @tdumitrescu , apologies for the inconvenience.
Removed them from our records and and will deploy the changes soon.
We have a filter for malicous domains - I don't know how this one got through (will check).

Updating that the package is now updated:
https://rapidsec.com/csp-packages/mixpanel

I had to configure CSP to send events to mixpanel, but it worked with "connect-src api-js.mixpanel.com", so I share it.
It should be better if it is mentioned in the documentation, so please consider it!


[SPARK-23433][CORE] avoid more than one active task set managers for a stage

What changes were proposed in this pull request?
This is another attempt to fix the more-than-one-active-task-set-managers bug.
https://github.com/apache/spark/pull/17208 is the first attempt. It marks the TSM as zombie before sending a task completion event to DAGScheduler. This is necessary, because when the DAGScheduler gets the task completion event, and it's for the last partition, then the stage is finished. However, if it's a shuffle stage and it has missing map outputs, DAGScheduler will resubmit it(see the code) and create a new TSM for this stage. This leads to more than one active TSM of a stage and fail.
This fix has a hole: Let's say a stage has 10 partitions and 2 task set managers: TSM1(zombie) and TSM2(active). TSM1 has a running task for partition 10 and it completes. TSM2 finishes tasks for partitions 1-9, and thinks he is still active because he hasn't finished partition 10 yet. However, DAGScheduler gets task completion events for all the 10 partitions and thinks the stage is finished. Then the same problem occurs: DAGScheduler may resubmit the stage and cause more than one actice TSM error.
https://github.com/apache/spark/pull/21131 fixed this hole by notifying all the task set managers when a task finishes. For the above case, TSM2 will know that partition 10 is already completed, so he can mark himself as zombie after partitions 1-9 are completed.
However, #21131 still has a hole: TSM2 may be created after the task from TSM1 is completed. Then TSM2 can't get notified about the task completion, and leads to the more than one active TSM error.
#22806 and #23871 are created to fix this hole. However the fix is complicated and there are still ongoing discussions.
This PR proposes a simple fix, which can be easy to backport: mark all existing task set managers as zombie when trying to create a new task set manager.
After this PR, #21131 becomes a pure optimization, to avoid launching unnecessary tasks. #22806 and #23871 are still valuable to improve this optimization.
How was this patch tested?
existing tests.

Test build #102913 has started for PR 23927 at commit 424a3c8.

cc @squito @pgandhi999 @Ngone51

This fixes https://issues.apache.org/jira/browse/SPARK-25250 as well, because zombie TSM won't launch new tasks.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8387/
Test PASSed.

Test build #102913 has finished for PR 23927 at commit 424a3c8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102913/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8405/
Test PASSed.

Test build #102935 has started for PR 23927 at commit 53c6ed8.

Test build #102935 has finished for PR 23927 at commit 53c6ed8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102935/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8434/
Test PASSed.

Test build #102965 has started for PR 23927 at commit f94809d.

Test build #102965 has finished for PR 23927 at commit f94809d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102965/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8441/
Test PASSed.

Test build #102972 has started for PR 23927 at commit f94809d.

Test build #102972 has finished for PR 23927 at commit f94809d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102972/
Test PASSed.

Test build #103022 has started for PR 23927 at commit 0ca733d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8488/
Test PASSed.

Test build #103022 has finished for PR 23927 at commit 0ca733d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103022/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8503/
Test PASSed.

Test build #103039 has started for PR 23927 at commit 0ca733d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8508/
Test PASSed.

Test build #103046 has started for PR 23927 at commit 07d7de9.

Test build #103039 has finished for PR 23927 at commit 0ca733d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103039/
Test PASSed.

Test build #103046 has finished for PR 23927 at commit 07d7de9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103046/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8520/
Test PASSed.

Test build #103058 has started for PR 23927 at commit 07d7de9.

thanks all of you for the review! I've updated the PR description.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8522/
Test PASSed.

Test build #103060 has started for PR 23927 at commit 58f646e.

Test build #103060 has finished for PR 23927 at commit 58f646e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103060/
Test FAILed.

Jenkins, retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8532/
Test PASSed.

Test build #103069 has started for PR 23927 at commit 58f646e.

Test build #103058 has finished for PR 23927 at commit 07d7de9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103058/
Test PASSed.

Test build #103069 has finished for PR 23927 at commit 58f646e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103069/
Test PASSed.

lgtm

merged to master, 2.4, & 2.3.  Thanks!


Advances hapi versions and skips xhtml fields

Description of what I changed

For why we are doing this, see #1014 and in particular this comment.
E2E test

TESTED:
Relied on unit/e2e tests.
Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 51.43%. Comparing base (5eb6819) to head (870916b).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1240      +/-   ##
============================================
- Coverage     51.48%   51.43%   -0.06%
+ Complexity      716      715       -1
============================================
  Files            95       95
  Lines          5907     5909       +2
  Branches        787      787
============================================
- Hits           3041     3039       -2
- Misses         2578     2581       +3
- Partials        288      289       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@chandrashekar-s I think the e2e failure is transient, so I think this is ready for review (I have restarted the e2e run).


Update meeting-expiration.md

OSDP --> ODSP

@KarliStites PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Update SharedUtilsFunctions.ps1 for ExaGrid

Add ExaGrid support

Description
Add support of ExaGrid Appliance
Related Issue
Exagrid Appliance not display in graph repository
Motivation and Context
Some veeam user have Exagrid Appliance
How Has This Been Tested?
Test with Exagrid appliance succesfull
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

Hi,
First of all,  thanks for all your work for this project.
A little pull request to support exagrid dedup appliance
We use it and i need to add repository on ExaGrid
Thanks
Best regards

Thanks for your contribution!

Hi, First of all, thanks for all your work for this project. A little pull request to support exagrid dedup appliance We use it and i need to add repository on ExaGrid
Thanks Best regards

Question for you:
Do you use the AsBuiltReport.Veeam.VBR or do you just use Veeam.Diagrammer to create your documentation?

Hello, i discovered this project with AsBuiltReport.Veeam.VBR.
At this time, i have just some test with Veeam.Diagrammer but i will try report on next days
Thanks for commit


IPv6 support

Is your feature request related to a problem? Please describe.
Not a problem as such, but it would be nice if relays are tested on their IPv6 address too (because it might be dual stacked and working on IPv4, but not on IPv6, which could lead to problems).
Describe the solution you'd like
Besides IPv4 addresses in the detailed dashboard, also show IPv6 availability. Perhaps even in the main dashboard?
Describe alternatives you've considered
None.
Additional context
N/A

nostr.watch uses the cloudflare DOH resolver to detect DNS. According to these docs, it should be possible, however, when I was attempting to complete this a while back (you're not the first to ask), I kept hitting roadblocks. I was timeboxed and had to move on. I will revisit when I'm not in the middle of a release.
cheers!

Additionally, the IP-API of choice (the one that's affordable and has robust support for client-side apps) does not support IPv6 (probably not ever), so the Geo Data would need to be queried based on IPv4.

nostr.watch uses the cloudflare DOH resolver to detect DNS. According to these docs, it should be possible

That link refers to DNS64 (and consequently NAT64), which is not what I meant. I'm talking 'native IPv6 connectivity', so just do an 'AAAA' query besides an 'A' query. Geolocation is something else. I suppose I could live with a page that would only show the geolocation of the IPv4 address.


UIImagePickerController not dismissed

public func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
isGalleryShowing = false
}
This function is missing: dismiss(animated: true, completion: nil)

This is resolved in 2.2.0.