Bug for removing a pin

Create a PIN
Close the app
Open Security & Backup
Click deactivate PIN
Enter the correct PIN

Shows an error message for half a second but it does not accept it.
https://github.com/user-attachments/assets/895a29d1-bc63-4596-9969-cdc88a3feca1

When removing the PIN if I enter the wrong one the message should stay on the screen so I know what happened.
Additionally, when creating the PIN you can see that I entered the PIN 3 times the 3rd is not right.

@Nackoo2000 Can you reproduce this with new builds? Please share logs if you do.
This could be an issue specific to your Android OS version.
I have not been able to reproduce the issue on Android & iOS with latest builds.

Now there is no problem it works as it should.


Logging

Is there a option to enable somekind of logging feature?

There isn't one now but I can work on adding in some logging.

I recently added http://serilog.net/ to a project and was very happy with
it. Would love for this project to use it as well.
On Wed, Jan 13, 2016 at 8:11 AM, Robert Kerber<EMAIL_ADDRESS>wrote:

There isn't one now but I can work on adding in some logging.
—
Reply to this email directly or view it on GitHub
https://github.com/Lone-Coder/letsencrypt-win-simple/issues/54#issuecomment-171324459
.

I noticed the log4net library in the download package so i thought maybe some hidden option to enable logging was already in place, would be a nice feature to have in a future release, thx!

I have used log4net in the past, but I have heard good thing about serilog so i'll probably go down that road.

I added logging in PR #68.


Fix changes introduce by update of go-swagger.

Related to #1719 (that wasn't completely fixed) and #1722 (following)
I also added the same fix for cycling dependencies (in repository obj parent) not handled by swagger-ui

Is this ready to review?

@lunny yes

LGTM

LGTM


[proxy] intermittent failure in entry point test

I'm seeing the following every now and then...
$ ./620_proxy_entrypoint_command_test.sh
Proxy uses correct entrypoint and command with weavewait
...
test #3 "proxy docker_on <IP_ADDRESS> run -e 'WEAVE_CIDR=<IP_ADDRESS>/24' --entrypoint='grep' false ^1$ /sys/class/net/ethwe/carrier" failed:
	program terminated with code 1 instead of 0

Running the failing command repeatedly on the host produces a failure about half the time...
root@host1:~# docker -H localhost:12375 run -e 'WEAVE_CIDR=<IP_ADDRESS>/24' --entrypoint='grep' false ^1$ /sys/class/net/ethwe/carrier
1
root@host1:~# docker -H localhost:12375 run -e 'WEAVE_CIDR=<IP_ADDRESS>/24' --entrypoint='grep' false ^1$ /sys/class/net/ethwe/carrier
1
FATA[0000] Error response from daemon: Container bf6d0a5ae5735d4443613cc165e3e4ac4e62639e720120c98aead839db59092d died
root@host1:~# docker -H localhost:12375 run -e 'WEAVE_CIDR=<IP_ADDRESS>/24' --entrypoint='grep' false ^1$ /sys/class/net/ethwe/carrier
1
FATA[0000] Error response from daemon: Container 885c2204b90952cc0274c196b63d5b890cc84750ff0934bf67153280e7a0be3f died

The log of the failed container contains 1.

I've also had test#2 and test#5 fail.

weave attach adds the network interface, which causes weavewait to run the main command. Then, weave attach carries on doing network-stuff. If the container exits before weave attach is finished, then weave attach throws the Container <...> died error. Making the weavewait interval longer causes the race condition to be less likely, but doesn't fix it.
Options:

Special-case and ignore that error (bleh)
Make weave attach atomic, from container's perspective (probably not possible...)
Change weavewait to block on USR2 signal, not the interface (IMO, the nicest option)

So, we can't make weave attach "atomic" because the route needs to be setup after the interface is up, and the arp update needs to use the interface to ping our neighbours.


Fix typos

This fixes https://github.com/apache/beam/issues/29783

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @AnandInguva

Retest this please


Add nightscout_parser

Add a nightscout parser so that we can check predictions up against loop in real time.

[ ] Specify start and end dates
[ ] Verify that the results compared to tidepool parser are equal

Duplicate


host in manifes.yml is deprecated

Hi Harini, The "host" in manifest.yml is deprecated and hence getting the error during CF push as  "Deprecation warning: Route component attributes 'domain', 'domains', 'host', 'hosts' and 'no-hostname' are deprecated. Found: host"

Hi Joel,
some versions of CLI still support host. https://github.com/cloudfoundry/cli/issues/1403
you can fix the issue by removing the host parameter and adding random-route: true in the manifest.yml file.


Label non-persons in LHOM index

@vanessa612 Could you please go through the LHOM Index https://docs.google.com/spreadsheets/d/1ujiT91ua3sA-WX86OWpuE-gDD_E-zONpI1dP70pXdWw/edit#gid=0 and label in Column F: Is Person? which items are a single person? Just put a Y if it is an individual person and a N if it is a group or something else. Thanks!

@vanessa612 Fantastic that this is almost ready. Just let me know when you're done and ready for me to check this.


lint: add semicolons automatically

Used Eslint's fix option.
And I would like to add a "semi": ["error", "always"] after #3561  is merged.

There are missing semicolons because we are changing to StandardJS style https://github.com/expressjs/express/blob/master/Collaborator-Guide.md

@dougwilson Oh ok, should we delete all semicolons?

As long as doing so will not create merge conflicts between the open PRs and master. And also no merge conflicts between master and the 5.0 branch.


Adds support for adding a textdomain from an already parsed json object

Hi!
I have crossed the land and walked through fire for a plain gettext JavaScript library that accepts standardized JSON translation objects, but to no avail. So I checked the source and realized what a small change be needed for this excellent lib to support this.
The case is I want to use standardized gettext that's populated with translations at runtime in the browser, without doing HTTP requests to .po files (i.e. the translations are bundled in the JavaScript file).

Do you think this goes against the purpose of this library?
Is the truthiness of file.translations a stable enough check for an already parsed JSON object?

If not, yay! If so, I'll make a new npm package that simply extends this one with that simple check.

Hey, sorry for missing this

Awesome! No worries at all.


