[test-framework] Make helm values configurable

fixes: #9207

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nmittler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
ebb99d9f58499d97a7d4868451ccdda425df9d1e
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@nmittler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d2b83c5377d79009045ed8b576d24f44bd8ec4c
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nmittler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
7d2b83c5377d79009045ed8b576d24f44bd8ec4c
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Task/sidre data info

Summary

This PR adds methods to get info about Sidre contents in a conduit Node
Specifically, adds method to DataStore class to get Buffer info and adds method to Group class to get info about a Group subtree
Tests of methods re included.

Putting this up as a draft PR for discussion to see what folks think. Good enough for now? Add more? This stinks, start over!?  When the dust settles on the feature, I will update changelog and add user docs.
Addresses https://github.com/LLNL/axom/issues/297

This looks like a good way to accumulate information to track data usage, especially for users to check if buffers are being created and then abandoned.

This looks like a good way to accumulate information to track data usage, especially for users to check if buffers are being created and then abandoned.

@nselliott can you think of any other information to track that would also be useful? I wanted some feedback before I went too far with this.

Folks, this PR is ready for final review. I've addressed all review comments received so far, added user docs, and updated the changelog.

My comments are all minor and not blocking. Feel free to merge.

My comments are all minor and not blocking. Feel free to merge.

I believe I addressed them all.

ping! folks please review this and comment or approve. thank you.

ping! folks please review this and comment or approve. thank you.

I will review this today.  Apologies for the delay.

@kennyweiss I believe I've addressed all of your review comments.


