Not clear that tenant-wide admin consent is required after turning on "user assignment required"

As an Application Developer maintaining internal App Registrations, I wanted to restrict an app to a group, with "user assignment required" and assigning the app to a group.
However I started getting errors signing in about "Need admin approval" "App needs permission to access resources in your organisation that only an admin can grant. Please ask an admin to grant permission to this app before you can use it."
This was confusing as the app was only requesting "User.Read" which does not require admin consent. All the documentation and blog posts I found initially pointed to needing admin consent for roles that require it but could not explain this case.
I eventually worked out that it was the "user assignment required" setting causing the issue, and found a blog post that explained

"When the access to the application is restricted via the “user assignment required”, an Azure Active Directory administrator needs to consent to all the permissions requested by the application, no matter whether users can normally provide consent for them."

It would be helpful if the Azure documentation made this more obvious, and pointed to the page explaining how to grant admin consent tenant-wide.

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

ID: 163a1f88-c268-18aa-a1e6-89b4a11bfb01
Version Independent ID: 1136a613-48f8-fb67-39d3-d9abdd351a0f
Content: Assign a user or group to an enterprise app in Azure AD
Content Source: articles/active-directory/manage-apps/assign-user-or-group-access-portal.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @kenwith
Microsoft Alias: kenwith

@DouglasHeriot Thank you for sharing your query! We are currently investigating and will get back to you shortly on this.

@DouglasHeriot WE are engaging the content authors to have your suggestions reviewed . They will take action accordingly.

Thank you for the feedback in improving the doc. I have added this item to our backlog so that it can be prioritized and worked on as appropriate.
#please-close

Thank you for the feedback in improving the doc. I have added this item to our backlog so that it can be prioritized and worked on as appropriate.
#please-close

@kenwith just came across and still can´t find clear information / explanation why admin consent is required for apps that have "User assignment required". While it makes sense to enable that option if admin consent was granted for certain permissions, making it mandatory when the option is enabled seems unnecessary or at least not clear in the docs.

Just stumbled across this as well - i have been spending last two days coming to this conclusion, and feel "relieved" that i'm not the only one having this problem. It doesn't really make any sense.
Is this a bug or by design?

No it's by design I think. Microsoft seems to believe that if an application should be user assigned that the use should not be able to give consent to it / an administrator needs to give consent in the name of the whole organization. But well, in my case the world looks different. There is no way a global administrator is able to make such decision globally. Especially if you follow a "least privilege principle". So yeah, thanks Microsoft.

@martin-jaeger-maersk It's "by design" (but the design isn't awesome and we plan to improve it). During sign in, Azure AD checks for assignment after checking for (and prompting for, if needed) consent. Also, when a user grants consent for themselves, they get assigned the app. Put together, this means a user could cause themselves to be assigned by granting consent—defeating the whole purpose of requiring assignment. So, currently, Azure AD simply disallows all user consent when the app requires assignment. It's not a perfect solution, but it adresses the majority of scenarios.
@magichappens89 Regarding least privilege: keep in mind that a delegated permission grant is the permission to act on behalf of a signed-in user. If a user is not allowed to sign in to an app, that app cannot exercise those delegated permissions on behalf of that user.
One approach we're considering is the following:

Check for assignment before checking for consent.
Stop blocking user consent only because assignment is required.
If assignment is required, do not assign the user when they consent for themselves.

We don't currently have an estimate for when we'll be able to make changes here, but we are aware of the challenges this causes for some customers.
I've updated the document here to add a note about this behavior.

@psignoret Thank you for the feedback 👍

but the design isn't awesome and we plan to improve it

Is there any information on when this improvement is planned?

Also curious about that. It´s bad user experience and a potential security risk as you seem to expect that either an admin is knowing which app should get which consent in the name of the whole company (we don´t) or train users muscle memory to always give consent when they login to an app that doesn´t require user assignment. And it might just be my feeling but checking app assignment before consent seems kinda obvious.

@bjoljo There is no timeline we can share at the moment, but I can tell you it is very unlikely it will be in the next six months.
@magichappens89 Thanks for your feedback!

There seems to be a regression here since the linked document https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/assign-user-or-group-access-portal currently does not mention anything about admin consent -- or it has never been updated.

@commakoerschgen The content that was being discussed several months ago in this (closed) issue has since been moved to a this page. Is that what you meant by "a regression"?

Yes, thanks for the link!

@bjoljo There is no timeline we can share at the moment, but I can tell you it is very unlikely it will be in the next six months.
@magichappens89 Thanks for your feedback!

@psignoret Is there any status update you could share with us?

@audunsolemdal Same status, unfortunately: There is no timeline we can share at the moment, but I can tell you it is very unlikely it will be in the next six months. (Starting now.)
What I can say is that this is a little bit closer to getting work started on it, but we're not quite there yet. (But still: unlikely to be in the next six months.)

@audunsolemdal Same status, unfortunately: There is no timeline we can share at the moment, but I can tell you it is very unlikely it will be in the next six months. (Starting now.)
What I can say is that this is a little bit closer to getting work started on it, but we're not quite there yet. (But still: unlikely to be in the next six months.)

Thanks for the update but given the case is 2 years old and, as mentioned, a feature that trains users muscle memory to be an easy target of illicit consent attacks it would be great to get a less vague timeline.

@magic-happenz With the current behavior, there are fewer scenario where a user would be prompted for consent, so I'm not sure I fully understand your comment about muscle memory. Would you mind elaborating?
The most common case (though I understand there are exceptions), is that an app that requires assignment is managed by someone in the organization, and interrupting users to prompt them for consent is undesirable. (Whereas third-party apps that nobody is actively managing do require each user to consent (if their organization allows them to.)

@psignoret I was hoping that with the redesign of the feature you would also improve that no global admin is required to actually allow an application to read user profiles. Currently I can only decide whether an admin or a user can consent to my application permissions but for the basic permissions an application needs, it should be more the responsibility of an application owner to make that decision. An internal auditor I could also imagine, but assigning everyone who may have such use case to the global admin role is not good practice.

@magic-happenz No, we have no plans here to remove the requirement that someone with sufficient privilege (either the user themselves or an administrator) needs to authorize an app's access to user profile details.

@audunsolemdal Same status, unfortunately: There is no timeline we can share at the moment, but I can tell you it is very unlikely it will be in the next six months. (Starting now.)
What I can say is that this is a little bit closer to getting work started on it, but we're not quite there yet. (But still: unlikely to be in the next six months.)

Hello @psignoret! So, after 11 months, I just wanted to check: is this planned for anytime soon? Thanks!

@fdelucaandrea, same status: Highly unlikely to be in the next six months.
To keep the hope alive: Some work has happened that takes us closer to this (e.g. if you experiment a bit, you'll notice we now check for assignment before checking for consent), but we're still too far away from the full solution to know when (or even if) we'll get to this.

I just lost a half a day coming to understand this situation and finding this issue. Updating documentation in the Azure Portal UI (and, ideally, sending a PR to the terraform docs) would save a lot of wasted time by Azure customers.
Once you understand the situation, you might be unhappy with it, but at least you know what's going on. Figuring out this confusing situation is almost as bad as the situation itself.


css:Increase padding for 'Default for stream' option (#28850).

Fixes:https://github.com/zulip/zulip/issues/28850
Before :-

After:

Modified the CSS styles responsible for the padding around the "Default for stream" option.

Increased the padding values to create more space around the option, addressing the reported issue.
Tested the changes across various devices and screen sizes to ensure consistent and satisfactory results.

Thanks! Please add screenshots showing how this looks in other languages, with longer and shorter strings.

Thanks! Please add screenshots showing how this looks in other languages, with longer and shorter strings.

Screenshots in different languages:-

hey @alya can you please review this and give your opinion on it.

Hm, I think leaving as much space on the left of the arrow as on the right would look better.

After making space eqaul in left and right of arrow:-

Please clean up your commit history and post again to request a review. See here for guidelines.

Closing as #28850 was completed through another PR, thanks for working on this @pankaj87072.


Error Virtual gamepad emulation

I have an error in the installation, hopefully they can solve me.

Thanks for ignoring the issue template, spam, closed.


Draft 09 changes

Should fail the web socket connection when receiving nonzero RSV* bits which are not understood (which means for now that any nonzero bits should fail the connection). Currently RSV* bits are simply ignored

Given that we're on draft 13, is this even worth the effort? (same for draft 10 bug?)


[Refactor] [Podfile] Updating Calling SDK to GA

Purpose
Updating the CallingSDK to GA 2.0.0
Removing the instance of how the CallAgent is created.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Test the code

pod repo update
pod install

Update the pod and quick smoke test of the demo app.

We are going directly into main?

Closing since the GA release does not have diagnostics tag


Versión 10.0.17763.1697 Windows 10 LTSC no found

termsrv.dll here
termsrv.zip

termsrv.dll here
termsrv.zip

nop we need ini update

nop we need ini update

nop we need ini update

))
file for help finding offsets

nop we need ini update

))
file for help finding offsets

I achieved something but the session is closed continuously

I achieved something but the session is closed continuously

[10.0.17763.1697]
LocalOnlyPatch.x64 =1
LocalOnlyOffset.x64 =7E381
LocalOnlyCode.x64 =jmpshort
SingleUserPatch.x64 =1
SingleUserOffset.x64=1386C
SingleUserCode.x64 =Zero
DefPolicyPatch.x64 =1
DefPolicyOffset.x64 =184F5
DefPolicyCode.x64 =CDefPolicy_Query_eax_rcx
SLInitHook.x64 =1
SLInitOffset.x64 =2198C
SLInitFunc.x64 =New_CSLQuery_Initialize

[10.0.17763.1697]
LocalOnlyPatch.x64 =1
LocalOnlyOffset.x64 =7E381
LocalOnlyCode.x64 =jmpshort
SingleUserPatch.x64 =1
SingleUserOffset.x64=1386C
SingleUserCode.x64 =Zero
DefPolicyPatch.x64 =1
DefPolicyOffset.x64 =184F5
DefPolicyCode.x64 =CDefPolicy_Query_eax_rcx
SLInitHook.x64 =1
SLInitOffset.x64 =2198C
SLInitFunc.x64 =New_CSLQuery_Initialize

