Eagerly resolve toolchains to canonical, fixed reference

This also gets rid of the update-hashes/ directory (it won't literally get rid of it on existing installs but the contents do not matter anymore).

I would love to see this change! Avoiding multiple installs of the same toolchain under different names would be wonderful (and save me disk space). What will be procedure for users updating elan while already having such toolchain installed (and ones for stable/nightly)?

What will be procedure for users updating elan while already having such toolchain installed (and ones for stable/nightly)?

I would prefer the same simple strategy as getting rid of unused toolchains so far: rm :)


flask plugins

Support UT at flask plugins

[x] linux
[x] macos

package
https://test.pypi.org/project/pinpointPy/1.0.38/


fix(TTY): export GPG_TTY before gpg signing

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


cant run with usage

when i run with python -m EV_sim i got error
AttributeError: 'DriveCycle' object has no attribute 't'
python：3.11.4
system: windows 10
ide: pycharm

Hi, thank you for letting me know of the error. It seems to be working at my end. If it's not a problem, can you copy all of the error messages displayed when you execute that command?

Hi, thank you for letting me know of the error. It seems to be working at my end. If it's not a problem, can you copy all of the error messages displayed when you execute that command?

Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\AppData\Local\miniconda3\Lib\tkinter\__init__.py", line 1948, in __call__
    return self.func(*args)
           ^^^^^^^^^^^^^^^^
  File "<string>", line 1, in <lambda>
  File "C:\Users\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 210, in lstbox_callback
    self.parent.parent.DisplayFrameInstance.create_widget(self.main_heading_text + '-' + category +
  File "C:\Users\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 384, in create_widget
    SubDisplayPlots(self, plot_info_dict=self.plot_info_dict[category],user_selection=user_selection)
                                         ^^^^^^^^^^^^^^^^^^^
  File "C:\Users\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 329, in plot_info_dict
    'Desired Acceleration': [sol_obj.t, sol_obj.des_acc, 'Time [min]', r'Desired Acceleration [m/$s^2$]'],
                             ^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 't'
Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\AppData\Local\miniconda3\Lib\tkinter\__init__.py", line 1948, in __call__
    return self.func(*args)
           ^^^^^^^^^^^^^^^^
  File "<string>", line 1, in <lambda>
  File "C:\Users\Desktop\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 210, in lstbox_callback
    self.parent.parent.DisplayFrameInstance.create_widget(self.main_heading_text + '-' + category +
  File "C:\Users\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 384, in create_widget
    SubDisplayPlots(self, plot_info_dict=self.plot_info_dict[category],user_selection=user_selection)
                                         ^^^^^^^^^^^^^^^^^^^
  File "C:\Users\batteries_smiu\EV_sim\EV_sim-main\EV_sim\tkinter_gui\main.py", line 329, in plot_info_dict
    'Desired Acceleration': [sol_obj.t, sol_obj.des_acc, 'Time [min]', r'Desired Acceleration [m/$s^2$]'],
                             ^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 't'

here is the error stack info

In the GUI, if you click on the External Conditions > input the values of the air pressure, road grade, and road force > Press Simulate (on the top bar).
Then you will be able to view the simulation results by clicking on any of the results in the results options.
An error is shown in case of missing inputs.


Lower title (h3) margins in refine & some small other stuff

Some changes happened to refine recently that made it more tight, so the margins for the h3 ended up not fitting the new size
Before:

After:

Some refine category & refine searchbox responsive size improvements to go along, as to avoid either of them looking either too big or too small at all resolutions aesthethical-wise
Put log-in icon at a lower height level and add a title to it with the text, useful in the case the log-in text is too big

Coverage remained the same at 24.341% when pulling 08f6c225ff9a015b1a2cb3c894a018c020f56414 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling 08f6c225ff9a015b1a2cb3c894a018c020f56414 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling 08f6c225ff9a015b1a2cb3c894a018c020f56414 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling 08f6c225ff9a015b1a2cb3c894a018c020f56414 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling a34385ea9acb29f4c0af6bf76fc7d698f6ccdde3 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling a34385ea9acb29f4c0af6bf76fc7d698f6ccdde3 on Kiloutre:patch-27 into eeaeb644bfefd8ef403e727f924390c5f2078b99 on NyaaPantsu:dev.

Coverage remained the same at 24.341% when pulling d4def3c257b14f647eb60e3e7c00bfc0c4e25c57 on Kiloutre:patch-27 into 645a711222b94f042e0c2184533aa5e7b77b9ecb on NyaaPantsu:dev.


Revised VSS RFC

Replaces and extends the previous PR for VSS module.

Other open questions that would document for completeness:

Will ACLs be supported?
ACLs are not supported in the current version.

Is active defragmentation supported?
Memory allocated by the module is not subject to active defragmentation.

Is the RDB compatible with redis search.
No, the RDB format is not compatible with RediSearch. The module utilizes a proprietary RDB format based on protobuf, which serializes both index metadata and content. In contrast, RediSearch's RDB format only serializes the index metadata.

I assume this a subset of redis search functionality. but would be good to document that along with what is missing.
Yes, we should have a section which enumerates key missing functionality as bullet points to provide clarity.

How will slot migration work, if at all. Will we also copy indexes between nodes when using the cli?
Slot migration is supported. Keys from the migrated slot trigger keyspace mutation events, which are processed in the same manner as client-originated mutations, except that they do not block the client during execution.

Are any core changes required? We talked about memory sharing and some new keysspace notifications.
While the module can function without engine changes, two engine changes have been introduced to enhance user experience:

Blocking client on keyspace mutation events: Ensures visibility of mutations in consecutive queries on the same connection. This change extends the behavior of the module API RedisModule_BlockClient.
Memory deduplication: Adds new module APIs to enable memory sharing between the module and the engine, reducing duplication.

How does redirecting read requests to the primary work in cluster mode? Can you force strongly consistent reads? By the consistency model do we care?
Valkey doesn't support distributed transactions so I don't see any consistency guarantee with scatter-gather across multiple shards.
Additional context: In cluster mode, a query can be received by any node, which fans out the request to all shards. Each shard processes the query locally and returns results to the initiating node, which selects the top responses across shards. Currently, forcing reads from primary nodes only are not supported. The current logic for instance selection is designed to improve the performance, increase throughput and lower latency, by load balancing between the instances.

Are we allowing searcg in Lua and multi-exec? Will it cause latency issues? Does redis allow it?
Will ACLs be supported?

Multi-exec: Supported, but it introduces significant performance impacts. Mutation processing cannot be parallelized, and queries inside a multi-exec context must wait for preceding mutations to complete.
Lua scripts: Supported, and the client is not blocked during mutation operations triggered by a Lua script.

Made some inline editing to bullet #8 regarding to LUA.

Can we close RFC https://github.com/valkey-io/valkey-rfc/pull/8?

I just add comments for this RFC, I am not sure if we could implment more index algorithms and query method, but the link Using Vector Indexes for Data Search in an Elasticsearch Cluster give me more hint https://support.huaweicloud.com/intl/en-us/usermanual-css/css_01_0123.html


App Crashing

Hi Alex,
It has been a  very helpful resource for me.But just draw your attention  to one tiny issue in the demo project that while a section is added to the footer it works fine but when the section is deleted from the section header button ...App crashes... :|

Thanks a lot @mohshin-shah, we will look into this.


From "wiki": "Every bot uses the same GUID for authentication..."

wiki -> native Go Plugins ...
"

Example plugin .go file:

package main
import "fmt"
var Utterances = []string{"test plugin"}
var Name = "Test Plugin"
func Action(transcribedText string, botSerial string) string {
fmt.Println("(in testPlugin) Transcribed text: " + transcribedText ", serial number: " + botSerial)
return "intent_imperative_praise"
}
The plugin should be built with sudo /usr/local/go/bin/go build -buildmode=plugin plugin.go (has to be built with the same version of Go wire-pod is using and same user)
The resulting .so should be placed in ~/wire-pod/chipper/plugins/. wire-pod will load it upon startup.
.> ... blablabla..
Every bot uses the same GUID for authentication:
tni1TRsTRTaNSapjo0Y+Sw==

"
WTF? Why ???

Node-red also performs this function much more easily with a serial number...

What is it that all vectors use the same GUID authentication? Is this meant to be a joke?

Weather commands are easy to setup via weatherapi.com or > openweathermap.org
Knowledge-graph ("I have a question") commands are easy to setup via houndify

Node-Red without chipper same..

If you want to make wire-pod run in the background (as a daemon), run the following commands (press CTRL+C to stop wire-pod if you are running it):
sudo ./setup.sh daemon-enable
sudo systemctl start wire-pod

./setup.sh daemon-enable <-- ??? Really?  this is not  a typo?

I couldn't figure out the relation between the client token and token hashes. If/when I figure that out, I will probably implement unique GUIDs for every authenticated bot. Otherwise, it shouldn't affect functionality and that is all that matters to me right now.
This is not a node-red alternative.
May you point out the typo to me? A more descriptive/accurate command would be ./setup.sh daemon-install but it isn't very different


Look up on desktop does not work

Id tags used in scripts are not unique and are used multiple times.

[ ] Remove duplicate header
[ ] Make search bar longer on mobile
[ ] Mobile setup needs padding


Android - Profile - Unable to select any listed year when prompted with search results

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.31-8
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4942630&group_by=cases:section_id&group_order=asc&group_id=229064
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Open app
Tap on Profile icon
Tap on Profile
Scroll down and select Date of birth
Tap on Year selector
Write search prompt e.g. "197"
Select any listed year

Expected Result:
Unable to select a year from the list displayed in the search results
Actual Result:
Nothing happens; the user remains on the search results page.
Workaround:
Unknown
Platforms:

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/6463310a-a1a9-48e1-b873-a251a0754cb5

View all open jobs on GitHub

@kadiealexander  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I also cannot  reproduce

Swapping with @jliexpensify for our android <> ios bug swap.


core.remote action throwing an error when ssh connection using private key

SUMMARY
When running core.remote action to ssh to host using private key error is thrown
STACKSTORM VERSION
st2 --version
st2 3.7.0, on Python 3.8.10
OS, environment, install method
K8s helm chart installation
Steps to reproduce the problem
st2 run core.remote cmd=whoami hosts=xxx username=stanley private_key=/home/stanley/.ssh/stanley_rsa
Expected Results
This action should run whoami command on remote host and print output.
SSH connection using this private_key is working properly password less when running ssh command from action_runner pod:
ssh -i stanley_rsa stanley@xxx
Exactly the same private_key we have on our old stackstorm instance(st2 3.5dev (596c60c23), on Python 3.6.9) and same core.remote action is able to run successfully. I have also verified that paramiko library has different versions:

old stackstorm instance: paramiko-2.7.2.dist-info
new stackstorm instance: paramiko-2.10.1.dist-info

Actual Results
st2 run core.remote cmd=whoami hosts=xxx username=stanley private_key=/home/stanley/.ssh/stanley_rsa

.

id: 63f779e3308e12af9365df26

action.ref: core.remote

context.user: st2admin

parameters:

 cmd: whoami

 hosts: xxx

 private_key: '********'

 username: stanley

status: failed

start_timestamp: Thu, 23 Feb 2023 14:36:19 UTC

end_timestamp: Thu, 23 Feb 2023 14:36:21 UTC

result:

 error: "Unable to connect to any one of the hosts: ['xxx'].

connect_errors={

 "xxx": {

   "failed": true,

   "succeeded": false,

   "timeout": false,

   "return_code": 255,

   "stdout": "",

   "stderr": "",

   "error": "Failed connecting to host xxx. q must be exactly 160, 224, or 256 bits long",

   "traceback": "Traceback (most recent call last):\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/parallel_ssh.py\\", line 278, in _connect\

   client.connect()\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/paramiko_ssh.py\\", line 171, in connect\

   self.client = self._connect(host=self.hostname, socket=self.bastion_socket)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/paramiko_ssh.py\\", line 787, in _connect\

   client.connect(**conninfo)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/client.py\\", line 435, in connect\

   self._auth(\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/client.py\\", line 682, in _auth\

   self._transport.auth_publickey(username, key)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/transport.py\\", line 1634, in auth_publickey\

   return self.auth_handler.wait_for_response(my_event)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/auth_handler.py\\", line 244, in wait_for_response\

   raise e\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/transport.py\\", line 2163, in run\

   handler(self.auth_handler, m)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/auth_handler.py\\", line 375, in _parse_service_accept\

   sig = self.private_key.sign_ssh_data(blob, algorithm)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/paramiko/dsskey.py\\", line 109, in sign_ssh_data\

   key = dsa.DSAPrivateNumbers(\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/cryptography/hazmat/primitives/asymmetric/dsa.py\\", line 244, in private_key\

   return backend.load_dsa_private_numbers(self)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/cryptography/hazmat/backends/openssl/backend.py\\", line 826, in load_dsa_private_numbers\

   dsa._check_dsa_private_numbers(numbers)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/cryptography/hazmat/primitives/asymmetric/dsa.py\\", line 282, in _check_dsa_private_numbers\

   _check_dsa_parameters(parameters)\

 File \\"/opt/stackstorm/st2/lib/python3.8/site-packages/cryptography/hazmat/primitives/asymmetric/dsa.py\\", line 274, in _check_dsa_parameters\

   raise ValueError(\\"q must be exactly 160, 224, or 256 bits long\\")\

ValueError: q must be exactly 160, 224, or 256 bits long\

"

 }

}"

 traceback: "  File "/opt/stackstorm/st2/lib/python3.8/site-packages/st2actions/container/base.py", line 117, in _do_run

   runner.pre_run()

 File "/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/paramiko_ssh_runner.py", line 206, in pre_run

   self._parallel_ssh_client = ParallelSSHClient(**client_kwargs)

 File "/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/parallel_ssh.py", line 90, in __init__

   connect_results = self.connect(raise_on_any_error=raise_on_any_error)

 File "/opt/stackstorm/st2/lib/python3.8/site-packages/st2common/runners/parallel_ssh.py", line 131, in connect

   raise NoHostsConnectedToException(msg)

"

Any suggestion what needs to be changed and how to make core.remote action work correctly?

same error on my side :/

I have downgraded st2 from 3.7 to 3.6 and it starts to work as expected. I tried st2 3.8 with the same errors. I think that clearly indicates that issue is related paramiko python library version that was upgraded in st2 3.7. So there is a workaround, but it is still not working on st2 3.7 and 3.8 so we can't use the latest versions.

This sounds like this issue: https://github.com/paramiko/paramiko/issues/2048

Looks like we upgraded paramiko just because of dependabot alerts - https://github.com/StackStorm/st2/commit/2dc9d9003d82b63f19ce1b3a577c0b5f99ffa500.
Will try and reproduce and see if downgrading paramiko to 2.8.1 resolves issue.

Interestingly I couldn't reproduce this problem. Steps taken:

Install single instance ST2 3.8 on rocky
Setup passwordless ssh to remote node using stanley_rsa key. In my case I had stanley on ST2 logging into ubuntu user on another node.
All the following commands all succeeded:

st2 run core.remote cmd=ls username=ubuntu hosts=xxx private_key=/home/stanley/.ssh/stanley_rsa
st2 run core.remote cmd=ls username=ubuntu hosts=xxx

This is with putting the newly generated rsa key onto the remote server.
There's some more info on the paramiko error which might explain why working for me:
Typically, when the given private key is a RSA key and publickey authentication passes, the issue doesn't occur. However, when the publickey authentication fails using RSAKey, SSHClient is trying to load and authenticate using DSSKey. As the result, the issue occurs.
So I managed to reproduce this error when I put the rsa key into a seperate directory, but didn't setup on the remote server - and then I got the error about the DSA key being reported. But when I then put the public key of that key onto the remote server, then it all worked fine.
So in my testing the error:
q must be exactly 160, 224, or 256 bits long
is only happening when it fails to connect remotely with the private key. But when the key validation passes then it all works fine.
Can you double-check the location and permissions of files, and that you have matching private/public keys, as I cannot verify the problem.


CharUtilTest : Tests added

Hi,
PR consists of new tests for following jodd.util.CharUtil-methods:

toAscii
toSimpleByteArray
toAsciiByteArray
isUppercaseAlpha
isHexDigit
isGenericDelimiter
isSubDelimiter
isLowercaseAlpha
isWhitespace

I have a question to method jodd.util.CharUtil#isWhitespace:
	/**
	 * Returns true if character is a white space ({@code <= ' '}).
	 * White space definition is taken from String class (see: trim()).
	 */
	public static boolean isWhitespace(char c) {
		return c <= ' ';
	}

Why does not this method delegate to Character.isWhitespace ?
see CharUtilTest#testIsWhitespace for different results (as System.out).
Bye,
Sascha

The reason for this is the performance. The Character.isWhitespace (as far as i remember) was doing many things, so we have the simple comparison here. I found this definition elsewhere, too.
Maybe we can change the method name to explain the intent better?

I suggest some updates to javadoc, too -> "this method has other results as Character#isWhitespace."
Any suggestion for changed method name?

Just found a definition of a whitespace:

White space characters are the following Unicode characters:
Members of the SpaceSeparator category, which includes the characters SPACE (U+0020), OGHAM SPACE MARK (U+1680), EN QUAD (U+2000), EM QUAD (U+2001), EN SPACE (U+2002), EM SPACE (U+2003), THREE-PER-EM SPACE (U+2004), FOUR-PER-EM SPACE (U+2005), SIX-PER-EM SPACE (U+2006), FIGURE SPACE (U+2007), PUNCTUATION SPACE (U+2008), THIN SPACE (U+2009), HAIR SPACE (U+200A), NARROW NO-BREAK SPACE (U+202F), MEDIUM MATHEMATICAL SPACE (U+205F), and IDEOGRAPHIC SPACE (U+3000).
Members of the LineSeparator category, which consists solely of the LINE SEPARATOR character (U+2028).
Members of the ParagraphSeparator category, which consists solely of the PARAGRAPH SEPARATOR character (U+2029).
The characters CHARACTER TABULATION (U+0009), LINE FEED (U+000A), LINE TABULATION (U+000B), FORM FEED (U+000C), CARRIAGE RETURN (U+000D), NEXT LINE (U+0085), and NO-BREAK SPACE (U+00A0).

isWhitechar?
isSpacechar?

Ascii table defines <32 chars as control chars

I guess the isSpaceOrControlChar is the most precise name. Somehow feel bad as "Whitespace" word is not part of it.

Or we can simply remove the method, hm. Im really open to suggestions :)

isAsciiControlChar ?

Or we can simply remove the method, hm. Im really open to suggestions :)

the last option :-)

The reason for this is the performance

Well, i have written a benchmark -> #477 :-)

Lets see the results for the bechmark :)
btw, can you also put results of the benchmark in the javadoc of the benchmark class, so ppl can easily see the findings :)

