Assertions triggering during moves coalescing

https://github.com/bytecodealliance/wasmtime/pull/1586 introduces explicit moves from SP (not allocatable) to other registers. However, this triggers an assertion failure in the backtracking allocator, during coalescing:
assert!(iix_bounds.uses_len == 1); (backtracking.rs:572)
I checked and the value was 0, instead of the expected 1. I don't think the change in the PR above is_move ought to be correct, so it'd be nice to see if there's something else to do to fix this.
It might be a bug in Cranelift itself, which might be feeding bad data to regalloc.
cc @julian-seward1

Having looked at this .. I think the assertion is wrong.  It should be <= 1 rather than == 1.  That would make it consistent with the assertion for defs_len on the next line.  The overall effect would be to ignore coalescing for moves in which either the source or dest regs have been "sanitised away".
The reason this happened is kind-of slackness on my part.  Until this bug, I had only ever seen
mov sanitised_reg, normal_reg
hence the assertions worked.  However, your example provides
mov normal_reg, sanitised_reg (where sanitised_reg = SP)
Which the proposed assertion change, I think we should be good to go.  However, I can't easily test it.  Can you?

Fixed, 091ca7e5fedb823ba370b930d64677f6cc43afd7.  Thanks for the bug and for testing the fix.


Instrument Type: Cryostat

Right now the Instrument Type "Cryostat" supports methods related to:

Temperature Controller
Magnets
Rotator
Liquid Level

Question/Proposal:

Should these be separate instrument types?
Should cryostat somehow aggregate these distinct instrument types?

This is straightforward except for PPMS systems...

The problem statement is summarized here:
https://boringengineer.com/2020/05/03/labview-g-interfaces-solving-a-decade-old-problem/
Perhaps when we can move to LV2020+ I will address this issue. Unless there is a way to support PPMS with multiple types (Temperature, Magnet, etc)


Make Memory Action

The one line memory is not being saved by the bot.  And if you request the bot to save the one line memory, it assumes that it needs to be incorporated to the recent memory that was saved.

Honestly, not even having this problem half the time, now Scriz wants to save everything - I'm happy to have a session to go over some of the possibilities here

@sheikita While this one has been half-handled with the copy/paste button, I'll leave it open for now, as we could get back to offering options to the member giving direction on when memories are preferred to be "made"

Make Memory
PRINT Memory Functionality
Needing some sort of "That's all I have to say about that" protocol into the interface requires something in addition to, or a coopting of the send button - I would want to provide an alternative hard return button that indicates you are finished discussing that memory.
It has not been effective to leave this to the LLM via instructions, nor would I ever expect it to compete with member clarity (even if a keyword, such as "done" or "print"-- I do like print...), so...
I'm proposing to utilize the ACTIVE MEMORY BAR to
Note: The frontend and backend concerns retain independence. If we choose to alter the interface, there is likely no backend impact, so long as the ## PRINT protocol is followed. I have altered the name of the function from "Make Memory" to PRINT MEMORY as the double entendre is desirable for a companionable frontend, backend confusion is widely undesirable, and print a memory tends to work even without being directly addressed in LLM instructions
Todo

[ ] frontend

[ ] co-opt the "active" bar as follows:

[ ] defaults to: Make a Memory (which can have a GO button on it to force create, as per bubble-tab?) when biographer is selected, disappears on other bots
[ ] dropdown/scrollable ticker,
[ ] dropdown: always show 'Make A Memory' but below that, selectable titles to activate
[ ] scroll buttons (+arrow keys) can tick through memories to activate

[ ] if MAKE MEMORY active, should, upon receipt, by default (or chosen in biog options) to open the summary for more prompting, now, prompting can be based on the content/summary review itself (since last make)

[ ] backend

[ ] instructions: explain in both avatar and biographer functionality and startup
[ ] 'mylife-avatar.mjs'