add 10.0.17763.1697 (x64)
autoupdate (recommended)
Info: please use/downlaod the latest autoupdate.bat file (added "Wait for network connection" :: important fix)
or
rdpwrap.ini

[10.0.17763.1697]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=7E421
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=138BC
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=18545
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=21A3C
SLInitFunc.x64=New_CSLQuery_Initialize

[10.0.17763.1697-SLInit]
bInitialized.x64      =F3B8C
bServerSku.x64        =F3B90
lMaxUserSessions.x64  =F3B94
bAppServerAllowed.x64 =F3B9C
bRemoteConnAllowed.x64=F3BA0
bMultimonAllowed.x64  =F3BA4
ulMaxDebugSessions.x64=F3BA8
bFUSEnabled.x64       =F3BAC

add 10.0.17763.1697 (x64)
autoupdate (recommended)
Info: please use/downlaod the latest autoupdate.bat file (added "Wait for network connection" :: important fix)
or
rdpwrap.ini

[10.0.17763.1697]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=7E421
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=138BC
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=18545
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=21A3C
SLInitFunc.x64=New_CSLQuery_Initialize

[10.0.17763.1697-SLInit]
bInitialized.x64      =F3B8C
bServerSku.x64        =F3B90
lMaxUserSessions.x64  =F3B94
bAppServerAllowed.x64 =F3B9C
bRemoteConnAllowed.x64=F3BA0
bMultimonAllowed.x64  =F3BA4
ulMaxDebugSessions.x64=F3BA8
bFUSEnabled.x64       =F3BAC

With these adaptations to the INI, there is no status under the version, and it keeps saying that it cannot connect because there is already another user

With these adaptations to the INI, there is no status under the version, and it keeps saying that it cannot connect because there is already another user

solved with auto update thank you very much !!!

solved with auto update thank you very much !!!

I followed the install on autoupdate, but it doesn't work. Can you explain more?

I followed the install on autoupdate, but it doesn't work. Can you explain more?

Franckino14 Friend I have the Windows Ltsc, coml I get it to work for me. Thank you


@types should be in devDependencies

Since 8.1 some type definitions were added to dependencies. I think these should be in the devDependencies so they don't get installed along with gulp-tslint?

Fixed in 8.1.1.

Types with global definitions (environment/globals) should be installed by the consumer, but all the other types should be regular dependencies so type definitions work reliably.
In practice, it means that types for normal dependencies should also be normal dependencies.


site: wip updated home page

preview URL for vetting on devices https://deploy-preview-1229--wazero.netlify.app/

draft until RC assets uploaded (as that's what the install.sh gets)

$ curl https://deploy-preview-1229--wazero.netlify.app/install.sh | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  9790  100  9790    0     0  15294      0 --:--:-- --:--:-- --:--:-- 15393
tetratelabs/wazero info checking GitHub for latest tag
tetratelabs/wazero info found version: 1.0.0-rc.2 for v1.0.0-rc.2/darwin/arm64
tetratelabs/wazero info installed ./bin/wazero

$  ./bin/wazero version
1.0.0-rc.2


State did not change to unsafe

System Health details
System Information

version
core-2024.9.1

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.4

os_name
Linux

os_version
6.6.46-haos

arch
x86_64

timezone
Asia/Jerusalem

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

HACS Data
ok

GitHub API Calls Remaining
4994

Installed Version
2.0.1

Stage
running

Available Repositories
1407

Downloaded Repositories
38

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 13.1

update_channel
stable

supervisor_version
supervisor-2024.09.1

agent_version
1.6.0

docker_version
26.1.4

disk_total
65.2 GB

disk_used
20.7 GB

healthy
true

supported
true

host_connectivity
true

supervisor_connectivity
true

ntp_synchronized
true

virtualization
kvm

board
ova

supervisor_api
ok

version_api
ok

installed_addons
eWeLink Smart Home (1.4.3), Advanced SSH & Web Terminal (19.0.0), Studio Code Server (5.15.0), Nginx Proxy Manager (1.0.1), File editor (5.8.0), Docker Wyze Bridge (2.10.1), Home Assistant Google Drive Backup (0.112.1), Anylist (1.7.2), Whisper (2.1.2), Piper (1.5.2), Govee to MQTT Bridge (2024.07.13-82ddc6e9), Mosquitto broker (6.4.1)

Dashboards

dashboards
9

resources
24

views
33

mode
storage

Recorder

oldest_recorder_run
September 5, 2024 at 4:43 AM

current_recorder_run
September 15, 2024 at 9:39 AM

estimated_db_size
669.86 MiB

database_engine
sqlite

database_version
3.45.3

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
The sensor binary_sensor_oref_alert did not change to unsafe for my area:

Reproduction steps
N/A
Debug logs
N/A

Diagnostics dump
No response

4 days ago Pikud HaOref changed the list of areas in Israel. The area "מודיעין" was replaced by "מודיעין מכבים רעות":

https://github.com/amitfin/oref_alert/commit/8c609da3759f9dbe2e26cd7c92e736a531dddf26#diff-498feada52613d24e53ae7f3840c28d14b52338dd9d3acd56c85f8d15fdb6db9
This can also be seen in the above screenshot where the alert was for "מודיעין מכבים רעות" and not for "מודיעין":

To fix the definition for your area:

Make sure the integration version is v2.12.0 or newer. This is the version with the updated list from Pikud HaOref.
Reconfigure the integration by removing the old area and choosing the new one:

I understand it sucks. Such a breaking change should happen with more consideration and careful. However, the list of areas is controlled by Pikud HaOref and all we can do is follow their changes.

v2.12.2 adds a more proactive approach for this type of issues by raising a repair ticket when an area doesn't exist anymore.
For example, in the above case a repair ticket will be raised explaining that "מודיעין" is an unknown area, it should be removed, and a different relevant area should be selected.

@amitfin thank you! I have updated the addon with the correct area.
Thank you for creating (and maintaining!) this addon


Combining Boolean and non-boolean vairables into single vector

I am doing a mixed-integer optimization that involves several matrix operations on my Boolean variable vector:
w = cp.Variable(n, boolean=True)
Currently the variable w is initialized as Boolean with length n, but now I want to set the first element to be non-Boolean. Is there a way to change the Boolean property for a subset of the variable vector w? Or, could I create two variable vectors and then combine them into a single vector?

@armarion You can do something like:
y_real = cvxpy.Variable(n)
y_bool = cvxpy.Variable(m, boolean=True)
y = cvxpy.hstack([y_real, y_bool])

Hope that helps :)

Yes! Exactly what I needed. Thank you!


A few features useful for AppServices

This PR implements:

A new directive copy in mx-tester.yml, to copy files into the Docker guest.
A new directive appservices to generate an appservice yml configuration.

For my own personal testing, I have made use of copy. However, it is my understanding that @jesopo would benefit from what we have in appservices.
Note The appservices code is not really tested. I'd be grateful if @jesopo could test-drive it :)

Sorry for getting back late on this
FileNotFoundError: [Errno 2] No such file or directory: 'appservices/mjolnir.yml'
*******************************************************************
 Error during initialisation:
    [Errno 2] No such file or directory: 'appservices/mjolnir.yml'
 There may be more information in the logs.
*******************************************************************

Synapse now seems to be struggling to find the registration for Mjolnir


Require minimum version of Ansible

It would be great if Drupal VM asserted that a minimum version of Ansible is in use.
You could use the assert module for this.
Something like:
- hosts: all
  tasks:
    - name: Get current Ansible version on local host
      local_action: command ansible --version
      register: local_ansible_version
    - assert:
        that:
          - "'2.1' in local_ansible_version.stdout"

+1 to this; There are a number of people who have run into issues like #891, #890, #771, and #713... would help stomp out those errors in the future.
This would need to run prior to anything else.

I think I may spend some time later turning this into a module, but here's a python script that will fail if the installed version of Ansible isn't at least 2.1.0:
# -*- encoding:utf-8 -*-
from distutils.version import LooseVersion

import sys

import ansible

def print_red_bold(text):
    print('\x1b[31;1m' + text + '\x1b[0m')

def __init__(self):
    # Can't use `on_X` because this isn't forwards compatible with Ansible 2.0+
    required_version = '2.1.0'
    installed_version = ansible.__version__
    if LooseVersion(installed_version) < LooseVersion(required_version):
        print_red_bold(
            "drupal-vm restriction: only Ansible version greater than {version} is supported.\n"
            .format(version=required_version)
        )
        sys.exit(1)

For easier maintenance (e.g. when 2.2+ comes out), to test a minimum version, we can use version_compare... I hope :)

Nice... looks like there's already a globally-accessible ansible_version variable that lets us do this more easily than string comparisons.
Test playbook:
---
- hosts: localhost
  pre_tasks:
    - debug: var=ansible_version

Output:
TASK [debug] *******************************************************************
ok: [localhost] => {
    "ansible_version": {
        "full": "<IP_ADDRESS>",
        "major": 2,
        "minor": 1,
        "revision": 1,
        "string": "<IP_ADDRESS>"
    }
}

I'll be adding a new variable for this and for vagrant min version (was going to hardcode it in playbook, but it's nicer to use a var):
drupalvm_vagrant_version_min: '1.8.5'
drupalvm_ansible_version_min: '2.1'

@geerlingguy nice catch. I looked for both a version_compare function and an ansible_version var and couldn't find either. ansible_version wasn't listed when I ran setup ad-hoc

One thing to note is that if someone has a really old version of Ansible, it's likely to fail on a syntax check before it ever gets to running your version check task. The only way I know to get around this is to write something as a callback module that is compatible with both pre and post 2.0

@smiller171 - True, mentioned in blog post here: http://www.jeffgeerling.com/blog/2016/require-minimum-ansible-version-your-playbook
I'd rather Ansible solves this upstream... everyone adding their own callback plugin for a version check seems a bit crazy.