benchmark results are added in javadoc.
Thanx for your hint!

The difference in performances is huge. Let's keep the method :)
I will add the javadoc. Not sure about the isAsciiControlChar, it's does not imply the fact that we are removing the whitechars.
I would leave it for now, but add a javadoc - what I will do right now ;)
Thank you!

Not sure about the isAsciiControlChar, it's does not imply the fact that we are removing the whitechars.

removing the whitechars?!
Thanx for explanation!


Rama front end

please resolve conflicts

pull request already approved in https://github.com/Esc-IngComp-UNC/computer-laboratory-web-page/pull/5


[Bug] [Module Name] cannot sart dolphinescheduler

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
OS： ubuntu20.04.6
deploymnet:  pseudo docker-compose
configuration: VM 4Core8G 32G SSD
follow the official docs step-by-step
all the components which the name start with dolphinscheduler were killed. only zk and pg were normal.
logs
NAME                                           IMAGE                                        COMMAND                  SERVICE                               CREATED             STATUS                      PORTS
docker-dolphinscheduler-alert-1                apache/dolphinscheduler-alert-server:3.1.6   "/bin/bash ./bin/sta…"   dolphinscheduler-alert                35 seconds ago      Exited (1) 32 seconds ago
docker-dolphinscheduler-api-1                  apache/dolphinscheduler-api:3.1.6            "/bin/bash ./bin/sta…"   dolphinscheduler-api                  35 seconds ago      Exited (1) 26 seconds ago
docker-dolphinscheduler-master-1               apache/dolphinscheduler-master:3.1.6         "/bin/bash ./bin/sta…"   dolphinscheduler-master               35 seconds ago      Exited (1) 26 seconds ago
docker-dolphinscheduler-postgresql-1           bitnami/postgresql:11.11.0                   "/opt/bitnami/script…"   dolphinscheduler-postgresql           35 seconds ago      Up 33 seconds (healthy)     <IP_ADDRESS>:5432->5432/tcp, :::5432->5432/tcp
docker-dolphinscheduler-schema-initializer-1   apache/dolphinscheduler-tools:3.1.6          "/bin/bash tools/bin…"   dolphinscheduler-schema-initializer   34 hours ago        Exited (1) 34 hours ago
docker-dolphinscheduler-worker-1               apache/dolphinscheduler-worker:3.1.6         "/bin/bash ./bin/sta…"   dolphinscheduler-worker               35 seconds ago      Exited (1) 26 seconds ago
docker-dolphinscheduler-zookeeper-1            bitnami/zookeeper:3.6.2                      "/opt/bitnami/script…"   dolphinscheduler-zookeeper            35 seconds ago      Up 33 seconds (healthy)     2181/tcp, 2888/tcp, 3888/tcp, 8080/tcp

