Firejail module: exposed wrappedBins

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

Motivation?

Motivation?

Let me guess: somewhere the variable is needed. Doesn't matter to much to be honest.

Do nixos modules support passthru? If so that should be used.

Yeah, the motivation was to be able to use the wrapped binary elsewhere (mainly in makeDesktopItem or to edit an existing deskop item).
However, I think a better solution would be to expose this as function in all-packages. Though something feels a bit wrong with this approach as it would require that the firejail wrapper be enabled in order for this function to work.

Side note: Is there any reason in particular why the firejail package is both added to systemPackages and the security wrapper?

#79104: "this way the man page etc. becomes available"

Thanks

Also, I'm closing since, I no longer have the motivation to fix this into a format for the repo. But is the code I'm using to wrap packages
{ lib
, firejail
, runtimeShell
}:

{ package
, paths
, args ? ""
, name ? lib.getName package
, profile ? "${firejail}/etc/firejail/${name}.profile"
}:
let
  wrapFileMapper = p: ''wrapProgram "$out/${p}"'';
in
package.overrideAttrs (old: {
  postFixup = /* bash */ ''
    ${old.postFixup or ""}

    # ensure profile exists
    if [[ ! -e ${profile} ]]; then
      echo '"${profile}"' does not exist
      exit 1
    fi

    wrapProgram() {
      local prog="$1"
      local hidden

      hidden="$(dirname "$prog")/.$(basename "$prog")"-wrapped
      while [ -e "$hidden" ]; do
        hidden="''${hidden}_"
      done

      hidden2="''${hidden}_"
      while [ -e "$hidden2" ]; do
        hidden2="''${hidden2}_"
      done

      echo $prog $hidden $hidden2

      # prog - firejail
      # hidden - change name of exec
      # hidden2 - original

      mv "$prog" "$hidden2"

      echo "#!${runtimeShell} -e" > "$hidden"
      echo "exec -a \"$(basename "$prog")\" \"$hidden2\" \"\$@\"" >> "$hidden"
      chmod +x "$hidden"

      echo "#!${runtimeShell} -e" > "$prog"
      echo "exec /run/wrappers/bin/firejail --profile=\"${profile}\" ${args} -- \"$hidden\" \"\$@\"" >> "$prog"
      chmod +x "$prog"

    }

    ${lib.concatMapStringsSep "\n" wrapFileMapper paths}
  '';
})

Its kind of dirty since it overrides the original package, but I think I did it that way to be able to easily keep desktop files. Also, for some reason zoom's desktop file has a hardcoded full path, so I just took the easy route.


Editing an agent while training results in false up to date message

If you start training an agent, and then add a saying to the sayings list while it is training, the agent will show as Out of Date. Once the training process completes it will change the agent status to trained. This message is misleading since there have been changes to the agent, which have not been trained. The expected status of the agent should still be out of date.

Closed by https://github.com/samtecspg/articulate/commit/7beeaf40c596f0c1ea6e06ed00982f20f42cde83


Uncaught ReferenceError: process is not defined

Reproducing the Error

Run the project using yarn start
Make some changes in the code
Check the browser console

Hello there, could I be assigned to this issue?


Typechat agent v2

A PR has been merged in the agent extension to use TypeChat which doesn't changes in the service extensions.


fix: resolve issue of double click to select all in checkbox group

Description

In CheckBoxGroupComponent, when isSelectAll is enabled, it takes two clicks to check/uncheck it rather then one. The reason for this issue was, the custom className select-all given to StyledCheckbox component. The same className is available from tailwind which adds the user-select: all property, making the label text of select all option to be selectable. So it used to select the text on first click, and then used to check the select all option on second click.

Add a TL;DR when description is extra long (helps content team)

Fixes #18880
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=b3d69dd

LGTM, we can merge @jsartisan


Syncing times and coordinates

Is it possible to synchronize the coordTimes and coordinates when converting GPX to geoJSON?  Your browser based tool seems to strip the times completely, and the command line implementation separates the coordinates (in geometry) from the coordTimes (in properties).

the command line implementation separates the coordinates (in geometry) from the coordTimes (in properties)

This is by design: putting time values in coordinates in GeoJSON is not specified and probably a dangerous idea. See https://github.com/mapbox/geojson-coordinate-properties for the draft spec.

OK, thanks.  The data is still keyed so I can extract a time/coordinate pair correct?

Yes, the coordinates array and properties arrays are symmetrical.


Pachyderm Dashboard Pod won't start in OpenShift

What is the goal / desired outcome?
If there is a way to accomplish this today via workaround, what does that require?
(Optional) What is your proposal for a feature to solve this?

Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

What happened?:
When deploying Pachyderm in OpenShift, the pod will not start due to OpenShift not allowing privileged containers to run by default. The logs for the dash pod display the following:
bash: run.sh: Permission denied
What you expected to happen?:
Dash Pod should start without any issues
How to reproduce it (as minimally and precisely as possible)?:

Deploy OpenShift 3.10
Deploy Pachyderm with an appropriate cloud backend

Anything else we need to know?:
According to the docs in openshift, the container image needs to be able to "Support Arbitrary User IDs" https://docs.openshift.com/container-platform/3.10/creating_images/guidelines.html.
Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

The run.sh script is located in /root , and is consequently owned by the root user. As a result, when the container runs in Openshift with an unprivileged service account, then we get the permission denied error mentioned above.
I encountered this issue on Openshift 4.7.12

Hi @OchiengEd thanks for resurfacing this!
At this point, Pachyderm's current dash (and associated dash pod) aren't being maintained anymore, so I think we're unlikely to be able to get the dash pod running in OpenShift. The goal is currently to re-add support for non-root usage (i.e. OpenShift) in the next version of Pachyderm's dash, which will work with Pachyderm 2.0.
We can help over slack with working around this.


is it available for Sublime 3?

I am not able to find this plugin using package control using sublime text 3.

Yes, probably as case of https://github.com/sindresorhus/sublime-autoprefixer/issues/41.

You are correct. Sorry For raising it before checking my sublime packages. Thanks for the response


Patching files with latest commit and adding skipibmpower for test

Added new patch files/ocs-ci/ocs-ci-17-node-mantainance-skip.patch for skipping tier-4a tests
Deleted patch files/ocs-ci/ocs-ci-17-scipy-fix.patch as the fix is no longer needed
        "numpy==1.22.4",
        "scipy==1.12.0"

Above versions are getting successfully installed

Before Changing setup logs
ls: cannot access '../../files/ocs-ci/powervm/ocs-ci-*[0-9][0-9]-*.patch': No such file or directory
platform_patchfiles=
Generating consolidated patch file /root/ocs-ci.patch from /root/ocs-upi-kvm/files/ocs-ci/
checking file ocs_ci/ocs/benchmark_operator.py
checking file ocs_ci/ocs/amq.py
checking file ocs_ci/templates/workloads/amq/benchmark/values.yaml
checking file ocs_ci/templates/workloads/amq/hello-world-consumer.yaml
checking file ocs_ci/templates/workloads/amq/hello-world-producer.yaml
checking file tests/cross_functional/scale/test_scale_osds_fill_75%_reboot_workers.py
checking file tests/functional/z_cluster/cluster_expansion/test_add_capacity_entry_exit_criteria.py
checking file tests/functional/z_cluster/cluster_expansion/test_add_capacity_with_node_restart.py
checking file tests/functional/z_cluster/cluster_expansion/test_delete_pod.py
checking file tests/functional/z_cluster/cluster_expansion/test_node_expansion.py
checking file tests/functional/z_cluster/cluster_expansion/test_verify_ceph_csidriver_runs_on_non_ocs_nodes.py
checking file tests/functional/z_cluster/cluster_expansion/test_crashcollector_pod_existence_on_ceph_pods_running_nodes.py
Hunk #1 succeeded at 5 with fuzz 2.

Hunk #2 succeeded at 114 (offset 14 lines).
checking file tests/functional/z_cluster/test_ceph_default_values_check.py
checking file tests/functional/z_cluster/test_must_gather.py
checking file tests/functional/object/mcg/test_s3_with_java_sdk.py
checking file tests/functional/object/mcg/test_noobaa_secret.py
Hunk #1 succeeded at 21 (offset 6 lines).
  Has git submodule ocs-ci HEAD changed?


NTR: endocrine-disrupting chemical

New term requested: endocrine-disrupting chemical
Suggested text definition: Chemicals that can interfere with endocrine (or hormone) systems at certain doses.
SOURCE: https://en.wikipedia.org/wiki/Endocrine_disruptor
Example: CHEBI:76988 - xenoestrogen
Related to ticket: https://github.com/cmungall/environmental-conditions/issues/6

