Remove void parameter from onConnect, Swift4 will would require passi…

…ng () into it
@daltoniam Noticed in the Swift 4 preview that this would require passing () into the onCallback callback, which is a bit funky.

This is is backwards compatible.

Agreed. I haven't look at an actual Swift 4 snapshot yet, probably should soonish 😄. Thanks!


Cannot edit shared documents after https enabled

Hi all,
I have recently tried ONLYOFFICE Community Server coupled with ONLYOFFICE Document Server installed onto two separete CentOS7 VMs. as far as there is not any https redirection everything works as expected: you can edit any of your document as well as any shared one.
For security reasons I had to move to https, so I changed the conf files as follows:
## For Community Server
upstream fastcgi_backend {
	server unix:/var/run/onlyoffice/onlyoffice.socket;
	server unix:/var/run/onlyoffice/onlyoffice2.socket;
	keepalive 64;
}

fastcgi_cache_path /var/cache/nginx/onlyoffice
	levels=1:2
	keys_zone=onlyoffice:16m
	max_size=256m
	inactive=1d;

server {
	listen 80;
	server_name localhost;
	return 301 https://$host$request_uri;
}

server {
   listen 443 ssl;
   server_name localhost;
   server_tokens off;

   ssl on;
   ssl_certificate /var/www/onlyoffice/Data/certs/onlyoffice.crt;
   ssl_certificate_key /var/www/onlyoffice/Data/certs/onlyoffice.key;

   ssl_session_timeout 5m;

   ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
   ssl_ciphers "EECDH+AESGCM:EDH+AESGCM:ECDHE-RSA-AES128-GCM-SHA256:AES256+EECDH:DHE-RSA-AES128-GCM-SHA256:AES256+EDH:ECDHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA:ECDHE-RSA-AES128-SHA:DHE-RSA-AES256-SHA256:DHE-RSA-AES128-SHA256:DHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA:ECDHE-RSA-DES-CBC3-SHA:EDH-RSA-DES-CBC3-SHA:AES256-GCM-SHA384:AES128-GCM-SHA256:AES256-SHA256:AES128-SHA256:AES256-SHA:AES128-SHA:DES-CBC3-SHA:HIGH:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
   ssl_prefer_server_ciphers on;

   	location / {
		root    /var/www/onlyoffice/WebStudio/;
		...
		...
		...
	}
	location /filesData {
		rewrite /filesData(.*) /$1  break;
		root /;
		internal;
	}
}

upstream fastcgi_backend_apisystem {
	server unix:/var/run/onlyoffice/onlyofficeApiSystem.socket;
	keepalive 32;
}
server {
	listen 8080;

	location / {
		root    /var/www/onlyoffice/ApiSystem/;
		...
		...
		...
	}
}

and
## For Document Server

include /etc/nginx/includes/onlyoffice-http.conf;

## Normal HTTP host
server {
	## http://nginx.org/en/docs/http/ngx_http_core_module.html
	listen 80;
	listen [::]:80 default_server;
	server_name localhost;
	server_tokens off;
	return 301 https://$host$request_uri;	## redirect to https and return http code 301
}

## HTTPS host
server {
	listen 443 ssl;
	listen [::]:443 ssl default_server;
	server_tokens off;
##	root /usr/share/nginx/html;

	## Increase this if you want to upload large attachments
##	client_max_body_size 100m;

	## Strong SSL Security
	## https://raymii.org/s/tutorials/Strong_SSL_Security_On_nginx.html
	ssl on;
	ssl_certificate /var/www/onlyoffice/Data/certs/onlyoffice.crt;
	ssl_certificate_key /var/www/onlyoffice/Data/certs/onlyoffice.key;

	ssl_ciphers "EECDH+AESGCM:EDH+AESGCM:ECDHE-RSA-AES128-GCM-SHA256:AES256+EECDH:DHE-RSA-AES128-GCM-SHA256:AES256+EDH:ECDHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA:ECDHE-RSA-AES128-SHA:DHE-RSA-AES256-SHA256:DHE-RSA-AES128-SHA256:DHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA:ECDHE-RSA-DES-CBC3-SHA:EDH-RSA-DES-CBC3-SHA:AES256-GCM-SHA384:AES128-GCM-SHA256:AES256-SHA256:AES128-SHA256:AES256-SHA:AES128-SHA:DES-CBC3-SHA:HIGH:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";

	ssl_protocols  TLSv1 TLSv1.1 TLSv1.2;
	ssl_session_cache  builtin:1000  shared:SSL:10m;
	ssl_prefer_server_ciphers   on;

	add_header Strict-Transport-Security max-age=31536000;
	add_header X-Content-Type-Options nosniff;

	ssl_dhparam /var/www/onlyoffice/Data/certs/dhparam.pem;

	include /etc/nginx/includes/onlyoffice-documentserver-*.conf;
}

Where ellipsis mean that the content of the location is the same as for the default conf files.
After these changes everything worked as expected, but the editing of shared documents, for which I got this error message:

I also tried to redirect api from port 8080 to 8443 modifying the last part of Community Server conf file as follows
[...]
server {
        listen 8080;
        return 301 https://$host:8443$request_uri;
}