Hi there, I`m AEX-O. I did Installed P-ify but when I start my phone he stays home black.

Description

Steps to reproduce

ROM info

Module version (build version)

Xposed log / logcat

give

Please include all information specified in the issue template.


Update scalafmt-core to 2.7.2

Updates org.scalameta:scalafmt-core from 2.1.0 to 2.7.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-minor

Superseded by #352


vscode-extensions: clean up

Description of changes
A continuation of #223308

Removes extensions that have vanished from the marketplace or been deprecated
Renames an extension that moved
Some more cleanup for consistency

During the creation of this PR, I noticed a few inconsistently capitalized extensions. I could attempt to fix them in this PR, a new PR, or just leave them be:

Arjun.swagger-viewer (-> arjun.swagger-viewer)
ms-vscode.PowerShell (-> ms-vscode.powershell)
rioj7.commandOnAllFiles (-> rioj7.commandonallfiles)
WakaTime.vscode-wakatime (-> wakatime.vscode-wakatime)

I also noticed that some aliases throw while others just point to the correct name. Not sure why.
I also noticed that elixir-lsp.vscode-elixir-ls is actually jakebecker.elixir-ls (which is currently an alias), not the other way around. I assume there is a good reason for this, as per the comment at the top of the file.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release 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

[X] Fits CONTRIBUTING.md.

These are correct:

Arjun.swagger-viewer
ms-vscode.PowerShell
rioj7.commandOnAllFiles
WakaTime.vscode-wakatime
Notice: Upstream provides names in uppercase.

As upstream provides names in uppercase, these are correct:

Arjun.swagger-viewer
ms-vscode.PowerShell
rioj7.commandOnAllFiles
WakaTime.vscode-wakatime

In that case, these are inconsistent instead:

angular.ng-template -> Angular.ng-template
antyos.openscad -> Antyos.openscad
azdavis.millet -> azdavis.Millet
b4dm4n.vscode-nixpkgs-fmt -> B4dM4n.nixpkgs-fmt (the name is different from source as well)
bbenoist.nix -> bbenoist.Nix
davidanson.vscode-markdownlint -> DavidAnson.vscode-markdownlint
divyanshuagrawal.competitive-programming-helper -> DivyanshuAgrawal.competitive-programming-helper
editorconfig.editorconfig -> EditorConfig.EditorConfig
elmtooling.elm-ls-vscode -> Elmtooling.elm-ls-vscode
golang.go -> golang.Go
graphql.vscode-graphql -> GraphQL.vscode-graphql
gruntfuggly.todo-tree -> Gruntfuggly.todo-tree
hookyqr.beautify -> HookyQR.beautify
ibm.output-colorizer -> IBM.output-colorizer
ionide.ionide-fsharp -> Ionide.Ionide-fsharp
james-yu.latex-workshop -> James-Yu.latex-workshop
and so on

As upstream provides names in uppercase, these are correct:

Arjun.swagger-viewer
ms-vscode.PowerShell
rioj7.commandOnAllFiles
WakaTime.vscode-wakatime

In that case, these are inconsistent instead:
* `angular.ng-template` -> `Angular.ng-template`

* `antyos.openscad` -> `Antyos.openscad`

* `azdavis.millet` -> `azdavis.Millet`

* `b4dm4n.vscode-nixpkgs-fmt` -> `B4dM4n.nixpkgs-fmt` (the name is different from source as well)

* `bbenoist.nix` -> `bbenoist.Nix`

* `davidanson.vscode-markdownlint` -> `DavidAnson.vscode-markdownlint`

* `divyanshuagrawal.competitive-programming-helper` -> `DivyanshuAgrawal.competitive-programming-helper`

* `editorconfig.editorconfig` -> `EditorConfig.EditorConfig`

* `elmtooling.elm-ls-vscode` -> `Elmtooling.elm-ls-vscode`

* `golang.go` -> `golang.Go`

* `graphql.vscode-graphql` -> `GraphQL.vscode-graphql`

* `gruntfuggly.todo-tree` -> `Gruntfuggly.todo-tree`

* `hookyqr.beautify` -> `HookyQR.beautify`

* `ibm.output-colorizer` -> `IBM.output-colorizer`

* `ionide.ionide-fsharp` -> `Ionide.Ionide-fsharp`

* `james-yu.latex-workshop` -> `James-Yu.latex-workshop`
  and so on

:-(
What should we do about this?

I suggest turning the four I mentioned into lowercase, adding aliases for their uppercase versions, and documenting this de-facto standard in the comment at the top of the file.

I suggest turning the four I mentioned into lowercase, adding aliases for their uppercase versions, and documenting this de-facto standard in the comment at the top of the file.

I suggest turning the four I mentioned into lowercase, adding aliases for their uppercase versions, and documenting this de-facto standard in the comment at the top of the file.

Suggestion accepted.

Apparently #222171 added the throwing aliases just a week ago. Maybe they should be turned into proper aliases instead of errors?

Jeez. I should have squashed this. But won't revert.


is this really MIT?

git is GPLv2 https://github.com/git/git/blob/master/COPYING is this project truly an independent implementation that has not been based on the viewing of any git code?

hmm, the https://libgit2.github.com/ website says "GPLv2 with Linking Exception." but I don't see how that is legally possible without the permission of all git contributors (or a single copyright authority such as Linus).

oh doh, this is the website repo isn't it.


Release of version <IP_ADDRESS> of xSQLServer

Releasing version <IP_ADDRESS>

This change is

@kwirkykat This should be version <IP_ADDRESS> because there is breaking changes. Have I missed adding BREAKING CHANGE to PR's?

@johlju You're right. It should be <IP_ADDRESS>. I'll fix it

Codecov Report

Merging #683 into master will increase coverage by 16%.
The diff coverage is 99%.

@@          Coverage Diff           @@
##           master   #683    +/-   ##
======================================
+ Coverage      81%    97%   +16%
======================================
  Files          32     30     -2
  Lines        3434   3250   -184
======================================
+ Hits         2788   3165   +377
+ Misses        646     85   -561


chore: bump npm packages for security

Description

bump npm packages for security

General Checklist

[ ] Added new tests to cover change, if needed
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Project coverage is 100.00%. Comparing base (461c45b) to head (7a233fe).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #55   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           25        25
  Lines         1179      1179
  Branches       218       231   +13
=========================================
  Hits          1179      1179

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


TagUI - RUN misbehave with parameters between double quote

Hi there,
I'm running an external command with arguments using RUN function as below.
Command   : "C:\tagui\set_focus.exe"
Argument 1 : --windows_name="test file"
Argument 2 : --windows_mode="restore"
It seems the double quote does not have any impact on parsing the arguments.
Is there any workaround available. Thanks in advance.
//TagUI Script
run "C:\tagui\set_focus.exe" --windows_name="test file" --windows_mode="restore"
echo run_result
//Results
START - automation started - Fri Nov 25 2022 16:24:13 GMT+0800 (Malay Peninsula Standard Time)
run "C:\tagui\set_focus.exe" --windows_name="test file" --windows_mode="restore"
usage: set_focus.exe [-h] --windows_name WINDOWS_NAME --windows_mode
WINDOWS_MODE
set_focus.exe: error: unrecognized arguments: file"
FINISH - automation finished - 0.9s

Hi @GarneshMike can you try this to run your external programme?
run cmd /c start C:\\tagui\\set_focus.exe

https://github.com/aisingapore/TagUI/issues/1135

Closing but please ping back if this is still relevant of an issue.


give developer the option to ignore struct tag "-"

I appreciate your excellent pkg, it helps me a lot. I need a new feature, please consider it,thank you!
There is an id field in my struct, I don't want to expose this field to outside users, so I tag it with "-", it works well.
when I  marshal  the struct  to  cache in redis and unmarshal to the struct from redis,I found that the fields I tagged with "-" are all zero values, it works well too, but that is not what I want
I hope there is an option for me to decide whether to ignore the "-" tag, when I marshal/unmarshal to cache in redis, I want to ignore the "-"tag and marshal/unmarshal all fields, when I marshal/unmarshal to frontend, the "-" works as it is now

type Config struct {
	IndentionStep                 int
	MarshalFloatWith6Digits       bool
	EscapeHTML                    bool
	SortMapKeys                   bool
	UseNumber                     bool
	DisallowUnknownFields         bool
	TagKey                        string
	OnlyTaggedField               bool
	ValidateJsonRawMessage        bool
	ObjectFieldMustBeSimpleString bool
	CaseSensitive                 bool
}

// ConfigFastest marshals float with only 6 digits precision
var ConfigFastest = Config{
	EscapeHTML:                    false,
	MarshalFloatWith6Digits:       true, // will lose precession
	ObjectFieldMustBeSimpleString: true, // do not unescape object field
}.Froze()

config the TagKey option by yourself.

thanks for your reply.
the tagKey -  is ok for me, I don't want to change it globally.  what I really need is  when I marshall unmarshall, allow me to decide whether to ignore the tagKey
the application scenario is described in my last post

thanks for your reply.
the tagKey - is ok for me, I don't want to change it globally. what I really need is when I marshall unmarshall, allow me to decide whether to ignore the tagKey
the application scenario is described in my last post

Oh, you didn't understand, you can just make a new and private config in you project, or a function, it will not effect the global config.
For example:
	jsoniter.Config{
		TagKey: "XXX",
	}.Froze().Marshal(v interface{})

private config in the function...  I got it!
thank you so much


Improved cleaning of URL to prevent validation errors

Build will fail until fluo-io/fluo-recipes#36 is merged

+1


Couldn't find plugin "ui5" service when running example test script.

Steps to reproduce the behavior:

download the whole project
go to folder /wdi5/examples/ui5-js-app/, run command npm install
run command npm install wdio-ui5-service under folder ui5-js-app/

I've got below errro:
[0-24] 2023-02-07T02:55:28.672Z ERROR @wdio/utils:initialiseServices: Error: Couldn't find plugin "ui5" service, neither as wdio scoped package "@wdio/ui5-service" nor as community package "wdio-ui5-service". Please make sure you have it installed!
[0-24]     at initialisePlugin (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/utils/build/initialisePlugin.js:41:11)
[0-24]     at initialiseServices (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/utils/build/initialiseServices.js:59:56)
[0-24]     at initialiseWorkerService (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/utils/build/initialiseServices.js:140:26)
[0-24]     at Runner.run (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/runner/build/index.js:75:45)
[0-24]     at processTicksAndRejections (internal/process/task_queues.js:95:5)
[0-24] RUNNING in chrome - /webapp/test/e2e/wdioBridge.test.js
[0-24] 2023-02-07T02:55:33.283Z ERROR @wdio/runner: Error: Unable to load spec files quite likely because they rely on `browser` object that is not fully initialised.
[0-24] `browser` object has only `capabilities` and some flags like `isMobile`.
[0-24] Helper files that use other `browser` commands have to be moved to `before` hook.
[0-24] Spec file(s): /Users/I310978/Documents/03Code/wdi5/examples/ui5-js-app/webapp/test/e2e/wdioBridge.test.js
[0-24] Error: Error: Cannot find module '/Users/I310978/Documents/03Code/wdi5/node_modules/wdio-ui5-service/dist/index.js'. Please verify that the package.json has a valid "main" entry
[0-24]     at tryPackage (internal/modules/cjs/loader.js:321:19)
[0-24]     at Function.Module._findPath (internal/modules/cjs/loader.js:534:18)
[0-24]     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:888:27)
[0-24]     at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as _resolveFilename] (/Users/I310978/Documents/03Code/wdi5/node_modules/@cspotcode/source-map-support/source-map-support.js:811:30)
[0-24]     at Function.Module._load (internal/modules/cjs/loader.js:746:27)
[0-24]     at Module.require (internal/modules/cjs/loader.js:974:19)
[0-24]     at require (internal/modules/cjs/helpers.js:93:18)
[0-24]     at Object.<anonymous> (/Users/I310978/Documents/03Code/wdi5/examples/ui5-js-app/webapp/test/e2e/pageObjects/Page.js:1:18)
[0-24]     at Module._compile (internal/modules/cjs/loader.js:1085:14)
[0-24]     at Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
[0-24]     at MochaAdapter._loadFiles (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/mocha-framework/build/index.js:86:35)
[0-24]     at async MochaAdapter.init (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/mocha-framework/build/index.js:55:9)
[0-24]     at async Object.adapterFactory.init (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/mocha-framework/build/index.js:319:22)
[0-24]     at async Runner.run (/Users/I310978/Documents/03Code/wdi5/node_modules/@wdio/runner/build/index.js:83:27)

Runtime Env (please complete the following information):

node-version (output of node --version): v14.18.3
OS: MacOS

the examples in the repo are part of the overall wdi5 monorepo setup.
you'll need at least npm 8 and do a npm i on root level.
Please see the contributing doc https://ui5-community.github.io/wdi5/#/contributing for more detailed instructions.


Fatal error after updating plugin

I updated the plugin, my Wordpress stopped working and displays the following error on the screen:
Fatal error: Using $this when not in object context in /home/krebs284/public_html/wp-content/plugins/enlighter/class/ShortcodeFilter.php on line 85
I had to disable it to make the site work.

Dublicate #52
The issue has already been fixed in the master branch. the release will follow within the next few days
best, regards, Andi

v3.3 is out including the fix


Capital D will not print in any code window

Type: Bug
I'm not joking, vs code will seriously not print anything when I try and type capital d. I can use the command palette to capitalise a lower case 'd' to an uppercase one no problem, I just can't type the character directly.
It's totally mad!

Open any text or code file....

Hold shift....

press the 'd' key.

Nothing is printed (but I can print 'D' on any other app and indeed in this issue reporter window).
Obviously, I've restarted the system (several times, to no effect) and I had hoped that upgrading to the latest version of vs code would help - but it's still an issue.
I literally have to use the command palette to capitalise lowercase 'd' manually.
Any insights/thoughts/suggestions would be very gratefully received 😢
VS Code version: Code 1.70.2 (e4503b30fc78200f846c62cf8091b76ff5547662, 2022-08-16T05:35:13.448Z)
OS version: Windows_NT x64 10.0.22000
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 5700G with Radeon Graphics          (16 x 3793)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
27.91GB (13.98GB free)

Process Argv
--crash-reporter-id 3cccc56f-1d20-4443-8476-779fe107a959

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

vscode-postgres
cko
1.4.0

gitlens
eam
12.1.2

RunOnSave
eme
0.2.0

prettier-vscode
esb
9.8.0

terraform
has
2.24.1

git-graph
mhu
1.30.0

vscode-docker
ms-
1.22.1

python
ms-
2022.12.1

vscode-pylance
ms-
2022.8.40

jupyter
ms-
2022.7.1102252217

jupyter-keymap
ms-
1.0.0

remote-containers
ms-
0.245.2

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

powershell
ms-
2022.7.2

vscode-yaml
red
1.9.1

mdx
sil
0.1.1

ayu
tea
1.0.5

vscode-icons
vsc
11.15.0

jinja
who
0.0.8

grammarly
znc
0.22.1

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
wslgetstarted:30449410
cppdebug:30492333
pylanb8912:30545647
vsclangdf:30486550
c4g48928:30535728
hb751961:30553087
dsvsc012cf:30540253
azure-dev_surveyone:30548225
i497e931:30553904

OMG!!!! I somehow managed to create a keybinding for shift-D!!!!


Handling of lockfile on crash/restart

I've spent the period since the last release beating uhppoted-tunnel in some pretty terrible ways; the vendor's app in monitor mode (5s updates) constantly, there's a python script running as a daemon to pull status and events parsing them into its own db (5s updates), there's another python daemon running as a UDP CRUD service on the same port that monitors the vendor's app MS Access database and if it sees any changes in the group and username tables it pushes them out to the remote site db using the same secure tunnel service (just uses a 'magic' device serial number, opcodes, and special data hash.)
Topology:
Win10-udp---udp-RPi-tls--------tls-VPS-tls------tls-RPi-udp---udp-UHPPOTE
db daemon                                                           db daemon
udp  db client                                                           udp db server
udb db client
This entire mess has been working pretty well all this time with the new release, and a few random network hits, the system recovers after about 10minutes or so.  And I've been doing development of the local and remote db applications, which means I've been nuking the event tables and re-syncing thousands of events several times a day over the tunnel all while the app is up and running.
Today, however, with several power hits hitting pretty much every node (and sub-node) in every permutation imaginable, the remote tunnel node got lost in such a way that it left the lockfile behind.  Looking through syslog, there's a ton of lines, starting off after a reboot:
Nov  4 15:32:14 netmon systemd[1]: session-5.scope: Succeeded.
Nov  4 15:55:11 netmon rc.local[841]: fatal error: concurrent map read and map write
Nov  4 15:55:11 netmon rc.local[841]: goroutine 5329 [running]:
Nov  4 15:55:11 netmon rc.local[841]: runtime.throw({0x3c4576, 0x21})
Nov  4 15:55:11 netmon rc.local[841]: #011runtime/panic.go:992 +0x5c fp=0x2856ee0 sp=0x2856ecc pc=0x47bf8
Nov  4 15:55:11 netmon rc.local[841]: runtime.mapaccess2_fast32(0x378680, 0x2800480, 0x306ee)
Nov  4 15:55:11 netmon rc.local[841]: #011runtime/map_fast32.go:62 +0x194 fp=0x2856ef0 sp=0x2856ee0 pc=0x202ac
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/router.(*Router).get(0x625478, 0x306ee)
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/router/router.go:92 +0x3c fp=0x2856f50 sp=0x2856ef0 pc=0x2b74c8
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/router.(*Switch).Received(0x280e988, 0x306ee, {0x2a73006, 0x1c, 0x7fa}, 0x2a700f0)
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/router/router.go:62 +0x24 fp=0x2856f64 sp=0x2856f50 pc=0x2b7108
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/tunnel/tls.(*tlsClient).received(0x28a00a0, {0x2a73000, 0x22, 0x800}, 0x280e988, {0x4415b4, 0x29a2000})
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/tunnel/tls/tls_client.go:162 +0x128 fp=0x2856fcc sp=0x2856f64 pc=0x339c24
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/tunnel/tls.(*tlsClient).listen.func1()
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/tunnel/tls/tls_client.go:150 +0x54 fp=0x2856fec sp=0x2856fcc pc=0x339a88
Nov  4 15:55:11 netmon rc.local[841]: runtime.goexit()
Nov  4 15:55:11 netmon rc.local[841]: #011runtime/asm_arm.s:824 +0x4 fp=0x2856fec sp=0x2856fec pc=0x7a140
Nov  4 15:55:11 netmon rc.local[841]: created by github.com/uhppoted/uhppoted-tunnel/tunnel/tls.(*tlsClient).listen
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/tunnel/tls/tls_client.go:149 +0xfc
Nov  4 15:55:11 netmon rc.local[841]: goroutine 1 [chan receive, 77 minutes]:
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/commands.(*Run).run(0x625900, 0x28a80d8, {0x440ad8, 0x284e180}, 0x280e6b8, 0x289c140)
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/commands/run.go:259 +0x228
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/commands.(*Run).exec(0x625900, 0x28a80d8, {0x440ad8, 0x284e180}, 0x280e6b8)
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/commands/run_linux.go:66 +0x27c
Nov  4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/commands.(*Run).Execute.func1(0x28a80d8, {0x440ad8, 0x284e180}, 0x280e6b8)
Nov  4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/commands/run_linux.go:33 +0x3c
.
then ends up
.
Nov  4 15:55:12 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/tunnel/tunnel.go:60 +0x24c
Nov  4 15:55:12 netmon rc.local[841]: goroutine 33 [select, 77 minutes]:
Nov  4 15:55:12 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/tunnel/tls.(*tlsClient).connect.func1()
Nov  4 15:55:12 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/tunnel/tls/tls_client.go:109 +0x154
Nov  4 15:55:12 netmon rc.local[841]: created by github.com/uhppoted/uhppoted-tunnel/tunnel/tls.(*tlsClient).connect
Nov  4 15:55:12 netmon systemd[1]: rc-local.service: Control process exited, code=exited, status=2/INVALIDARGUMENT
All of this seems to happen before the system has really booted.  Maybe the unit file needs to be modified to wait for full multi-user or maybe network?  On this RPi4 running Bullseye, I have about a 1min network wait to deal with the dumpster fire that is Debian-11/RPi's handling of the WiFi drivers.
In any event, there seems to be something missing in clean-up; if it crashes, it should take the lockfile with it, or the unit script should look for the pid and 86 both it and the file and start anew.

Those storms must have been quite something!
There's quite a bit to unpack in that log:

There might be a mutex missing around one of the router tables (concurrent map read and map write) - will look into that
The goroutine 33 [select, 77 minutes] ... goexit is something I've never seen - will have to dig into it to see exactly what is happening there
Good point on waiting for multi-user/network in the service file - will implement that (actually that needs to be implemented across the board)

The lockfile  needs some thinking about though - basically there's no way to guarantee that a crash won't leave a lockfile behind so the recovery needs to happen on service start. There are a couple of options I can think of right off the bat:

Allow the lockfile to be opted out in the conf file (e.g. lockfile=(none) or somesuch). Probably not such a good idea but it's simple
Make the lockfile a soft lock a lá the fix for the uhppoted-mqtt client lockfile.
Deleting it the service start (also a lá the uhppoted-mqtt fix) . Am a bit reluctant to make that the default but it could be an 'advanced user' option
An "I'm alive" hook so something like an external service or cron job could monitor the daemon and clean up if necessary

Let me give it some thought..

There is a lot more in the log, if you give me a few tidbits of keywords to look for I can try to ferret out more gotchas, but we'd all like the ultimate goal of this beast to be unattended and robust.  First and foremost, I need it to forward alarms and fortunately I'm using Pushover for that, which does a nice job of sending an event summary, a snap of the door, etc. to my phone/desktop/whatever so it does not depend on Tunnel.  But who's in the building, door control, siren silence, and all that rubbish is dependent on Tunnel and I don't want to have to futz with trying to manage a wayward process from my phone (or write a watchdog script if I can help it).

Thanks - think there's probably enough for me to start looking at it, but may ask you for more info once I get deeper into it. Or ... if there's nothing sensitive in the log file maybe attach it as a zip?

Hi,
Are you running tunnel v0.8.2+ on the machine that crashed? I'm curious because:

The fatal error: concurrent map read and map write issue was fixed in v0.8.2 (cf. https://github.com/uhppoted/uhppoted-tunnel/issues/4)
router.go:line 62 in v0.8.2+ doesn't seem to match the log:

Nov 4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/router.(*Switch).Received(0x280e988, 0x306ee, {0x2a73006, 0x1c, 0x7fa}, 0x2a700f0)
Nov 4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/router/router.go:62 +0x24 fp=0x2856f64 sp=0x2856f50 pc=0x2b7108

(Iin the actual code Received is around router.go:67 and it hasn't changed since the v0.8.2 release)

That would be strange, because v0.8.1 does not run as a daemon.  Now you got me wondering if I managed some sort of frankeninstall, double-checking:
uhppoted-tunnel installed in /etc/uhppoted/uhpoted-tunnel and referenced as such in /etc/systemd/system/uhpoted-tunnel.service
MD5 of /etc/uhppoted/uhppoted-tunnel bcca16c461b9f2b721b0f3630ca74659
going over to the releases dir cloned from git as uhppoted_v0.8.2/uhppoted-tunnel, the MD5 is bcca16c461b9f2b721b0f3630ca74659, so I'm pretty sure it is what it is...

No, I'm much too lazy, lol...
got from https://github.com/uhppoted/uhppoted/releases/download/v0.8.2/uhppoted_v0.8.2-arm7.tar.gz
just did a wget to a test directory, gunzip, tar xvf, to make sure it IS what I installed, and got the same MD5:
bcca16c461b9f2b721b0f3630ca74659

Ok, I get the same checksum now ... which is bizarre. I really hope it doesn't mean all those top level binaries are wrong - but you might want to switch to the uhppoted-tunnel releases while I figure out what is going on.

FYI, turns out the checksums are different because the top-level release builds were not being built with GO WORK=off - not catastrophic but I'll fix the release script and add a checksum verification step.
Which, however, does not explain why the logs and the source code do not match..

Ok, at this point in time I am reasonably convinced that when your machine crashed it was running tunnel v0.8.1:

the log entries match the source code of v0.8.1 exactly (and v0.8.2 not at all)
I've dumped and checked the symbol tables for both the uhppoted-tunnel v0.8.2 and the uhppoted v0.8.2 releases and they're both correct for v0.8.2 (and not v0.8.1) so as far as I can tell neither of the executables would have produced those log entries.

I'm afraid I have absolutely no idea how that could have happened - but here are some things you could look at if you wanted to:

The tunnel daemon prints its version on startup - there may be something in the logs for 4th November like:

Nov 10 21:32:25 uhppoted uhppoted-tunnel[14728]: 2022/11/10 21:32:25 uhppoted-tunnel service v0.8.2 - Linux (PID 14728)

sudo systemctl status "uhppoted-tunnel" should show the tunnel version when the daemon starts:

sudo systemctl status "uhppoted-tunnel-qwerty"
uhppoted-tunnel-qwerty.service - UHPPOTE UTO311-L0x access card controllers UDP tunnel service/daemon
     Loaded: loaded (/etc/systemd/system/uhppoted-tunnel-qwerty.service; disabled; preset: enabled)
     Active: active (running) since Thu 2022-11-10 21:32:25 UTC; 1min 48s ago
       Docs: https://github.com/uhppoted/uhppoted-tunnel
   Main PID: 14728 (uhppoted-tunnel)
      Tasks: 5 (limit: 1020)
     Memory: 1.3M
        CPU: 7ms
     CGroup: /system.slice/uhppoted-tunnel-qwerty.service
             └─14728 /opt/uhppoted/uhppoted_v0.8.2/uhppoted-tunnel --lockfile /var/uhppoted/uhppoted-tunnel-qwerty.pid --in udp/listen:<IP_ADDRESS>:60000 --out tcp/server:<IP_ADDRESS>:12345

Nov 10 21:32:25 uhppoted systemd[1]: Started UHPPOTE UTO311-L0x access card controllers UDP tunnel service/daemon.
Nov 10 21:32:25 uhppoted uhppoted-tunnel[14728]: 2022/11/10 21:32:25 uhppoted-tunnel service v0.8.2 - Linux (PID 14728)

Is (was ?) the service file (/etc/systemd/system/uhppoted-tunnel.service) pointing to the correct executable?

I'm going to look at the goroutine 33 [select, 77 minutes]: thing just to try understand that - don't think there's much else I can offer around the fatal error: concurrent map read and map write log entry.

Going back in time to find when this run was launched before this epic storm...
Oct 24 13:32:23 netmon systemd[1]: Started UHPPOTE UTO311-L0x access card controllers UDP tunnel service/daemon.
Oct 24 13:32:23 netmon uhppoted-tunnel[5463]: 2022/10/24 13:32:23 uhppoted-tunnel service v0.8.2 - Linux (PID 5463)
Oct 24 13:34:06 netmon systemd[1]: session-606.scope: Succeeded.
(PID 5463 was up for nearly a week until the storm hit, then it was rock-n-roll for several hours, several power and network outages).
Went forward in the logs and pulled the part where it crashed when it found the stale pid file...
Nov  4 21:15:00 netmon systemd[1]: Started UHPPOTE UTO311-L0x access card controllers UDP tunnel service/daemon.
Nov  4 21:15:00 netmon uhppoted-tunnel[1563]: 2022/11/04 21:15:00 uhppoted-tunnel service v0.8.2 - Linux (PID 1563)
Nov  4 21:15:01 netmon uhppoted-tunnel[1563]: ERROR: PID lockfile '/var/uhppoted/uhppoted-tunnel.pid' already in use
Nov  4 21:15:01 netmon systemd[1]: uhppoted-tunnel.service: Main process exited, code=exited, status=1/FAILURE
Nov  4 21:15:01 netmon systemd[1]: uhppoted-tunnel.service: Failed with result 'exit-code'.
Nov  4 21:16:25 netmon systemd[1]: Started UHPPOTE UTO311-L0x access card controllers UDP tunnel service/daemon.
Nov  4 21:16:25 netmon uhppoted-tunnel[1591]: 2022/11/04 21:16:25 uhppoted-tunnel service v0.8.2 - Linux (PID 1591)
This is the RPi-4/Debian-11 (Bullseye) build, pulled as bin off git, as lazy as possible.
The pivot node is x86 running on a VPS, but still Deb-11, tunnel v0.8.2 if that matters at all, it's been rock-solid with no crashes. The x86/win10 side has been solid as well.

Ok, am going to admit to being completely baffled by the concurrent map access issue:

Nov 4 15:55:11 netmon rc.local[841]: fatal error: concurrent map read and map write
Nov 4 15:55:11 netmon rc.local[841]: goroutine 5329 [running]:
Nov 4 15:55:11 netmon rc.local[841]: runtime.throw({0x3c4576, 0x21})
Nov 4 15:55:11 netmon rc.local[841]: #011runtime/panic.go:992 +0x5c fp=0x2856ee0 sp=0x2856ecc pc=0x47bf8
Nov 4 15:55:11 netmon rc.local[841]: runtime.mapaccess2_fast32(0x378680, 0x2800480, 0x306ee)
Nov 4 15:55:11 netmon rc.local[841]: #011runtime/map_fast32.go:62 +0x194 fp=0x2856ef0 sp=0x2856ee0 pc=0x202ac
Nov 4 15:55:11 netmon rc.local[841]: github.com/uhppoted/uhppoted-tunnel/router.(*Router).get(0x625478, 0x306ee)
Nov 4 15:55:11 netmon rc.local[841]: #011github.com/uhppoted/uhppoted-tunnel/router/router.go:92 +0x3c fp=0x2856f50 sp=0x2856ef0 pc=0x2b74c8

As far as I can see that can only occur in v0.8.1. Am going to put that on the back burner for now (pending another storm and/or inspiration :-)) and focus on the lockfile problem which is at least tangible.

TODO

[ ] Abstract Lockfile as interface and move to lib
[ ] Default to ephemeral tmp folder for lockfiles
[ ] Default to flock for Linux and MacOS
[ ] (optional) 'soft' lock
[ ] (optional) 'socket' lock
[ ] Rework other modules to use lib Lockfile

[ ] uhppoted-rest
[ ] uhppoted-mqtt
[ ] uhppoted-httpd
[ ] uhppoted-app-wildapricot
[ ] uhppoted-app-s3
[ ] uhppoted-app-sheets

NOTES

https://github.com/golang/go/issues/33974
https://go.googlesource.com/proposal/+/master/design/33974-add-public-lockedfile-pkg.md
https://perl.plover.com/yak/flock
https://stackoverflow.com/questions/5210945/atomic-file-creation-on-linux
https://www.diskodev.com/posts/linux-atomic-operations-on-files
https://stackoverflow.com/questions/29261648/atomic-writing-to-file-on-linux
https://stackoverflow.com/questions/34873151/how-can-i-delete-a-unix-domain-socket-file-when-i-exit-my-application
https://gavv.net/articles/unix-socket-reuse
https://stackoverflow.com/questions/7405932/how-to-know-whether-any-process-is-bound-to-a-unix-domain-socke
https://www.unix.com/man-page/centos/1/lockfile
https://security.stackexchange.com/questions/11976/can-unix-domain-sockets-be-locked-by-user-id
https://pythonhosted.org/lockfile/lockfile.html

That's quite an area you live in! Have you considered using something like e.g. Tailscale for the SSH?
re. lockfile. Am working on it - turns out there's a lot more to lock files than one would think (the official Go implementation has been stuck in limbo for like 5 years). However - as an interim fix you could set the lock file to live in the /tmp folder which is (supposedly) cleared on reboot e.g.
uhppoted-tunnel --lockfile /tmp/uhppoted-tunnel.lock .....

re. camera. Huh! Interesting idea!! And yeah, a Pi 4 is way overkill for most of this stuff :-).

Tailscale is neat and does seem to have pretty much got it right - unlike all the other hacky and marginal solutions that peope have resorted to over the years.
Although I see they do have a CVE today :-):

Just read through that, since my install was today, I have the latest, but #7 is nasty, like log4j nasty.  Not running http through it, but ran the curl tests to be sure there are no gotchas.
While I'm at the task of modifying the system d unit file, is there a directive to have it shut off logging the packet transactions (or at least point them to /dev/null?)  Since this thing is running headless and mostly in RAM, I want to minimize SD use from logging and keep it only for alarm event recordings.  It's a high endurance card, but still logs tend to scribble blocks like crazy...

Ref. #7. Eeesh, security researchers do not think in straight lines - looks like Tailscale is more or less on top of it though.
You managed to figure out logging to /dev/null ? By default tunnel logs to stdout and stderr which means the service typically logs to syslog (depending on how your system is configured of course). Editing the command-line in the unit file to redirect stdout to /dev/null should do it.

logging:  I saw the log level directive in the description, set it to warn in the Service description.
Hey, here's a weird inconsistency; In my typical laziness I pull the 0.8.2 arm linux binary to load up on the pi-zero-w.  It crashes out with Illegal Instruction, so I go, oh, built with 64bit.  No big deal, install golang and build it from source.  Pi-zero, so go make dinner, it'll be a while...  come back, stick the binary where I typically do (as /etc/uhppoted/uhppoted-tunnel) only this time when I go to deamonize it, it complains about not finding the .toml file.  That's funny, the 64bit binary downloaded from git does not do that on the pi4.  Made a uhppoted-tunnel.toml with some defaults I liked, including cert and key file locations, etc.  Well, it ignores those directives, but it stopped complaining.
TL;DR: The .toml madness hints there is something out of sync with the pre-built binaries on git and the source.

OMG, you're going to love this, and I feel like an idiot:  I have no idea how this happened, but there were both versions of uhppoted-tunnel running concurrently on this system, and looking at /etc/systemd/system I don't see how it is even possible.  It was hidden because I am running different client sessions to different hosts, so I expect to see a few out there when running a ps, but until I stopped all the services and saw one still hanging around, and pointing to /home/pi which is not even an active user directory(!) I got suspicious.
This Pi is definitely haunted...  mentioned before this was a Buster->Bullseye upgrade, which is contraindicated due to hits on the networking stack, well, methinks there's other weirdness with apt and make.  This one gets wiped and a do-over.  The pi-zero is now up and running and is a clean build from 0.8.2.

Ha! I knew it :-) :-) :-) - but basically 'Whew!' - am really, really happy to have that very puzzling anomaly explained.
Oh, BTW - figured out the hack for routing service logs to /dev/null - if you run the service with the--console argument then the logs go to stdout and you can redirect them to wherever you like. Not recommended though because if anything goes wrong there won't be a scrap of evidence to work with - --log-level is definitely the way to go.

Oh, that sounds great!!! Really glad to hear that, and thanks for the update!
The flock syscall is the last piece of the puzzle I think - it's honestly the correct way to do locking and should in theory add a whole new level of robustness to the lockfile thing .. been an interesting delve in the innards of systems.

v0.8.3 is available for whenever you have time to try it out (and a convenient storm). I'll leave this issue open for a bit..

'k, closing this - feel free to reopen if you need to.


quickly print full link to repo?

I am thinking to implement gitea.info to do this:
gitea info myuser/myrepo
This would print:
https url: https://gitea.example.com/myuser/myrepo.git
SSH url: git@gitea.example.com:myuser/myrepo.git
(Maybe) Last commit: blah blah commit message #commithashash

Can this be done and how?

As of the latest commit, you can do something like:
gitea.info() {
    gitea exists "$1" && echo "$REPLY" | jq .
}

To output a nicely-formatted JSON dump of the repo data.  If you want it in something other than JSON, you can change the jq . to something like jq -r '"https url: \(.html_url)\nSSH url: \(.ssh_url)"', expanding upon it to include other data from the JSON as desired.  You'll probably want to read the jq manual for more info if you're going to take that approach.


initial add of guided registration

Added our guided registration ui.

Can be reviewed again. Changed anchor and added .gitignore for .DS_Store as well as correction of url in Dockerfile.


angular approach for building components

Hello,
I am looking for a way to create reusable components like I would in an angular app. What I mean is something like this:

   <page-router-outlet>
   <toolbar>
   <side-menu>

This would be ideal, but I understand page-router-outlet works differently then router-outlet. So I started doing this in every component:

<...my component elements...>
<toolbar>
<side-menu>

This is a lot of repetition, and I would like to avoid it, yet I am failing to do so. Now because I need AOT, I had to update RadSideDrawer to 1.6.1. My side-menu was a component that contained RadSideDrawer and all it's link-loading logic. With the new way of RadSideDrawer logic ( it is wrapping the content of the 'parent' page) I now have to write it's code on every component.

<RadSideDrawerCode>
.....
<...my component elements...>
....
</RadSideDrawerCode>
<toolbar>

I tried wrapping up ActionBar in a component, which would have all the logic and icon placement I needed, but have failed. So I need to specify ActionBar in every component again - I know I can use directives for buttons and title, but I still need to define the ActionBar, which I wanted to avoid.
<ActionBar class="actionBar">
	<NavigationButton android.systemIcon="ic_menu_back" text=""></NavigationButton>
	<ActionItem  ios.position="right" android.position="left">
		<Image width="30" height="30" src="res://hamburger" (tap)="openDrawer()"></Image>
	</ActionItem>
</ActionBar>
<StackLayout>
<RadSideDrawer>
    <StackLayout tkDrawerContent class="sideStackLayout">
        <StackLayout class="sideTitleStackLayout">

        </StackLayout>
        <StackLayout class="sideStackLayout">

        </StackLayout>
    </StackLayout>
    <StackLayout tkMainContent>
      <... my component elements>
    </StackLayout>
</RadSideDrawer>
<toolbar></toolbar>
<StackLayout>

I pretty much end up writing the code above in every component, which seems wasteful, counter-intuitive, unmaintainable and just plain dirty. I basically only want to do the <... my component elements...>. I know the example above is a bit excessive, but when I checked repos of official samples, it seems you are doing it in almost the same way (with the exception of action bar directives).
I am sure I must be missing something. Can you show me how to achieve the least amount of empty copying? I know native works different then web, that all views are actual views and not just changed context inside one loaded page. But I still feel I should be able to easily reuse actionbar, sidemenu and toolbar, which almost all apps have.
I have asked and discussed this multiple times, but I always end up in the last example.
Thanks

Hi @lukashlobil,
If your side menu (aka. side drawer) content is same, then you have something like below-
In app.component.html-
<ActionBar>
optional, this can come from different feature component
</ActionBar>
<StackLayout>
 <RadSideDrawer>
    <StackLayout tkDrawerContent class="sideStackLayout">

       --- fixed side drawer content

        </StackLayout>
    </StackLayout>
    <StackLayout tkMainContent>
      **<page-router-outlet>** Or **<router-outlet>**
    </StackLayout>
</RadSideDrawer>
<toolbar></toolbar>
<StackLayout>

You should use  if you care about page transitions(fade, slide, etc) and automatic back button for iOS and Android. And then you can nest them as needed.
Or you can simply use  and its nesting like typical Angular App.
The above mentioned code approach can possibly save your SideDrawer code in all the components.
Hope this helps you.
Thanks,

any thoughts on this

The answer given by @hiddencaliber is more than enough. Adding your reuse-able components such as action-bar onto your app.component.html or where you're hosting your page-router is all you need to do.


Add list view expansion to Angular-Patternfly

Added new unit test for the functionality and tested in ng-docs

Please add more comments to this PR so that release notes can be created with details.

Add the list-view-pf-view selector added a 30px top margin inside the list view scrolling container. Why would we want that?

The columns are no longer aligned and there is very little space (28px) left for the item name.

Adding the expand indicator makes the rows grow in height for no good reason. Looks like the PF class list-view-pf-expand is adding a touch too much margin.


Screen mirroring (airplay) is broken

Description
The IOS screen mirroring feature is broken when a native alert is displayed.
The alert will only appear on the mirroring screen and not on the Iphone, preventing any interaction with it.
Occurs on IOS
The minimal reproduction uses expo managed, but ejecting won't solve the issue.
React Native Version
0.71.7
Output of npx react-native info
OS: macOS 13.3.1
CPU: (8) arm64 Apple M1 Pro
Memory: 102.20 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
Yarn: 1.22.17 - ~/.nvm/versions/node/v16.13.2/bin/yarn
npm: 8.1.2 - ~/.nvm/versions/node/v16.13.2/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7935034
Xcode: 14.3.1/14E300c - /usr/bin/xcodebuild
Languages:
Java: <IP_ADDRESS> - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.8 => 0.71.8
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

Start the project on your phone
Start mirroring on a compatible screen like your Macbook screen
On your phone, press the alert button.

-> The alert will only be shown on the mirroring screen but not on the phone
Snack, code example, screenshot, or link to a repository
https://github.com/Chayuan/expo-screen-mirroring-issue

Edit : We were able to resolve the issue with airplay by reverting this commit.

Edit : We were able to resolve the issue with airplay by reverting this commit.
Edit 2 : Upgrading to react-native 0.71.10 does not fix the issue.

Cc: @cipolleschi

Hi @Chayuan, thank you for opening the issue.
To properly investigate the issue, we would need a repro with an app created with the CLI, not with expo.
Could you double check that by:

creating a new app with the CLI
moving your App.js in the newly created app
airplaying and clicking the button
The issue is still there? With that setup, we can work something out.

I don't think that reverting https://github.com/facebook/react-native/commit/0c53420a7af306d629350e1244e8e2ccae08a312 is a good long term solution as we introduced it to support the SceneDelegate approach with is the standard for modern iOS apps.

Hi @cipolleschi, thanks for the answer.
Here is the minimal reproduction using the cli, I can confirm the issue is still there.

Thanks! I hope to find some time this or next week to look into that. 👍

Hi we're facing the same issue.. is there any update on this?

Not yet, sorry. Had been dragged away by other issues. :(

I see :( is there any plan to work on this soon-ish? It would help a lot to know an estimation for timeline

Thanks! I hope to find some time this or next week to look into that. 👍

@cipolleschi Did you manage to find some time to look into this issue?

Could you provide a repro for this using React Native vanilla, and with this template?
I saw that the provided repro above is using Expo, and that's not good for me.

Could you provide a repro for this using React Native vanilla, and with this template?
I saw that the provided repro above is using Expo, and that's not good for me.

Updated this repo using the mentioned template and find the reproduction steps here.

I had a look at the issue recently but I couldn't test it properly as I don't have any mirroring enabled device at home or in the office.
However, we don't do anything specific with React Native to interact with AirPlay.
Can you check how a native app which just display an alert interact in that scenario?

One thing I noticed with external screens is that the alert is presented in that screen:

While a Native App seems not to present the same behavior:

This difference can be due to how React Native handles the Screens. With two screens, we can choose which one to use with the UIScreen APIs.
However, while mirroring, we should still have a single Screen, so I don't think this is super relevant.

I had a look at the issue recently but I couldn't test it properly as I don't have any mirroring enabled device at home or in the office. However, we don't do anything specific with React Native to interact with AirPlay. Can you check how a native app which just display an alert interact in that scenario?
One thing I noticed with external screens is that the alert is presented in that screen:
While a Native App seems not to present the same behavior:
This difference can be due to how React Native handles the Screens. With two screens, we can choose which one to use with the UIScreen APIs. However, while mirroring, we should still have a single Screen, so I don't think this is super relevant.

Thanks for looking into it, I have checked the alerts with native apps and it's working without an issue.
FYI: This issue does not occur in react-native version 0.71.0.


Fix layout differences of Header v.s. Header

This fixes https://github.com/servo/tendril/issues/58, which shows UB when converting between non-atomic and atomic Tendril. This conversion is exposed without unsafe through SendTendril and implemented with transmute, so matching layout is essential.

r? @jdm

CI is failing on nightly while testing the C API. Should we just rip it out?

Yes. I thought we had already.

struct Header should be changed to be #[repr(C)], Rust won't otherwise guarantee that Header<Atomic> and Header<NonAtomic> have fields in the same order. Currently UCG doesn't guarantee deterministic layout (https://github.com/rust-lang/unsafe-code-guidelines/issues/35).

@bors-servo r+

:pushpin: Commit bdd07bf has been approved by jdm

:sunny: Test successful - checks-github
Approved by: jdm
Pushing 3df7383ac023b2ba5aa4bcb6694fba8459c268ed to master...


[avatar] Unit Tests for Avatar Helper

As title. implement unit test for:

[ ] getSizeFromUrl
[ ] gravatar
[ ] uiAvatar

merge to #222


Tallow not dropping in ATM9 v0.0.22

Describe the bug
Tallow isn't dropping when killing animals with the butcher knife
To Reproduce
Steps to reproduce the behavior:

Kill animal with butcher knife
No tallow drops after like 30 kills with looting butcher knife

Expected behavior
Mob's drop tallow when killed with a butcher knife
System (please complete the following information):

Occultism Version: 1.80.12
OS: Windows
Minecraft Version: 1.20.1
Modpack Link and Version, or list of mods: https://www.curseforge.com/minecraft/modpacks/all-the-mods-9

This is a forge Bug, latest version of forge might have fixed it, see also #934
You can use vanilla candles instead of occultism candles to progress


travis: consider building latest commit only

Consider using https://docs.travis-ci.com/user/customizing-the-build/#building-only-the-latest-commit preventing unnecessary rebuilds of PR's

That's already on.


Change internal build scheme for python patch versions.

Despite official semver rules, Python has it's own opinions.
Official rules:
<valid semver> ::= <version core>
                 | <version core> "-" <pre-release>
                 | <version core> "+" <build>
                 | <version core> "-" <pre-release> "+" <build>

ValueError: Invalid version: 1.6.13+build1
To make this more complicated, remember dbt Labs ignores the -.

Not needed after 1.8 😥


Dynamic query using a runtime parameter fails after compilation with "The monad composition can't be expressed using applicative joins"

Version: 3.0.1
Module: quill-jdbc
Database: postgresql
Expected behavior
No runtime error.
Actual behavior
Runtime error.
Steps to reproduce the behavior
https://scastie.scala-lang.org/HOpz81UYR4OHsoCrQNBtZw
or more complex case
https://scastie.scala-lang.org/1JdMW7tATvGuohT4aJI8Mg
Workaround
None that I know.
@getquill/maintainers

import ctx._

case class Users(loginToken: String)

case class Params(serverLoginToken: Option[String])

val params = Params(Some("server"))

val q =
  dynamicQuery[Users].filter { t =>
    params.serverLoginToken match {
      case None => quote(false)
      case Some(id) => quote(t.loginToken == lift(id))
    }
  }

println(ctx.run(q))

This works for me. You had to lift(id) and move dynamicQuery outside of run(I don't know why, but it works)

I would vote to not close this bug.

if you need to lift the id then it is a "better-error-messages" bug.
I actually tried lifting the id before, but then the macro expanding the lift died. It should not die.

@arturaz  Feel free to reopen this.

Have run into this issue as well multiple times, the error message is very confusing, obviously, there're no joins in the query yet it fails at the runtime with: "The monad composition can't be expressed using applicative joins".


Removed warnings from rust 1 6

Hi @kenpratt ,
I just wanted to play around with the project and check if I could still build it. I found that it actually compiles. I updated a new warning message and added the authors line to Cargo.toml.
What do you think about publishing to crates.io?

Looks good, thanks! It's probably not useful to publish to crates.io since it isn't a library? What do you think?

Maybe it would be useful to other peoples to find the project. I am not sure if cargo can be used as equivalent to npm. In the npm it is possible to distribute command line apps, for examples grunt-cli. Is this possible with crates.io?


How to get script content by goquery？

<script>
    var zodiac_content = {
    "data" : [
        {
             "type" : "headline",
             "text" : "TODAY'S STAR RATINGS"
        }
    ]}
</script>

I want to get
{
   "data" : [
       {
            "type" : "headline",
            "text" : "TODAY'S STAR RATINGS"
       }
   ]}

Thank you very muck!

Hello,
Goquery manipulates HTML, so it can help you get the content of the <script> tag, but it won't help you parsing that content, you will need a javascript parser for that.
To get the content of the <script> tag, something like that would work:
var data = `
<html>
  <body>
  <script>
    var zodiac_content = {
    "data" : [
        {
             "type" : "headline",
             "text" : "TODAY'S STAR RATINGS"
        }
    ]}
  </script>
  </body>