Thinking about relationship to existing 'hormone' term in CHEBI...
The ED role can be realized in different ways:

mimicking endogenous hormones (as agonist or antagonist)
altering metabolism of endogenous hormones
altering natural hormone receptors

Should we have 3 subclasses for these? What would their relationship to existing CHEBI roles be?
current polyhierarchy:
  is_a CHEBI:24432 ! biological role
   is_a CHEBI:33280 ! molecular messenger
    is_a CHEBI:24621 ! hormone ***
   is_a CHEBI:52210 ! pharmacological role
    is_a CHEBI:48705 ! agonist
     is_a CHEBI:24621 ! hormone ***

CHEBI treats "hormone" generically as inclusive of non-natural as well as endogenous. I recommend the CHEBI def of hormone is changed (in phrasing but not in meaning):
"Originally referring to an endogenous compound that is formed in specialized organ or group of cells and carried to another organ or group of cells, in the same organism, upon which it has a specific regulatory function, the term is now commonly used to include non-endogenous, semi-synthetic and fully synthetic analogues of such compounds."
OBO definitions should not start with discursive history of terminology, they should concisely and precisely state the meaning, following a genus-differentia form. Additional gloss and clarification can be placed at the end.
The current parent is agonist, so a candidate def would be
"An agonist that is transported by the circulatory system and executes a regulatory function, or any compound that mimics this."
(OK, that could probably do with work).
Are hormones always agonists? The parent def is also a little awkward:
CHEBI:48705 ! agonist [DEF: "Substance which binds to cell receptors normally responding to naturally occurring substances and which produces a response of its own."]

Have now created endocrine disruptor (CHEBI:138015) and populated it with some classes and a few individual compounds.
Will leave the ticket open while we think about rephrasing our definition of 'hormone'!


[Windows] \dev\tty: No such a device

Hi! I developed a package for atom, and when trying to publish it from windows I get this error:
bash: /dev/tty: No such device or address
error: failed to execute prompt script (exit code 1)
fatal: could not read Username for 'https://github.com': No error

After running apm publish minor from the repo I'm storing the package into.
Why is this happening?

Looks like you're using Cygwin? Try using a normal Windows shell such as CMD or PowerShell.

I'm running that from cmd.exe, with git installed overriding windows tools like find and such so everything should in principle be as git wants. It's puzzling.

Same problem but I worked around it by changing the git remote url to include my git username and password.
List your remote URL using:
git remote -v

Update the remote URL to include username and password just before github.com:
git remote set-url<EMAIL_ADDRESS>
Not sure why this was necessary,  I thought I'd be prompted for my credentials if git didn't know them.
More details, hopefully relevant:

OS Windows 10
Git version is 2.6.0 from http://git-scm.com/
Recently done a Windows re-install

How does one report an issue to git itself?

Just ran into this after updating Git to v2.6.0.  I have allowed Git to be used from cmd, but not to override the default cmd commands.
C:\Users\user\Documents\GitHub\notifications>apm publish minor
Preparing and tagging a new version done
Pushing v0.60.0 tag failed
bash: /dev/tty: No such device or address
error: failed to execute prompt script (exit code 1)
fatal: could not read Username for 'https://github.com': Invalid argument

> git --version
git version 2.6.1.windows.1

I maintain another git-wrapping tool, and my Window's users at git 2.5 and higher are experiencing the same issue.

same issue,
i think cygwin or other bash tool interupt shell for git@sublime,
.. ㅠㅠ

I had the same error and maybe the way I fixed it could help you too.
I'm using GitExtension 2.48.05. From settings I'm using openSSH (configured via Cygwin).
I changed the following things:

Go to Tools->Settings and in "Git"/ "Path to linux" put the bin forlder of your cygwin install
In SSH panel, set openSSH (openSSH must be installed from Cygwin and configured to run from command line
use ssh instead of HTTPS links. (I configured "config" file in .ssh on my home directory to use my own private rsa key )

Everything works

Try adding this into your git config
[credential]
	helper = wincred

@blowsie
it does not work

Thanks @blowsie, worked for me.

I got this error when I changed my CygWIn home to my windows home by adding the HOME environment variable and setting it to %UserProfile%

I think that install winpty may help you.
And run apm this like below:
winpty apm publish minor

What i did was opened the C:\Users\logedinuser.gitconfig file and replace triple slahes  \\ to single
slash \  e.i.
helper = !\"C:/Program Files (x86)/GitExtensions/GitCredentialWinStore/git-credential-winstore.exe\"
into
helper = !"C:/Program Files (x86)/GitExtensions/GitCredentialWinStore/git-credential-winstore.exe"

It looks like there are some workarounds here and that the problem is either in Git or in APM. You can report issues with the latter at https://github.com/atom/apm/issues

@damieng yep, this is https://github.com/git-for-windows/git/issues/327. The atom bug tracker is definitely the wrong one.

Is is possible to check the error level on Windows? If the files cannot be uploaded with the error, the package is still posted. Then, all we have is an empty post on atom.io/packages. It would be great it the the upload were atomic, not posted if it is not successful.

Thanks @blowsie, worked for me!

[credential]
helper = wincred

@bphamous please note that the Git Credential Manager is shipped with Git for Windows as of March 15th. It would configure your credential helper automatically, but of course, if you edited your $HOME/.gitconfig, the latter would override Git for Windows' setting.

setting the credential helper was the key to success on my side!
Thanks all!

For people having issue with the credential helper in general, I would recommend using SSH, it can solve a lot of issues.

I use GitExtensions as my git wrapper and have been having this issue with new repositories ever since upgrading my local git client to 2.8.4. For some reason the wrapper did not offer to enter the username and password while trying to fetch the data from the repo. Configuring wincred has solved the problem once and for all.
P.S.
I did not have any issues using the native git client regardless of the credential helper setting.

@KV-VadimGusev something tells me that you did not upgrade your local Git client to 2.8.4...

@dscho, no I did not. The latest version on the Git download page is 2.8.3, released 4 days ago

The latest version on the Git download page is 2.8.3, released 4 days ago

I know. I released it :smile:

@gavynriebau you missed the remote name, it should be:
git remote set-url remote-name<EMAIL_ADDRESS>

@blowsie tks so much

@blowsie it works for me in MobaXterm

@blowsie  youre the man worked instantly

@blowsie Nice, now it works!
git config --global credential.helper wincred

@blowsie  it worked. Thanks

HP@HP-PC MINGW64 /f/data science project (master)
$ git push -u origin master
Username for 'https://github.com': h
HP@HP-PC MINGW64 /f/data science project (master)
$ herror: failed to execute prompt script (exit code 1)
how to fix it


Fix panic when using .load_folder() with absolute paths

Fixes https://github.com/bevyengine/bevy/issues/9458.
On case-insensitive filesystems (Windows, Mac, NTFS mounted in Linux, etc.), a path can be represented in a multiple ways:

c:\users\user\rust\assets\hello\world
c:/users/user/rust/assets/hello/world
C:\USERS\USER\rust\assets\hello\world

If user specifies a path variant that doesn't match asset folder path bevy calculates, path.strip_prefix() will fail, as demonstrated below:
dbg!(Path::new("c:/foo/bar/baz").strip_prefix("c:/foo"));
// Ok("bar/baz")

dbg!(Path::new("c:/FOO/bar/baz").strip_prefix("c:/foo"));
// StripPrefixError(())

This commit rewrites the code in question in a way that prefix stripping is no longer necessary.
I've tested with the following paths on my computer:
let res = asset_server.load_folder("C:\\Users\\user\\rust\\assets\\foo\\bar");
dbg!(res);

let res = asset_server.load_folder("c:\\users\\user\\rust\\assets\\foo\\bar");
dbg!(res);

let res = asset_server.load_folder("C:/Users/user/rust/assets/foo/bar");
dbg!(res);

Just tested some examples, nice change! As a note, it does change the normal behaviour of read_directory when given a path that is relative to the root. Now it will always return absolute paths instead of paths relative to AssetIo::root_path. As the examples still work I don't think this is a breaking change, but it does change how this function works beyond just fixing the panic!.

Now it will always return absolute paths instead of paths relative to AssetIo::root_path.

It will return relative path if path parameter is relative (that's the most common case, and there're no changes there).
It will return absolute path if path parameter is absolute (before it was sometimes relative, and sometimes panic).

@bushrat011899 if you're happy with the change and think it's ready to merge, can you please leave a review using Github's system? Bevy works on "community reviews", and your opinion will get us halfway there to merging this :)

Great, this fixes #5770.


Implement oik-sb/sb-debug-block for debugging FSE templates and template parts

Background
In Fizzie, SB and ThisIs... I've used some custom HTML to add debug information to each template, and some template parts.
It appears at the top of each template enabling me to know which template I'm viewing.
eg In 404.html

404.html

If I change the text in the .html file I can tell whether or not the template has been loaded from the file or the database.
It doesn't let me know which language version is being used though.
Similarly, it's nice to know where a template part starts, and possibly where it ends.
eg in the 404.html template part

404 template-part

The first
 with class WP_DEBUG is styled with CSS in the theme's style.css stylesheet to appear in the wp-admin top bar.
Others have a dotted red border and appear when hovered over.
In order to turn off debugging it's necessary to either remove the custom HTML or change the CSS.
Finally, the output-input.html template uses a template part called contents-shortcode.html which uses the [contents] shortcode to display the raw HTML of the post content - escaped to make it visible in the browser.
This can be used to check that what we see in the front end was produced from the blocks in the post's content
Requirements

Use the sb-debug-block to implement a similar solution to the hardcoded solution above
Dynamically adjust the output based on the value of certain constants
Provide a simple block UI to enable the user to indicate what they want the block to do

Proposed solution

[x] Server Side Rendered block
[x] Automatically determines the template name / template part name and where it was sourced.
[ ] If that's not possible display the value defined in the block's debug attribute
[ ] Generates the required CSS based on WP_DEBUG and/or SCRIPT_DEBUG and/or a block setting -eg Additional CSS class(es)
[ ] Displays the equivalent of the [contents] if the showContents block setting is toggled on. Default: false.
[ ] Display the HTML source for a particular block or group of blocks - to assist documentation.

Automatically determines the template name / template part name and where it was sourced.

This is only possible with changes to Gutenberg. Accessing the template information is easier than accessing template part information.

Accessing the template information is easier than accessing template part information.

Actually, now that I've written it, accessing the template part information was easier. I was able to extend the logic in my template part override function to call an action hook rendering_template_part both before and after the rendering.
The call before passes the $attributes and the $seen_ids array.
The call after passes a null parameter for the attributes and the updated $seen_ids array.
This allows the debug block to be used after a template-part block.
It should show the current level of template part nesting.
Now to attempt to rework the mechanism to indicate which template is being rendered by adding
a do_action( 'rendering_template' ) hook to replace the global I used in the first instance.


using TweenJS

Hello,
I don't know why I can't use twwen.js (https://github.com/twee njs/tween.js/) in Nunustudio but I already made it work in others threeJS project.
Do you have any example using this library ?
(also having a real console, like in chrome would be really helpfull for debugging)
Thanks for your help

I noticed something weird (well at least it is something I don't understand)
I managed to make it work by adding
[...]
requestAnimationFrame(update)
myTween.update(delta)
[...]
in my update function.

Hello
I have some bits of tweenjs inside of the nunuStudio source. It should be fairly simple to integrate:
It is a bit strange that you have to requestAnimationFrame separately for the tween.
I will try it out to see if i can put an example in the docs.
Thank you.
Cheers

+1 for seeing an example of tween.js at work. Thanks!


Look at optimizing how we determine which PV types a BlockStore supports

Currently, during backup, we determine if a BlockStore supports a given PV by calling the GetVolumeID function, which returns the cloud-provider volume ID if the BlockStore supports that PV, or an empty string/(error?) otherwise.
Look into whether it makes sense to add a Handles() or AppliesTo() function to the BlockStore interface that can tell pkg/backup upfront what types of PVs the BlockStore supports, to make mapping PVs to block stores more efficient.

Removing this from the epic as it's non-critical.

Thanks for this request. We are not currently prioritizing this item within our roadmap, and as such will be closing out the issue. Please feel free to reach out as needed in this issue or a new one.


State/CommandFile and repository design changes

These should be final. This pull request removes .dvc/state and introduces *.dvc files, that combine cache_state, state and command files. I.e. from now on dvc repository will look like this:
data/Posts.xml.tgz
data/Posts.xml.tgz.dvc
.dvc/config
.dvc/cache/
.gitignore
Where Posts.xml.tgz.dvc looks like:
cmd: null
deps: []
locked: true
md5: ec7cea7f176edab12d7006919393b8e8
out:

data/Posts.xml.tgz
out-git: []

Would it be possible to store the .dvc  files under a subdirectory? This would make it less cluttered. Having the .dvc files in the same directory effectively doubles the number of files and makes it harder to interoperate with people not using dvc.

Hi @kskyten !
That is a great point! We've actually just discussed it with @dmpetrov and decided on using per-directory 'Dvcfile's, that should hide all that cluttering. Patch for that should be ready shortly.
Thanks,
Ruslan


Registry Keys are Missing Data, with No Explanation as to Why

The instructions above state "Use this list of Windows 10, version 1709 registry keys as your starting point". However, the following apps are missing from "Registry keys for provisioned apps" listed above:
Microsoft.3DBuilder
Microsoft.HEVCVideoExtension
Microsoft.Messaging
So if the starting list you have documented does not include 3DBuilder, HEVCVideoExtension or Messaging, does this mean they cannot be removed, you recommend they are not removed, the registry keys do not work for those apps, or what??
Or is it just that the "Registry keys for provisioned apps" is not accurate?
There is a mistake somewhere......

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

ID: c9c3129d-3c8d-b508-f298-3333499bb180
Version Independent ID: 97b70720-82d8-60a4-3958-00ceed34aa1b
Content: How to keep apps removed from Windows 10 from returning during an update
Content Source: windows/application-management/remove-provisioned-apps-during-update.md
Product: w10
Technology: windows
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@officedocsbot assign @mypil

@MikhailCompo - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@MikhailCompo .I am adding 3 new registry keys via creating PR .

@MikhailCompo Done. Thank you

@MikhailCompo - From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Add extension to marketplace

Cheers guys! Awesome job with coder! We've been using it for a while and it's been working great for us! Could you add this extension to the marketplace:
https://marketplace.visualstudio.com/items?itemName=stroveio.stroveteams
Thanks from the whole strove.io team for the opensource contribution, we genuinely love what you're doing!

Glad to hear you're enjoying the project. Unfortunately, at this time we require extensions in our marketplace to have a public Github repository from which we can build the .vsix artifact. For this reason, we cannot add stroveteams.

I made the repository public @cmoog  https://github.com/stroveio/stroveTeams
Can you reopen?

Perfect. The extension is now live in our marketplace. Thanks!


Line endings not accounted for in logging

Reported by @runcom during testing (it was a TODO in conmon ;) )
envar-demo.yaml
apiVersion: v1
kind: Pod
metadata:
  name: envar-demo
  labels:
    purpose: demonstrate-envars
spec:
  restartPolicy: Never
  containers:
  - name: envar-demo-container
    image: gcr.io/google_containers/busybox:1.24
    command: ["sh", "-c", "env"]
    env:
    - name: FOO
      value: "foo-value"
    - name: BAR
      value: "bar-value"
    - name: FOOBAR
      value: "$(FOO);;$(BAR)"

Only one line is printed from output of env.
$ kubectl create -f envar-demo.yaml
$ pod "envar-demo" created
$ kubectl logs envar-demo
KUBERNETES_PORT=tcp://<IP_ADDRESS>:443

Closed by #436


undefined is not an object (evaluting 'topLevel[property]')

@swapanil Did you get this fixed , I have the same error

Fixed this by removing "prism" from the highlighter prop value
BEFORE
<SyntaxHighlighter
  	language='javascript'
  	style={docco}
  	highlighter={"prism" || "hljs"}
  >
  	{codeString}
  </SyntaxHighlighter>;

AFTER
<SyntaxHighlighter
  	language='javascript'
  	style={docco}
  	highlighter={"hljs"}
  >
  	{codeString}
  </SyntaxHighlighter>;

same. removing "prism" changed nothing

Solved. My problem was that we cannot use just style name. We need to import appropriate stylesheet file and use it for style prop
Example: import { docco } from 'react-syntax-highlighter/styles/hljs';


error with existing webrtc app

The app is built with easyrtc and running well in chrome and ff. but when i try to connect with bowser i get the following error:
set-remote-description: Failed to set remote offer sdp: Called with SDP without DTLS fingerprint.

I think it is a side effect of https://github.com/EricssonResearch/openwebrtc/issues/62 which are working to fix. There is a pull request for the fingerprint part that is up for review: https://github.com/EricssonResearch/openwebrtc/pull/427 .
I'm going to close this as the issue should be tracked in the main OpenWebRTC repository as noted above.


Please support deterministic output (PROV-N)

With Python 3 (3.6, 3.7), the output with PROV-N (at least) is not deterministic:
import prov.model as prov

d = prov.ProvDocument()
d.set_default_namespace('https://example.com')
d.entity('id', [(prov.PROV_TYPE, 'foo'), (prov.PROV_TYPE, 'bar')])
print(d.get_provn())

Gives sometimes
document
  default <https://example.com>

  entity(id, [prov:type="foo", prov:type="bar"])
endDocument

and sometimes
document
  default <https://example.com>

  entity(id, [prov:type="bar", prov:type="foo"])
endDocument

This makes it difficult to create doctests here.

Hi @olebole,
Attributes are stored internally in a set of values. I think this is why the values do not follow a deterministic order when listed. I can see why it is an issue for your testing, but I wonder if you could have the same test done in a different way. The two documents that you provided above are equivalent.

Could that be wrapped by sorted()? Having a deterministic output would greatly improve doctests. Currently, I write
'''Example:

>>> import prov.model as prov
>>> d = prov.ProvDocument()
>>> d.set_default_namespace('https://example.com')
>>> d.entity('id', [(prov.PROV_TYPE, 'foo'), (prov.PROV_TYPE, 'bar')])
>>> print(d.get_provn())
document
  default <https://example.com>

  entity(id, [prov:type="foo", prov:type="bar"])
endDocument
'''

which is nicely readable as tutorial for the user, and would be a good doctest. Putting the output into a string, parsing it, and comparing with the input doc would probably work (right?), but is then not usable as documentation anymore.


Update to ratatui 0.26

Hi, tui-textarea currently targets ratatui 0.24.0, but the latest version is 0.26.1. Could you update the dependency and publish a new release?

Tui-textarea's >=0.23 version spec means you only have to run cargo update and you'll be using the latest version released. (0.26.2 at time of writing). This is actually bad behavior as it causes the problems mentioned in:

https://doc.rust-lang.org/cargo/reference/resolver.html#unexpected-dependency-duplication
https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#multiple-version-requirements

This issue was solved by #69.


Default cluster alerts / alert groups don't seem to work.

Hi,
we use Rancher v2.2.3 and the "integrated" prometheus/alert manager instances.
The default alerts that ship with a new Rancher K8s cluster (and own alerts created in the default alert-groups) don't seem to work at all (they never seem to fire if the limit is hit).
As far as I can tell this is due to the alert manager that gets deployed only if you create your own alert groups.
What kind of request is this:

Bug Report

Steps to reproduce:

create a fresh cluster (we used K8s v1.13.5-rancher1-2)
enable monitoring
create an alert in an existing alert group (e.g. "A set of alerts for kube components") that should fire due  to the defined limit (e.g. "cluster:memory_usage:ratio > 0.001") and note that it doesn't ever fire (switch its state from "active" to "alerting")
have a look at "cattle-prometheus" namespace and note there is no "alertmanager-cluster-alerting" deployed
create a new/own alert-group with the very same alert
have a look at "cattle-prometheus" namespace and note "alertmanager-cluster-alerting" now is deployed
note that the alert (at least the one in the new alert-group, for the other see "Other details that may be helpful") will fire now

Result:

Before creating an own alert-group: no alert manager is present, therefore no alerts ever fire
After creating an own alert-group: alert manager is present, therefore alerts work now

Other details that may be helpful:
There is still a portion of the problem I don't understand:
While writing this I could not get the original alert (the one under "A set of alerts for kube components") to fire, even after the alert manager was deployed.
However I'm absolutely certain that in similar experiments yesterday I managed to get firing alerts in the default alert-groups, too.
Environment information

Rancher version: v2.2.3
Installation option (single install/HA): HA, vmware, on-premise

Cluster information

Cluster type: HA, vmware, on-premise, created via rancher
Machine type and specifications: TYPE: vmware CPU: 2 MEM: 4GB
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1", GitCommit:"b7394102d6ef778017f2ca4046abbaa23b88c290", GitTreeState:"clean", BuildDate:"2019-04-08T17:11:31Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:19:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           18.09.6
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        481bc77156
 Built:             Sat May  4 02:34:58 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.5
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       e8ff056
  Built:            Thu Apr 11 04:13:40 2019
  OS/Arch:          linux/amd64
  Experimental:     false

@Heiko-san Could you check:
If the alertmanager hasn't deployed:

check the rancher server's logs for the deploy related error

if alertmanager was deployed and satisfy the  alert condition but alert doesn't fire:

If you use express metric alert, first check whether prometheus trigger the alert(click the link in cluster-monitoring app then you could access prometheus UI)
then check whether alertmanager receive the trigger from prometheus(alertmanager UI could be accessed by clicking the link in cluster-alerting app).
If the alert manager receives the alert event, but email/slack... haven't received alert event, check the alertmanager log and notifier configure.


Run all formatting jobs on Ubuntu 22.04

Description
Ubuntu 22.04 ships all versions of clang-format we're using so we can easily use this single OS to test formatting. This ensures that none of these jobs will unexpectedly upgrade their clang-format versions.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.


Similar project

I've been doing some experiments for including real lyrics: https://github.com/0x2b3bfa0/blob-opera

You've starred my project and are aware of its existence; closing this issue.

Thanks for letting me know about this @0x2b3bfa0 , your implementation is really cool!

🔔   @OverlappingElvis, if wou want to take a look now...  🎉   It's in PyPi, has some documentation and interesting options like automated upload/download and reverse conversion (Blob Opera to MusicXML).

🔔   @OverlappingElvis, if wou want to take a look now...  🎉   It's in PyPi, has some documentation and interesting options like automated upload/download and reverse conversion (Blob Opera to MusicXML).


Migrate to java17

Consider releasing a java17 version of ranger. A lot of clients may still be using the old one, so will continue to maintain it, unless it is a bug that is functional, needn't have to make any improvements onto the old one, Should migrate to 17 and push a new major version from then on. The 2 series maybe.

Not possible right now. Will take it up later along with code cleanup to make it properly J17 compliant.


INC-970: Fix for 'OUT has no available incentive levels'

We're seeing a lot of these exceptions in dev (probably caused by the same domain event) but it's very noisy and makes it hard to see other more important problems.
We haven't seen this in production yet but it's feasable for this to happen there as well if, for whatever reason, at the time we process a domain even for a prisoner this is no longer in prison (their current agencyId would be OUT which stands for "Outside").
Try to determine the default Incentive level for OUT (which is not a valid prison) would cause an exception so it seems reason to just ignore these domain events (and log a warning message).

I have cleared the DQL in dev which should reduce the noise for now. If this happens again and/or in production we can investigate further why this is happening and potentially introduce this fix.


Google sign-in Cancelling throws an error

When I cancel the sign in it sends an error that I can't catch
try{
 GoogleSignInAccount googleSignInAccount = await _googleSignIn.signIn();
}catch(e){
  print('error');
}

duplicate of https://github.com/flutter/flutter/issues/44431


Ont 435 for case 0.2.1

Note that this PR is built on the branch backing PR 53.  Hence, PR 53 should be merged before this branch is merged.
CI is not set to run on this branch, but I can confirm I've downloaded the branch and run make clean && make check successfully.

This PR was prematurely merged.  Comms error between Paul and I, I apologize.  Opening new PR to re-do.


Title of Page - The .NET Framework Analyzer or The .NET Security Analyzers

I apperciate the work that goes into this documentation. I know it is challenging to keep updated. :smile:
It seems like this page is broadly about The .NET Framework Analyzer, but the HTML <title> is The .NET Security Analyzers - .NET | Microsoft Docs, and it also reads:

The .NET Security Analyzers must be installed as a NuGet package on every project where you want them to run. Only one developer needs to add them to the project. The analyzer package is a project dependency and will run on every developer's machine once it has the updated solution.

Where is this package?

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

ID: 171973c7-c225-a776-793f-cf67869ea0b8
Version Independent ID: f26f0129-8073-cc7e-1b3a-0002ad3af932
Content: The .NET Security Analyzers - .NET
Content Source: docs/standard/analyzers/framework-analyzer.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Good catch. I think that it's the same thing refereed in two different ways. Perhaps in the past it was referred to as the Security Analyzer. Other links going to this page do refer to it as the .NET Framework Analyzer.
We should probably remove reference to it as the .NET Security Analyzer.
https://www.nuget.org/packages/Microsoft.NetFramework.Analyzers/ is the package you're looking for.

@Thraka , thank you for the clarification regarding the Security Analyzer.


Remove unused argument 'implicitSolvent' from SystemGenerator in tests

fixes issue #886

:tada: this should fix our nightly builds


[Suggestion] Trade Shop

Can we have shop where people can trade items? So like trading 3 slime for 1 diamond, and something similar like that? I'd be greatly appreciated if you would add that.
Or if it's too complicated, would it be okay if I pay for the feature request?

No, sorry, I won't add this


hasMessageAvailable should return false when Reader read from latest messages

Fixes #4912
Motivation
When a reader that configured reading from the latest message, hasMessageAvailable will timeout or hang on.
Modifications
Make sure lastDequeuedMessage initialised with right position.

@zymap please address the review comments

@zymap please address the comments

@sijie ok.

@sijie  @merlimat  Sorry for the late response. PTAL

@sijie @merlimat PTAL. Thanks

@merlimat can you review this again as @zymap added the tests?


train.py -> function check_accuracy -> accuracy formula is wrong

need to fix this way:
acc = float(num_correct) / num_samples / ( preds.size(1)*preds.size(2))
    logging.info('number of samples %d, accuracy (perc.) (%.2f)' % ( num_samples, 100 * acc))

Thank you Markus. In the refactored version I'll change it to
accuracy = (y == preds).float().mean(), averaging over all samples in the batch and all pixels. I'll merge the refactored version (currently on branch batchai) in the coming weeks.

Merged.


Updated readme

@aloisbarreras @beckysteele

Current coverage is 100% (diff: 100%)

Merging #7 into master will not change coverage

@@           master    #7   diff @@
===================================
  Files           1     1
  Lines          26    26
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           26    26
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 038242f...002fc90

Wrong repo


Custom Validators

I am trying to create a custom validator to test if passwords match when a user is signing up.
confirmPasswordValidator(value, field, model) {     let pass = model.password;     let confPass = value;     if (pass != confPass) {         return ["Passwords don't match."];     } else {         return [];     } };
When I submit my form I have a submit button created like this:
{                                 type: "submit",                                 buttonText: "Register",                                 onSubmit: this.onValidated,                                 validateBeforeSubmit: true                             }
The form submits even though the password fields do not match.
{                                 type: "input",                                 inputType: "password",                                 label: "Password",                                 model: "password",                                 min: 6,                                 required: true,                                 hint: "Minimum 6 characters",                                 validator: VueFormGenerator.validators.string                             }
                             {                                 type: "input",                                 inputType: "password",                                 label: "Confirm Password",                                 model: "confirmPassword",                                 min: 6,                                 required: true,                                 hint: "Minimum 6 characters",                                 validator: this.confirmPasswordValidator                             }
How do I stop the form from submitting when there are errors?

See this example
https://jsfiddle.net/0mg1v81e/695/
Add v-on:submit.prevent modifier on your form to prevent submit and make sure your data is a function so this.onValidated will call the appropriate function

I will try that out later tonight.  Thanks.  I will let you know if it did not work.


Fixed :meth:.CoordinateSystem.get_area when using few plot points and a boundary graph

Overview: What does this pull request change?

See title.

Motivation and Explanation: Why and how do your changes improve the library?

Code example:
class GraphPoints(Scene):
    def construct(self):
        ax = Axes(x_range=[-2, 2], y_range=[-2, 2], color=WHITE)
        f1 = ax.plot(lambda t: t, [-1, 1, 0.5])
        f2 = ax.plot(lambda t: 1, [-1, 1, 0.5])
        a1 = ax.get_area(f1, [-1, 0.75], color=RED)
        a2 = ax.get_area(f1, [-0.75, 1], bounded_graph=f2, color=GREEN)

        self.add(ax, f1, f2, a1, a2)

Before changes:

After changes:

Links to added or changed documentation pages

Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Can confirm that this was broken and is fixed by this PR. Cannot say I fully understand the logic switch behind the bounded graph after a very brief glance at the code... but the issue seems to be only the former part for if bounded_graph is None: and the change on 1296 seems like the missing piece of logic.

Thank you for reviewing!
The changed logic basically generates two sets of points (which serve as the boundary for the area polygon); one set of points along the original graph starting with (a, f(a)), then choosing all "existing" points on the graph, up until (b, f(b)), and one set of points for the boundary graph. The points are then assembled in the correct order to become the boundary points of the area polygon (travelling along the points of the original graph, and travelling back along the points of the boundary in reverse order).
The logic that was missing in both the bounded graph case as well as the normal case was that the points (a, f(a)), (b, f(b)) have not been added.


⚠ Bofrost.de has degraded performance

In 2f314b6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 931 ms

Resolved: Bofrost.de performance has improved in cf14ad9 after 6 minutes.


Down button not working in :Tcd

Fel upptäcktes vid bearbetning av function remote#define#CommandBootstrap[5]..remote#define#request:
rad    2:
error caught in request handler '/Users/david/dev/VimPlugs/bolt.nvim/rplugin/python3/vim_tc_explorer:command:TcExpDown
[[], [1, 1]]':
Traceback (most recent call last):
File "/Users/david/dev/VimPlugs/bolt.nvim/rplugin/python3/vim_tc_explorer/init.py", line 35, in tc_down
self.TcExplorer.tc_down(args, range)
File "/Users/david/dev/VimPlugs/bolt.nvim/rplugin/python3/vim_tc_explorer/vim_tc_explorer.py", line 216, in tc_down
exp.window.cursor = (exp.selected + exp.headerLength, 0)
AttributeError: 'explorer' object has no attribute 'window'

To reproduce

nvim
:Tcd
Press key Down

The error was introduced in 8f83f70

Should be easy to fix, will give it a go.

Solved in cd1f57b


Eclipse Temurin JRE 17 upgrade issue

Please provide a brief summary of the bug
Upgrading from Eclipse Temurin JRE 17.0.4+8 to <IP_ADDRESS>+1 does not uninstall previous version.
Please provide steps to reproduce where possible

install OpenJDK17U-jre_x64_windows_hotspot_17.0.4_8.msi
install OpenJDK17U-jre_x64_windows_hotspot_<IP_ADDRESS>_1.msi
check programs and features

Expected Results
upgrading removes previous (n-1) version
Actual Results
upgrading does not remove previous (n-1) version
What Java Version are you using?
<IP_ADDRESS>, <IP_ADDRESS>
What is your operating system and platform?
Windows 10 21H2
How did you install Java?
OpenJDK17U-jre_x64_windows_hotspot_17.0.4_8.msi
OpenJDK17U-jre_x64_windows_hotspot_<IP_ADDRESS>_1.msi
Did it work before?
yes, this worked with prior versions

Did you test with other Java versions?
issue only occurs when upgrading from 17.0.4 to <IP_ADDRESS>

Relevant log output
No response

I believe this is a limitation of server as outlined in the installation docs (https://adoptium.net/installation/windows)

Ahh, thank you for pointing that out.


Question: transluscent vs transparent

It seems like setting the color to transluscent also changes some underlyign behavior in how the content behind it is rendered, while transparent doesn't do the same. Is this expected behavior and is it possible to make transparent work like transluscent in that it is rendered on top of the content behind it?

Hey @mekwall, this is a behavior that is expected to change the screen size when the color is set to translucent. You can see that React Native behaves the same way by using <Status Bar translucent={true} backgroundColor={"transparent"} /> in its own StatusBar. I think this is actually a situation related to React Native.

Thanks for the quick reply @kadiraydinli! It's kinda what I figured. Would it be possible to add an option to act like translucent but for all colors? So, that when you use a color with transparency the content behind would be partly visible.

@mekwall As far as I have researched, it seems that it is not possible to make bar colors both translucent and colored. If you can find any examples where this has been done, it would be great if you could share it with me.

@mekwall As far as I have researched, it seems that it is not possible to make bar colors both translucent and colored. If you can find any examples where this has been done, it would be great if you could share it with me.

thanks your compent， now i can use like facebook，  homep page transparent and other page with translucent。。。

not tranparent。。。

just setting：
SystemNavigationBar.setNavigationColor('translucent');  or
SystemNavigationBar.setNavigationColor('transparent');
in your case， setting ，transparent in the navigationbar， and other compent with translucent color will do the same， i think

but a problem， it seems the     stautsBar also become transparent no matter what it setting，  when setting the navigationbar to transparent。。。  but its not a big problem

I'm seeing the same issue on my end. Using setNavigationColor('transparent', 'light', 'navigation') will also make the status bar transparent and will actually push my content under the camera notch. When I use the React Native StatusBar component to do the same thing, the content is not being pushed under the camera. All I am trying to do is to set the background of the bottom navigation to transparent so that it looks better (rather than having a slight dark opacity)


Fixed typo in README.md.

I consider fixing this typo important, because without fixing it it may be a bit hard to understand what exactly your software does.

Pull Request Test Coverage Report for Build 59

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.644%

Totals

Change from base Build 58:
0.0%

Covered Lines:
288

Relevant Lines:
298

💛  - Coveralls

Thanks for your contribution. I tried to improve the README in 039aebf5538d38a0eb2770f92bff50f71f01411f.
Please let me know if it’s better to understand.


Use of Claim variables in rewrite / proxypass

On CentOS / mod_auth_openidc 2.1.2 I can't use claim* variables in rewrite / proxypass. It's really strange as the claim_* variables are available in a (test) cgi but not during rewrite.
Relevant config entries:
OIDCScope "openid name email profile"
RewriteEngine On
RewriteMap lookup "txt:/var/my/lookup.txt"
RewriteRule ^ - [E=MYPORT::${lookup:%{HTTP:OIDC_CLAIM_specific_user_metadata}}]
ProxyPassInterpolateEnv On
ProxyPass "/" "http://<IP_ADDRESS>${MYPORT}/" interpolate
ProxyPassReverse "/" "http://<IP_ADDRESS>${MYPORT}/" interpolate
The rewritemap logic works, have verified that multiple times.
Many thanks for reading this. Otherwise mod_auth_openidc works like a charm for me.

I believe the headers may not be available in this step, but the environment variables should be, which should be available by default, see: https://github.com/pingidentity/mod_auth_openidc/blob/master/auth_openidc.conf#L598

see also this thread: https://github.com/pingidentity/mod_auth_openidc/issues/76

Thanks for the tips and quick reply! It does not appear to be set, even if setting OIDCPassClaimsAs environment
Read through the the recommended thread but AFAIK all that happens after rewrite/proxypass does its thing. Any other directions appreciated.

I'm pretty sure they would be available (from other threads like https://groups.google.com/forum/#!topic/mod_auth_openidc/d3mFE4V4Ujs), certainly for ProxyPass even the headers are available, but I would have to reproduce this to verify.
Remember the environment variable would be called %{OIDC_CLAIM_specific_user_metadata} not %{HTTP:OIDC_CLAIM_specific_user_metadata}

I am able to see mod_rewrite do the right thing, i.e. set the MYPORT variable, when I configure:
RewriteEngine On
RewriteMap lookup "txt:/var/my/lookup.txt"
RewriteRule ^ - [E=MYPORT:${lookup:%{ENV:OIDC_CLAIM_email}}]

Note that you have an superfluous : in your expression.
I'm not able to make this work with ProxyPass though but I assume that is up to the interaction between those modules, not mod_auth_openidc.

yes @zandbelt that's my conclusion as well. Thanks for your feedack! However even though you can proxy with the rewriteengine, you would need proxypass. So I resorted to a workaround but it's fine, mod_auth_openidc does take care of the heavy lifting. Cheers!

I am trying to mimic the working example in Zandbelt's last comment but no success so far. I have one more step to set the Authorization header (base64 ignored for now). Here is the config I have:
    <Location /oauth2/>
            AuthType openid-connect
            Require valid-user
            ProxyPass  http://my.host.url/ nocanon
            ProxyPassReverse http://my.host.url/

            RewriteRule ^ - [E=AUTHNXP:%{ENV:OIDC_CLAIM_sub}]
            RequestHeader set Authorization "Basic %{AUTHNXP}e"
    </Location>

TCPDump shows:             Authorization: Basic
without the value of ENV:OIDC_CLAIM_sub. But I can see the http header OIDC_CLAIM_sub in the tcpdump with the right value.
My goal is to get the value of ENV:OIDC_CLAIM_sub, base64 encode it and set the result to Authorization header and forward to backend.
Any suggestion where I am doing wrong?
Thanks,

Well, I mentioned in that last comment "I'm not able to make this work with ProxyPass" so that would be it. I also don't see how you would be able to apply base64 encoding to it.

Sorry I missed that part.
The base64 part can be done with a rewriterule like:
RewriteMap base64map "prg:/usr/bin/base64"
RewriteRule .* - [E=AUTHN:${base64map:%{LA-U:REMOTE_USER}:%{LA-U:REMOTE_PASS}},NE]
as claimed here: http://serverfault.com/questions/647591/apache-as-authentication-proxy
Thanks for the quick response.

I was able to do this using:
RewriteMap base64map "prg:/tmp/base64"
RewriteRule ^ - [E=AUTHNXP:${base64map:%{HTTP:OIDC_CLAIM_sub}}]
RequestHeader set Authorization "Basic %{AUTHNXP}e"

I needed a special script that turns off buffering:
#!/usr/bin/perl
use MIME::Base64 qw(encode_base64);
$| = 1;
print encode_base64(<STDIN>);

Something interesting happened. I used your base64 script and configuration as:
            RewriteEngine on
            RewriteMap base64map "prg:/usr/local/bin/base64"
    <Location /oauth2/>
            AuthType openid-connect
            Require valid-user
            RewriteRule ^ - [E=AUTHNXP:${base64map:%{HTTP:OIDC_CLAIM_sub}:}]
            RequestHeader set Authorization "Basic %{AUTHNXP}e"
            ProxyPass  http://my.gerrit.host:8080/ nocanon
            ProxyPassReverse http://my.gerrit.host:8080/
    </Location>

    AllowEncodedSlashes On
    ProxyPass /  http://my.gerrit.host:8080/ nocanon
    ProxyPassReverse /  http://my.gerrit.host:8080/

With the above configuration, I DID get the right Authorization header sent from Apache to my Gerrit listening on port 8080. However, somehow this rewriting rule only works once per Apache session. If I run my test the second time, I will not have the right Authorization header, only see empty 'Authorization Basic'. If I restart Apache, I will observe the exactly above. (You may think this should work for me at least for the first user, but not since somehow Apache needs to forward two requests, one to '/', the other to '/login/' of gerrit. And the first one to '/' will contain the right header but the second one to 'login' will not due to above mentioned rewriting-rule-only-work-once problem. But it looks like it can work with ProxyPass at least once.
Any comments, ideas?

Interestingly enough I observe the same but the problem seems to be in the mapping rather than in the interaction between mod_rewrite and mod_auth_openidc.
In my case I can see that the 2nd time around the mapping is called (again) with the right key, but no value is returned. So the input obtained from the headers is OK. Also when I use a simple lookup table i.e RewriteMap base64map "txt:/tmp/lookup.txt" it all works reliably. You should be able to confirm that as well by setting LogLevel rewrite:trace8
If confirmed it should be a matter of finding the right tool/script to do the base64 conversion. Mine has a few problems with buffering of stdin/stdout.

@zandbelt You are absolutely correct that it is the script. Here is what I have and it works for me now in the context of ProxyPass:
#!/usr/bin/perl
use MIME::Base64 qw(encode_base64);
$| = 1;
while($a = ) {
print encode_base64($a);
}
The script starts once per Apache session and is used by the following rewrite rule calls. Hence a 'while' look did the trick.
Thank you very much for all the help!


Add touch support to graph hover

📝 Changed routes:

/[slug]

(view full size)

Commit 2f62e9573948f0b3a6efb8068772ac0325902b34 (https://routes-fead5wie0-samuelkraft.vercel.app).


Fixing warnings and code cleanup.

Everything seems to be working as expected. Still need to look into other dynamic method versions to see which one speeds things up.
Benchmark is about 3/3.5x slower than similar. Need to speed things up.
Need to add more benchmarks as well.


Provide the event consumer more information about the event

Providing the event object to the event consumer will make the user have more flexibility.
ex. I use the event object to get whether the end user has pressed shift keyboard key when he choose an entry, this way I can decide whether to open the selection to a new tab or to navigate current page.
onSelection: feedback => {
  if (feedback.event.shiftKey) {
    window.open(url);
  }else {
    window.location.href = url;
  }
}

Hello @Liano,
This is a good addition actually, thanks for your highly valuable contribution.
v8.3.0 Github & CDN
Have a nice day, cheers! :)

Hello @Liano,
This is a good addition actually, thanks for your highly valuable contribution.
v8.3.0 Github & CDN
Have a nice day, cheers! :)

You are welcome @TarekRaafat I am happy I could help.
Have a nice weekend.

You are welcome @TarekRaafat I am happy I could help.
Have a nice weekend.


Add two columns in FI's view of all applications

Closing as duplicated https://github.com/open-contracting/credere-backend/issues/398


[BUG] Shortcut for switching markdown editor doesn't work

OS: Windows
Extension Version: 3.3.6

The button for this action works, but shortcut does not.

Here's a pull requrest for a solution that hopefully works; although i've not build vscode-office to test it
https://github.com/cweijan/vscode-office/pull/324


Fix section tree in navigation bar for missing translated documentations

This fixes section tree in a navigation bar of translated documentations.
Ideally, if a part of sections is translated, they and missing translations (EN docs) must be listed on same section tree.
The current document creates an additional section tree for translation, but the original section tree may remain ("Advanced User Guide" is translated into "高度なユーザーガイド" on the just bottom of "Advanced User Guide").

This happens where missing "index.md" but other subsections is translated.
Here improves logic for collecting "nav" section name.
(Sorry for inconvenience that my example is Japanese translation because I can read only that 😢 .

Docs previewe shows successfully as follows ("Advanced User Guide" is unified with "高度なユーザーガイド" (translation for "Advanced User Guide")):

Docs previewe shows successfully as follows ("Advanced User Guide" is unified with "高度なユーザーガイド" (translation for "Advanced User Guide")):

Thank you @tokusumi! I updated how the docs are generated somewhat recently, and now that there's no content in any additional mkdocs.yml, and with the new setup structure, this is fixed. 🚀
Given that, I'll now close this one. But thanks for all your help with the translations! 🍰


Move to asyncio.timeout from async_timeout

New minimum requirement HA 2023.8 and Python 3.11

Wait for release of HA 2023.8.0 before making this change.


Update README.md

Update title to Monaco Demo
Minor formatting, spelling, and grammar adjustments

Much appreciated!


[Compile]: Compilation Issue with Connectivity_plus pluggin For Bonsoir Dependency

Because no versions of cast match >2.0.0 <3.0.0 and cast 2.0.0 depends on bonsoir ^3.0.0, cast ^2.0.0 requires bonsoir ^3.0.0.
And because bonsoir ^3.0.0 depends on bonsoir_platform_interface ^3.0.0, cast ^2.0.0 requires bonsoir_platform_interface ^3.0.0.
Because bonsoir_platform_interface 3.0.0 depends on connectivity_plus >=4.0.0 <4.1.0 and no versions of bonsoir_platform_interface match >3.0.0 <4.0.0, bonsoir_platform_interface ^3.0.0 requires connectivity_plus >=4.0.0 <4.1.0.
Thus, cast ^2.0.0 requires connectivity_plus >=4.0.0 <4.1.0.

Hello,
I've release a new version 2.1.0 without the upper limit for bonsoir.

Thank you for the quick response. It matters a lot.


lib: improve error message

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (516e575) 81.93% compared to head (d987c5a) 82.55%.
Report is 1 commits behind head on master.

Files
Patch %
Lines

config/profile.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #920      +/-   ##
==========================================
+ Coverage   81.93%   82.55%   +0.61%
==========================================
  Files          46       46
  Lines        3870     3869       -1
==========================================
+ Hits         3171     3194      +23
+ Misses        568      544      -24
  Partials      131      131

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


parse: use same algorithm that reflect.Lookup to detect wire tags #179

Fixes #179
This fix replaces tag detection using the same algorithm of reflect.Lookup into strings.Contains .

Hi,
I was just about tackling this issue, too. But then I saw you already submitted a pull request which is awesome!
I just wonder, couldn't the same behaviour be achieved by changing the isPrevented method to the approach below?
import "reflect"
...

// isPrevented checks whether field i is prevented by tag "-".
// Since this is the only tag used by wire, we can do string comparison
// without using reflect.
func isPrevented(tag string) bool {
	return reflect.StructTag(tag).Get("wire") == "-"
}

I might be wrong, but the tests still succeed.
The only disadvantage I see is the import of the reflect package. But this shouldn't matter I think because parse.go imports fmt which imports reflect anyways.
The benefit would be that all future improvements to StructTag.Get would be automatically applied here.
Why did you decide to copy the method from the reflect.StructTag type instead of reusing it?

@jotdl Thank you for your reviewing. I just did the same approach you said but I thought whether this comment might have been special meaning.
https://github.com/google/wire/blob/ed1f30e1e8cc03225f15f4bb986f870c4c6dd394/internal/wire/parse.go#L865
Unless there are particular reason, I greate agree with your approach and I immediately fix using reflect.
What do you think @shantuo ?

Hi @u5surf and @jotdl , thanks for the contribution!
I guess this one was my fault. I didn't know the shortcut of doing reflect.StructTag conversion while I wrote down the comment. I think doing what @jotdl suggest is fine.

@shantuo: Thanks for your response! Great to hear that this looks fine.
@u5surf: Do you want to modify your pull request or should I submit a new one? What do you prefer?

@shantuo Thanks your comments, I certainly understand that comment is unintentional.
@jotdl  I’ll modify that


There may be a decoding problem

One device out of three shows an incorrect local key:
Local Key = ts<+N?_PN(_EDu+m
What can you do to find the correct key?

The IoT website has the same information.

What makes you think it's incorrect?  They switched to including symbols a few months ago https://github.com/jasonacox/tinytuya/discussions/341


Add support for the Board category

Add support for all endpoints in the Board category.

Support for the Board category was implemented in #2.


Mono - Missing constructor or Method for Quat

with a fresh godot 3 with mono from master.
In gdscript, there is a constructor (or method) for Quat to "convert" basis to Quat. It can be found in the documentation: http://docs.godotengine.org/en/latest/classes/class_quat.html?highlight=Quat
It's the third method..
Quat | Quat ( Basis from )
example: 	var this_rotation = Quat(get_transform().basis) - Work like a charm in GD script !
Maybe I'm just on the wrong path but I can't find the way to do the same in C#.. there is no constructor or method who take a basis parameter..
I can't do something like this in C#..
Transfrom t = GetTransform();
Quat thisRotation = new Quat(t.basis); -- Won't work.. and there is no methods either for this.
It's a missing feature or i'm just wrong ? (I can be.. I'm just a human after all :D)

There's a Quat() method in the Basis struct that you can use for this:
 Quat thisRotation = GetTransform().basis.Quat();

This is not a constructor but an implicit conversion (the documentation displays it as a copy constructor). I didn't add them to the C# code yet. I plan to add those as explicit operators (that Quat() should be renamed). The only exception is NodePath < --- > String which is an implicit operator.

@neikeq what should we do with this?


crash in android

java.lang.IncompatibleClassChangeError: Found interface com.google.android.gms.location.SettingsClient, but class was expected (declaration of 'com.google.android.gms.location.SettingsClient' appears in /data/app/~~n6Huw2QsDW-xwP2Tie6VxQ==/com.azooz.driverr-OKSmBNpK8FV8oWawPK30Hw==/base.apk)

Any update on this, I am also facing the same issue. Let me know if any work around is there.

check fl_location: ^2.0.0


UI Issue - Raidable Realms not showing

[ ] Filtering to "Raidable" realms on the atlas does not show the realms that are raidable only non-raidable realms.

@ponderingdemocritus

We need to now search through all the Balances of the resources generated - then return asc order of these.
@tenoxious you have a good understanding of what needs to be done?

have added a basic filter in #327
You can now filter by raidable but it does not order them by total amount - this might require a bit more thought, but it is an improvement over what currently exists


Wrong default password on documentation

On the Nugget packaging, it states the Wi-Fi password to access the web interface is "password123" but in the current binary, it's password.
Update the binary with the right password?

This has been fixed - update to the latest .BIN file and the default password should match the password on the packaging


Implement metasync.firstOf

metasync.firstOf([f1...fN], (err, args...) => {
...
});

FYI: @aqrln @belochub

@tshemsedinov, could you explain what that function has to do, please?

I think it is interesting control flow utility to catch which function returns first. It will be also errback compatible and need two implementations: parallel and sequential. @DzyubSpirit

@tshemsedinov Could you explain how to do that function sequential? As I understand 'sequential' means second function executes after first callback has been called. It it's true then purpose of firstOf will be lost

@DzyubSpirit agree, just parallel :)


Report: Community Partners - Data Discrepency

The numbers in this report don't seem to be adding up - compare the following screenshots:

@EdemD  QA successfully tested both Public and private report.

https://docs.google.com/spreadsheets/d/1nNtAaIzJT-1kUt7YJrOfPcUxFAu2ojQcYqN11wgq_Eo/edit#gid=547627265


Number of input channels

Hi there,
I'm trying to reproduce the pre-training of the SatClip based on S100 datset. I downloaded S100 and changed the paths in the config file default.yaml and in s2geo_dataset.py.
Now, this is the output error that I'm trying to solve:
using vision transformer
GPU available: True (cuda), used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/loops/utilities.py:73: `max_epochs` was not set. Setting it to 1000 epochs. To train without an epoch limit, set `max_epochs=-1`.
skipped 8142/100000 images because they were smaller than 10000 bytes... they probably contained nodata pixels
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0]

  | Name     | Type        | Params
-----------------------------------------
0 | model    | SatCLIP     | 88.9 M
1 | loss_fun | SatCLIPLoss | 0
-----------------------------------------
88.9 M    Trainable params
0         Non-trainable params
88.9 M    Total params
355.445   Total estimated model params size (MB)
SLURM auto-requeueing enabled. Setting signal handlers.
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0]

  | Name     | Type        | Params
-----------------------------------------
0 | model    | SatCLIP     | 88.9 M
1 | loss_fun | SatCLIPLoss | 0
-----------------------------------------
88.9 M    Trainable params
0         Non-trainable params
88.9 M    Total params
355.445   Total estimated model params size (MB)
SLURM auto-requeueing enabled. Setting signal handlers.
Sanity Checking DataLoader 0:   0%|                                                                                | 0/2 [00:00<?, ?it/s]Traceback (most recent call last):
  File "/data/eplekh/code/satclip/satclip/main.py", line 159, in <module>
    cli_main(config_fn)
  File "/data/eplekh/code/satclip/satclip/main.py", line 144, in cli_main
    cli.trainer.fit(
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 544, in fit
    call._call_and_handle_interrupt(
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/call.py", line 44, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 580, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 987, in _run
    results = self._run_stage()
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 1031, in _run_stage
    self._run_sanity_check()
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/trainer.py", line 1060, in _run_sanity_check
    val_loop.run()
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/loops/utilities.py", line 182, in _decorator
    return loop_run(self, *args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/loops/evaluation_loop.py", line 135, in run
    self._evaluation_step(batch, batch_idx, dataloader_idx, dataloader_iter)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/loops/evaluation_loop.py", line 396, in _evaluation_step
    output = call._call_strategy_hook(trainer, hook_name, *step_args)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/trainer/call.py", line 309, in _call_strategy_hook
    output = fn(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/lightning/pytorch/strategies/strategy.py", line 412, in validation_step
    return self.lightning_module.validation_step(*args, **kwargs)
  File "/data/eplekh/code/satclip/satclip/main.py", line 75, in validation_step
    loss = self.common_step(batch, batch_idx)
  File "/data/eplekh/code/satclip/satclip/main.py", line 66, in common_step
    logits_per_image, logits_per_coord = self.model(images, t_points)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/data/eplekh/code/satclip/satclip/model.py", line 365, in forward
    image_features = self.encode_image(image)
  File "/data/eplekh/code/satclip/satclip/model.py", line 358, in encode_image
    return self.visual(image.type(self.dtype))
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/data/eplekh/code/satclip/satclip/model.py", line 230, in forward
    x = self.conv1(x)  # shape = [*, width, grid, grid]
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 460, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/eplekh/.local/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 456, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Given groups=1, weight of size [768, 4, 32, 32], expected input[64, 13, 256, 256] to have 4 channels, but got 13 channels instead

It seems the images are found, but somehow the CNN expects 4 channels and get 13, and I'm not sure why.
I tried to change in the ./satclip/configs/default.yaml the row
in_channels: 4
to
in_channels: 13,
but this did not help. Also the file
"/data/eplekh/code/satclip/lightning_logs/version_14077673/./configs/default-latest.yaml" that is created while running the script contains
in_channels: 4
despite I changed the ./satclip/configs/default.yaml.
This might be the reason, but I don't know how to fix it.
Small additional question: is "LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0]" okay output or does it mean that it does not see the GPU?
Would very much appreciate any help,
Kind rehards,
Elena

Are you using the default.yaml from here: https://github.com/microsoft/satclip/blob/main/satclip/configs/default.yaml?
You'd need to change in_channels to 13, then it should run. If you want to use a pretrained vision encoder you need to change vision_layer to e.g. moco_resnet50.
For more details on the vision encoders and how they are used check: https://github.com/microsoft/satclip/blob/main/satclip/model.py

Thanks for your reply!
Changing parameters in default.yaml didn't help, but changing them in the class SatCLIPLightningModule of main.py script actually did.
So it's running now, thank you!


I don't believe the requirements are being installed

`pi@raspberrypi:~/internet-pi $ sudo ansible-galaxy install -r ./requirements.yml -vvv
ansible-galaxy 2.9.23
config file = /home/pi/internet-pi/ansible.cfg
configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/lib/python2.7/dist-packages/ansible
executable location = /usr/bin/ansible-galaxy
python version = 2.7.16 (default, Oct 10 2019, 22:02:15) [GCC 8.3.0]
Using /home/pi/internet-pi/ansible.cfg as config file
Reading requirement file at './requirements.yml'
pi@raspberrypi:~/internet-pi $ ansible-playbook main.yml
ERROR! couldn't resolve module/action 'community.docker.docker_compose'. This often indicates a misspelling, missing collection, or incorrect module path.
The error appears to be in '/home/pi/internet-pi/tasks/internet-monitoring.yml': line 25, column 3, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:
to the docker group, so this task may fail.

name: Ensure internet-monitoring environment is running.
^ here
`

Following the Setup instructions, you need to install the requirements: ansible-galaxy install -r requirements.yml — note that you need to run Ansible 2.9 or later for this playbook and the collections it needs to work.

The command shown in the installation guide doesn't match with the one used in the video. The command is missing its type (collection) and should be
ansible-galaxy install -r requirements.yml

I am running Ansible 2.9:
@.:/home/pi/internet-pi# ansible --version
ansible 2.9.23
config file = /home/pi/internet-pi/ansible.cfg
configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/lib/python2.7/dist-packages/ansible
executable location = /usr/bin/ansible
python version = 2.7.16 (default, Oct 10 2019, 22:02:15) [GCC 8.3.0]
@.:/home/pi/internet-pi# ansible-galaxy install -r requirements.yml
@.***:/home/pi/internet-pi#
I run this cammand, but nothing gets installed:
@.:/home/pi/internet-pi# ansible-galaxy install -r requirements.yml
@.:/home/pi/internet-pi#

On Jun 23, 2021, at 12:34 PM, Gerrit Wellen @.***> wrote:
ansible-galaxy install -r requirements.yml

@gerritwellen - If running Ansible 2.10 or later (I use 4.0), you can drop the collection from the command and it will install all collections and roles correctly.

@mikev1963 - Please try running ansible-galaxy collection install -r requirements.yml as @gerritwellen describes. That will work with 2.9, I think. I keep forgetting that 2.9 has a lot of annoying deficiencies for using collections.

That worked.  I was following your readme in this repo.  Thank you .  Working  great.


WKYoutubePlayer layout is wrong in landscape mode

This issue is not consistent. But happens very frequently while repetitive rotation of device.
On rotation from portrait to landscape, player keeps having portrait frame, occupies 1/4 part of the screen and has alignment of top left.
Frame of player(WKWebView) has correct frame, but several subviews(WKCompositingView) has wrong frames(smaller than player's frame)
In my code I'm using layout constraints for making layout and not using autoResizingMasks:
view.addSubview(player)
player.translatesAutoresizingMaskIntoConstraints = false
NSLayoutConstraint.activate([
    player.topAnchor.constraint(equalTo: view.topAnchor),
    player.bottomAnchor.constraint(equalTo: view.bottomAnchor),
    player.leadingAnchor.constraint(equalTo: view.leadingAnchor),
    player.widthAnchor.constraint(equalTo: player.heightAnchor, multiplier: 16 / 9)
])
Issue is identified on both Simulator and iPhone devices on iOS 12.x.x-13.x.x.

Issue was fixed by modifying window.onresize method in YTPlayerView.html template:
window.onresize = function() {         setTimeout(function() { player.setSize(window.innerWidth, window.innerHeight) }, 100);     }

Thank you for your solve code, It's bug happend very often when I try device from portrait to land scape, I'll fix on v1.1.0


proxy: use a real proxy conn id

What problem does this PR solve?

Issue Number: close #345
Problem Summary: Pass a real connection id.
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

/retest


plugin: streamlive.to - how to get stream. please help me

i use command: livestreamer [Url] best -o "D:\test.mp4" but livestreamer can;t get link

When a streamlive.io stream is playing, in firefox hit F12, then goto network tab, type .m3u8 in the filter box, this will show the source url of stream. right click to copy URL
the source url can be played by simply > vlc 'http://sourceurlinquotes.m3u8'
: Note this method does not use any livestreamer/streamlink plugins.
i strongly suggest you ditch livestreamer in favour of the new improved streamlink