docker logs
docker logs -f docker-dolphinscheduler-api-1
#
# There is insufficient memory for the Java Runtime Environment to continue.
# Cannot create GC thread. Out of system resources.
# An error report file with more information is saved as:
# /opt/dolphinscheduler/hs_err_pid8.log

What you expected to happen
every component was running
How to reproduce
not sure.on my laptop, it always happens.
Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The root cause is There is insufficient memory for the Java Runtime Environment to continue.
Please check the system memory usage.

thanks for your reply.
but I have 8G memory.
how much RAM should the product need?

#
# There is insufficient memory for the Java Runtime Environment to continue.
# Cannot create GC thread. Out of system resources.
# An error report file with more information is saved as:
# /opt/dolphinscheduler/hs_err_pid11.log
root@cfbd1ded3a66:/opt/dolphinscheduler# df -h
Filesystem                         Size  Used Avail Use% Mounted on
overlay                             43G  8.5G   32G  22% /
tmpfs                               64M     0   64M   0% /dev
tmpfs                              5.9G     0  5.9G   0% /sys/fs/cgroup
shm                                 64M     0   64M   0% /dev/shm
/dev/mapper/ubuntu--vg-ubuntu--lv   43G  8.5G   32G  22% /etc/hosts
tmpfs                              5.9G     0  5.9G   0% /proc/asound
tmpfs                              5.9G     0  5.9G   0% /proc/acpi
tmpfs                              5.9G     0  5.9G   0% /proc/scsi
tmpfs                              5.9G     0  5.9G   0% /sys/firmware
root@cfbd1ded3a66:/opt/dolphinscheduler# free -h
               total        used        free      shared  buff/cache   available
Mem:            11Gi       391Mi        10Gi       1.0Mi       616Mi        11Gi
Swap:          3.1Gi          0B       3.1Gi

i think the RAM is enough.but it still error
@SbloodyS


ANALOG_SUFFIXES ?

This variable doesn't seem necessary and probably shouldn't be hardcoded into the timeline module in any case?
If you have a timeline then you can extract the suffices from the variable names, such that they're always up to date.

The variable is used in display_new. Plus it could be used for sanitization? (Which is the reason why I defined it in timeline.py)

Not sure what you mean. If these suffices are extractable from the timeline (and display uses the timeline), then why are they needed here?
More importantly, this seems like a very specific usecase. Other users would have more or fewer suffices and changing the constant in the module doesn't seem the right place. This should be at the user's layer of abstraction. In any case, if you're thinking about ADwin then it should be in that module.


How to call event Close dialog from inside dialog component?

Hi I made my own close button but didn't find the dialog close event.
How to call event Close dialog from inside dialog component?

It will be easier if you send out the action of your button.

this.$emit('submit', { ...some result })


MobileScannerController.analyzeImage() method return wrong value when we click image using camera

Capture a photo return correct value
Capture another photo with different QR code it returns the old value(returns value from image captured in 1 step)

should this is accept as the answer  ?

I had the same problem: LOADING a QR code image from an gallery, I couldn't recognize it, the analyzeImage method always returned false

Please check with latest version v3.0.0-beta.4

mobile_scanner: ^3.4.1
I also have same problem. Also, screenshot image is not working.


Support controlling whether or not provenance/attestation information is included

Recent versions of BuildKit support (and maybe enable by default?) including provenance information in the images they generate. https://github.com/docker/build-push-action recently changed to enable this by default (in v4), however offers a mechanism to turn it off.
It would be great if this builder action also supported control over these properties. My main use-case is that I want to turn it off, though I appreciate others may want to keep it and/or have finer grained control.

yes we can support attestation since the buildx backend should already support this. I will work on this sometime later this week.

For clarity: I believe that the images produced already include attestation -- I'd like to be able to turn it off.

Yes this should be synonymous to build-push-action attestation support.


Projectile Improvements

If you shoot at a person that is critted now it will only hit if you aim at that person otherwise go "above" him and hit other targets.

[ ] Make dead people hitable
[ ] Make it work with hitscan weapons

(Still W.I.P do not merge)

i think you accidentally committed your firelock construction graph on this branch

Yes I am working on a bigger projectole rework in another branch


XPath.name and XPath::HTML.name should follow Ruby conventions