[Duplicate of #5490]: In Page Filtered Search With Vanilla JavaScript

Formatting and structure checks

[X] Have you used the correct folder and file structure? - see Contribution Guidelines

[X] Is your article properly formatted in Markdown? - see Github Markdown guide

[X] Have you used our preferred heading structure? - H3 (###) and up

[X] Does your article follow any of the suggested structure formats? - see example formats

[X] Is your article over 750 words? - Tool: Wordcounter.net

[X] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[X] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[X] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[X] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[X] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[X] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[X] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[X] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[X] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[X] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes
This closes #4703

@M0nicah could have a look at this

@Shee254
I remember reviewing this article a few days ago. It's duplicate of #5490.
Please make sure to ping the original reviewer who had closed the article. This avoids duplicate reviews, and enables the reviewer to keep track of the changes made.
cc. @M0nicah Re-assigning this PR to myself.

Hello @srishilesh I have made all the changes, sorry I forgot to ping you,the article had passed plagiarism check kindly

Removing plagiarism label, since plagiarism check was already done in the previous PR.

Thank you @srishilesh,I will have a look

jumping on this one


Typing support for app type for changeFieldControl

Expected Behavior
Passing 'app' as an argument for changeFieldControl's second parameter shouldn't throw typing error.

Actual Behavior
Passing 'app' as an argument for changeFieldControl's second parameter throws typing error.

Possible Solution
Add required typing.

Steps to Reproduce

Create a typescript migration file
Install cloudinary app
Create a content type
Add a cloudinary image field using this code

model.createField(id, {
        name,
        type: "Object",
        required,
    })
this.model.changeFieldControl(id, 'app', this.CLOUDINARY_WIDGET_ID)

Context

I encountered this while creating migrations with typescript which deals with creating custom fields offered by Contentful apps.
Environment

Node Version: 10.15.3
Package Manager Version: 1.17.3 yarn -v
Operating System: darwin C02XP20RJG5H 19.5
Package Version: 1.6.2

@ruderngespra @Khaledgarbaya Can any of you please take a look at this? It's just a typing change.


Fix uninitialized memory bug in GdiEngine

ed27737 contains a regression were a RECT in GdiEngine wasn't properly
initialized anymore. Due to this, rendering during scrolling behaved erratic.
To find other cases of this bug in ed27737 the following regex was used:
^-.* = \{\s*\d*\s*\};

It appears that only GdiEngine was affected by a bug of this kind,
but just to be sure, this PR reverts all other instances.
This bug was likely caused when I tried to undo some of the changes in
ed27737 to make the PR smaller, but failed to revert the code properly.
PR Checklist

[x] Closes #13270
[x] I work here

Validation Steps Performed
I'm unable to reproduce the issue on my hardware and am unable to test
this change, but the uninitialized struct is clearly a bug regardless.
Co-authored-by: James Holderness<EMAIL_ADDRESS>

I still have this weird "ghosting" effect when I scroll in my Windows 10 VM. But I'm confident that this should fix the bug. If you happen to get a chance, would you mind checking whether initializing the RECT properly fixes the bug, @j4james? (If you haven't done so already.)

How did the audit build miss this?

That project doesn't get built in the AuditMode configuration. 😿

How did the audit build miss this?

@DHowett Terrifying fun fact: I made RendererGdi compile under AuditMode and this bug doesn't get caught at all.


python naming differences

on linux the command "python" points to python v2 and "python3" to v3. the ladderserver currently runs "python" while python-sc2 needs python >v3.6.
on debian and ubuntu which are the most used desktop linux os, "python3" points to python v3.5.
if you install python v3.6, you will need to call "python3.6".
On windows it depends how python is installed as anaconda might call python v3.6 while an installation via cygwin will use python2 by default.
to solve all this mess, i suggest to add a config file entry

python=python3.6

and make it configurable

Probably that should b3 set per bot (i.e. in the bots config) as we could have pysc2 bots too.

Could we make it mandatory to use python3.6 on the ladder? (The version is needed for python-sc2, while pysc2 can still be used with 3.6)

https://github.com/Cryptyc/Sc2LadderServer/commit/c9f352ed9a30a82b3e4067f4e4fdc215afdee88d


Failure to delete onefile build folders on Windows

OS: Win 10
Nuitka: 2.3
Python: 3.11.9
cmd:
python -m nuitka --onefile --remove-output --windows-disable-console --noinclude-dlls="*libcrypto-3.dll" --noinclude-dlls="*libcrypto-3.dll" --onefile-tempdir-spec="{TEMP}/_ZoomIt" --windows-icon-from-ico="ZoomIt.ico" --output-filename="ZoomIt" Zoomit.py
[..]
Nuitka-Onefile: Onefile payload compression ratio (28.61%) size 20772084 to 5943316.
Nuitka-Onefile: Removing onefile build directory 'Zoomit_GDI_13.onefile-build'.
Traceback (most recent call last):
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 161, in
main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 147, in main
MainControl.main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1130, in main
_main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1043, in _main
packDistFolderToOnefile(dist_dir)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 52, in packDistFolderToOnefile
packDistFolderToOnefileBootstrap(onefile_output_filename, dist_dir)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 259, in packDistFolderToOnefileBootstrap
removeDirectory(path=source_dir, ignore_errors=False)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\utils\FileOperations.py", line 754, in removeDirectory
shutil.rmtree(path, ignore_errors=ignore_errors)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 787, in rmtree
return _rmtree_unsafe(path, onerror)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 615, in _rmtree_unsafe
onerror(os.scandir, path, sys.exc_info())
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 612, in _rmtree_unsafe
with os.scandir(path) as scandir_it:
^^^^^^^^^^^^^^^^
PermissionError: [WinError 5] Access is denied: 'Zoomit_GDI_13.onefile-build'
Nuitka-Reports: Compilation crash report written to file 'nuitka-crash-report.xml'.
I've tried two different projects, with the same result. In both cases, if I run the exe produced inside the (non-removed) 'xxx.dist' directory, it runs with no problems.

Permission error: There would have to be some sort of Windows issue, e.g., an AV tool still checking those files and not allowing us to look at them. We have an extra ball of 2.3 after a sleep of 0.1, but maybe that is insufficient for more people, in 2.3 this might have become shorter, as we realized that retrying operations after a sleep is not the solution, the delay may have become too small.

I will add retry decorations with extra reasons beside AV, e.g. dist folders, your binary may still be running (or AV checked), and that should deal with this more thoroughly and give at least some form of user dialog.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback on whether this is working is very welcome. Please do not share plans to do it; only confirm or deny that it is working.

I have tried the latest factory version, but it has not resolved the issue. Version used:
2.4rc1
Commercial: None
Python: 3.11.9 (tags/v3.11.9:de54cf5, Apr  2 2024, 10:12:12) [MSC v.1938 64 bit (AMD64)]
Flavor: CPython Official
Executable: C:\Users\User1\AppData\Local\Programs\Python\Python311\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: ~\AppData\Local\Nuitka\Nuitka\Cache\DOWNLO~1\gcc\x86_64\13.2.0-16.0.6-11.0.1-msvcrt-r1\mingw64\bin\gcc.exe (gcc 13.2.0).
[...]
Nuitka: Starting Python compilation with Nuitka '2.4rc1' on Python '3.11' commercial grade 'not installed'.
Nuitka: Completed Python level compilation and optimization.
Nuitka: Generating source code for C backend compiler.
Nuitka: Running data composer tool for optimal constant value handling.
Nuitka: Running C compilation via Scons.
Nuitka-Scons: Backend C compiler: gcc (gcc 13.2.0).
Nuitka-Scons: Backend linking program with 18 files (no progress information available for this stage).
Nuitka-Scons: Compiled 18 C files using ccache.
Nuitka-Scons: Cached C files (using ccache) with result 'cache hit': 13
Nuitka-Scons: Cached C files (using ccache) with result 'cache miss': 5
Nuitka-Postprocessing: Adding 3 icon(s) from icon file 'ZoomIt.ico'.
Nuitka-Postprocessing: Creating single file from dist folder, this may take a while.
Nuitka-Onefile: Running bootstrap binary compilation via Scons.
Nuitka-Scons: Onefile C compiler: gcc (gcc 13.2.0).
Nuitka-Scons: Onefile linking program with 1 files (no progress information available for this stage).
Nuitka-Scons: Compiled 1 C files using ccache.
Nuitka-Scons: Cached C files (using ccache) with result 'cache miss': 1
Nuitka-Postprocessing: Adding 3 icon(s) from icon file 'ZoomIt.ico'.
Nuitka-Onefile: Using compression for onefile payload.
Nuitka-Onefile: Onefile payload compression ratio (28.63%) size 20766452 to 5944926.
Nuitka-Onefile: Removing onefile build directory 'Zoomit_GDI_13.onefile-build'.
Nuitka-Onefile:WARNING: Failed to delete 'Zoomit_GDI_13.onefile-build' in attempt 1.
Disable Anti-Virus, e.g. Windows Defender for build folders.
Retrying after a second of delay.
Traceback (most recent call last):
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 612, in _rmtree_unsafe
with os.scandir(path) as scandir_it:
^^^^^^^^^^^^^^^^
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'Zoomit_GDI_13.onefile-build'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 161, in
main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 147, in main
MainControl.main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1147, in main
_main()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1050, in _main
packDistFolderToOnefile(dist_dir)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 52, in packDistFolderToOnefile
packDistFolderToOnefileBootstrap(onefile_output_filename, dist_dir)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 259, in packDistFolderToOnefileBootstrap
removeDirectory(
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\utils\FileOperations.py", line 769, in removeDirectory
_removeDirectory()
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\utils\Utils.py", line 405, in retryingFunction
result = func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\utils\FileOperations.py", line 752, in _removeDirectory
shutil.rmtree(path, ignore_errors=False, onerror=onError)
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 787, in rmtree
return _rmtree_unsafe(path, onerror)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 615, in _rmtree_unsafe
onerror(os.scandir, path, sys.exc_info())
File "C:\Users\User1\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\utils\FileOperations.py", line 745, in onError
last_error.append(func, path)
TypeError: list.append() takes exactly one argument (2 given)
Note: I have Windows Defender turned off and no other anti-virus software running.

Yeah, that is just a programming mistake I made. The real solution will be what is currently on factory, and to include that for 2.3.1 hotfix, namely to just use the retry decorator, avoiding the special made code entirely.


Any plan to support advfirewall?

The last commit in this repository is  created on Jul 13, 2017.
Is this repository status in dormant?

Yes, I have not had any time to put into this project in several years - it is still pretty straightforward if you would like to add to it, but it definitely is not complete by any means


Add azurerm_data_protection_backup_instance_kubernetes_cluster feature

Description of your changes
This simple change adds support for the resource azurerm_data_protection_backup_instance_kubernetes_cluster.

Fixes #
I have:

[X] Read and followed Crossplane's contribution process.
[X] Run make reviewable to ensure this PR is ready for review.
[X] Local tests with make e2e passed
[N/A] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Manual deployment
Local uptests with make e2e passed

/test-examples="examples/dataprotection/v1beta1/backupinstancekubernetescluster.yaml"


Authentication failed for user

I have tried everything I could find online like doing ssh-add comments, ssh-agent commands, adding ed25519 and bcrypt_pbkdf but I keep having the same issue.
When I run cap <environment> deploy, in different projects, at the first task deploy:check:directories, it gets raised an exception and process completely exit saying Authentication failed for the user. If I try do a simple ssh connection to the server, everything works fine, same as if I create a custom task using exec command. With Capistrano tasks instead I get the error:
#<Thread:0x00007fa72c2c45e0@/Users/antoniodisanto/.rvm/gems/ruby-2.5.3@gemset/gems/sshkit-1.18.0/lib/sshkit/runners/parallel.rb:10 run> terminated with exception (report_on_exception is true):
Traceback (most recent call last):
	1: from /Users/antoniodisanto/.rvm/gems/ruby-2.5.3@gemset/gems/sshkit-1.18.0/lib/sshkit/runners/parallel.rb:11:in `block (2 levels) in execute'
/Users/antoniodisanto/.rvm/gems/ruby-2.5.3@gemset/gems/sshkit-1.18.0/lib/sshkit/runners/parallel.rb:15:in `rescue in block (2 levels) in execute': Exception while executing as deploy@********: Authentication failed for user deploy@******** (SSHKit::Runner::ExecuteError)
(Backtrace restricted to imported tasks)
cap aborted!
SSHKit::Runner::ExecuteError: Exception while executing as deploy@*********: Authentication failed for user deploy@*******

Caused by:
Net::SSH::AuthenticationFailed: Authentication failed for user deploy@*********

As I've said, if I run a normal ssh deploy@******* or even with -vvvv everything works fine with no issues at all. No passphrase is asked either to make it works
Below are the files
Gemfile:
source 'https://rubygems.org'

group :development do
  gem 'capistrano', '3.11.0'
  gem 'ed25519', '>= 1.2', '< 2.0'
  gem 'bcrypt_pbkdf', '>= 1.0', '< 2.0'
end

Capfile:
require "capistrano/setup"
require "capistrano/deploy"
require_relative "config/scm/local_deploy.rb"
install_plugin Capistrano::LocalDeploy

Dir.glob("config/tasks/*.rake").each { |r| import r }

deploy.rb:
lock '~> 3.11.0'

set :deploy_to, '/home/deploy/directory'
set :user, 'deploy'
set :branch, 'master'
set :tmp_folder, '/tmp/localfolder'
set :repo_url, '**************'

before 'deploy:publishing', 'app:build'
after 'rsync:deploy', 'app:clean'

staging.rb:
server '**********', roles: %w(app web db staging), user: 'deploy'

set :build_command, 'build-staging'

ask :branch, proc { `git rev-parse --abbrev-ref HEAD`.chomp }

I've done a deploy on 2022/01/25 and after, the only thing i've done, is updating MacOSX.
This is the doctor command:
cap staging doctor

Environment

    Ruby     ruby 2.5.3p105 (2018-10-18 revision 65156) [x86_64-darwin21]
    Rubygems 3.0.9
    Bundler  2.1.4
    Command  /Users/antoniodisanto/.rvm/gems/ruby-2.5.3@gemset/bin/cap staging doctor

Gems

    capistrano 3.11.0 (update available)
    airbrussh  1.3.0  (update available)
    rake       12.3.1 (update available)
    sshkit     1.18.0 (update available)
    net-ssh    6.1.0

Variables

    :branch        <ask>
    :build_command "build-staging"
    :default_env   {}
    :deploy_to     "/home/deploy/directory"
    :format        :airbrussh
    :keep_releases 5
    :local_user    "antoniodisanto"
    :log_level     :debug
    :pty           false
    :repo_url      "*************"
    :stage         :staging
    :tmp_dir       "/tmp"
    :tmp_folder    "/tmp/directory"
    :user          "deploy"

    :build_command is not a recognized Capistrano setting (/Users/antoniodisanto/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/delegate.rb:83)
    :tmp_folder is not a recognized Capistrano setting (/Users/antoniodisanto/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/delegate.rb:83)
    :user is not a recognized Capistrano setting (/Users/antoniodisanto/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/delegate.rb:83)

Servers (1)

    deploy@************* [:app, :web, :db, :staging]

Thank you!

Hi, I will close the issue. It wasn't on Capistrano side but was on server side. New openSsh version on server pretend to use a PEM file instead of classical ssh-rsa key.


Updated Caddyfile for Seafile

Updated config file to more closely match Seafile's documentation. This would close https://github.com/caddyserver/examples/issues/44
'without' trims the /seafhttp prefix from requests, like the regexp in Seafile's example nginx config: https://manual.seafile.com/deploy/https_with_nginx.html
Also included /media folder for static serving. This could alternatively be done with root and except directives in the first domain block.
gzip for good measure

Thanks for working through this, and for the contribution! Gladly accepted. :)


[Feature Request] within el-form, hope to set up tabindex attribute in el-input

Existing Component
Yes
Component Name
el-input
Description
how to set tabIndex within el-input please?
I have a form that built by el-form, and I wanna move between the input field by tab key.

You can already do that without explicitly setting tabindex: https://jsfiddle.net/qomxyr94/ . Actually you don't have to set tabindex on form elements.

@Leopoldthecoder What if we want to set the index to -1?


add docs for the 'g:cursorword_highlight' option

https://github.com/itchyny/vim-cursorword/issues/16

Thank you.


Fixed README.md. The project runs without any errors on terminal

Description
The readme.md is fixed and now the project can run without generating any errors.
Fixes #156
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
This has been tested on my terminal and my local machine

Checklist:

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

Closing as stale.


Allow users to follow organizations

It doesn't seem possible to follow organizations, but it would be a really useful feature to have. Why can users only follow other users, and not other organizations?

Hi, what is the use case to follow organization since in Gogs it does not generate any feeds?


Token for pipelines

What is the correct procedure to provide a token in pipeline jobs? I can create a personal access token and include that in my gradle file in my repos but obviously that is less than ideal since then it is essentially "public" and will also eventually expire.
I see there are pipeline tasks for nuget to push to artifacts but there doesn't seem to be an equivalent for maven/gradle artifacts.

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

ID: 52da1009-6733-d555-3c00-1a1bd77f1759
Version Independent ID: 72b916de-2836-460b-2889-67981ffea5a4
Content: Publish a Maven artifact using Gradle and Azure DevOps Services
Content Source: docs/artifacts/gradle/publish-package-gradle.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Hi @gschrader , you should be able to use the Maven build task. Are you using the designer or YAML, and are you just trying to push to an Azure Artifacts feed in your org?

@elbatk I assume you mean the Gradle task. Yes I've started with the designer, I guess YAML will probably offer me more possibilities and I should look to convert to that. Yes I'm just trying to push to a feed in my org. I think I'm going to have the same issue with reading from that same feed from another project, although maybe I'm jumping to conclusion and the pipeline agents will already have access.

I thought maybe I figured it out with $(System.AccessToken) but it looks like that token doesn't work, at least not with the AZURE_ARTIFACTS username anyway.

I got this to work, I found a blog post that suggested VSTS as a user, it still didn't work but then found a spot where I wasn't setting the environment variable and after that it worked! So I'm not sure I needed to change the user. I'll see if I can find some time to raise a PR for the docs with an example.

Sorry I was on a flight today! I'm glad you got it working, and I'd be very open to a PR to the docs with an example because I feel like this scenario should be covered. Going to close this, if you don't think you'll have time to do a PR, let me know so I can try to get one going

No worries. I hope your flight was good. It's maybe worth holding off on the documentation until the upstream maven source feature has been released, I'll be one of the first to try that out.

Good to hear! Holding off isn't a bad idea, since we pretty much recommend all of our users to utilize upstreams.


Not able to share QR as an image in iOS

Getting this message only "Scan the QR and receive bitcoin in your Bitcoin Tribe bitcoin wallet."

Duplicate


Shape inference for RNN ops: RNNOp, LSTMOp, and GRUOp

This patch creates a common function for inferring shape for RNN operations.

This patch will be merged into PR #73 so that E2E tests can be enabled in PR #73.


Mat/Pat - Bugs in Submission

While testing #895 & #1334, i noticed some bugs. I submitted a form for only maternity leave

[x] On submission i saw this error, but did not see any validation error.

Private Zenhub Image

[x] I also noticed duplicate drafts for the form that I saw an error on submission. However, the form is not in Review & Process tab.

Private Zenhub Image

Ayush to perform 4 to 5 more tests and close the ticket.

Submitted a couple more forms, and did not face this error.
Closing this ticket.

Submitted a couple more forms, and did not face this error.
Closing this ticket.


Add status subresource to fleet and Gameserverset

Switch to use UpdateStatus function for Fleets when it is necessary, add RBAC
permissions for "fleets/status" and "gameserversets/status".

The small regression after applying this changes:
If the spec.Replicas in GSS was 0 initially then there are empty values on Status.Replicas , Ready and Allocated:
NAME                                                        SCHEDULING   DESIRED   CURRENT   ALLOCATED   READY     AGE
gameserverset.stable.agones.dev/scale-fleet-0-45ksj-5qrkp   Packed       30        30        0           30        25s
gameserverset.stable.agones.dev/scale-fleet-1-szfqc-g5b8g   Packed       0                                         25s

@aLekSer I'm guessing this is because there is no default values for those status items on creation? I'd be tempted to ask #sig-apimachinery to see why 0 values come as blank? (although not quite sure what the reason for this is -- feels like a k8s bug?)
I'm wondering if its something we need to worry about it?  WDYT?

I think this is because status fields are not defined in yaml file and the only place Status are changed (inited) is here:
https://github.com/GoogleCloudPlatform/agones/blob/3a0e6d1c335413ee30dd8ebce555e18bd8620d75/pkg/gameserversets/controller.go#L557
I am looking on a fix, without it we would see empty printing columns if Spec.Replicas was always 0.

Some new error on a test step:
rm: cannot remove './public': No such file or directory
includes/website.mk:35: recipe for target 'site-static' failed
mkdir /workspace/site/public
make[2]: [site-static] Error 1 (ignored)

I am looking for a fix, without it we would see empty printing columns if Spec.Replicas was always 0.

Is the issue that the value is 0, or that status hasn't been set at all?
If you scale a fleet up to 5, and then back down to 0, does it go back to being blank?
I'm wondering if its because the status hasn't been created yet?

Hello @markmandel ,
I have fixed the issue. As you can see I added one more updateStatus on creating GameServerSet from fleet controller.

Is the issue that the value is 0, or that status hasn't been set at all?
Status were not set at all, but now it is set on creating GSS with aditional step (above).
If you scale a fleet up to 5, and then back down to 0, does it go back to being blank?
Now it always set right after create of GSS.
I'm wondering if its because the status hasn't been created yet?
Yes, that was the case, no all should be fine with yesterday fix.


🧠 feat: Implement Multi-Model Support and Extended API Integration

Multi-Model Support and Extended API Integration
Overview
This pull request introduces multi-model support and extends our API integration to include additional AI providers. The changes encompass the following major features:
1. Model Selection UI

Added a dropdown in the chat interface to allow users to select different AI models.
Options include Claude 3.5, GPT-4o, o1-preview, o1-mini, AWS Bedrock Claude, and Gemini (currently disabled).

2. Enhanced API Integration

Implemented support for OpenAI and AWS Bedrock APIs alongside our existing Anthropic integration.
Added functions to retrieve API keys and credentials for multiple providers.
Created new model creation and text streaming functions for OpenAI and AWS Bedrock.

3. Backend Changes

Updated the chat API route to handle different model selections and use the appropriate streaming function.
Modified the enhancer API to use OpenAI's streaming function.

4. Dependency Updates

Added the AWS Bedrock SDK as a new dependency.

Impact
These changes provide a more flexible and extensible architecture for our AI chat application, allowing easy integration of multiple AI models and providers.

I will remake it again based on #531!


🐛 Cannot read properties of undefined (reading 'organization')

TypeError: Cannot read properties of undefined (reading 'organization')
    at Z (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:123659)
    at https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:122945
    at Array.map (<anonymous>)
    at R (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:122931)
    at G (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:128678)
    at ak (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:60903)
    at up (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:72785)
    at i (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:120317)
    at oD (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:99073)
    at https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:98940

https://app.moderne.io/marketplace

Hi @sbenzal thanks for reporting your issue here! When I load https://app.moderne.io/marketplace here in my browser I don't see the above issue. Did you already try if a hard refresh solves the issue for you? Or might it have been a temporary issue?

Hi @timtebeek , thanks for answering, the problem happens when I try to change the organization in app.moderne.io, since yesterday it does not work and when I click on it I get the following error message
![image](https://github.com/openrewrite/rewrite/assets/147181266/25a6f003-00dd-40ec-b8e1-4ff048bb374
I dont know if its possible to see it, but it says:
TypeError: Cannot read properties of undefined (reading 'organization')
at Z (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:123659)
at https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:122945
at Array.map ()
at R (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:122931)
at G (https://app.moderne.io/_next/static/chunks/pages/_app-239a58656a093b2e.js:206:128678)
at ak (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:60903)
at up (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:72785)
at i (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:120317)
at oD (https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:99073)
at https://app.moderne.io/_next/static/chunks/framework-314c182fa7e2bf37.js:9:98940

I also tried in another browser and its not working either

Hmm tried that here as well, but not seeing any such issues; I'm hoping Scott has more insights once he comes online.

@sbenzal We have deployed a fix for this issue. You may need to refresh your browser. Please try again.

Yes, now it works, thanks! 😊


Insomnia Requests

Insomnia Requests
Fix content of the requests file for the Insomnia (as it is - has no content). Do so for the minimal project.

Done

Done :-)


Add a command to recompute dependencies state.

When changing check_file_uptodate the states of dependencies are dropped, so all the tasks are considered out-of-date. This is an issue when the tasks takes a long time to run, and if the file deps  / targets are already existing.
So I would like to add a command which recompute the states of dependencies without executing the tasks. I think I will take example on the forget command, to find all tasks/subtasks and call self.dep_manager.save_success for each task. Seems reasonable ?
Also suggestions for the cmd name are welcome ;-), something like rebuilddb ?

I guess it is reasonable. You just need to pay attention that tasks save other values not only information about file_dep... Worse than that if you just call save_success() you will remove previously saved values, so I would not use this function.
The command name should not be called rebuilddb because it is not being completely rebuilt!
set-file-dep-as-uptodate is the best description but too long so what about reset-dep?

Ah yes indeed, so I need to update the checker class name, the list and the state of file deps. reset-dep seems ok, thanks.


Add support for multiple data elements in notifications

The notify API can now process notifications whose data array contains more than one element.
Previously it returned an error message with the HTTP status 400.
reporter.reporter.notify method now loops through each data element
performing the operations previously done to the single element such as adding the time index and validating.
translators.crate.CrateTranslator._preprocess_values method now adds None to the values list if an entity update doesnot have a value for a given attribute.
Before it threw a KeyError exception. This allows _insert_entities_of_type to successfully insert
entities that have different attributes.
Tests were modified to test for this new feature:

Test data was added to conftest
test_notify.test_no_multiple_data_elements was renamed and modified to test the notify API with the test data.
test_insert_same_type_entities_with_different_attrs test was added to test_crate.
It uses the test data to check that the above described modification to CrateTranslator works.
Utils.common.check_notifications_record method was modified to use None if there is no value for an attribute in an entity update.
This was required for the method to work in the test_insert_same_type_entities_with_different_attrs test.

This pull request solves issue #185

@chicco785, @c0c0n3, @taliaga I think I have addressed all of your comments with commits or my own comments. Since you have not responded for a few days I wanted to check if you are expecting something from me or if you just have not yet had time to look into this which I naturally understand.

@ohylli sorry for the slow reply :-)
we actually talked about this PR yesterday and are thinking of merging it in. You can ignore my comments about error handling and batch processing, since we decided to implement those features in a separate PR. For the rest, we're all very happy with your work and would like to thank you very much for taking the time to do this.