Agreed. I wonder if Ansible would accept a pull request to add in that functionality. Having it as a top-level parameter for a play seems to make sense:
- hosts: all
   minimum_ansible_version: 1.9.2
   tasks:
.....

Of course this isn't helpful for users of old versions, since the old versions wouldn't support the minimum_ansible_version parameter. Perhaps you could also ship a callback plugin with instructions on how to include it with your playbooks.


throw er; // Unhandled 'error' event

node 9.3.0
win 10 x64
Console log
`[06:14:34] Did you forget to signal async completion?
events.js:136
throw er; // Unhandled 'error' event
^
Error: Sending illegal frame (DATA) in CLOSED state.
at Stream.transition [as _transition] (D:\LAN\WORK\7777Taxi\7777taxi.com\no
de_modules\http2\lib\protocol\stream.js:628:33)
at Stream._pushUpstream (D:\LAN\WORK\7777Taxi\7777taxi.com\node_modules\htt
p2\lib\protocol\stream.js:230:8)
at Stream._write (D:\LAN\WORK\7777Taxi\7777taxi.com\node_modules\http2\lib
protocol\stream.js:307:25)
at doWrite (_stream_writable.js:399:12)
at writeOrBuffer (_stream_writable.js:385:5)
at Stream.Writable.write (_stream_writable.js:292:11)
at OutgoingResponse._write (D:\LAN\WORK\7777Taxi\7777taxi.com\node_modules
http2\lib\http.js:336:17)
at doWrite (_stream_writable.js:399:12)
at writeOrBuffer (_stream_writable.js:385:5)
at OutgoingResponse.Writable.write (_stream_writable.js:292:11)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this com
mand. `
i catch this bug after 5-10 repeat pushing button 'reload page' in chrome canary x64

hello?

close


Improve GitHub Actions for pull requests

See https://github.com/Senzing/community-roadmap/issues/71

Pull requests:

https://github.com/Senzing/docker-web-app-demo/pull/81


Should be way of switching settings (right now it's immutable Singleton)

In Alpha4 we are using "hack" for doing this. We will have more correct way of doing this later in Alpha5. Will create this issue in Alpah5


Incorrect include url crashes FMark

[x] Should check if file exists before trying to open the file.
[x] On Windows file includes put in a weird double c drive "C:\c:"

Fixed by #137


Playground lost syntax highlighting

Bug Report
Playground now doesn't do syntax highlighting
To Reproduce

Create a page with playground
Code is not highlighted

Expected behavior
Code should be highlighted
Environment
<EMAIL_ADDRESS>Additional context/Screenshots

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

Should be fixed in<EMAIL_ADDRESS>Please feel free to re-open if it's not 👍


Do something about the clipboard on insecure domains and Firefox

Current workarounds:
On Firefox you can use middle-mouse click or shift+insert (possibly depends on your OS). Pasting with the right-click menu will not work on Firefox but you can press shift+right click to get the native context menu and use that to paste. ctrl+v will work in the editor but not the terminal.
On Chrome ctrl+shift+v should work in the terminal and ctrl+v should work in the editor. If you use Chrome over a secure domain and accept the clipboard prompt then the right click menu to paste will work for the terminal but not the editor.

Summary:
Currently only native paste or the clipboard API will work so there are certain cases where pasting will not work with the keybinding you'd expect (ctrl+shift+v in the terminal) or with the right-click menu (see my comments below for details).

Original post:
Currently if you try to copy on an insecure domain you get "cannot read property writeText of undefined` because it's trying to access the clipboard which is undefined in insecure contexts.
We could fall back to copying using document.execCommand("copy") which I think is what the editor does. That seems to work fine.
For paste I believe the editor listens to the native paste command. That's not an option for the terminal because it requires we use the native keybinding (ctrl+v) and that has a different function in the terminal.
I've experimented with document.execCommand("paste") in the past but it didn't work so I assume this is why VS Code didn't use it here although perhaps more research is warranted here. That means our only option is the browser clipboard API which is only available in secure contexts.
Having only copy but not paste available on the terminal doesn't seem like a good solution so I think our best bet is to detect when the context is insecure and display a message describing the need to use https.
Service workers don't work in an insecure context either so maybe it would be a good idea to display a message when the browser loads as well just so people are aware they're accessing code-server in an insecure way and that things might be broken (as well as just generally dangerous).
While we're at it we could also implement the editor paste command using the clipboard API which would resolve https://github.com/cdr/code-server/issues/1105.

document.copyText() seems to be the only way we can support HTTP clipboard access but I'm not really certain if WHATWG will still keep it up, and that'll be bad for localhost dev if they decide to axe it.

We can work around copying, it's the pasting that's a problem. :(

why is it so hard to generate a self signed cert for general development use and allow that in the browser via import? You do that once you can reuse the certs for your own dev projects always and its done where is the problem=?

Not everyone has the expense to do that, please consider those implications and never assume people to do one thing and do it correctly. This is not open for debate per se.

why is it so hard to generate a self signed cert for general development use and allow that in the browser via import? You do that once you can reuse the certs for your own dev projects always and its done where is the problem=?

Well, some cloud platforms set certificate as enterprise version paid function, so some personal small projects have no way to enable https.

@easychen show me a link to such a offer. In general you can always get public signed certs from letsencrypt and private once for your company via any PC in that company

@easychen to make my point clear if your using http + auth or without it does not matter your working with clear text over many hops this is not acceptable in 2019 as you expose everything over the line.
Even More Clear
A Login / JWT Token in the http Header of every Request in Clear Text is like Opening it for the Internet.

Is it really so hard to make copy-paste working in terminal even on not trusted connections?
Is confusing error message: "cannot read property writeText" really better than making it work (maybe with some warning about the certificate issue)?
It works in Jupyterlab terminal for example..
When you check number of issues related to this problem, you see that this is quite common/serious thing which kills usability for quite a number of people...

@easychen show me a link to such a offer. In general you can always get public signed certs from letsencrypt and private once for your company via any PC in that company

https://www.sinacloud.com/index/price.html

@easychen to make my point clear if your using http + auth or without it does not matter your working with clear text over many hops this is not acceptable in 2019 as you expose everything over the line.
Even More Clear
A Login / JWT Token in the http Header of every Request in Clear Text is like Opening it for the Internet. A Cookie makes not much diffrence

We DO login via https,  but just  use this for temp online sandbox under a  separate  subdomain for every users.
And not all cloud platform has api to upload the certs ...  welcome to the real world.

If we really need to make the clipboard work on insecure domains the only thing we can do is allow the native keybindings through and listen to the paste event (as far as I'm aware; happy to be proven wrong though).
As long as the browser supports Ctrl+Shift+V for pasting then that will work since we can't use Ctrl/Cmd+v.
Chrome supports Ctrl+Shift+V everywhere while Firefox only supports it for contenteditable=true elements so that would have to be taken into consideration. I don't know about Safari.
Also note you can use Shift+Insert although this might depend on your OS.
This is probably a change that should be made upstream, though, rather than patched in code-server.
It would be wise to use SSL even for a temporary sandbox though, right?

It would be wise to use SSL even for a temporary sandbox though, right?

We will when possible, it's not a tech problem but a financial one. Version 1 is good enough for us, great thanks for your excellent work.

For the pasting problem, I see it works ( cmd+v ) under insecurity domain, maybe I missing something?

code-server
Version: 1.39.2
Commit: d81d5f499f09bc887db2fb9bb35e0222bf48d80c
Date: 2019-10-24T21:52:45.195Z
Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.120 Safari/537.36

My guess is that cmd+v has no assignment in VS Code which allows it to perform a native paste instead. You could check by searching for "cmd+v" in the keyboard shortcuts.
That actually presents an interesting workaround. If you're using Chrome and Ctrl+Shift+V doesn't work you can open the keyboard shortcuts and assign the terminal paste keybinding to something obscure you won't use which allows Ctrl+Shift+V to perform a native paste instead.

Neat! So for anyone else having the problem, currently the workaround is to search in the keyboard shortcuts for "workbench.action.terminal.paste" then remove the keybinding (Delete or right-click and click Remove Keybinding).
The long-term plan: when the clipboard API isn't available, don't bind those keys.

this is also the case in upstream https://github.com/microsoft/vscode/issues/83595, https://github.com/microsoft/vscode/issues/83598

We'll wait for upstream fix and see what we can do.

Another note about this on Firefox. Firefox doesn't support the clipboard API at all apparently (at least not on websites) and it also doesn't bind Ctrl+Shift+V so the above workaround doesn't work. The only solution as far as I can tell is to use contenteditable=true on Firefox which makes Ctrl+Shift+V work.
So there's currently no workaround for Firefox and it won't work even in a secure context.

Oh actually as previously mentioned Shift+Insert is a workaround.

@code-asher the problem even exist in theia

Solution:
Open web console and past this code, just it.
<script>
// A minimal polyfill for `navigator.clipboard.writeText()` that works most of the time in most modern browsers.
// Note that on Edge this may call `resolve()` even if copying failed.
// See https://github.com/lgarron/clipboard-polyfill for a more robust solution.
// License: public domain
function writeText(str) {
  return new Promise(function(resolve, reject) {

    /********************************/
    var range = document.createRange();
    range.selectNodeContents(document.body);
    document.getSelection().addRange(range);
    /********************************/

    var success = false;
    function listener(e) {
      e.clipboardData.setData("text/plain", str);
      e.preventDefault();
      success = true;
    }
    document.addEventListener("copy", listener);
    document.execCommand("copy");
    document.removeEventListener("copy", listener);

    /********************************/
    document.getSelection().removeAllRanges();
    /********************************/

    success ? resolve(): reject();
  });
};
</script>

Solution
@code-asher Open web console and past this code, just it.
<script>
// A minimal polyfill for `navigator.clipboard.writeText()` that works most of the time in most modern browsers.
// Note that on Edge this may call `resolve()` even if copying failed.
// See https://github.com/lgarron/clipboard-polyfill for a more robust solution.
// License: public domain
function writeText(str) {
  return new Promise(function(resolve, reject) {

    /********************************/
    var range = document.createRange();
    range.selectNodeContents(document.body);
    document.getSelection().addRange(range);
    /********************************/

    var success = false;
    function listener(e) {
      e.clipboardData.setData("text/plain", str);
      e.preventDefault();
      success = true;
    }
    document.addEventListener("copy", listener);
    document.execCommand("copy");
    document.removeEventListener("copy", listener);

    /********************************/
    document.getSelection().removeAllRanges();
    /********************************/

    success ? resolve(): reject();
  });
};
</script>

Another workaround for Firefox: Shift + Right-click opens the context menu, and you can simply click paste. Middle-click did not work for me.

I can confirm that the problem persists in Firefox also with HTTPS (secure certificate, no self-signed). Firefox v. 68.8.0esr and code-server v. 3.3.1

Does not work with Signed certificate using Safari, Chrome, or Firefox on iPad with latest release. No copy/paste functionality unless in the actual terminal window.

From https://github.com/cdr/code-server/issues/1725

I think the move will be to always show the paste option but when it's clicked display a message describing why it won't work and possible solutions (i.e. switch to Chrome).

@LindsayWeir Not sure why it doesn't work on iPad, maybe the clipboard API isn't implemented there. We should show a message there as well @code-asher

I was updating from 3.2.0 to 3.4.1. When I use shift + right click I see now a different native context menu where I can bookmark or save the content. But no pase option is on that context menu. I guess on the older version the terminal was kind of textarea where the context menu shows me copy and paste option? So for me it is not possible to paste any content to the terminal :-(

Is this just on Firefox? What about using the native context menu adding the VS Code ones with this? It would be slightly more cluttered and wouldn't fit the VS Code theme, but I think that's worth it (or it could be optional).

How it would look:

yes it is only a problem with firefox, on chrome it works very well. If I use the shift + righ click now it looks like the screenshot below. As you can see, there is no chance to paste anything.....

yes it is only a problem with firefox, on chrome it works very well. If I use the shift + righ click now it looks like the screenshot below. As you can see, there is no chance to paste anything.....

You have to right click in the actual terminal, it looks like you're right-clicking on the right-click menu

that was a fast answer ;-)
I cklick on the terminal and select a text with shift + left click and than I use shift + right click to open the context menu. As you can see now the native context menu is rendered above the terminal context menu, that happens somtimes.

I have the same issue with Chrome, what's the workaround for it?

SHIFT + INSERT works with Firefox on Windows 10 version 2004

For Linux laptop users without an Insert key, three finger click on touchpad equals middle click.

I found a workaround for Firefox!
Just set dom.events.testing.asyncClipboard to true.

I found a workaround for Firefox!
Just set dom.events.testing.asyncClipboard to true.

THANK YOU SO MUCH!

@Miguelklappes If you are not being sarcastic, why did you give my comment thumbs down?

@Miguelklappes If you are not being sarcastic, why did you give my comment a thumbs down?

By mistake!! I'm not being sarcastic haha

hey it seems something changed? with the latest FF version (Codeserver Version 3.10.1) I can copy paste (CTRL+C, CTRL+V)in the terminal? I am totally surprised ;-)

Oh really? Maybe something got fixed upstream. Thanks for the note @hgw77 😄


Reading (Partial Strings) into char[]

Discussed in https://github.com/stephenberry/glaze/discussions/264

Originally posted by raddy April 25, 2023
If we've got a struct:
struct A
{
    int  x;
    char buf[12];
};

And we've got some JSON: std::string buffer = R"({"x":"999", "some_string": "SHORT_STRING_NAME"})";
Is there a way to read directly into the struct's buf? (I guess writing that which fits + null term)

Solves issue #264

This is now supported.  If the input string is too long, an error will be returned.


2.0-wip: no debugger

Yep, YUI Compressor crashes. Took a while to figure that out.

thanks!  sorry about that


docs: Establish VISION for "One Layer, All Storage"

Which issue does this PR close?

Result of https://github.com/apache/opendal/discussions/5301
Rationale for this change

Establish VISION for "One Layer, All Storage"
What changes are included in this PR?
Update docs that refer to vision.
Are there any user-facing changes?
Not in the short term, but it will affect the entire future.

I also invite all @apache/opendal-committers to join the review.

Thank you all for the review, especially @tisonkun for the detailed comments and @xxchan for optimizing the expression. I'm going to merge this now.
Let's build "One Layer" for "All Storage"!


Not working with parentheses

Consider this example:
import pandas as pd
(
    pd.to_datetime("2011-01-07") \
    # convert to period
    .to_period(freq="W") \
    # convert back to timestamp
    .to_timestamp()
)

Trying to run the block in parentheses throws this error:

Thank you

That snippet works fine for me. You might need to check that you installed the package correctly?
https://github.com/kylebarron/vscode-jupyter-python/assets/15164633/d2ce3ab6-8b4d-4fce-85bc-1af2fd3b09de

I restarted VScode and now it's working fine, thank you @kylebarron


SLING-4428 -  Sightly: scriptComment and styleComment contexts are not doing anything

Added support for multiline comment validation in XSS API.
Added implementation and test.
Added styleComment context to Sightly.
Added proper validation for scriptComment and styleComment contexts.

Change merged by @rombert in https://svn.apache.org/r1660420


client.uploadPhoto not works

const FileCookieStore = require('tough-cookie-filestore2')
const Instagram = require('instagram-web-api')
const { username, password, accounts } = process.env
const cookieStore = new FileCookieStore('./cookies.json')
const client = new Instagram({ username, password, cookieStore });
const uploadPhoto = async (photo) => {
const { media } = await client.uploadPhoto(photo)
return media;
}
(async () => {
await client.login()
await client.uploadPhoto("https://habrastorage.org/webt/or/77/t4/or77t4go_qqok-v8qnenx5rxcjy.png")
// Or Even Like This
// ./or77t4go_qqok-v8qnenx5rxcjy.png
})()
(node:31679) UnhandledPromiseRejectionWarning: TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
    at Object.open (fs.js:423:10)
    at _openReadFs (internal/fs/streams.js:158:15)
    at new ReadStream (internal/fs/streams.js:135:5)
    at Object.createReadStream (fs.js:1835:10)
    at Instagram._uploadPhoto (/Users/0x77/0x77.memes/node_modules/instagram-web-api/lib/index.js:330:51)
    at Instagram.uploadPhoto (/Users/0x77/0x77.memes/node_modules/instagram-web-api/lib/index.js:345:17)
    at /Users/0x77/0x77.memes/index.js:15:18
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
(node:31679) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 3)
(node:31679) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
 ~/0x77.memes  wget ^C
 ~/0x77.memes  curl -O https://habrastorage.org/webt/or/77/t4/or77t4go_qqok-v8qnenx5rxcjy.png                                                                                1356ms < четверг,  9 апреля 2020 г. 22:43:55
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   138  100   138    0     0   1169      0 --:--:-- --:--:-- --:--:--  1169
 ~/0x77.memes  node index.js                                                                                158ms < четверг,  9 апреля 2020 г. 22:43:56
(node:31748) UnhandledPromiseRejectionWarning: TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
    at Object.open (fs.js:423:10)
    at _openReadFs (internal/fs/streams.js:158:15)
    at new ReadStream (internal/fs/streams.js:135:5)
    at Object.createReadStream (fs.js:1835:10)
    at Instagram._uploadPhoto (/Users/0x77/0x77.memes/node_modules/instagram-web-api/lib/index.js:330:51)
    at Instagram.uploadPhoto (/Users/0x77/0x77.memes/node_modules/instagram-web-api/lib/index.js:345:17)
    at /Users/0x77/0x77.memes/index.js:15:18
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
(node:31748) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 3)
(node:31748) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Wrong Code!
just  do:
    await client.uploadPhoto({photo: 'url image or path'}).catch((error) => {
    	console.log(`An error occurred on uploading photo: ${error}`);
    }).then((responseUpload) => {
    	console.loog(responseUpload)
    })