The mutant gem, and other code tools, depend on each class defining a singleton_method called name that returns a String or nil. XPath and XPath::HTML don't follow that convention.
See mbj/mutant#338 and the tickets referenced from there for more details.
In particular, see sparklemotion/nokogiri#1635, which I mistakenly opened thinking that this issue was coming from nokogiri when in reality it was coming from xpath.
I am using xpath 2.0.0 (indirectly--it's a dependency of capybara which I am using).

@jnicklas Any thoughts on this?  Possibly renaming it to qname or something like that?  Obviously it would be a breaking change (along with the XPath::HTML removal) and would probably be best to have a deprecation release first.

@jnicklas  FYI - I don't believe the method is used in Capybara.


Upload Progress Issue

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Track POST upload progress.
What did you expect to happen?
Get frequent progress updates in callback block with values ranging from 0.0f to 1.0f.
What happened instead?
Block is invoked just once with a value of 1.0f
AFNetworking Environment
AFNetworking version:4.0.1
Xcode version:13.2.1
Swift version:5.0.0
Platform(s) running AFNetworking:iOS
macOS version running Xcode:11.5.2
Demo Project
https://stackoverflow.com/questions/67874854/afnetworking-4-upload-progress-issue)

I have this problem as well. Has anyone figured out the cause?


Unable to create addition interface in Deployment

I am using a Rancher k3s cluster with calico as the main network plugin and using multus to provide an additional interface to one of my pods. I can successfully create a pod with the additional interface but when I use a Deployment resource the additional interface is not appearing on the pod.
I first placed the annotation in the deployment metadata
apiVersion: apps/v1
kind: Deployment
metadata:
name: ...
labels:
app: ...
annotations:
k8s.v1.cni.cncf.io/networks: macvlan-conf
.
.
.
and when it didn't work I realized the annotation was on the pod so it needed to be in the pod template metadata:
apiVersion: apps/v1
kind: Deployment
metadata:
name: ...
labels:
app: ...
spec:
replicas: 1
selector:
matchLabels:
app: ...
template:
metadata:
labels:
app: ....
annotations:
k8s.v1.cni.cncf.io/networks: macvlan-conf
spec:
containers:
.
.
.
This hasn't worked for me either.
In examining closed issues here I see moving the annotation to the template metadata section was sufficient for resolving some users issues but it isn't working for me.
I see one user submitted this same issue a couple years ago
https://github.com/intel/multus-cni/issues/196
and indicated his problem was resolved by enabling an RBAC permission so that Multus can read the Pod information from the Deployment vs the Pod. However this user user did not describe his particular solution and I am unsure what I need to do.
Both my network-attachment-definition and my deployment reside in the default namespace for now.
Any help would be appreciated.

Out of curiosity, does it work when you use it against a pod? (as opposed to a deployment)

Yes, it works fine against a pod. But when applied to a deployment the interface does not appear.

Okay. My fault for failing to reapply changes to my Deployment file. Adding the k8s.v1.cni.cncf.io/networks: macvlan-conf annotation to the metadata section of the Pod template in side the Deployment file was sufficient. This issue is closed.


Transaction refactor

Why we need this refactor
Since we support UDF, user created macros should have different visibilities to read/write transactions. E.g. user created macros should only be visible to the statements within that write transaction until it commits. To achieve this, binder needs to use the transaction pointer when matching macros from the catalog.
Plan:

Move transaction pointer from executionContext to clientContext so binder can access this transaction pointer as well. Client context should also be saved in the executionContext, so processors (operators) can access the transaction pointer.
We should prepare statements in a transaction. When the user calls to prepare(), we should check whether there is an active transaction. If not, we should start an auto-commit transaction.
Refactors the catalog APIs:
We should stop exposing the getReadOnlyVersion(), getWriteVersion() to the caller. Instead, each get API in the catalog should take in a transaction pointer.

This is no longer relevant.


RFE: read config from environment variables

It would be great to have chacractl read the API key from environment variable(s) that Jenkins can set.
This would allow us to avoid writing the API key to disk on the Jenkins slaves, and it would give us cleanup for free as well.

This is already done: https://github.com/ceph/chacractl/blob/master/chacractl/main.py#L48


no-redundant-story-name: add CSF2 support

Issue: N/A
What Changed
As title says
Checklist
Check the ones applicable to your change:

[ ] Ran yarn update-all
[x] Tests are updated
[ ] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] maintenance
[ ] documentation
[x] patch
[ ] minor
[ ] major

self merging @shilman


Add Main Icon to Windows

As there is a new icon for ProDBG it would be good to have the Windows version include it (currently only in Mac version) it may be that minifb needs to be updated to support it.

Not sure if this issue means what I think it means, but here is an example of linking an icon into a Rust Windows executable https://github.com/klutzy/rust-windows/tree/master/examples

Yeah the task is pretty much to get a icon on the Windows version which isn't the default one. I think Tundra supports compilation of resources files so linking that one it should be fine but likely need to add some things to minifb so it can be set on window creation also.


adjust size of table to screen size (proposal)

This adds scroll bars to the table, and makes the specified width a max width.
I use this on my website; I specified the width in React code, and users with large monitors can use the very large table effectively. If they are offsite on their mobile, they can still access the table by scrolling through the rows and columns.
Perhaps this should be a configurable option; if I had a hint towards a good approach, I would be happy to implement it.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Thanks for your pull request. I apologize for leaving this project in limbo and not addressing your concerns. I have archived this project after doing a final release to make it compatible with React v16.


Make coding inside notebook has the same experience as coding in vscode tab

I'm loving the ability to directly open and edit notebook in vscode 1.39.1. However, many of the things I love about vscode does not work in the notebook tab. For example:

Jump to definition of variable by Cmd + Click
Search (Cmd + F does not work)
Cmd + S does not save the notebook

Is it possible to give coding inside notebook the same experience as coding in a normal vscode tab? Thanks!

The short answer, sorry but not likely.
The longer answer - we need VS code to support a bunch of stuff before we can make this happen.
See these issues:
https://github.com/microsoft/vscode-python/issues/7903
https://github.com/microsoft/vscode-python/issues/5959
https://github.com/microsoft/vscode-python/issues/7875

Remaining issues in this bug are covered by others.

Jump to variable def: See https://github.com/microsoft/vscode/issues/34739
Search (Ctrl-F): This is supported in the latest released build of the extension
Cmd-S to save the notebook is coming with https://github.com/microsoft/vscode-python/issues/7903


Is there a way to provide a port other than 1935?

In the streamJsonStr object if I use server1.com or server1.com:1935 as the value for the "rtmp" key, then it works fine, but if I try server1.com:2935 it fails with IO error
Is providing non-default port allowed?

Any updates?

Sorry for the late reply.
You can do like this:
mProfile.setDnsManager(null)

thanks, I will try this setting and let you know how it works.
Could you please share some details on how/why this setting fits in the entire PLDroidStreaming scenario. I mean why was DNS manager introduced in first place and why setting this to null would help connect to ports other than 1935?

The port didn't write back into the final URL after DNS manager working. It's a bug.

Is it already fixed in 1.7.0 or will it be fixed in later versions?

Is this already fixed in v2.0.1


win: restore file position after positional read/write operation

File read or write from specified position will move file pointer on Windows but not on POSIX. This makes Windows behave as other supported platforms: for writes/reads with provided offset, file position before operation is saved, and then restored once the operation completes.
Ref: https://github.com/nodejs/node/issues/9671

I've noticed this before, it surfaces as an undocumented aspect of the node fs.read/write calls. The windows implementation can't be atomic, as pread()/pwrite() are, but I don't think that matters, and it would be good to bring Unix/Windows into alignment here.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/279/

Updated, PTAL

Rebased, PTAL

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/285/

Ping?

Updated, PTAL

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/291/

@saghul, can I get +1 from you?

Landed in https://github.com/libuv/libuv/commit/0bd8f5bf36b4164b071af6b0bb68c814cf7dedf8


fix: fix metadata retry backoff invalid when get metadata failed

fixes https://github.com/Shopify/sarama/issues/1711

fixes https://github.com/Shopify/sarama/issues/1711


"Launch With" integration with SevenBridges

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 2.X
┆Issue Number: DOCK-442
┆Epic: Launch-with Support

Woo!
:fireworks: :champagne:
https://github.com/dockstore/dockstore-ui2/pull/766


Use Mustache for docs/ and test/?

It would be nice if it is possible to tweak templates for docs/make.jl, docs/src/index.md, and test/runtests.jl.
For example, I often make docs/make.jl an "sandboxed executable" which makes the Travis setup as trivial as script: docs/make.jl.  Also, I want to wrap code in test/runtests.jl as a module to make the test reproducible in REPL.  I'd like to use those settings from the get-go.  However, it is not easy to do this at the moment.

When #61 is done (soon I hope) you should be able to do this.


Fix bug where IOUPreview is not show for requests following a send money action

CC @tgolen, as you were assigned to the original issue
Details
Since the send money flow was introduced, a bug has existed where the IOUPreview will not show for outstanding requests. THis is because our 'shouldShowIOUPreview' condition was only looking for IOUActions, instead of a specific type of action (create).
Fixed Issues

$ https://github.com/Expensify/App/issues/15435
Tests

Verify that IOUPreview components are visible

Log in with two users: userA and userB
As user B, make three iou requests from userB
For both users, the full IOUPreview should be shown only for the most recent request
As any user, send money to the other
An IOUPreview should display for both the SendMoney, and most recent IOU request action

Offline tests