</html>
`

func main() {
	doc, err := goquery.NewDocumentFromReader(strings.NewReader(data))
	if err != nil {
		log.Fatal(err)
	}

	s := doc.Find("script").Text()
	fmt.Println(s)
}

Hope this helps,
Martin

Hello,
Thanks for your answer.
But the content is anomalous.
I 'm use regexp solved ,
The html is
   $(window).resize(function(event) {
        $('#share-aside-bar').css('left', $('.main-wrapper').offset().left - 52);
    });
</script>
            <script>
    var zodiac_content = {
    "data" : [
        {
             "type" : "headline",
             "text" : "TODAY'S STAR RATINGS"
        },
        {
            "type" : "rating",
            "data" : [
                {
                    "name" : "Overall",
                    "rating" : 4
                },
                {
                    "name" : "Love",
                    "rating" : 4
                },
                {
                    "name" : "Career",
                    "rating" : 5
                },
                {
                    "name" : "Money",
                    "rating" : 3
                },
                {
                    "name" : "Health",
                    "rating" : 4
                }
            ]
        },
        {
             "type" : "headline",
             "text" : "TODAY'S ARIES"
        },

        {
            "type": "paragraph",
            "text": "Be ambitious. Be assertive. Above all believe, really, truly, totally believe, that you deserve the best and are capable of getting it. This can be one of the most productive times of the year for you, so stop complaining and get to work."
        },

        {
		 	"type" : "headline",
		 	"text" : "TODAY'S MATCHES"
		},
        {
             "type" : "matches",
             "data" : [
                       {
                           "name" : "Friendship",
                           "match" : 9
                       },
                       {
                           "name" : "Love",
                           "match" : 3
                       },
                       {
                           "name" : "Career",
                           "match" : 1
                       }
             ]
        }
    ]
};
    $(function () {
        $zodiac_desc = $('#zodiac_desc');

        var data = zodiac_content.data;

        var current_period_str = '' + current_period;
        var current_date = new Date(current_period_str.slice(0, 4) + '-' + current_period_str.slice(4, 6) + '-' + current_period_str.slice(6, 8));

My code is :
alltext := string(body)  //  html content
var exp, _ = regexp.Compile(`zodiac_content = ([\s\S]*]\n})`)
matchs := exp.FindStringSubmatch(alltext)
fmt.Printf("%+v", matchs[1])

I'm not sure I understand your question or how it's related to goquery? Please provide a minimal example (full code + html, runnable) and explain what you get vs what you expect. Then I can try to help you out.
Martin

is there a way to get the complete html label using goquery????

@mna it is an old thread, but I could use a hint here.
Assume my html input
<!DOCTYPE html>
<html>
<head>
    <meta charset="UTF-8">
</head>
<body>
    <script>
        console.log(hi);
    </script>

</body>
</html>

I'm trying to get the script text as you suggested above (I have the res.Body already):
doc, err := goquery.NewDocumentFromReader(res.Body)
if err != nil {
params.Error(err.Error())
}
s := doc.Find("body").Text()
fmt.Println(s)

, but all I get is non-clear data:
�Qt�w�
                             pU�(�ͱ㲁Q��)v\
@`��Z�����XT�Zb��k�T��Vb���R    UY�\�YPဴ&�������kddjZC�Ӈ)��
                                                           4
                                                            l9T���

I'm pretty sure I'm missing something here. Or..?
Thanks 😊

Hello Shay,
This is most likely that your res.Body returns binary data, not the html you expect (probably because the response's body is e.g. gzipped). You can check this by saving the res.Body content to a file (or print it to stdout) instead of passing it to goquery. To test that your code is otherwise correct, you can use a static string of html instead of the res.Body as input and you'll see the doc.Find(body) work as expected.
Please open a separate issue if there's more to discuss, as it is unrelated to the current issue and it has been closed anyway.
Hope this helps,
Martin


Feedback for "Create Space Quota Plans Using the Cloud Foundry Command Line Interface"

https://help.sap.com/docs/BTP/65de2977205c403bbc107264b8eccf4b/504fde98e8bb4bff889e49d0b2f6c28b.html
There's a typo in the code sample. The parameter should be --allow-paid-service-plans instead of --allow-paid-service-plan.

Thank you very much for pointing this out. I have fixed the typo.

Thank you for your valuable feedback contribution, @danachkov! So that we can recognize your contribution in SAP Community, please tell us your SAP Community profile URL in a reply to this comment; don't include any other text, just the URL on its own, like this:
https://people.sap.com/your-user-name

Thanks!

https://people.sap.com/dragomir.anachkov

https://people.sap.com/dragomir.anachkov


iOS 9 and ReachabilitySwift

Hey guys, there is a compile error with ReachabilitySwift and I can't seem to fix it. Could you update the files and merge the new branches perhaps please? :)

Bumped version to 0.1.3 which includes fixes to work with latest versions of the pod dependencies.
Let me know if you're hitting any more problems.
I still need to make a few updates to the README, add a CHANGELOG, refactor a few bits of code, and add support for tvOS (and maybe watchOS?). Those last two depend on this first though: https://github.com/krzyzanowskim/CryptoSwift/pull/168


MB-12592 Convert ghcapi tests

Jira ticket for this change
Summary
Converts all tests in ./pkg/handlers/ghcapi to use transactions.
Pattern for server tests conversion explains more about the approach used.
Setup to Run Your Code
A clean run of make server_test is sufficient.
Verification Steps

[x] No subtests are run with suite.T().Run(...) - they all use suite.Run(...)
[x] There is no unjustified usage of Truncate in the tests
[x] There is no unjustified usage of suite.AppContextForTest().DB() - instead suite.DB() is used directly
[x] Fatalf is deprecated in favor of other assertions/checks
[x] Go's testing package is only imported in the setup test file. (In this case there was no separate setup file)

Messages

:book:
:link: MB-12592

  Generated by :no_entry_sign: dangerJS against 84bfa82f31fd036afdfd010b825cda49f373749f

@shimonacarvalho can you say more about this:

There is no unjustified usage of suite.AppContextForTest().DB() - instead suite.DB() is used directly

I thought we wanted to be using the DB from the appctx

@shimonacarvalho can you say more about this:

There is no unjustified usage of suite.AppContextForTest().DB() - instead suite.DB() is used directly

I thought we wanted to be using the DB from the appctx

Been using the same PR text that Sandy came up with. I believe in the tests, it makes no difference within the test and suite.DB() is cleaner. We definitely want to use suite.AppContextForTest if we need an appcontext, though.

@shimonacarvalho can you say more about this:

There is no unjustified usage of suite.AppContextForTest().DB() - instead suite.DB() is used directly

I thought we wanted to be using the DB from the appctx

Been using the same PR text that Sandy came up with. I believe in the tests, it makes no difference within the test and suite.DB() is cleaner. We definitely want to use suite.AppContextForTest if we need an appcontext, though.

Ah ok, could you add that to the docs? At least, I don't think I saw it. I did find this: https://transcom.github.io/mymove-docs/docs/backend/guides/use-stateless-services-with-app-context#testing-with-appcontext
but that's actually a bit misleading since we don't actually have to add that per test suite since we add it at the top level for all test suites.

Ah ok, could you add that to the docs? At least, I don't think I saw it. I did find this: https://transcom.github.io/mymove-docs/docs/backend/guides/use-stateless-services-with-app-context#testing-with-appcontext
but that's actually a bit misleading since we don't actually have to add that per test suite since we add it at the top level for all test suites.

I could add to docs but I think we realized over time that it's pretty rare we've seen that so not that important of a pattern to correct. Just might as well watch out for it as we are cleaning up.
That doc though does need to be updated because it seems to refer to TestAppContext? When really that function (I think) is now called AppContextForTest. Not part of this ticket I think but I'll try to get that in soon.

@felipe-lee anything further you'd like fixed on this pr?


Add a way to show schedule in UI

Fix #102
Add a "ViewingPanelManager.ViewType viewType" to CommandResult
This Enum is used to figure tell the MainWindow app what to display in the UI on the right side
For example, if you type the command "schedule", the Enum will be SCHEDULE which shows the schedule, for now all other commands shows DEFAULT which is the original view.
Also added attendees interface so that I can know who is attending each lesson.

Codecov Report

Merging #103 (f671c93) into master (ce8222a) will decrease coverage by 1.41%.
The diff coverage is 15.05%.

@@             Coverage Diff              @@
##             master     #103      +/-   ##
============================================
- Coverage     61.29%   59.87%   -1.42%
- Complexity      597      598       +1
============================================
  Files           115      120       +5
  Lines          2320     2385      +65
  Branches        260      265       +5
============================================
+ Hits           1422     1428       +6
- Misses          814      872      +58
- Partials         84       85       +1

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
71.42% <0.00%> (-3.58%)
:arrow_down:

.../seedu/address/logic/commands/ScheduleCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
81.25% <0.00%> (-5.42%)
:arrow_down:

...logic/parser/persons/PersonRemoveLessonParser.java
100.00% <ø> (ø)

src/main/java/seedu/address/model/AddressBook.java
63.79% <0.00%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
74.68% <0.00%> (-0.96%)
:arrow_down:

...eedu/address/model/lesson/LessonWithAttendees.java
0.00% <0.00%> (ø)

...model/lesson/exceptions/CannotAssignException.java
0.00% <0.00%> (ø)

...lesson/exceptions/OverlappingLessonsException.java
0.00% <0.00%> (ø)

... and 12 more

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 ce8222a...f671c93. Read the comment docs.


refactor(behavior_path_start_planner_module)!: prefix package and namespace with autoware

Description
Part of https://github.com/autowarefoundation/autoware/issues/4569
Related links

Tests performed
PSIM
Notes for reviewers

Interface changes

ROS Topic Changes

ROS Parameter Changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

this PR did not add autoware namespace


Add exhaustive option to digest

Add exhaustive option to digest, making it process the full buffer (multiple state transitions), usable for DMA based applications the add entire buffers to the internal buf at a time.
For example handle some status msg\r\nAT+GMR\r\r\nOK\r\nAT+USORD=3,16\r\n+USORD: 3,16,\"16 bytes of data\"\r\nOK\r\n in a single digest

That is very valid point! Upon which i think i will rework this a bit, to make it always exhaustive.. Shouldn't be made this way though, as i'm not a huge fan of the recursive call stack.. Will give it a rework over the next couple of days!


Prepare project for Weblate integration [UA]

Proposed change

Prepare project for Weblate integration [UA]
I need to whitelist at least one .pot file to setup a test Weblate project and further play with the integration.
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've added references to all holidays information sources used in this PR
[x] The code style looks good: make pre-commit command generates no changes
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 99.978%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10275

Relevant Lines:
10275

💛  - Coveralls


WIP: 3.0.

[x] Event.capture* should redirect to Raven.capture*
[ ] Add performance improvements
[x] Fix rubocop discrepancies
[ ] Allow some fields to be excluded from sanitization
[ ] Fix Sidekiq integration via #635

Ultimately I decided not to include local variable capture - it's too unstable and buggy and could never be used in production, so the use case is too limited.

Just realized we'll need a 2.x series release where to_hash uses string keys and still supports symbol keys in order to provide a smooth transition to 3.0. Otherwise, peoples events in background job processors will get eaten up.


Error: Exception - Cannot compile typed context: 'location': member names cannot be the same as their enclosing type (line 221)

Hi,
When the Linqpad's driver is fetching the schema I get the following error:
Error: Exception - Cannot compile typed context: 'location': member names cannot be the same as their enclosing type (line 221)
I don't have a any location column in my schema so I'm not sure where this comes from.
It used to work well (3 months ago).
Any idea?
Thanks in advance

Hi, finally found the issue, we had a different dataset there with record named location with internal string member also named location. So the exception is probably when trying to compile a class named Location with same member name which is not allowed.
Up to you if you wanna solve it..
For now I just set only the dataset I needed which doesn't have this problem.


adding workflow file

adding workflow.yml
Badge is commented right now. checking if the Test/build process works fine.

missing pyproject.toml


Deduplicate test_basic image_array_copy

Merge image_array_copy, image_array_copy3d
as they share a lot of common code.
Signed-off-by: John Kesapides<EMAIL_ADDRESS>

2024/07/02 teleconference: @JohnKesapidesARM This needs rebasing.

2024/07/02 teleconference: @JohnKesapidesARM This needs rebasing.

@kpet Rebased and all CI checks passing.

Merging as discussed in the July 9th teleconference.


GraphView Height

This libraray is good i have used this and also custmize this according to requirement but the one thing that i am unable to correct is height of graph view. I want to show this graph view in dialogFragment and the height of fragment is wrapContent and height of graphView is also wrap content but it is still showimg on full screen. Any idea how can i reduce its height or how can i correct its height according to my graph.

The ZoomLayout we're using is measuring itself as match_parent. I'm not quite sure why they do this, but as a workaround you could create a class which is derived from GraphView and override the onMeasure method and measure the view to be as big as the child. The child (our internal node container view) will always be as big as your graph.

I'm closing this because of inactivity. Please re-open if you want to add something to this issue.


engine: make otel proxy shutdown error non-fatal

Fixes https://github.com/dagger/dagger/issues/8934
I haven't tracked down why, but it appears that sometimes in our test
workflows the "shutdown otel proxy" cleanup step takes over 10s, hits
its timeout and then causes the exec to fail.
It shouldn't take that long, but in the meantime there's no reason for
this to be fatal to the exec. The error is now just logged.

@vito let me know what you think of this workaround. I'm a bit mystified and concerned why shutting down the otel proxy server is taking >10s sometimes. The only recent change I can think of that's plausibly related is getting rid of the transactions, which I suppose could somehow be causing the http handlers take longer? Maybe? Pretty shot in the dark guess.
But for now it at least stops this from breaking user execs.

Caught it after a few re-runs: https://dagger.cloud/dagger/traces/6d46046448a86fe4dcc83f4e0f9761b6?span=f64e19849bc74e52&logs
Was indeed a cleanup func, specifically shutdown otel proxy.
Need to look into why this started happening recently. Not sure if something is just taking longer than it used to (maybe related to more otel data flowing due to metrics?) or if something is actually going wrong.
Either way, that doesn't seem like something that should be fatal to an exec at least at face value.

Follow-up: https://github.com/dagger/dagger/pull/8950


Can't run headless client integration tests

Great project there.
I ve tried a lot of things but I can't run client integration tests. I've versioned up and down meteor-jasmine. And checked the logs but I cant see what's wrong. Here is a lame sample project to reproduce what I get : https://github.com/cwartel/jasmine-lame
when I JASMINE_BROWSER=PhantomJS meteor --test
I get
stream error Network error: ws://localhost:3000/websocket: connect ECONNREFUSED
[[[[[ ~/jasmine-lame ]]]]]

=> Started proxy.
=> Meteor <IP_ADDRESS> is available. Update this project with 'meteor update'.
=> Started MongoDB.
=> Started your app.

=> App running at: http://localhost:3000/
W20150322-18:25:04.347(1)? (STDERR) connections property is deprecated. Use getConnections() method
PASSED jasmine-server-unit : Lame test => Lame test passes 2
PASSED jasmine-server-integration : Lame test => Lame test passes 2
PASSED jasmine-client-unit : Lame test => Lame test passes 2

jasmine-client-integratin never shows and the command is stuck.
tail .meteor/local/log/jasmine-client-integration.log shows this
[velocity-mirror] Mirror is initializing...
LISTENING
[velocity-mirror] Mirror is ready.

I've also reproduced it inside a docker. I could provide the docker file if needed. What am I missing ?

Is it working when you run your tests in Chrome?

Its worse : client unit tests are stuck too. But I've come across that one, I needed to load the karma page that would pop on its own. I'm more focused on headless tests.

When it is working in Chrome but not PhantomJS its probably:

You use something that is not supported by PhantomJS. For example Object.bind.
There seems to be a problem with iron:router redirects that I haven't figured out myself yet.

Yes I know there's something with iron router but it's not included in the example project I provided. And I dont get where I could Object.bind as I just added lame tests to the base meteor project : https://github.com/cwartel/jasmine-lame.

Thanks for the repo. It works fine on my Macbook.
Could be related if your machine is slow: https://github.com/meteor/meteor/pull/3957 (this is not yet released)
Also the mirror uses a random port. This could maybe cause problems with Docker. You can fix the port by using JASMINE_MIRROR_PORT=5000 JASMINE_BROWSER=PhantomJS meteor --test.

Tried JASMINE_MIRROR_PORT=5000 JASMINE_BROWSER=PhantomJS meteor --test without success. will try on a faster machine and/or wait for meteor/meteor#3957

Are you maybe using something like this in your route configuration?
Router.onBeforeAction(ensureLogin, {
only: [ "home", "user" ]
});
After removing the ensureLogin from the home route, the client integration tests began to work. (The test might simply to login first?)

Updating to the latest Meteor and sanjo:jasmine andr unning the tests with this command should hopefully work: https://github.com/Sanjo/meteor-jasmine#running-tests-in-continuous-integration
Worked for me.

It works ! thanks for your time on that matter :
JASMINE_BROWSER=PhantomJS meteor --test --release<EMAIL_ADDRESS>--port 4000
=> Using<EMAIL_ADDRESS>as requested (overriding Meteor <IP_ADDRESS>)

stream error Network error: ws://localhost:4000/websocket: connect ECONNREFUSED
I20150422-23:03:50.620(2)? [velocity] jasmine-client-integration is starting a mirror at http://localhost:48267/.
I20150422-23:03:50.638(2)? [velocity] jasmine-server-integration is starting a mirror at http://localhost:56377/.
I20150422-23:03:50.645(2)? [velocity] You can see the mirror logs at: tail -f /home/wartel/jasmine-lame/.meteor/local/log/jasmine-client-integration.log
I20150422-23:03:50.645(2)? [velocity] You can see the mirror logs at: tail -f /home/wartel/jasmine-lame/.meteor/local/log/jasmine-server-integration.log
W20150422-23:03:50.654(2)? (STDERR) connections property is deprecated. Use getConnections() method
W20150422-23:03:50.654(2)? (STDERR) Could not mock the export WebAppInternals of the package webapp. Will continue anyway.
PASSED jasmine-server-unit : Lame test => Lame test passes 2
PASSED jasmine-client-unit : Lame test => Lame test passes 2
I20150422-23:04:12.307(2)? [sanjo:jasmine]: You no longer need to wrap your server integration tests in Jasmine.onTest ;-)
PASSED jasmine-server-integration : Lame test => Lame test passes 2
PASSED jasmine-client-integration : Lame test => Lame test passes 2
TESTS RAN SUCCESSFULLY