Hi @ohylli and @c0c0n3 Sir,
I am also facing the same issue in our project and so I am interested in this feature to be merged as soon as possible.
Could please merge this issue or let me know in case any contribution is required. I am happy to help.
Thanks

Hi @SourabhChourasiya-NEC, thanks for offering to help :-)
We should be able to work on this next week and will definitely let you know if we need an extra pair of hands, thanks!


Shared AuthorizeAttribute

We should create a new metadata-only AuthorizeAttribute in Microsoft.AspNet.Security that will be respected by both MVC and SignalR.
This attribute will implement an interface (IAuthorizeMetadata?) that will have a single Policy property. We are considering whether or not this attribute should be sealed.
Optional framework-specific metadata can be attached to separate interfaces. For example, SignalR can have it's own HubAuthorizeAttribute that will implement an additional interface that includes the RequireOutgoing property. However, using the shared AuthorizeAttribute should be sufficient for most use cases.

Added in 5094b85ac92df0035f15e94b87d72cd4bc716905, still need to update SignalR to consume


Fix gemlite import

We do not need to import gemlite cache in the benchmark scripts.
They should already be set from python/sglang/srt/layers/torchao_utils.py

cc @jerryzh168


[Bug Fix]: Fixed the same size for all the display images for the games

PR Description 📜
I have fixed a default size i.e. (213.75px X 123.37px) for all the images for the games uploaded by the contributors in the /assets/images folder ,previously all the images were of different size and the home page looked very odd but now everything is symmetric and perfect😊
Fixes #881