[After running the regular test, go offline
Send money again, using the PayPal option
An IOUPreview should display for both of the SendMoney actions, and also the most recent IOU request action

QA Steps

Run above tests
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@tgolen @eVoloshchak One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@tgolen @eVoloshchak One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

Wow, nice coincidental assignment.

@tgolen friendly bump for review. In case you missed it, Pullerbear assigned you after I CC'd you :)

Ready for review again @eVoloshchak, @tgolen

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Thanks both!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.2.81-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/roryabraham in version: 1.2.81-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Addon not working since last update of HA. Could you please fix this?

Version of the custom_component

Configuration

Add your logs here.

Describe the bug
A clear and concise description of what the bug is.
Not working since last update of HA
Debug log

Unable to install package ourgroceries==1.5.0: ERROR: Cannot install ourgroceries==1.5.0 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip available: 22.3 -> 22.3.1 [notice] To update, run: pip install --upgrade pip

Hello getting same issue as well.

Same.
`Logger: homeassistant.setup
Source: setup.py:202
First occurred: 7:08:50 PM (1 occurrences)
Last logged: 7:08:50 PM
Setup failed for custom integration ourgroceries: Requirements for ourgroceries not found: ['ourgroceries==1.5.0'].`

Was broken initially after 2022.12.0 and beta but then was fixed after ha-our-groceries 1.5.0.
Seems to be broken again since 2022.12.2 or 2022.12.3.

Still broken on 12.4 release.

The issue is caused by the pins here:
https://github.com/ljmerza/py-our-groceries/blob/32d6600fa602cd452dfd56a6836fe926f6646fa1/setup.py#L6-L7
    'aiohttp==3.8.3',
    'beautifulsoup4==4.11.1'

Packages/libraries should not pin strictly but instead, pin a minimal version or ping a range with a minimal and max version for subdependencies.
For example, is there a specific reason that library only works with aiohttp 3.8.3 and not with 3.8.1? Or why wouldn't it work with 3.7.x?
../Frenck

Is there a way to fix this issue?

Just found out about Ourgroceries & the HA integration today. I was ripping out what's left of my hair trying to get it working in HA.

Is there a way to fix this issue?

@rucanunes Yes, see my reply above.
If your question is: Can you fix it yourself? No, not without forking the library and custom integration and build your own.

Same issue. Our Groceries 1.5.0 doesn't work with HA 2022.12.5. Happened previously with v 1.4.1 but was very quickly fixed by @ljmerza. He is a real mvp :)
Unable to install package ourgroceries==1.5.0: ERROR: Cannot install ourgroceries==1.5.0 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip available: 22.3 -> 22.3.1 [notice] To update, run: pip install --upgrade pip
Thank you

that error message is supposed to tell you what package is conflicting. if it doesnt tell; me not sure how to fix it. there's only two packages and they both match the HA version

@ljmerza They don't and they should not match. They should provide a range, see also my explanation here:
https://github.com/ljmerza/ha-our-groceries/issues/31#issuecomment-1349066916
As a general rule of thumb: Don't ever pin exact versions in libraries (unless the library itself has a hard requirement of some sort, which is not the case for this library as hand).
../Frenck

okay should work again on v1.5.1


Support for Ruby<2.3 broken

Since the dig function used in /lib/jekyll/tidy.rb was introduced in Ruby 2.3, support for older Ruby versions is broken and produces the following error message
path/to/ruby/2.2.0/gems/jekyll-tidy-0.2.0/lib/jekyll/tidy.rb:35:in `ignore_env?': undefined method `dig' for #<Jekyll::Configuration:0x000000024e3750> (NoMethodError)

@abeMedia We're now testing for support back to Ruby 2.0.0

Awesome response time, thanks!


Add support for Math ML

https://developer.mozilla.org/en-US/docs/Web/MathML

Closing this as we've since added the Katex renderer and EquationNode.


Country information

Where will we display the country information and how will the dataset title look like?

country name is currently part of the dataset title. In this case Spain is country and Zaragoza is a title of the dataset. Will we have titles for all datasets? Because now I see collections like Moldova, Colombia and we also have a lot of Digiwhist collections that only differs in country but don't have special title.

We'll have bilingual application (EN + ES). How should work the Country starting with filter? In english version Spain should be selected by clicking S but in Spanish version it should be selected by clicking E for Espana?

@hrubyjan I think we should have titles for all datasets. For instance Spain Zaragoza and Spain Digiwhist.
Would it be possible to get a list of all the datasets? I can then draft what all the titles could be and maybe @jpmckinney can sign off.
On the bilingual issue, yes the user will need to select S for Spain when in English and E for Espana when in Spanish.

@hrubyjan you can find the titles as well in the document that I shared in #54

https://docs.google.com/spreadsheets/d/1SiGQ78mRaEmG7OKRSs-cHd2RV2Ra-UOLK_qLSkijUec/edit#gid=0 has a mapping between spider names (the "Spider" column to the far right) and the publisher names that OCP uses in its reports. We might change the publisher names for the registry (especially if they are very long), but it is a starting point. Let me know if you need access to the document.

Yes, please share it with me. thank you.

Done!


Add EXPIRE, TTL commands and EX keyword

Add the eviction commands EXPIRE, TTL, and the EX keyword to the SET command.
Examples
# Remove truck1 from fleet after 5.4 seconds.
EXPIRE fleet truck1 5.4
# Check how many second before truck1 is removed from fleet.
TTL fleet truck1
# Add truck1 to fleet and remove after 5.4 seconds.
SET fleet truck1 EX 5.4 POINT 33.5 -115.6

EXPIRE, PERSIST, TTL, and the EX keyword for the SET command have been pushed.


ShallowCopy for byte::Bytes

Seems like byte::Bytes can not be used as type, which is unfortunate, as it should be more or less among the most favored types as it is made for fast data transfer, continuous memory footprint and such.
copy in Bytes is shallow by design, so the following compiler error might be avoidable...
error[E0277]: the trait bound bytes::Bytes: evmap::ShallowCopy is not satisfied
--> src/foo.rs:143:5
|
143 |     w:WriteHandle<[u8; 32], Bytes, (), BuildNoHashHasher>,
|     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait evmap::ShallowCopy is not implemented for bytes::Bytes
|
= note: required by evmap::WriteHandle
error: aborting due to previous error

Hmm.. I think it should be okay do have:
impl ShallowCopy for Bytes {
    unsafe fn shallow_copy(&mut self) -> Self {
        Bytes::from_static(self.as_ref())
    }
}

Along those lines, I also think it would be good to add
impl<'a, T> ShallowCopy for &'a [T] {
    unsafe fn shallow_copy(&mut self) -> Self {
        std::slice::from_raw_parts(self.as_ptr(), self.len())
    }
}

I'd be happy to take a look at a PR that adds these :)

Sorry 'PR' ?

You want to look into a program that uses those trait implementations? I will try them eventually. What minimal code do you need and why you want to see it?

I'm not sure I understand what you mean? Those trait implementations would need to be added to evmap itself for you to be able to rely on them in an external crate.

A really? Ok I was under the impression, that the user software implement those traits for Bytes in their code.

Ah, no, we're limited here by Rust's orphan rule:

One restriction to note with trait implementations is that we can implement a
trait on a type only if either the trait or the type is local to our crate.
For example, we can implement standard library traits like Display on a
custom type like Tweet as part of our aggregator crate functionality,
because the type Tweet is local to our aggregator crate. We can also
implement Summary on Vec<T> in our aggregator crate, because the trait
Summary is local to our aggregator crate.
But we can’t implement external traits on external types. For example, we
can’t implement the Display trait on Vec<T> within our aggregator crate,
because Display and Vec<T> are defined in the standard library and aren’t
local to our aggregator crate. This restriction is part of a property of
programs called coherence, and more specifically the orphan rule, so
named because the parent type is not present. This rule ensures that other
people’s code can’t break your code and vice versa. Without the rule, two
crates could implement the same trait for the same type, and Rust wouldn’t
know which implementation to use.

In other words, since evmap defines the ShallowCopy trait, either the byte crate would have to implement ShallowCopy for Bytes (which seems unlikely) or that implementation has to exist inside of evmap. Since byte::Bytes is so common, I think I'd be okay with including that impl directly here :)

I see.

Or you can submit a pull request yourself that makes the necessary changes (adding a dependency on byte, the impls above, and tests with the corresponding types) which I could then review and merge :)

Fixed in #30!


Need ChangedOn date in RetrieveAttributeChangeHistory

I was looking to retrieve one attribute changes datewise, I thought RetrieveAttributeChangeHistory will do the job, but it is just returning old value and new value, not the date. I'm making below shown GET request, Do I need to pass any extra parameter to get changed date or this will not retrieve it at all.
Do we have any other Function/Action for my scenario?
https://instance.crm.dynamics.com/api/data/v9.1/RetrieveAttributeChangeHistory(Target=@p1,AttributeLogicalName=@p2)?@p1={'@odata.id':'incidents(01D707F5-MY-GUID-a000D3AE0E155)'}&@p2='statuscode'

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

ID: db6d9d15-455a-2793-80f1-19955810a901
Version Independent ID: 2dd868ce-f6b5-a656-5091-ca1049ca0878
Content: Audit Entity Reference (Dynamics 365 for Customer Engagement)
Content Source: ce/developer/entities/audit.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Any updates on this?

While you are adding the ChangedOn date, please consider also adding the userid of the person who made the change.
While using the D365 API, I often have to determine "who was the user who last updated this field".  Right now that is a very byzantine process that expends a lot of API calls to get the answer.
If you added these two fields, a single API call would give me my answer.
Thanks.
Bryan Hunt

Oh, this is still open. 4 years and counting.


Question: Can Unstructured track hierarchical header info, such as H1, H2, H3, etc.?

In HTML and Markdown, we have a concept of nested and hierarchical header text, and I think it could be helpful to track for TitleText.
Does Unstructured currently have any tracking for?:

Identifying the hierarchical level of a title during parsing.
Storing this hierarchy level (H1, H2, etc.) in the metadata.
Exposing the hierarchy of relevant headers on sub-elements, such as on metadata of a NarrativeText element?

