feat: support multiple resource definition of the same type

Related issue: https://github.com/seal-io/walrus/issues/1784

Rebased


Cross Positioning

Test:

[ ] Ubuntu
[ ] Manjaro KDE
[ ] Mint XFCE
[ ] Mint Cinnamon
[ ] Mint Mate

/CC @ekonstantinidis
Theoretically, there can be problems with Linux, If are several screens with different resolution screens.

@mrmlnc Looks good to me! If it works with the default settings of OSX, Windows & Ubuntu there's nothing to works! So I guess we can merge it :sparkles: :sparkles: :sparkles:


[crashtracker] Enable collecting to a UNIX socket

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
Attention: Patch coverage is 0% with 173 lines in your changes are missing coverage. Please review.

Project coverage is 68.29%. Comparing base (de8d3b6) to head (08ca16f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #420      +/-   ##
==========================================
- Coverage   68.56%   68.29%   -0.28%
==========================================
  Files         193      193
  Lines       25086    25188     +102
==========================================
  Hits        17201    17201
- Misses       7885     7987     +102

Components
Coverage Δ

crashtracker
17.73% <0.00%> (-1.62%)
:arrow_down:

datadog-alloc
98.76% <ø> (ø)

data-pipeline
51.45% <ø> (ø)

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
85.24% <ø> (ø)

ddcommon-ffi
74.93% <ø> (ø)

ddtelemetry
56.09% <ø> (ø)

ipc
81.69% <ø> (ø)

profiling
77.84% <0.00%> (-0.15%)
:arrow_down:

profiling-ffi
59.66% <0.00%> (-0.39%)
:arrow_down:

serverless
0.00% <ø> (ø)

sidecar
37.89% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
54.98% <ø> (ø)

trace-mini-agent
69.12% <ø> (ø)

trace-normalization
97.79% <ø> (ø)

trace-obfuscation
95.74% <ø> (ø)

trace-protobuf
30.76% <ø> (ø)

trace-utils
90.00% <ø> (ø)


Center loading icon that appears at first open

Right now, it's a bit low on the screen
General idea: https://i.redditmedia.com/3ezLhRjypSa_55X-IOFSVCGM129HtGdMJ33N1UWrb-E.jpg?s=041d3c0bc296511191363d4c11f65343

It was off center for a reason at some point, but I'll make it centered again


Small Hungarian update

And some of the translations in the catalog needs correction:

"Séta hó" -> "Hó"
"Óceán hullámai" -> "Óceán"
"Könyv oldal" -> "Könyv lapozás"
"Repülőgép kabin" -> "Repülőgép"
"Vákuum" -> "Porszívó"
"Karácsonyi hang" -> "Karácsony"

Thanks as always. And I've updated the catalogue sound names as well! Restart the app, and the fixed translations should start to work.

Thanks as always. And I've updated the catalogue sound names as well! Restart the app, and the fixed translations should start to work.


Rename Visual Studio Online -> Visual Studio Team Services

Visual Studio Online has been renamed to Visual Studio Team Services.

Hi @changeworld, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

This is a dupe of the other one.  The approach is each team is changing their owned tasks as they update them.  Not batch changing.


Added Christmas theme along with textures

As suggested I have added a Christmas theme having a texture background with the given color palette. #735

@shaminmeerankutty Thank you for your contribution!
I'll check your code tomorrow.


WIP: Add basic tests

This PR based off https://github.com/buildkite/lifecycled/pull/45, hence it contains all the commits from that one. Started off writing some basic tests that at least cover cases like cleaning up queues after the listener exits etc. When testing we need to use logrus.NewNullLogger(), so I had to rewrite some code to avoid using the default/global logger in logrus - ended up with the following pattern:

Daemon includes a logrus.Logger.
Listener's accept a logrus.Entry which is used for log entries. This allows us to set fields and have them included in all log entries (e.g. the instance id and/or listener type).

Example of the log entries below:
INFO[0000] Starting listener                             instanceId=test listener=autoscaling
INFO[0000] Waiting for termination notices               instanceId=test
DEBU[0000] Creating sqs queue                            instanceId=test listener=autoscaling queue=lifecycled-test
DEBU[0000] Subscribing queue to sns topic                instanceId=test listener=autoscaling topic="arn:aws:sns:eu-west-1:#:lifecycled-example-lifecycle"
DEBU[0001] Polling sqs for messages                      instanceId=test listener=autoscaling queueURL="https://sqs.eu-west-1.amazonaws.com/#/lifecycled-test"
DEBU[0021] Polling sqs for messages                      instanceId=test listener=autoscaling queueURL="https://sqs.eu-west-1.amazonaws.com/#/lifecycled-test"
INFO[0025] Received signal: shutting down...             signal=interrupt
DEBU[0025] Deleting sns subscription                     arn="arn:aws:sns:eu-west-1:#:lifecycled-example-lifecycle:35ff34d7-edc8-4413-9065-066bcce3e3c4" instanceId=test listener=autoscaling
DEBU[0025] Deleting sqs queue                            instanceId=test listener=autoscaling queue=lifecycled-test
INFO[0026] Stopped listener                              instanceId=test listener=autoscaling

Stuff that needs fixing:

All the references to itsdalmo/lifecycled (after this PR and #45 is merged, I'll stop using my fork to make PR's)
time.NewTicker only triggers after the first duration has passed, which means that a few of spot listener tests take 5 seconds to complete.
I'm wondering if perhaps the create/delete queue log entries should be info instead of debug since it only happens once in the lifetime of the program and gives you greater confidence that it has started correctly etc.

Closing in favor of #52


Why are you sending spam emails?

While this seems like legitimate project with good intentions, you violate various laws and the GitHub Terms of Usage by scraping GitHub emails and sending spam advertisement emails for this project.
Please stop sending spam emails and never do this again.

Thank you for bringing this to our attention. While we did receive cooperations from tech podcasts and influencers to help enlarge the impact of this project, we never authorize to send the spam emails. Please block the email address sending this and forward the email to me so that we can did further investigations. Thank you!


 At first, it was ok, but after a while, we couldn't get access to them via the RPS and soon after we were unable to communicate with them at all. Some of them reverted back to a DHCP mode and that blocks us completely from sending IPMI commands from other services like MAAS.
I recently discovered that using the docker image, we can force a connection to the /dev/mei0 port that would allow us to re-establish a connection to the RPS server, if they crash for whatever reason. The only catch being we need to do this in the ME setup timeframe (usually 2 mins from reboot)
Is there a way that we can re-enable the 16992 or 16993 ports using rpc-go remotely (I have limited physical access to the hosts and even if we could have access to them, they are utilized 99% of the time)?
With that in mind, is there a better way to reconnect hosts that have lost connection to the RPS? Manually connecting to a host and hoping that they have /dev/mei0 running could also prove difficult.

While migrating to the toolkit, did you create a ACM or CCM Profile with CIRA (see steps 11 and 12 in docs)? If yes, that means AMT is configured to connect to MPS using CIRA and ports 16992/16993 won't be accessible remotely.
There are 2 ways to connect to AMT, Direct Connect and CIRA

Direct connection is recommended for LAN type (flat) networks. In this mode, a console or application can directly talk to AMT using TCP/TLS connection. And Ports 16992/16993 remain open and accessible remotely.
CIRA is recommended for Cloud or WAN use cases. In CIRA, all the traffic to AMT can only be routed via MPS, which makes the ports 16992/16993 inaccessible remotely.

Coming back to your question, to enable ports 16992 and 16993, follow these steps:

Create a Profile with TLS (see this doc)
At step 11, select Server and Non-TLS Authentication from the dropdown
Run the rpc-go activate command on the AMT machine with the new Profile

Awesome thank you very much! I'll give this a try.
From my understanding, we could only do CIRA from the demo that I followed. But I can take a look at that again to see where I might have missed this part!

Working! Thank you!


What-if for FrontDoor throws python exception

Describe the bug
Command Name
az deployment group what-if
Errors:
cli.azure.cli.core.util: azure.cli.core.util.handle_exception is called with an exception:
cli.azure.cli.core.util: Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/custom.py", line 752, in what_if_deploy_arm_template_at_resource_group
    return _what_if_deploy_arm_template_core(cmd.cli_ctx, what_if_poller, no_pretty_print, exclude_change_types)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/custom.py", line 833, in _what_if_deploy_arm_template_core
    print(format_what_if_operation_result(what_if_result, cli_ctx.enable_color))
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 67, in format_what_if_operation_result
    _format_resource_changes(builder, what_if_operation_result.changes)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 160, in _format_resource_changes
    _format_resource_changes_in_scope(builder, scope, resource_changes_in_scope_list)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 176, in _format_resource_changes_in_scope
    _format_resource_change(builder, resource_change, is_last)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 196, in _format_resource_change
    _format_property_changes(
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 239, in _format_property_changes
    _format_property_change(builder, property_change, max_path_length, indent_level)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 258, in _format_property_change
    _format_property_array_change(builder, children, indent_level + 1)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 342, in _format_property_array_change
    _format_property_changes(
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 239, in _format_property_changes
    _format_property_change(builder, property_change, max_path_length, indent_level)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 255, in _format_property_change
    _format_property_modify(builder, before, after, children, indent_level + 1)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 307, in _format_property_modify
    _format_property_changes(
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 239, in _format_property_changes
    _format_property_change(builder, property_change, max_path_length, indent_level)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 255, in _format_property_change
    _format_property_modify(builder, before, after, children, indent_level + 1)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 318, in _format_property_modify
    _format_indent(indent_level)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/command_modules/resource/_formatters.py", line 491, in _format_indent
    builder.append(str(Symbol.WHITE_SPACE) * 2 * indent_level)
AttributeError: 'int' object has no attribute 'append'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/arm.py", line 100, in handle_template_based_exception
    raise CLIError(ex.inner_exception.error.message)
AttributeError: 'AttributeError' object has no attribute 'inner_exception'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 712, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/arm.py", line 102, in handle_template_based_exception
    raise CLIError(ex)
knack.util.CLIError: 'int' object has no attribute 'append'

cli.azure.cli.core.azclierror: 'int' object has no attribute 'append'
az_command_data_logger: 'int' object has no attribute 'append'

To Reproduce:
This is most probably related to some specific that I applied for front door - for a simple example this works
Expected Behavior
What-if will succeed and show applied changes
Environment Summary
macOS-11.4-x86_64-i386-64bit
Python 3.8.11
Installer: HOMEBREW
azure-cli 2.25.0

Additional Context
I experience this on cli 2.23.0, 2.24.0 and 2.25.0, on MacOS and in Docker.
CorrelationId: ece931bc-0951-400b-92ca-34a3147e24bc

I just reviewed what-if output with the fixed cli bug and it turned out it came out because I used invalid variable (my bug), and the deployment would fail anyway.
Nevertheless, the bug in python is still there and it might arise at the point we do not expect.

route to service team


[Feature] full scan using ffmpeg

Is your feature request related to a problem? Please describe.
The problem I encountered is that video files were not completely transferred and so not playing fully (sometimes containing gaps of 5-10 minutes), yet ffprobe marked these files as having no issues.
Describe the solution you'd like
Full analysis of files using ffmpeg.
Describe alternatives you've considered
Manually running ffmpeg on newly copied files which was able to identify the issue.
Additional context
Use the same database of a quick or full check.

@jakepog do you happen to have details on what command you ran to id these and problem files and what the output was in ffmpeg? I'd like to get this into the 3.4.0 release

The code I now run manually, after navigating into the appropriate folder:
find . -name "*.mkv" -exec sh -c "echo '{}' >> ../../../root/errors.log; ffmpeg -v error -i '{}' -map 0:1 -f null - 2>> ../../../root/errors.log" \;
I cannot recall the error sorry, and no longer have any files that have errors to replicate.
By trial and error I made code from this stack overflow post.

I should be able to get something going based on that, thanks! Currently planning to have this land in 3.5.0 if there's no issue with the implementation.


Add AndroidX support.

Addresses #1280
It would be nice if we could get this functionality tested within ButterKnife. Not sure the best way to go about since we have to add androidx artifacts to the processor classpath.
For the time being I was able to deploy to maven local and successfully built an AndroidX app using this change. Strangely I had to declare an explicit dependency on ButterKnife's annotations artifact otherwise they didn't appear on my classpath.

Travis timed out but this builds just fine.

Great work. Thanks!


Failed to initialize crash reporting

HI,
I'm facing this type of issue.
Could you help me resolve the problem?

    java.lang.NullPointerException: Attempt to invoke virtual method 'void android.app.ActivityManager.getMemoryInfo(android.app.ActivityManager$MemoryInfo)' on a null object reference         at bgh.<init>(:com.google.android.gms.dynamite_dynamitemodulesc<EMAIL_ADDRESS>(040308-192802242):29)         at com.google.firebase.crash.internal.api.FirebaseCrashApiImpl.init(:com.google.android.gms.dynamite_dynamitemodulesc<EMAIL_ADDRESS>(040308-192802242):12)         at bfv.onTransact(:com.google.android.gms.dynamite_dynamitemodulesc<EMAIL_ADDRESS>(040308-192802242):22)         at android.os.Binder.transact(Binder.java:507)         at com.google.android.gms.internal.measurement.zzn.transactAndReadExceptionReturnVoid(Unknown Source)         at com.google.android.gms.internal.measurement.zzxa.zza(Unknown Source)         at com.google.firebase.crash.zze.run(Unknown Source)         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:428)         at java.util.concurrent.FutureTask.run(FutureTask.java:237)         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)         at java.lang.Thread.run(Thread.java:762) 04-30 12:36:17.333 17645-17667/ir.ba24.key E/FirebaseCrash: Failed waiting for crash api to load.     java.lang.InterruptedException         at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedNanos(AbstractQueuedSynchronizer.java:1036)         at java.util.concurrent.locks.AbstractQueuedSynchronizer.tryAcquireSharedNanos(AbstractQueuedSynchronizer.java:1327)         at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:249)         at com.google.firebase.crash.FirebaseCrash.zzrl(Unknown Source)         at com.google.firebase.crash.FirebaseCrash.zza(Unknown Source)         at com.google.firebase.crash.zza.run(Unknown Source)         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)         at java.lang.Thread.run(Thread.java:762)

Step 2: Describe your environment

Android device:  Samsung galaxy S6
Android OS version: 7.0
Google Play Services version:  com.google.gms:google-services:3.2.1
Firebase/Play Services SDK version: 15.0.0

@pesehr thanks for reporting this.  Can you reproduce this issue locally or is this something your users are seeing?
Also can you confirm what version of Google Play services is installed on the Samsung Galaxy S6 device?

@samtstern thanks for your response.
Yes, I can reproduce it.
The version of Google play services on my device is: 12.5.29

@pesehr can you show the minimal amount of code needed to reproduce it on that device?

Ok so the sequence of events is:

See a crash
Launch the app again
Get this error in the logcat

Does this cause the app to crash or is this just reported in the logs?

this just reported in the logs.

@pesehr thank you, good to know it's not fatal.  We'll try to reproduce and then forward it on to the crash reporting engineering team.
By the way, you might want to consider moving to Crashlytics:
https://firebase.google.com/docs/crashlytics/upgrade-from-crash-reporting
It's the best option going forward and the migration is really simple.

@samtstern
Actually, I use Crashlytics now but it doesn't report crashes so I thought maybe it's related to this error.

@pesehr I apologize for not responding, did you get Crashlytics to work?

@samtstern yes, that's my fault.
but I still get the reported error :(

Could you show your dependencies block for me?

@pesehr I hope you solved this problem!  Closing this issue because it's an old thread with no recent responses.  If you are still seeing this, please let me know and I will re-open it.

i got same issue


Add aws error fields and their values to log

Change Overview
Add aws error fields and their values to log
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[x] :sunflower: Logs
[ ] :world_map: Documentation
[ ] :robot: Test

@ewhamilton please review the changes in the PR that you suggested, when you get time. Thanks!


Allow ChangeLogs to be instantiated outside MongoBee and passed in

This is motivated by the desire to allow use with a wider set of DI frameworks which could instantiate change log objects and pass them to MongoBee.
This allows change allows a developer to give MongoBee a predefinied list of change log objects which it will execute on, allowing the setup of change log objects using DI through Guice, Spring, or any other method.
This makes ChangeService an abstract class with implementations of PackageScannedChangeService (current behavior) and PreInstantiatedChangeService. The latter is used when Mongobee is given a list of pre-instantiated change log objects to use before the execute method is called.
This potentially breaks any callers that instantiate ChangeService directly, but ChangeService is never mentioned in current documentation nor used for Mongobee setup, so risk is small.

Decided this doesn't work as well as I might like with external DI frameworks... rethinking how this can be done without explicit external dependencies.


fixing production Order manager and print pdf

Coverage decreased (-0.06%) to 63.507% when pulling 6353ca3e68f84aba1a384e0398bb3fc009259d5d on achensetyawati:dev into 105842cb099bccf9b0ca495b5238e9225e400358 on danliris:dev.


Value not transformed when it is 0

Tranform values when we get actual 0.

https://github.com/yesoreyeram/yesoreyeram-boomtable-panel/commit/e32b884be9b738898a269661179c8762913cd9fd should close this.


Impossible to export comments

Good job for this plugin !
I have a little problem. Comments outside artboard don't export to PDF. Do you have a solution for that ?
Thinks

This is by design, since the plugin ignores anything that's outside an Artboard.
Will tag this as a feature request and see what we can do.

I'd love to get this feature implemented, as it is so common to annotate behaviours outside the artboard (to not mix design with behavioral requirements). Exporting Artboards + Links + outside elements would be a huge step forward as you can see the whole user journey at a glance.
See example of annotations that are not exported:


Add code examples for plugins

Operator JWT: Parse JWT Token
Action Challenge: Redirect the user to a bot challenge
Transformation SplitSpaceRight:  Split the value by spaces and get the last value from left to right
Logger zap: A ZAP logger
Body processor CVS: A CVS body processor

Codecov Report
Base: 77.52% // Head: 77.44% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (2ba21e9) compared to base (cc0c325).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           v3/dev     #498      +/-   ##
==========================================
- Coverage   77.52%   77.44%   -0.08%
==========================================
  Files         144      137       -7
  Lines        7474     7430      -44
==========================================
- Hits         5794     5754      -40
+ Misses       1448     1445       -3
+ Partials      232      231       -1

Flag
Coverage Δ

default
72.93% <ø> (ø)

examples
28.40% <ø> (ø)

ftw
54.72% <ø> (ø)

ftw-multiphase
?

tinygo
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bodyprocessors/multipart.go
78.48% <0.00%> (-3.80%)
:arrow_down:

internal/corazawaf/rule.go
67.47% <0.00%> (-1.42%)
:arrow_down:

operators/rbl_tinygo.go

loggers/init_tinygo.go

loggers/noop_formater.go

internal/sync/pool_tinygo.go

operators/inspect_file_tinygo.go

bodyprocessors/xml_tinygo.go

loggers/noop_writer.go

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

What is missing here?


Browsing LOD published as JSON-LD doesn't work

For example:
http://en.lodlive.it/?http://musicbrainz.org/release-group/d3bc1a64-7561-3787-b680-0003aa50f8f1
Is it possible to add support for JSON-LD?

LodLive support JSON-LD via a simple Jena Proxy, does musicbrainz.org support content negotiation?

Yes MusicBrainz does support conneg if you try with "Accept: application/ld+json" but I am not sure the support is perfect (like if you use q factor).
What Accept header is the proxy sending in the request?
Regards,
John
On 8 Feb 2016, at 23:37, Diego Valerio Camarda<EMAIL_ADDRESS>wrote:

LodLive support JSON-LD via a simple Jena Proxy, does musicbrainz.org support content negotiation?
—
Reply to this email directly or view it on GitHub.

Has this problem been resolved? I am trying to browse JSON-LD resources with LodLive but it does not work.
Example: https://json-ld.org/test-suite/tests/compact-0025-in.jsonld

@CaptSolo lodlive will never work with that example because the URL that you are opening:
https://json-ld.org/test-suite/tests/compact-0025-in.jsonld
is not the URI of the resource http://example.com/queen
@jaw111 sorry but the old sesame proxy don't understand your resource
http://en.lodlive.it/?http://musicbrainz.org/release-group/d3bc1a64-7561-3787-b680-0003aa50f8f1
Jena instead does it
https://lodview.it/lodview/?IRI=https%3A%2F%2Fmusicbrainz.org%2Frelease-group%2Fd3bc1a64-7561-3787-b680-0003aa50f8f1&sparql=<>&prefix=https%3A%2F%2Fmusicbrainz.org%2Frelease-group%2Fd3bc1a64-7561-3787-b680-0003aa50f8f1
I hope to find time someday to update the proxy


Blacklist a key while an IP is using it

Add the ability to ignore a key while there's an IP whitelisted that got access through that key.
Might help curb key sharing if they can't be used from multiple IPs at the same time.

Implemented in 20a9a0c.


Quick Find Definition doesn't update on paste (when copying between files)

Steps to reproduce:

Load two different files in Brackets that both have the same function definition.
Copy the name of the function in File 1.
Go to File 2 in brackets via the Working Files list.
Press Command + T to open the Quick Find Definition input box
Press Command + V to paste into the Quick Find

Note that the Quick Find doesn't update the results. If you press some arrow keys or any other keydown event -- it will eventually fire.
What should happen:
The Quick Find input should detect a paste command and execute the search for that value.

This is fixed in the upcoming 1.3 release


CODE REVIEW for Carnelos and De Zen: Utility app to manage 3 Volume Streams, and Vibration

KNOWN ISSUES:

Android-testing needs to be updated
layout may go out of the frame (missing layout adapters)

another known issue is that, for now, Multiple vibration is setted with a default pattern chosen by me.. In the future, I'm planning to make it more flexible.

The code is well formatted
Naming conventions are kept consistent
Some variables' names aren't entirely clear, as in buttons 0 to 5
Some method names are relatively ambiguous
No hardwiring
Specifics are fairly explanatory
No explanation at the top of files, a couple lines of comment would make it easier to understand what the class is supposed to do overall
AudioUtilityManager class is reusable in other contexts
Not bad overall, I added comments with suggestions or doubts on specific parts of the code.


How to make a Wiegand Keypad work ?

I am working with the Interrupt example on an ESP32 and I have no idea how to make a Wiegand Keypad work.
If I set wiegand.begin(26, true); and press on the keypad I get the Card read error: Message size unexpected - Raw data: 8bits  error and the 8bit raw data of each key, which I guess is HEX:
1 = E1; 2 = D2; 3 = C3;
4 = B4; 5 =A5 ; 6 = 96;
7 = 87; 8 = 78; 9 = 69;
* = 5A; 0 = F0; # = 4B;

If I set wiegand.begin(Wiegand::LENGTH_ANY, true); and press on the keypad I get nothing. No reads, no messages, no errors.
void IRAM_ATTR receivedData(uint8_t *data, uint8_t bits, const char *message)
{
    uint8_t bytes = (bits + 7) / 8;
    // Wiegand 26, first and last bits are for control, remaining 24 bits
    if (bits == 24)
    {
        wiegandFacilityCode = String(data[0] >> 4, 16) + String(data[0] & 0xF, 16);
        wiegandCardNumber = "";

        for (int i = 1; i < bytes; i++)
        {
            wiegandCardNumber += String(data[i] >> 4, 16);
            wiegandCardNumber += String(data[i] & 0xF, 16);
        }
        didReadWiegandCard = true;
    }
}

// Notifies when an invalid transmission is detected
void receivedDataError(Wiegand::DataError error, uint8_t *rawData, uint8_t rawBits, const char *message)
{
    Serial.print(message);
    Serial.print(Wiegand::DataErrorStr(error));
    Serial.print(" - Raw data: ");
    Serial.print(rawBits);
    Serial.print("bits / ");

    // Print value in HEX
    uint8_t bytes = (rawBits + 7) / 8;
    for (int i = 0; i < bytes; i++)
    {
        Serial.print(rawData[i] >> 4, 16);
        Serial.print(rawData[i] & 0xF, 16);
    }
    Serial.println();
}

Hello, @MetriciRO
Keypads are usually sent in 4 or 8 bit  messages, and from the code you sent it looks like you are ignoring anything but 26 bits.

Hello, @MetriciRO
Keypads are usually sent in 4 or 8 bit messages, and from the code you sent it looks like you are ignoring anything but 26 bits.

You are right. I've made it work. Thank you.


Compose : Trim extra spaces while rendering inline code

The in-line code inserted using  two `  must also be rendered with trimming extra spaces from the beginning and end of content as it happens with the case of quoting something or adding code.
for example... this

is rendered as

whereas this

is rendered as

@SatyendraBanjare we have intentionally allowed white spaces in code(``) elements. Any discussion about this at chat.zulip.org?
@rishig can confirm this.

@zulipbot claim

@shubhamdhama @Shobitha-M  very very sorry, i looked into this now only (my bad :cry: :cry: ) and on asking @hackerkid , he says its kind of intentional.
Next time I will be more careful with addressing issues.

@shubhamdhama  @Shobitha-M  closing this. Very very sorry for inconvenience.


react-addons-test-utils still in published 1.0.8

Looks like something went wrong with the publish to npm:
npm view<EMAIL_ADDRESS>peerDependencies
{ react: '^0.14.8 || ^15.1.0',
  'react-dom': '^0.14.8 || ^15.1.0',
  'react-addons-test-utils': '^0.14.8 || ^15.1.0' }

Just re-deployed<EMAIL_ADDRESS>fixes the issue.


feat(Datagrid): FilterFlyout responsiveness

What will this achieve?
Currently the FilterFlyout is not responsive.

How will success be measured?
Follow the guidance on creating a responsive layout for the FilterFlyout.
https://pages.github.ibm.com/cdai-design/pal/components/data-table/filters/#responsive

Additional information

Designs
Existing code
etc

FYI @sdignum @vsnichols — we can change the breakpoint in which the responsive version of the toolbar occurs, but that may not be ideal for all cases.

@AlexanderMelox to prototype ways to handle the responsiveness.

Hey @matthewgallo
Signing up for that review


Notifier improvements and fixes for Linux

Correct an issue where if libnotify-bin is missing then RuneLite doesn't do anything
Add an icon to the Linux notifications using libnotify-bin if it's present

Sample on Ubuntu Budgie:

This requires rebasing now from #735 . Notifier is now guice managed so you can just inject the execution service into it.

Code's rebased now.

I have no idea, at all, if this is even relevant anymore.
But upstream libnotify devs fixed the "notify-send never tells you if it errored anymore" bug.
I have no idea if this is useful.
I honestly forgot which project this was for and had to use GitHub search to find this issue.


README: added regedit command to set settings

Source:
https://looking-glass.hostfission.com/wiki/Using_Scream
I named my key wrong and spent alot of time figuring it out. I feel that this command would prevent alot of people from entering the wrong name for the key. Feel free to change the wording.

Thanks for the suggestion. I've added it to README.md with a slightly different wording.


compiled binary of shapely.box segfaults

bash-4.2$ nuitka3 --version
<IP_ADDRESS>
Python: 3.8.2 (default, Jun 16 2020, 17:24:50)
Executable: /vendor/tools/FOSS/python/3.8.2_Apex_20200616a/x86_64/bin/python3.8
OS: Linux
Arch: x86_64
I've tried this on 0.6.6 and it has the identical behavior, and haven't seen any issues created for this specifically. I believe this has to do with the geos references built into shapely. Here is a simple testcase:
nuitka command:
nuitka3 --standalone --follow-imports --show-modules --show-progress --plugin-enable=numpy --include-matplotlib shapely_test.py
shapely_test.py:
from shapely.geometry import box
VAR = {}
VAR['x'] = 963.072
VAR['y'] = 706.56
print("this prints before the shapely code", 0, 0, VAR['x'], VAR['y'])
b = box(0, 0, VAR['x'], VAR['y'], ccw=False)
print("this prints after the shapely code", b)
The shapely pip installation is based on the default build, so I'm also experimenting with the installation to rebuild the geos references locally. Once I have results I will share and post to this issue.

I'm able to solve this issue by:

uninstall of shapely through pip -uninstall
reinstall shapely through pip3 install shapely --no-binary=shapely

implicit in these steps is have shapely source code installed AFTER related geos dependencies are installed (geos, numpy, etc). The nuitka compiled code would not have the same segfault once it can follow the source code based shapely build.


v3.0 Breaking Changes

This is a list of planned breaking changes in v3.0.0.

[x] r, s values will be returned QUANTITY encoded for eth_getTransactionByHash
[x] eth_sign should return chainId-relative signature with high-level v values
[x] fix bug where ganache treats all transactions as having a chainId of 1, but RPC eth_chainId returns 1337 (we need to just make chainId configurable). We need investigate aligning net_version and chain_id as metamask uses net_version instead of chain_id for signing transactions.
[x] move reporting of tx hash on error to error field to prevent poorly-written clients which assume that the existence of the "result" field implies no errors from breaking.
[x] relative to above: don't return both result and an err, ever. Maybe. Need to discuss further.
[x] flip the VM errors on RPC response flag
[ ] old db format will no longer be valid. Investigate how to update old dbs to the new one without forcing the user into the new release.
[x] Update web3_clientVersion to include Ganache in its return value.
[x] improve precision of internal timeAdjustments (this is more of a bug fix that could break test that use specific block times via the time option, evm_increaseTime, or evm_setTime)

Does web3.eth.accounts.sign generate signatures with a v of 27/28? It seems that it does but does not generate correct signatures according to this test
https://travis-ci.org/ethvault/ens-registrar-contract/jobs/561935235#L612
Otherwise is there a workaround for writing tests for code that uses ECDSA.recover?

@moodysalem did you find a solution? I'm wondering if there is a workaround as well. Currently cannot verify signatures correctly using Ganache CLI v6.7.0-beta.0 (ganache-core: 2.8.0-beta.0) due to the v parameter.

@mrwillis I reverted to an older version of the ECDSA code.

move reporting of tx hash on error to error field to prevent poorly-written clients which assume that the existence of the "result" field implies no errors from breaking.

The JSON-RPC specification explicitly states that

[result] MUST NOT exist if there was an error invoking the method

and

[error] MUST NOT exist if there was no error triggered during invocation

I wouldn't call someone who wrote a JSON-RPC client to-spec "poorly-written".

Thanks for the clarity on this, @MicahZoltu. This line was copy-pasted from a TODO in our source code: https://github.com/trufflesuite/ganache-core/commit/0ed5d175030137a36bb8d3ee3afb440b213966d9#diff-1c25f2af06a7bfd6c36abff0f88b9a2bR189-R191
I think the comment in the code there was written because the author is a huge proponent of The Robustness Principle:

Be conservative in what you do, be liberal in what you accept from others (often reworded as "Be conservative in what you send, be liberal in what you accept").

In other words, programs that send messages to other machines (or to other programs on the same machine) should conform completely to the specifications, but programs that receive messages should accept non-conformant input as long as the meaning is clear.

I am a big fan of the Robustness Principle.  However, in JSON-RPC the presence/non-presence of those two variables are the intended mechanism for discriminating between success and failure, and having both is considered an error (per the specification).  Because of that, I don't think the robustness principle applies here as the correct way to discriminate is to look for either a success or an error and expect never both.

Any progress on this, particularly the chainId hardcoded to 1? Still having the issues with Ganache 2.4.0

@balthazar yes! You can follow along in the ts branch of this repository.

Sorry to bump this. Are there any updates on 3.0? I can't find the ts branch.

@frangio It was merged into develop. We're working on buttoning some things up and porting the last feature: forking.


Not able to set custom location for WebDriver

Hi,
I'm trying to change the Chromedriver location to run test cases but I'm always getting the error:
WebDriverException: Message: 'chromedriver' executable needs to be in PATH. Please see https://sites.google.com/a/chromium.org/chromedriver/home
I'm changing the webdriver location using an init.robot file, the keyword succeeds but still getting the error, I'm using this keyword to change the location path:
Set Environment Variable | webdriver.chrome.driver | 'C:\WebDrivers\chromedriver.exe'
Thanks for the help!!

This is not the issue tracker for Selenium2Library, which is what you want to report this.


Remove [[Enumerate]] and associated reflective capabilities

At the January TC39 meeting and subsequent conversations we got consensus on removal of the [[Enumerate]] MOP operation and the associated Reflect.enumerate and proxy trap enumerate. A PR implementing this consensus is here: https://github.com/tc39/ecma262/pull/367.

https://github.com/tc39/test262/pull/496

@Yongqu
This is failing following ESNext kangax test case as well
print((function()
{
var passed = true;
var proxy = new Proxy({}, {
  enumerate: function() {
    passed = false;
  }
});
for(var key in proxy); // Should not throw, nor execute the 'enumerate' method.
return passed;
})());

Expected: true
Actual:Exception.

Removing 1.2 milestone due to backward compatibility concern. Please fix this in Master.


ecc.isValidPublic('EOS5K6ag7t87u3F4jdkmt73wQtd9oprQFyJsp6vskrrwoQoPqcPdE7') return false

Version of EOSJS
16.0.9
Describe the bug

ecc.isValidPublic() return false on 2 public keys of account 1255gdpaftgs
Point.decodeFrom() report 'Invalid sequence tag'

To Reproduce

get the public keys of 1255gdpaftgs

validate these keys using ecc.isValidPublic()

# umu @ UMU-MBP in ~/github/meet-one/mainnet-snapshot on git:master x [21:45:27]
$ cleos -u https://mainnet.meet.one get account 1255gdpaftgs
created: 2018-11-08T01:25:13.500
permissions:
     owner     1:    1 EOS5K6ag7t87u3F4jdkmt73wQtd9oprQFyJsp6vskrrwoQoPqcPdE7
        active     1:    1 EOS5Kkxpsx71txnR8niEmZmcG6d6AeSQqYw1qfXYQBDkgh5v4tbGxH
memory:
     quota:     5.345 KiB    used:     3.365 KiB

net bandwidth:
     staked:          0.1000 EOS           (total stake delegated from account to self)
     delegated:       0.0000 EOS           (total staked delegated to account from others)
     used:                 0 bytes
     available:        79.93 KiB
     limit:            79.93 KiB

cpu bandwidth:
     staked:          0.1000 EOS           (total stake delegated from account to self)
     delegated:       0.0000 EOS           (total staked delegated to account from others)
     used:                 0 us
     available:        12.63 ms
     limit:            12.63 ms

producers:     <not voted>

# umu @ UMU-MBP in ~/github/meet-one/mainnet-snapshot on git:master x [21:45:33]
$ node
> const ecc = require('eosjs-ecc')
undefined
> ecc.isValidPublic('EOS5K6ag7t87u3F4jdkmt73wQtd9oprQFyJsp6vskrrwoQoPqcPdE7')
false
> ecc.isValidPublic('EOS5Kkxpsx71txnR8niEmZmcG6d6AeSQqYw1qfXYQBDkgh5v4tbGxH')
false

Expected behavior
true, but return false.

@jeffreyssmith2nd


Open Pervasives.JsxModules by default.

See https://github.com/rescript-lang/rescript-compiler/pull/6091 where the jsx modules are moved inside Pervasives, to provide 2 different copies for curried and uncurried mode.

One still gets e.g. JsxC and JsxU in the autocompletion list, just because they exist.
@zth I guess there are other similar issues elsewhere. I was wondering whether one could add some sort of annotation to some modules so they don't show up in autocomplete.


Add missing language in codeql.yml

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

Please upload report for BASE (main@86ff5e2). Learn more about missing BASE report.

Additional details and impacted files
@@          Coverage Diff           @@
##             main    #250   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?      20
  Lines           ?    1512
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?    1512
  Partials        ?       0

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


fix: change default test client host to localhost

Signed-off-by: slayerjain<EMAIL_ADDRESS>

This is important in cases like HTTP redirection. In that case, may web frameworks can use the host header to figure out where to redirect requests. localhost seems more common than <IP_ADDRESS>.


Request Timed Out

Describe the bug
Getting a request timed out error (see #312)
-- exact same issue
Versions
Node: v16.15.1
node-steam-user: 8.14.0
Screenshots and Error Logs
https://i.lunardev.group/xozRPFk/direct

There's no such thing as node-steam-user 8.14.0. I need to see your code.

Apologies -
node-steam-user version: 4.24.3
I'm really just using a provided example: https://i.lunardev.group/iCKE2hP/direct

@WinterFe is just me or i cannot see anything at the link? Expired mby?

Same problem. The bot works for 2 minutes, then get a timeout. After reconnecting, it works again for 2 minutes. The problem started in October and is still there.
steam-user v4.26.0


Better to scope firebase-admin to auth user permissions?

In the "with-firebase-authentication" example the server attaches a unrestricted firebase-admin instance to the request for later use in getInitialProps. This is ok, but wouldn't it be better to scope the firebase-admin down to just the permission level of the authed user? That way any query you run server-side has the same permissions as the user. I believe this is easily accomplished by omitting:
server.use((req, res, next) => {
    req.firebaseServer = firebase
    next()
  })

and instead adding in an additional express middleware (after the 'api/login' route):
server.use((req, res, next) => {
      try {
        admin.app('userScopedServer').delete()
      } catch (error) {
        console.log("creating new user scoped firebase-admin instance")
      }

      if (req.session && req.session.decodedToken && req.session.decodedToken.uid) {
        const userScopedFirebaseServer = admin.initializeApp({
          credential: admin.credential.cert(require('./credentials/server')),
          databaseAuthVariableOverride: {
            uid: req.session.decodedToken.uid
          }
        }, 'userScopedServer')

        req.userScopedFirebaseServer = userScopedFirebaseServer
      }
      next()
    })

Or am I thinking about this incorrectly?

@jakelowen It is certainly a very good idea to limit thescope of the server-side permissions to the user. The only issue I see is that it doesn't seem like you can make anonymous firebase requests in your example since the firebase-server is only set for authenticated users.

@jakelowen, this the solution I came up with to ensure matching scopes on the client and server...
    // Server.js --->
    // verify auth token and extract the user's uid
    const uid = await admin
      .app()
      .auth()
      .verifyIdToken(token)
      .then(({ uid }) => uid)

    // create a `customToken` with the user's uid
    const customToken = uid ? await admin
      .auth()
      .createCustomToken(uid) : null

    // Page.js --->
    // sign-in using the `customAuthToken` created above
    await firebase
      .auth()
      .signInWithCustomToken(customAuthToken)
      ...


Detect new sectors since last run and tics included

We are already providing an updater based on a timestamp.


Mautic documentation

What do these changes do?

Added documentation for Mautic

Type of documentation

[x] New documentation
[ ] Documentation update
[ ] Hotfix

This PR is ready for review -identify and group snippets to be shared by @anantjain45823

@anantjain45823 @shrouti1507 all the latest changes as per the spec are now included in the doc. Please review this PR and share your feedback.

Group call can be used to link as well as delink the contacts with segment, campaign, or a company.


does not set the column width once the streamwrite has been flushed

https://github.com/qax-os/excelize/blob/933159f9391f9be1b41b51e85885722124f8a7aa/col.go#L454
I will not know the max width of my columns until I have read the data from the database query... and while the streamwrite seems to expect that the column width is set before the first row is written to the sheet... so I figured I could flush the streamwriter then use the file.setcolwidth() but that did not work.

Thanks for your issue, if so, please consider using the normal API instead of streaming API.

I tried this a number of different ways. I tried using file.setcolwidth() while writing to the stream and after I closed the stream. Nothing worked.

As stated in the documentation, the common API and stream API can't be work mixed to writing data on the worksheets, this issue is similar to issues #725, #777 and #826, you should be call streamWriter.SetColWidth instead of f.SetColWidth in the streaming mode.


os-bind - SafeSearch - Google domains

The Google SafeSearch only redirects two of Googles domains, there are lots more that should be included.
On this page https://support.google.com/websearch/answer/186669?hl=en, under the heading 'Lock SafeSearch by making changes to your router' there is an entry (and any other Google ccTLD country subdomains your users may use) which links to this list https://www.google.com/supported_domain.
Could you include all the domains in that list?
Thanks,
Steve

Can you do me a favor and make a list like in this format?
https://github.com/opnsense/plugins/blob/master/dns/bind/src/opnsense/service/templates/OPNsense/Bind/google.db
Then I can include it to the template ...
Thanks!

Thanks,
Is this what you meant?
google.doms.zip

Look good, thanks! I added it here: https://github.com/opnsense/plugins/pull/1176/files
As you can see it's very easy, so when you want to start contributing on your own I'd be glad to help :)


Refactor Eventing in LiveClient(s)

Proposed changes
A refactor can be done to handle events generically. This has been on my list for a while and started the implementation a bit ago.. just waiting for things to stabilize with new features.
Context

Why is this change important to you? How would you use it? How can it benefit other users?

Possible Implementation

Not obligatory, but suggest an idea for implementing addition or change

Other information

Anything else we should know? (e.g. detailed explanation, related issues, links for us to have context, eg. stack overflow, codepen, etc)

Closing as it's available in this release
https://github.com/deepgram/deepgram-python-sdk/releases/tag/v3.7.2


Not working at all as of 19/1/

Restarted discord today and after reinstalling betterdiscord version 3.0 and making sure the quicksave plugin is still in the folder and activated in the discord tab, I can't get the icon to show up whatsoever and can't save any images. Previous to that I was also experiencing the bug of the icons not appearing but the little grey box showing up and saving images. Currently I'm not getting the grey box or any icons and can't save anything.

Fixed. Sorry about the slow updates.

Thank you for the update !
While it's nice to see it functioning again and actually saving images even from links and in full size & quality, I'd still really love it even if there was an optional version that kept the little grey box just because it allows for 1 click saving and it was available in the 'search' directory in servers (discord doesn't open images or allow right clicking in the search column), since I use this plugin to mass-save images off a server the difference between doing 1 click in the search column or having to jump back in chat, open image, save image, makes quite a difference !
I understand that takes extra time and effort to make but theres nothing that mimics the original way this plugin functioned and it was the most useful and wonderful thing I've found for the app !

Few others wanted it back as well, didn't realize people loved it that much. Re-implemented it in my own way.

Thank you soooo much, 10000% perfect now !


use folder name as source for uploadDirectory command

Some testing for #46
Not sure if this is the best way to proceed, rule_directories should end with separator and the command creates automatically all the sources.

Coverage decreased (-0.2%) to 60.549% when pulling 5194c384d8f9dc4fcc955916015622442fc2818f on dadokkio:master into 9bf704dab48733dc722b58852e782c0dcf0c5f60 on PUNCH-Cyber:master.

using commit dd6ae27 in lieu of pull


Revert "Remove most uses of CompareInBounds (#13244)"

This reverts commit f785168aacf79e2923f0d140e17fdd4c9364e51d (PR #13244)
The error logged to NVDA was caused by the following line of code in _getTextValue():
THROW_HR_IF(E_FAIL, !bufferSize.IsInBounds(_start) || !bufferSize.IsInBounds(_end));
NVDA would expand a text range to encompass the document in the alt buffer. This means that the "end" would be set to the dangling "endExclusive" point (x = left, y = one past the end of the buffer). This is a valid range!
However, upon extracting the text, we would hit the code above. The exclusive end doesn't actually point to anything in the buffer, so we would falsly throw E_FAIL.
Though this could be fixed by adding a special check for the endExclusive in the line above, I suspect there are other places throughout the UIA code that hit this problem too. The safest course of action is to revert this commit entirely since it was a code health commit (it doesn't actually close an issue).
Closes #13866

FYI @DHowett

thanks a lot @carlos-zamora! What would be the best way to test this?

@leonardder The simplest way that I've found to test terminal PRs:

Click "details" under "Terminal CI (Build x64 Build x64 Release Dev) " in the CI results.
On the page that opens, click "View more details on Azure Pipelines".
Search the page for "artifacts" (note spelled with an i). You'll find a link labelled n artifacts produced where n is some number. Click that link.
This page is a bit fiddly. Find the table on the page, then switch to browse mode. Down arrow through the rows until you reach one labelled "drop", then hit right arrow.
In the menu that opens, down arrow to "download artifact" and press enter.
Extract the zip you downloaded. Inside the drop\appx folder, you'll find a CascadiaPackage_<IP_ADDRESS>_x64.msix. Rename it to a .zip and extract.
From that directory, run WindowsTerminal.exe.

@carlos-zamora moving the cursor around in a WSL nano or vim session doesn't explode after this, right (ala #13183)? I suppose that was fixed in #13250, not #13866...

Wait, but how am I supposed to remove the Viewport class in my upcoming commits then. ._.

Wait, but how am I supposed to remove the Viewport class in my upcoming commits then. ._.
To be precise, I don't see how this
bufferSize.CompareInBounds(A, B, true) <=> 0

is any different mathematically from this
A <=> B

since both do the exact same thing.

Replacing CompareInBounds() with <=> is fine.
The regression was mainly caused by the IsInBounds() check because endExclusive was no longer accepted after #13244 went through. I can create a separate commit solely replacing CompareInBounds() if you'd like, but I figured reverting #13244 entirely would be the safest thing to do for 1.15/1.16 since we're approaching release. You could also just replace CompareInBounds() in your PR yourself, up to you haha

@msftbot merge this in 10 minutes


Add sampling with Data Prepper use case

Description
Adds sampling with Data Prepper use case to common use cases section on doc site
Issues Resolved
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @lizsnyder: Tagging you for review at your availability. Thanks for your time and support.

@dlvenable or @Naarcha-AWS: Please provide a doc review of this PR. This use case is one in a series of uses cases we're bringing over from OpenSearch Service to Data Prepper. I've updated or removed language as applicable to OpenSearch open source and Data Prepper. I'll tag you in the other PRs for your doc reviews. Thanks, @vagimeli

@dlvenable I've addressed your comments. Please re-review at your availability. Thanks, Melissa


Redesigned Privacy Policies Page | Attractive UI | Modern Styling | Issue #280

fix #280

This update enhances the "Privacy Policies" page by implementing a fixed sidebar for easy navigation. The sidebar contains links to various sections, ensuring users can quickly access information.
The main content area has been adjusted to prevent overlap with the sidebar, providing a cleaner and more organized layout.
Additionally, sections are now smoothly animated for a better user experience.

Overall, these changes improve usability and accessibility on the page, making it easier for participants to engage with the content.
Video
https://github.com/user-attachments/assets/009e351b-fbcb-45dc-9e47-5b156ec731f7

@rishicds please give level 2 for this as this is revamping of existed page


Running Mojave in VirtualBox under Big Sur

I am running Big Sur on a MacBook Pro 2017 with 16Gb memory. Everytime I run the script when the Virtual Box is launched I allways get the same
image.pdf
I tried with other OSX images other then Mojave (all generated by installinstallmacos.py) I always get the same message. As you see from the log everything goes smothly
runMacOSVirtualbox.log
Hope you can help me sorting it out
António
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Additional context

log file (see ~/Library/Logs/runMojaveVirtualbox.log that is accessible via Console.app).
screen shot

I forgot to mention that I am using Virtualbox 1.16 r140961

I forgot to mention that I am using Virtualbox 1.16 r140961

Have you tried to delete all old files with clean and stash / stashvm? See make help. However, I'd assume something with using installinstallmacos.py is wrong. At the same time, the log doesn't look too bad...

Have you tried to delete all old files with clean and stash / stashvm? See make help. However, I'd assume something with using installinstallmacos.py is wrong. At the same time, the log doesn't look too bad...

I repeated tyhe whole process after running make clean and creating the Mojave install again.
The drive created in VirtualBox 6.1 boots properly but when I select Install MacOS I always get the message
"This copy of the Install macOS Mojave application is damaged, and can't be used to install macOS"
Any clues on what I should do? If needed I  can shared the Installation disk with you (about 500 Mb).
Thanks

I repeated tyhe whole process after running make clean and creating the Mojave install again.
The drive created in VirtualBox 6.1 boots properly but when I select Install MacOS I always get the message
"This copy of the Install macOS Mojave application is damaged, and can't be used to install macOS"
Any clues on what I should do? If needed I  can shared the Installation disk with you (about 500 Mb).
Thanks

Instead of using the menu option "Install MacOS" I tried to launch the installer directly from the Terminal using
/Install macOS Mojave.app/Contents/MacOS/InstallAssistant
and the system installed properly

Instead of using the menu option "Install MacOS" I tried to launch the installer directly from the Terminal using
/Install macOS Mojave.app/Contents/MacOS/InstallAssistant
and the system installed properly


Provide support for a config file for most common variables

So it would be nice to support a config file to avoid the common things needed on the command line:

SiteName (for most clients this never or rarely changes)
UserName (for most clients this rarely changes)
Version (we default to v4 but for clients on older versions it would be nice for them to be able to override)

I think the biggest need for now is support for a METASYS_DEFAULT_REST_API_VERSION variable. If a user has this set it should be honored. This allows a client on 11.0 to always get version of the API without always having to specify it.

I tried this but there's some corner cases. Like does the client know what version will be used if they specify a default, and then make a call, but on the next call they specify a different version.
Plus this is mitigated by the fact that you only need to specify the Version on the first call of a session. After that it's saved.

This has been complete for a while see configuration file


Make a school an Object rather than a String

This is not an issue but a suggestion.
Currently, we're returning the list of schools as a text. However, we might want to add other data as part of a school in the future.
For the list =>
Current: Nigeria.universities = List
Proposed: Nigeria.universities = List
For a school =>
Current: value = String
Proposed: value = School (and you can access school using value.name)
Then, when we want to add other school info such as faculty lists, department lists, it'd be easy.

Hmm... A nice idea!
Thank you so much. I will work on that.

Can you reopen this issue? Also can I send in a pull request to fix the issue?

It is opened already. Alright you can go ahead.


Prepare for PyPI

Switch to Poetry for package management
Add black for code formatting
Add license (MIT)
Declare support for Django ^2.1 and Python ^3.6
Make Django Rest Framework an optional dependency
Add the "drf" extra to install Django Rest Framework
Remove tox test runner

Travis is behind on jobs this morning, and I really want to get this tested before I merge, and this merged before I release. Hopefully it will finish soon.
https://www.traviscistatus.com/


Initialize with keepDirty and empty array creates an empty object in form values

Are you submitting a bug report or a feature request?

Bug report
What is the current behavior?

Hi, I'm working on an array structure form. When I delete a field in the array list that was before set to a 'empty-like' value {name: ""}, and triggers initialize with keepDirty option and an new empty initial value to make it pristine, the ReduxForm reducer creates an empty object in the state values.
What is the expected behavior?
The values array should not be altered by the reducer as the keepDirty parameter is set to true and the field indeed dirty.
Sandbox Link

https://www.webpackbin.com/bins/-KnHcmQvY3yKJ4ZF8m0C
What's your environment?

Any environment
Other information
import { reducer, initialize } from 'redux-form';
import structure from 'redux-form/es/structure/plain';
const { deepEqual, fromJS } = structure;

const before = {
  myForm: {
    registeredFields: {
      // field has been deleted in array before so fields have been unregistered
      myList: { name: 'myList', type: 'Field', count: 0 },
      'myList.0.name': { name: 'myList.0.name', type: 'Field', count: 0 },
    },
    values: {
      myList: [], // field has been deleted in array before
    },
    initial: {
      myList: [{ name: '' }], // but was still present in initial values, with an empty string value
    },
  },
};

// let's submit the data to the server, and the form will update its initialValues
// either programmatically with an action or with enableReinitialize for example
// so the new initialValues should now be
const actionInitialValues = [];
const actionKeepDirty = true;

const state = reducer(
  fromJS(before),
  initialize('myForm', actionInitialValues, actionKeepDirty),
);

console.log(JSON.stringify({
  myForm: {
    registeredFields: {
      // this part is not affected
      myList: { name: 'myList', type: 'Field', count: 0 },
      'myList.0.name': { name: 'myList.0.name', type: 'Field', count: 0 },
    },
    values: {
      // this is the problematic part, as it should be empty
      myList: [],
    },
    initial: {
      // new initial values are indeed modified to empty array
      myList: [],
    },
  },
}, null, 2));

console.log(JSON.stringify(state, null, 2));

       }
     },
     "values": {
-      "myList": []
+      "myList": [
+        {}
+      ]
     },
     "initial": {
       "myList": []

In the createReducer/js file :
The deepEqual function is guilty in part, because it considers [{name: ''}] and [] equals regarding the myList.0.name field. Therefore, since the old value is equal to the old initial value, it sets the new value equal to the new initial value to keep it pristine, which creates an empty object to set the 0.name property to undefined.

Fix published in v7.0.0.


$validate is not working

$validate does not work.
For example,
https://cloud.alphora.com/sandbox/r4/cqm/fhir/Library/SafeUseofOpioidsConcurrentPrescribingFHIR/$validate
returns
{
"resourceType": "OperationOutcome",
"text": {
"status": "generated",
"div": ...
},
"issue": [
{
"severity": "error",
"code": "processing",
"diagnostics": "Failed to call access method: java.lang.AbstractMethodError: Receiver class org.hl7.fhir.common.hapi.validation.validator.FhirInstanceValidator$NullEvaluationContext does not define or inherit an implementation of the resolved method 'abstract java.util.List resolveConstant(java.lang.Object, java.lang.String, boolean)' of interface org.hl7.fhir.r5.utils.FHIRPathEngine$IEvaluationContext."
}
]
}
This prevents tools like http://gb2.clinfhir.com/ from saving to a CQF Ruler because that tool tries to do $validate before saving

This is caused by a mismatch between the fhir core versions and the hapi versions referenced by the cqf-ruler. It's fixed on master, I need to redeploy the sandboxes so they pick up the fix.

The $validate operation is now functioning as intended.


Fix isco key via funciton edits

Consider either edits to read_pdf() and/or best_matches(). Also consider adjusting parameters. Bottom line is that we need to fix #114

Working by discovery that it seems to be valid to throw overlapping x lim parameters for the columns since the purrr function essentially operates independently. Now we need to fix the fact that best_matches() presents the incorrect match value for each row. Branching...

isco key merges correctly in 2017


Cmd-W closes tabs when About window has focus

When viewing the Code > About window, Cmd-W closes the open tab, not the About window, despite the About window having focus.

VSCode Version: 1.11.2 (1.11.2)
OS Version: MacOS 10.12.4 (16E195)

Steps to Reproduce:

Open a tab.
Click the Code menu.
Click the About menu item.
Press Cmd-W.

You can also dismiss the about dialog using ESC.

Yes, or by clicking the close button with the mouse, but this breaks MacOS conventions of Cmd-W closing the active window, and it can lead to unintentionally closing an open file.

We are not even creating the dialog, but just setting a role:about, I am not sure why commands can still execute when that window opens. Might be related: https://github.com/Microsoft/vscode/issues/11928


Replication restoration, whats the best strategy for maintaining replication after volume loss

I have a master setup with 001 default replication with 2 volume servers, 'node1' and 'node 2'.
Lets say I have a volId of 1 which is replicated across 'node 1' and 'node 2'. In the event that node 1  goes down and is completely lost what is the best path to recreate the replica of volId 1 on node 2? Lets say I can add a new 'node 3' into the cluster for the purpose of recreating the volId 1 replica.

There is a "weed backup" to continuously backup to some place.
For "node 3", the simplest way is just to rsync the file over, since volume 1 will be readonly at that time.


Integrated Modelling using Distributed Repositories

This feature adds the support of multiple different repositories.

[x] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit
[x] Ensure that the commit message is a good commit message
[x] Ensure to use auto format in all files
[x] Ensure that you appear in NOTICE at Copyright Holders
[x] Tests created for changes
[x] Documentation updated (if needed)
[x] Screenshots added (for UI changes)

Filed b9e838a as CQ 20519.


Allow parsing left-unbounded range (..10)

This allows parsing ..10 and similar as a left-unbounded range.
Some points:

Valid path should not be parseable as a valid range because the only case you can have a path separator in a range is this: 0..(4 / 2)..10 which involves spaces. Paths should not have spaces around the separator (I hope?).
.. is not allowed to be parsed as a range. Range always needs at least one bound, otherwise it could be easily confused with parent directory.
With this change, external commands with names that could be a valid range (e.g. ..10) are not allowed since they would parse as a range.
Internal commands named ..10 etc. are allowed.

looks good


Data in gimme_genome function in models.py

Hi @cdiener,
I hope my issue won't take much time, I have a pop-science talk in 2 weeks about microbiome research and was thinking of "stealing" a very nice interactive gimme_genome feature from your course:) Of course, with appropriate references to your course and ISB, just wanna add some interaction after the talk.
So, I tracked down the bug and it seems to be related to the data.
If you run models.py, !bootstrap command does not work and you will get HTTP Error 500 which leads to the reading of some assembly files.
I'm just wondering if it is a bug or maybe you made those files private intentionally, please, let me know, then I will come up with some other interactive activity.
Full error message:

Traceback (most recent call last):
File "/usr/local/bin/bootstrap", line 5, in
from course.bootstrap import main
File "/usr/local/lib/python3.7/dist-packages/course/init.py", line 1, in
from .utils import gimme_genome, gimme_exchanges, gimme_media, download, add_r_squared
File "/usr/local/lib/python3.7/dist-packages/course/utils.py", line 15, in
"https://github.com/Gibbons-Lab/2021_microbiome_course_data/"
File "/usr/local/lib/python3.7/dist-packages/pandas/util/_decorators.py", line 311, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/readers.py", line 586, in read_csv
return _read(filepath_or_buffer, kwds)
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/readers.py", line 482, in _read
parser = TextFileReader(filepath_or_buffer, **kwds)
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/readers.py", line 811, in init
self._engine = self._make_engine(self.engine)
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/readers.py", line 1040, in _make_engine
return mapping[engine](self.f, **self.options)  # type: ignore[call-arg]
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/c_parser_wrapper.py", line 51, in init
self._open_handles(src, kwds)
File "/usr/local/lib/python3.7/dist-packages/pandas/io/parsers/base_parser.py", line 229, in _open_handles
errors=kwds.get("encoding_errors", "strict"),
File "/usr/local/lib/python3.7/dist-packages/pandas/io/common.py", line 614, in get_handle
storage_options=storage_options,
File "/usr/local/lib/python3.7/dist-packages/pandas/io/common.py", line 312, in _get_filepath_or_buffer
with urlopen(req_info) as req:
File "/usr/local/lib/python3.7/dist-packages/pandas/io/common.py", line 212, in urlopen
return urllib.request.urlopen(*args, **kwargs)
File "/usr/lib/python3.7/urllib/request.py", line 222, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.7/urllib/request.py", line 531, in open
response = meth(req, response)
File "/usr/lib/python3.7/urllib/request.py", line 641, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.7/urllib/request.py", line 569, in error
return self._call_chain(*args)
File "/usr/lib/python3.7/urllib/request.py", line 503, in _call_chain
result = func(*args)
File "/usr/lib/python3.7/urllib/request.py", line 649, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)

That was error was on Github's side. For some reason accessing subfolders in the repo did not work. I put in a ticket and it seems to be working now. There was another issue with the bootstrap which should be fixed now as well. You will have to reset the colab runtime for that though.


Add Review 02 FEB

The code is very well organized and easy to follow. Most of the comments are really style preferences so do with them as you will.
Great job with this!

#104 closes this


Fix previous authorship mentions

In order to properly fit with the corresponding licenses, references concerning authorship have to use explicit links to their appropriate authors and note automatic mentions.
Fix previous authorship mentions in files:

[x] <git> head.md / tag.md
[x] <gpg> key.md
[x] <vscode> README.md

Close by #34


Move constant bodies to be a FunDecl

In crate_of_json we do a little hack where we transform each global into a function declaration that contains its body and a new global type that points to that function. I think we should do this on the rust side too: a constant would not contain a body and instead would contain a FunDeclId of its initializer function.

Isn't it what we actually used to do? (I agree with your proposal)

Maybe before I was here; as far as I know we've always done like today.

Fixed by #461


[AMS] Fix #14021: az ams account sp is not idempotent

Description
Fix for #14021
Testing Guide
Running the following command multiple times should output the same SP object
az ams sp create -n [sp-name] -a [account-name] -g [resource-group]
To change the name or role of an existing SP run the following command:
az ams sp create -n [sp-name] --new-sp-name [new-sp-name] -a [account-name] -g [resource-group] --role [role]
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

hi @fengzhou-msft could you pls help reviewing this PR? thanks


Fail XSLT on Error

From Radu:
at some stage the XSLT processing failed but the build ended with "Build
succesfull" so the build file would probably need a "failonerror"
attribute set on the Java tasks which run the XSLT transformations.

Fix committed on develop branch.


org.bouncycastle.crypto.fips.FipsSelfTestFailedError: Exception on self test: Array index out of range: 32: EC

getting the following stacktrace when upgraded bc-fips-<IP_ADDRESS>.jar to  bc-fips-<IP_ADDRESS>.jar. No issues when bc-fips-<IP_ADDRESS>.jar is used.
This is with IBM Java 8.
ERROR c.s.s.p.SecurityProperties - org.bouncycastle.crypto.fips.FipsSelfTestFailedError: Exception on self test: Array index out of range: 32: EC
          at org.bouncycastle.crypto.fips.SelfTestExecutor.validate(Unknown Source)

          at org.bouncycastle.crypto.fips.FipsEC.f2mDsaTest(Unknown Source)

          at org.bouncycastle.crypto.fips.FipsEC.<clinit>(Unknown Source)

          at java.lang.Class.forNameImpl(Native Method)

          at java.lang.Class.forName(Class.java:340)

          at org.bouncycastle.crypto.fips.FipsStatus.loadClass(Unknown Source)

          at org.bouncycastle.crypto.fips.FipsStatus.access$200(Unknown Source)

          at org.bouncycastle.crypto.fips.FipsStatus$Loader.run(Unknown Source)

          at org.bouncycastle.crypto.fips.FipsStatus.isReady(Unknown Source)

          at org.bouncycastle.crypto.CryptoServicesRegistrar.getDefaultMode(Unknown Source)

          at org.bouncycastle.crypto.CryptoServicesRegistrar.<clinit>(Unknown Source)

          at org.bouncycastle.jcajce.provider.ProvSecureHash$MD5.configure(Unknown Source)

          at org.bouncycastle.jcajce.provider.BouncyCastleFipsProvider.<init>(Unknown Source)

          at org.bouncycastle.jcajce.provider.BouncyCastleFipsProvider.<init>(Unknown Source)

          at org.bouncycastle.jcajce.provider.BouncyCastleFipsProvider.<init>(Unknown Source)

          at com.sterlingcommerce.hadrian.common.util.BCSecurityFipsUtilsImpl.<clinit>(BCSecurityFipsUtilsImpl.java:126)

          at java.lang.Class.forNameImpl(Native Method)

          at java.lang.Class.forName(Class.java:340)

          at com.sterlingcommerce.security.provider.SecurityProperties.getBCSecurityUtils(SecurityProperties.java:465)

          at com.sterlingcommerce.security.provider.SecurityProperties.initialize(SecurityProperties.java:705)

          at com.sterlingcommerce.security.provider.SecurityProperties.load(SecurityProperties.java:224)

          at com.sterlingcommerce.security.provider.SecurityProperties.load(SecurityProperties.java:158)

          at com.sterlingcommerce.hadrian.Main.isBootStrapEnabled(Main.java:762)

          at com.sterlingcommerce.hadrian.Main.main(Main.java:279)

Including email from David Hook on this issue:
From: David Hook
Sent: Wednesday, November 1, 2023 8:03 PM
To:
Subject: Re: [dev-crypto] Issue after FIPS upgrade from
Okay, I've been able to reproduce it.
The IBM BigInteger class uses a different method to the regular BigInteger class for building a BigInteger from a SecureRandom. Unfortunately it appears the new KAT from EC f2m results in the BigInteger constructor reading off the end of the fixed "random" data provided. This is something we can fix, but it would require updating the module's certificate which makes it unlikely for now.
Apologies,
David

This is now fixed in the early access version. Still in negotiations about organizing a release date.

Thank you David.
Naru
From: David Hook @.>
Sent: Monday, November 13, 2023 1:07 AM
To: bcgit/bc-java @.>
Cc: Manda, Naru @.>; Author @.>
Subject: Re: [bcgit/bc-java] org.bouncycastle.crypto.fips.FipsSelfTestFailedError: Exception on self test: Array index out of range: 32: EC (Issue #1532)
This mail has been sent from an external source. Do not reply to it, or open any links/attachments unless you are sure of the sender's identity.
This is now fixed in the early access version. Still in negotiations about organizing a release date.
—
Reply to this email directly, view it on GitHubhttps://github.com/bcgit/bc-java/issues/1532#issuecomment-1807574757, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BD5KY5HJ3YCBRQLZU7RG47LYEHBJVAVCNFSM6AAAAAA7ISVNHSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGU3TINZVG4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

If you've got a support contract <IP_ADDRESS> is now in early access, otherwise you'll need to role back to <IP_ADDRESS>.

I downloaded <IP_ADDRESS> and I no longer get the FipsSelfTestFailedError. However, I noticed that there is a vulnerability in that jar:  CVE-2022-45146. Has this vulnerability been fixed?
Thanks,
smartycardpants

Yes, in <IP_ADDRESS>

Yes, I guessed it was fixed in <IP_ADDRESS>, but that's where I ran into the FipsSelfTestFailedError which I cannot get around.

Thank you, David.
Another question - <IP_ADDRESS> has the FipsSelfTestFailedError that you mentioned is fixed in the early access version (<IP_ADDRESS>). When will <IP_ADDRESS> be publicly available without a support contract? Also, what does a support contract entail - cost? duration? etc. (Thanks again.)

Details about support and early access are available here https://www.keyfactor.com/open-source/bouncy-castle-support/ Contracts start at 25K USD. I'd guess we're currently looking at about 6 months for public release - the changes we are making are small so we're not anticipating any issues there, however the CMVP is under a lot of strain at the moment.


get current path via req.path instead of req.route.path

fixing https://github.com/msiebuhr/node-statsd-client/issues/71
Seems like req.path will do the trick and return the currently matched path as expected, according to docs its supported in express 3/4.

@cainvommars Nice work!
To prevent it from breaking in the future, could you please add a test with a server that uses routes defined this way? Looks like changing just one of the routes in the server set up in the before block of https://github.com/msiebuhr/node-statsd-client/blob/master/test/helpers.js could do the trick.

@papandreou sure thing - however, do tests run for you properly? I get a list or errors when running them locally. (node 6.10, npm 5)
`✖ 2 of 54 tests failed:

EphemeralSocket Sends data immediately with maxBufferSize = 0:
AssertionError: expected 8 to be close to 0 +/- 5

Helpers response times GET /foo/:param/bar → "GET_foo_param_bar":
Error: timeout of 2000ms exceeded
at Timeout. (/Users/peter/git/tmp/node-statsd-client/node_modules/mocha/lib/runnable.js:158:14)
at ontimeout (timers.js:380:14)
at tryOnTimeout (timers.js:244:5)
at Timer.listOnTimeout (timers.js:214:5)`

The tests run fine for me. From the error messages it seems like you're running into some timing/timeout related problems. I've tried tweaking the timeouts on master -- could you try rebasing and see if that helps?

this should do the trick,
what about all those js hint complaints btw?

I don't see any jshint complaints? If they're about stuff in the node_modules folder, you might have run jshint .? I've added a .jshintignore on master now so that won't fail with hundreds of errors in the future :)
It could also be that this project doesn't have a .jshintrc, so in effect you're linting using a rule set located somewhere in your home directory. We should probably do something about that.
But since it builds fine on CI, I think this is good to merge in its current state!

yes, I also figured out that there are no rules right after posting the question ;-)

Just a thought - we don't test against a route defined as a regex, which I imagine could be wired up differently internally in express.

added a testcase for regex defined routes, weird thing is i never get a positive test run for this one it('/foo/:param/bar → "GET_foo_param_bar"' it always a timeout,

I have a hunch that req.path matches the materialized paths (i.e. what came in, not what the route is called inside express).
Could you also try cherry-picking https://github.com/msiebuhr/node-statsd-client/commit/f33029844980494fce8f5e812e3c4f3c4f71dc92 and see if that also fail?
Perhaps extending https://github.com/msiebuhr/node-statsd-client/blob/master/lib/helpers/getExpressMiddleware.js#L4 to accept arrays (or however else express does it internally):
function sanitize(name) {
    var n = Array.isArray(name) ? name.map(sanitize).join("_") : name
    return n.replace(...)...;

}
Or, if possible, pick just the matching path. That way it could semantically show up as a number of different paths.

I've updated the PR. The goal of this is to enable route definitions via array. We have three types of routes ("literal", "parameterised", "regex") all of them can occur in an array.

[x] support literal routes in array
[ ] support parameterised routes in array (its not ideal yet: https://github.com/msiebuhr/node-statsd-client/pull/72/files#diff-9c9124fb66323f829b84dfd0bd727237R21)
[x] support regex routes in array

Bonus:

[ ] also added tests for regex routes

I expected the test for per regex defined routes to fail, have the same result locally as well. It would be great if one of you has a look at this :/ I've tried the new routes individually in a standalone express app and they all work and return 200s for me. Any idea what I could have missed?


Remove panicking/None Font default impls

E.g. Font::font_data added to make then addition non-breaking to external Font impls.

Probably also applies to Font::glyph_svg_image


Wrong links for GB Cartridge

The 4 links to flipc.org for the "gb cartridge" leads all to "gb link-camera"
The erroneous links are here, all4 variants leads to "gb link camera"
https://github.com/EstebanFuentealba/MALVEKE-Flipper-Zero/tree/main/flipper_companion_apps/applications/external/malveke_gb_cartridge
and the real fap has some kind of build error
https://flipc.org/EstebanFuentealba/MALVEKE-Flipper-Zero?branch=main&root=flipper_companion_apps%2Fapplications%2Fexternal%2Fmalveke_gb_cartridge

Thank you for reporting the issue! I've just fixed it.
Best regards!


Long press support

First off, this is a fantastic integration - easy to configure and incredibly useful.
It would be nice if we could trigger a separate action on a long-press. For example, I have buttons where each press cycles through a series of states; a long press could move backwards through the cycle instead of forwards.

Great suggestion!
This has been requested in https://github.com/basnijholt/home-assistant-streamdeck-yaml/issues/93 too 😄
So I will close this one.


Change LDAP Library and Improve LDAP Handling

This PR substitutes #1231 and is related to issue #1228
It changes the LDAP library to go-ldap, and improves LDAP handling, mainly with configurable Dial Timeout.
Also this makes possible the evolution of Harbor with LDAP.

@reasonerjt @yhua123 Again sorry for the mess.

Coverage decreased (-3.5%) to 47.738% when pulling 86146dcc1e5edb14bc3e2beee8d0359d270aee4b on rikatz:dev into 64ae107fe50345c0e19b48e628055daaf5052b99 on vmware:dev.

Coverage decreased (-0.3%) to 50.963% when pulling 0055e9b08127e7bf0422fdb0e23d1651498dff99 on rikatz:dev into 64ae107fe50345c0e19b48e628055daaf5052b99 on vmware:dev.

@rikatz:
Thanks for your hard work~~~
I have a suggestion:
I saw you help correct some typo and error for document, can you separate your PR to 2 parts, one for document update and one for update LDAP library?
The document PR can merge quickly.
Update LDAP PR, I will use your dev repo testing first and then merge to harbor dev. Maybe need sometime.
sorry for trouble your busy week :)

@yhua123 No problems. Pretty busy this week here on the work, but will try to fix this at home!

Coverage decreased (-0.2%) to 51.012% when pulling 5bc1d0cea9e4a6b1c779a4662e41e66dc45791c3 on rikatz:dev into 64ae107fe50345c0e19b48e628055daaf5052b99 on vmware:dev.

Coverage decreased (-3.4%) to 47.765% when pulling 2f80ee1916072a562cd20d8c902fc245d77609e2 on rikatz:dev into 64ae107fe50345c0e19b48e628055daaf5052b99 on vmware:dev.

Coverage decreased (-0.2%) to 50.998% when pulling 2f80ee1916072a562cd20d8c902fc245d77609e2 on rikatz:dev into 64ae107fe50345c0e19b48e628055daaf5052b99 on vmware:dev.

@yhua123 LGTM.
About the license issue, don't think it's going to be a problem anymore, right?
About the docs, the thing is that I've run lint and gofmt against all the source code, and then all files where rewritten. Anyway, I can undo this, and do on separate commits (maybe separate branches).
What's the best approach to do this? Close this PR again and open other two?
Best rgds.

@rikatz:
Yes, no license issue now.
Better divide into two parts, thank you~

@yhua123 Back from holydays. Will check this tomorrow, just to make this easier, may I:

close this PR
Create two new branches, one with the changes of LDAP Library and make PR for this one, and other to the Docs improvements?

Or is there another bether way to do this?
Thanks

It's up to you. thanks~

Closing this, and opening a new one, better organized :smile:


Unable to connect AWS MongoDb

MongoDB \ Driver \ Exception \ ConnectionTimeoutException (13053)
No suitable servers found (serverSelectionTryOnce set): [TLS handshake failed: error:1408F10B:SSL routines:ssl3_get_record:wrong version number calling ismaster on 'reachalldbinstance.cxwewiihbz0x.us-east-2.rds.amazonaws.com:3306']
'mongodb' => [
'driver' => 'mongodb',
'host' => env('DB_HOST', 'localhost'),
'port' => env('DB_PORT', 27017),
'database' => env('DB_DATABASE'),
'username' => env('DB_USERNAME'),
'password' => env('DB_PASSWORD'),
'options' => [
'ssl' => true,
],
'driver_options' => stream_context_create([
'context' => [
'ssl' => [
'local_cert' => base_path('rds-combined-ca-bundle.pem'),
'allow_self_signed' => true,
'verify_peer' => true,
'verify_peer_name' => true,
'verify_expiry' => true,
]
]
])
],

This error usually occurs if you attempt to connect to something that isn't using SSL/TLS. Check this openssl#6805

Hi! 👋🏻 I run into this issue - silly mistake. The config/database.php file calls 'port'     => env('DB_PORT', 27017), but, in my case, .env is setting DB_PORT=3306 and should be DB_PORT=27017.


google.auth.exceptions.RefreshError: Internal Failure. Random exception with a few concurrent authentications with service account in json file

I'm running a couple of concurrent Google Speech requests and quite frequently an exception occurs with the following traceback:
ERROR:root:AuthMetadataPluginCallback "<google.auth.transport.grpc.AuthMetadataPlugin object at 0x7f5ce8fd20f0>" raised exception!
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/grpc/_plugin_wrapping.py", line 77, in __call__
    callback_state, callback))
  File "/usr/lib/python3.6/site-packages/google/auth/transport/grpc.py", line 77, in __call__
    callback(self._get_authorization_headers(context), None)
  File "/usr/lib/python3.6/site-packages/google/auth/transport/grpc.py", line 65, in _get_authorization_headers
    headers)
  File "/usr/lib/python3.6/site-packages/google/auth/credentials.py", line 121, in before_request
    self.refresh(request)
  File "/usr/lib/python3.6/site-packages/google/oauth2/service_account.py", line 322, in refresh
    request, self._token_uri, assertion)
  File "/usr/lib/python3.6/site-packages/google/oauth2/_client.py", line 145, in jwt_grant
    response_data = _token_endpoint_request(request, token_uri, body)
  File "/usr/lib/python3.6/site-packages/google/oauth2/_client.py", line 111, in _token_endpoint_request
    _handle_error_response(response_body)
  File "/usr/lib/python3.6/site-packages/google/oauth2/_client.py", line 61, in _handle_error_response
    error_details, response_body)
google.auth.exceptions.RefreshError: ('internal_failure: None', '{\n  "error" : "internal_failure"\n}')
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/google/api_core/grpc_helpers.py", line 54, in error_remapped_callable
    return callable_(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/grpc/_channel.py", line 341, in _next
    raise self
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with (StatusCode.UNAVAILABLE, Getting metadata from plugin failed with error: ('internal_failure: None', '{\n  "error" : "internal_failure"\n}'))>

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/var/lib/asterisk/agi-bin/agi_devel/gasr.py", line 112, in <module>
    for response in responses:
  File "/usr/lib/python3.6/site-packages/grpc/_channel.py", line 347, in __next__
    return self._next()
  File "/usr/lib/python3.6/site-packages/google/api_core/grpc_helpers.py", line 56, in error_remapped_callable
    six.raise_from(exceptions.from_grpc_error(exc), exc)
  File "<string>", line 3, in raise_from
google.api_core.exceptions.ServiceUnavailable: 503 Getting metadata from plugin failed with error: ('internal_failure: None', '{\n  "error" : "internal_failure"\n}')

Authentication is handled with a service account and a json file. Here's the relevant code bit, it's not much:
os.environ["GOOGLE_APPLICATION_CREDENTIALS"] = "/home/arch/agi/google_service_account.json"
from google.cloud import speech
speech = speech.SpeechClient()

The issue is somewhat random, but only appears when there are a couple of concurrent recognition requests. It never got it when running one request at a time.

Hrm. We don't go out of our way to ensure thread safety but the worst case scenario is that the credentials get refreshed multiple times within a second.
If you manually refresh these credentials before running your script, what do you see?
import google.auth
import google.auth.transport.requests

credentials, _ = google.auth.default(scopes=['https://www.googleapis.com/auth/cloud-platform'])
credentials.refresh(google.auth.transport.requests.Request())

from google.cloud import speech
speech = speech.SpeechClient(credentials=credentials)

...

I'm closing this due to inactivity. If you can reproduce this with the latest grpcio, let us know and we'll re-open.

I am in the same situation, what do you suggest ?
Traceback (most recent call last):
  File "aadhaar_template_test_suite.py", line 17, in <module>
    id = IdFactory(image)
  File "/Users/mohammadumair/workspace/dolphin-env/dolphin/ekyc/core/base.py", line 144, in IdFactory
    return detect_id(image_content)
  File "/Users/mohammadumair/workspace/dolphin-env/dolphin/ekyc/core/base.py", line 65, in detect_id
    detected_words = detect_words(image)
  File "/Users/mohammadumair/workspace/dolphin-env/dolphin/ekyc/core/utils.py", line 295, in detect_words
    detected_text = str(detect_document_text(image).full_text_annotation.text)
  File "/Users/mohammadumair/workspace/dolphin-env/dolphin/ekyc/core/utils.py", line 281, in detect_document_text
    client = vision.ImageAnnotatorClient(credentials=get_refreshed_credentials())
  File "/Users/mohammadumair/workspace/dolphin-env/dolphin/ekyc/core/utils.py", line 275, in get_refreshed_credentials
    credentials.refresh(google.auth.transport.requests.Request())
  File "/Users/mohammadumair/workspace/dolphin-env/lib/python3.6/site-packages/google/oauth2/service_account.py", line 322, in refresh
    request, self._token_uri, assertion)
  File "/Users/mohammadumair/workspace/dolphin-env/lib/python3.6/site-packages/google/oauth2/_client.py", line 145, in jwt_grant
    response_data = _token_endpoint_request(request, token_uri, body)
  File "/Users/mohammadumair/workspace/dolphin-env/lib/python3.6/site-packages/google/oauth2/_client.py", line 106, in _token_endpoint_request
    method='POST', url=token_uri, headers=headers, body=body)
  File "/Users/mohammadumair/workspace/dolphin-env/lib/python3.6/site-packages/google/auth/transport/requests.py", line 124, in __call__
    six.raise_from(new_exc, caught_exc)
  File "<string>", line 3, in raise_from
google.auth.exceptions.TransportError: HTTPSConnectionPool(host='accounts.google.com', port=443): Max retries exceeded with url: /o/oauth2/token (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x1102a9be0>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))

Thanks theacodes,
The manually refresh thing worked for me.


[Feature Request] service-endpoint create support to read from stdin

Creating a service connection to an arbitrary service (such as Docker Hub) with:
az devops service-endpoint create --service-endpoint-configuration foo.json

requires the use of --service-endpoint-configuration to read from a JSON-encoded file containing the POST request to be sent. But in a CI environment this will typically require first creating a (hopefully temporary) file with cleartext credentials which is not desirable. On UNIX system you can work around this with:
sed -e 's/MY_PASSWORD_TOKEN/MY_ACTUAL_PASSWORD/' foo.json | az devops service end-point create --service-endpoint-configuration /dev/stdin

but I'm not sure how you would do this on Windows? It would be useful if there was an alternate parameter named something like --service-endpoint-configuration-from-stdin that would explicitly support reading the required JSON snippet from stdin in a cross platform way.

Ah I created #1044 and only after, I saw that thsi existed.
The standard way to enable file reading from stdin is to use '-' as a filename.
So: sed -e 's/MY_PASSWORD_TOKEN/MY_ACTUAL_PASSWORD/' foo.json | az devops service end-point create --service-endpoint-configuration -


peer DIDs marshalled with empty authentication

Test case:
ariesCtx := createAriesContext()...
peerDID, err := ariesCtx.VDRIRegistry().Create(
	"peer",
	vdri.WithServiceEndpoint("http://example.com/didcomm"),
)
bits, err := peerDID.JSONBytes()
fmt.Printf("%s", bits)

Output looks like this:
{
    "@context": [
        "https://w3id.org/did/v1"
    ],
    "authentication": [
        ""
    ],
    "created": "2020-07-24T17:08:31.370411561-04:00",
    "id": "did:peer:1zQmXJ88C99EXHjkpBEfaxKFeN9qnGadEZfQWs8SSH6rbVwy",
    "publicKey": [
        {
            "controller": "#id",
            "id": "",
            "publicKeyBase58": "FMAHL95Quyt1WofUmKqWPYvjw3tnXKJ9MtWAmYfCg3M2",
            "type": "Ed25519VerificationKey2018"
        }
    ],
    "service": [
        {
            "id": "#agent",
            "priority": 0,
            "recipientKeys": [
                "FMAHL95Quyt1WofUmKqWPYvjw3tnXKJ9MtWAmYfCg3M2"
            ],
            "routingKeys": null,
            "serviceEndpoint": "http://example.com/didcomm",
            "type": "did-communication"
        }
    ],
    "updated": "2020-07-24T17:08:31.370411561-04:00"
}

unable to reproduce now...


handleActions doesn't work...

Hi there,
I'm stuck on using handleActions.
This snippet works well.
export const reducer = handleActions({
  [getPerson]: (state, action) => {
     const newState = Object.assign({}, state, {
      person : {
        name: action.payload.name,
        age: [action.payload.age,
      }
    })
   return newState;
  },
 initialState);

But this one doesn't work.
export const reducer = handleActions({
  [getPerson]: (state, action) => {
     const newState = Object.assign({}, state);
     newState.person.name = action.payload.name;
     newState.person.age = action.payload.age;
   return newState;
  },
 initialState);

And this works...
export const reducer = handleActions({
  [getPerson]: (state, action) => {
     const newState = Object.assign({}, state);
     const person = {
       name: action.payload.name,
       age: action.payload.age
     }
     newState.person = person;

   return newState;
  },
 initialState);

Can anybody tell me what is wrong with the second one ???
Please please help me out.
I've been wondering for 4 hours !!! :(
Here is initialState
const initialState = {
  person: {
    name: null,
    age: null
  }
}

Thanks

Second variant mutate your state.
Object.assign doesn't merge object deeply:
const state = {person: {name: null}};
const newState = Object.assign({}, state);
console.log(state.person === newState.person); // true

Don't mutate your data in react-redux apps. And use object destructuring, it much more convenient than Object.assign. Your reducer would look something like that:
export const reducer = handleActions({
  [getPerson]: (state, {payload: {person}}) => ({
     ...state,
     person,
  })
}, initialState);

+1