[X] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[X] I have performed a self-review of my own code or work.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generates no new warnings.
[X] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[X] I have added screenshot for website preview in assets/images
[X] I have added entries for my game in main README.md
[X] I have added README.md in my folder
[X] I have added working video of the game in README.md (optional)
[X] I have specified the respective issue number for which I have requested the new game.

Thank you @trishanu-init ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @trishanu-init !
we are already thinking to re design all the images so we will be having that really soon !!

Thank you  @trishanu-init , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


How to clear content on return?

I'd like to clear the content state after enter is pressed.
Something like this:
<DraftEditor editorState={this.state.editorState} handleReturn={onReturn}/>
...
handleReturn = () => {
  this.setState({
    editorState: EditorState.createEmpty()
  });
}

This clears the input but the next key input doesn't properly move the text cursor.
Any ideas?

There are several ways to do this, you could just create a new content state altogether and push it in the editor, but you'd need to reset the selection to the new block key and offset 0. Alternatively, you could use Modifier.replaceText on the block you wish to clear by creating a selection from 0 to the length of the block, then use EditorState.push with the new content state and EditorState.forceSelection(editorState, contentState.getSelectionAfter());

I'm still pretty new to draft-js, how would i reset the selection to the new block key and offset? The APIs are a little bit more complex than one would hope.

Sure, sorry about that. It's still early days for the API that's for sure!
Disclaimer: I didn't test this, and there are certainly other ways to achieve the same effect. This makes no assumptions about the length of the document and clears the entire thing.
handleReturn = () => {
  let {editorState} = this.state;
  let contentState = editorState.getCurrentContent();
  const selection = editorState.getSelection();
  const firstBlock = contentState.getFirstBlock();
  const lastBlock = contentState.getLastBlock();
  const allSelected = new SelectionState({
    anchorKey: firstBlock.getKey()
    anchorOffset: 0,
    focusKey: lastBlock.getKey(),
    focusOffset: lastBlock.getLength()
    hasFocus: true
  });
  contentState = Modifier.removeRange(contentState, allSelected, 'backward');
  editorState = EditorState.push(editorState, contentState, 'remove-range');
  editorState = EditorState.forceSelection(contentState, contentState.getSelectionAfter());
  this.setState({editorState});
}

Alright, that seems to have worked. It looks kinda cumbersome to just clear the content. May fit as a utility method in EditorState.
Thanks!


Improvements/bug fix for removeStrategy

Fixes https://github.com/OriginProtocol/origin-dollar/issues/324 and https://github.com/OriginProtocol/origin-dollar/issues/322.

This requires an upgrade of Vault. I didn't add it as not sure if its worth it just for these changes. Hopefully it'll make its way into an upgrade shortly.


message.Encode optimization

some ideas:

[ ] Need to think whether to continue using map or move to slice for the Entries
[ ] change to other serialization, protobuf?
[ ] incremental encoding

I think for entries , we can keep using map because :

Insertion on map would take O(1) and slice would take O(n)
Getting data on map would take O(1) and slice would take O(n)

So for performance, I think it would be good if we keep using map data structure instead ofslice

So for performance, I think it would be good if we keep using map data structure instead ofslice

Please note that this is more about encoding performance.

Insertion on map would take O(1) and slice would take O(n)

hmm..yes, insertion operation performance will affect.
do you have any data that map is faster than slice?

Getting data on map would take O(1) and slice would take O(n)

we can exclude this, there is no get operation on the message

we can exclude this, there is no get operation on the message

Oh my bad
hmm..yes, insertion operation performance will affect.
do you have any data that map is faster than slice?

Since someone already did benchmark on map vs slice in this ref : https://shakedos.com/maps-vs-slices-vs-arrays, I revoked my statement. For insertion, slice is faster than map.
So I agree for changing from map to slice

did some profiling, with this scenario:

40 concurrent key set, with 4K of total keys
sleep for 2 seconds
40 concurrent key get, with 4K of total keys
repeat forever

max keys is 100K.
There is no sign of issue in message.Encode performance.
So, i think we don't need optimization on this thing, at least for now.
cpu.pdf
heap.pdf


Missing /marked/marked.min.js