Totally understandable if this is out of scope or perhaps not relevant to mainstream use cases. For my part, I just wanted to better understand if this is something that Unstructured can do, or if not if there are existing plans to add something like this in the future.
Thanks!

Hi @aaronsteers ! There is currently an effort to discover and expose hierarchy metadata for element types.
Currently though, we do store tag info in the tag and ancestortags properties of instances of the HTMLText class, which are the element types that come back from partitioning an HTML file. This also applies to certain other file types that get converted to HTML for processing, including markdown.
However, I don't believe this information is retained in the metadata when the element is converted to JSON, so if you're using the API this may not be helpful. This is something we could add, though.

There are plans to support distinguishing between H1, H2, etc. This will be accomplished by the category_depth metadata field being introduced in https://github.com/Unstructured-IO/unstructured/pull/1308 .
E.g., H3 would be a Title element that has metadata.category_depth=3.

This feature was added in #1268! Let us know if you run into any issues


Code clean

Pull Request Test Coverage Report for Build 128

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 126:
0.0%

Covered Lines:
88

Relevant Lines:
88

💛  - Coveralls


Add reCAPTCHA v3 support

Purpose
Add reCAPTCHA v3 support
Related Issues

https://github.com/wso2/product-is/issues/11576

Related PRs

https://github.com/wso2/identity-apps/pull/3224
https://github.com/wso2-extensions/identity-governance/pull/608

Remove the threshold value from the deployment.toml file, since this is required only for v3


How to decrypt the python code

I was thinking a creating an android app from the python code but it seems to be encrypted. How do I decrypt it?

SourceCode_ESET-KeyGen-v<IP_ADDRESS>.zip

I will need to change the method of code packing because I can't automate code unpacking because of byte problems.

Use https://github.com/rzc0d3r/PyCompressor

Thanks!

I removed the encryption and returned the normal source code. I was also able to trick eset antivirus. Now the source code is no longer removed by antivirus when scanning and running!!!

Lmao


Error console when importing any

Description the bug
The plugin is not working. not with the commands, not with the icon in sidebar.
getting this error in console output:
Error importing Apple Books highlights: SyntaxError: Unexpected end of JSON input
eval @ plugin:apple-books-import-highlights:6154
Promise.catch (async)
eval @ plugin:apple-books-import-highlights:6152

Also include the following details:

Error behavior: intermittent/permanent.
Error occurs when:

[x] Importing a single book
[x] Importing all books
[x] All the above

Number of books and highlights (approximately)

General information

Plugin version: 1.2.1
Obsidian version: 1.5.12
MacOS version: Sonoma 14.4.1

To Reproduce

option+command --> Apple Books import command
with icon in sidebar

Expected behavior
importing some highlights
Screenshots
no screeny - error text
Additional context
getting this error in console:

Getting the same issue. Same error, same line number. This seems to be happening because the query in getAnnotations()
SELECT ZANNOTATIONASSETID, ZFUTUREPROOFING5, ZANNOTATIONREPRESENTATIVETEXT, ZANNOTATIONSELECTEDTEXT, ZANNOTATIONNOTE, ZANNOTATIONCREATIONDATE, ZANNOTATIONMODIFICATIONDATE, ZANNOTATIONSTYLE 		FROM ZAEANNOTATION 		WHERE ZANNOTATIONSELECTEDTEXT IS NOT NULL 		AND ZANNOTATIONDELETED IS 0;
is returning 0 results. and causes the JSON.parse(stdout) to fail.
I confirmed I had no highlighted "Books", only "PDFs." Pulled out a random Book and added a highlight and the plugin worked as expected. I converted one of my PDFs to EPUB, added a highlight and it showed up in Obsidian when I reran the sync as well.
Little sad to see PDFs don't seem supported. Poking at the sqlite databases in question it seems like PDF annotations may be stored differently or elsewhere.

@dschense Thanks for the report
@JeminiRiver Thanks for the report and for the explanation of the error.
@dschense The error you're receiving is caused (as @JeminiRiver mentioned in her message) by an empty response after querying the database, which means that you probably have no books in your Apple Books application. Could you please confirm that?

it seems like PDF annotations may be stored differently or elsewhere.

@JeminiRiver Yes, you're right. However, it's a bit tricky. Apple Books is a convenient place to store PDFs but the tricky thing is that even they're getting added to the library database, all the highlights are not stored in the database and seem to be added to the document itself. You can check the highlights using Preview, for example, via Highlights and Notes pane (Command + Option + 4). Converting PDF to ePub helps to workaround this, for sure. So, I believe it's more like lack of native support for highlights rather than the plugin's missing feature. But I will check the possibility of adding it anyway.

sorry for the late response:
there are a lot books in my library. They are all epubs or downloaded direktly from the apple store.
for me there is still the same error.
Dont know what to do, to fix this.

@dschense Thanks for the information. I will definitely go deeper into the issue because now, I was able to reproduce your issue on an empty database only. I'm also planning to do a partial refactoring of the fetching logic and write tests, so it should also help to avoid that kind of issue in the future. Sorry for the inconvenience.

Thank you for your commitment @bandantonio
I have carried out a few more tests to definitely rule out an error on my part.
I'm pretty sure that it can't be my library.
installed this Plugin for a test:
https://github.com/atfzl/obsidian-apple-books-plugin
With this plugin a sync to Obsidian works without any problems.
Maybe this information helps to find the bug.

@dschense Thank you too for your help, that will definitely come in handy.

Reopened as the issue still exists.


CLI binaries (CLI plugins) should be pushed as OCI images

To support tanzu CLI's new OCI source, we must publish our CLI binaries as OCI assets.
The destination for each CLI plugin should be:
projects.registry.vmware.com/tce/cli/plugins/tanzu-${PLUGIN_NAME}-${OS-}${ARCH}:${PLUGIN_TAG}

We should be able to push periodically, unsigned images, for example, something equivalent to:
imgpkg push \
    -f artifacts/linux/amd64/cli/unmanaged-cluster/v0.10.0-dev.5/tanzu-unmanaged-cluster-linux_amd64 \
    -i projects.registry.vmware.com/tce/cli/plugins/tanzu-unmanaged-cluster-linux-amd64:v0.10.0-dev.5

For a release, the same process should be follow, but a signed binary should be pushed.

@karuppiah7890 is added here. @dvonthenen owns the initial work. Release engineering should then handle the triggering of it (via automation) and maintencne of it over time.
@karuppiah7890 feel free to assign someone from your team if you're not the right human.

This won't be relevant for v0.12.0 because we cannot afford to change this model until potentially v0.13.0.


Add Upgrade Guide notes on Deprecations/Removals

Related to #17151 - need to add Upgrade Guide notes for all the deprecations and removals.
However, merging all these is annoying enough as is, so I'll wait until they're all (or at least most of them are) merged first.  This ticket is to ensure I don't forget to do this.

Also Amanda asked for a quick presentation on this.


Change log level dynamically

This will allow to change log level on per request/user basis for OW transactions.
If a specific header is provided in the http request to the controller, it will log messages on the debug level for that given transaction id.
@mhenke1
regards, Vadim and Martin.

Looks useful!. Logback provides support for [TurboFilters] which can be used to enable log for specific thread of execution. We used this in Apache Sling to enable log for specific logger for specific thread via HTTP Header and make those logs available in Chrome plugin. Something like below
curl -s -D - -H "X-OW-EXTRA-LOGGING : true"
-H "X-OW-EXTRA-LOGGING-CONFIG : org.apache.kafka.clients;level=trace" \
 http://localhost:8000/api/v1/namespaces/_/actions/hello?overwrite=true -o /dev/null

This would allow us to say see what all outbound calls are made in specific request. Going further probably the log directive can be added to activation message and picked up by invoker those allowing tracing logs for specific category on invoker side to debug any potential issue
All this is just a possible suggestion/idea for future work in this area! For now we can keep it simple and start with current approach in this PR

@chetanmeh Thank you for your feedback. Learning from proven solutions like Apache Sling is always a valuable asset to avoid making the same things wrong :-)
Indeed we also looked into using Turbofilter, but had to learn that in our case they do not work well.
The reason is that we use Akka Logging. Log message are passed through the Akka Event Bus (we win asynchronicity on this way) before passed to Logback. When running on INFO level all DEBUG level messaged are not put into the event bus and therefore do not reach Logback and it's filters. The only way to exploit Logback filtering would have been to put all messages into the event bus and use Logback  to filter out the one with the appropriate level. But in this case we would would again see increased memory pressure due to bigger internal buffers needed by the event bus. So we decided for this simpler solution.


bug: optimism gas calc.

within the gas tracking branch, we have implemented a transaction entity that interacts with the optimism gas oracle to return all of the necessary data fields to calculate the gas cost of the transaction. currently, the l1_gas_used by the transaction is underestimated by ~1% on average. from my best guess, this is a problem with how we are passing in the transaction data to the contract function.
the oracle contract function being called can be found here.
an example of interacting with the oracle contract can be found in the optimism sdk here
the current convention used for passing transaction data to the function can be found here
without the serialize function of the ethers package, we are restricted to using the ethereum class in graph-ts