How about license?

Oh we have no license notation in the repository.
It looks theres no license convention in ensime project(ensime-server is 3BSD, ensime-emacs is GPL, ensime-atom is MIT, etc).  Which should we choose?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27169668-how-about-license?utm_campaign=plugin&utm_content=tracker%2F18164593&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F18164593&utm_medium=issues&utm_source=github).

I'd be inclined to choose a permissive license like MIT, but maybe we can put this to a vote, here are some alternatives, feel free to add some:

GPLv3
AGPLv3
MIT
BSD 3-Clause License

Ok, let's vote then, I'll be closing the votes on sunday.
My vote:
MIT

+1 to MIT :)


Added test case for #233

I created a test case for the ordering issue named in #233. The test fails at the moment but I think shows the issue.
The test only contains synced ids, not synced dictionaries. Maybe those should be tested as well. Futhermore, in this test are no new items added, existing ones are just re-ordered.
Maybe someone has a proper solution to fix the issue by using this test case.

A test case is a great step towards finding a solution. Awesome @dersvenhesse!


Programmatically open the menu fully with width & overdraw set?

Hi, I'm trying to configure SWRevealViewController so that the left rear controller is always visible by ~50pt or so, and when an "Open" button is tapped on the menu is fully opened. When closed, the menu returns to it's original 50pt width. The idea is to always have a sliver of the menu visible to indicate to users that there's a menu there.
By setting rearViewRevealWidth to 50 I can achieve the first part of the left rear menu always being partially visible, but I can't figure out how to fully open the thing programmatically. Calling             self.revealViewController().revealToggleAnimated(true) hides the menu, and calling it again reveals the menu but only to 50pt.
Is this possible?