I followed the procedure described in "Self-Hosting TeXMe" in order to locally try TexMe.
The file  "/marked/marked.min.js" has been removed from the current "marked" repository.
So, it is notified as missing by  the http server :
<IP_ADDRESS> - - [28/Nov/2023 08:56:18] "GET /marked/marked.min.js HTTP/1.1" 404 -
and your provided "euler.html" example is displaying an empty html page.
I revert my "marked" repository to the 8492faf1ee3acd4118f62a2fae65a34f9210903a commit which make this file to reappear and the html page to correctly display.
May be "euler.html" should be updated according to the new "marked" repository ?
Unfortunately, as I am totally incompetent in both javascript and html, I am unable to suggest a fix.

Hi @Tanavit-M,
Thank you for the detailed description of the issue.  Indeed the Marked project has removed marked.min.js from its source code.  Here is the commit in the Marked project where this file was removed: https://github.com/markedjs/marked/commit/7d95a91093f8b5cbdb30b642d06b0c9dd0ab269d
Since the project maintainers of Marked now want us to download marked.min.js directly from the CDN URL, the steps to self-host TeXMe would look like this now:
git clone https://github.com/susam/texme.git
git clone https://github.com/mathjax/mathjax.git
curl -sSL --create-dirs -o marked/marked.min.js https://cdn.jsdelivr.net/npm/marked/marked.min.js

Please let me know if this works for you.  I'll update the README sometime later.

Hello @susam
Your proposal works perfectly.
Thanks.


Fix default Python version declaration in pre-commit config

See also https://github.com/mozilla/bugbug/pull/584.

Coverage remained the same at 25.752% when pulling 46b459370cd74da13ddc2964d910870ceecd8749 on marco-c:fix_py_ver into c04f06591ccb1353f59986f92806fa3028dde61f on mozilla:master.


Crash on iOS 14 devices

I have an issue where the navigation works flawlessly on iOS 15 devices, but immediately crashes the app upon pressing a link on an iOS 14 device. I had a look at DestinationBuilderModifier and the content there is printed out differently.
on iOS 15 devices (Including simulator)
SwiftUI._ViewModifier_Content<NavigationBackport.DestinationBuilderModifier<MyProject.Event>>()
on iOS 14 devices (Including simulator)
(SwiftUI._ViewModifier_Content<NavigationBackport.DestinationBuilderModifier<MyProjectt.Event>>) content = {}
This is one of the places implemented that crashes the app. The navigation is done by NBNavigationDestination attached to a scroll view
NBNavigationStack {
	ScrollView {
		NBNavigationLink(value: account) {
			Text("CLICK ME")
				.Shadow()
		}
		.buttonStyle(.plain)
	}
	.setBackground()
	.nbNavigationDestination(for: Account.self) { account in
		DestinationView(for: account)
	}
}

Tested cases:

Navigation Stack with no path, does not navigate at all
Navigation Stack with path, crashes the app on second level
Navigation Stack that is added through a ViewModifier crashes the app, if generics is used

The error printed out is
2022-07-06 14:34:52.810190+0200 MyProject[99017:5417116] Fatal error: No view builder found for key MyProject.Account: file NavigationBackport/DestinationBuilderHolder.swift, line 34
Here is also another example where it works on iOS 15 and not on iOS 14
struct DestinationView: View {
    let account: Account

    @Binding var path: NBNavigationPath

    var body: some View {
      VStack {
        Text(account.name)

        Button {
          $path.withDelaysIfUnsupported { $0.append(Account()) }
        } label: {
          Text("Try me?")
        }

        NBNavigationLink(value: Account()) {
          Text("Or... try me?")
        }
      }
      .nbNavigationDestination(for: Account.self, destination: { person in
        Text("CRASHES")
      })
    }
  }

Thanks for raising this issue @GioPalusa! I tried to replicate the issue by running your code on the iOS 14.4 (iPhone 12 Pro) simulator, but wasn't able to reproduce any strange behaviour or crashes. Do you only see the issue on device or can it be reproduced on simulator? I wasn't able to include everything in your sample code (e.g. .Shadow() and .setBackground()), and I had to guess at the missing details. Are you able to provide the entire code to reproduce the issue please? Thanks!

I have tried to replicate the issue in a separate project, but haven't been able to make it crash yet. Adding things one by one and testing what it might be that crashes the app. I'll let you know!

Glad to hear it's resolved!


Fix merge operator in dhall-render bootstrap

Ah, I didn't know about the // operator. This isn't broken, closing!


Update README to add ffmpeg in requirements

Update README to add ffmpeg in requirements

❤
Thanks, @deepakmahakale!


Added .clang-format

You might want to add the Microsoft license header as a comment at the top of the file.

@djee-ms
I don't mind doing so, but we are not doing it here: https://github.com/microsoft/MixedReality-WebRTC/blob/master/.clang-format
Also, it appears to be impossible to reply to a general comments. Next time, please comment on some line of code.

Right, I fixed it on MixedReality-WebRTC too, good catch.
For the comments I didn't know. That's... not very helpful from GitHub :(


Update Alpine County, CA

Looks like there might be a typo in the schema 🤔

Ah nope, typo in the source file here. I misread the error from pre-commit. Updated!

The regexp function uses this code here. I wonder if re.sub() has a different behavior when the pattern doesn't match at all 🤔

This PR should be closed because https://github.com/openaddresses/openaddresses/pull/7520 was merged instead.  And my regexes are a lot easier on the eyes :)


Refactor Properties

This is just a few minor changes/improvements I found in the process of working on #201, but wanted to treat them separately. If it makes sense, we could hold off on this and address it as part of #193 .Though at that point it would probably be a major, breaking change, and better to incorporate as part of the STIX 2.1 branch.

Regarding @gtback comment, I think it would be easier for me to pull from master if this PR gets merged...


#91 dropDatabaseTables to include sessions table

Added the session table to the models to delete from bistarter_test database, there maybe more elegant ways how to solve it but I found it simple and clear.
any other idea?

Did you run the tests? :) I am seeing this error:
Error: ER_NO_SUCH_TABLE: Table 'bitstarter_test.sessions' doesn't exist

@karliatto Still a problem here with dropping the sessions table. See my previous comment for details.


[Data Structures using C/insertion_sort.c] Implement a program to sort an array using Insertion Sort.

Working Directory

Hey, can you please assign this issue to me!


redesign: Dropdown/misc actions

requirements: https://github.com/kodadot/nft-gallery/issues/4111#misc-section

I've already started but I might need oruga-ui (from #4305) in order to create new button component

@vladtheboi iirc had ideas on dropdown custom one we could reuse somewhere in collection view 👀


Debian script should perform upgrade

Some of the installations are failing because an apt upgrade is not performed before package installation.

I think this works now.


Trimming not working properly?

I have this as my config.yml
containers:
  bedrock:
    - name: mcbedrock
      worlds:
        - "/server/worlds/Rad World - Redux"
        - "/server/worlds/Rad World"
schedule:
  interval: 3h
trim:
  trimDays: 2
  keepDays: 7
  minKeep: 1
loggingLevel: trace

Per my understanding of the documentation, I should end up with 16 hourly backups for the last day with every day thereafter trimmed to the latest backup.
However, it doesn't appear to have been trimming the hourly backups for the previous days... I set it to 7 keepDays just to test though and it did properly trim the files to that max number of days (although still every 3 hours for each day)

Hmm, while I generally leave the trimming code alone, I did have to make some changes to support the new “extras” feature. It looks like the regression is due to that. Let me take a look tonight and see why it broke.
I should have double checked my own backups folder at the end of the testing period before moving things between the test and latest tag. It would have prevented this from rolling out to users.

Also need to make sure that I include a unit test covering this scenario to prevent this sort of regression from happening again. With the conversion of this code to something that uses protocols, unit testing the logic around trimming can be done with mocks, so unit tests need to be written.

Fix is merged, and will appear in the test tag shortly. It can take 20 minutes due to building multi-arch docker images with QEMU.
Things appear to be cleaned up on my local server with the fix, but I'd appreciate confirmation that the test tag works for you as well.


webapplication does not appear.

os: win10 64bit  .net core 2.0.2 vs2017  15.4
i followed the readme's step,finally the console window appear:
Start Electron Desktop Application...
Microsoft Windows [版本 10.0.14393]
(c) 2016 Microsoft Corporation。保留所有权利。
C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2>dotnet publish -r win10-x64 --output "C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\obj\Host\bin"
用于 .NET Core 的 Microsoft (R) 生成引擎版本 <IP_ADDRESS>001
版权所有(C) Microsoft Corporation。保留所有权利。
ASPNETCORE2 -> C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\bin\Debug\netcoreapp2.0\win10-x64\ASPNETCORE2.dll
ASPNETCORE2 -> C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\obj\Host\bin\
C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2>
Start npm install...
Microsoft Windows [版本 10.0.14393]
(c) 2016 Microsoft Corporation。保留所有权利。
C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\obj\Host>npm install

<EMAIL_ADDRESS>postinstall C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\obj\Host\node_modules\electron
> node install.js

and hanged.
then I  switch to visual stuido and pressed f5
a console weindow appeared:
info: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[0]
User profile is available. Using 'C:\Users\Think\AppData\Local\ASP.NET\DataProtection-Keys' as key repository and Windows DPAPI to encrypt keys at rest.
Hosting environment: Production
Content root path: C:\Users\Think\Documents\Visual Studio 2017\Projects\WebApplication1\ASPNETCORE2\bin\Debug\netcoreapp2.0\win10-x64
Now listening on: http://localhost:5000
Application started. Press Ctrl+C to shut down.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/home/index
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 313.9332ms 404
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 2.569ms 404
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 0.4535ms 404
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/home
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 0.2ms 404
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 0.7621ms 404
then I  open chrome and input "http://localhost:5000" ,the browser return "HTTP ERROR 404",
I don't know why, perhaps some of  my operations are  wrong?.   please help, thank you!

Mhh - do you have node.js installed? If yes, which version?