i really liked the idea of a big subgraph that collected and formatted every piece of data you could want and was written in a very modular way (writing each entity in its own file (?) and other good stuff) that could be forked and used as boilerplate for any type of rubicon-related subgraph you could encounter (im talking about you future messari orderbook template). but, this goes against our event-driven lifestyle at the moment.
i believe we will meet again, #2


⚠ Processors has degraded performance

In fd6c1ae, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 46 ms

Resolved: Processors performance has improved in 3f9008a.


Note Migration Failed Stuck Spinning [BUG]

Hi,
I upgraded from Version v1.3.75 to Version 2.0.30 I received the V1 to V2 popup window and the first two checks were ticked green.
The third migrating notes to journal entry process was stuck with a spinning wheel for hours, eventually I relaunched the server, the popup didn't open again and nothing has migrated over and I'm missing a years worth of calendar notes.
Can you please help or advise me how I can import the notes?

Also it opens the calendar now every time I open the game.

Sorry to hear you had a bug with the migration.
It sounds like it had an issue processing your notes and covering them to journal entries. No worries all of your note data is still there.
To start you can re run the migration by running this code in the browser console.
Hit F12 and this should open up the developer console. There should be a tab (or section) labeled console. In there paste this text and hit enter SimpleCalendar.api.runMigration()
This will cause the migration dialog to open again and run the migration. If the migration gets stuck an error should be logged to the console. If you can share that error with me it can help figure out what went wrong.
Alternatively if you wish you can share your  settings.db for that world with me and I can see what's going on. This article, https://foundryvtt.com/article/user-data/ , shows where the world data is stored. Under the world folder there will be a folder for the world that is having the issue and in that folder will be the settings.db
As for Simple Calendar opening every time you load that is a new setting in version 2. You can turn it off in simple calendars configuration or under the module settings. The setting is called "open on load". More details and other settings can be seen here
https://simplecalendar.info/pages/docs/global-configuration/settings.html#client-settings

That worked thanks. Had intermittent internet drops and was monitoring the console if I had a connection drop and then reestablish connection the process wouldn't kick off again. So I had one of my players with more stable internet do it for me, once he did it it finished and got it sorted. Thanks for the support and thanks for this awesome module you're a great dev.

Glad that worked for you!

I'm stuck with this error myself, but haven't been able to resolve by manually re-running the migration.
There are a number of errors:

"Cannot read properties of undefined (reading '0')
"Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '0')
"Error: You are accessing JournalEntry#data object which is no longer used. Since V10 the Document class and its contained DataModel are merged into a combined data structure. You should now reference keys which were previously contained with the data object directly.

I've attempted to restore from backup (which works great on the old version), and re-attempted the upgrade from v1 to v2 again. No dice. I am wondering if this upgrade will only work on foundry 10, and I've already moved onto foundry 11. I wasn't aware this was out of date until a recent compatibility notification, because I was stuck on an older version, but it was reporting it was up to date.
I'm betting that this is mostly a me problem since my version issues are not the typical upgrade path.
Short of manually re-typing the old notes/dates, then manually creating them in the new version, I'm wondering if there's anything else I can do. I'm wondering if there is a way to export the old note content into a txt or json, or any somewhat readable format to make the transition easier.

@BlackfishStudios sorry you are having migration issues. The good news is that all the data is still there in foundry 11 we just need to get it
The migration should work, but there may be a bug in in foundry version 11. What would help me, if you are comfortable with doing so, is send me your world backup so I can try on my computer and see what's going on in more detail. If not (which is ok) it will just take me a bit to reproduce the issue and try to fix it.
If you are ok with sending me the world backup you should be able to attach it in a GitHub comment or send it to me on discord (my name on there is vigorator).
If you don't mind making a new issue for this as well that would be great.
We will get this sorted out!


Handle max_token more gracefully

A bug was found in the way max_token is managed. Since the number of tokens generated is updated every time FSMState is called, when we generate tokens in batch the sequence generator returns max_tokens / batch_size tokens. The solution implemented in #391 consists in adding an idx argument to the next_state method of the FSMs, but this does not respect the a priori interface between the generator and FSMs.
There are two solutions to this problem:

Create a FSM for each sequence in the batch.
Handle the constraint in SequenceGenerator.

Either of these allows us to remove the idx argument. #417 suggests that the stop_at constraint should be implemented at the SequenceGenerator level (since we need to decode the token ids), and the fact that we should let users control the maximum number of tokens generated manually makes me lean towards (2).

Makes sense. Let's merge #391 first.

An alternate solution that I implemented in the vLLM integration example was to initialize every sequence-dependent variable as a default dict and pass the sequence id. The solution of copying the FSM is much more elegant, and there's some work planned with the vLLM team to generate FSMs dynamically for each new sequence indeed.


TokenId invalid error when using serviceworker, keycloak in an nx react app

Issue and Steps to Reproduce
I am integrating react-oidc lib with an nx monorepo react app. Because of the complex folder structure it's not able to locate the public folder directly. Hence I copied the OidcServiceWorker.js and OidcTrustedDomains.js from  example repo and it looks to be setup correctly. The service worker is able to fetch the token with keycloak but the issue is with the nounce mismatch which gives token id invalid error. The isTokenOidcValid method recieves separate nonce - formattedTokens has NONCE_SECURED_BY_SERVICE_WORKER while nonceData.once has nonce from sessionStorage value.
Steps to reproduce:-

Setup an integrated nx react app
Setup keycloak using docker
Integrate lib with

   configuration:export const configuration = {
    client_id: 'test',
    redirect_uri: 'http://localhost:4200/*',
    silent_redirect_uri: 'http://localhost:4200/home',
    scope: 'openid profile email api offline_access',
    authority: 'http://localhost:8080/realms/test-app'
    service_worker_relative_url: '/OidcServiceWorker.js',
    service_worker_only: false,
};

Versions
react-oidc: 6.24.1
Expected
To have NONCE_SECURED_BY_SERVICE_WORKER instead of undefined 'default' in sendMessageAsync in login.ts on a getNonce event. This flows down and in the verification fails as it defaults to sessionStore nounce value
Am I missing anything in serviceWorker configuration?

Hi @Sneham14 , thank you very much for your issue. Did you upgrade react oodc version recently ?
If yes ServiceWorker.js is not up to date that may do this error occur.

Thank you for the swift response on a Sunday. Yes, this is my first time working with react-oidc so using the latest. Should I use another version or update my OidcServiceWorker code somewhere? To note, even if the login fails next time when I reload the ServiceWorker calls seems to be working well. I can also see userinfo using service worker in network tab. The issue occurs only on first login where the workflow is:-
Authenticating msg -> Keycloak IDP login -> Authentication success screen msg -> Authentication failed (nonce issue) with the payload in the url.

Hi @Sneham14 , thank you for your answer.
Does it work if you disable service worker mode (comment serviceworker file from configuration and unistall service worker from dev tool)?
Which browser are you using?
I will publish tomorrow a version with more detailled logs about nonce check.

Yes it works without serviceworker using session storage. I tried on Google Chrome and Microsoft Edge. Thanks

hi @Sneham14 , i need to wait for a big PullRequest to be done before adding more logs.
Can you wait a litle bit before doing the tests with more log ?

Hi @Sneham14 ,
version 7.2.2 add a lot of more log detail to try to understand your problem.

Hi @Sneham14 , do you have any news about the logs?

Hi @Sneham14 , do you have any news?


Heroic Strike Rank 11 is unavailable to players

Rank 10 is the max rank learnable by players, rank 11 was scrapped in TBC beta (original) and does not become available until wotlk

Great find, thanks!


Coin control

It would be great if there were a way to select specific utxos when spending, bypassing whatever existing coin selection algorithm exists.
Brainstorming some ideas for how this could work

Pass list of addresses/utxos via a flag --spend-from - perhaps poor UX but also extensible for gun as a cli
Like above, but instead pass address indexes with a flag - would probably want address list to include an index column so these are accessible. We would also have to make some distinction on internal/external..
Interactive menu where you can toggle UTXOs - would be nice, but also not very CLI-like (could be built on top of --spend-from)

Other ideas?

I think we have to do the first bullet first, see #86


platform.getOffsetParent is not entirely decoupled and is still called when overridden

Here is a toy example that highlights the issue: https://codesandbox.io/s/floating-ui-dom-template-forked-qkf5uz?file=/src/index.js
https://github.com/floating-ui/floating-ui/blob/777c9377c5acfc1f252b58807eb076c30bad0ceb/packages/dom/src/platform.ts#L21
In the above line, you can see that getElementRects references the original getOffsetParent implementation.

Yeah, you'd need to also replace getElementRects with something custom :\

I guess something like this can work:
// Your custom function
const getOffsetParent = () => x;

computePosition(a, b, {
  platform: {
    ...platform,
    getOffsetParent,
    // Inject your custom one into the existing platform method
    getElementRects: (args) => platform.getElementRects({
      ...args,
      getOffsetParent,
    }),
  },
});

  getElementRects: async ({
    reference,
    floating,
    strategy,
    getOffsetParent: getCustomOffsetParent,
  }) => ({
    reference: getRectRelativeToOffsetParent(
      reference,
      getCustomOffsetParent
        ? await getCustomOffsetParent(floating)
        : getOffsetParent(floating),
      strategy
    ),
    floating: {x: 0, y: 0, ...getDimensions(floating)},
  }),