It helped, thanks!

Wrong Code!
just do:
    await client.uploadPhoto({photo: 'url image or path'}).catch((error) => {
    	console.log(`An error occurred on uploading photo: ${error}`);
    }).then((responseUpload) => {
    	console.log(responseUpload)
    })

thank you codex


Why is the size of the exported mnist_test.bin file 0 bytes?

https://github.com/iree-org/iree/tree/f9fa934c649749b30fc4be05d9cef78eb043f0e9/samples/vision_inference

I tried it quickly and didn't have any issues. Maybe cat or python3 failed (e.g. mnist_test.png was deleted or in wrong dir)?

Thank you. This problem has been solved.

I tried it quickly and didn't have any issues. Maybe cat or python3 failed (e.g. mnist_test.png was deleted or in wrong dir)?

Thank you. This problem has been solved.


Integrated terminal text is horionztally squished

VSCode Version: 1.17.1
OS Version: Windows 10

Steps to Reproduce:

Open integrated terminal (Ctrl + `)

Reproduces without extensions: Yes
Image is attached showing squish.

I get this as well:

The font looks extremely small and narrow.

I think it's related to the update, not sure, I updated a couple versions at once.

I am getting issues with my terminal setup as well.

duplicate of #36059

Duplicate Duplicate https://github.com/Microsoft/vscode/issues/35991


[WIP] Add axe linter config file

@tisto Is this still relevant?

Closing this since it's incomplete and 4 years old


Add missing release file for java 17

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

This is the same fix that was merged in https://github.com/conda-forge/openjdk-feedstock/pull/154 applied to the java17 branch.

@conda-forge-admin, please rerender


Document dependent pipelines

Tasks

[x] Create Depenendent Pipelines example
[x] Documentation updated

ok to test

ok to test


shim nodejs 'http' and 'https' for duktape

Hi everyone, I'm curious if it's possible to return a custom implementation of http/https whenever bundled code requires these built-in node packages. The use case being I'd like to redirect http method calls to a custom http implementation provided via go-duktape. Ideally, the custom implementation would be used when requiring statically and when requiring with expressions.
e.g., static: const http = require('http'); and with expression: require(url.split(':')[0])
I'm currently using a npm package that contains a require with an expression like in the example above, however, this results in an error being thrown - Cannot find module 'http'
My first thought was to monkey patch the require function to intercept calls for 'http' but looking at the bundled code, I see that calls with require(url.split(':')[0]) are replaced with __webpack_require__(3) which points to a webpackEmptyContext module.
I've searched plenty for this kind of thing but haven't had any luck. Any help is appreciated

require(url.split(':')[0])

You have to use the ContextReplacementPlugin to fix that. It's too dynamic for webpack.

Answer above, please use the issue template in future


Add GCC 11 builder to jenkins

This also triggered the error reported in https://github.com/STEllAR-GROUP/hpx/issues/5275. I've simply made the conversion explicit, but I don't know if there's a more proper fix for the issue.

This also triggered the error reported in #5275. I've simply made the conversion explicit, but I don't know if there's a more proper fix for the issue.

That works as well. Thanks!

Sigh... one more try. This was missing the slurm constraint for gcc-10.


Feature: share tcc phase 1 and 2 context

Ⅰ. Describe what this PR did
Share TCC model phase 1 and 2 context

TCC phase 1 business params can be passed to phase 2
The parameters in the execution of service or return results can be passed to the phase 2 by annotations on the returned result fields. see ActionContextField

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2742 into develop will decrease coverage by 0.19%.
The diff coverage is 25.92%.

@@              Coverage Diff              @@
##             develop    #2742      +/-   ##
=============================================
- Coverage      50.89%   50.70%   -0.20%
+ Complexity      2849     2847       -2
=============================================
  Files            567      567
  Lines          18173    18175       +2
  Branches        2166     2129      -37
=============================================
- Hits            9250     9215      -35
- Misses          8029     8041      +12
- Partials         894      919      +25

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/io/seata/common/util/CollectionUtils.java
94.54% <0.00%> (-3.57%)
32.00 <0.00> (ø)

...seata/core/store/db/sql/log/MysqlLogStoreSqls.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

...eata/core/store/db/sql/log/OracleLogStoreSqls.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

.../core/store/db/sql/log/PostgresqlLogStoreSqls.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

...ava/io/seata/rm/tcc/api/BusinessActionContext.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...a/rm/tcc/interceptor/ActionInterceptorHandler.java
20.98% <11.76%> (-4.39%)
7.00 <0.00> (ø)

...in/java/io/seata/server/session/GlobalSession.java
83.87% <50.00%> (-0.30%)
71.00 <2.00> (-1.00)

...java/io/seata/server/coordinator/AbstractCore.java
55.81% <100.00%> (+0.51%)
12.00 <0.00> (ø)

...a/server/storage/db/store/LogStoreDataBaseDAO.java
71.76% <100.00%> (ø)
34.00 <0.00> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

... and 29 more

Please add ：
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

please handle ci errors

Please add a switch to '@TwoPhaseBusinessAction' for this feature, because not all TCC need to share phare 1 and 2 context.
This will cause unnecessary performance degradation.

There is a problem: branchReport and localTransactionCommit can't guarantee atomic,
so branchReport must be transferred into the local transaction.
目前还存在一个问题：分支事务上报和本地事务提交无法保证原子性。
所以需要把分支事务上报功能转移到本地事务之内。

如果 #3062 先进行合并的话，那这里的RedisTransactionStoreManager#updateBranchTransactionDO也要进行对应的修改；
如果这个pr先合并的话， #3062 的那个pr的 RedisTransactionStoreManager#updateBranchTransactionDO也要进行对应的修改。
这个后续的reviewer在看的时候关注一下。

Codecov Report

Merging #2742 (af24dda) into develop (d28274d) will decrease coverage by 1.77%.
 这个PR就关闭喽。


Add deassignGroup Command

The complement to the assignGroup command, deassignGroup, does not exist. This is a feature flaw, as we effectively cannot undo assignGroup. This must be rectified before v1.3 ends.
Related to #168.

This feature has been deemed unimportant, as we have fine-grained deassignment on the individual level and on the address book level now.


#99 Run Unit Tests as Pre-Push hook

Please also test on Windows :)
It shouldn't actually RUN the tests on push, if nothing has changed in the frontend dir for the frontend tests and backend dir for backend tests.
To try out whether the hooks run or not:
pre-commit run --hook-stage push -a test-backend
pre-commit run --hook-stage push -a test-frontend

Frontend tests work Backend test don't:
pre-commit run --hook-stage push -a test-backend
[WARNING] hook id `test-backend` uses deprecated stage names (push) which will be removed in a future version.  run: `pre-commit migrate-config` to automatically fix this.
[WARNING] hook id `test-frontend` uses deprecated stage names (push) which will be removed in a future version.  run: `pre-commit migrate-config` to automatically fix this.
Run backend tests........................................................Failed
- hook id: test-backend
- exit code: 1

Running gradle task: test
Traceback (most recent call last):
  File "C:\Users\danie\IdeaProjects\corewar\scripts\run_gradle.py", line 29, in <module>
    run_gradle()
  File "C:\Users\danie\IdeaProjects\corewar\scripts\run_gradle.py", line 24, in run_gradle
    subprocess.run("ls")
  File "C:\Users\danie\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 501, in run
    with Popen(*popenargs, **kwargs) as process:
  File "C:\Users\danie\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 969, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\danie\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 1438, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] Das System kann die angegebene Datei nicht finden

Maybe because it tries to run ls which is not available on windows?

I used that to debug and forgot about it being there :sweat_smile:

Now everything seems to work, but there is no console output at all about what is happening. Is it possible to somehow display the test status while they are running and / or the test results after they are finished?

There is now output if the hook fails. This is the default behavior for pre-commit, and I don't think it's easily changed.
But only showing output if the tests fail should be fine, right?


What is the purpose of creating an image for your pipelines?

Not sure if there is a form where I can post this question, but what are the main benefits of creating an image of your pipelines? I noticed in Kabanero 0.6 there are 3 scripts env.sh package.sh and release.sh, in the env.sh if you put your docker username and password it pushes your pipelines to docker hub. I am just wondering why is this needed? And what are the main benefits as opposed to create a release on github?
https://hub.docker.com/repository/docker/yellocabins/pipelines-index
https://github.com/oiricaud/pipelines

The docker image that is being created is one that can serve the pipeline(s) archive.  It is an edge case scenario where, for reasons of your own, you can choose not to have git host and serve the pipeline(s) archive and instead deploy an NGINX container which will serve them.

Thanks!


Lua API error: main:291: attempt to index a non-table object(nil) with key 'capabilities'

Description of the problem or steps to reproduce
When invoking help (Ctrl+G) or command, for example Ctrl+E tree  micro faults
❯ micro
Lua API error: main:291: attempt to index a non-table object(nil) with key 'capabilities'
stack traceback:
	main:291: in main chunk
	[G]: ?

This also happens when I build locally from the main branch.
Specifications

Version: 2.0.11
Commit hash: 225927b9
Compiled on July 24, 2022
OS: macOS Monterey 12.6.1  MBP 14 M1 Pro 32GB
Terminal: iTerm 2 Build 3.4.16

Note that  cat ~/.config/micro/settings.json contains an empty JSON object  {}

I think this is a problem with the lsp plugin, I will try to look into why this is happening.

I have go 1.19.3 installed in case that matters and am running VSCode


fix(deps): update module github.com/stackitcloud/stackit-sdk-go/services/iaas to v0.4.0

This PR contains the following updates:

Package
Type
Update
Change

github.com/stackitcloud/stackit-sdk-go/services/iaas
require
minor
v0.3.0 -> v0.4.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Investigate CGPath Behavior

Investigate how CGPath behaves when modifying the path while attempting to draw the path.

We are not sure what this is, so closing this for now.


Getting workspace name in settings.js

In settings files, I can use ${workspaceRoot} to get full path  of workspace but is that a way to get name of workspace only?

${workspaceRoot}

Please try ${workspaceRootFolderName}


Python unit tests fail with new proto files

In our CI pipeline, we regenerate all sources rather than relying on committed files. This is causing master build failures since this commit: https://github.com/gojek/feast/pull/438
Expected Behavior
Build and tests pass
Current Behavior
Unit test python fails
Steps to reproduce

Re-generate python files with cd protos; make gen-python
Run unit tests with cd sdk/python; pip install -r requirements-ci.txt && pip install -e . && pytest --junitxml=python-sdk-test-report.xml

Specifications
latest master
Possible Solution
Add required dependencies to requirements. tensorflow_metadata?

@Yanson I believe this is resolved. Please let me know if we should reopen.

@woop  Hi, I tried to run the example notebook (basic.ipynb) but I got this error: "ModuleNotFoundError: No module named 'feast.core'" at Step 2 (Import necessary modules)

@woop same for me "ModuleNotFoundError: No module named 'feast.core'" whenever I try to import any feast module.


solve the problem of aggregation query losing precision.

when i use sql tools to execute sql。not lose precision

when i use sql tools to execute sql。not lose precision

Do you have a reproducible example of this issue? Because from my testing I think it is the other way around - if 25095.505999999998 is converted to a string by SQLite then it turns it into 25095.506. The GUI tool you are comparing too might just always use sqlite3_column_text, which is why it produces the wrong answer. You can observe the same issue with the standard sqlite3 CLI by running SELECT 25095.505999999998;.

Here's my code ,I wish to be a contributor
package main

import (
	"database/sql"
	"fmt"
	_ "github.com/mattn/go-sqlite3"
	"log"
	"os"
)

func main() {
	os.Remove("./foo.db")

	db, err := sql.Open("sqlite3", "./foo.db")
	if err != nil {
		log.Fatal(err)
	}
	defer db.Close()

	sqlStmt := `
	create table foo (id integer not null primary key, value text);
	delete from foo;
	`
	_, err = db.Exec(sqlStmt)
	if err != nil {
		log.Printf("%q: %s\n", err, sqlStmt)
		return
	}

	tx, err := db.Begin()
	if err != nil {
		log.Fatal(err)
	}
	stmt, err := tx.Prepare("insert into foo(id, value) values(?, ?)")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()
	fmt.Println("mock data:")
	for i := 0; i < 10; i++ {
		_, err = stmt.Exec(i, "2509.5506")
		if err != nil {
			log.Fatal(err)
		}
	}
	err = tx.Commit()
	if err != nil {
		log.Fatal(err)
	}

	rows, err := db.Query("select id, value from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer rows.Close()
	for rows.Next() {
		var id int
		var name string
		err = rows.Scan(&id, &name)
		if err != nil {
			log.Fatal(err)
		}
		fmt.Println(id, name)
	}
	err = rows.Err()
	if err != nil {
		log.Fatal(err)
	}

	var value string

	fmt.Println("sum error")
	stmt, err = db.Prepare("select sum(value) from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()

	err = stmt.QueryRow().Scan(&value)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Println(value)

	fmt.Println("avg error")
	stmt, err = db.Prepare("select avg(value) from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()
	err = stmt.QueryRow().Scan(&value)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Println(value)

	fmt.Println("max OK")
	stmt, err = db.Prepare("select max(value) from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()
	err = stmt.QueryRow().Scan(&value)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Println(value)

	fmt.Println("min OK")
	stmt, err = db.Prepare("select min(value) from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()
	err = stmt.QueryRow().Scan(&value)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Println(value)

	fmt.Println("count OK")
	stmt, err = db.Prepare("select count(value) from foo")
	if err != nil {
		log.Fatal(err)
	}
	defer stmt.Close()
	err = stmt.QueryRow().Scan(&value)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Println(value)

}

this is execute result

It's my honor to contribute code. thanks a lot

These sorts of issues are par for the course when working with floating point numbers. For example, Go produces the exact same results. https://go.dev/play/p/i1AWFLb7KSv
Thanks for the contribution, but the current code is actually working as expected.

Did you run the example code I gave you?
This is a bug in your packaging of c-sqlite,beacause you use the wrong way!
This cannot be blamed on the precision of the go language itself.
If the result is 25095.50599999998,how can peolple to use?The user does not know that the result of the source is wrong！
Are you make a  joke?

This package is not using SQLite the wrong way. The issue you are experiencing has nothing to do with SQLite in particular, and everything to do with how IEEE standard floating point numbers work. The Go example I linked was meant to show exactly that. I suggest looking more into this topic if you are not familiar with it.
In other words, SQLite itself produces the correct value as per standard floating point arithmetic. This is the value that this library is returning verbatim.
The reason you are getting a different answer from other applications is because they are always converting the floating point result to a string. And there appears to be some peculiarity in SQLite itself here where this process turns 25095.505999999998 into 25095.506. In your specific case you found that peculiarity beneficial as it happened to counteract the round-off issue inherent to floating point arithmetic.
If your application cannot tolerate the nuances of floating point arithmetic, it needs to be dealt with there. You may choose to perform some rounding explicitly, or leverage the aforementioned peculiarity by casting to string, or use fixed point, to name a few possibilities.

What I need to clarify is i am not converting the floating point result to a string.
I was only try get this select sum(value) from db which really value is 25095.506.
The source code here turns the data into 25095.505999999998. In fact, the data stored in the database is 25095.506.
https://github.com/mattn/go-sqlite3/blob/master/sqlite3.go#L1960
Have you carefully looked at the table structure in the sample code I gave?
create table foo (id integer not null primary key, value text)

the type of value is text.Sqlite enable make a sum calucation, if this text value is type of number

foo.tar.gz
This is the database file.you can make a query by your tools.
select sum(value) from foo

the type of value is text

SQLite is flexibly typed, meaning that the declared column type is irrelevant (unless you use strict tables). What actually matters is the type of each individual value. In this case, those are also strings because you wrote stmt.Exec(i, "2509.5506") as opposed to stmt.Exec(i, 2509.5506). See https://sqlite.org/datatype3.html for more information.
That aside, because you are calculating SUM, each of those values must be converted from string to floating point, because that is how SQLite does it. And the actual result of SUM is a floating point number. See https://www.sqlite.org/lang_aggfunc.html#sumunc

If any input to sum() is neither an integer nor a NULL, then sum() returns a floating point value which is an approximation of the mathematical sum.

As I demonstrated, any application that does standard floating point arithmetic to add 25095.506 10 times is going to get 25095.505999999998 as a result.
If you convert this value to a string via sqlite3_column_text, it happens to produce the value "25095.506". I am not really sure why, but my guess is it is due to some implicit rounding. Thus the reason that whatever graphical tool you are using is giving you 25095.506 is because that tool likely is just calling sqlite3_column_text regardless of what SQLite says the type of the value is.


chore(release): Preparing for 3.1.1 release

Coverage decreased (-0.002%) to 99.947% when pulling 938b54242b6ef14b84d88024db64399a1fb6bb3c on asa/release-3-1-1 into 22eafd6086f2fae92295c479ad02167fa0ccd090 on master.


test-branch/execute-job/quoted-authorization/be645a8c-9b4b-4287-a0b0-f1b3fd23a4bb

PR for testing 37a0b47f738d4ab778818c0be4726d33b140d4f8

/canonical/self-hosted-runners/run-workflows 37a0b47f738d4ab778818c0be4726d33b140d4f8


Image picker

New Control Image Picker

@joelfmrodrigues  ready to merge validated

Thanks Joao, I'll review this in the next few days.

@joelfmrodrigues  @michaelmaillot merge the PR validated and tested


node: neo-cli should load config.json from working directory, not binary directory

neo-cli should be loading the config.json file from the current working folder, not the folder where the neo-cli binary is located. This would allow multiple nodes to run on a single machine (such as in a privatenet scenario) without needing multiple folders with neo-cli binaries.
If desired, we can fallback to the config.json from the binary folder if there is no config.json file in the current working folder.

Old, if remains, please re-open


Old setOutput function before @actions/core 1.10.0 is deprecated

Describe the bug
The old setOutput function is deprecated and its use leads to the following deprecation warning:
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

To reproduce
Use the action and observe that calling it produces a deprecation warning.
See https://github.com/Kas-tle/java2bedrock.sh/actions/runs/3247277630 for example.
Expected behaviour
No deprecation warning occurs.
Additional context
This can be remedied by upgrading the @actions/core package to 1.10.0 as described here:
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Thanks, will take care of that with the next update.

Thanks so much!


Momomia logo in different font

The logo of Momomia in the homepage is different from the one in contact us page. I can change it, please assign me the task.

@exotic123567 issue is already resolved


Add a "hotstart" mode to the CLI to start the service, but not analyze

As pulling the container and starting up the service is slow, we should have a CLI mode where we just pull and start the container (and any specified analyzers).

Upgrading to P1, since this will make it possible to do some nicer demos.


Implement Stack (Python)

Implement Stack in Python
We need a simple stack implementation written in Python for our repository.
This issue is great for beginners looking to contribute in open source. If you wish to fix this issue please comment that you are working on it. Please read the readme.md and contributing.md files to learn how to contribute.
If you wish to contribute another algorithm/data structure or the same algorithm in a different language, please see if it hasn't been implemented and issue a new request if no issue has been made.
If you are going to work on the issue yourself, make sure you comment on the issue to let others know.

i would like to work on this. please assign this this task to me.

i have solved the problem and pushed it with branch "stack-python".

@aayushsinha44 Please tell everyone that you are working on it before putting up a pull request. This is your first time, so you are excused. Please take that into consideration from next time

@asterisk30 So sorry but someone seems to have already solved the issue. Can you please take something else?


Full data sync to sqlite

Implement a feature that will sync all data from the ring (steps, heart rate, spo2) into a sqlite database.
The goal is to have a single colmi_r02_client sync database_file.sqlite command that will pull all available data off the ring. It should be smart enough to know when the last sync was so that it can fetch just the new data
TODO:

[ ] pick library for sqlite
[ ] decide if I want to use an ORM
[ ] design the table schema

ORM would be a nice touch!


engine: don't kick off a build while tiltfile execution is pending

Hello @landism, @maiamcc,
Please review the following commits I made in branch nicks/ch1826/manifest:
b12b1646ad721a79ec3b126eea4df5545511995a (2019-03-12 16:10:45 -0400)
engine: don't kick off a build while tiltfile execution is pending

re: moving everything into build controller: ya, possibly? for some definition of "build controller" (it obviously would obviously have to evolve a bit).
i've been thinking a lot about what the internals of other modern graph-based build systems look like (e.g., Bazel, Buck). I would expect BuildController to just pick the next target to build, and dispatch abstract BuildStarted / BuildCompleted events. Then it would have some sort of Handler table that knew how to kick off the builds of each type (image builds vs k8s deploys vs tiltfile runs)


"WhatsApp works with Google Chrome 36+" Issue

The WhatsApp website displays users the message "WhatsApp works with Google Chrome 36+"

I have the same issue.

This is happening with me too. Have anyone found a workaround?

Same issue here. Any update on this?

Same here!

Same here =/

El proyecto tiene muy buena pinta. Una pena que este tan descontinuado y después de meses, sigan sin solucionar este problema con Whatsapp. Franz y Rambox tienen un mejor soporte. Tendrán que ponerse las pilas si quieren competir con ellos.
The project looks very good. A pity that is so discontinued and after months, still not solve this problem with WhatsApp. Franz and Rambox have better support. They will have to put the batteries if they want to compete with them.


No clear information on identity management for managed applications

How does identity management work with respect to managed apps? Are all managed apps given a service principal or system assigned managed identity? How can I reference that identity from within my application definition template?
There really should be a page that lays out how identity management for managed applications works. What identities are used when? What roles do those identities have? What AD administrative roles do they have? The Appliance Resource Provider SP should be included on this page.
As an example problem that this would solve, I want my managed app to run a deployment script on creation. If I don't create a user managed identity to run my deployment script, what identity is used in the OBO flow described in the second "Important" box here? Is it a system-assigned managed identity, or the Appliance Resource Provider SP, is it the user that requested the managed app to be provisioned, or is it somehow something else?

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

ID: 1823c33d-0d6d-7cc3-15e2-721b8941f338
Version Independent ID: 34e77461-b289-0cb9-52fb-321b84048016
Content: Overview of managed applications - Azure Managed Applications
Content Source: articles/azure-resource-manager/managed-applications/overview.md
Service: managed-applications
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@benjamincburns Thanks for your feedback.
Assigning to content author @tfitzmac for review.

#reassign:davidsmatlak

@benjamincburns Thanks for your feedback about the documentation. Unfortunately, your issue wasn't reviewed in a timely manner. Your issue remains open and will be reviewed, and the timeline for resolution may vary based on resourcing. There was a recent change to the content's author and I'm working through the open GitHub issues and updating documentation. I'll post a comment when I'm working on your issue.

Thanks for the acknowledgement @davidsmatlak. I'm no longer affiliated with the project that caused me to open this issue, but @jamesstanleystewart is, and he will likely want to track progress on this issue.

Unfortunately your issue wasn't reviewed in a timely manner. I'm closing this issue, but if you feel that it's still a concern, please post a comment or submit a new issue.
#please-close


Custom fields not created on objects if Django ORM methods are used

Environment

Nautobot version (Docker tag too if applicable): 1.5.10
Python version: 3.8.10
Database platform, version: postgresql, Ubuntu 12.12-0ubuntu<IP_ADDRESS>
Middleware(s): -

Steps to Reproduce

Create a custom field of any type with ("") or without (null) as default value and assign it to the object type interface.
Create a new job and in that job create a new interface for an existing device using Django ORM methods:

nautobot_ip_interface = Interface(
    device=device,
    name=ip_interface_name,
    type=InterfaceTypeChoices.TYPE_VIRTUAL,
    status=status_active,
    label=ip_interface_name,
    mgmt_only=mgmt_only,
    enabled=True,
    untagged_vlan=untagged_vlan,
    mode="access",
)
nautobot_ip_interface.validated_save()

List the custom field keys by calling nautobot_ip_interface.custom_field_data.keys() or navigate to the Device Change Log site in Nautobot and open the interface created entry for the interface, which you have created in step 2. Under Difference note the value Null and the new value (JSON) with the property

 "custom_fields": {
       "your_custom_field": null,
  }

On the right side of this change log page under Object Data the custom field section your_custom_field is missing.

Expected Behavior
All custom fields should be created with default values or null on an object if Django ORM methods are used.

Observed Behavior
None custom fields are created.
If you add the custom field data manually before saving, the field will be created:
nautobot_ip_interface.custom_field_data["your_custom_field"] = ""
nautobot_ip_interface.validated_save()

If pynautobot (REST-API) is used, the custom fields will be created during object creation.
This issue has been discussed here: https://github.com/nautobot/nautobot/discussions/3298#discussioncomment-4960531

@phoenixswiss Do you have a celery worker running? And if so after you save the object can you run a refresh_from_db after waiting a minute after the object save?

@bryanculver Yes, I have celery worker running. Saving the object and waiting 60 seconds and then refreshing the object from db had the same effect.

@bryanculver Yes, I have celery worker running. Saving the object and waiting 60 seconds and then refreshing the object from db had the same effect.

Have you tried this outside a job as well?

@bryanculver Yes, I did test it outside a job:
>>> device = Device.objects.get(name="testdevice")
>>> ip_interface_name = "test"
>>> status_active = Status.objects.get(slug="active")
>>> nautobot_ip_interface = Interface(
...         device=device,
...         name=ip_interface_name,
...         type="virtual",
...         status=status_active,
...         label=ip_interface_name,
...         mgmt_only=False,
...         enabled=True,
...         mode="access",
... )
>>> nautobot_ip_interface.validated_save()
>>> nautobot_ip_interface.custom_field_data.keys()
dict_keys([])
>>> time.sleep(60)
>>> nautobot_ip_interface.refresh_from_db()
>>> nautobot_ip_interface.custom_field_data.keys()
dict_keys([])

Thanks for the extra extra confirmation @phoenixswiss!


feat(state-keeper): circuits seal criterion

What ❔

the number of circuits needed is estimated in VM
replaces ComputationGasCriterion with more precise CircuitsCriterion

Why ❔
Introduce CircuitsCriterion for efficient batch sealing
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.
[x] Spellcheck has been run via cargo spellcheck --cfg=./spellcheck/era.cfg --code 1.

Nice! Most of my comments are nits, but I am a bit worried about lack of tests, it would it be better if we had those (esp. for precompiles)

I'm working on tests. Wanted to get confirmation that the approach makes sense before starting


refactor: sized_box_for_whitespace linter rule

Summary
Use SizedBox to add whitespace to a layout.
A Container is a heavier Widget than a SizedBox, and as bonus, SizedBox has a const constructor.
refactor code to pass following linter rules:

sized_box_for_whitespace

ping

@tenshiAMD if this PR is approved, can you merge it to alpha?


Roadmap v2 / PHPUnit 10

PHPUnit 10 has been released beginning of February.
As this is a major release with breaking changes, upgrading should be a done deliberately and managed.
This effectively means that upgrading to a PHPUnit Polyfills version which supports PHPUnit 10 should also be done deliberately and managed, so the Polyfills should get a new major release.
To this end, the develop branch in this repo will be renamed to 1.x and a new 2.x branch will be added to contain the code for the 2.x series.
The intention is for the 1.x series to continue to be maintained, alongside the 2.x series, for as long as PHPUnit continues to make PHPUnit 9.x compatible with new PHP versions.
Refs:

https://phpunit.de/announcements/phpunit-10.html
https://github.com/sebastianbergmann/phpunit/blob/main/ChangeLog-10.0.md

Proposed Roadmap for PHPUnit Polyfills 2.0 / PHPUnit 10.0 support

[x] Rename the develop branch to 1.x and add a 2.x branch.
[x] Drop support for PHPUnit < 5.7 / PHP < 5.6. - PR #102
[x] Make all assertion methods in the public API final (following in PHPUnit's footsteps). PR #104
[ ] Remove the AssertAttributeHelper which was only intended to buy people a little more time to upgrade to PHPUnit 9.x.
Open question: should the AssertAttributeHelper still remain for the time being ? And if so, for how long ?
[x] Remove the ExpectPHPException Polyfill as this is functionality no longer supported in PHPUnit 10.0. - PR #108
I wouldn't be adverse to adding a PHPErrorHelper class/trait for the time being to buy people a little more time though (which similar to the AssertAttributeHelper would still require test code changes to use, so people will still have to evaluate the tests which use(d) this functionality).
[x] Add a new polyfill for the new assertStringEqualsStringIgnoringLineEndings() and assertStringContainsStringIgnoringLineEndings() assertions. - PR #109
[x] Add a new polyfill for the new assertIsList() assertion. - PR #110
[x] Add a new polyfill for the new PHPUnit 10.1 assertObject[Not]HasProperty() assertions. - PR #116
[ ] Handle the change from TestListener to Event system for the TestListener Polyfill.
[x] Verify the documentation is in line with the changes.
[x] Allow PHPUnit 10.x. - PR #130

Planning
Providing I manage to get everything working, release version 2.x before the end of March 2023.
Request for comments
This roadmap is open for comments and I'd especially like to hear opinions on the AssertAttributeHelper and about a potential PHPErrorHelper.

Nearly there.... just some final touches needed for the test listener implementation.
Also, looks like PHPUnit 10.1 (expected mid-April) is introducing two new assertions assertObjectHasProperty() and assertObjectNotHasProperty(), so would be good to get those polyfilled as well.

Closing as version 2.0.0 has been released.
A separate ticket is now open to track the PHPUnit 10 TestListener implementation: #128.


View cleanup

Hi,
I believe that views 'onRemove' and '_onRemove' functions are being called twice on clean up. If you look here:
https://github.com/lazojs/lazo/blob/master/lib/client/execute.js#L23 'cleanupView' calls both remove functions, then on the next line the flexo view base class also calls both functions again.

Fixed in v2.x.x-alpha.


Feature/#103 publish payload convenience method byte array

Why are Davids commits part of this PR? They are already merged into develop.
Try rebasing on develop.


Update sigs.k8s.io/controller-tools from 0.9.2 to 0.10.0 and regenerate bundles

What type of PR is this?

What this PR does / why we need it:
This is pretty much PR#1181 except I also regenrated the manifests and the bundle to make CI happy.

build(deps): bump sigs.k8s.io/controller-tools from 0.9.2 to 0.10.0
update bundle after controller-gen bump

Which issue(s) this PR fixes:
None

Does this PR have test?
N/A

Special notes for your reviewer:
N/A
Does this PR introduce a user-facing change?

NONE

Codecov Report

Merging #1195 (a724553) into main (e2286b3) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1195   +/-   ##
=======================================
  Coverage   47.51%   47.51%
=======================================
  Files          44       44
  Lines        4927     4927
=======================================
  Hits         2341     2341
  Misses       2487     2487
  Partials       99       99

/lgtm


UWP Support?

Any chance to get support for UWP together with the TypeScript support for some total cross platform integration in ionic 2?
https://github.com/webrtc-uwp/webrtc-uwp-sdk

Its nothing we got planned. We do have a consulting/contracting arm if you need it and have money, but as it stands, using cordova means less fuss with native libraries and is feature-full enough for most applications.


tab-rs: mark broken on aarch64-darwin

Description of changes
ZHF: #265948
https://hydra.nixos.org/build/239218160
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

LGTM


dose this package support zebra auto-scanning devices?

Hi ,
thanks for this package, i want to ask if this package support zebra auto-scanning ?

Hello  everyone,
I want to let you know that I currently do not have access to a Zebra scanner device. However, I've developed a Zebra scanner integration and made it available publicly here.
I encourage you to explore the project, contribute, and help enhance its functionality. Your contributions will not only help improve the project but also benefit other developers in the community.
Thank you for your support and collaboration!
Best regards,
Sanket Warule


feature: support get model distinct field

这个 PR 实现什么功能/修复什么问题:
feature: support get model distinct field
是否需要 backport 到之前的 release 分支:
release/2.10.0
/cc @swordqiu
/area region

Codecov Report

Merging #2060 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2060      +/-   ##
=========================================
- Coverage    7.18%   7.17%   -0.02%
=========================================
  Files         445     445
  Lines       69397   69424      +27
=========================================
- Hits         4987    4980       -7
- Misses      63999   64030      +31
- Partials      411     414       +3

Flag
Coverage Δ

#aFlag
7.17% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/cloudcommon/db/modelbase.go
1.95% <0%> (-0.3%)
:arrow_down:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/appsrv/response.go
66.15% <0%> (-3.08%)
:arrow_down:

pkg/util/rbacutils/rbac.go
73.37% <0%> (-0.93%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e91552...428da62. Read the comment docs.

/hold

/hold cancel

/hold

/hold cancel

/lgtm

/approve


Cannot change cursor during dragging

Hello,
sorry but I need your helps, I need to change the dragging cursor (the 'copy' cursor, the arrow with the green + button).
I can do it by changing the ".-- dragging-event" class, but the original cursor is reverted when i move the event in another column days.
How can I get my cursor to always be visible when dragging?
Thanks

Hi @Hellense, please provide a reproduction example so we can help you. Thank you.

Hello,
I need to change the cursor that is seen when dragging an event.
I would like to modify the cursor + with the grab.
I've tried with css
---> this was my last desperate attempt:
.--dragging-event,
.--dragging-event *,
.vuecal__event--static,
.vuecal__event--static *,
.vuecal__event--static > *,
.vuecal__event--dragging,
.vuecal__event--dragging *,
.vuecal__event--dragging > * {
cursor: grabbing !important;
}
.my-event[draggable="true"]:hover {
cursor: grab;
}
}
.dragging-helper {
opacity: 0 !important;
}

I see.
If you inspect the source, the cursor is already a grab, but sometimes while dragging starts, the browser overrides the cursor with its own default for dragging.
As far as I know, this is not something we can handle from within Vue Cal. 🤷♂
https://user-images.githubusercontent.com/22971289/188729279-1a7edc82-0189-4f2e-8fd6-321881e05be7.mov

I'm also having the same issue here. If anyone would have any ideas on how to use a different cursor during the dragging phase, that would be great!


New zoom effect for product gallery images.

What is the motivation for adding / enhancing this feature?
Someone wrote that product detail image zoom functionality is not ux friendly.
We could change the zoom so that users can see a larger zoom, more details of the image, using a magnifier icon. See example:

What are the acceptance criteria
User should see every details of product.
To displaying zoomed image just hover mouse on image.

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[x] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[x] This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[ ] (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information

I'll have a go at this !


[feat] column type aware reconciler

Reconciler should aware of column type during comparison and apply column cast (implicit or explicit) when needed.
For example comparing integer vs varchar, reconciler should cast integer to varchar first and compare, the result json could indicate as casted.
Similar to profiler setup, Reconciler should have column type specific compare function and supports threshold configs

varchar: =
integer: base - target = 0
datetime: datediff('time_interval', base - target) = 0

Done in #8


🎮 Game Request
Game logic and basic description
Blackjack is a popular casino card game with simple rules and strategic depth. Here’s an overview of the game logic and how it’s typically played:
Objective
The goal of blackjack is to beat the dealer by having a hand value closest to 21 without exceeding it.
Card Values
Number cards (2-10): Face value.
Face cards (J, Q, K): 10 points each.
Aces: Can be either 1 point or 11 points, depending on which value is more favorable for the hand.
Game Setup
Deck: Blackjack is usually played with one to eight decks of 52 cards.
Players: One or more players compete against the dealer.
Initial Deal: Each player and the dealer are dealt two cards. Players' cards are typically dealt face up, while the dealer has one card
face up (the "upcard") and one card face down (the "hole card").
Point down the features
Game points
Card Values
Number Cards (2-10): These cards are worth their face value.
Example: A 5 is worth 5 points.
Face Cards (J, Q, K): These cards are each worth 10 points.
Example: A Queen is worth 10 points.
Aces (A): Aces can be worth either 1 point or 11 points, depending on which value benefits the hand most without causing the hand to bust (exceed 21 points).
Example: An Ace can be worth 11 points if it doesn’t cause the hand to exceed 21; otherwise, it’s worth 1 point.
Example Hands and Their Point Values
Hand: 5♣, 7♦
Total Points: 5 + 7 = 12
Hand: K♠, 9♥
Total Points: 10 (for the King) + 9 = 19
Hand: A♠, 8♦
Total Points: 11 (for the Ace) + 8 = 19
Hand: A♣, 7♠, 3♦
Total Points: Ace can be 11 or 1. If 11, total is 21 (11 + 7 + 3). If 1, total is 11 (1 + 7 + 3), but 21 is the preferred value in this case.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @codewithishu , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#3447

Hey @codewithishu ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @codewithishu !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Update: Added MinuteMedia logo to users list

Summary
Added the MinuteMedia logo to the list of companies that use Jest in their testing process

Hi @kingerez, thanks for the PR!
Because a lot of companies use Jest and we can't list them all, we've put just a few very well-known ones on the website. So I think at the moment we'll only add more if they're a very big deal, sorry :|

Hey @jeysal
Thanks for letting me know...


Backport $maxNormalizeItemCount et al. from master to 1.x.

Please consider backporting NormalizerFormatter's $maxNormalizeItemCount et al. from master to 1.x. and refreshing composer package.
TIA,
--Vladimir

This won't be backported sorry, we are moving forward to try and get a release out both for 1.x and 2.x.


Add new infiniteScrollReset parameter to allow for easier list updates

This parameter is needed if you want to reuse the same infinite scroll directive and your items will completely change. A use case is a file browser where you navigate the folder structure and you want all the levels of the hierarchy to benefit from infinite scroll. Without a reset such as this, as soon as you scroll to the bottom of the list, subsequent lists that are not bigger than that will no longer have infinite scroll.
You can call the reset by adding an event listener and setting the reset to true. e.g.
<div infinite-scroll="addMoreItems()" infinite-scroll-listen-for-event="list:changed" infinite-scroll-reset="true">
  [...]
</div>

@tricinel please add tests and documentation for this change, and squash + rebase.


skip entities if location returns multiple similar entities.

previously if a location returned a whole lot of entities some
of which were duplicated, then the whole batch were failing.
this changes to to instead log an error where there are duplicates
and continue to load the remaining good enities.

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4720-ohaav3cjjldavkg29iahjhtz55msxn93.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/60391ef5fcb22d277161bd0c

Tugboat has finished building the preview for this pull request!
Link:

https://pr4720-ohaav3cjjldavkg29iahjhtz55msxn93.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/60391ef5fcb22d277161bd0c

Tugboat has finished building the preview for this pull request!
Link:

https://pr4720-ohaav3cjjldavkg29iahjhtz55msxn93.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/60391ef5fcb22d277161bd0c

This sounds very dangerous, and can lead to a "takeover" where a near-duplicate of an entity is put in place by a malicious user, so that the new entity "competes" for the slot taken by the old one.
The actual problem, rather, may be that the batch writes fail at a too large granularity. There should be logic that when a batch write fails, it instead tries to write the individual entities one by one. Right?
@Rugvip foe visibility

@freben thanks for the feedback. I wonder if the batch sizes were smaller, and if the first duplicate makes it into into an earlier batch, then it might succeed and the second might fail.
How can it be known which of the duplicates is valid?
I may be misundering the context fully. But I wonder, in the absence of knowing which is the true valid entity,  would it be safer to reject all the duplicates.

Discussed this a bit more, and thinking that it's likely best to skip this for now and instead consider it as part of the upcoming catalog work. We'll be splitting the processing up into much smaller chunks, so the reasoning around this will likely get a bit clearer. This change is also a lil bit scary as mentioned above.
Do you want to create an issue for this instead and we'll add it to https://github.com/backstage/backstage/milestone/23 ?

@punkle Thanks! 👍 , added to the milestone


Key press only registered when skhd --profile is running

I'm running macOS Sonoma Version 14.2 and skhd (seems) to only register key presses when skhd --profile is running.
Anybody experiencing similar issues?
Running skhd --profile is a workaround that I'm happy to live with given the value of skhd.
$ system_profiler SPSoftwareDataType SPHardwareDataType
Software:

    System Software Overview:

      System Version: macOS 14.2 (23C64)
      Kernel Version: Darwin 23.2.0
      Boot Volume: …
      Boot Mode: Normal
      Computer Name: …
      User Name: …
      Secure Virtual Memory: Enabled
      System Integrity Protection: Disabled
      Time since boot: 35 minutes, 24 seconds
Hardware:

    Hardware Overview:

      Model Name: MacBook Pro
      Model Identifier: MacBookPro16,1
      Processor Name: 8-Core Intel Core i9
      Processor Speed: 2,3 GHz
      Number of Processors: 1
      Total Number of Cores: 8
      L2 Cache (per Core): 256 KB
      L3 Cache: 16 MB
      Hyper-Threading Technology: Enabled
      Memory: 32 GB
      System Firmware Version: 2<IP_ADDRESS>.0 (iBridge: 21.16.2057.0.0,0)
      OS Loader Version: 580~1053
      Serial Number (system): C02DD3E0MD6T
      Hardware UUID: 6F015C4A-A3F9-59F4-8428-C8B0305B5F8E
      Provisioning UDID: 6F015C4A-A3F9-59F4-8428-C8B0305B5F8E
      Activation Lock Status: Disabled

$ skhd --version
skhd-v0.3.9

Hi @rimi-itk , what was your solution to this? I see you marked this as completed without additional context. I'm facing the same problem so would appreciate your input. Thank you.

Hi @zoominGiladSher, I found out that just running skhd, i.e. with --profile worked, and then running skhd via the service started working. Therefore, I closed this issue, and in my excitement forgot to tell why I closed it. Sorry about that.
However, the skhd service has just stopped working on my machine, so now I'm back to running skhd in a terminal to enjoy it.

Thanks for your answer @rimi-itk, I'm facing the same issue, unfortunately. Thanks for letting me know!