[ ] assign and capture button press (i.e., will be received as ## PRINT, may want to perform internal MyLife triggers inside of Conversation or elsewhere)

[ ] change unique summary function to itemSummary()


Feature Request: Time "left" ("to go", remaining, etc)

When a negative difference is encountered, we only see "just now", can we please instead get "3 minutes left" or similar?

The jquery Version http://timeago.yarp.com/ allows future dates, would be great to have vue-timeago supporting this as well.

This would be really nice to have. The current behavior is almost a bug as future dates are displayed as "just now".

It's now supported in vue-timeago v4 which uses date-fns.


[BULK] - DocuTune remediation - Sensitive terms with GUIDs (part 16)

Applying sensitive terms with GUID changes as part of Content SFI and outlined in Overview - Writing content securely - Platform Manual. Changes are part of the Microsoft-wide SFI effort. Point of contact: @CelesteDG
DocuTune v<IP_ADDRESS>
CorrelationId: ac15aa43-4e2b-437f-ab1c-fdd7e79cd4db
#docutune

#docutune-review

LGTM


[Dropdown] Usage of the onLabelCreate($label) callback

According to the document it seems a label can be modified before it get added (in a allowAdditions = true context).
What I want is to lowercase the label before it get added. But I have no clue how to modify the being-added label since the callback function only get value and text arguments which are both passed by value. Probably add an example to the document?
Many thanks!

onLabelCreate receives the $label element which is expected to be returned.
To lowercase you could use something like return $label.html( $label.html().toLowerCase() );

It seems onLabelCreate receives value and text, and then returns $(this) (source). Am I got anything wrong?
Many thanks again.

You can modify data-value parameter to modify its value, relevant underlying bits


STAR: Update freshness check by district

Doing some clean up on this monitoring, related to https://github.com/studentinsights/studentinsights/pull/2768.  The mechanics of these syncs are different across districts, and so as a result so are some of the update cycles.

selfie


Validation

What is added New
Add the

[ ] following validations:

For the User model:

Name must not be blank.
PostsCounter must be an integer greater than or equal to zero.

For the Post model:

Title must not be blank.

Title must not exceed 250 characters.

CommentsCounter must be an integer greater than or equal to zero.

LikesCounter must be an integer greater than or equal to zero.

[ ] Add unit specs for all of your models' methods and validations.

Hello @MMhaidari 👋,
This is an additional review as a result of the second opinion request.
I can confirm that there are failing tests, and the reason behind this is the absence of the 'user_id' column in the comments and likes tables. According to the schema.rb file, there is only an 'author_id' column in comments and likes tables. Let's focus on resolving these issues.

Keep rocking 🚀!


fix: partial export for git connected apps

Description
Fix partial export for git connected apps.
PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/28997
Type of change

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

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

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

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/build-deploy-preview


Package conflicts on Ubuntu 14.04

The following packages contain conflicting files, preventing the installation of python3-rosinstall
(Reading database ... 64690 files and directories currently installed.)
Preparing to unpack .../python3-catkin-pkg_0.2.6-1_all.deb ...
Unpacking python3-catkin-pkg (0.2.6-1) ...
dpkg: error processing archive /var/cache/apt/archives/python3-catkin-pkg_0.2.6-1_all.deb (--unpack):
 trying to overwrite '/usr/bin/catkin_test_changelog', which is also in package python-catkin-pkg 0.2.6-1
Preparing to unpack .../python3-rospkg_1.0.34-1_all.deb ...
Unpacking python3-rospkg (1.0.34-1) ...
dpkg: error processing archive /var/cache/apt/archives/python3-rospkg_1.0.34-1_all.deb (--unpack):
 trying to overwrite '/usr/bin/rosversion', which is also in package python-rospkg 1.0.34-1
Preparing to unpack .../python3-rosdistro_0.4.0-1_all.deb ...
Unpacking python3-rosdistro (0.4.0-1) ...
dpkg: error processing archive /var/cache/apt/archives/python3-rosdistro_0.4.0-1_all.deb (--unpack):
 trying to overwrite '/usr/bin/rosdistro_reformat', which is also in package python-rosdistro 0.4.0-1
Errors were encountered while processing:
 /var/cache/apt/archives/python3-catkin-pkg_0.2.6-1_all.deb
 /var/cache/apt/archives/python3-rospkg_1.0.34-1_all.deb
 /var/cache/apt/archives/python3-rosdistro_0.4.0-1_all.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

The Python 2 and Python 3 ROS packages are known to be not side-by-side installable. This has been reported for specific packages before (e.g. https://github.com/ros-infrastructure/rospkg/issues/71). There is not obvious way to fix this (since some packages provide global binaries). Therefore the best approach is to keep the Python 3 related stuff in a virtual env.
We also can't express the conflict in the Debian control file until stdeb (which we use for releasing the Python packages) merges and releases our pull request to add that feature: https://github.com/astraw/stdeb/pull/88

Should we close this issue in favor of re-opening #71 until the upstream PR is merged and released?

As soon as stdeb supports the feature we will add the conflict entries to all Python packages. But this won't make them side-by-side installable. They will still conflict with each other - just in a more graceful way.

stdeb 0.8.4 has just been released (https://github.com/astraw/stdeb/commit/c1c4eac6a28cdd6d37c64e298acdb30bb73572c4). We should release all Python packages (which have global binaries) with their conflicts listed in the stdeb.cfg file:

[ ] bloom @wjwwood
[x] catkin_pkg @dirk-thomas 0.2.7 (https://github.com/ros-infrastructure/catkin_pkg/commit/d82efa91b7da077396db03b3197a0ccbf779d7ce)
[ ] catkin_tools @wjwwood
[ ] rosdep @wjwwood
[x] rosdistro @dirk-thomas 0.4.1
[ ] rosinstall @wjwwood
[ ] rosinstall_generator @wjwwood
[x] rospkg @dirk-thomas 1.0.35
[ ] vcstool @dirk-thomas
[ ] vcstools @wjwwood
[ ] wstool @wjwwood

Please make sure to update your local environment before releasing new Python packages: https://github.com/ros-infrastructure/ros_release_python/commit/7c4c3c7dd344b069b3250e470c1804dee84a47b0

Does vcstools need this? It doesn't install any scripts.

If it doesn't it does not need to declare them. It should be perfectly possible to install the Python 2 and 3 packages of it side-by-side then.

I'll try it.

I'm able to install and use it in both Python and Python3 at the same time.

Ok, my packages are done. I'm about to sync to public and tryout the installations on the new versions now.

Ok, I was able to install all of the new versions from pip and apt-get.

all packages checked off, except vcstool which is parallel installable. They correctly conflict now.

I am a bit puzzled. You were able to install both vcstool Debian packages at the same time?
It does not work for me and I wouldn't expect it to because both try to install the global vcs script.

Yeah, it definitely conflicts. I must have done vcstools by accident instead.

Released vcstool 0.1.8.


How to pu a Recaptcha widget between Flutter widgets?

I've checked almost any other issue here,
What I've understand is that we need a html file working in a API/Website, and then we need to redirect user to that View by using WebView widget than the google will handle the visual part.
My question is that, what if we want that visual part with in a page,
For example we have a login page, an username input, a password input, and then a submit button.
When user fails to enter password right for at least 2 times I want to check that if user is a human by recaptcha.
A Webview should apper between  the password and submit button (or at the bottom)  to show user that they have to submit recaptcha.
The problem is if I basicly put a Weview there, I dont know how much height I should give it. And will it expand when the bigger modal shown. Will it be responsive to other screen aspect ratios?
If I use a Stack Widget, that contains all other widgets in a Column, and captcha as other child, I can maybe simulate the height requested by the Recaptcha on the other hand and the modal pop-up will be shown normally.
Anyone tried that ? How to put a recaptcha in between a FlutterUI design? Basicly I dont want to put a HTML page in my app.

+1


Convert Paise to Rupees

Description
Input  ( Ps ) : 100
Output ( Rs ) : 1

Program added successfully :tada:
Thanks for your contribution :hugs:


adding a logger

currently we are using fmt to log errors etc.. we might consider to use a logger for ex: zap.

Resolved.

Resolved.


[YouTube] Can't download region restricted videos

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
This video is probably only available in Japan.
But I'm trying to download from Japan.
Is there a way to download it?
Verbose log
[debug] Command-line config: ['-vU', '-f', 'b', 'https://www.youtube.com/watch?v=jwCz9KmGuYY']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2022.04.08 [7884ade65] (zip)
[debug] Python version 3.8.10 (CPython 64bit) - Linux-5.4.0-94-generic-x86_64-with-glibc2.29
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: certifi, secretstorage, sqlite
[debug] Proxy map: {}
Latest version: 2022.04.08, Current version: 2022.04.08
yt-dlp is up to date (2022.04.08)
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=jwCz9KmGuYY
[youtube] jwCz9KmGuYY: Downloading webpage
[youtube] jwCz9KmGuYY: Downloading android player API JSON
ERROR: [youtube] jwCz9KmGuYY: Video unavailable. The uploader has not made this video available in your country
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 641, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/youtube.py", line 3377, in _real_extract
    self.raise_no_formats(reason, expected=True)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1161, in raise_no_formats
    raise ExtractorError(msg, expected=expected, video_id=video_id)

Use proxy or VPN.

In general, yt-dlp cannot download videos that you cannot access in browser

Thank you for your reply.
I live in Japan so I can access it with a browser.
I can watch this video normally.

Are you using the same IP for yt-dlp as you are in browser? Confirm that you can access the video in an incognito tab without login

--source-address

What IP you used?

--source-address was irrelevant.
It was my mistake.


add upgrade deletion previous role/rolebinding support

Issue: https://github.com/open-cluster-management/backlog/issues/10003
Updates:

added logic to handle leftover role/rolebindings during acm upgrade

Tests:

[x] unit test
[ ] fresh install still works
[ ] when upgrade ACM, old role/rolebindings are deleted
[ ] after upgrade, still works

/test all

/test all

/lgtm


Object.entries is not defined

Latest build is broken for me due to Object.entries
https://github.com/michaelcontento/redux-storage/blob/master/src/reducer.js#L23
Does babel support this? Could just use lodash pair

Thank you for reporting this!
Fixed with lodash.pairs and v1.3.2 has been published to npm :)


Add blacklist function

Thanks @cxmplex, this has been included in https://github.com/inference-labs-inc/omron-subnet/releases/tag/0.1.3


Directories not escaped in ci/build_libs.sh results in failure when path contains spaces

The LIBDAISY_DIR and DAISYSP_DIR usages in cd trigger error "cd: too many arguments" when the values contain spaces (e.g. I have the files under a lop level "Google Drive" fold that is everything I have cloud sync set up for). Putting quotes around the usages seems to do the trick,

Good to know! We'll look into fixing this!

Good to know! We'll look into fixing this!


zipl: On Alpine Linux

Hello,
I am trying to build s390-tools and run zipl on musl-libc-based Alpine Linux. The compilation is good, with some patches (https://github.com/tmh1999/wip/tree/master/s390-tools). Running zipl sometimes succeeds, sometimes fails. The environment a KVM vm, boot with a kernel, initramfs, and an ext4 filesystem containing the rootfs.
I narrow down the bug to this line : https://github.com/ibm-s390-tools/s390-tools/blob/master/zipl/src/boot.c#L106
Sometimes it shows Arithmetic Exception SIGFPE, sometimes Illegal Instruction.
I pasted build log, dump output at : https://gist.github.com/tmh1999.
Also, I have to stick atm with version 1.37.0 since libutil in later releases started using GNU extension argz. But for zipl, it doesn't mean much since there isn't any significant changes. We also have to use -no-pie.
Would you mind having a look? I have a publicly reachable Alpine s390x instance here if you need.
Thanks,

@stefan-haberland Could you please have a look please?

Sure, I will have a look but this will take some time.

This is my bad.
It turns out zipl does require other programs (which one? zdev ? ) in s390-tools in order to function properly at run time. I just built zipl only by modifying TOOL_DIRS in Makefile.
Build-time deps only require libiraries in lib*. There is no run-time deps as far as I know. Maybe I miss the documentation somewhere ?
Sorry for the fuss. I'm closing this issue.


[CI] Add pre-commit workflow, pre-commit hooks for opentofu, and check-yaml

This PR adds pre-commit hooks for opentofu and check-yaml as well as a GitHub Action to run the pre-commit hooks for every pull request.
For opentofu, currently only tofu_fmt is enabled, but I propose that we also add tofu_tflint. Adding tofu_tflint would require developers to install tflint locally on their machines.

Ok let's use tofu_tflint.  Applying it on this one might cause issues on your WIP PR. Enable/apply on whatever PR that is most convenient.


Inline menu refactor

Refactor inline menu to use react portal. Moving it to portal will also enable using prosemirror state from context, do required refactor to use context to get this data.

Related to https://github.com/tinacms/tinacms/issues/862


Upgraded go toolchain in our product from go 1.18.9 to go1.21.6 and ran the image in the target. We use yocto build system to compile our product modules. go1.21.6 is built from source.
What did you see happen?
All go programs failed to start with index out of range in one of the sha1, md5 and flag packages.
panic: runtime error: index out of range [3] with length 0
goroutine 1 [running]:
crypto.RegisterHash(...)
/usr/lib/go/src/crypto/crypto.go:149
crypto/sha1.init.0()
/usr/lib/go/src/crypto/sha1/sha1.go:19 +0x65
panic: runtime error: index out of range [0] with length 0
goroutine 1 [running]:
flag.init()
/usr/lib/go/src/flag/flag.go:1199 +0xf9
What did you expect to see?
Go programs to start normally.

Strange. This looks like somehow your program is not being initialized correctly (or at all).
I suspect a corrupt installation. Maybe you installed 1.21.6 overtop 1.18.9? Nevertheless, try blowing away your 1.21.6 and reinstalling fresh.

Thanks for your response @randall77 .
We tried 1.20.13 and that works fine and all go programs in our product came up fine. When we tried 1.21.0, it failed with the above mentioned error.
Is there a way to verify the installation after the toolchain is upgraded? The go env seems ok. Any other way to check?

Build a hello world that prints os.Args[0] and see if that works. That's one of the expressions your program crashed on that should never fail.

@randall77  Thanks for your suggestion. Tried the same and the programs seems to run fine.
./hello-world
Hello World printing args[0]
1 [./hello-world]

Try adding import _ "flag" to your hello world program?
Frankly, I'm at a loss for what this could be. Clearly something is getting screwed up very early in initialization.
Can you share a program on which this occurs?

 GOROOT, CGO_CFLAGS, CGO_LDFLAGS, etc.
Might also try go clean -cache.

After importing flag package, program did not start with the same error.
No. We are not using go program from C library.
Unsetting env variables and cleaning cache did not help either.

So this program
package main

import (
	_ "flag"
	"fmt"
)

func main() {
	fmt.Println("hello world")
}

Builds ok but crashes when you try to run it?

That's correct.
And FYI - we used the yocto patches available at<EMAIL_ADDRESS>

Try without those patches.

We have been able to build the go 1.21.6 toolchain in yocto build system and the programs are running fine on the target after using the patches provided by yocto contributions here https://patchwork.yoctoproject.org/project/oe-core/list/?series=22831.

I'm going to close this as I don't think there's anything here for the Go project to do.


Try (again) to make GitHub's Linguist .gitattributes work

Our present Linguist configuration in .gitattributes doesn't seem to be having any effect, as can be seen when viewing this commit: https://github.com/gleam-lang/tree-sitter-gleam/commit/7159ce961592192b0e7cdf88782cda0fdf41a4cb.
Historically we've tried the style employed by tree-sitter-elixir (here) to no avail. This new attempt combines usage of the shorthand from tree-sitter-rust with the no-leading-slash style from Linguist's own documentation. Hopefully it works this time :crossed_fingers:

Ah the

As an added bonus, unlike vendored and documentation files, these files are suppressed in diffs

is the difference for generated, that makes sense! I think it makes sense to prefer linguist-generated then so it will hide those changes in PR diffs.


SARIF docs are not rendered

I'm very happy to see SARIF was added to bandit recently, thank you :)
I'm not sure why but the docs at https://bandit.readthedocs.io/en/latest/formatters/sarif.html are not showing the content for the SARIF formatter though it looks to me like they were defined in https://github.com/PyCQA/bandit/blob/main/bandit/formatters/sarif.py

Thank you.


Is it possible to add agent pools at a later point of time?

Hello,
I would like to know how we can add agent pools to an existing cluster DC/OS created using acs-engine.
Thanks,
Sasidhar.

@skasturi yes, this is supported.  Using the apimodel.json from the output folder, add the extra agent pool and redeploy the template.  Ensure you are using the same ACS-Engine you created the original cluster with, otherwise you will have to trim out the other VM resources.


Error Capture Page Screenshot' could not be run on failure

[ WARN ] Keyword 'Capture Page Screenshot' could not be run on failure: Multiple keywords with name 'Capture Page Screenshot' found. Give the full name of the keyword you want to use:
AppiumLibrary.Capture Page Screenshot
Selenium2Library.Capture Page Screenshot
I have got the issue while run the pybot demo_8_0_full_workflow.robot for odoo automation test case purpose

You have two libraries with same keyword in use. Like the error message says, you need to explicitly specify which one you want to use. If this happens in some demo, please report the issue to the demo project directly. General questions are best sent to robotframework-users mailing list, to Slack, or other such forums. This issue tracker is for bug reports and enhancement requests.

How to specify explicitly which library to be used, please give an example

Like it says in the warning message:
# Use 'capture screenshot' from Appium:
AppiumLibrary.Capture Page Screenshot
# Use 'capture screenshot' from Selenium library:
Selenium2Library.Capture Page Screenshot

Like it says in the warning message:
Use 'capture screenshot' from Appium:
AppiumLibrary.Capture Page Screenshot
Use 'capture screenshot' from Selenium library:
Selenium2Library.Capture Page Screenshot

Thank you for the quick response

do this test setup ""Set Library Search Order  AppiumLibrary  SeleniumLibrary""


Mismatch between origin and baseUrl (dev).

hi,
i think fruition stopped working ?
i added email routing to my domain and some days later i notice my domain / notion was showing this error, i think now every website i knew is showing the error , even fruitiosite.com itself , is there a fix ?
i removed all routing and rules and othe r stuff to start fruition tutorial all over again , but the site is now and now i dont know if its me missing some rule or other fruitions requirement , or if ever script i try is with error

its back up again


OAuth 2.0 Google + OAuth 1.0 Yahoo

I tried live demo at: http://authomatic-example.appspot.com/
With Yahoo, I got result as below:
Hi
You are logged in with:
Yahoo
Your email is:
Your ID is:
<Get recent emails - button>
{
    "error": {
        "lang": "en-US",
        "description": "Table social.profile requires https but requested through http"
    }
}

Yes, no display name, no email which are very important for my application.
With Google, I got all necessary information. But I read this issue: https://github.com/peterhudec/authomatic/issues/1. So I need to enable Google+ API to get information like email, name... This API is limited (10k requests/day). I'm using velruse and it doesn't require to enable this API.
I really hope you can resolve these problems. Thanks!
Edit:
Additionally, I have to tick one of options in user private data access of my yahoo project (e.g. Contact), but I just need to retrieve email and name only. If I don't do that, I'll get oauth_problem=consumer_key_rejected response. I think email and name are basic information and don't need to tick any option if no user private data is required to access.

@peterhudec
I get this error when login via Yahoo:
Error: Server Error

The server encountered an error and could not complete your request.
Please try again in 30 seconds.


Do you have any benchmark for those graph-conv layers?

❓ Questions & Help

Do you have any benchmark for those graph-conv layers? (in public open source dataset)

Sure, you can find our complete benchmark suite in benchmark/. Results of those experiments are given in our paper.

@rusty1s Thanks~


CLI Reports not writing out upon completion

CLI appears to run successfully and tells you that the report are written.  But they aren't there.
Two issues:

Was using a save 'config' file that had an override of the output-directory.  Need to fix that when -o isn't set.
The jar files for the driver couldn't be located.  As seen in the log output.  So the process failed to load the hive jdbc driver.

Fixed Runtime Exception that skirted report output.


Example on clearing angucompletes via ng-click

This should show that the clearing as wrongly stated in #225 works with angular 1.4.0

Thank you!


Fix cargo audit

Run cargo update, with one PR to dev , and a second PR to the private repo release-1.4.5 if required

Solved in #2626


'no such table' when generating unit test from artisan

Laravel Version: 6.9.0
PHP Version: 7.3
Database Driver & Version: 5

In new laravel version when we generates a unit test from artisan it set up a different namespace from previous versions which was Tests\TestCase but now it uses PHPUnit\Framework\TestCase
And when i try to use factory()->create() under this namespace ill got the 'no such table' error, Note that when i change it to previous one it works.

See #30879

Just wondering if i unable to use factories inside if unit tests, How to test my models?

extend the skeleton app's base test class


Set mouse spotlight zone fully transparent instead of white

Description of the new feature / enhancement

make the center transpartent like

Scenario when this would be used?
looks better
Supporting information
No response

/dup #15512


Remove thrift field TabletStats.majors

While working on #3592 noticed this thrift field will not longer be used as its was set in the tserver based on compaction activity in the tserver.  The field seems to be used in the monitor.

The field is no longer populated after #3604, but the field is still there and need to be remove from thrift and monitor code.

The following comments are based on #3618
The TabletStats also has a split related field that is no longer used. This could be removed.
Also there is a class in the tablet server module named TabletStatsKeeper. This class does not do much anymore, need to evaluate if its still needed.


When instancing nodes in runtime, nodes with unique names on @onready are null.

Tested versions
Reproducible on 4.2.2-stable.
System information
Windows 11. Godot 4.2.2-stable
Issue description
If you instance a node in runtime that references an unique named node on @onready, the unique named node is always null.
Steps to reproduce

Create a scene Spawner with a script that instantiate SpawnedNode during runtime and add it to the tree as a child.
Add a node as a child of Spawner. Right click and select "% Access as Unique Name".
In the SpawnedNode's script, add the standard @onready for the unique named node.
Run the game.

Minimal reproduction project (MRP)
UniqueNameBugMRP.zip

I tested the project, I can confirm that this problem exists

Scene unique names are scoped to the owner scene. i.e., owner's subtree.
The UniqueNameNode can only be accessed by nodes stored in scene.tscn.
Your onready variable is inside a script attached to the root node of spawned_instance.tscn, so it can only access scene unique names defined spawned_instance.tscn. And there're no nodes named UniqueNameNode there.

Scene unique names are scoped to the owner scene. i.e., owner's subtree.
The UniqueNameNode can only be accessed by nodes stored in scene.tscn.
Your onready variable is inside a script attached to the root node of spawned_instance.tscn, so it can only access scene unique names defined spawned_instance.tscn. And there're no nodes named UniqueNameNode there.

Understood. So there is no way to dynamically instantiate a script that references a node using the % unique name? I don't know how things are handled internally, but that's a kind of big limitation. I believe it would be good to make it possible.
I tried setting the owner of the SpawnedInstance node both before and after calling add_child(), and both to self (Spawner) and get_tree().get_edited_scene_root(), but %UniqueNamedNode was still null.

This limitation is by design, see https://github.com/godotengine/godot-proposals/issues/4096.
get_tree().current_scene.get_node("%UniqueNamedNode") should work.
By the way, hard-coding name/path of a node that is outside the current scene is making an assumption about the structure of that scene. It might be closer to best practice to have spawned_instance expose a set_the_important_node(node: Node) method.

I'd go one further @export var some_node: MyNode and then it can picked in the inspector window (but maybe that's not what you're after?)

My usage is for spawning enemies instances where there are many enemies already in the scene (which would make sense to assume its structure).
Making the spawner script set_important_node(node: Node), would require that node to have reference to the unique named nodes I need, which atm, it doesn't really need to know them.
Adding an exported variable would require me to update them in the inspector for every single enemy that starts in the scene, which is not really viable.
I appreciate all your replies. I think the best approach for now it is just using an enemy manager of some sort to update the "important nodes" refs inside its _ready() method.
Maybe we can change this to a proposal to have instantiated nodes to still have access to unique named nodes? I don't know the implications of this changes, though.

This limitation is by design, see godotengine/godot-proposals#4096.
get_tree().current_scene.get_node("%UniqueNamedNode") should work.
By the way, hard-coding name/path of a node that is outside the current scene is making an assumption about the structure of that scene. It might be closer to best practice to have spawned_instance expose a set_the_important_node(node: Node) method.

I tried using:

get_tree().current_scene.get_node("%UniqueNamedNode") should work.

And I was still getting null ref in the MRP.

My usage is for spawning enemies instances where there are many enemies already in the scene (which would make sense to assume its structure).

If it's just letting enemies know where the player is, then assigning a group to the player will let the enemies be able to get a list of all players instances.
Otherwise yeah, "enemy manager"...

My usage is for spawning enemies instances where there are many enemies already in the scene (which would make sense to assume its structure).

If it's just letting enemies know where the player is, then assigning a group to the player will let the enemies be able to get a list of all players instances.
Otherwise yeah, "enemy manager"...

It is not related to the player. More for navigation and AI. But your group idea is great, thanks for that!

This may be just a case of "wrong tool for the job". Scene unique nodes are intended for scenes working with their own nodes. And the name is meant quite literally, too: They are unique to their scene, not globally, so when your HUD has a unique node named "Score", the game won't break the instant you add a game over message that also has a node named "Score". They can both happily exist next to each other, each with their own unique node. You can also spawn multiple HUDs into the game and their respective unique nodes will still work as expected without them getting confused by name clashes.
As a consequence, however, if your SpawnedNode scene says "give me my unique node 'Child'" and it doesn't have a unique node "Child", then it won't simply return the unique node of a different scene (the Spawner) that happens to use the same name.
So for your use case, you're not actually looking for a "scene unique node" but rather a "node with a global name" that you can find from everywhere in your game. And that's indeed what Godot calls a "node group". They're called "groups" because here multiple nodes with the same (group) name may exist at the same time, so get_nodes_in_group() may return more than one node.
If this kind of coupling is a bit too loose for your taste, there's also the "NodePath parameter" concept mentioned above. Your inner scene can export a parameter and expect the parent scene to put a reference to an external node there when it spawns a new child (that's how an AnimationPlayer is associated with its "root" node, for instance). The idea is that you don't create a hard dependency on the inner structure of another scene but rather have that scene tell you where the node that you need access to is.

Thanks @bs-mwoerner !
I decided to use the group approach, and it worked fine for my case.


DisplayCaseManager raw types

from https://kolmafia.us/threads/reducing-number-of-warnings-raw-types-displaycasemanager.26004/
Now, the discussion over there actually ended with an unfinished talk about adding a test to this class.
The only non-solely-generics-related change of this PR is a change from
move(<List>.toArray(), [...]);

void move(Object[], [...]) {
  <turn the array into a List>
  [...]
}

to
move(<List>, [...]);

void move(List, [...]) {
  [...]
}

I would like to push the fact that this change should be straightforward/simple enough to not warrant a test.
N.B. this fact was not mentioned in the discussion so far.
If the test is still mandated, I would ask for help in adding it, because I don't really know where to start...

I noticed a @SuppressWarnings.  My philosophy at the moment is that we should not do that because we are much more tolerant of PRs that deal with fixing compiler complaints.  If there is a warning we cannot address, perhaps a comment in the code about the warning and why it remains to be suppressed?
I'm not happy with the test situation, or lack there of but since you asked for help I will either get you some or let things pass as is.

I noticed a @SuppressWarnings. My philosophy at the moment is that we should not do that because we are much more tolerant of PRs that deal with fixing compiler complaints. If there is a warning we cannot address, perhaps a comment in the code about the warning and why it remains to be suppressed?
I'm not happy with the test situation, or lack there of but since you asked for help I will either get you some or let things pass as is.

It's because of an array of generics.
I'm generally surprised that the compiler even says anything about it, since there is, AFAIK, literally no way to make it "safe"
new ArrayList[]; // <<< Allowed, but whines about raw types
new ArrayList<Object>[]; // <<< Not allowed

I added a simple test.  You could add to it and make a better one.  Grab the html from a display case with shelves, put it in a file in test\root\data.  Copy and rename itShouldHaveSomeContents(), change it to read your file and then add some assertions to deal with the shelves defined in your file.  For extra credit look at the test report and see if there is something you can do that covers uncovered code.
@heeheehee-kolmafia might look at the test but I think we're almost there.


Deleting its own prefabs at random?

Version 2.12, Unity Version 2020.3.12f1
So I have a new project, and have Mirror, and UMA in it to play with since never used it before
Made a new scene, added the UMA Dynamic Character, and the UMA GLIB Prefabs
then added in some UI stuff for editing chars, and the script to manage the ui controls.
Sometimes when I Play in editor, it just stops working and looking threw everything I noticed sometimes the UMA_GLib prefab just 'vanishes' from the scene. Nothing in the scripts I wrote effect anything for Adding or removing from a scene any components or otherwise, so I am not sure why it just vanishes.
The only added things is the ui controller, and all it does is look for changes on the slider bars, then assigns the dna, and generats.
I have a long UI Controller since I added all the Dna stuff to its own methods, but a example method is
   private void Updated(UMAData data) // Called when the Listenders sees a Change
    {
        dna = Avatar.GetDNA();
        heightSize.value = dna["height"].Get();

}
private void OnEnable()
{
Avatar.CharacterUpdated.AddListener(Updated);
heightSize.onValueChanged.AddListener(HeightChange);
}
    public void HeightChange(float val) // called each time the height slider changes
    {
        dna["height"].Set(val);
        Avatar.BuildCharacter();
    }

Is there a way to lock or prevent the GLIB from being deleted?

This has been fixed in my development branch.
I no longer add a "EditorContext" automatically. Instead, I prompt the user to add either a permanent copy, or a temporary copy that will last until the scene is closed. It's never deleted automatically anymore. This will appear in the next update.


Flyway migration fails on an existing database

To reproduce the issue, run the first script under bootstrap/sql for that database type (this will reflect the production environment for 3.0.1 release), now invoke bootstrap-storage with "migrate" option.

@raju-saravanan Thanks for addressing this issue.


Add UnicodeCodePoint.IsScalarValue

[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static bool IsScalarValue(UnicodeCodePoint codePoint) {
    return IsSupportedCodePoint(codePoint) && !IsSurrogate(codePoint);
}

I think it's a great suggestion. @DerpMcDerp, would you like to submit a PR?

The type UnicodeCodePoint no longer exists, so this is not needed.


Join thread notification should only link to existing threads

After leaving thread, a joined thread notice will attempt to take you to the thread, which no longer 'exists'. In fact, it seems to resurrect the thread, but without a title?

This should go away when new notification stuff lands... notifications related to a deleted thread will also be deleted.


IBC hooks: Allow custom prefix for derive intermediate sender

Background
IBC hooks exposes a function to derive the intermediate sender: https://github.com/osmosis-labs/osmosis/blob/main/x/ibc-hooks/client/cli/query.go#L62. This is very similar to the function used by PFM, the only difference is the prefix used.
Allowing users to customize the prefix would make it easy to predict the PFM addresses using this function. This will be useful in tests (https://github.com/osmosis-labs/osmosis/pull/5954/files#diff-7d1fc1e232b55677582b51e2623aae6d8c15b5f0960ac6100fc7d733cb795dacR1308) and when crafting packets to forward that may depend on the sender (XCSv2)
Suggested Design
Modify the query to take two optional parameters --hash-prefix and bech32-prefix and pass them to a helper function. The basic function used in ibc hooks should still have the hash prefix hardcoded and just use the helper internally
Acceptance Criteria
osmosisd q ibchooks wasmsender should support the two extra params

hey @nicolaslara! is this free to pick up?

yes!

I have one question: is hash-prefix this issue refers to and the one we want to provide as arg is this?

const types.SenderPrefix untyped string = "ibc-wasm-hook-intermediary"

Yeah, exactly


pruned nodes compatibility issue

Ideally forkscanner will be connected to pruned btc nodes, we should have a check which ignores a block if gets pruned status back from btc node for the first get.
right now forkscanner keeps on sending the same get block request again and again, and goes into an infinite loop and gets blocked by the btc node.

I am not seeing this behavior, I do see the forkscanner getting a message about pruned blocks:
JSON-RPC error for getblock: RpcError { code: -1, message: "Block not available (pruned data)", data: None }
but it seems to continue on past this, can you describe steps to reproduce what you're seeing?


Fields missing from json search results needed for bento search

Currently no results are showing for dpul in the bento search. Probably this broke with the blacklight upgrade.
Here is some client code showing previous expected behavior, with attempts to find data in the new format: https://github.com/pulibrary/discoveryutils/pull/201/files#diff-69331d0313a44ff9b652e6ee7f61c7d05c3588ea17d8b44afd02d064b4d63fecR26-R42
However, some of the required fields are not published at all. Currently it looks like only collection and creator are coming through: https://dpul.princeton.edu/catalog.json?utf8=✓&search_field=all_fields&q=music
We at least need title and url.

Our bento search is a primary means of folks finding our content. I'm going to mark this a sudden priority, but recommend we don't tackle it until next week.


Clean for Arrow code

Please can you add a "clean" rule that ultimately gets invoked from the clean rule in cava/Makefile to remove auto-generated files, perhaps by adding them to a .gitignore that is used in a clean rule? Thank you kindly.
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	ArrowExamples.vok
	ArrowExamples.vos
	ArrowPHOAS.vok
	ArrowPHOAS.vos
	ArrowStream.vok
	ArrowStream.vos

Closed via #34


0 temperature in time-series should be presented differently

Now we represent a system in 'off' with a temperature equal to 0. this is convenient as we do not mix text with float or integer in a same vector. However, there is not such a thing as zero temperature in the system so we should think on a better way to represent it in general.

@JIMENOFONSECA, I kind of have to guess what you mean here... So here is what I assume:

you have found a case where a state variable in the simulation is not known and that is currently represented by a zero 0
might have been from an np.zeros(HOURS_IN_YEAR) initialization
you would prefer to have a value outside of the domain of possible values for temperatures (in this case)

Since I am also assuming that the variable is a real of some sorts (float, double, whatever) then the value to use should probably be NaN, "Not A Number".
If you specify where this happened, we can fix it. Do we need to update all the initializers?

NaN i believe would not be an alternative as ArcGIS will fail to read it when we want to represent it in a temporal scale. We will also have trouble on summing up vectors with NAN right?

We could just test that and see. We really need to specify where this is happening. What variables are we talking about? Why do they not have a value? What does not having a value mean? Only then can we actually have a meaningful discussion.

@gabriel-happle @daren-thomas i suggest to close this issue if it is not as important as other issues

@gabriel-happle what do you think?

I still think it's important. we should just come up with a strategy or guideline and start working according to it from now on. But in general I still think we should avoid zero in cases where we actually mean "no data".

@gabriel-happle it has been a year. I suggest we close the issue if there is no strategy and things work across the full stack of tools.


Makes $runsMigrations property be always a boolean

This pull request makes $runsMigrations to be of type bool instead of being bool|null.

Eh, this changes the actual logic of what I was going for.


feat: add custom middlewares for init plugins

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
A new feature for init plugin to add custom middlewares.

What is the current behavior? (You can also link to an open issue here)
Currently there's no easy way to add custom middlewares.

What is the new behavior (if this is a feature change)?
A new function WithCustomMiddlewares.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.

Other information:

Looking into this now!

Overall it looks like a fine idea, a lot of flexibility is added and the only notable downside is the added adherence to gin in initPlugins. I think that's a worthy tradeoff.
@rbeuque74 thoughts ?

LGTM, thanks for the contrib !
Merging now


Updated Default Theme

We made an error in judgement with our 2023 default theme having custom CSS.
The new default theme should be 100% vanilla bootstrap 5, with no custom CSS what-so-ever. This will make it easier for people to build on quickly.
Working on it here: https://demo.eastus.raytha.app/

Will be deployed with v1.1.5


Why is the CWA able to scan vaccination certificates without valid signature?

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[ ] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository - I don't know, someone should check on iOS
[x] Bug is not already reported in another issue

Technical details

Device name: LG-G4 (LGH-815)
Android version: 6.0
App version: 2.3.2

Describe the bug
CWA is now able to store DCC vaccination certificates - that's great, and thanks to everyone involved for accomplishing that work! ❤
As I don't own my personal DCC bar code yet, I tried out to scan an example code found on
https://github.com/Digitaler-Impfnachweis/certification-apis/tree/master/examples which uses this bar code:
https://github.com/Digitaler-Impfnachweis/certification-apis/blob/master/examples/01_example.png
This bar code was added flawlessly to CWA, but it was not possible to add it to CovPass app or to verify it successfully with CovPassCheck app.
CWA's representation of example DCC vacc cert scanned from GH source:

CovPass app's error dialog when scanning example DCC vacc cert scanned from same GH source:

CovPassCheck app's error dialog when scanning example DCC vacc cert scanned from same GH source:

It's obvious, that the example QR code does not have a valid signature issued by an authorized health authority.
While CWA checks the json payload of the DCC for validity afaik, it does not check its signature.
Steps to reproduce the issue

Scan QR code from https://github.com/Digitaler-Impfnachweis/certification-apis/tree/master/examples with CWA
Scan QR code from https://github.com/Digitaler-Impfnachweis/certification-apis/tree/master/examples with CovPass app
Scan QR code from https://github.com/Digitaler-Impfnachweis/certification-apis/tree/master/examples with CovPassCheck app
See differences

Expected behaviour
I would have expected CWA to refuse a DCC that does not contain a valid signature issued by an authorized health authority.
For me it's not clear, if this behaviour is intended, or maybe a bug, or maybe there is already a validation algorithm planned for a future release?
Possible Fix
I would propose to add a signature check in a future release of CWA to prevent

occasionally distracted people (like me) to scan the wrong QR code out of their desk's paper mountain, especially when being stressed while preparing for a holiday, and then later experiencing a bad surprise at border control ;)
users to being able to abuse sample codes when they're used as gate passes in other situations: theoretically, the gate keeper should always see another personal document (like id etc.) to validate the presented code against the holder. Practically, this won't  likely be the case in many situations for future events (like entrance to cinema, stadium, restaurants etc...). While people still may scan other peoples' valid DCCs, it's at least increasing the level for abuse compared to scanning arbitrary codes found in internet into CWA.

I don't know if it might be a good idea to add an explanation to the FAQ about the current issue.
Cheers, v.

Related to topic: Check signature of certificates
Internal Tracking ID: EXPOSUREAPP-8010 Android
Internal Tracking ID: EXPOSUREAPP-8330 iOS

@vaubaehn
We can leave it open for visibility.
I've moved it to cwa-documentation and changed the title to a question. Maybe we can make an FAQ entry with the info, but I don't know if that's necessary.

Corona-Warn-App Open Source Team

I was also able to create own qr-codes that were successfully imported to cwa app resulting in a blue colored vaccination box.
I can understand your argument, but:

CovPassApp IS validating the COSE‘s Signature - and people expect identical behaviour for acceptance if your app
As a user of your app I would like to know wether my vaccination cert is invalid BEFORE the gatekeeper tells me!

@AnTu2702

I was also able to create own qr-codes that were successfully imported to cwa app resulting in a blue colored vaccination box.

Did you by any chance try out the CovPassCheck-App?

@AnTu2702
In general I'm agreeing, but

2\. As a user of your app I would like to know wether my vaccination cert is invalid BEFORE the gatekeeper tells me!

There are two types of invalidity: either the issuer's signature key is/became invalid (afaik some kind of expiration was considered), or the health certificate itself expired (i.e., the vaccination/test/immunity "expired"). For the latter, I hope CWA is validating the json payload against expiration while scanning... did you try that out also?
And that the issuer's key expires will usually be after the health certificate expired, so I guess that won't be a critical point.

There will be a rules available at the EU DCC Gateway. This rules will be applied on the user device locally when the enters specific parameter like county and date.
More information:
https://github.com/ehn-dcc-development/dgc-business-rules

Would be great if that was available also as a list for better overview, not having to query every single country for an extended trip. E.g. show a list with all countries, next column with current status or from what date travel will be allowed/denied.

@treysis

There will be a rules available at the EU DCC Gateway. This rules will be applied on the user device locally when the enters specific parameter like county and date.
More information:
https://github.com/ehn-dcc-development/dgc-business-rules

Would be great if that was available also as a list for better overview, not having to query every single country for an extended trip. E.g. show a list with all countries, next column with current status or from what date travel will be allowed/denied.

It would probably be better to continue this discussion as a separate topic on https://github.com/corona-warn-app/cwa-wishlist since the topic about rules doesn't fit the title of the original subject "Why is the CWA able to scan vaccination certificates without valid signature?" here in https://github.com/corona-warn-app/cwa-documentation/issues/634 and you are asking for new functionality.

There will be a rules available at the EU DCC Gateway. This rules will be applied on the user device locally when the enters specific parameter like county and date.

Work on this on the CWA-Server: https://github.com/corona-warn-app/cwa-server/pull/1446

You will find this list on https://reopen.europa.eu/de

Yeah, it would be great to have this as a table, not having to query every country manually and do manual calculation.

no matter what "EU guidelines" we have,
CWA and CovPass/Check have different concepts and this can lead to different PASS/FAIL for exactly the same QR-code.
the reason(s) for this difference is irrelevant. it happens "in the field" right now.
both concepts should merge/share QR-math/sourcecode.
if this is not possible asap, CWA needs to remove its QR function. with a link to CovPass/Check.
so, instead of having several apps, "trying to do" multiple tasks, it should be one app only, making "one" perfect job.
if "too many criteria" is the problem, then a binary "pass/fail" is not the solution. (but maybe a table).
simplicity (for the user) is not (always) the right answer.
coding and development is not the problem. but bad concepts are often just setup by "bad architects" (or management).
easy statements for me. I know ;)
forgive me for posting "opinions", maybe even in the wrong place

Can you please elaborate where exactly is the issue to scan an invalid or expired certificate except it is different.
You cannot prevent the user to show a screenshot or website which looks like the app.

I understand your argument.
Imagine somebody providing "illegal" QR-codes to a CWA user (no tech skills). Maybe even for money.
"gate-keeping" is still secure. (e.g. CovPass/Check)
but CWA becomes the "fav target" for bad guys.
no "use case"? wired idea?

Illegal QR-Codes? If you scan them with CovPassCheck, they will still turn out as invalid. I don't see your point.

please read my 1st statement again. CWA could be a vulnerable target. this could harm your reputation.
I can cross-check with 3 apps.
normal users will not.
they will be happy having a PASS in CWA.
then they will e.g. travel ... and meet a gate-keeper using more/different/updated criteria = FAIL.
both apps pretend to conduct the same task (QR-code validation).
but with different critera = potential conflict.
I still dont know if this scenario is possible. But I guess it is. due to e.g. the missing signature check of CWA.

I agree I would say that some kinda cert check should be done if possible. if the device is very old the app could tell the user that it loaded the cert but could not verify it.

I'm sorry to say this but the press writes that this is a "schwerwiegende Sicherheitslücke" (https://www.br.de/nachrichten/wirtschaft/impf-zertifikat-in-der-corona-warn-app-maengel).
I recommend to add signature checks, not only to prevent bad press but also to catch up with nearly all other EU wallet apps, which are verifying the signature.
I know, if everybody would use the CovPassCheck App and look at the id card this wouldn't be necessary and I also know that this has to be done according to the guidelines, but it makes no sense to say that the guidelines are like this if nobody else cares about the guidelines.
Yes for sure you could argue that the people should follow the guidelines but they just don't and they won't because they're lazy or just think that it's enough to take a look at the certificate.
Try to explain somebody who's responsible for the guest registration and that the guests are tested, vaccinated or recovered what a signature is and explain that he/she has to check every QR code with an extra app and also has to see the ID document.
Everybody will say that this is not practical.
So we should really adopt to the reality here and add signature checks here.
Have a good night.

 The only problem is that nobody anticipated that the general public has no idea about IT (and even less about encryption), and that press will jump anything in regard to these apps.
Imagine this scenario: you add a signature check. Somebody will then write an article that you can just fool the bouncers by showing a manipulated screenshot and how insecure the whole system is.
I don't know if that would result in less bad press, or just different bad press. So, if there is a lot of "bad press" it might rather raise awareness for the requirement of using CovPassCheck (or whatever the other countries use) and ID checks.
In the end, too much is just clickbait articles and there's little to do against it.

Yes, I know. But the people reading this (my father sent me this article) think that it's true and that the security is really bad.

Somebody will then write an article that you can just fool the bouncers by showing a manipulated screenshot and how insecure the whole system is.

I assume this would result in another bad press, but somehow this is obvious and shouldn't be a scandal.
Your last idea is a good one, feel free to open a new wishlist issue for this!
But another valid point (which was raised here already) which speaks for signature checks is that if I get a QR-Code from a pharmacy, I scan it with the CWA and then travel to Spain (for example). They ask me for the QR-Code and I show it, they scan it and then they say that the QR-Code is not valid. I assume that their scanner app has the same general error message like CovPassCheck, which doesn't really tell one reason why this particular QR-Code is invalid.
Now I'm at the border in Spain, didn't took my yellow booklet with my because I thought yeahy we're going digital.
I do some investigation (phone the pharmacy) and then  it becomes clear that the QR-Codes were somehow issued with a signature that's not valid. Why didn't CWA tell me when importing the QR-Code? In the worst case the user will directly uninstall the app because of this.

Imagine this scenario: you add a signature check. Somebody will then write an article that you can just fool the bouncers by showing a manipulated screenshot and how insecure the whole system is.

this is not true. as many understand already: the SCANNING is the point. NOT just "showing".
if you SHOW a fake QR, CWA could SCAN and yield GREEN, CovPass/Check will scan and yield RED.
if you SHOW a correct QR, CovPass/Check will scan + yield GREEN.
all this is independent from "show-by-app", "screenshot" or "paper".
this is the baseline of the security for QR codes. the code is the KEY, the scanning (criteria eval) is the LOCK.
I will not open my door if you just show me a key. it needs to FIT. (metaphor for kids)
get used to it!
repeatedly saying, that CovPass/Check will yield correct status, is just a bad excuse for keeping a bad concept in CWA.
a concept bug is always more severe as a development bug. if this is due to EU-guidelines, then they have a concept bug IN the guidelines. and CWA copied that.
the term "wallet app" implies that the key(QR-code) is JUST stored.
but surprise.... the "wallet app" also shows the STATUS. so it NEEDS a valid unLOCK function. (= ALL criteria checks)
so if you really want to have the wallet-function only - REMOVE the status GREEN/RED at least.
the PRESS is just like a HACKER. they will exploit every weakness. even with "bullshit statements".
but no matter how stupid a statement is, somebody will RECHECK in depth. nothing will vanish over time.
tick tock...
BTW:
Ein-Tim is right. "zero-info-in-case-of-failing" is another problem. see covpass-android-issue-18

Dear community!
This topic and implications are discussed internally and hopefully we can give you more information here soon.
Best,

Corona-Warn-App Open Source Team
Related to topic: Check signature of certificates
Internal Tracking ID: EXPOSUREAPP-8010

this is not true. as many understand already: the SCANNING is the point. NOT just "showing".

Yes, I never stated that scanning isn't the point. I am saying that if the app shows the status, bouncers (or whoever) will assume more and more often that this is already sufficient proof ("because the app said so...") and that checking with the CovPassCheck app is optional. If the app wouldn't display the status, bouncers would maybe learn better that "hey, this is just a QR code, what does it actually mean? I need to check with CovPassCheck" instead of just accepting what might actually just be a manipulated pic.

this is not true. as many understand already: the SCANNING is the point. NOT just "showing".

... If the app wouldn't display the status, bouncers would maybe learn better that "hey, this is just a QR code, what does it actually mean? I need to check with CovPassCheck" instead of just accepting what might actually just be a manipulated pic.

its not a "maybe". they are forced to scan with e.g. CovPassCheck.
security by concept/design.
my "wallet" is black and with. no green or red lights.

I'm sorry to say this but the press writes that this is a "schwerwiegende Sicherheitslücke" (https://www.br.de/nachrichten/wirtschaft/impf-zertifikat-in-der-corona-warn-app-maengel).

The article is based on a profound and dangerous misunderstanding of the way certificates have to be validated. Vaccination certificates must always be verified on a trusted third party device (e.g. by using the CovPassCheck app). Viewing the QR code display in the CWA is not a verification. User devices cannot be trusted as they can contain a manipulated version of the app or show a forgery of the CWA display via screenshot. The CWA basically cannot prevent manipulation on the user's device, therefore an external check is always necessary.

Yes for sure you could argue that the people should follow the guidelines but they just don't and they won't because they're lazy or just think that it's enough to take a look at the certificate.
Try to explain somebody who's responsible for the guest registration and that the guests are tested, vaccinated or recovered what a signature is and explain that he/she has to check every QR code with an extra app and also has to see the ID document.
Everybody will say that this is not practical.

Without following the guidelines (i.e. using CovPassCheck on a trusted third-party device), the certificate verification is not secure (regardless of whether it is practical or not), even if CWA checks the signature. People need to understand this sooner rather than later. Digital signatures do not create trust by existing. Security requires that specific protocols are followed to be effective.
Introducing measures that check the signature inside the CWA would only "fix" the perception of security rather than be an actual security measure; they would essentially be security theater. Implementing such a thin is a double-edged sword because it may lead people to believe that the wrong verification process is now safe. In the end, this might compromise security even more.

Corona-Warn-App Open Source Team

Viewing the QR code display in the CWA is not a verification

while true actually verifying in app also helps users to see that their cert actually is valid, which was a point noted multiple times.

@My1

while true actually verifying in app also helps users to see that their cert actually is valid, which was a point noted multiple times.

This is correct and a valid consideration. I wanted to stress that this would purely be a convenience feature and should not be seen as a security measure.

Corona-Warn-App Open Source Team

This is correct and a valid consideration. I wanted to stress that this would purely be a convenience feature and should not be seen as a security measure.

Therefor the additional information should only be shown on the "Certificate" overview screen, not when displaying the QR code. Display of the QR code should ideally only contain the code. No name, no DOB, no DOV, no current vac status. This should make it even more clear that evaluation has to be done on a separate external device.

The article is based on a profound and dangerous misunderstanding of the way certificates have to be validated. Vaccination certificates must always be verified on a trusted third party device (e.g. by using the CovPassCheck app). Viewing the QR code display in the CWA is not a verification. User devices cannot be trusted as they can contain a manipulated version of the app or show a forgery of the CWA display via screenshot. The CWA basically cannot prevent manipulation on the user's device, therefore an external check is always necessary.

I know, you know, the community here knows. 8 out of 10 (according to another article) restaurants in Munich don't know 🤷🏻♂ It makes not sense to just argue something that is not done. In the end the user decides how a product is used, and not some PDF documents somewhere in the internet.

Introducing measures that check the signature inside the CWA would only "fix" the perception of security rather than be an actual security measure; they would essentially be security theater. Implementing such a thin is a double-edged sword because it may lead people to believe that the wrong verification process is now safe. In the end, this might compromise security even more.

I see your point here, but at least it wouldn't be possible anymore to just scan any certificate without a valid signature into CWA so the gatekeepers can't be fooled with this anymore.
But I'll now just leave this over to you, if there was the internal decision not to implement signature checks, that's fine for me too. You know what you do (I don't mean this ironically!)
Thank you @heinezen!

It makes not sense to just argue something that is not done. In the end the user decides how a product is used, and not some PDF documents somewhere in the internet.

Unfortunately, this is some of the rare cases where the user behaviour needs fixing. Cryptography is not magic, it is a tool that needs to be used properly. That does not mean that the user is to blame, btw. However, if the users do not follow the correct process, the process will not be secure. I have checked if RKI may have some official communication about this, but I currently don't know how that would look like. Right now, we only respond to the discussion on Github.

Corona-Warn-App Open Source Team

This is correct and a valid consideration. I wanted to stress that this would purely be a convenience feature and should not be seen as a security measure.

sure, I have no problem with that.

Therefor the additional information should only be shown on the "Certificate" overview screen, not when displaying the QR code. Display of the QR code should ideally only contain the code. No name, no DOB, no DOV, no current vac status. This should make it even more clear that evaluation has to be done on a separate external device.

totally agree. aside from maybe a heading like it being the corona app or whatever to just immediately see that you actually are on a corona qr rather than a contextless qr, it shouldnt show much extra stuff except maybe an on screen hint to have it scanned by the covpasscheck and having your ID card verified.

CWA is able to scan and store fake QR codes = severe design/concept flaw.
did "you" forget already ?
blaming the user will lead to a specific impression in public = failure and arrogance
saying "I dont blame the user, but he did sth wrong"
= is even more arrogant because its a illegal statement on logic level
no matter how many "nice statements or explainations" in-app or outside-app there will be

If the CWA check the signature it will not prevent to show fake qr codes.  It will encourage that is enough to scan this in an app and not check with CovPass Check.
But it is also planned to add signature and expiration check in the CWA. But again this will not add any security or helps to prevent fake qr-codes

@thomasaugsten while true we need to consider both sides and a user should be able to know whether his qr is valid so verifying it isnt something we should just leave out. but as others said the qr screen obviously shouldnt show much about validity and stuff for obvious reasons.

FYI: https://github.com/corona-warn-app/cwa-app-android/pull/3664 will add a hint that the CWA doesn't check whether the certificates are valid or not.

If the CWA check the signature it will not prevent to show fake qr codes. It will encourage that is enough to scan this in an app and not check with CovPass Check.
But it is also planned to add signature and expiration check in the CWA. But again this will not add any security or helps to prevent fake qr-codes

A signature verification check by the CWA would not increase security. But as someone who wants to check if the printed document with the QR-Code has been successfully signed by the RKI, this would be useful.
That is, because the CovPassCheck-App either says "proof of vaccination valid" or "proof of vaccination not valid". So you have to wait 14 days until you have full protection by vaccination before you can verify if the originally printed QR-Code has a correct signature. A wrong signature or just not signed QR-Code would not be detected in the moment you receive it. Maybe there are scammers who sell those QR-Codes and someone who bought it believes that he has to wait 14 days or there was a technical failure in the process of signing by the legitimate pharmacy / vaccination centre / medical.

Device: mi9tpro
CPC Version: 1.28.4
Related to: https://github.com/Digitaler-Impfnachweis/covpass-android/issues/18
2 Certs EU Vaccinations printed from Pharmacy :
Check with CPC:
-2/2 : Not Valid
-1/1 Recovered: Not Valid
best regards

@skyerjoe

2 Certs EU Vaccinations Certifiacte printed from Pharmacy :
Check with CPC:
-2/2 : Not Valid
-1/1 Recovered: Not Valid

Was the vaccination date more than 14 days ago?
If yes, did you try again with CovPassCheck-App?
There were reports that the CovPass backend was having problems and incorrectly rejecting valid certificates. This is supposed to be resolved now.
What does the Corona-Warn-App say about these certificates?
Are the vaccination certificates shown as complete?

@MikeMcC399
Hello Mike
I dont have the cwa i have the cctg and these are working in the app.
Yeah may it due the 14 days duration time cause my cert was issued with an wrong date, today i was at the pharamacy again got a new cert with right date and now the cert is valid.
But if cpc cannot difference between techincal problems and duration time it's really weird and confusing.
Thanks for helping me
Ps: i've delete my thread is there any possibilty to recover it ?

But if cpc cannot difference between technical problems and duration time it's really weird and confusing.

totally agree.
https://github.com/Digitaler-Impfnachweis/covpass-android/issues/21

Seems like Version 2.7 will add signature checks: https://github.com/corona-warn-app/cwa-app-ios/pull/3194

Is it to early to apply the Fix 2.7 label here?

Signature checks were implemented in today released version 2.7. I can confirm that they are working.

As I'm currently debugging business rules with self created certificates, would be nice not to have DCC validation... But I could open a wishlist issue for this.
🏃 💨
😉

I think I will close this issue now, as I have the right button for it.
Cheers, and thanks for discussion and implementation!

With this latest release 2.7.1 it seems that ALL previously valid UK vaccination certificates have now become invalid.
My certificate comes from the NHS and I was able to import and validate it successfully in the previous version.
It is now invalid.
This topic is now causing some waves in the UK for anfew million EU citizens.

@markil01
Could you please provide some more details about what error/message you see where by providing a screenshot/stacktrace/description.
Thanks in advance

@markil01 Thanks for reporting. I will cross-check internally, if the following "theory" is correct: If you were able to import UK certificates, apparently the UK NHS used the same scheme/format as the EU does. However, there is no digital signature (or a different format, or not distributed amongst the EU) that these are valid documents. See the map and the table here: https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en#how-can-citizens-get-the-certificate
Apparently the UK is not participating. However, I will come back to you and confirm the "theory".

AFAIK you were able to import the UK certificates but they were never valid. So if you could try to scan the QR-Code you received from the NHS with the CovPassCheck-App, I think it will say that the certificate is not valid.

AFAIK you were able to import the UK certificates but they were never valid. So if you @markil01 could try to scan the QR-Code you received from the NHS with the CovPassCheck-App, I think it will say that the certificate is not valid.

Not entirely correct.
I was successful in importing the UK certificate into the Corona Warn App and having it validated until last week. Since 2.7.1., that previously uploaded certificate is now reporting the error of an invalid signature.
A re-upload of an updated

QR code is not resulting into that problem straight away.

@markil01 Thanks for reporting. I will cross-check internally, if the following "theory" is correct: If you were able to import UK certificates, apparently the UK NHS used the same scheme/format as the EU does. However, there is no digital signature (or a different format, or not distributed amongst the EU) that these are valid documents. See the map and the table here: https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en#how-can-citizens-get-the-certificate
Apparently the UK is not participating. However, I will come back to you and confirm the "theory".
Corona-Warn-App Open Source Team

Correct.
It seems the NHS does not include a valid signature into their QR codes.

AFAIK you were able to import the UK certificates but they were never valid. So if you @markil01 could try to scan the QR-Code you received from the NHS with the CovPassCheck-App, I think it will say that the certificate is not valid.

Since 2.7.1 that is indeed the behaviour.  Even a previously (before 2.7.1)  uploaded certificate which had been marked as valid became now invalid, without a new upload.

@markil01
Do you still have access to the QR code you received from the NHS?
If yes, please download the CovPassCheck-App and scan the QR code with it and tell us the result!
Thank you!

@markil01
Do you still have access to the QR code you received from the NHS?
If yes, please download the CovPassCheck-App and scan the QR code with it and tell us the result!
Thank you!

Unfortunately,  due to country restrictions I am not able to download the above app. But I tested the same QR code with the Swiss version of the QR code check with the same result. Seems they also already included the digital signature check.

@markil01
Okay, and here you see, you were able to import the certificate into the Corona-Warn-App but it was never really valid.

at least the swiss app actually tells you this compared to the CovPassCheck which tells you practically nothing

@markil01
You wrote above

A re-upload of an updated QR code is not resulting into that problem straight away.

What exactly do you mean? I'm getting it like, from some point in time on NHS issued certificates that contain valid digital signatures, and these can be imported into Corona-Warn-App without any problems. Did I understand it correct?

According to @thomasaugsten the status of the UK NHS vaccination certificates in the EU environment was still not finalised. Maybe he could let us know the latest official status?
See also https://github.com/corona-warn-app/cwa-wishlist/issues/567#issuecomment-871663665

@MikeMcC399 @markil01 I just raised this topic internally: There are negotiations to include the UK NHS vaccination certificates (with their signature) in the EU environment, but at this stage we still don't know when it will be implemented (we assume that it will happen).

@markil01
You wrote above

A re-upload of an updated QR code is not resulting into that problem straight away.

What exactly do you mean? I'm getting it like, from some point in time on NHS issued certificates that contain valid digital signatures, and these can be imported into Corona-Warn-App (including v2.7.1 with digital signature check) without any problems. Did I understand it correct?

No, I uploaded the NHS certificate before the 2.7.1 release. It seem the previous version did not check for a valid signature.  I don't know if the certificate included a valid signature in July (when I uploaded my certificate for the first time). But now it does not.

@markil01

No, I uploaded the NHS certificate before the 2.7.1 release

Thanks, I got it now!
@dsarkar Big thank you to you for providing information so quickly!
@My1

at least the swiss app actually tells you this compared to the CovPassCheck which tells you practically nothing

This is intended because of data privacy ("Datenschutz"): The verifier shall not receive any information why the certificate is invalid, or what kind of certificate it is actually, because this reveals more information about the presenting person, than is necessary to do the job as gate keeper. For this purpose, only binary data "valid/invalid" would be enough.
For my taste, it's also a bit too restrictive.
About the NHS issue in general: I'm a bit shocked and sad that NHS is not yet fully participating. Without following these issues before, I had a kind of expectation (or: wishful thinking) that NHS was fully integrated. Let's hope that this can be achieved soon.

NHS England's page on https://www.nhs.uk/conditions/coronavirus-covid-19/covid-pass/ describes certificates similar to EU Digital COVID certificates, but they are not identical. There are links at the bottom of that page to Scotland, Wales and Northern Ireland. Hopefully they can all be made interoperable with the EU scheme.
Apart from the EU countries, https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en shows Iceland, Liechtenstein, Norway, San Marino, Switzerland & Vatican City, but not the United Kingdom currently participating in the EU Digital COVID certificates scheme.


Update package description to reflect Zeal support

I've tested my zazu-dash plugin with Zeal, and am changing the description to reflect that.

Thanks!


Added pointer-events:none to disabled button styling

Fixes #3070

Added pointer-events:none to disabled button styling.

@nlhkabu @di Can you please review this?
Thanks


Validate unwrap calls at mwc-wallet

Just need to review the unwrap calls. We don't want do that on tokio calls, they might fail.  We can't think about then as reliable.

Nothing critical was found. Just few minor.


InvalidSignature

I receive this error quite a bit when invoking the Search endpoint:
 com.yelp.clientlib.exception.exceptions.InvalidSignature
An issue was closed last month that had some similarities (Issue #36), but I'm not sure it was fully resolved as the solution was listed as simply just using RX libraries.
Here is give or take my exact implementation, some days it works, some days I get the above error:
            YelpAPIFactory apiFactory = new YelpAPIFactory(Constants.YELP_CONSUMER_KEY,
                    Constants.YELP_CONSUMER_SECRET,
                    Constants.YELP_TOKEN,
                    Constants.YELP_TOKEN_SECRET);
            YelpAPI yelpAPI = apiFactory.createAPI();

            Map<String, String> params = new HashMap<>();

            // general params
            params.put("limit", "10");
            params.put("radius_filter","16094");
            params.put("actionlinks","true");
            params.put("term", "restaurants");
            params.put("cll", latitude + "," + longitude);

            final Call<SearchResponse> yelpDataCall = yelpAPI.search(getCity(latitude,longitude), params);
            yelpDataCall.enqueue(new Callback<SearchResponse>() {
                @Override
                public void onResponse(Call<SearchResponse> call, Response<SearchResponse> response) {
                    int statusCode = response.code();
                    SearchResponse yelpData = response.body();
                }

                @Override
                public void onFailure(Call<SearchResponse> call, Throwable t) {
                    // com.yelp.clientlib.exception.exceptions.InvalidSignature
                    Log.d("Yelp", t.toString());
                }
            });

Hello @MalcolmMcFly, thanks for reporting this issue. I saw you are logging the stacktrace, do you mind to share one? (Please remove any key or sensitive data from the string) Thanks.

My apologies, I should have included this the first time:
D/Fragment:145: Yelp restaurants response error code: com.yelp.clientlib.exception.exceptions.InvalidSignature:  D/Fragment:146: GET <-- https://api.yelp.com/v2/search?location=Los%20Angeles&radius_filter=16094&actionlinks=true&sort=1&limit=10&cll=34.052234,-118.243685&term=restaurants W/System.err: com.yelp.clientlib.exception.exceptions.InvalidSignature:  W/System.err:     at com.yelp.clientlib.exception.ErrorHandlingInterceptor.parseError(ErrorHandlingInterceptor.java:85) W/System.err:     at com.yelp.clientlib.exception.ErrorHandlingInterceptor.intercept(ErrorHandlingInterceptor.java:45) W/System.err:     at okhttp3.RealCall$ApplicationInterceptorChain.proceed(RealCall.java:190) W/System.err:     at se.akerfeldt.okhttp.signpost.SigningInterceptor.intercept(SigningInterceptor.java:46) W/System.err:     at okhttp3.RealCall$ApplicationInterceptorChain.proceed(RealCall.java:190) W/System.err:     at okhttp3.RealCall.getResponseWithInterceptorChain(RealCall.java:163) W/System.err:     at okhttp3.RealCall.access$100(RealCall.java:30) W/System.err:     at okhttp3.RealCall$AsyncCall.execute(RealCall.java:127) W/System.err:     at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32) W/System.err:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133) W/System.err:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607) W/System.err:     at java.lang.Thread.run(Thread.java:761)

I failed to reproduce this.
@MalcolmMcFly, you say this error happens occasionally, is there any other thing you observes that might help on debugging? When the error happens, does that happen on few queries or does that happen for the whole day? Does this happen on other endpoints for you?

@tzuhanwu It's actually been occurring several days in a row now with no hint to why. I am currently in Thailand, with the geolocation settings of my emulator set to California if that means anything. Would sending you a gist with the fragment calling the API help at all? I am not seeing any additional error outside of the one I posted above.

Interesting, I'll see whether I can reproduce that.

@tzuhanwu I can confirm it works on an actual Android device. Now investigating why it doesn't work on the emulator.

I am experiencing a similar issue on my Android device. I am hitting the APIs from India. Is my location the reason for the Invalid Signature error?

@MalcolmMcFly, how are the things going? Are you still experiencing the problem with emulator?
@jgphilip, have you tried to run on emulator? Does the API work for you on emulator?

I am no longer using this API, sorry I cannot provide more details. I was not able to figure out what the underlying issue was.


CARDS-1110: Unsaved data lost when using the browser's back feature

I'm not sure if the call to setSaveInProgress is what is causing the data to not be saved upon clicking the browser's Back button. Perhaps it is a data race and the call to setSaveInProgress simply adds a delay and the value of formNode.current is empty at that point in time?
What happens if you change the line window.addEventListener("beforeunload", saveDataWithCheckin) to window.addEventListener("beforeunload", saveDataWithCheckin, true) so that the event handler executes on the capture phase of the beforeunload event instead of the bubble phase? It seemed to work fine for me, but then again, I was not able to locally reproduce this bug at all.

I confirm that either solution works. I'll switch to changing the event handler phase.


FT: Add recent metrics listing option

Adds the option to list recent metrics using the Utapi route with action ListRecentMetrics. This will return the latest metrics of between Date.now() and the second most recent fifteen minute interval (e.g., if it is 6:31 when the request is made, then list metrics starting from 6:15).
If start or end options are provided along with the recent option, they will be ignored and metrics of the "recent" time range will be returned.
Depends on https://github.com/scality/utapi/pull/114.

Yes, the route should be tested with Integration—I will be adding those. Also, tests for this command line tool are probably in order there.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "rel/6.4", "SCALITY_S3_BRANCH": "ft/add_get_recent_utapi_metrics"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11195 for CI status.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "rel/6.4", "SCALITY_S3_BRANCH": "ft/add_get_recent_utapi_metrics"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11211 for CI status.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "rel/6.4", "SCALITY_S3_BRANCH": "ft/add_get_recent_utapi_metrics"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11216 for CI status.

E2E passed.


Shared debug console

Visual Studio Code
Hi, I hope it's the right section for this question.
When I'm in a live share session with my friends and the host presses f5 to run the code, the result on debug console appears for a second to us, while on the host remains.
I think we missed some options, but we can not get through them.
Thank you all
Video for detail:
https://youtu.be/8RGM3mHvE-E

@Delirioo Hmm, that's unexpected. Are you in Visual Studio or Visual Studio Code?

@Delirioo Hmm, that's unexpected. Are you in Visual Studio or Visual Studio Code?

Visual Studio Code

@Delirioo What kind of app are you trying to debug? Also, when you say that the debug console appears for a second, are you saying that the Debug Console pane goes away? Or it just stops updating with new content?

@Delirioo What kind of app are you trying to debug? Also, when you say that the debug console appears for a second, are you saying that the Debug Console pane goes away? Or it just stops updating with new content?
Hi i made a video for a bettr explain of the problem:
https://youtu.be/8RGM3mHvE-E

Any ideas?

Hi, @Delirioo, I don't think this is a Live Share issue. I just tested your scenario and got the exact same result outside the context of a Live Share session. Closing this issue for now, let me know if I'm missing something and I'll reopen it.


Webcam Remains On even after StopRunning is Called

Issue: Webcam doesn't stop running on the device even though StopRunning is called on the VideoKitCameraManager on Windows (Logitech webcam still flashes blue even when camera should stop playing).
Expected Output: Camera should stop flashing blue when StopRunning is called.
Technical Specs:
Unity Version 2022.3.20f1
VideoKit 0.0.22-alpha-2 (from npm Package Manager)
Platform - Windows 64 bit (Untested on Mac, iOS, & Android)
How to Reproduce (In the Unity Editor):
Step 1: Enable the camera manager on awake
Step 2: Run the game

Experiencing same issue with Unity 6000.0.23f1 and VideoKit 0.0.22


Namespace confusion on imported "wrap" library

I'm attempting to create a new ASP.NET front end for our existing Silverlight application and ran into what seems like a bug for imported projects.  It seems like if there is a namespace "collision" -- the same namespace in two referenced assemblies, ASP.NET does not aggregate all the classes across all referenced assemblies.
We have a MyApp.DataAccess.Security.User object which previously lived in a MyApp.DataAccess.Security.dll project.  It was extracted so that we could move it to a portable class library, but the namespace remained the same.  It now lives in MyApp.DataModel.dll.
I generated "wrap" project.json files for each of those libraries (MyApp.DataAccess.Security.dll, MyApp.DataModel.dll) and see both of them as referenced by the ASP.NET project.
When I type new MyApp.DataAccess.Security. I only get classes from the MyApp.DataAccess.Security.dll and I do not get MyApp.DataAccess.Security.User.  Entering the full name only gives me a compile error.
Any help would be greatly appreciated.
I'm on VS 2015 Update 1 Community, with ASP.NET 5 RC1-Update 1.  I did not see a known issue filed against this -- sorry if I missed it.

@dythim can you log this in the https://github.com/aspnet/dnx/issues/ repo? This isn't specific to MVC.


Missing package keys are not shown in 1.5.0+

Error in 0.01s: bundler failed. invalid-package-exception: required property "main" doesn't exist in the package.json file.
This message is not shown in the new version and the build fails without any information why.

I think all errors are not shown Error in 0.01s: bundler failed. type-error: cannot read properties of undefined (reading 'javax.portlet.resource-bundle')
this is a major bug which need to be fixed asap.