server{
        listen 8443 ssl;
        server_name localhost;
        server_tokens off;

        ssl on;
        ssl_certificate /var/www/onlyoffice/Data/certs/onlyoffice.crt;
        ssl_certificate_key /var/www/onlyoffice/Data/certs/onlyoffice.key;

        ssl_session_timeout 5m;

        ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
        ssl_ciphers "EECDH+AESGCM:EDH+AESGCM:ECDHE-RSA-AES128-GCM-SHA256:AES256+EECDH:DHE-RSA-AES128-GCM-SHA256:AES256+EDH:ECDHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA:ECDHE-RSA-AES128-SHA:DHE-RSA-AES256-SHA256:DHE-RSA-AES128-SHA256:DHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA:ECDHE-RSA-DES-CBC3-SHA:EDH-RSA-DES-CBC3-SHA:AES256-GCM-SHA384:AES128-GCM-SHA256:AES256-SHA256:AES128-SHA256:AES256-SHA:AES128-SHA:DES-CBC3-SHA:HIGH:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
        ssl_prefer_server_ciphers on;

        location / {
                root    /var/www/onlyoffice/ApiSystem/;
                ...
                ...
        }

With this change, no documents were editable any more.
Can anyone help me?
Thanks in advance.

NOTE:
Since I was not sure about https://$host:8443$request_uri, I also tried with $host:8443$request_uri. But this didn't change the behavior.

Hello!
Here you may see a detailed insctruction on configuring nginx for switching Document Server to https.

Does this mean that I need an https proxy and that a simple https redirect is not enough?

You can use installation script http://bit.ly/2frc3Vh that allows you to put not just Docker, but in deb/rpm packages.
use
/var/www/onlyoffice/Tools/letsencrypt.sh yourdomain.com
you can generate certificates
/etc/nginx/includes/onlyoffice-communityserver-common-ssl.conf.template
contains the default template to configure a https connection.

If you install on different machines, check the correct connection settings document editors in portal settings "Setting->Integration->Document Service".

I do all with https://helpcenter.onlyoffice.com/installation/docs-community-install-docker.aspx
but had error when open example document,

i set ssl config :

and run command is :

Hello @little3201 You need to open a new issue in DocumentServer repo.


New Test Open Badges 3.0

Please create some additional new test badges to support these issues:

https://github.com/digitalcredentials/learner-credential-wallet/issues/460
https://github.com/digitalcredentials/learner-credential-wallet/issues/456

Note that one of them pertains to displaying markdown in criteria.narrative. It would be helpful if one or two test credentials contained some markdown with various styling (bullets, bold, etc) and URLs.
Also, newly added to display is Achievment.AchievementType.

Here are several credentials that Atomic Jolt can start to use: credentials.zip. Each subfolder name describes the content therein. I have tried to include as many fields as possible (e.g., expirationDate, revoked) and I have also include a number of examples of markdown. Let me know if you need more samples.

Thanks @kezike - would you please create an "example-credentials" folder at the top of: https://github.com/digitalcredentials/docs
And add the subfolders in there?

@kayaelle Here is a PR that adds the credentials to the docs repo.

Thanks @kezike. Going to follow-up soon with issues to create a few more examples specific to upcoming pilot on degrees and for PDF printing.


Tools icons not uploading

[ ] Ask to be assigned
[ ] Icons of the different tools on the toolbar are not showing
[ ] Cursor of the respective tools are not changing

Can I work on this?

ok am start working on this issue


The leader should receive ADDITIONAL compensation to cover gas costs

we have with a very “loose” gas fee structure, but need to add the logic of compensate gas cost for leader.

found where --fees flag is when node started

exists as a flag


Open by Default

if you set Colombo as your default browser and you open a link from an email it just launches the homepage i just wanted to bring that to your attention i'm currently having a look at trying to fix it

I know, is too annoying


Answer an FAQ: How to use Greek letters?

I have added a FAQ section to the user guide. Your assistance with this is greatly appreciated. @LabAsim #63

there is an example with greek letters but there is only use fontspec so no workaround for pdflatex : https://www.overleaf.com/learn/latex/Multilingual_typesetting_on_Overleaf_using_polyglossia_and_fontspec

It's been a while since I have typeset some serious documents but my go-to solution for multilingual source documents (and system font usage) has always been XeLaTeX (brief comparison). Now there seems to be LuaLaTeX, too.


Could not find a version that satisfies the requirement gpt-code-assistant

Hi,
Just wanted to try out "gpt-code-assistant" but it seems not published on the pip registry whereas it's mentioned in the setup step https://github.com/narenmanoharan/gpt-code-assistant#installation . Did I miss something?
Note that on the other side, I do see https://pypi.org/project/gpt-code-search/ (by the way I tried this older version but not working at all... always getting the error "Dictionary values must be serializeable to JSON" even on your own codebase).
Thank you,
 ~  pip install gpt-code-assistant                                                                                 INT ✘  10:35:59
ERROR: Could not find a version that satisfies the requirement gpt-code-assistant (from versions: none)
ERROR: No matching distribution found for gpt-code-assistant

EDIT: my workaround for now is to: poetry build && pip install aaa.whl && gpt-code-assistant ...

Sorry about that! I just published the package on pypi - https://pypi.org/project/gpt-code-assistant/
Looks like I skipped that step 😓


Starting Vitis-AI docker

docker: Error response from daemon: error while creating mount source path '/opt/xilinx/dsa': mkdir /opt/xilinx: read-only file system.

Hi @nevilshah235 ,
Can you check that the user account that you use to run Vitis AI docker has write permissions to /opt/xilinx ?

Hi @hanxue
Thank you for the support. I tried running it with sudo as well. But still I get the same error.

Hi @nevilshah235 ,
Hmm, the error seems to be no write permission to /opt/xilinx, or /opt. Could you do me a favour and paste the output of the following commands
df -hTl
ls -ld /opt/xilinx
ls -ld /opt

?

Sure.

nevil@nevil-ASUS-TUF-Gaming-A17-FA706IH-FA706IH:~$ df -hTl
Filesystem     Type      Size  Used Avail Use% Mounted on
udev           devtmpfs  3.6G     0  3.6G   0% /dev
tmpfs          tmpfs     744M  2.4M  742M   1% /run
/dev/nvme0n1p9 ext4      288G  142G  132G  52% /
tmpfs          tmpfs     3.7G   70M  3.6G   2% /dev/shm
tmpfs          tmpfs     5.0M  4.0K  5.0M   1% /run/lock
tmpfs          tmpfs     3.7G     0  3.7G   0% /sys/fs/cgroup
/dev/loop0     squashfs  100M  100M     0 100% /snap/core/10859
/dev/loop2     squashfs   56M   56M     0 100% /snap/core18/1988
/dev/loop3     squashfs  151M  151M     0 100% /snap/okular/109
/dev/loop4     squashfs  132M  132M     0 100% /snap/docker/796
/dev/loop5     squashfs  9.2M  9.2M     0 100% /snap/canonical-livepatch/95
/dev/loop6     squashfs   65M   65M     0 100% /snap/gtk-common-themes/1514
/dev/loop1     squashfs  100M  100M     0 100% /snap/core/10908
/dev/loop7     squashfs  219M  219M     0 100% /snap/gnome-3-34-1804/66
/dev/loop9     squashfs  163M  163M     0 100% /snap/gnome-3-28-1804/145
/dev/loop8     squashfs  324M  324M     0 100% /snap/kde-frameworks-5-qt-5-15-core20/14
/dev/loop10    squashfs  165M  165M     0 100% /snap/signal-desktop/351
/dev/loop11    squashfs  146M  146M     0 100% /snap/slack/37
/dev/loop12    squashfs  426M  426M     0 100% /snap/pycharm-community/232
/dev/nvme0n1p1 vfat      256M   34M  223M  13% /boot/efi
/dev/loop13    squashfs   33M   33M     0 100% /snap/snapd/11402
/dev/loop14    squashfs  163M  163M     0 100% /snap/gitkraken/172
/dev/loop15    squashfs   62M   62M     0 100% /snap/core20/904
/dev/loop16    squashfs  425M  425M     0 100% /snap/pycharm-community/226
/dev/loop17    squashfs   52M   52M     0 100% /snap/snap-store/518
/dev/loop18    squashfs   33M   33M     0 100% /snap/snapd/11107
tmpfs          tmpfs     744M   28K  744M   1% /run/user/125
tmpfs          tmpfs     744M  4.3M  740M   1% /run/user/1000
/dev/loop19    squashfs  147M  147M     0 100% /snap/slack/38
nevil@nevil-ASUS-TUF-Gaming-A17-FA706IH-FA706IH:~$ ls -ld /opt/xilinx
drwxr-xr-x 3 root root 4096 Mar 24 16:36 /opt/xilinx
nevil@nevil-ASUS-TUF-Gaming-A17-FA706IH-FA706IH:~$ ls -ld /opt
drwxr-xr-x 5 root root 4096 Mar 25 18:46 /opt
nevil@nevil-ASUS-TUF-Gaming-A17-FA706IH-FA706IH:~$

The /opt directory looks OK. Can you please list out the contents of
ls -l /opt/xilinx

please?
Just to ensure the problem is due to this directory, temporarily delete this line in your local copy of docker_run.sh https://github.com/Xilinx/Vitis-AI/blob/master/docker_run.sh#L87
    -v /opt/xilinx/dsa:/opt/xilinx/dsa \

A quick question. Are you by any chance running Docker inside a Mac or Windows machine?
We have seen weird mount issues like these, and at the moment Vitis AI is only supported on a Linux host with Docker.

Hi @hanxue
I also deleted line no. 88 in my local installation of docker_run.sh
-v /opt/xilinx/overlaybins:/opt/xilinx/overlaybins \
After that I was able to successfully run the docker_run.sh script
I am using an Ubuntu 20.04 OS.

Sharing the output of ls -l /opt/xilinx
I had created the directory dsa, to check whether it works after I include that directory, but same issue is there

Which workflow are you using? If it is Whole App Acceleration or Alveo related, you will need to have XRT installed on your host. Download it from https://www.xilinx.com/products/design-tools/vitis/xrt.html#gettingstarted
This version should work
https://www.xilinx.com/bin/public/openDownload?filename=xrt_202<IP_ADDRESS>7_18.04-amd64-xrt.deb
You might want to delete /opt/xilinx/dsa that you created prior to installing XRT.

Thanks for the fix suggestion. I am unable to install XRT library on Ubuntu
20.04, due to problems with prerequisite libraries.
The packages required have different version numbers. So I had not
installed XRT earlier, still facing the same issue.
On Fri, Mar 26, 2021 at 6:04 PM Hanxue @.***> wrote:

Which workflow are you using? If it is Whole App Acceleration or Alveo
related, you will need to have XRT installed on your host. Download it from
https://www.xilinx.com/products/design-tools/vitis/xrt.html#gettingstarted
This version should work
https://www.xilinx.com/bin/public/openDownload?filename=xrt_202<IP_ADDRESS>7_18.04-amd64-xrt.deb
You might want to delete /opt/xilinx/dsa that you created prior to
installing XRT.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Xilinx/Vitis-AI/issues/347#issuecomment-808180759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDTXDSVIPRTIEJJAEE7THDTFR5MVANCNFSM4ZXA4LUQ
.

There are a couple of options for installing XRT for Ubuntu 20.04:
Download from https://www.xilinx.com/bin/public/openDownload?filename=xrt_202<IP_ADDRESS>7_20.04-amd64-xrt.deb
Build and install XRT from source per the instructions https://xilinx.github.io/XRT/2020.1/html/build.html  . i.e. Download the XRT 2020.2 official release from https://codeload.github.com/Xilinx/XRT/tar.gz/refs/tags/202<IP_ADDRESS>3 , and within the XRT source directory, there is a src/runtime_src/tools/scripts/xrtdeps.sh script that will install all the required tools.

Thanks a ton for the support. I installed XRT for Ubuntu 20.04, and now the Vitis-AI docker installation is working perfectly fine.

Hi @hanxue
I installed XRT, but it's still not working. Earlier when I had tested, I forgot to add
-v /opt/xilinx/dsa:/opt/xilinx/dsa
-v /opt/xilinx/overlaybins:/opt/xilinx/overlaybins
in the docker_run.sh script, so the installation worked fine. But I am still facing the same issue as mentioned earlier in starting VITIS-AI docker.

Hi @nevilshah235 ,
What is the exact error you are seeing? Is it the following?
docker: Error response from daemon: error while creating mount source path '/opt/xilinx/dsa': mkdir /opt/xilinx: read-only file system.

At this point , it is easier to verify if there is something wrong with the Docker installation

Trying running Vitis AI docker on another machine, even if there is no FPGA and XRT installed
Reinstall Docker completely on your complete machine

back up any Docker images, if you need to preserve them
uninstall Docker - sudo apt-get remove docker docker.io docker-cli containerd containerd.io docker-engine runc
Remove all local docker files - sudo rm -Rf /var/lib/docker
Reinstall docker again

I did some googling and and notice this issue is quite common if Docker is installed via snap.
How did you install Docker?
This should fix the snap installation
snap remove docker

Then follow the official instructions for installing Docker https://docs.docker.com/engine/install/ubuntu/

Thanks @hanxue. The issue was with the docker installation, once I reinstalled docker, it worked out fine.


translate debug authorization

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

Needs approval from an approver in each of these files:

content_zh/OWNERS

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

Hi @xionchen. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

感谢review，问题已经修改

/lgtm

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

Needs approval from an approver in each of these files:

content_zh/OWNERS [rootsongjc]

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


Updated to fix SUDOERS not root

Cleaned up to change sudoers issue

Kind of a duplicate of #218,
Just edits the lines instead of adding new ones to support older versions of sudo.

Your absolutely right... i thought I was in my fork.. srry... just trying to get this back working...
I 86'd my install i did last month and can't get this latest version to boot into kde or console...
—
Ty Carter
Strategic Network Consultants Inc
524 E. 9th
Washington, NC. 27888
252-946-0351
On Feb 9, 2022, at 11:33 AM, Code-Ac @.***> wrote:

Kind of a duplicate of #218https://github.com/ChrisTitusTech/ArchTitus/pull/218,
Just edits the lines instead of adding new ones to support older versions of sudo.
—
Reply to this email directly, view it on GitHubhttps://github.com/ChrisTitusTech/ArchTitus/pull/224#issuecomment-1033955617, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAATGJOFZLM3ZAJBFKDIRULU2KJNDANCNFSM5N6ACMYA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Accidental duplicate of an previous bug ID that I didn't catch and has been addressed but still not combined.  Sorry about the duplication.


fix: timestamps caching error

default timestamp cached object to {}
fix error for missing dispute in store
add getParties to Multiple Arbitrable Tx (this may not be needed since aliases is used instead anyways)

Pull Request Test Coverage Report for Build 867

2 of 5 (40.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 49.967%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/contracts/implementations/arbitrable/MultipleArbitrableTransaction.js
0
3
0.0%

Totals

Change from base Build 865:
-0.1%

Covered Lines:
598

Relevant Lines:
1085

💛  - Coveralls


Cleaned-up readme

Image resource no longer exists + improved formatting

Thank you @blrhc! When I first saw this PR I thought it was a frivolous change, but now I see spammers have replaced that original book image with a hosting image and you were fixing that! Next time please do leave a comment about so I don't miss it. Thanks again for catching that and taking the time to fix. I'll get this merged today.


have way to delete test runs

Testing microsoft/vscode-python#22081
This gets pretty busy, would be nice to have an X button to delete past runs

The 🗑 in the top right corner of the view does that


Screenshots not working in Windows release

Any tips to resolve this problem? Even if i setting bid delay, screenshots don't creating and im getting screenshot timed out. :(

I unfortunately don't have access to a windows machine to debug on. Are you using the latest version of Chromium?

I run aquatone on Mac and don't have any problems there. It should also work properly on Linux, so you might want to try running aquatone in a virtual machine, e.g. Kali linux.

UPD: I've figured it out on linux. Fuck. I've mistaked chromium-bsu(2d game) with the real browser. LOL

Maybe im doing something wrong? My command is
type 1.txt | aquatone.exe -chrome-path "chrome-win/chromium.exe"
But getting this "screenshot timed out". Chromium and chrome is the last version

Its working for me in windows 10.
Try to give the full path to the chromium :
type 1.txt | aquatone.exe -chrome-path "C:\path\to\chromium.exe"

Its working for me in windows 10.
Try to give the full path to the chromium :
type 1.txt | aquatone.exe -chrome-path "C:\path\to\chromium.exe"

What version of chromium you are using?  Give me the link where you downloaded your chromium

https://chromium.woolyss.com/ This chromium was made by third-party developers.
I recommend using this : https://www.chromium.org/getting-involved/download-chromium

@stueotue I did give the full path, though :/
Gonna try it again and let you know

I get similar issue in december 2018 but when i give the full path to chrome.exe its working. But now i try using aquatone again in my windows with cmd like this :
type 1.txt | aquatone.exe -chrome-path "C:\path\to\chromium.exe" and the screenshots not working.
However when i use my batch file for recon it works, here is the batch file https://github.com/stueotue/personal-recon and you need to change the variable if you want to use it.
I really need to know what is happening.
SS : https://imgur.com/a/fdgH7Tn

Can confirm that this is still an issue on Windows 10 with the latest available binary:
aquatone_windows_amd64_1.4.3
Also tried ridiculously long timeouts as well as I thought that may have been the issue, as well as running everything with administrative privs incase that was somehow causing the lack of screenshots.

Still got the same issue with latest release 1.7.0

@stueotue your batch didn't working with latest version :c Or its working with you?

Can reproduce this too with aquatone_windows_amd64_1.7.0.zip. I've also tried different chromium releases, all with the same effect

For me everything works fine with 1.7.0! Be sure to pass the path parameters in the correct way especially backslash (I think this could the reason for your command not working @shadowzoom):
type C:\Users\Bla\urls.txt | C:\Users\Bla\aquatone.exe -out C:\Users\Bla\aquatone-report\ -chrome-path C:\Users\Bla\Chromium\Application\chrome.exe

@Max-Trouble
This is my command:
type .\domain_result.txt | ..\..\aquatone\aquatone.exe -out report\ -chrome-path C:\Users\fuomag9\Desktop\chromium\chrome.exe

I still get the screenshot timed out error, while on my ubuntu vps the screenshot works perfectly

@fuomag9 Thats strange, your command looks right.
Have you tried this bat script with fixed path? https://github.com/stueotue/personal-recon

any update?

hey @shadowzoom  can you please describe how did you fix that problem of screenshots not storing to a file? I am having the same problem.

It's still not working on windows
I tested chromium v.727690

This is because you're probably using powershell to execute the command as you can use cat file.txt | aquatone. This will work:
cd C:\Program Files (x86)\Google\Chrome\Application
cat "C:\file\domains.txt" | aquatone -chrome-path ".\chrome.exe" -ports large -out "C:\file\att"

The issue is the default timeout setting.
By default -screenshot-timeout is set to 30000, changed it to 100000 when scanning one host and got success on taking a screenshot on port 443. Failed on port 80.
Tried using 1000000 and then the screenshot worked for both ports.
Used the same timeout when scanning 5 hosts it was very fast for all but one screenshot that finished though in the end.

Chrome trying to save screenshot to a relative path that doesn't exists (under path/to/chrome/screenshots next to executable).
Maybe the pull-request #243 will help you fixing the issue : the path now refering to destDir/screenshots folder.

Same issue getting screenshot timed out

Same issue can confirm

Fixed by pull-request from @glafarge comment.
Can confirm that now i can make screenshots on Windows 10.
Compile this version and add to this fork: https://github.com/vionde/aquatone/releases
Thanks, man!


Automatically set up TailwindCSS paths if Rails project is bootstrapped with Tailwind

Is your feature request related to a problem? Please describe.
My project uses Tailwind. Generated with --css=tailwind option.
I'm also a heavy user of tailwindui.com, so naturally I wrap TailwindUI components into View Components
However, a fresh install of view_components-contrib makes it clear that no styles are going to be compiled by bin/rails tailwindcss:watch just because Tailwind doesn't know about those paths
Describe the solution you'd like
By default, config/tailwind.config.js contains those lines:
  content: [
    './public/*.html',
    './app/helpers/**/*.rb',
    './app/javascript/**/*.js',
    './app/views/**/*.{erb,haml,html,slim}'
  ],

We might just go ahead and add app/frontend/components
  content: [
    './public/*.html',
    './app/helpers/**/*.rb',
    './app/javascript/**/*.js',
    './app/frontend/components/**/*.{erb,haml,html,slim}',
    './app/views/**/*.{erb,haml,html,slim}'
  ],

voilà, problem solved
Describe alternatives you've considered
It might be OK to just add this to readme instead of automating this as a feature
Additional context
None

Thanks for the configuration examples. For now, we collect non-Webpack configuration in the discussion. Added a link to the issue there.


Design: Create designs for Checkout components

Issue summary
There is currently no design for the checkout component and in order for the project to have a uniform styleguide, a design has to be created for each of the components.
Proposed fix
Create a design for the checkout component using the colors of the project which is already given in the README.
Tasks

[ ] Address form
[ ] Payment options
[ ] Order summary
[ ] Shipping options
[ ] Order confirmation

Before working on the issue, dm me or @AdityaSingh-02 and @purnima143 to be added to the Figma file.

@Lovelyfin00 I would like to work on this, please assign it to me

You can go ahead and work on it. Have I added you to the figma file?

Hi @Lovelyfin00 ,
I would like to work on this issue, assign it to me.


Created Dockerfile for Spring Boot Minimal Example

Added Dockerfile with Alpine Linux based OpenJDK for building Docker image

You might want to close this PR because we update this repo by copying the examples of armeria repo when a new Armeria version has been released.

Not required


[BUG] 'error_code': 31299, 'error_msg': 'Invalid param keylist

Describe the bug
A clear and concise description of what the bug is.
When I want to upload after split for chunk then when upload process failed
error code
'error_code': 31299, 'error_msg': 'Invalid param keylist
Additional context
Add any other context about the problem here.


Allow specifying the output registers for equivalence checking

Just add a flag to the plugin that allows specifying a set of strings which represent the set of output registers to compare for equivalence.

Handled in https://github.com/draperlaboratory/cbat_tools/pull/56


Replace lwIP byte order functions with built-in functions (IDFGH-6512)

This replaces the default software implementation (less efficient) with compiler builtin functions (much efficient and possibly assembler implemented).
These byte order swap functions are used intensively and are on top of the list of the recommended improvements for performance. Please see Detailed Description here and here.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution.

sha=dd9fe9761ac714b1ea60f3b4ad4226c7ef1a288b


VSCODE Version 1.9.1 on vagrant 1.9.1 Not working

HI,
thanks a lot for your plugin, i'm new to vscode and just installed vscode Version 1.9.1, and the plugin does'nt seem to work at all . i mean there is no output ! when i have a running vagrant vm launched via the mac os terminal, then i switched to vscode and issue a "> vagrant: status " nothing seems to happen nothing shows in output view.
is the plugin still compatible with vscode 1.9.1

I have the exact same experience trying to use the extension running vagrant 1.8.7. Typed "> vagrant: status", nothing happens. Opened "> vagant: Show log", console opens up, it's blank.

Thanks for the feedback! Sorry for the late reply, I had to concentrate on the first weeks of my new dream job 😉
Issue should have been fixed with latest version of the extension.
Please let me know if the fix is effective on your side too.
I'll reopen this issue if it's not.


VPNs: recommend WireGuard

This is probably what conference attendees should be using if it's available to them. Open Source, better crypto, smaller attack surface, stealthy by default, etc.

I'm happy with listing Wireguard as an option. I'm hesitant in recommending a non-mature and unaudited piece of software that exclusively works on some Linux distros. You personal preference is, well your personal preference.

Okay, pushed a change to reflect your view.

recommending a non-mature and unaudited piece of software

"Guaranteed fewer 0-day than the competition!"


Can't use except validator condition

Hi!
I'm trying to exclude my validation rules from creation scenario, which name specified in ActiveController::createScenario
Example
public function rules()
{
    return [
         ['name', 'required', 'except' => 'create'],
    ];
}

As you can see, I have no other rules for 'create' scenario, I'm using the scenario just to exclude validation rule.
But when I call Model::validate() it throws "Unknown scenario" exception.
Where is the bug? In docs or in the code?

you could define scenarios yourself to work around this. Model should generate scenarios by also including information from except.

@cebe By the way, maybe it is also a bug: If rule has specified no $on, and no $except, it would not be common  for all scenarios

Resolved by commit a4f760e


FEAT: Add i18n Multi-language support

Pull Request Type

[x ] ✨ feat
[ ] 🐛 fix
[ ] ♻ refactor
[ ] 💄 style
[ ] 🔨 chore
[ ] 📝 docs

What is in this change?

Multi-language support base on i18n

Add English and Chinese language choose in Appearance page

update menu item for i18n

Additional Information

Developer Validations

[ ] I ran yarn lint from the root of the repo & committed changes
[ ] Relevant documentation has been updated
[x ] I have tested my code functionality
[ ] Docker build succeeds locally

This wont work for us, we are not a typescript project so there is a lot of things here we cannot merge. Will take the translation work you have done in settings.json and add those in the modified PR

https://github.com/Mintplex-Labs/anything-llm/pull/1317


Proguard rules update

add missing rules
-keep class io.realm.kotlin.types.EmbeddedRealmObject
-keep class * implements io.realm.kotlin.types.EmbeddedRealmObject { *; }

Thanks @tarasmorskyi-mediamonks
@rorbech I think we missed to add proguard rules in https://github.com/realm/realm-kotlin/pull/797/files

fixed via https://github.com/realm/realm-kotlin/pull/1109


Add connector for datart and ingest dashboards and charts metadata from it

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@zhaizhenhua would you be interested in contributing the connetor?

@zhaizhenhua would you be interested in contributing the connetor?

As a loyal user of the project, considering that Datart is also a good open-source project, I hope you can include Datart in OpenMetadata in future versions


[test][fast-client] Removed TC and DVC based metadata

Removed TC and DVC based metadata
We have fully migrated to server/request based metadata for FC. Cleanup the code for the other two metadata implementation (TC and DVC based) for FC.
How was this PR tested?
Existing unit and integration tests
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Parked this for too long and the rebase is not worth it. Will create a new PR from latest main for the removal of these tests.


Classes created in plugin using class_name not accessible via the editor

Godot version
4.0 alpha 2
System information
Windows 10
Issue description
Named classes should show up in the GODOT editor for enabled plugins when adding a new node, but the editor does not recognize them.
Steps to reproduce

Create new plugin
Add new script (under you newly created plugin folder) and give it a class name (class_name TestClass)
Enable this plugin
Make a new scene and click 'Add Child Node' under the 'Scene' tab
Search for your newly created class (TestClass)

If you find it, then I'm just a dunce. If you don't, then this is a bug.
Minimal reproduction project
PluginTest.zip

This is intended. Godot does not scan addons/ for class_names. You are meant to register the classes in your plugin script.

Duplicate of https://github.com/godotengine/godot/issues/30048.


Add WebSocket permessage-deflate

Resolves #1733
Hi!
Here's the draft of the "permessage-deflate" compression for WebSocket.
It has at least one unresolved issue discussed in https://github.com/square/okio/issues/570.
It would be great to get your feedback on the overall approach.
Thanks!

Thanks a lot for the review, @ming13 , @JakeWharton , @swankjesse !
Working on my broken git client)
Please disregard this PR till explicit notice.
I'll also change the status to non-draft once ready.

This is one of the nicest PRs I’ve seen in a while. Thanks for your contribution @AntonRutkevich !

Thanks for your great review @swankjesse !

OK, now it's ready for review.

Added WebSocketOptions parsing which is closer to RFC
Added tests

These changes are in the last 2 commits to make them easier to review. Once review is complete, I'll rebase it onto master branch, and squash into one commit.
@swankjesse , are there any other blockers for this to be merged (after review is done)?

@swankjesse

Can you mark the two new methods on OkHttpClient as internal? I’d like to restrict OkHttp 4.0 to ZERO new behavior + API changes, adding those in 4.1 and beyond.

Sure, done.
I'm totally fine with it on by default, but the trick with the header does not work here as nicely as with gzip, as far as I can tell.
For gzip, there is a way to disable it with a request header, with Accept-Encoding: identity. I could not find a way disable the websocket compression with "Sec-WebSocket-Extensions" header present. Maybe just something hacky like sending a non-existing compression algorithm, like Sec-WebSocket-Extensions: none, forcing the server utilize this part of RFC:
   A server declining all offered PMCEs MUST NOT include any element
   with PMCE names.  If a server responds with no PMCE element in the
   "Sec-WebSocket-Extensions" header, both endpoints proceed without
   per-message compression once _the WebSocket Connection is
   established_.

But if we do so, this PR and the source code will be the only place to learn how to do it for okhttp users. It's non-standard too.
Do you see a better option here?

Turn compression on and see what breaks?

OK, no problem.
So I'll basically remove all the new stuff from OkHttpClient.
And fix the options parsing to treat empty PCME list as "no compression".

Done in da99030.
Now compression is on by default.

@AntonRutkevich
Hello and thank your for the feature you made!
I'm using okhttp from your fork since yesterday. Before that I was using 3.x version.
Interesting error appears in logs in onFailure() method:
source exhausted prematurely

Didn't see this error before (my app have been working more than a year) so it's probably related to your PR. Could you check on your side?
For the info: this error shows itself in two different websockets from two different crypto exchanges so the websocket data can't be wrong. I got tens of the errors per day.

Hello @avently ,
Your error might definitely be related to this PR. I've seen it during development, when I was trying to get all these buffers work correctly.
Could you please share more details:

How often does the issue arise? 10s of errors per day, how much is it compared to the daily user count (just the order of magnitude)? Or, what percentage of users gets affected by this daily?
Do you have any idea about the pattern or any other details besides the error message? Message size? Common OS version, common device model, else? Could you reproduce it locally?

From my side, I've integrated a version of this PR rebased onto okhttp 3.13.1 (as I didn't want to include pre-release 4 version into a production app), and we'll be using it internally during at least next 1-2 weeks. We do collect error logs and other metrics, so I'll let you know if I find something.