Check the position of front view controller and change the value of rearViewRevealWidth consequently before reveal toggle

Sorry, I guess I'm not following. Are you suggesting I do something like this?
class BaseViewController : UIViewController, SWRevealViewControllerDelegate {
    private var isMenuOpen = false

    override func viewDidLoad() {
        super.viewDidLoad()

        let revealController = self.revealViewController()
        revealController.rearViewRevealWidth = 50.0
        revealController.setFrontViewPosition(.Right, animated: true)

        let tapRecognizer = UITapGestureRecognizer(target: self, action: "handleTap:")
        tapRecognizer.allowedPressTypes = [NSNumber(integer: UIPressType.Menu.rawValue)]
        self.view.addGestureRecognizer(tapRecognizer)
    }

    func handleTap(sender: UITapGestureRecognizer) {
        if sender.state == UIGestureRecognizerState.Ended {
            let revealController = self.revealViewController()
            revealController.rearViewRevealWidth = isMenuOpen ? 50.0 : 600.0
            revealController.revealToggleAnimated(true)
        }
    }
}

For reference, I'm shiming this into a tvOS app (hence the UIPressType.Menu gesture recognizer).


Test Issue created at Sun, 23 Jan 2022 02:18:56 GMT

Test issue body  Sun, 23 Jan 2022 02:18:56 GMT