@robertmuehsig ,thank you for your quickly answer first! yes node  version 8.9.0 and npm version: 5.3.0

Could you share our project on GitHub?

@niufwshd try to attach the process with visual studio and debug your startup code.. (See the steps on readme.md)
For debug startup code, you need currenty a Thread.Sleep() on begin the startup.

@niufwshd Did you find the problem?

@robertmuehsig ,GregorBiswanger,sorry for my later reply. Today I saw https://www.youtube.com/watch?v=nuM6AojRFHk guide video. then I found my Fault. I am not very famailar with the .net core or my carelessness, In the program.cs I put this  "UseElectron(args)"  before  "UseStartup()", and  followed the Video, I switch their order,and execute dotnet restore, dotnet electronize  init,and dotnet electronize start, at last I saw the miracle!!!,very excited ! thank you very much!

If you have any questions left, just open a new issue. The order is quite important - not only for Electron, but for other ASP.NET Core packages as well. Glad Gregors video helped!


vb-315 specification change for visit filtering

Yes I think we need the param style change also - as we've agreed to go with filters.
I would expect this endpoint to now be something similar to:
/visit-sessions?prison={prisonID}


Blacklist class

Hello all,
I am new to smoothstate,  I have a light box script that does not play well with smoothstate.
How can I add a blacklist class to smoothstate?
Thank you in advance.

Hello,
You can add a blacklist class by adding a blacklist option and specifying the class/es like this:
blacklist: 'form, .btn--big, .menu-item-278 > a',
Happy to help further if needed.
Thanks


Warning in test/http.cpp while compile

[ 83%] Building CXX object test/CMakeFiles/run_all_tests.dir/http.cpp.o
/root/td/test/http.cpp: In member function 'virtual void Test_Http_reader::run()':
/root/td/test/http.cpp:148:7: warning: moving 'a' of type 'td::BufferSlice' to itself [-Wself-move]
  148 |     a = std::move(a);
      |     ~~^~~~~~~~~~~~~~
/root/td/test/http.cpp:148:7: note: remove 'std::move' call
/root/td/test/http.cpp:149:7: warning: moving 'b' of type 'td::BufferSlice' to itself [-Wself-move]
  149 |     b = std::move(b);
      |     ~~^~~~~~~~~~~~~~
/root/td/test/http.cpp:149:7: note: remove 'std::move' call
[ 83%] Building CXX object test/CMakeFiles/run_all_tests.dir/link.cpp.o

Alpine 3.19
Cmake 3.28.4

The diagnostics was added in GCC 13 and must be suppressed in the mentioned place in the same way as it has already been done for similar clang warning..


Automated cherry pick of #6451: fix: policy create fail on initialization phrase

Cherry pick of #6451 on release/3.2.
#6451: fix: policy create fail on initialization phrase

Codecov Report

Merging #6452 into release/3.2 will decrease coverage by 0.00%.
The diff coverage is 0.39%.

@@               Coverage Diff               @@
##           release/3.2    #6452      +/-   ##
===============================================
- Coverage         7.05%    7.04%   -0.01%
===============================================
  Files              649      649
  Lines           111327   111481     +154
===============================================
  Hits              7856     7856
- Misses          102862   103016     +154
  Partials           609      609

Flag
Coverage Δ

#aFlag
7.04% <0.39%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/manager.go
0.50% <0.00%> (-0.01%)
:arrow_down:

pkg/baremetal/tasks/baseprepare.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/sharablebase.go
7.08% <0.00%> (-0.09%)
:arrow_down:

pkg/cloudcommon/policy/policy.go
1.50% <0.00%> (-0.06%)
:arrow_down:

pkg/compute/models/cloudaccounts.go
0.69% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/cloudproviderregions.go
0.27% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/cloudproviders.go
0.90% <0.00%> (-0.04%)
:arrow_down:

pkg/compute/models/dbinstanceresource.go
0.00% <0.00%> (ø)

pkg/compute/models/disks.go
0.53% <0.00%> (-0.01%)
:arrow_down:

... and 20 more

Continue to review full report at Codecov.

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


Pragma(s) missing from docs

It's not in the index, but if you do a site search the word 'pragma' occurs several times. I feel it should have its own section with explanations of the various pragmas, e.g., fatal, v6, variables.

New page: https://docs.perl6.org/language/pragmas
⚠ feel free to edit this list