Show published bot endpoint after publishing is completed or better yet, provide a way to test the published bot from inside Composer

After publishing from Composer, user has to go to Azure portal to test the recently published bot.  It'll be great if the newly published bot endpoind is displayed in in the publish bot UI.

Describe the solution you'd like
Show the published bot endpoint,  better yet, provide a way to test the published bot from inside Composer
Describe alternatives you've considered

Additional context

+1 on having a way to easily open your published bot in the webchat panel or emulator.

Shiproom - moving our of R13

@GeoffCoxMSFT when publishing to PVA we render a "Test in Power Virtual Agents" link. I do not know if we have the data & capability to similarly construct a deep-link to the Azure Portal Web Chat blade. Assigning to you for now to push this forward and understand what gaps we may have here.

@Dewain27 Let's chat about this in the next connection meeting.  I'm not sure what UI needs to happen here


Tailwind Dark Mode doesn't work side-by-side

Describe the bug
Tailwind Dark Mode doesn't show in Preview. It only shows when Open a New Window is clicked
Link to the blitz that caused the error
https://stackblitz.com/edit/github-6frqvs-qvxcvt
Steps to reproduce
Just open the Stackblitz & see. Click on the toggle which u probably can't see so highlight the text.
Expected behavior
Dark Mode should be visible
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 92.0.4515.107
Major version = 92
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.107 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 78894922,
  "usedJSHeapSize": 72427146,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 548843700a830a9170c7b4f676310f668ec52ffc

Additional context
No response

Thanks for the report @deadcoder0904! Seems like there's something wonky going on with it being in an iframe where the body background isn't changing color properly 🤔  Will investigate with the team!

Thought like I had replied here before, but apparently not!
The issue is that browsers:

Keep the iframe's background transparent;
May change the iframe document's text color to white or black or something else depending on

the OS color scheme,
the browser's theme,
the iframe document's <meta name="color-scheme" value="dark|light">,
or the iframe document's :root { color-scheme: dark|light } CSS property;

DO NOT set the iframe document's background-color when setting its text color.

On our side, our editor UI uses a dark theme by default. Most users still have a light mode OS setting, so by default text in the preview iframe will be black. To avoid showing black text on top of the dark background of our editor UI, which would be unreadable, we set a white background behind the iframe. This works well for the majority use case.
In the minority use case where users use a dark mode OS, or a dark theme for their browser (and the browser uses that as a signal for the default text color of documents), then the browser may set the iframe's text to white or some other light color, and then you'll end up with white text in a transparent iframe on top of a white background, which obviously is not great.
The problem doesn't happen when opening the preview in a new tab, because the browser will set the tab's background color (not the document's background color) to something dark, so you have light text on a dark background and that's readable. It's a problem that mostly affects iframes.
We might want to try to work around this by using prefers-color-scheme in the parent document, but I don't think we have a guarantee that matching (prefers-color-scheme: dark) in the parent document means that it would match in the iframe document, and it doesn't guarantee that the iframe document doesn't explicitly sets :root { color-scheme: light } and ends up with dark text, so using this signal to replace our white backdrop with a dark one could break as many projects as it fixes. Sounds like a game of whack-a-mole.
IMO the solution is for authors of web pages to make sure that they do not rely on browser defaults that may fail to provide correct contrast, and instead explicitly set text and background colors for their pages, e.g. for a light theme:
:root {
  color-scheme: light;
  color: black /* or some dark color */;
  background-color: white /* or some light color */;
}

and for a dark theme:
:root {
  color-scheme: dark;
  color: white /* or some light color */;
  background-color: white /* or some dark color */;
}

After revisiting this issue, we identified specific combinations of parameters where showing a transparent iframe on top of a white backdrop would result in unreadable text:
https://docs.google.com/spreadsheets/d/1R9knWZN7KB-y_3JAov4LDYWfHIzenGCtMJrQikttTBs/edit
For now we've decided to fix this issue by removing the color-scheme declaration in the parent document (https://stackblitz.com/edit/), which gives better results in Google Chrome. In our tests, there are differences in how browsers treat color-scheme, so for best results we still recommend that pages should:

Declare a correct color-scheme value on the root element.
And also declare a matching color and background-color pair of colors on the root element.


Create SECURITY.md

Closes #527
As per the linked issue, this PR adds a security policy to the repository.
The policy currently requests that vulnerabilities be reported to GitHub's private vulnerability reporting feature.
This feature should be enabled in order for it to start working:

Open the repo's settings
Click on Code security & analysis
Click "Enable" for "Private vulnerability reporting (Beta)"

Let me know if there are any changes you'd like to make to the security policy to be more precise.

Private vulnerability reporting is enabled, as instructed.


Buildmastering/update dependencies

Description
Update dependencies and move act method to React.

Processing this pull request

Pull request rejected by merge tool. No change entry file found

Processing this pull request

Pull request rejected by merge tool. No change entry file found

Processing this pull request


add logic to only show NECCC on production sent

What's wrong

we don't want to launch the new regions to production until the councils have had a chance to review and approve

What done looks like

add an env variable to .env called REACT_APP_ENVIRONMENT and set it to development or production locally based on what you want to test. if REACT_APP_ENVIRONMENT === 'production'  then only show NECCC. if REACT_APP_ENVIRONMENT === 'development' then show all councils. i will update these env variables in the pipelines

Where the changes need to be made

.env, multiple places

updated so only NECCC is avaliable on branch thumbnail_images.  filter and sorting updates in review on same PR


tag.AddAttachedPicture() not working

The album art doesn't seem to get added in the music file. Here path is a valid mp3 file and metadata.Image is a valid jpeg image byte array.
	tag, err := id3v2.Open(path, id3v2.Options{Parse: true})
	checkErr(err)
	defer tag.Close()

	pic := id3v2.PictureFrame{
		Encoding:    id3v2.EncodingUTF8,
		MimeType:    "image/jpeg",
		PictureType: id3v2.PTFrontCover,
		Description: "Front cover",
		Picture:     metadata.Image,
	}
	tag.AddAttachedPicture(pic)
	if err = tag.Save(); err != nil {
		log.Fatal("Error : ", err)
	}

Found the problem. Turns out there's a problem with MacOS High Sierra, and it sometimes fails to show album covers in Finder.
Sorry for raising this issue. Its not related to your program.

@kalbhor Sometimes I encounter this issue too. It's better to check ID3 tags in iTunes.


DOC: Fix PrintExceptionMessageHandler example

@hosmanadam Thanks for the PR!
The index.html file is generated from the index.adoc file. We should fix the index.adoc file too or your changes will be overwritten the next time index.html is generated.
To test, run:
gradlew ascii

Then check the generated HTML  in build/docs/html5/index.html
Thanks again for helping to make picocli a better library!

@remkop Thanks for the heads up, I made that change as well.
Least I can do, thanks for an awesome tool :)

Merged and updated release notes.
Thank you again for the contribution!
Can I ask:

what is it that you like about picocli?
what would you do if picocli did not exist?

@remkop Sure. I like that it works well and is conveniently annotation-configured, which is a familiar workflow when you come from Spring. If it didn't exist, I'd build our CLI in Click instead, but then no one else on the team could touch it, because they're Java guys.

Thank you!
Enjoy picocli! 😉


CFException does not seem to be auto-loaded by composer.

Using composer 1.0-dev (40c14709f79f9d7ea35ac969cfbd7f41beb525bb), requiring this project with composer and loading the composer autoloader, an application will fail when it reaches a CFException for any reason:
Fatal error: Class 'okw\CF\Exception\CFException' not found in ./vendor/okwinza/cloudflare-api/okw/CF/CF.php on line 167
If then I add:
require "vendor/okwinza/cloudflare-api/okw/CF/CFException.php";
to the top of my PHP file, above:
require_once 'vendor/autoload.php';
it works as expected.

I'm having the same issue. I am using this with Symfony 2.8 and looks like the CFException is not being loaded.

Hi!
I think this is due to an old version being the last stable release in Packagist.

At the 1.0.2 tag, there is no CF/Exception folder (see this tree)
At the last commit on master, there is a CF/Exception folder (see this tree)

I believe the missing folder is the reason why Composer fails to autoload the CFException class, since this is assumed to be located in that folder.
I guess the best way to solve it would be for @okwinza to create tag a new release, but in the meantime I think one could solve it by changing the composer requirement for the package to something along the lines:
"okwinza/cloudflare-api": "dev-master"

Or perhaps this if you want to make sure that it's the currently latest commit to the master branch that is being installed:
"okwinza/cloudflare-api": "dev-master#c3b3ace495c3ab7fd335ec56607aa6fa72868d4a"

Much thanks @ManneW for digging into this issue and sorry it took so long.
Sadly, i've barely had any time for open source stuff lately but it is going to change very soon. I'm looking at the code right now and there are many things i'd change(abstract away from curl for example), so expect major version bump in the near future.
As for now, i pushed new tag and updated the package on packagist just in case.
Please, check back if the issue is still relevant.
Cheers!