Comment on issue at Sun, 23 Jan 2022 02:18:58 GMT


How to get an entity data in blockRendererFn if DraftEntity.get is deprecated?

I want to bulid some custom block components ( like image ).
Then, I hope they are in their own file.
Below is a approach I found in draft-js-image-plugin

But, how to get an entity data in blockRendererFn if DraftEntity.get is deprecated?

You could make use of currying, and pass a getter function for your editorState:
blockRendererFn: (getEditorState) => (block) => {
  const editorState = getEditorState()
  const contentState = editorState.getCurrentContent()
  // Now you can use your existing code, but with contentState.getEntity() instead
}

Use it like this with your editor component:
<Editor blockRendererFn={blockRenderFn(this.getEditorState)} {...yourOtherProps} />
Where the getEditorState function only returns the editorState:
getEditorState = () => this.props.editorState

Good idea. I will try it,thank you very much.


kde-rounded-corners: support on Plasma 6

Description of changes

switch package to kdePackages and Qt6 for Plasma 6 support
add devusb to maintainers

Closes #312034
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release 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.

Add a :+1: reaction to pull requests you find important.

can confirm this works on plasma 6.0.4

@ofborg eval


Fix start of autocompletion

There was a bug that caused the omnicompletion to start the completion one column ahead. This PR fixes this and also adds no constraints to run autocompletion when the user asks for it. For instance, now we allow it even if you don't have written any seed.