How often does the issue arise?

6-7 erros per hour. Sometimes 2-3 per minute and then silence for an hour. I don't use the fork in production so it happens only on two of my devices: PC with JDK 8 and Android 7.1.

Do you have any idea about the pattern or any other details besides the error message?

I have no idea how your code works (didn't read it) but maybe there is somewhere Integer.MAX_VALUE instead of Long.MAX_VALUE in bytes size. Some websocket messages can be larger than one frame's size.

Payload size of the message that failed?

How to debug it? I only can duplicate websocket without encryption and see what the frame was when it's failed on encrypted websocket.
Looks like a have to debug your changes if you can't reproduce the error.

Thanks for the details!

6-7 erros per hour.
That's a lot. Then the chances are high I'll see it once we run it in the staging env internally.

I have no idea how your code works (didn't read it) but maybe there is somewhere Integer.MAX_VALUE instead of Long.MAX_VALUE in bytes size. Some websocket messages can be larger than one frame's size.
Sure, I was just wandering if you happened to have any ideas based on knowledge of your web socket traffic patterns, or something else local to your case.

How to debug it? I only can duplicate websocket without encryption and see what the frame was when it's failed on encrypted websocket.
I was thinking you could know it in advance, for example. It can also be debugged in wireshark.

Anyway, I'll try to take a closer look next week.

New info.
I added some logs and try-catch to the function inflate(). Parsed websocket message is correct even after exception is catched:
fun inflate(buffer: Buffer): Buffer {
    require(source.size == 0L)

    source.writeAll(buffer)
    println("LALAL WEBSOCKET inflate WRITEALL, source size " + source.size)
    // Adds 4 bytes
    source.writeInt(OCTETS_TO_ADD_BEFORE_INFLATION)

    totalBytesToInflate += source.size
    println("LALAL WEBSOCKET inflate totalbytestoinflate " + totalBytesToInflate)

    return buffer.apply {
      // We cannot read all, as the source does not close.
      // Instead, we ensure that all bytes from source have been processed by inflater.
      try {
        while (true) {
          println("LALAL WEBSOCKET inflate read bytesBefore " + inflater.bytesRead +
                  " value " + inflaterSource.read(this, Long.MAX_VALUE) +
                  " bytesRead " + inflater.bytesRead)
          if (inflater.bytesRead == totalBytesToInflate) {
            break
          }
        }
      } catch (e: Exception) {
        e.printStackTrace()
        println("LALAL WEBSOCKET inflate failed, bytes read " + inflater.bytesRead)
      }
    }
  }

And this is what I see in the logs:
Normal situation, 31 bytes + 4 bytes (from OCTETS_TO_ADD_BEFORE_INFLATION) were read.
LALAL WEBSOCKET inflate WRITEALL, source size 31
LALAL WEBSOCKET inflate totalbytestoinflate 18016
LALAL WEBSOCKET inflate read bytesBefore 17981 value 156 bytesRead 18016

Bad situation
LALAL WEBSOCKET inflate WRITEALL, source size 1507
LALAL WEBSOCKET inflate totalbytestoinflate 7146
LALAL WEBSOCKET inflate read bytesBefore 5635 value 5248 bytesRead 7142
java.io.EOFException: source exhausted prematurely
        at okio.InflaterSource.read(InflaterSource.kt:75)
        at okhttp3.internal.ws.MessageInflater.inflate(MessageInflater.kt:56)
        at okhttp3.internal.ws.WebSocketReader.readMessageFrame(WebSocketReader.java:254)
        at okhttp3.internal.ws.WebSocketReader.processNextFrame(WebSocketReader.java:117)
        at okhttp3.internal.ws.RealWebSocket.loopReader(RealWebSocket.java:321)
        at okhttp3.internal.ws.RealWebSocket$1.onResponse(RealWebSocket.java:253)
        at okhttp3.RealCall$AsyncCall.execute(RealCall.kt:138)
        at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
LALAL WEBSOCKET inflate failed, bytes read 7146

In the end you see that source was read fully. The difference here is the last 4 bytes. For some reason these 4 bytes wasn't read from first iteration. Then Okio decided that source is exhausted prematurely but read another 4 bytes.
And this is another example of normal situation but with more bytes:
LALAL WEBSOCKET inflate WRITEALL, source size 29222
LALAL WEBSOCKET inflate totalbytestoinflate 259013
LALAL WEBSOCKET inflate read bytesBefore 229787 value 8192 bytesRead 232122
LALAL WEBSOCKET inflate read bytesBefore 232122 value 8192 bytesRead 234303
LALAL WEBSOCKET inflate read bytesBefore 234303 value 8192 bytesRead 236446
LALAL WEBSOCKET inflate read bytesBefore 236446 value 5822 bytesRead 237975
LALAL WEBSOCKET inflate read bytesBefore 237975 value 44 bytesRead 237979
LALAL WEBSOCKET inflate read bytesBefore 237979 value 2326 bytesRead 238612
LALAL WEBSOCKET inflate read bytesBefore 238612 value 8192 bytesRead 240758
LALAL WEBSOCKET inflate read bytesBefore 240758 value 8192 bytesRead 242939
LALAL WEBSOCKET inflate read bytesBefore 242939 value 8192 bytesRead 245109
LALAL WEBSOCKET inflate read bytesBefore 245109 value 4071 bytesRead 246171
LALAL WEBSOCKET inflate read bytesBefore 246171 value 4121 bytesRead 247373
LALAL WEBSOCKET inflate read bytesBefore 247373 value 8192 bytesRead 249631
LALAL WEBSOCKET inflate read bytesBefore 249631 value 8192 bytesRead 251931
LALAL WEBSOCKET inflate read bytesBefore 251931 value 8192 bytesRead 254222
LALAL WEBSOCKET inflate read bytesBefore 254222 value 577 bytesRead 254363
LALAL WEBSOCKET inflate read bytesBefore 254363 value 7615 bytesRead 256454
LALAL WEBSOCKET inflate read bytesBefore 256454 value 8192 bytesRead 258691
LALAL WEBSOCKET inflate read bytesBefore 258691 value 1219 bytesRead 259013

Okio divides the source into chunks with different sizes (size shown after value), max is 8192 as defined inside Segment.SIZE variable.
Bad situation with more bytes:
LALAL WEBSOCKET inflate WRITEALL, source size 3810
LALAL WEBSOCKET inflate totalbytestoinflate 2489316
LALAL WEBSOCKET inflate read bytesBefore 2485502 value 8192 bytesRead 2487047
LALAL WEBSOCKET inflate read bytesBefore 2487047 value 8192 bytesRead 2488688
LALAL WEBSOCKET inflate read bytesBefore 2488688 value 3050 bytesRead 2489312
java.io.EOFException: source exhausted prematurely
        at okio.InflaterSource.read(InflaterSource.kt:75)
        at okhttp3.internal.ws.MessageInflater.inflate(MessageInflater.kt:56)
        at okhttp3.internal.ws.WebSocketReader.readMessageFrame(WebSocketReader.java:254)
        at okhttp3.internal.ws.WebSocketReader.processNextFrame(WebSocketReader.java:117)
        at okhttp3.internal.ws.RealWebSocket.loopReader(RealWebSocket.java:321)
        at okhttp3.internal.ws.RealWebSocket$1.onResponse(RealWebSocket.java:253)
        at okhttp3.RealCall$AsyncCall.execute(RealCall.kt:138)
        at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
LALAL WEBSOCKET inflate failed, bytes read 2489316

From my point of view your code is correct. Why Okio throws exception but successfully reads the source? Who knows.

Hmm..
I have a guess.
In
// Adds 4 bytes
source.writeInt(OCTETS_TO_ADD_BEFORE_INFLATION)

we rely on the fact that the writer (server) has stripped last 4 octets 0x0000ffff from the compressed data (as we do in MessageDeflater).
If this is not the case, it might be possible that we get a data ending with EMPTY_DEFLATE_BLOCK = 0x000000ffff (see MessageDeflater), add our 4 octets of 0x0000ffff.
Then okio stops reading at the empty deflate block, treating it as the end of the input, but we still have our 0x0000ffff left in the Buffer. So, okio throws the EOFException.
If this is the case, then fix like
if (!endsWithEmptyDeflateBlock()) {
 source.writeInt(OCTETS_TO_ADD_BEFORE_INFLATION)
}
...
// As in MessageInflater
private fun Buffer.endsWithEmptyDeflateBlock(): Boolean = rangeEquals(
      size - EMPTY_DEFLATE_BLOCK.size, EMPTY_DEFLATE_BLOCK)

should help.
Again, I'm not sure it's the case, as I'm not a Deflate RFC pro.
@avently Could you try this one?
If this does not work, then the println() of last 6 bytes of the block before exception would be useful. It can be obtained with Buffer.getByte(int pos).

endsWithEmptyDeflateBlock() doesn't work.
The info you asked:
for (i in buffer.size - 9 until buffer.size) {
          println("$i is " + buffer[i])
        }

9455 is 49
9456 is 54
9457 is 48
9458 is 57
9459 is 57
9460 is 125
9461 is 93
9462 is 125
9463 is 93
5733 is 34
5734 is 58
5735 is 55
5736 is 54
5737 is 48
5738 is 125
5739 is 93
5740 is 125
5741 is 93
6873 is 56
6874 is 52
6875 is 56
6876 is 57
6877 is 53
6878 is 125
6879 is 93
6880 is 125
6881 is 93
The last four bytes is always 125, 93, 125, 93. This "snapshot" was taken in  catch() block

Thanks for the logs!
I would expect every deflated message to end with 0x00, so that we add 0x0000ffff to get the empty deflate block. Looks like it's not always the case for both successful and failed ws messages.
Do you know what's the web socket implementation on the server side?
BTW, I still cannot reproduce it locally.

Do you know what's the web socket implementation on the server side?

Nope.
Attaching a sample app. It was initially made for reproducing another bug so it has more code than needed it the situation.
Change a version of okhttp lib in the build.gradle to your local version. I use the version 4.0.0-SNAPSHOT (just named it like that for myself).
Run with ./gradlew build runOkhttp
Wait a liitle bit and you'll see a crash.
sample.zip

Thanks a lot for the sample!
I was able to reproduce the issue, actually your sample throws the exception almost immediately.
While I still don't have a proper solution, here is what I found:

Looks like the backend implementation you are listening to is not following the ws permessage-deflate spec directly (7.2.1) as there are messages which do not end with 0x00. Still, I do confirm that there seems to be no explicit correlation between that and the exception: 0x00 messages produce the exception sometimes, and non-0x00 messages are parsed correctly sometimes.
Again, I do confirm that if inflater.bytesRead == totalBytesToInflate - 4 after the read call, the exception is guaranteed to be thrown.
This happens because on the next iteration inflater source

tries to inflate these 0x0000ffff bytes, but gets 0 bytes inflated
so, it runs another cycle of the inner while loop, where the buffer is already exhausted, reads 0 bytes again -> throws the exception.

I have once been able to reproduce this with 5, not 4 bytes left to read: 0x000000ffff. The message ended with 0x38e061042c090000.
Also, the traffic pattern of the backend you are using is worth mentioning.
Here is the example sizes of the first messages (159, 64, 29, 10, 16, 1755, 105657, 279888, 90, 32, 46, ...). So first some small meta info, then 1-2 huge snapshots, and then more smaller updates.
From my experience, these huge messages throw the exception most often.

Here is what I tried with no success (in pseudocode):
// Only write 4 bytes if it will make the message end with empty deflate block.
if (source.endsWith(0x00)) {
  source.writeInt(0x0000ffff)
}

if (source.endsWith(0x00)) {
  source.writeInt(0x0000ffff)
} else {
  // Write full 5 bytes of empty deflate block with no compression: 0x000000ffff.
  source.writeByte(0x00)
  source.writeInt(0x0000ffff)
}

The only hack that seems to solve the issue I see now is
if (inflater.bytesRead == totalBytesToInflate || (inflater.bytesRead == totalBytesToInflate - 4)) {
  break
}

but it's not much better than try-catch of EOF with silent ignore.
For now, I wasn't able to find the reason, why these 4 bytes are left unread, which seems to be the root cause.

I have
if (!source.endsWithEmptyDeflateBlock()) {
      source.writeInt(OCTETS_TO_ADD_BEFORE_INFLATION)
}

with
if (inflater.bytesRead == totalBytesToInflate || (inflater.bytesRead == totalBytesToInflate - 4)) {
     break
}

And it doesn't help me to prevent crashes. So only try-catch block works for now

Remove the
if (!source.endsWithEmptyDeflateBlock()) {
      source.writeInt(OCTETS_TO_ADD_BEFORE_INFLATION)
}

First, if this was correct, it would make sense to check the last one byte to be 0x00, as I understood after answering to you and debugging the code. Second, it just does not work unfortunately, the data gets corrupted this way.

@AntonRutkevich do you want to work this PR through the kotlin migration?

@yschimke yep, I do.

Sorry for the delay.
Please take a look at the rebased (and migrated to Kotlin) version of this PR.
There are 2 conceptual issue which I'm not sure how to solve. Mentioned them inlined in the code.

@AntonRutkevich hello. Did you fix the issue we talked about? I use your old code with try-catch block for now.

Hi @avently,
Not really. Looks like it goes down to the implementation of the java inflater and requires deeper investigation on how deflate algorithm actually works.
For now I thinks that try-catch is an acceptable solution, taking into account that inflater does not collapse after this exception. But we definitely need an approve from Jesse for that.
Do you have other ideas?

@swankjesse, hey, sorry to ping you about this but can we ask you to take a look at the PR once again? It has been more than four months since it was submitted and it is kind of stuck in the limbo. We would like to make it happen since it will allow us to use WebSockets more efficiently (we actually pay for user data usage in one of the applications we have). Maybe we can make a call with @AntonRutkevich to discuss further steps and how you imagine the implementation should look in the end. We understand that this change is not a trivial one and requires proper thinking about consequences.

Do you have other ideas?

Nope. Works ok for me with try-catch. Maybe someone who knows inflater better has answers. But this guy probably saw this discussion here and in okio thread. No suggestions were made so let's leave it with try-catch. It doesn't ruin anything

@ming13 yeah, this is ready for attention. Now that the dust around our Kotlin migration has recently settled, I’m ready.

Hey @swankjesse , sorry for the delay.
Could you please confirm what needs to be done from my side to get this PR merged?

@swankjesse ping - any thoughts?

Builds are green, think it's just the merge conflict blocking this now.

@AntonRutkevich yesterday I saw a video made by you about testing a code. Thank you.
I thought, I saw Anton Rutkevich somewhere. It turns out I use your build since June.

@yschimke  Done. Had to move the diff of #5800 by @dave-r12 from RealWebSocketTest.java to RealWebSocketTest.kt.
Not sure how to preserve Dave's authorship, as the .java file is gone now.
@avently, glad it was useful :)

@AntonRutkevich
Does the latest code have a fix for situation we talked earlier?
https://github.com/square/okhttp/pull/4871#issuecomment-498684696
For those who curious how much effective this pr is:
it reduces bandwidth of my app by 85.5%. It's alot!

@avently
No, the current code does not have the fix for the above situation. The reason is that the only way to fix it completely I could find was the ugly try-catch from https://github.com/square/okhttp/pull/4871#issuecomment-499140329.
It's great to see such traffic reduction!

@AntonRutkevich
So

Ugly but working for a half of year without any problems
Good but crashes every time on two leading crypto exchanges from two.

And you chose 1 option? I think we should have a try-catch block because it proved it's usefulness in real world application

I'm fine with 1) as long as I get an approve from @swankjesse for that :)

I built a code from this PR and the problem is still here... Just to confirm.

Hi @avently,
you might want to check if the Jesse's implementation in #5871 still has the EOF issue on you input. It differs a lot in the details from this PR (much simpler also), so the issue might be gone there.

@AntonRutkevich thank you for letting me know. I will read the changes you mentioned. Glad that your work received continuation.


Check function: unzipFiles

Function documentation in order with SRS (Roxygen2)
Functionality in order with function documentation
Unit test in order with TestPlan (usethis, testthat)
Code review of function (other dev than whoever picked up this ticket) (lintr)

Deprecated


⚠ Processors has degraded performance

In 24edce6, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 88 ms

Resolved: Processors performance has improved in 3972ac4.


HIP-RA fails to parse parameters with blank line

Resolved by https://github.com/softwareengineerprogrammer/geothermal-ui/commit/2e7959e2ffa63c55fea7f122428a8224667554d8


BUG [USER] Endless progress is shown on 'Sign Up' button after leaving 'Last name' field empty

Steps to reproduce:

Go to https://ita-social-projects.github.io/GreenCityClient/#/auth/sign-up
Enter valid value in 'First name' field (e.g. 'Nadya')
Leave 'Last name' field empty
Enter valid email adress (e.g<EMAIL_ADDRESS>Enter valid values in 'Password' and 'Confirm password' fields (e.g. Nadya*13)
Click on 'Sign Up' button

Actual result:
Endless progress is shown on 'Sign Up' button after leaving 'Last name' field empty
Expected result:
'You have successfully registered on the site. Please confirm your email address in the email box' confirmation message is shown after clicking on 'Sign Up' button
Environment:
OS: Windows 10
Browser: Mozilla Firefox 72.0.2 (64-біт)
Google Chrome Version 80.0.3987.132 (64-bit)
#385

Not relevant


The current release of polkabot is lacking critical features in discord

The release of polkabot lacks the key component that differentiate itself to other discord bot which is the lack of urine. with this missing from the core features, the cord must have a suisei bot or else there will be a immediate degradation of the channel. without the piss, even one of the old members of okbh unfortunately leaved the server such as "brook". god i miss him uoooogh

how would polkabot piss work? lol


Driver should not require sudo

The driver should not require being run with sudo. Instead, README should include instructions to set up udev rules (e.g. creating /etc/udev/rules.d/99-huion-tablet.rules that sets mode = 0664 to a specific group, e.g. plugdev or dialout).
I would create a pull request, but I'm not very familiar with the udev syntax. My rule:
SUBSYSTEM=="usb", ENV{DEVTYPE}=="usb_device", MODE="0664", GROUP="dialout"
gives write access to USB devices period. I think something like
SUBSYSTEM=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="256c", ATTRS{idProduct}=="006d", MODE="0664", GROUP="dialout"
SUBSYSTEM=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="256c", ATTRS{idProduct}=="006e", MODE="0664", GROUP="dialout"
might work.

Damn, this is much better than what I am currently doing, I should try this out soon.

This issue is more than just an "enhancement", It is a security issue and it should be treated as such. At least that is how I view it.
Even if users trust the developer here, there is always the possibility that as this driver is adopted by intermediate users that it could be intercepted or modified to be malicious if 3rd parties start providing it downstream.
A new user with lower tech literacy, naive, and not knowing the difference may then refer to your install instructions - if they even go out of there way take even that step -. Only to see that "sudo" was officially used upstream in the instructions and then just assume "it must be safe" since its not an "oddity" between your instructions and the 3rd partys. Then proceed to literally just cut and paste the install instructions into a terminal without a second thought.
Additionally requiring the use of sudo when its not absolutely a requisite is going to reduce the likely-hood this package is going to be supported by other projects or adopted by intermediate users, distro user repos, and those with workstation usecases.
With all that said, thank you for creating this driver. It is very much appreciated that someone is properly working to support Huion products and are not just slapping them together along with Wacom drivers. Particularly since Huion on average is really Wacom's best competitor in the space.


Broken Vue Dev Tools

Version
6.0.0-beta.2
Browser and OS info
Chrome Version 86.0.4240.111 (Official Build) (64-bit) on Windows 10
Steps to reproduce
When I open Vue Dev Tools I'm getting two errors;
"Uncaught (in promise) TypeError: Cannot set property 'VUE_DEVTOOLS_UID' of null
at Object.registerApp (backend.js:1204)"
and:
"backend.js:570 Uncaught TypeError: Cannot read property 'id' of null
at Bridge. (backend.js:570)
at Bridge.emit (backend.js:3998)
at Bridge._emit (backend.js:903)
at backend.js:835
at Array.forEach ()
at backend.js:835
at listener (backend.js:1477)"
What is expected?
To Vue Devtools to work without errors.
What is actually happening?
Vue Dev Tools are not working at all.

Same exact problem for me, just after creating a basic project with the Vue CLI.
I searched a lot for a fix, but this bug seems "different" from the previous months/years one.
I wonder if I'm missing something in the docs, but I also tried to follow the "Force enable the devtools" section and nothing happened.
(Is it possible to fix it with some config?)

The best and only way for now is to uninstall Vue Devtools extension and install the older, stable version that do work (https://chrome.google.com/webstore/detail/vuejs-devtools/nhdogjmejiglipccpnnnanhbledajbpd?hl=en), I couldn't do anything with the new beta version, it's sadly unusable at the moment.

@stiltet Thank you for the response and I confirm the stable version is working!
Yes anyway, I tried everything and this seems the only solution at the moment.


What do you think? @GoodForOneFare @lemonmade

@fandy tests+code in the same commit.  If you write the tests first, you'll notice stuff like the test runner ignoring them.

PR should be ready now. Also added npm run setup to the README since it can be confusing to new Shopifolk.

Just pushed an update - it's looking much cleaner now :)

Updated with sanity checks:

don't remove returns in nested blocks
don't remove returns with arguments

👍 looks great! Remember to squash and this is good to go 🐑

👍 Good stuff.  Thank you!


CI: enable github workflows

Enable CI for builds/PRs.
Switches away from actions-rs as this is unmaintained. See thread

I'm definitly not a CI expert, but LGTM from what I understand !


feat: created the StarSearch button CTA

Description
Implements the StarSearcButton component used to open StarSearch in Workspaces.
Related Tickets & Documents
Closes #3587
Mobile & Desktop Screenshots/Recordings

Steps to QA
You can view the StarSearchButton component in Storybook.
Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[x] Tier 3
[ ] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

Looks almost to have an internal box where maybe it should be a gradient? Notice the red edges on the bottom and right side:

Looks almost to have an internal box where maybe it should be a gradient? Notice the red edges on the bottom and right side:

Yeah, our current StarSearch logo is like that as well. I'll definitely get it tweaked and there's some feedback coni mentioned to Isa up above  as well.

Looks almost to have an internal box where maybe it should be a gradient? Notice the red edges on the bottom and right side:

Fixed in 19eaf17293d38f1f0a5e25a717008b52d4ec44d4


[paused] feat: snos job

⏸ On hold
This PR is currently paused until the snos repository has been updated (since the way to run Snos in the context of Madara will change a lot and we don't have any spec/design yet).

Resolves: #11
Added

Implementation of the SnosJob
StateJob::DummyState - used as a replacement until we can import the structure BlockifierStateAdapter from Madara. Currently not possible dues to conflicting dependencies versions between Snos <=> Madara <=> Cairo-vm. (It may not be needed at all to import that depending on the next updates from snos, their goal is to make it easier to run the Snos in the context of Madara)
HttpRpcClient to our Config for RPC requests to L1/Madara (gas price & snos input at the moment)
conversions & time utils functions

Hi @apoorvsadana , I just want to know if there are any plans for SNOS. I've noticed that this PR has been on hold for a while.

Hi @apoorvsadana , I just want to know if there are any plans for SNOS. I've noticed that this PR has been on hold for a while.

Hey @Tranduy1dol - this PR is on hold until the snos repository is ready to be integrated with Madara. Until then, we can't really continue this work.


Support for Lazy like Dagger

I've implemented a version of Dagger's Lazy<T> in Sangria.  But it's somewhat limited; Lazy<Dependency> works automatically but @Annotated Lazy<Dependency> requires you to explicitly add a binding.  That's about as good as an extension can do.  There are some alternatives:

Have Guice support its own Lazy<T> out of the box, in a similar way to how Provider<T> injection works now.
Have Guice expose some kind of support for "pluggable bindings."  Then I could write something like bind(new LazyKeyMatcher()).to(new LazyGuicePlugin()) and the plugin would generate bindings on the fly.

Is there any interest in doing something like 2?  If so I can code up a proof of concept.

I started looking into the generic binding thing, and I realised it's not quite powerful enough to implement Lazy<T> on top of, since it wouldn't work for @Named("foo") Lazy<Foo>.  I can't think of a simple way to express such a "wildcard" binding either; maybe a special @Any annotation would work, but how would the implementation actually retrieve the real annotation?
So with that aside, would you guys accept a patch that added a guice-specific Lazy?  Then if JSR 330 gets updated, it could be retrofitted on top just like Provider was.

Ping?  I'm happy to implement a Guice-specific version of Lazy as long as there's a chance it would get accepted.

We're cooking up something to make this work cross Dagger/Guice.  I'll ping
here when it's ready.
On Wed, Dec 9, 2015 at 4:40 PM Tavian Barnes<EMAIL_ADDRESS>wrote:

Ping? I'm happy to implement a Guice-specific version of Lazy as long as
there's a chance it would get accepted.
—
Reply to this email directly or view it on GitHub
https://github.com/google/guice/issues/852#issuecomment-163402304.


View source link includes URL fragment

The Chrome extension adds a 'view source' link to the class reference pages, but it doesn't seem to be stripping out URL fragments (and maybe other URL components as well?).

Navigate to https://developer.android.com/reference/android/widget/Toolbar#collapseActionView()

'view source' links to https://android.googlesource.com/platform/frameworks/base/+/refs/heads/master/core/java/android/widget/Toolbar#collapseActionView().java

'view source' should link to https://android.googlesource.com/platform/frameworks/base/+/refs/heads/master/core/java/android/widget/Toolbar.java

Apparently node.js doesn't let me use URL without require('url') which you don't need when running in a browser. As a result I've spent 100x longer trying to get the tests working than fixing the bug and writing the tests. Ugh. 🤦♂


Loading Bar working when reload page only

Hello
why loading bar working when reload page only
when navigate to pages not working
Thanks

This is likely something in your implementation.  If you post a fiddler or plunker someone may be able to help you out.

` ngular.module('app')
.run(['$rootScope', 'cfpLoadingBar',
function ($rootScope,  cfpLoadingBar) {
$rootScope.$on('$stateChangeStart', function (event, toState, toParams, fromState, fromParams) {
            cfpLoadingBar.start();
        });

        $rootScope.$on('$stateChangeSuccess', function (event, toState, toParams, fromState, fromParams) {
            cfpLoadingBar.complete();
        });

}])`


Add alternate temperature conversion for big_forecast

Allow alternate temp conversion for any location
Use alternate temp for NYC
Update layout of big_forecast to correctly position alt temp
Alternate temperature data will be available to all widgets but will only currently appear in big_forecast

Rationale: New York seems to have a disproportionate number of non-native Fahrenheit readers and it could benefit all of us to develop a bit of short hand knowledge for both.

Hey Micah. This looks great. Thanks for doing this!
One question: I don't see the NYC dashboard actually using this yet. Did you forget to check in dashboards/nyc.erb?

I hadn't actually pushed it up yet. If it looks ok, I'll merge the pull request and get it up there.

Cool. Thanks!

I realized the alt units looked strange without the °C/°F units so with the last commit now it does, though a side-effect is that it does so for both big_forecast and forecast. I smoke-tested each board on staging and I think they look better with the units but we can change this if needed.

@spilth Deployed to pivotal-life. Let me know what you think.

Looks good! I put it back up on the NYC monitor.


TaggingLogger not setting record.extra['tags'] properly

According to the docstring for TaggingLogger, "The tags themselves are stored as list named 'tags' in the extra dictionary." However, if you expose the contents of the tags variable via a format string, you find that when using the methods generated in TaggingLogger's init, record.extra['tags'] is overwritten with the last tag in the list for all the generated methods.
Thus, this code:
from logbook import StderrHandler
from logbook.more import TaggingLogger

h = StderrHandler(format_string="{record.msg}|{record.extra[tags]}")

l = TaggingLogger('tagged', ['a', 'b'])

with h.applicationbound():
    l.a('a')
    l.b('b')

outputs:
a|['b']
b|['b']

The unit test in test_more.py did not test the generated functions and did not check that the tags entry in the extras dict was properly filled.
This pull request takes care of both issues, by using functools.partial instead of a lambda for generating the functions in TaggingLogger's init, and adding a unit test similar to the above.

Thanks!


Rewrite as WebExtension (FF56+) (Updated with toolbar icon & locales)

NOTE: the icons-feature doesn't work with Firefox 55 or lower - see ff55 for a branch without the icons
Not everything that was in the old version has been brought over yet. This is
mainly:

all the locales (English only currently) (old locales auto converted)
toolbar button (added)
hotkey (Ctrl-Shift-U)
network error page

However, the main feature is there:
resurrect

the current page
the link right-clicked on
the selected string of text

in either

the current
a new foreground tab
a new background tab
a new window

using

Google Cache
Google Cache (Text-Only)
The Internet Archive's Wayback Machine
Archive.is
WebCite

See also:
https://github.com/arantius/resurrect-pages/issues/26

I haven't tested it yet (I will), but I think this is a great start! Are all of the context items added to the main menu, or is it a submenu?

here's a screen shot

implemented the popup

What does resurrect "the selected string of text" even mean?

it was my attempt at replicating this: https://i.imgur.com/0ZGIu86.png
my version currently lacks the detection if it actually looks like a url. (I've got no problem with removing that feature until it's completed)


Dataview query to SQL

Hello there!
Do you think it would be possible for you to create such a method, that could populate an SQL table using the results of a dataview query, rather than the frontmatter?
Table name would be the note's name, and there would be a single dataview query on the note, and your code would upload the result as it is rendered by the dataview (with column names and values)
I am desperate to find some sort of solution so i can easily do an SQL query on the data. The current method of a json string being uploaded into a single record is not suitable for me unfortunately...
Thank you for your kind answer!

Hi, sure, can you send me an example data view query + Markdown files?
Also, I would be interested in the SQL query that you are planning to use.

Camille

18 nov. 2022, 23:27 de @.***:

Hello there!
Do you think it would be possible for you to create such a method, that could populate an SQL table using the results of a dataview query, rather than the frontmatter?
Table name would be the note's name, and there would be a single dataview query on the note, and your code would upload the result as it is rendered by the dataview (with column names and values)
I am desperate to find some sort of solution so i can easily do an SQL query on the data. The current method of a json string being uploaded into a single record is not suitable for me unfortunately...
Thank you for your kind answer!
—
Reply to this email directly, > view it on GitHub https://github.com/clouedoc/postgresql-obsidian/issues/5> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/ADKG2SWGTKOCM6H33CKVYNLWI77ELANCNFSM6AAAAAASE557XE> .
You are receiving this because you are subscribed to this thread.> Message ID: > <clouedoc/postgresql-obsidian/issues/5> @> github> .> com>

Please find a very basic set of markdown files on the link : https://drive.google.com/file/d/19xncVZ782mDXk1ZbksOiAnQAlOZV45GF/view?usp=sharing
My use case would be that i build all the RolePlaying elements in Obsidian (characters, items, etc) and upload them using their metadata to SQL, so my gameserver can query it and use all the provided data for the online game.
Dataview would be needed in obsidian to "precompose" the datasets.
Should a reverse query would be possible, please let me know, because that would be non plus ultra!
I mean making a query from the SQL database and with those values updating the corresponding files' metadata.
In case anything is changed in the SQL by a different software, so i could have a two way sync between Obsidian and the SQL.
Oh and i would love to buy you a coffe, or somehow show my gratitude! Please give me a paypal or anything where i can do that! ;)

That's a cool use case 😮
I will try to throw a few hours at it, but can't promise anything.
User experience
Here's what I'm thinking about, based on your suggestions:

The user creates a note that will be dedicated to uploading the data
The note has frontmatter with postgresql-table: true
There is a single dataview query inside the note
The results of the dataview query automatically gets uploaded on trigger.

Triggers:

the user updates the database note
the user updates the frontmatter of a note referenced in the results of the dataview query
the user asks the plugin to "upload all the vault to SQL"
the user executes a "upload dataview to database" command

Technical challenge
The problems here are:

inferring the right data types for the SQL columns
fetching and applying updates from the database
not destroying/re-creating rows (could cause problems with Foreign Keys on the user's side)

As for the technical challenges:
I think we can agree that this method can not be code-controlled enough to eliminate user error. Whoever would use this has to have certain knowledge and discipline.

inferring the right data types for the SQL columns

I am not sure of the available methods, but i would approach by doing simple math with all columns' values and if it fails then stick to varchar().

not destroying/re-creating rows (could cause problems with Foreign Keys on the user's side)

I would be perfectly fine with destroying all tables and re-creating them when uploading from Obsidian to SQL, to eliminate any overwrite or duplication errors.

fetching and applying updates from the database

This i think can actually be skipped. Let's take the Obsidian as the ultimate vault of data and information. The backway synchronization from SQL could cause severe data loss if the database would become corrupt...


Code of certain pattern can not execute correctly

Here is the source code block in Emacs:
#+begin_src racket :lang sicp
(define (fibgen a b) (cons-stream a (fibgen b (+ a b))))
(define fibs (fibgen 0 1))
fibs
#+end_src

org-babel-expand-src-block gives the following output:
#lang racket

(write (let ()

(define (fibgen a b) (cons-stream a (fibgen b (+ a b))))
(define fibs (fibgen 0 1))
fibs))

Execute this piece of code yields the following error:
: fibgen: undefined;
:  cannot use before initialization
:   context...:
:    c:\Users\Administrator\AppData\Local\Temp\babel-gE7w5t\org-babel-BEF6AJ.rkt:3:0
:    body of "c:\Users\Administrator\AppData\Local\Temp\babel-gE7w5t\org-babel-BEF6AJ.rkt"

Copy and paste the code to DrRacket yields the same error.
However, removing the (write (let part gives code that can be correctly executed:
#lang sicp

(define (fibgen a b) (cons-stream a (fibgen b (+ a b))))
(define fibs (fibgen 0 1))
fibs

DrRacket outputs (0 . #<promise>) for the final expression.
Why (write (let is necesssary? Is there any way to avoid such problem?

So the particular piece of code does evaluate without error at the sicp
module level, and it also produces a result in racket, even with the
implicit (let () ....) wrapping:
#+begin_src racket :results value :require "(only-in sicp cons-stream)"
(define (fibgen a b) (cons-stream a (fibgen b (+ a b))))
(define fibs (fibgen 0 1))
fibs
#+end_src
#+RESULTS:
: {0 . #}
The difference between racket and sicp would appear to be the way in
which the local define forms get transformed to more primitive binding
forms. One solution then would be to transform the code yourself to be
more explicit about the desired evaluation order:
#+begin_src racket :lang sicp :results value
(letrec ((fibgen (lambda (a b) (cons-stream a (fibgen b (+ a b))))))
(let ((fibs (fibgen 0 1)))
fibs))
#+end_src
#+RESULTS:
: (0 . #)
The assumption made by ob-racket is that in :results value mode we can
take the contents of a code block and wrap it in a (let () ....) form to
get an expression that gives a value for the block, which can then be
further processed by writeing it out, or in some cases printing it out
as an Emacs Lisp datum.
It turns out this assumption doesn't hold for your sicp code sample
since evaluation of the resulting expression produces an error rather
than a result. You can avoid that kind of incompatibility by instead
using :results output mode, in which no value printing (write (let
....)) expression gets constructed:
#+begin_src racket :lang sicp :results output
(define (fibgen a b) (cons-stream a (fibgen b (+ a b))))
(define fibs (fibgen 0 1))
fibs
#+end_src
#+RESULTS:
: (0 . #)


[BUG] Un-used Timer code

Describe the bug
Hi,
Some of the existing notebooks contains the following code, which is not used anywhere:

# calculate the time required by a cell to run
from timeit import default_timer

class Timer(object):
    def __init__(self):
        self._timer = default_timer

    def __enter__(self):
        self.start()
        return self

    def __exit__(self, *args):
        self.stop()

    def start(self):
        """Start the timer."""
        self.start = self._timer()

    def stop(self):
        """Stop the timer. Calculate the interval in seconds."""
        self.end = self._timer()
        self.interval = self.end - self.start

It can be found in several notebooks: linear_regression_demo.ipynb, dbscan_demo.ipynb...
Thanks!
Expected behavior
Removed that code if not used anymore.

Thanks @miguelangel. Fixed with https://github.com/rapidsai/notebooks/pull/159


fix: fix aws database generator

What type of PR is this?

/kind bug
What this PR does / why we need it:
This PR fixes some bugs for aws database generator.
Which issue(s) this PR fixes:

Fixes #540
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 74.045%

Files with Coverage Reduction
New Missed Lines
%

pkg/cmd/destroy/options.go
7
84.08%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
6259

Relevant Lines:
8453

💛  - Coveralls


AMBARI-24326. Upgrade creates flume kerberos principal, after removin…

…g flume service (dlysnichenko)
clone of #1828

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3213/
Test FAILed.
Test FAILured.

Rescheduled to 3.0


Validating JSON with Schema

I want to validate the input against json schema
eg
{
 "name": "abc" ,    //Required Not null String
 "email"<EMAIL_ADDRESS>,  //Can be null String
 "age": 25    //Required Not null Integer
}

Is there any function which can help me validate against all the inputs

+1 curious about this as well

I think it'd be a very interesting feature for SwiftyJSON that would allow it to differentiate itself from competitors. I'm gonna tag this as help wanted in case someone would like to work on this, please let us know in this issue so we can assign you.

OK I can help in contributing.

@sujaykakkad Awesome! Assigning you here then. Would you mind discussing the architecture in this issue before creating a PR? Just so we're on the same page. Thanks!

import JSONSchema

let jsonSchema: JSONSchema = [
    "type": "object",    //SwiftyJSON object
    "properties": [
        "id": ["type": "number"],
        "name": ["type": "string"],
        "address":[
            "type": "object",
            "properties":[
                 "city": ["type": "string"],
                 "state": ["type": "string"],
                 "zip": ["type": "number"]
             ],
            "required": ["city", "zip"]]
        ]
       "required": ["name", "address"]
    ]
]

let address = JSON(["city": "cityX", "zip": 1234])
jsonSchema.validate(JSON(["name": "abc", "address": address]))

Something like this as library

Hi, Where should I include JSONSchema.swift file in xcode project? Should I use
swift package generate-xcodeproj command and then include my file. The git repository has Source folder instead of Sources folder.

@sujaykakkad Are you talking about https://cocoapods.org/pods/JSONSchema, or are you planning to write JSONSchema yourself?

I am planning to write the Schema myself. This uses Swift Dictionary. I am planning to use SwiftyJSON object to validate. I might refer from it though but my main reference is from http://json-schema.org/.

So I repo has Source folder instead of Sources. Where should I include the Schema module in the repo?

@sujaykakkad I think Source folder is fine. What's the progress on this? Would you need someone to help out?

Actually I am caught up with some work. I will get back to this next week.

Hi @wongzigii, I like to solve this problem.
Demo code:
import SwiftyJSON
let json = JSON(parseJSON: "{"stat": {"test":"test1"}, "hello": "test" }")
//print(json)
extension JSON {
func validateJson (_ keys: [String]) -> Bool {
print(self)
let x = keys.flatMap{
[$0.components(separatedBy: ".")]
}
for y in x{
var d = self
for z in y {
if d[z] != JSON(NSNull()) {
d = d[z]
print(d.type)
} else {
return false
}
}
}
return true
}
}
json.validateJson(["hello","stat.test"])
Can we do something like this.


re-org Framework package structure to avoid import cycles under tests

          @harshanarayana I am coming back here so it doesn't fall the radar. `decoder_test`  means our dependency graph has an issue. Let's open a PR (if you haven't done so) for package re-org to avoid this situation.

Originally posted by @vladimirvivien in https://github.com/kubernetes-sigs/e2e-framework/pull/246#discussion_r1248849162

Hi @harshanarayana @vladimirvivien I am interested in working on this issue. can you add more description to this?

/assign @reetasingh


Use pkgutil to load video_reqs.json

This is to fix these errors I'm having while using steve-cmd webedit and clicking on a file to edit.
(I'm not running the command from steve repo clone)
----------------------------------------
Exception happened during processing of request from ('<IP_ADDRESS>', 52774)
Traceback (most recent call last):
  File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python2.7/SocketServer.py", line 655, in __init__
    self.handle()
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
    self.handle_one_request()
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 328, in handle_one_request
    method()
  File "build/bdist.linux-x86_64/egg/steve/webedit.py", line 80, in do_GET
    return self.route_edit(path)
  File "build/bdist.linux-x86_64/egg/steve/webedit.py", line 105, in route_edit
    reqs = get_video_requirements()
  File "build/bdist.linux-x86_64/egg/steve/util.py", line 305, in get_video_requirements
    fp = open(fn)
IOError: [Errno 20] Not a directory: '/home/paul/.virtualenvs/pyvideo/local/lib/python2.7/site-packages/steve-0.5.dev0-py2.7.egg/steve/video_reqs.json'
----------------------------------------

Can you add a test for this?

@willkg , is that ok?


use struct{} as the value type for "sets"

https://www.davidkaya.com/sets-in-golang/
We have map[X]bool in various places; this can be converted to map[X]struct{}, saving a small amount of space. We may want to try and use this as well: https://github.com/deckarep/golang-set

This is also applicable to the semaphore implementation; compare https://github.com/golang/go/issues/7747


Update BASE_VERSION to 1.21-2024-01-31T19-03-43

This is incorrect, as the 1.21 should be against the release-1.21 branch. Will need to rerun so it is agasint the correct branch. Looking into the change now. @istio/release-managers-1-21

https://github.com/istio/release-builder/pull/1743 created and merged and re-running the job.


Remove numpy dependencies <1.22.0 due to security issue with…

…numpy.core component. CVE-2021-34141
snowflake-connector-python latest version bumped dependency version of NumPy from <1.23.0 to <1.24.0. Although it is a pretty harmless security issue, there is an easy fix.

Unfortunately, it is conflicting with 3.7 support. Numpy 1.22.0 and onward requires Python 3.8 or higher.

Then I suggest we keep it as is, i.e. not restricted, and let the users update their final app environments accordingly.
Anyway, the CVE mentions "constructing specific string objects", but data-diff does not accept input from arbitrary users. And it does not use numpy directly or indirectly — that is probably some form of "dataframe storing/fetching" in the connector, which we do not do.
So it should be safe.


TAMAYA-349: Ensure Tamaya builds on JDK 11

This updates the Javadoc plugin to 3.0.1 and the JaCoCo plugin to 0.8.2. It also fixes a javadoc tag that, apparently, is not valid under JDK 11.

Thanks - why did you remove jdk9?

I replaced jdk9 with jdk11 since jdk9 is already obsolete. Would you prefer to continue testing under jdk9? Or perhaps test on jdk8, jdk10 and jdk11?


Need Help with Custom Shares

Hey guys, great plugin!
Can you help me with something please, I need to incorporate Slideshare and Xing.
Can you advise me how to do that, I mean, what code needs to be added to jssocials.js in order for that to work?
Thanks!

Thank you for the feedback!
for Slideshare: are you sure there is sharing with slideshare?
for XING: the share url is available here https://dev.xing.com/plugins/share_button/docs (Share on XING with custom design section).
How to add a new custom share is described in the docs http://js-socials.com/docs/#custom-share

Thank you for fast response!
You have been very helpful!

Hi
I have problem with custom sharing, I need  add  Xing to sharing   block,
I am writing  this code , but Xing  icon doesn't appear
    jsSocials.shares.xing = {
label: "xing",
logo: "fab fa-xing",
shareUrl: "https://www.xing.com/spi/shares/new?url=[URL_TO_BE_SHARED]",
countUrl: "#",
getCount: function(data) {
    return data.count;
}

thank you


Support for ESP32 4MB flash and ILI9341

Hi Sukesh, I wondered if this would be ok? I added a device config for ESP32-WROOM 4MB flash with ILI9341(XPT2046).
I fixed a missing reference to the ESP-IDF qrcode library as well.

@RobertUK
Thank you. Will check it out soon.
The project uses QR Code library from lvgl and not from ESP-IDF.
To add support, the only addition required is the config file which defines PINS and probably calibration portion if its resistive touch. Though I prefer to keep the project focused on capacitive touch devices only for better experience.

Hi Sukesh, I wondered if this would be ok? I added a device config for ESP32-WROOM 4MB flash with ILI9341(XPT2046).
I fixed a missing reference to the ESP-IDF qrcode library as well.

hello @RobertUK  Can you operate the touchscreen??
my problem https://github.com/sukesh-ak/ESP32-TUX/issues/34


Slideshow cutoff on Windows

The slideshow renders well in all browsers on Mac, but seems to cutoff in all browsers on Windows 7 & 10 (see below). @pleary is there an easy fix for this?
Windows 10 Chrome

Windows 10 Edge (Explorer replacement)

Windows 10 Firefox

Windows 7 Chrome

Windows 7 Explorer

Windows 7 Firefox:

Chrome 62.0.3202.94 on my Mac:

I don't think this is specific to Windows. The slideshow was designed to be displayed on a 16:9 ratio screen (specifically 1920x1080 television screens). We could tweak the design to work on wider screens, but just keep in mind that we use the same ratio regardless of the screen. So if we optimize it for wider screens, it'll look worse on 16:9 screens. Alternatively, it should look OK if the user uses fullscreen, or make the browser narrower

thnx, let me ask James...


Support System tables at PostgreSQLClient

If someone was having an access control for postgresql catalog, then exposing pg_catalog may be a security issue.
Maybe we do this with a flag?
@electrum @kokosing

Having a flag for this sounds like a good idea. We should probably default to false by default, as it could be a potential security issue.

Thanks for the review, I've modified the test case.
Also I'm going to add a flag at PostgreSqlConfig to include SYSTEM TABLE and SYSTEM VIEW

@findepi I’ve added a flag for system tables

@findepi Thanks for the comments. Rebased and squashed

Merged as 87ce835b700a1444cf841b5ccaa73c52f1446258, thanks!

@miniway as a follow-up, would you like to mention this in https://prestosql.io/docs/current/connector/postgresql.html ?


[App]: Graphs are empty

Current Behavior
Hi, none of the runs are being synced to wandb; all of my graphs are shown as having no data even though they have been running for quite a while. There is no issue with the code as far as I know because we had many runs synced in the past with no issues. In fact, this issue only seemed to come up this morning because we had a few runs yesterday with no problems with the graphing
Expected Behavior
Graphs are supposed to created for all the runs
Steps To Reproduce
Try to log anything using wandb and try to visualize the graphs on the webpage. The error we get is: "There's no data for the elected runs.Try a different X axis setting.Current X axis: _step" even though there definitely is data that could be synced.
Screenshots

Environment
OS: Linux
Browsers: Chrome, Firefox
Version: 0.15.1
Additional Context
No response

I got the same error. Everything was running fine several hours ago and then stopped working.

I think this is acknowledged by their team now: https://status.wandb.com/incidents/q7tfd1d60kqw


TagUI from docker container error launching Chrome - Docker/Chrome issue

Hi! I'm trying to mount a dockerized enviroment for a possible use in production of Tagui. In my local linux machine everything works fine, but in the container I'm stuck with an error of google-chrome. When I excecute tagui with the command:
$ tagui flows/samples/1_google.tag -h
I got this error:
/usr/local/bin/tagui: line 329: 107643 Illegal instruction     (core dumped) $chrome_command --user-data-dir="$TAGUI_DIR/chrome/tagui_user_profile" $chrome_switches $window_size $headless_switch > /dev/null 2>&1
I dont understand the cause of that error.
This is my DockerFile:
Dockerfile.txt
Thanks for help in advance.

Hi @agustin-tecso ! Can you tell me if you are running this as normal user or root user? Can you try editing tagui/src/tagui file, change the first line to #!/usr/bin/env bash -x by adding -x at the end. Then you run again. This will let you see exactly the command that is run that triggers the error.
For example on my macOS, it will be the following line -
+ '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome' '--user-data-dir=/Users/kensoh/Cloud Drive/Marketing/Website/api/tagui/src/chrome/tagui_user_profile' --remote-debugging-port=9222 about:blank --window-size=1366,842

With this info, you can copy that line and run it manually to know what exactly are the parameters that cause the error when launching Chrome in the docker image. Let me know how it goes! Both me and my team-mate we don't use Docker so we can't test directly and immediately from our side, but if Docker usage is common and we have more demand, we may have to do that in order to replicate.

Oh I see, this is totally new to me this solution, because I have no experience with Docker. Thanks for sharing back!!

I'll close the issue now but feel free to ping back if anything.

Hi! Thanks for the quick reply @kensoh! I was running tagui with non-root user, because chrome doesn't allow you to excecute whitout UI and root privileges. I was able to solve the problem. It was an issue with chrome and docker container. I had to run the container with --cap-add option in docker and export enviroment variable before execute tagui export OPENSSL_CONF=/etc/ssl
Thanks alot!

Hello tecso, I also encountered this problem. Can you tell me how you solved it? Thanks alot!

Tagging @agustin-tecso


fingers crossed

every PR that gets merged to main will have an additional CI/CD pipeline to pass
we will install all dependencies, bring up all of the docker services, start the webserver, and start the client


Remove BibTex output item count limits

Originally on 2011-09-08
From hoc:
It would probably be better for this to work like the LaTeX CV format, which
can display over 1,000 records at a time.  Certainly, a limit of 1,000 would be
more suitable than the 100 limit as the point of the bibtex format is not to
read the result but cut-and-paste to a file.
The limit is inside search_engine.py, but the effect is seen in the output format.  There is machinery in search_engine meant to raise (remove?) this limit for xml output formats; perhaps that is helpful here.

Fixed in Invenio 3 using Records-REST serialisers and Elasticsearch max search result limit.


Update font-courier-new to 2.82

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as de773fd573575aaee9f2bbee508898b5a99f7d9d. Your contribution is still credited to you.


fix: docs minor grammar issues [api/interfaces]

I am currently working on translating the Pinia documentation to Spanish, specifically in the [api/interfaces] section. To keep the PR history clean, I will be adding any minor issues I come across to this PR. I expect to finish my section within a week.
I am open to feedback and suggestions ✌🏻

Thank you for this! Please see this Discussion and the mentioned documents to create a PR to the correct repository. We will link to it here once it's ready (more in the translation guidelines)

Looking at my issue comment I was not very clear 😅 what I meant was that I'm working on the translation at the repo mentioned in that discussion and while in the process of translation I was going to be comitting fixes to grammar issues I found in the official docs. But now that I think about it, I should've created a single PR unrelated to that. Thank you for your welcoming reply, I appreciate it.


Uncaught TypeError: domNode.replaceData is not a function

Issue
Using nested tagger result in either an exception or the wrong domnode being modified
Patch indices are collapsed for nested tagger in diffHelp  but not in addDomNodesHelp
Reproduction
import Html exposing (div, text, button)
import Html.App exposing (beginnerProgram, map)
import Html.Events exposing (onClick)

main =
    beginnerProgram
        { model = Page1
        , view = view
        , update = update
        }

type Page
    = Page1
    | Page2

view model =
    case model of
        Page1 ->
            div []
                [ div []
                      [ button [ onClick Click ] [ text "change page" ] ]
                , text "page 1"
                ]
            |> map (always (always IntermediateMsg))
            |> map (always (ChangePage Page2))
        Page2 ->
            div []
                [ button [ onClick Click ] [ text "change page" ]
                , text "page 2"
                ]
            |> map (always (always IntermediateMsg))
            |> map (always (ChangePage Page1))

type Msg
    = ChangePage Page

type IntermediateMsg
    = IntermediateMsg

type SubMsg
    = Click

update msg model =
    case msg of
        ChangePage page ->
            page

Same as https://github.com/elm-lang/virtual-dom/issues/21?

Simpler SSCCE in #21, so follow along there.


Add support for casting ms and us timestamp

This PR solves https://github.com/jorgecarleitao/arrow2/issues/1541
This adds the minimal changes to support casting from a utf8 or largeutf8 to timestamp using ns, ms or us time units.

Closing this as I am implementing it on Polars nano-arrow