[x] use v6 (testing#Writing_tests)
[x]  use lib (modules#Finding_Modules)
[x]  use variables (variables#Default_Defined_Variables_Pragma)
[x]  use fatal (control#fail)
[x]  use soft (functions#Re-dispatching)
[x]  use MONKEY-TYPING (augment)
[x]  use MONKEY-SEE-NO-EVAL (EVAL)
[x]  use precompilation (see #798)
[x]  use newline (see #797)
[x]  use trace (see #800)
[x]  use strict (see #799)
[x]  use nqp (see #796)
[x]  use worries (see #801)
[x]  use internals (see #809)
[x]  use invocant (NYI) (see #810)
[x]  use parameters (NYI) (see #811)
[x]  use MONKEY-(BRAINS, GUTS, BUSINESS, TRAP, SHINE, WRENCH, BARS) (reserved, but I believe NYI)
[x]  use experimental
[x] ↑ and its experiments (see #812)

Let's create a list of all pragmas?
⚠ feel free to edit this list

use v6 (testing#Writing_tests)
use lib (modules#Finding_Modules)
use variables (variables#Default_Defined_Variables_Pragma)
use soft (functions#Re-dispatching)
use MONKEY-TYPING (augment)
use MONKEY-SEE-NO-EVAL (EVAL)

On Sunday, July 24, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Let's create a list of all pragmas?
⚠ feel free to edit this list

This description seems incomplete:

use MONKEY-SEE-NO-EVAL (EVAL https://docs.perl6.org/routine/EVAL)

The one use example doesn't show or explain the results.

@tbrowder sure! This list is for someone who will document these things properly.
By the way, should we have a separate page for all pragmas? Or perhaps we can just improve current docs, make them searchable and that's it?

On Sunday, July 24, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote:

@tbrowder https://github.com/tbrowder sure! This list is for someone
who will document these things properly.
By the way, should we have a separate page for all pragmas? Or perhaps we
can just improve current docs, make them searchable and that's it?

I like the idea of a separate page--easier to see some of the higher-level
organization of our HUGE, but wonderful, language.

On Sunday, July 24, 2016, Tom Browder<EMAIL_ADDRESS>wrote:

On Sunday, July 24, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

@tbrowder https://github.com/tbrowder sure! This list is for someone
who will document these things properly.
By the way, should we have a separate page for all pragmas? Or perhaps we
can just improve current docs, make them searchable and that's it?

I like the idea of a separate page--easier to see some of the higher-level
organization of our HUGE, but wonderful, language.

But on the page a link to the definitive source should be good enough.

On Sunday, July 24, 2016, Tom Browder<EMAIL_ADDRESS>wrote:

On Sunday, July 24, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote:

Let's create a list of all pragmas?
⚠ feel free to edit this list

This description seems incomplete:

use MONKEY-SEE-NO-EVAL (EVAL https://docs.perl6.org/routine/EVAL)

The one use example doesn't show or explain the results.

And I tried in rakudo REPL and got an error--I have on idea to make it
work--going to irc...

All pragmas are tied to some language feature. Without that context explaining them would be rather hard. So we would end up with a section that contains a short introduction followed by a table with a name and a description and a link that you have to click to understand what that stuff is for. Please apply for Approval of the Redundancy Ministry of Redundancy Approval.
All pragmas need to be in the index (easy to check) and also need to be marked as pramga.
So we need X<|use $pragmaname (pragma)> to have them all listen by any possible search term. Whereby we get redundancy already because 'use' and 'pragma' will both produce a complete list.
I'm not happy with how the index is handled in general but I can't really propose any change before we have a LaTeX randerer to see how we map our index entries to something that is a flat (or not so flat) list, in addition to a searchable list.

cosed by PR #786

@tbrowder nice!
However, I don't think that this issue is fully resolved. I have added a TODO list and marked this as big.
Also, these pragmas are not searchable yet.

On Tuesday, August 2, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote:

Reopened #752 https://github.com/perl6/doc/issues/752.

I forgot the X<> but will fix that today.

works now with my last PR ==> merge (PR #793 ); closing this again.

Nice, thanks!
However, some of these pragmas are still not described. I think that it is better to have an issue in this repo, so that people can easily see what is documented and what is not (without the need to grep for TODOs or TBDs), so I will reopen it.
In fact, maybe we should turn some other TBDs and TODOs into issues? Maybe that's not a bad idea?
Don't really want to start a close war on this, but I think that as long as the issue is not fully resolved we should not be closing the issues.

On Thursday, August 4, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote:

Nice, thanks!
However, some of these pragmas are still not described. I think that it is
better to have an issue in this repo, so that people can easily see what is
documented and what is not (without the need to grep for TODOs or TBDs), so
I will reopen it.
In fact, maybe we should turn some other TBDs and TODOs into issues? Maybe
that's not a bad idea?

Agree, putting the issues in plain site in bite-sized pieces brings more
attention and maybe encourages stomping them out.

Don't really want to start a close war on this, but I think that as long
as the issue is not fully resolved we should not be closing the issues.

No argument here!

On Thursday, August 4, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote
...

In fact, maybe we should turn some other TBDs and TODOs into issues? Maybe
that's not a bad idea?

So done.

Hm. Creating a separate issue for each item is OK too. It looks a bit weird to have so many open issues on a very similar topic, but it is probably much easier to work on the issues this way. However, when I said “other TBDs and TODOs” I meant non-pragma stuff, meaning that we should probably try to keep our TODOs on github instead of having some issues on github and other in the source. Having one ｢big｣ issue is sometimes OK, at least, I'm not willing to split every ｢big｣ issue into separate issues. But yeah, I should not be changing topics that fast :)
Having so many MONKEY-* issues around, does anybody have any idea why these pragmas were reserved? For example, when Larry reserved MONKEY-SHINE, what was he thinking? Why “shine”? And if nobody knows, what could it mean potentially?

OK, given that there is now a separate issue for each pragma we can close this.
#796 #797 #798 #799 #800 #801 #802 #803 #804 #805 #806 #807 #808 #809 #810 #811 #812

On Friday, August 5, 2016, Aleks-Daniel Jakimenko-Aleksejev <
<EMAIL_ADDRESS>wrote:

Hm. Creating a separate issue for each item is OK too. It looks a bit
weird to have so many open issues on a very similar topic, but it is
probably much easier to work on the issues this way. However, when I said
“other TBDs and TODOs” I meant non-pragma stuff, meaning that we should
probably try to keep our TODOs on github instead of having some issues on
github and other in the source. Having one ｢big｣ issue is sometimes OK, at
least, I'm not willing to split every ｢big｣ issue into separate issues. But
yeah, I should not be changing topics that fast :)

Maybe the pragma NYIs should be issues on rt.perl?

@tbrowder for “invocant” and “parameters”? Yes.
perl6 -e 'use parameters'
===SORRY!=== Error while compiling -e
use parameters not yet implemented. Sorry.
at -e:1
------> use parameters⏏<EOL>

alex@librebook:~$ perl6 -e 'use invocant'
===SORRY!=== Error while compiling -e
use invocant not yet implemented. Sorry.
at -e:1
------> use invocant⏏<EOL>

On Fri, Aug 5, 2016 at 3:01 PM, Aleks-Daniel Jakimenko-Aleksejev
<EMAIL_ADDRESS>wrote:

@tbrowder for “invocant” and “parameters”? Yes.
...

Also "internals":
perl6 -e 'use internals'
===SORRY!=== Error while compiling -e
use internals not yet implemented. Sorry.
But these give no response (e.g., implemented but not documented?):
perl6 -e 'use MONKEY-WRENCH'
perl6 -e 'use MONKEY-TYPING'
perl6 -e 'use MONKEY-TRAP'
perl6 -e 'use MONKEY-SHINE'
perl6 -e 'use MONKEY-BUSINESS'
perl6 -e 'use MONKEY-GUTS'
perl6 -e 'use MONKEY-BRAINS'
perl6 -e 'use MONKEY-BARS'

IRC log: http://irclog.perlgeek.de/perl6/2016-08-05#i_12975389

On Fri, Aug 5, 2016 at 5:12 PM, Aleks-Daniel Jakimenko-Aleksejev
<EMAIL_ADDRESS>wrote:

IRC log: http://irclog.perlgeek.de/perl6/2016-08-05#i_12975389

So, @Aleks-Daniel, what shall be done with the Pragmas page and the
doc issues and RT issues?

Alright, so after thinking about it:

Nothing should be done with the Pragmas page, except for filling in missing stuff. That is, we need actual explanations for every “TBD”. There are now issues for each missing description, so everything is fine.
I think that it is right to have an RT ticket for every NYI. So three tickets should be filed: invocant, parameters and internals (unless the tickets are already there, I was unable to find anything).
I am not sure what has to be done with doc issues. I don't like NOTSPECCED issues floating there like this (especially that amount), so I have the following idea: we should tag stuff as NOTSPECCED, but if nothing can be done now, we can close the issue. Once something is implemented, we can reopen it.

In other words: if there is nothing we can do, then we consider the issue solved at this particular moment in time. This would clear the list of open issues, and the NOTSPECCED label will make it possible to check the list periodically to make sure that nothing falls through the cracks.
Does it sound sane?

Yes, it does.
TL;DR: we close issue if the thing is not in roast, but we mark it as NOTSPECCED so that we can easily check it again later.

Also, remove NOTSPECCED tag if it turns out that this thing will never need documentation.

Unless you're adding a new pragma to the list above, I think all of them have been added, right?


Importing a group without e-mail defined in groups resource block

I am currently testing v1.1.3. I tried importing an Axual group into a TF resource and I got:
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to module.groups_shared.axual_group.<redacted>, provider "provider[\"registry.terraform.io/axual/axual\"].shared" produced an unexpected new value: .email_address: was null, but now cty.StringVal("").
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.
╵

The group resource looks like:
resource "axual_group" "<redacted>" {
  name          = "<redacted>"
  members = [
    <redacted>
  ]
}

Note, that e-mail here is not defined but since it is optional, that should not be a problem.
It seems like the import still worked though but still worth it to share I think.

I have closed this issue due to inability to reproduce it, if you are able to reproduce please open a new issue


fix(docs): div focus on link clicks

Previous Behaviour: Reload and focus
Current Behaviour: Direct Focus

Any progress?

Any progress?

We will merge it very soon =)


fast-glob>3.0.0 doesn't support backslashes as path separator, fix it by normalize the path

Hello,
The fast-glob package since 3.0.0 does not support backslashes as path separator. So i fix it by normalizing the path. I also change the test file so it can correctly detect this problem on windows platform.

Hey @qiaoruntao, thanks for your PR! I'm not using a Windows machine so I appreciate this.
Do you mind running yarn format before I merge this in?

Hello, I have run the yarn format. Thanks for you remind :smile:

@all-contributors please add @qiaoruntao for code and test


add functionality for multiple flights in a single circle.yaml

at the moment there is only functionality to have one flight in a yaml file for testing (with the get_simple_circle_times_from_yaml); it would be nice if there was an option to have a simple yaml with all the flights

closed with #115


Internal Server Error

Ever encounter any internal server errors? Running with TLS on so not sure what's up

Hmm, no, there's not a lot that can go wrong here!  I'm assuming you're running on Compute@Edge and not in the local development environment, Viceroy?
You could try log-tailing the service to see if there are panics or other runtime-related issues happening there.  If a 500 is happening, it could be a panic before the handler gets executed, but I can't imagine why.

Yeah that's true. TBH I just moved to Rust for now. I'm bummed by that but I'll hit up support.
The handler works fine except when I'm getting the JA3.
On a separate note... are you doing any normalizing of JA3's to counter Chrome's random implementation? https://hnull.org/2022/12/01/sorting-out-randomized-tls-fingerprints/

I hadn't seen this, but I'll take a look at it.  My original plan was to implement the JA3 generation myself, but I was able to farm it out to a third party library, so normalization will probably need to go upstream into that library.
I just pushed a commit which updates the tinygo build script and updates the Go SDK to the latest version.  Feel free to give that a shot, but I don't expect it to change anything.

https://github.com/AGWA/tlshacks
Blog post about it: https://www.agwa.name/blog/post/parsing_tls_client_hello_with_cryptobyte

Dang, I never saw a link to that repo. That would have saved me some time haha. Thanks for sharing it.


[storage][azblob]DownloadFile: Download large files serially

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[x] Updates to CHANGELOG.md are included.
[x] MIT license headers are included in each file.

============================================================
The existing DownloadFile method writes blocks of a large blob to the file in random order, which is not performant. This change will make DownloadFile method to write serially instead. The blocks are still downloaded parallelly.
This does not change public interface of the API. Also, this change does not impact performance of small blobs (Or blobs which fit in single chunk).
Below table makes a note of time taken to transfer a 10GiB file with existing and proposed change.
Concurrency    Current	        Proposed
Default = 5     3m22.915s	1m8.111s
8               2m22.286s       47.138s
16	        2m19.675s	47.377s

/azp run go - azblob

/azp run go - azblob

/azp run go - azblob


Add importMap to denols

I was just trying out deno and seems like the current lsp config doesn't resolve imports from importMaps because it doesn't have that capability. I added it with a sane (i think?) default.
Before:

After:

@David-Else I don't know enough about deno to know whether this is a common pattern/it will break common usecases. Can you advice? Maybe @williamboman knows as well.

https://www.npmjs.com/package/@jsenv/importmap-node-module uses project.importmap as the filename for the import map. I'm not particularly familiar with import maps but my experience so far has been that it's still highly experimental.
It's worth noting that simply adding this setting will not fix things - one would still have to actually create and manage the import_map.json file. I also don't know how well the denols server handles import_map filenames that doesn't exist, will it introduce undefined behavior or crashes?
I feel like it'd be best to leave this to user's to configure themselves, but again I don't use deno.

Thanks! Let's leave this to the users then.


Re-enable lint CI

Codecov Report

Merging #3 (c249c47) into main (b74d1ef) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          209       209
  Branches        14        14
=========================================
  Hits           209       209

Continue to review full report at Codecov.

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


Make the agent error message more informative

What problem or use case are you trying to solve?
I have tried OpenDevin for a few tasks. I have encountered a few times the error message "Agent encountered an error."  or "Agent encountered an unexpected error."
It took me a while to figure out how I could dig into docker logs to find out more.
Describe the UX of the solution you'd like
It would be helpful to directly display the error messages extracted from the relevant docker log.
Do you have thoughts on the technical implementation?
The user interface could have a new tab called agent error message to display docker log, after the tabs for Editor, Browser, Jupyter.

Duplicate of https://github.com/OpenDevin/OpenDevin/issues/1451


ECDSA: Add recovery ID to test cases

For testing an secp256k1_recover_pubkey implementation  (recover pubkey from message and signature), it would be very helpful to have the recovery ID that is available during the signing process.
The recovery ID is described here:

https://ethereum.stackexchange.com/questions/42455/during-ecdsa-signing-how-do-i-generate-the-recovery-id
https://docs.rs/ecdsa/latest/ecdsa/struct.RecoveryId.html

Right now I have it working by

Checking for comment "k*G has a large x-coordinate" and handle that separately
Try secp256k1_recover_pubkey with recover ID 0 and 1 and see if one of them worked

It does not seem to to be exactly trivial to get it from the Java implementation due to the high level APIs. This was discussed here: https://stackoverflow.com/questions/49247397/how-to-extract-public-key-from-ecdsa-signature. But maybe you have some better idea how to approach this.

I looked that this issue just briefly. It seems that public key recovery has its own set of potential implementation issues, so that it would make sense to have its own test and test vector format. For example it is possible to generate invalid signatures with the property that public key recovery leads to a public key with a point at infinity. Obviously an implementation should properly handle such situations.
Generating such a set of test vectors wouldn't be the main issue here. The first step would be to determine a test vector format that has sufficiently many use cases. There are a number of somewhat different protocols that all require public key recovery and hence it would be useful to have good coverage.  Another question is finding a good place for a potentially new set of test vectors. I.e., it would be useful to have test vectors for specialized protocols all in the same place.
The JCE interface does indeed not expose underlying EC arithmetic. BouncyCastle has some direct support, but having to rely on specific providers is of course not great. Test vector generation for the current test vectors does indeed try to generate a lot of edge cases. Hence the test vectors contain cases where recover ID 2 or 3 would be necessary.

Having dedicated tests would be ideal indeed. In the meantime I tried generating the recover IDs but did not find an implementationation that is flexible enough which I can run with reasonable setup effort.
The use case I'm most familiar with is Ethereum (which I guess is mostly the same as Ethereum). There you only require recover ID 0 and 1 since other IDs are considered invalid. It would be great to not run everything through DER but have signature = (r, s), message_hash and recover_id as inputs.
Personally I'm interested to use them for ecdsa/secp256{k,r}1. But of course, the more use cases can be covered, the better.

I would assume that the 65-byte vrs format is quite common [...] valid range for v.

The v variable is Ethereum specific and combines a network ID and a recovery ID in a single value: v = chain_id * 2 + 35 + recovery_id. Using a recovery ID of range 0, 1, 2, 3 is a generic representation.

I'm putting some experimental files here: https://github.com/bleichenbacher-daniel/Rooterberg/tree/main/ecdsa
Please note that these files are just experimental. The main goal is to simply determine the format of the test vectors.
There are essentially two options: the first one is to have test vectors with raw values, r,s and recovery_id. The second one is to have have specific encodings (such as RLP encoded message and signatures) for specific protocols. I don't have an opinion which one is more useful.
The test vector generation is nowhere near complete. The current version focuses on arithmetic errors that could occur during public key reconstruction. The construction of these test vectors depends on how EC points are represented internally. I currently use either Jacobian or affine coordinates during the construction. If there are other commonly used representations then I'll add them too.
I test my general ECDSA implementation against third party libraries and against itself, but I don't have an implementation for the Ethereum or bitcoin specific modifications. Hence it is possible that everything is completely wrong.

That's great stuff, thanks a lot for the effort!
For my purpose the split r, s, revovery_id would be ideal. IMO the protocol specifics can be done by the user of the test vectors.
My experience working with wycheproof was a bit painful as I had to write a custom DER encoder with all sorts of edge case handling just to use the test vectors but do not need DER anywhere in my project.
The expected pubkey could be included in compressed and uncompressed form maybe? Not too hard to do this in the caller but if you include both that would make it more convenient. Ethereum users will need the uncompressed pubkey (65 bytes), my API needs the compressed representation (33 bytes).

I did push some new test vectors with a new format. These are still experimental.
I agree that tester should not have to parse DER encoding if their library is not using DER encoded signatures. Currently, there are test vectors that use P1363 encoding, which are easier to parse, but it would still make sense to add additional encodings. For standards that are well defined it would make sense to add additional encoding as well. Figuring out a convenient format is one of the goals here, I think.

I implemented a prototype of some tests using these in here.
One small annoyance is that I currently have to check the comment for "(s not in range 1 .. n//2)" because all of them are marked as invalid, but we actually count high-s signatures as valid. Would be nice to add a flag for that.

I hope this works better now.
Eventually there should be some documentation listing protocols and corresponding parameters in order to simplify the selection of test vectors. Though I've just started collecting this kind of information.


Update AWS documentation for v1.0.1

Delete some legacy docs
Add more instructions to use IAM Roles for Service Account

This change is

/assign @sarahmaddox

Preview: https://deploy-preview-1804--competent-brattain-de2d6d.netlify.com/docs/aws/deploy/install-kubeflow/

Thanks @Jeffwan!
/lgtm
/approve


How use NavigationController

When I click a button on a ContentCard to go to another VC, I want that fullscreen VC to have a Navigation Bar with a "< Back" button.
How?
Thx for any help.

follow

The content card view controller is child of the VC presented to the card.
You can try setting up your navigation controller and passing it to the card.


Logbook does not support template

Made a script with:
script:
  alarm:
    alias: Alarm
    sequence:
      - event: LOGBOOK_ENTRY
        event_data:
          name: Alarm
          message: is turning {% if is_state('switch.hallen_camera_recording', 'on') and is_state('switch.printer_camera_recording', 'on') %}on{% else %}off{% endif %}

But logbook shows

This needs more information.

sorry about not providing info initially.

This functionality would not be possible using the current way logbook entries are recorded (they are just plain events).
We could provide this functionality by having the logbook expose a service that will take in a template string, render it and fire the event to record the entry.


podman run has no --type parameter

The example from README.md contains
sudo podman run … --type qcow2 --local quay.io/centos-bootc/centos-bootc:stream9

The documentation for podman run at https://docs.podman.io/en/latest/markdown/podman-run.1.html does not mention the --type and --local parameters.

These are arguments for bootc-image-builder, not for podman, they are specified after the image reference in the podman run command. Have you actually encountered an issue with the command?

Yes, you are right that the parameters are to the image, not to podman run.  I have not encountered problems with the command.
Where do I find information about the --local parameter?

ugh, it's not very well documented at the moment. The --local flag tells bootc-image-builder to use bootable container images from a local containers storage (mounted inside bootc-image-builder via -v /var/lib/containers/storage:/var/lib/containers/storage. This the recommended way to use bootc-image-builder. We even make it the default in https://github.com/osbuild/bootc-image-builder/pull/423.
If you don't specify --local, bootc-image-builder pulls the bootable container image itself.


Scan stops after sometime and there is no option to start scans from that point where the issue occured

Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 203, in _new_conn
sock = connection.create_connection(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/urllib3/util/connection.py", line 60, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/socket.py", line 962, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
socket.gaierror: [Errno -3] Temporary failure in name resolution
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 791, in urlopen
response = self._make_request(
^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 492, in _make_request
raise new_e
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 468, in _make_request
self._validate_conn(conn)
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 1097, in _validate_conn
conn.connect()
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 611, in connect
self.sock = sock = self._new_conn()
^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 210, in _new_conn
raise NameResolutionError(self.host, self, e) from e
urllib3.exceptions.NameResolutionError: <urllib3.connection.HTTPSConnection object at 0x7f010cd58e10>: Failed to resolve 'googlechromelabs.github.io' ([Errno -3] Temporary failure in name resolution)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/requests/adapters.py", line 486, in send
resp = conn.urlopen(
^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 845, in urlopen
retries = retries.increment(
^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/urllib3/util/retry.py", line 515, in increment
raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='googlechromelabs.github.io', port=443): Max retries exceeded with url: /chrome-for-testing/latest-patch-versions-per-build.json (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x7f010cd58e10>: Failed to resolve 'googlechromelabs.github.io' ([Errno -3] Temporary failure in name resolution)"))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/http.py", line 32, in get
resp = requests.get(
^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/requests/api.py", line 73, in get
return request("get", url, params=params, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/requests/api.py", line 59, in request
return session.request(method=method, url=url, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/requests/adapters.py", line 519, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='googlechromelabs.github.io', port=443): Max retries exceeded with url: /chrome-for-testing/latest-patch-versions-per-build.json (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x7f010cd58e10>: Failed to resolve 'googlechromelabs.github.io' ([Errno -3] Temporary failure in name resolution)"))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/lostools/loxs.py", line 1736, in
main()
File "/root/lostools/loxs.py", line 1732, in main
handle_selection(choice)
File "/root/lostools/loxs.py", line 1709, in handle_selection
run_xss_scanner()
File "/root/lostools/loxs.py", line 1208, in run_xss_scanner
main()
File "/root/lostools/loxs.py", line 1202, in main
scanner.run()
File "/root/lostools/loxs.py", line 1103, in run
asyncio.run(self.scan())
File "/usr/lib/python3.11/asyncio/runners.py", line 190, in run
return runner.run(main)
^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/asyncio/runners.py", line 118, in run
return self._loop.run_until_complete(task)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/asyncio/base_events.py", line 654, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "/root/lostools/loxs.py", line 1068, in scan
self.process_tasks(done)
File "/root/lostools/loxs.py", line 1037, in process_tasks
service = ChromeService(executable_path=ChromeDriverManager().install())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/chrome.py", line 40, in install
driver_path = self._get_driver_binary_path(self.driver)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/manager.py", line 35, in _get_driver_binary_path
binary_path = self._cache_manager.find_driver(driver)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/driver_cache.py", line 107, in find_driver
driver_version = self.get_cache_key_driver_version(driver)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/driver_cache.py", line 154, in get_cache_key_driver_version
return driver.get_driver_version_to_download()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/driver.py", line 48, in get_driver_version_to_download
return self.get_latest_release_version()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/drivers/chrome.py", line 59, in get_latest_release_version
response = self._http_client.get(url)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/webdriver_manager/core/http.py", line 35, in get
raise exceptions.ConnectionError(f"Could not reach host. Are you offline?")
requests.exceptions.ConnectionError: Could not reach host. Are you offline?

soon fixed ! so it can also when press ctrl+c it give only scanned results too in html report..