Review by @ChrisCoffey.

Please, LGTM as soon as you can as I want to merge this soon @ChrisCoffey :smile:

taking a look now

Other than my one comment this looks great


WFLY-8723 move elytron tests to core

This will almost certainly fail tests. I'll link a custom integration job.
Core PR: https://github.com/wildfly/wildfly-core/pull/2378

Integration test here: https://ci.wildfly.org/viewLog.html?buildId=59338&tab=buildResultsDiv&buildTypeId=WF_WildFlyCoreIntegrationExperiments

Merged via incubator.


Typo in "Calling a Web API"

https://github.com/rust-lang-nursery/rust-cookbook/blob/752b035c1ae1646c999b4d7c39c78d67215d5893/src/web/clients/api/rest-get.md?plain=1#L9
I think it should be [reqwest::get]  instead of [reqwet::get] .

sorry this is a duplicate, there is already a pull request for this: #651


activerecord: Fix #find does not take string(s) as primary key(s).

ActiveRecord supports automatic conversion from String to Integer on #find call.
The API document of ActiveRecord::FinderMethods#find says:

If the primary key is an integer, find by id coerces its arguments
by using to_i.
https://api.rubyonrails.org/classes/ActiveRecord/FinderMethods.html#method-i-find

Unfortunately, it's difficult to add the _ToI type only if the primary key is an integer.  So this adds it always.
Is it better to override the signature in rbs_rails side?

In my case, I passed a primary key typed as String read from CSV.
# image code
CSV.read(filename).each do |row|
  User.find(row[0])
end

@tk0miya Is the following correct?

CSV.read returns Array[Array[String?]]
row is Array[String?]
row[0] is String | nil
String | nil is not PrimaryKey(String)
You want this code to be error free.

Ah, sorry. I missed your message.

Is the following correct?

Almost right.
More precisely, our production code checks row[0] is not nil. So my sample code is not precise.
# image code
CSV.read(filename).each do |row|
  user_id = row[0]
  next if user_id.nil?

  User.find(user_id)  #=> User.find(String)
end

In my thought, it's not strange that ID is converted to String when we interoperate with other systems. Some communication passes ensure serializing IDs to String. For example, CSV and HTTP query params represent all data as String.
Rails and ActiveRecord support the use of String IDs easier by auto conversion. So I think adding _ToI is familiar with Rails philosophies.
As you said, the conversion works only if the primary key is an integer. So adding it to gem_rbs_collection might not be good. what do you think adding this to rbs_rails? If agreed, I'll post this to the repo later.

If the primary key is a string, PrimaryKey == String should hold. Since String satisfies _ToI, it seems that this change is unnecessary and type checking should pass.
However, there may be some issue that makes this assumption incorrect, which is why I am asking what the problem is.
With the current approach to the changes, I am unlikely to merge.

If the primary key is a string, PrimaryKey == String should hold. Since String satisfies _ToI, it seems that this change is unnecessary and type checking should pass.

In my case, the primary key is an integer.

However, there may be some issue that makes this assumption incorrect, which is why I am asking what the problem is.
With the current approach to the changes, I am unlikely to merge.

As I commented first, I think this is better to move rbs_rails. So I'm withdrawing this now.

Is it better to override the signature in rbs_rails side?

Thank you for your reviewing. Thanks,


Fix typo of wrapper from warpper

Thank you for your contribution. This PR has been manually cloned and submitted upstream, then automatically migrated here as https://github.com/google/quiche/commit/56edfdd61ab38a9eea92c06502ce62df0cb68f05.


Passwords in UI and CLI displayed in plain text

Moved from https://github.com/StackStorm/st2/issues/5145

Looking at that issue, it looks like we need to ensure obfuscation is happening in the rules widgets and possibly other places where something is marked secret: true

@cognifloyd Is this something that could be added to a shortlist of features for an upcoming version? Could be a potential barrier to enterprise adoption.

@ip-sf Contributors welcome!
I do not develop much with st2web, and I have a lot on my plate in other parts of the st2 project, so someone else will need to pick this up.

Understood. I wish i had the time to dive into this. Perhaps I will tackle this at somepoint and get my feet wet on more than just packs contributions :)
Thanks!


In Get Request, I try to encode "+"  to "%2b", but it not working.

Describe the bug
this is my requeset "  https://china-merchant.wish.com/api/v3//products/?sort_by=updated_at.asc&updated_at_min=2022-01-09T16:05:41.041%2B08:00  "
i need encode the plus (+) symbol in a URL in a Get request;
but this is the error message from wish api  "{"message":"'2022-01-09T16:05:41.041%2b08:00' is not a 'date-time' for parameter 'updated_at_min'. Acceptable format should be like YYYY-MM-DDThh:mm:ss.sss+hh:mm","code":1003,"data":{}}"
To Reproduce
Steps to reproduce the behavior:

String url =  url.replace("+","%2b")
String token = getToken(storeAcctId);
return Unirest.get(url)
.header("authorization", token)
.asString();

Expected behavior
this same url work in postman,so i wish it work  like in postman
Screenshots

Environmental Data:

Java Version
Version [ 1.8.0_281]

Unirestis not postman and we don't have any intent of mimicking it's behavior but as far as I can tell Unirest is doing exactly what you are asking for and sending the param with the encoded +. If I post to http bin I get this:
String body = Unirest.get("https://httpbin.org/get?date=2022-01-09T16:05:41.041%2B08:00")
.asString()
.getBody();
{
  "args": {
    "date": "2022-01-09T16:05:41.041+08:00"
  },
  "headers": {
    "Accept-Encoding": "gzip",
    "Host": "httpbin.org",
    "User-Agent": "unirest-java/3.1.00",
    "X-Amzn-Trace-Id": "Root=1-6203c7b9-255616fc4bbc7f287fbf86d8"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/get?date=2022-01-09T16:05:41.041%2b08:00"
}

It looks like from your screen shot that Postman is turning your encoded value back to +, or that might be a artifact of the logger, I don't know, I don't use Postman.
One note is that your datetime isn't really properly encoded, the colons should also be encoded to make a proper URL encoded param. You can do this with formal params like this:
String body = Unirest.get("https://httpbin.org/get")
.queryString("date", "2022-01-09T16:05:41.041+08:00")
.asString()
.getBody();
which results in
{
  "args": {
    "date": "2022-01-09T16:05:41.041+08:00"
  },
  "headers": {
    "Accept-Encoding": "gzip",
    "Host": "httpbin.org",
    "User-Agent": "unirest-java/3.1.00",
    "X-Amzn-Trace-Id": "Root=1-6203c896-44d49c2d2026bf712387fb30"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/get?date=2022-01-09T16%3A05%3A41.041%2B08%3A00"
}

Unirestis not postman and we don't have any intent of mimicking it's behavior but as far as I can tell Unirest is doing exactly what you are asking for and sending the param with the encoded +. If I post to http bin I get this:
String body = Unirest.get("https://httpbin.org/get?date=2022-01-09T16:05:41.041%2B08:00") .asString() .getBody();
{
  "args": {
    "date": "2022-01-09T16:05:41.041+08:00"
  },
  "headers": {
    "Accept-Encoding": "gzip",
    "Host": "httpbin.org",
    "User-Agent": "unirest-java/3.1.00",
    "X-Amzn-Trace-Id": "Root=1-6203c7b9-255616fc4bbc7f287fbf86d8"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/get?date=2022-01-09T16:05:41.041%2b08:00"
}

It looks like from your screen shot that Postman is turning your encoded value back to +, or that might be a artifact of the logger, I don't know, I don't use Postman.
One note is that your datetime isn't really properly encoded, the colons should also be encoded to make a proper URL encoded param. You can do this with formal params like this:
String body = Unirest.get("https://httpbin.org/get") .queryString("date", "2022-01-09T16:05:41.041+08:00") .asString() .getBody();
which results in
{
  "args": {
    "date": "2022-01-09T16:05:41.041+08:00"
  },
  "headers": {
    "Accept-Encoding": "gzip",
    "Host": "httpbin.org",
    "User-Agent": "unirest-java/3.1.00",
    "X-Amzn-Trace-Id": "Root=1-6203c896-44d49c2d2026bf712387fb30"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/get?date=2022-01-09T16%3A05%3A41.041%2B08%3A00"
}

same code,different result
{
"args": {
"date": "2022-01-09T16:05:41.041%2b08:00"
},
"headers": {
"Accept-Encoding": "gzip",
"Host": "httpbin.org",
"User-Agent": "unirest-java/1.3.11",
"X-Amzn-Trace-Id": "Root=1-620463a8-11c75bf1455b0aa1213bb224"
},
"origin": "<IP_ADDRESS>",
"url": "https://httpbin.org/get?date=2022-01-09T16:05:41.041%252b08:00"
}