Number Format Exception

Hi,
I have compiled the agent. I am using mmonit-3.3, and when I startup the agent, I get the following in the server logs:
2014-12-01 12:11:39 NumberFormatException: For input string<PHONE_NUMBER>6 -- Numerical result out of range (src/util/Str.c Str_parseInt 208) when processing message posted from Monit at host '<IP_ADDRESS>' (monitid=cff13a142a4071ae9d44d6a652e4bdb1)
Please let me know what I can update to get this working?
Thanks
Raymond

Hi Raymond,
we updated the whole agent and therefore also fixed a lot of errors.
Can you please try to reproduce this Exception using the new version?
Thanks
Max


Should not call out to external processes for checking file types in caching layer

Hi,
I noticed that cached_url_opener checks the file types of cached files before opening them: https://github.com/webdataset/webdataset/blob/21a73ded7512d076c1d82e03eccfe6b16fdad8ad/webdataset/cache.py#L140
To do so, it calls the get_filetype method (https://github.com/webdataset/webdataset/blob/21a73ded7512d076c1d82e03eccfe6b16fdad8ad/webdataset/cache.py#L93), which calls out to the external file command two times. This is bad because each of those calls creates a fork of the process, which is a slow operation that should not be performed each time reading an input shard.
Moreover, the os.system call actually calls out to /bin/sh, which reads configuration files in the user home (.profile). On systems where the user home is lying on a networked filesystem, this creates additional slowness.
I have to admit that I do not have an alternative way to perform filetype validation to suggest. One way would be to simply skip validation at that point. I believe the normal tar iterator also does not perform filetype validation.

This validation is carried out on cache files only, mostly to ensure that the initial download worked correctly. It gives better error messages.
I'll see whether I can add a user validation function and/or use the Python library.
The overhead is pretty small on modern systems, and the cache directory ought to be on local disk anyway.

This validation is carried out on cache files only, mostly to ensure that the initial download worked correctly. It gives better error messages.
I'll see whether I can add a user validation function and/or use the Python library.
The overhead is pretty small on modern systems, and the cache directory ought to be on local disk anyway.

Aside from whether there is a slowdown, the system call assert os.system("file .") == 0 will generate a one-line output of  .: directory from each worker. If we don't replace it eventually, it would be nice to simply suppress this output.


#2427 deployer:push Can't open user config file ~/.ssh/config: No suc…

[X] Bug fix #2427?
[ ] New feature?
[ ] BC breaks?
[ ] Deprecations?
[X] Tests added?
[X] Changelog updated?

realpath can't use ~ in the ssh_config, therefore deployer push breaks. (more on the issue)
This fixes the problem
BUT
I don't know wether we open any security problem by adding `` to the ssh command. Can someone with more knowledge about security and consoles have a look at that?
What I do is change
$ rsync [...] -F ~/.ssh/config [...]

to
$ rsync [...] -F `realpath ~/.ssh/config` [...]

but only if a ~ is contained in the ssh-file

Hi, please rebase PR.

Fixed in b8947cf0c24aa36f4499305478c88dd3711984f7


feat(root): add new example for dropdown button with popover menu

Summary of the changes
Add new example for dropdown button with popover menu
Related issue
#1184
Checklist

[x] I have manually accessibility tested any changes, if relevant.
[x] I have raised this pull request against the develop branch

The direction of the dropdown arrow doesn't seem to update\gets out of sync with the menu open\closed state:

eg when clicking a menu item:

It only seems to change when the button itself is clicked

The direction of the dropdown arrow doesn't seem to update\gets out of sync with the menu open\closed state:
eg when clicking a menu item:
It only seems to change when the button itself is clicked

This should be resolved now


Control of device (feature query/request)

Is it possible to control the device via the serial interface? I would like to turn the device on and off, so that the laser is not running all the time (according to the manual, the device should not be worn for extended periods (more than 2 hours))

No, I'm afraid not. To my knowledge there is no built-in way to turn the device on or off remotely from the PC. As far as I remember, there is no such feature in the bundled Windows software. Nothing is impossible, of course, but not without making hardware modifications to the device. I suppose you could add a small microcontroller that listens in on the serial line for custom turn on/off commands.


refactor(errors): refactor actix_web::ResponseError for ApiErrorResponse

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[x] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Two sources of truth for which http status_code that is sent in the final response. Fixed it in this PR.
Ideally impl actix_web::ResponseError for api_error_response::ApiErrorResponse  should be removed but it results in too many changes so implemented it differently in this PR.
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

compiles
Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

This definitely looks like a quality of life change. But, earlier we were planning to refactor/remove the macro that we were using to construct the ApiErrorResponse as it has a greater overhead. That's the reason why we have these information in 2 different places. I think it's alright to remove it for now


Run spell check and markdown linter

This PR is the result of a couple tools to improve spelling and formatting. It is not a detailed editing pass.

I just merge the PR approved last week and it shows a lot of merge conflict between this PR and main branch. Conflicts should be resolved or you can publish a new PR based on the latest main branch.

Another thing is, set the target branch for "dev" for PRs in future. The main branch is release branch, which means the content of this branch would directly used by the Dev Kit. We should merge our PRs to dev branch and merge the dev into main after we make sure it's well tested. The PRs before this were set to merge into main because the Dev Kit was not released yet. Considering we're going public preview this week, we need to start following the right and cautious way to update samples.

Another thing is, set the target branch for "dev" for PRs in future. The main branch is release branch, which means the content of this branch would directly used by the Dev Kit. We should merge our PRs to dev branch and merge the dev into main after we make sure it's well tested. The PRs before this were set to merge into main because the Dev Kit was not released yet. Considering we're going public preview this week, we need to start following the right and cautious way to update samples.

Is this process documented anywhere? I didn't see a "CONTRIBUTING.md" file defining expectations of public PRs.

@AlexJerabek I will add a contributing document. Thanks for reminding.


Remove the networked image from the example for FittedBox

Description
This removes the Image.network from the example, since that causes a race condition in the test, and an image isn't necessary for the example, it works just was well (maybe better) with a Placeholder widget that shows how the fitted box affects the layout.
Related

Caused a revert of the Dart roll because of the racy test: https://github.com/flutter/engine/pull/40767

Tests

No change to the tests because the example test doesn't cover the content of the widget drawn.

@CaseyHillers here's another instance of g3 testing being blocked on a failing but ignored target.
Landing manually to unblock the Dart -> Engine roll.


Uncaught Exception ENOENT

Reported by user

That's the same issue we had before where the daemon is deleted by windows for some reason. We could not reproduce. It happened to me when my pc shut down on me once.
User needs to reinstall latest version and the problem is fixed.


Manual initialization of  CloudReactiveFeign + Eureka + Ribbon + Hystrix

public interface GreetingReactive2 {
@RequestLine("GET /greeting")
Mono greeting();
}
GreetingReactive2 client = CloudReactiveFeign.builder(WebReactiveFeign.builder())
.enableLoadBalancer(s -> LoadBalancerCommand.builder()
.withLoadBalancer(AbstractLoadBalancer.class.cast(getNamedLoadBalancer("greeting-other")))
.withRetryHandler(new DefaultLoadBalancerRetryHandler(1, 1, true))
.build())
.target(GreetingReactive2.class, "http://" + "greeting-other");
client.greeting().subscribe()
application.yml
greeting-other:
ribbon:
# the eureka vipAddress of the target service
DeploymentContextBasedVipAddresses: web-flux-app
I am not able to configure following line
.withLoadBalancer(AbstractLoadBalancer.class.cast(getNamedLoadBalancer("greeting-other")))
Calling "client.greeting().subscribe()" throws timeout error.
I spoilded whole day figuring out how to get ribbon loadbalanced instance to be passed to
AbstractLoadBalancer.class.cast(getNamedLoadBalancer("greeting-other")) .
Please help.

The latest version doesn't support Ribbon Load balancer
Check this functional test. You may use it as sample.
https://github.com/Playtika/feign-reactive/blob/develop/feign-reactor-cloud/src/test/java/reactivefeign/cloud2/LoadBalancingReactiveHttpClientTest.java
Which version of library do you use?


[improvement](group commit) make get column function more reliable when replaying wal

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Make this package platform independent

Unclear if this package should keep the common setup.py or not. Moving it out might be the correct call.
Once that's figured out, this could consume vcorelib and we could start to build out a Python-based task /dependency resolver that takes precedent over running make. It can leverage multiprocessing

We should probably just make a pep517-compliant build backend that just wraps setuptools..

something like buildpolyfill or setuptools-polyfill ?

We were able to refactor out the build stuff in here, currently working on https://github.com/vkottler/workspace/issues/8

Added macos on https://github.com/vkottler/vmklib/pull/39

Blocked by https://github.com/vkottler/vmklib/issues/36 to get it running on Windows.

CI runs Windows now.


Footer does not show logos of the socials

The social logos are not visible in the footer
I want to fix that

@codervivek5 Please assign this

hey I have solved this issue and send pull request of it please accept it as I am a GSSoC'23 contributor.


Update to Ruby 3.0.0

This should improve performance across the board 🤞

We have a good number of blockers to work out before we can do this one. This PR attempts to add testing to CI which tests out how the app performs on Ruby 3. We have issues right now with ruby-shadow, inspec (fixed, but not yet merged to chef/chef) and ffi. It's happening, but there's a TON of little things to handle first.

We have a good number of blockers to work out before we can do this one. This PR attempts to add testing to CI which tests out how the app performs on Ruby 3. We have issues right now with ruby-shadow, inspec (fixed, but not yet merged to chef/chef) and ffi. It's happening, but there's a TON of little things to handle first.


PR: Forward and Raise actual error for python 2

Description of Changes

On Python 2 the log_lsp_handle_errors method was not working properly as stack_info is not a valid kwarg in Python 2
The following code was used as example:
# -*- coding: utf-8 -*-
"""Example."""

def hello(text='', number=1):
    """
    Returns
    -------
        Testing
    """
    print(text, number)

hello(

After typing the ( the error dialog was triggered.
Actual error on Python 2

Simulated error for Python 3

After fix on Python 2

Issue(s) Resolved

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

Hello @goanpeca! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/widgets/tests/test_codeeditor.py:

Line 9:42: W291 trailing whitespace


panic: runtime error: integer divide by zero

Trying to run a mixedworload and it crashes.
root@VM109:~/s3tester-v2.1.0-linux-x64# screen -S s3tester ./s3tester -concurrency=5 -size=1048576 -requests=2000 -endpoint=https://xxxxxxxxxx:443 -prefix=mixedworkload -bucket=bhaskar04022020 -workload test.json
panic: runtime error: integer divide by zero

goroutine 1 [running]:
main.calcStats(0xc000ba76a8, 0x5, 0x1d59de)
        /home/vlad/go/src/github.com/s3tester/s3tester/s3tester.go:460 +0x18b
main.setupResultStat(0xc000ba76a8)
        /home/vlad/go/src/github.com/s3tester/s3tester/s3tester.go:448 +0x49
main.processTestResult(0xc000ba76a8, 0x5, 0x100000, 0xc00007d800, 0x1, 0x1, 0x957efc, 0x3, 0x7fffcd918204, 0xf, ...)
        /home/vlad/go/src/github.com/s3tester/s3tester/s3tester.go:435 +0x74
main.runtest(0x5, 0x100000, 0xc00007d800, 0x1, 0x1, 0x957efc, 0x3, 0x7fffcd918204, 0xf, 0x7fffcd9181ee, ...)
        /home/vlad/go/src/github.com/s3tester/s3tester/s3tester.go:187 +0x1cd
main.main()
        /home/vlad/go/src/github.com/s3tester/s3tester/s3tester.go:576 +0x90d

root@VM109:~/s3tester-v2.1.0-linux-x64# cat test.json
{'mixedWorkload':[{'operation':'put','ratio':25},{'operationType':'get','ratio':25},{'operationType':'updatemeta','ratio':25},{'operationType':'delete','ratio':25}]}

Not sure what am I missing.

Same above issues observed when try running mixedworkloadprofile:
[root@eosnode-1 ~]# go run s3tester -bucket test1 -concurrency=10 -size=50 -requests=50 -endpoint="https://s3.seagate.com" -prefix=3 -workload=workload.json
panic: runtime error: integer divide by zero
goroutine 1 [running]:
main.calcStats(0xc0000f55b0, 0xa, 0xcfb72f)
/root/go/src/s3tester/s3tester.go:460 +0x188
main.setupResultStat(0xc0000f55b0)
/root/go/src/s3tester/s3tester.go:448 +0x49
main.processTestResult(0xc0000f55b0, 0xa, 0x32, 0xc0000819e0, 0x1, 0x1, 0x9bb445, 0x3, 0x7fff1b109767, 0x5, ...)
/root/go/src/s3tester/s3tester.go:435 +0x74
main.runtest(0xa, 0x32, 0xc0000819e0, 0x1, 0x1, 0x9bb445, 0x3, 0x7fff1b109767, 0x5, 0x7fff1b1097bc, ...)
/root/go/src/s3tester/s3tester.go:187 +0x1cd
main.main()
/root/go/src/s3tester/s3tester.go:576 +0x925
exit status 2
[root@eosnode-1 ~]#

This was due to an error int he workload file. The example in the readme was fixed by #23 . The same problem is discussed here and solved by changing the first occurrence of "operation" to "operationType". See #24

This was due to an error int he workload file. The example in the readme was fixed by #23 . The same problem is discussed here and solved by changing the first occurrence of "operation" to "operationType". See #24


show results on the ctrl+f highlighter without clicking

fixes #1521

with one view open it is exact as wanted. But we can open any view multiple times and also different views. When there are more occurencies we should better select which is wanted, see example

somehow the result is also interesting.  This would mean highlight all occurencies of ... may be we should have an option.
Besides navigation a user to a button/text I was also thinking about using this in  the tutorials. On different OS,Screen,Desktops the fonts are different. I want to simplify the navigation for pyautogui, e.g.
I can send the keystroke to pyautogui and then look for:
pag.locateCenterOnScreen(picture('view', 'hightligted.png'))
Please can you take care that when it is multiple times is in the selection to force the user to choose.
For the other case (highlight multiple times) we would need a checkbox to enable this.

Good Evening @ReimarBauer
I will look into the code and try to limit it to one occurrence.
To be sure these are the changes requested?
-> add a checkbox (highlight multiple times)(if True) to implement the filter as it is working now.
-> checkbox (highlight multiple times)(if False) the user has to manually go and click on the item to highlight it
-> in the case of only one view opening the filter should behave as it is working now.

Moin Shubh Gaur :)
thx. for looking into this.

-> add a checkbox (highlight multiple times)(if True) to implement the filter as it is working now. : Yes
-> checkbox (highlight multiple times)(if False) the user has to manually go and click on the item to highlight it (False,, should be default checked), Yes
-> in the case of only one view opening the filter should behave as it is working now. Yes

Thx
(Meaning of Moin, https://moinmo.in/MoinMoinEtymology, it is a timeless greetings, didn't know timezones ;) )

Great! Will get back to you by tomorrow.
I am from India about 4:30 hrs ahead of you (2:25 am here).

Great! Will get back to you by tomorrow. I am from India about 4:30 hrs ahead of you (2:25 am here).

Good night :)

For changing the ui_shortcut.ui please use the designer command and convert is to a py script by pyuic5. Both are in the environment installed.

I am looking into why the tests failed.

Hi
I looked into why the test develop was failing, so now when we change the state of the new checkbox we want it to trigger the filter_shortcuts but the state change function[self.cbHighlight.stateChanged.connect(self.filter_shortcuts)](Line 163) can not pass the text parameter to the filter_shortcuts method. So I edited the method to assign the value of text inside the method itself to integrate the new Checkbox feature.
Why the test is failing:-

when we run Line 103 the filter_shortcuts method is called and the text is assigned "Nothing", but inside the method, the text variable is reassigned [text = self.leShortcutFilter.text()](Line 302). as a result the treeWidget is not empty and the assert at Line 104 failed.
Fixes
-> We can change the way we are calling the filter_shortcuts method in the test_msui.py file

I will make a commit for the same

Hi I looked into why the test develop was failing, so now when we change the state of the new checkbox we want it to trigger the filter_shortcuts but the state change function[self.cbHighlight.stateChanged.connect(self.filter_shortcuts)](Line 163) can not pass the text parameter to the filter_shortcuts method. So I edited the method to assign the value of text inside the method itself to integrate the new Checkbox feature.
Why the test is failing:-  when we run Line 103 the filter_shortcuts method is called and the text is assigned "Nothing", but inside the method, the text variable is reassigned [text = self.leShortcutFilter.text()](Line 302). as a result the treeWidget is not empty and the assert at Line 104 failed.
Fixes -> We can change the way we are calling the filter_shortcuts method in the test_msui.py file

great analyses and good solution :)

This is now much better than before. One last cosmetic change please. It is more about what gets into our mind when we in a year read again the lines.

Sure! will do that.


Xiaomi Philips Ambient Light state is not updated

Home Assistant release with the issue:
0.80.0
Last working Home Assistant release (if known):
n/a
Operating environment (Hass.io/Docker/Windows/etc.):
Virtualenv on Raspberry Pi
Component/platform:
https://www.home-assistant.io/components/light.xiaomi_miio/
Description of problem:
After https://github.com/home-assistant/home-assistant/issues/16269 has been fixed, Xiaomi Philips Ambient Light finally started to work, but there are still some issues with updating the state:

If both main Philips and Philips Ambient Light are turned on and you turn off main Philips (either on the lamp or using HA), Philips Ambient Light turns off as well, but HA continues to think that it turned on forever.
If everything is turned off and you turn on Philips Ambient Light using HA, main Philips turns on as well, but HA continues to think that it turned off until next sync in XX seconds.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:
  - platform: xiaomi_miio
    name: philips
    model: philips.light.sread1
    host: <IP_ADDRESS>
    token: 846e47c324dbaeb7e9215fb85bae5378

Traceback (if applicable):
n/a
Additional information:
n/a

@OleksandrBerchenko I believe I've seen the same behaviour using Xiaomi Home app and that got me wondering if it's not a bug in lamp's firmware, etc. Can you let me know if your lamp behaves the same?

Mi Home application doesn't allow me to turn on/turn off Ambient Light and doesn't show its status at all.

This behavior is firmware related and cannot be changed.


build(nestjs): update nest dependencies

Just had to upgrade a project I work on and I noticed that this library, which we use, was a bit outdated :)

Did you test this version with a production app?

Did you test this version with a production app?

Doing so right now on a controlled environment, so far so good, and if everything goes right, I'll update my production environment as well (which won't have a different result from my controlled environment).
Give me a few hours from now (maybe around 16 hours, since that's when I'll get back to work), and I'll update this PR letting you know if everything is working as it should.

Sounds good. Thanks for the pull request.

Can you also upgrade the example repository? @KiriLucas

Can you also upgrade the example repository? @KiriLucas

I just did it! And my tests went fine, nothing out of the ordinary happened. I talked with some coworkers that work on another project and they have been testing this version for almost a week without any issues related :)

Thanks! I'll take it from here.


Fixs typo in README.md

Fixs typo: "vizualize" to "visualize" in line 312.

Rebuilding. It seems our gomodules build failed with an error similar to what was reported in https://github.com/prometheus/prometheus/pull/4802#issuecomment-451347507

Getting some gomodules build errors still:
go<EMAIL_ADDRESS>bzr branch --use-existing-dir https://launchpad.net/~niemeyer/gocheck/trunk . in /home/travis/gopath/pkg/mod/cache/vcs/f46ce2ae80d31f9b0a29099baa203e3b6d269dace4e5357a2cf74bd109e13339: exit status 3:
	bzr: ERROR: Cannot lock LockDir(file:///home/travis/gopath/pkg/mod/cache/vcs/f46ce2ae80d31f9b0a29099baa203e3b6d269dace4e5357a2cf74bd109e13339/.bzr/branch/lock): File exists: u'/home/travis/gopath/pkg/mod/cache/vcs/f46ce2ae80d31f9b0a29099baa203e3b6d269dace4e5357a2cf74bd109e13339/.bzr/branch/lock': [Errno 17] File exists: '/home/travis/gopath/pkg/mod/cache/vcs/f46ce2ae80d31f9b0a29099baa203e3b6d269dace4e5357a2cf74bd109e13339/.bzr/branch/lock'

I don't know exactly what happens and why it starts falling apart just now but https://github.com/prometheus/prometheus/pull/5057 should fix it eventually.

CI errors should be fixed by https://github.com/prometheus/alertmanager/pull/1692

@JoeWrightss you'd need to rebase on master to pass the CI.

@mxinden, @simonpasquier , Thank you for guiding me, I have rebase on master🐯.

Thanks @JoeWrightss.


Have a mini button to allow instant copy of wallet address

Works like "copy link" button on Google Drive.

We should do the same thing with new UI.


feat: introduce createFactory() and createHandlers()

I made #1722 a draft and create this new PR instead it.
This PR introduce Factory class, createFactory(), and createHandlers() in Factory exported from hono/factory.
import { createFactory } from 'hono/factory'
import { logger } from 'hono/logger'

const factory = createFactory<Env>()

const middleware = factory.createMiddleware(async (c, next) => {
  c.set('foo', 'bar')
  await next()
})

const handlers = factory.createHandlers(logger(), middleware, (c) => {
  return c.json(c.var.foo)
})

app.get('/api', ...handlers)

This is useful for defining handlers outside of app.get('/api', ...). Using this approach, you can structure your application in a Ruby on Rails-like Controller pattern, as discussed in #1072.
A key feature is its ability to correctly infer types.

Why we can't call createHandlers()directly
Ideally, you might write shortly as follows:
import { createHandlers } from 'helper/factory'

However, there is a problem that passing Env as follows will break other types:
createHandlers<Env>()

createMiddleware()
You can use createMiddleware as before:
import { createMiddleware } from 'helper/factory'

Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

Hi @usualoma
Please review this one!

@usualoma
Thanks! I'll merge this into the "next" later.


Fix SPM Usage in Xcode 12.5+

Description
On Xcode 12.5, apparently the umbrella header generation logic changed, so this stopped working on my project because my project couldn't find MXScrollView anymore, when importing this lib using SPM. The problem was that the generated umbrella header didn't contain references to all the public headers. This PR aims to fix that.
Motivation and Context
How Has This Been Tested?
It compiles on my project now 😄
Before:
import MXParallaxHeader
…
let scrollView = MXScrollView // Couldn't find reference to MXScrollView

Now, the same code above compiles just fine.
Types of changes

[x] 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)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation. - Nope
[ ] I have updated the documentation accordingly. - There's no need to update anything

This is a PR, it's not stale 😢

Still valid

Still valid.

@maxep could this PR be reviewed any time? 🙇

@maxep hi!👋 feel free to add me as a contributor! Happy to move things forward :)
Just make sure I have enough permissions to publish releases (and maybe pods?), so I don't have to bother you again 😄 cheers!

@rogerluan, super nice, thank you!
So I've added you as a collaborator on this project. I've tried to add your email (from your GH profile) to cocoapods, but it seems that you are not yet registered. Could you please register? 👉 https://guides.cocoapods.org/making/getting-setup-with-trunk.html#getting-started
Thanks again!

I'll do that tomorrow! 🤗

Done @maxep ! :) Same email as my GH profile


istio - rewriteAppHTTPProbers default value true causes connection pooling issues for probe requests

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Noticing that when sidecar.istio.io/rewriteAppHTTPProbers: 'true' nginx is not able to pool this connection between the istio-proxy sidecar and container (nginx).
I print access logs of nginx with $connection and $connection_requests and with both liveness and readiness probes running every 10 sec rewriteAppHTTPProbers with true and false here is what I get:-
/rewriteAppHTTPProbers: 'true
Every time new connection. Only 1 request per connection.
{ connection: 8,  connection_request: 1, path: "/liveness"}
{ connection: 7,  connection_request: 1, path: "/readiness"}
{ connection: 6,  connection_request: 1, path: "/liveness"}
{ connection: 5,  connection_request: 1, path: "/readiness"}

/rewriteAppHTTPProbers: 'true'
nginx is able to use same connection. Multiple requests per connection..
{ connection: 5,  connection_request: 4, path: "/liveness"}
{ connection: 5,  connection_request: 3, path: "/readiness"}
{ connection: 5,  connection_request: 2, path: "/liveness"}
{ connection: 5,  connection_request: 1, path: "/readiness"}

So by setting rewriteAppHTTPProbers: 'false' my app is benefiting from it as long as concurrency is NOT set to 0
. When concurrency is set to 0 I get no benefit of rewriteAppHTTPProbers: 'false'
proxy.istio.io/config: |
          holdApplicationUntilProxyStarts: true
          concurrency: 0

I think  rewriteAppHTTPProbers should not have any effects on connection pool on the container side.
Setting concurrency: 0 should affect only threads and resources on the sidecar while allowing containers todo connection pooling.
Version
# i do not have access to istio-system namespace but the version is Istio Version

    v1.20.2
docker.io/istio/proxyv2:1.20.2 is proxy

$kubectl version
Client Version: v1.29.1
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.5-eks-5e0fdde

### Additional Information

Additional details:
- probes are at http1.1

kubelet itself should not be reusing the connection; istio-agent mirrors that logic. Is that who is sending the traffic? Or some alternative 3rd party?


Phase-1 Validation

close #15

inOutZeroSum :: P1Validator
inOutZeroSum b@(unpack -> BB{..})
    | i <> m <> bbFee == o = Success b

zero sum check does it

Awesome, missed that.

I removed the type synonym, but now, I don't really like how checker is constructed. Since Checker cannot get functor nor contrafunctor, I don't really see a good way of combining them :thinking:

You can make Checker a synonym over a -> Validation (Acc CheckerError) (), which is happily Contravariant, Divisible and Decidable. We can do this because Checker doesn't return a useful value: this means we can do
runChecker :: forall (a :: Type) . Checker a -> a -> Validation (Acc CheckerError a)
runChecker (Checker comp) x = comp x $> x

Can we just make checkers return Acc CheckerError? It seems to work fine and we don't have to deal with Validation.
I'll push the update in a moment with this change.

That also works. Still has all the instances too.

This is superceded by #20, right?

That's correct.


Normalizing exported struct fields

Hey there :)
Please consider the following:
type Instance struct {
	PublicAddress  string
	PrivateAddress string
	Name           string
	Labels         map[string]string
}

program, err := expr.Compile(target, expr.Env(Instance{}), expr.AsBool())
.......

I can now evaluate the following expression Name == "foobar" but not name == "foobar" since Name is an exported field and thus starts with a capital letter. What would be the best practice here to be able to use lowercase/case insensitive field names in the expressions?
Thanks

What about adding expr.Option for case insensitive vars and names?
expr.CaseInsensitive()

That would be handy :)

PR are welcome ;)

Haha yep, i'm gonna look into it :)

Now Expr has expr:"name" tags!


Performance issues when used inside ng-repeat

Hello!
This datepicker is awesome. Thanks!
I have a list, which has dates for each item. Unfortunately it gets really slow to render all items.
Is there any better way to use it instead of using a  inside the repeat? Or am I missing something?
Thanks

May be you can give it a try for modal instead of a popup. The modal is working fast than the popup.

Actually I'm already using modal

Instead of using the component multiple times, you can use it only in one place and you can reuse the same for the other places.


new line after page size info print

A new line at the end of page size info printing would be good

:+1:


Feature/add integration test for chunking

This is a port of the chunking test from the Java SDK
(https://github.com/DKE-Data/agrirouter-sdk-java/blob/main/agrirouter-sdk-java-tests/src/test/java/com/dke/data/agrirouter/test/usecase/SendAndReceiveChunkedMessagesTest.java)

Funny difference... the Java Test (https://github.com/DKE-Data/agrirouter-sdk-java/blob/main/agrirouter-sdk-java-tests/src/test/java/com/dke/data/agrirouter/test/usecase/SendAndReceiveChunkedMessagesTest.java) comes out with 3 chunks for the same file?


feat(styles): set initial component visibility to hidden to prevent flickering

I see the styles being applied in Storybook examples, but not at the Component playground.

I see the styles being applied in Storybook examples, but not at the Component playground.

@myrta2302 the component playground probably does not load all the styles necessary. You can check the index.html in the components package src folder to see what's exactly being loaded.


Add binary payload format

#26

Hey @Larsus, great work -- I've tried the journal part and it works for me (haven't tried the snapshots though). @Aaronontheweb, is this PR going to be merged?

@felix-b let me take a look at the test results. I'm adding this to my to-do list for the week.

This still doesn't appear to work correctly with manifest serializers... Need to investigate further.


How do I integrate ollama as a repo explorer model

What problem or use case are you trying to solve?
Describe the UX of the solution you'd like
Do you have thoughts on the technical implementation?
Describe alternatives you've considered
Additional context
I would like to  use the latest ollama 3.2 model as the model for using as repo explorer model. How do I do this with ollama serve. what is the api key.
It would be great if its linked in some doc.

The API key for ollama can be whatever you like. I used "dummy" or "ollama" for example.
That will enable you to use models served via ollama in openhands. Please take a look at this doc too.


Support for ignore

Is Papertrail's ignore supported? If so, what is the syntax? Thanks!
https://github.com/paper-trail-gem/paper_trail#2c-choosing-attributes-to-monitor

Hey @bjacobs09, honestly looks like not.
I didn't support it for a while because of a missing sequel at the current project.
Please feel free to contribute :)


Input : interact_text, + sign causes error

On Windows 10 machine, using Powershell shell, pressing + sign while using Input prompt in interact_text mode, causes following error:

Code used is :
use dialoguer::Input;

fn main() -> std::io::Result<()> {
    let input = Input::<String>::new()
        .with_prompt("Data: ")
        .interact_text()?;

    Ok(())
}

I'd expect + sign to be rendered or at least not throw error. This error might occur for more characters, but this is the one I found.

Recently came across the same Not a terminal error in Powershell on Windows 10 (through VS Code's integrated terminal, although that probably doesn't matter much) when the SHIFT key was used as a modifier.

Fixed by 0.8.0 release


Don't render DayPickerSingleDateController when collapsible panels

When I have 'N' calendars, each one on a collapsible accordion bootstrap panel (react-bootstrap), if it's hidden first time when it renders, it doesn't appear again. Here is the example code https://codesandbox.io/s/6j10z2m8vn

This issue related with #46

Same happening with version 21.8.0.


Add AED currency to Blink

User has requested to add AED currency to Blink
https://en.wikipedia.org/wiki/United_Arab_Emirates_dirham
Represented with symbol AED in place of dollar sign

I have this symbol د.إ can you confirm?

"That's Arabic. In English it's "AED"
Sounds like best is to stick with english letters for symbol

added 3 months ago


Refresh specs for String#unpack and String#unpack1

Similar to  #2110:  a review of this change does not add anything.


on gpu :InternalError (see above for traceback): NormalizeWithDimensions launch failed

I run it use gpu,but i get this issue,how can i sovle it?Thanks very much.
InternalError (see above for traceback): NormalizeWithDimensions launch failed
[[node norm1 (defined at /home/linux/codes/test/old/Tensorflow-SegNet-master/model.py:177)  = LRNT=DT_FLOAT, alpha=0.0001, beta=0.75, bias=1, depth_radius=5, _device="/job:localhost/replica:0/task:0/device:GPU:0"]]
[[{{node loss/total_loss/_49}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_1720_loss/total_loss", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]

I met the same problem. Have you solved this problem?


[Status indicators] Need to update/confirm the Orange color on the Status palette

In the table that shows the icon it mentions orange 40 and orange 60, the status palette shows the  orange 50 instead.
We need to confirm if It meant to be 40 with 60 stroke or 50 with a 60 stroke. If 50, then that is the same on light or dark with only the 60 stroke as the difference in the themes.

Update (2024)
After discussing with Jeannie and Michael Gower:

[ ] Remove the stroke
[ ] Make sure the symbols meet the contrast requirement with the outer shape

Diana is also working on proposing consistent to colors on this page.

This issue will be resolved by #17117


See if Partial* estimators can be replaced with make_streamable function

make_streamable(Estimator) - > Estimator

This would also let users easily wrap models not in scikit-learn, like in https://github.com/arthurmensch/modl

Behavior-wise, it'll be identical to e.g. dask_ml.linear_model.SGDClassifier. The wrapped .fit method will pass each block of a dask array to the underlying scikit-learn model's .partial_fit method.
The goal is to make things a bit easier for users to wrap arbitrary scikit-learn compatible .partial_fit method.
It'd be make_streamable(MyCustomEstimator()) instead of
class MyStreamableEstimator(_BigPartialFitMixin, MyCustomEstimator)

est = MyStreamableEstimator()


Using Google Analytics

Hey. I see there's a file in the Node SDK for using Google Analytics. Is there a tutorial for setting this up? What does this track?
Best,
Michael

It's the analytics code for documentation not for SDK. If you want to use analytics for your bot you can integrate something npm like universal-analytics.


[$500] IOU - Delay in green dot and total amount update when making net amount as 0 and reverting it

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.19-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: User A has requested money from User B

Log in as User B
Go to 1:1 DM with User A > Click IOU preview
Request the same amount from User A to make the net owe become 0
Note the the green dot for the main chat in LHN
Return to main chat
Return to the IOU report
Delete the requested amount created in Step 3

Expected Result:
In Step 3, once the net owe becomes 0, the green dot for the main chat in LHN will disappear
In Step 7, the new amount should update instantly
Actual Result:
In Step 3, once the net owe becomes 0, the green dot for the main chat in LHN does not disappear, unless the main chat is visited
In Step 7, the total amount remains at 0, unless the report is revisited
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/9662059f-9076-4743-be9f-fdbbf9dbeaf2
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017d07091eedfc5a43
Upwork Job ID:<PHONE_NUMBER>445937664
Last Price Increase: 2023-12-29

Proposal
Please re-state the problem that we are trying to solve in this issue.
This is happening because we do not update the hasOutstandingChildRequest of the parent 1:1 report when requesting the money.
What is the root cause of that problem?
In IOU.getMoneyRequestInformation, we need to calculate the total amount of the IOU Report, and if it's 0, optimistically set the chat report's hasOutstandingChildRequest to false.
What changes do you think we should make in order to solve the problem?

What alternative solutions did you explore? (Optional)

@lanitochka17 I don't think the title is very clear.
Am I correct in my understanding that the title should be something like: Delay in updating green dot and total amount when the net amounts cancel each other out for 2 users?

@tienifr's proposal looks good to me.
🎀 👀 🎀 C+ Reviewed

@mountiny Can you clarify the above comment?

Well regarding that line, thats about request money and setting the GBR optimistically which only happens in the workspace/ expense report case. You cannot request money from yourself in IOU case

@jliexpensify Updated the title, thank you!

https://github.com/Expensify/App/issues/33774#issuecomment-1877360599
Is there an additional question that there is confusion with?

Ah interesting, yeah I did not consider the case of deleting money request or requesting more than what you are owed to remove the GBR.
So yeah in that case we have more routes to cover.
Requesting more than what you are owed in IOU case will remove the GBR.
This would also mean deleting money request on IOU or expense report, the GBR might change
Editing amount on the IOU report can change the report owner/ manager and update the GBR as well.
Could we handle all of these cases in this one issue? I think its a fair scope given the changes are laid out clearly code-wise

@tienifr @shubham1206agra Can you note if you need any help with the additional cases?

@thienlnam @tienifr I have made a backend PR for this https://github.com/Expensify/Auth/pull/9653
So this shoudl be resolved next week if there is anything else you need help with let us know

Not overdue

Not overdue, but am I correct in saying that Vit's PR was deployed and this should be now fixed?

Vit's PR just unblocked the piece needed for the FE PR. We're still waiting for a FE fix here

Aha, thank you for clarifying!

Not overdue!

@shubham1206agra Hi, looks like you need to review the PR?

I am OOO right now. Will review it on 30th

Payment Summary

Contributor: @tienifr $500
C+: @shubham1206agra $500 - Please complete checklist!

NEW UPWORKS JOB

Paid and job closed


prayer: use html-tidy 5.6

Description of changes
The prayer package requires an older version of html-tidy as dependency due to prayer relying on private library functions of libtidy which have become hidden in versions greater 5.6 of html-tidy.
ZHF: #265948

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

You're right. I just looked into getting this to build again, but there seems to be a lot of other stuff wrong with this now. the icons folder is placed right next to bin and share etc. it should probably be somewhere below share/prayer/ it also contains an etc which should probably not exist. I can confirm the reply from the server is empty. There's also a few hardcoded absolute paths in one of the config files in the etc folder, like /usr/bin/passwd.
There's also no maintainer and it hasn't been touched in almost 4 years. It just happen to break recently when html-tidy got updated. I guess it's not worth the effort since nobody seems to use it. What do you think?

Oops. Misclick.

Oh wow. Here's to more facts.

hydra build failed since 2021, which means no active users.
and the sources can't be fetched anymore, so if hydra ever got rid of them it'll be lost anyway.

Yeah I think it's overdue to be marked broken, or even just removed entirely.

This time intentionally.


database connection

AttachDbFilename=|DataDirectory|FlightReservationSystemContext-20151113121316.mdf
file not included. am having problem with this.
Error =
A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: SQL Network Interfaces, error: 50 - Local Database Runtime error occurred. The specified LocalDB instance does not exist.

Try to create new MySQL Database with name FlightReservationSystemContext-20151113121316 or with any name and mention it in web.config file.
The DBInitializer (FlightReservationSystem/Models/DbInitializer.cs) will populate the tables and data into it, when the project is run.

Got the same error from above

getting error database already exist.


Completion queue draining blocks until all pending alarms fire.

Re-creating #16591 as I found no way to reopen it.
Current behavior requires all alarms to be tracked by the application, so that they can be cancelled manually alongside the server shutdown. Otherwise, clean shutdown is impossible, or may take a long time.
I'm looking for confirmation that it is intended behavior.
What version of gRPC and what language are you using?
1.13, 1.14
C++
What operating system (Linux, Windows, …) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
GCC 8.2
What did you do?
#include <chrono>

#include <grpc/support/log.h>
#include <grpcpp/alarm.h>
#include <grpcpp/grpcpp.h>

using namespace std::chrono_literals;

int main() {
  ::grpc::ServerBuilder builder;
  builder.AddListeningPort("[::]:0", grpc::InsecureServerCredentials());
  auto          cq     = builder.AddCompletionQueue();
  auto          server = builder.BuildAndStart();
  void*         tag;
  bool          ok;
  ::grpc::Alarm alarm;
  alarm.Set(cq.get(), ::std::chrono::system_clock::now() + 5s, (void*)1);

  server->Shutdown();
  cq->Shutdown();

  GPR_ASSERT(cq->Next(&tag, &ok));
  GPR_ASSERT(ok);
  GPR_ASSERT(tag == (void*)1);

  return 0;
}

What did you expect to see?
I expected the application to exit immediately after shutdown. I expected the pending alarm to be cancelled.
What did you see instead?
The application blocked for the full duration of the alarm (5s in this case).
Anything else we should know about your project / environment?

I have reopened the original, will close this as duplicate of #16591


TLS Certificates

If a new domain name is decided (e.g. d-craft.diglib.org), new certificates will need to be deployed n the certbot chain.

Update WP domain settings (use http prefix)
Generate new cert on the wordpress.clir.org cert
Remove "old" domain name (so certs will properly upgrade themselves)
Update WP domain settings to use https prefix

Not necessary as they kept the same domain name


Implement testing tab to match the MacOS front-end

Following the OSX front-end, merge the skein and transcript into a new "Testing" tab. There are several related changes associated with this:

Remove the "Replay" menu.
Allow different selections in the two skein panels.
Selected knots in the skein don't need to be ones playing.
The skein is no longer horizontally centred in the tab if less wide than the tab.
Reduce spacing between skein knots.
"Test Me" knots have italic knots below them for the output they produce.
Drag and drop of skein nodes with animation.
Undo and redo of skein, change trim warning.

Started work on this on the "testing_tab" branch.

The main body of this work has now been merged to master.

The latest MacOS front-end does not support labels/annotations in the Testing tab, except for the case of one knot, which can be marked as the "winning knot" (i.e. an annotation of "***"), like this:

Commit b3745fa67c433b6cb7f5fc5c844cbac2d9f684e0 makes labels/annotations consistent with the MacOS front-end: labels are no longer directly available, except for the "Set Winning Command" menu option.

Commit 7e5d3bc0cb77f8f5fe4c13987d7c4746ff95918b adds splitting up the output of "test me" commands into test sub-item nodes in the skein.

Commit c092cefa8269bf19347d96b18b4b0eee9e0e75e3 adds drag and drop to the skein.


docs: Add heimatverein-niederjosbach.de to the showcase

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Hello again,
I've merged your contribution as  2ece1b3. Your website should be listed within the next minutes as well.

Awesome, thanks! :)


AOT Bug? : Calling function 'makeDecorator', function calls are not supported

I'm submitting a ...  (check one with "x")
[ x ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
$ ng b --prod
...
ERROR in Error encountered resolving symbol values statically. Calling function 'makeDecorator', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol Injectable in<EMAIL_ADDRESS>resolving symbol OpaqueToken in<EMAIL_ADDRESS>resolving symbol OpaqueToken in<EMAIL_ADDRESS>
ERROR in ./src/main.ts
Module not found: Error: Can't resolve './$$_gendir/app/app.module.ngfactory' in '_git/AzureFunctionsPortal/AzureFunctions.AngularClient/src'
 @ ./src/main.ts 4:0-74
 @ multi ./src/main.ts

Expected/desired behavior
AOT should work
Reproduction of the problem
Pull this branch and run npm i and then ng b --prod
https://github.com/projectkudu/AzureFunctionsPortal/tree/upgrade-angular-v4
Please tell us about your environment:

ngx-translate version: 6.0.1

Angular version: 4.0.0

Browser: [all]

Actually, this was busted for the old version of ng20translate, but seems to be fine now for this version.
The errors are now pointing to the same problem with another ng 3rd party library that is not AOT compatible.
closing

with Angular 4.1.0 I am getting following error
ERROR in Error encountered resolving symbol values statically. Calling function 'OpaqueToken',
function calls are not supported. Consider replacing the function or lambda with a reference to an
exported function, resolving symbol USE_STORE in /Developer/Work/iot/hub/node_modules/@ngx-
translate/core/src/translate.service.d.ts, resolving symbol TranslateModule.forRoot in
<EMAIL_ADDRESS>resolving symbol
AppModule in /Developer/Work/iot/hub/src/app/app.module.ts, resolving symbol AppModule in
/Developer/Work/iot/hub/src/app/app.module.ts


pre signed url is not accessible after 15 mins, throws clock skew error

Environment info

NooBaa Version: noobaa-core-5.17.0-20241023.el9.x86_64
Platform: standalone noobaa

Actual behavior
pre signed url creation is success and accessible via browser as well as curl command.
However after 15 mins (approximately) , the url becomes inaccessible and gives clockskew error.
Expected behavior
If I set the expiry time of the pre signed url, the url must be accessible till that expiry time.
Steps to reproduce

Upload a text file with some readable data in to bucket .
[root@redshirt1 ~]# cat test123.txt
abcd
[root@redshirt1 ~]# aws-test s3 cp test123.txt s3://bucket1001

Generate a pre signed url with expiration time of 7 days
[root@redshirt1 ~]# aws-test s3 presign s3://bucket1001/test123.txt --expires-in 604800

Now try to read the url via curl command or browser , Getting read properly.
Noobaa.log shows the following entry

Oct 30 02:45:36 enfield-x-master [1997121]: [nsfs/1997121]   [L0] core.sdk.bucketspace_fs:: BucketSpaceFS.read_bucket_sdk_info: bucket name bucket1001
Oct 30 02:45:36 enfield-x-master [1997121]: [nsfs/1997121]   [L0] core.sdk.bucketspace_fs:: BucketSpaceFS.read_bucket_sdk_info: bucket name bucket1001
Oct 30 02:45:36 enfield-x-master [1997121]: [nsfs/1997121]   [L0] core.sdk.namespace_fs:: NamespaceFS: read_object_stream { file_path: '/ibm/fs1/account1001/bucket1001/test123.txt', start: 0, end: Infinity, size: 5 }
Oct 30 02:45:36 enfield-x-master [1997121]: [nsfs/1997121]   [L0] core.sdk.namespace_fs:: NamespaceFS: read_object_stream completed file /ibm/fs1/account1001/bucket1001/test123.txt { num_bytes: 5, num_buffers: 1, avg_buffer: 5, log2_size_histogram: { '3': 1 } }

After roughly 15 mins do the same operations in step3 again.

Oct 30 03:08:47 enfield-x-master [1997120]: [nsfs/1997120] [ERROR] core.endpoint.s3.s3_rest:: S3 ERROR <?xml version="1.0" encoding="UTF-8"?><Error>RequestTimeTooSkewed<Message>The difference between the request time and the server's time is too large.</Message><Resource>/bucket1001/test123.txt?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=CoQUAQeCnOBNmROQxmuf%2F20241030%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20241030T094528Z&X-Amz-Expires=604800&X-Amz-SignedHeaders=host&X-Amz-Signature=e09fada28fccfbb058cf235d45758bb119a4d5ffaa317cc39f5ae137d85951a4</Resource><RequestId>m2vps331-2q0kh1-ndv</RequestId></Error> GET /bucket1001/test123.txt?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=CoQUAQeCnOBNmROQxmuf%2F20241030%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20241030T094528Z&X-Amz-Expires=604800&X-Amz-SignedHeaders=host&X-Amz-Signature=e09fada28fccfbb058cf235d45758bb119a4d5ffaa317cc39f5ae137d85951a4 {"host":"<IP_ADDRESS>:6443","connection":"keep-alive","cache-control":"max-age=0","sec-ch-ua":"\"Chromium\";v=\"130\", \"Google Chrome\";v=\"130\", \"Not?A_Brand\";v=\"99\"","sec-ch-ua-mobile":"?0","sec-ch-ua-platform":"\"macOS\"","upgrade-insecure-requests":"1","user-agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36","accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7","sec-fetch-site":"none","sec-fetch-mode":"navigate","sec-fetch-user":"?1","sec-fetch-dest":"document","accept-encoding":"gzip, deflate, br, zstd","accept-language":"en-US,en;q=0.9,ms;q=0.8,ml;q=0.7"} Error: The difference between the request time and the server's time is too large.    at Object.check_headers (/usr/local/noobaa-core/src/util/http_utils.js:602:15)    at handle_request (/usr/local/noobaa-core/src/endpoint/s3/s3_rest.js:106:16)    at Object.s3_rest [as handler] (/usr/local/noobaa-core/src/endpoint/s3/s3_rest.js:66:15)    at Server.endpoint_request_handler (/usr/local/noobaa-core/src/endpoint/endpoint.js:286:28)    at Server.emit (node:events:518:28)    at Server.emit (node:domain:488:12)    at parserOnIncoming (node:_http_server:1143:12)    at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)
More information - Screenshots / Logs / Other output

@romayalon @guymguym Can you please look in to this?

Our check in the code is indeed not adapted to the presigned url case correctly - it checks the request date is up to fixed 15 minutes (config.AMZ_DATE_MAX_TIME_SKEW_MILLIS) -
https://github.com/noobaa/noobaa-core/blob/0b9e0d1cb89f048a24f1043a5c41876c6ee2cf55/src/util/http_utils.js#L590-L603
AWS docs limit the presigned URL's to a maximum of 7 days:

A presigned URL remains valid for the period of time specified when the URL is generated. If you create a presigned URL with the Amazon S3 console, the expiration time can be set between 1 minute and 12 hours. If you use the AWS CLI or AWS SDKs, the expiration time can be set as high as 7 days.

I think we should fix to have a separate config for the limit of presigned requests, so for regular signed requests (aka headers_v4) we indeed check the 15 minutes (config.AMZ_DATE_MAX_TIME_SKEW_MILLIS) to avoid time skew of the client and server, but for presigned requests (aka query_v4) we will check for 7 days (e.g config.AMZ_DATE_MAX_PRESIGNED_MILLIS).
Note that we already check the expiration date itself during the signature verification, so no need to change that.

cc @nimrod-becker @romayalon

I merged a fix to master #8493 , closing this issue


Error retrieving playlists: 403

Hi! When I sing in with my account YouTube it shows this message: "Error retrieving playlists: Error retrieving playlists: 403"
I always used the same account on YouTube, and I was always very good.

Unfortunately....
This extension is no longer available in the Chrome extension store.
YouTube has revoked the Chrome extension's access to the YouTube API. YouTube audited the extension and required several changes that I am unable to, or will not, make (including changing the name of the extension and user interface to not use the word "YouTube"). I guess we'd have to call it "Playlist Sorter for That Really Popular Video Sharing Service". No thanks.
I'll leave the code here for now, in case anyone wants to run it locally, or make their own attempt with YouTube's auditors.


Enzyme shallow render of a component that has a Transition component fails

I am trying to do a render in Enzyme using either the shallow or mount functions to unit test a react component. The component has some nested child components that I'd like to confirm exist, and they are wrapped in Transition components, more specifically a Grow for animation effects. When I mount or shallow render the parent component and run react scripts test, I get the following error:
  console.error node_modules\react-dom\cjs\react-dom.development.js:9627
    The above error occurred in the <Transition> component:
        in Transition (created by Grow)
        in Grow (created by WithTheme(Grow))
        in WithTheme(Grow) (at NoteGroup.js:147)
        in div (at NoteGroup.js:146)
        in NoteGroup (created by WrapperComponent)
        in WrapperComponent

    Consider adding an error boundary to your tree to customize error handling behavior.
    Visit https://fb.me/react-error-boundaries to learn more about error boundaries.

My test suite (condensed) looks like...
import NoteGroup from 'components/NoteGroup'
import { shallow, configure, mount } from 'enzyme'
const props = { /* */ }
const TestNoteGroup = new NoteGroup(props)
const wrapper = mount(<NoteGroup {...props} />)

describe('Given a set of props', () => {
	describe('When I create a new NoteGroup', () => {
		describe('Then', () => {
			it('contains the passed props', () => {
				expect(TestNoteGroup.props).toEqual(props)
			})

			it('renders without crashing', () => {
				expect(wrapper).toHaveLength(1)
			})

			it('renders the proper number of draggable notes in the group', () => {
				const numberOfNotes = props.noteIDS.length
				expect(wrapper.find('.draggableNote')).toHaveLength(numberOfNotes)
			})
		})
	})
})

[x] This is a v1.x issue (v0.x is no longer maintained).
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Context
I am trying to unit test my react components to confirm it renders the correct number of child components
Your Environment

Tech
Version

Material-UI
^1.0.0

React
16.1.0

Resolved by using Material UI's createShallow testing utility, see https://material-ui.com/guides/testing/


Add stretchy data for U+20D7.  #1544

Add stretchy data for U+20D7 to HTML-CSS, CHTML, SVG data for TeX and STIX fonts.  Other fonts already have it.  Resolves issue #1544.

lgtm (after resolved conflicts).


Add docs and testing files

This PR aims to add docs and testing files.

@ldr426 The test functions should be outside of the package modules so you can test if the package works as expected. And then you should run them with pytest. Read about testing and make sure it works with continuous integration.
Can you split also documentation and testing in separate PRs. I see the file changes in this PR are mainly testing. Where are the documentation changes?

@ldr426 you can look at the echoregions CI setup for tests:
https://github.com/OSOceanAcoustics/echoregions/blob/main/.github/workflows/pytest.yml


Move non-secret information from Github secrets to variables and rename

Identical to #640, but testing that building from a fork still works


A warning thrown with jest

Description
There was a warning thrown as we ran test as following: TypeError: _reactNative.AppState.removeEventListener is not a function
CodeSandbox/Snack link
https://codesandbox.io/s/codesandbox-sse-example-forked-izm8zv?file=/sample.test.jsx
Steps to reproduce

Visit the above codesanbox
Run yarn test command then you will the see the warning

NativeBase Version
3.4.15
Platform

[ ] Android
[ ] CRA
[X] Expo
[ ] iOS
[ ] Next

Other Platform
No response
Additional Information
No response

my stack trace for this issue
    console.error
      Warning: Internal React error: Attempted to capture a commit phase error inside a detached tree. This indicates a bug in React. Likely causes include deleting the same fiber more than once, committing an already-finished tree, or an inconsistent return pointer.

      Error message:

      TypeError: _reactNative.AppState.removeEventListener is not a function
          at HybridProvider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/native-base/lib/commonjs/core/hybrid-overlay/HybridProvider.tsx:9:3)
          at ResponsiveQueryProvider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/native-base/lib/commonjs/utils/useResponsiveQuery/ResponsiveQueryProvider.tsx:20:12)
          at RNCSafeAreaProvider
          at Component (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/react-native/jest/mockNativeComponent.js:17:18)
          at SafeAreaProvider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/react-native-safe-area-context/lib/commonjs/SafeAreaContext.tsx:35:3)
          at Provider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/native-base/lib/commonjs/utils/createContext.tsx:9:13)
          at NativeBaseProvider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/node_modules/native-base/lib/commonjs/core/NativeBaseProvider.tsx:52:5)
          at TestingNativeBaseProvider (/Users/pmikolajczuk/Projects/priv/koncept-bobo/asystentciazy/tests/ui/TestingNativeBaseProvider.tsx:5:3)

Same issue when updating from version^3.4.12 to ^3.4.16.

Hey everyone, Thanks for reporting the issue. We'll fix it.

Hey guys, I've created temporary patch for this issue:
File: /patches/native-base+3.4.16.patch
diff --git a/node_modules/native-base/lib/commonjs/core/color-mode/hooks.js b/node_modules/native-base/lib/commonjs/core/color-mode/hooks.js
index 8e28efe..68490c8 100644
--- a/node_modules/native-base/lib/commonjs/core/color-mode/hooks.js
+++ b/node_modules/native-base/lib/commonjs/core/color-mode/hooks.js
@@ -45,9 +45,9 @@ const useAppState = () => {
   const subscription = _react.default.useMemo(() => ({
     getCurrentValue: () => _reactNative.AppState.currentState,
     subscribe: callback => {
-      _reactNative.AppState.addEventListener('change', callback);
+      const sb = _reactNative.AppState.addEventListener('change', callback);

-      return () => _reactNative.AppState.removeEventListener('change', callback);
+      return () => sb.remove()
     }
   }), []);

After next release, just delete this patch.

Getting the same error here when running tests with Jest
`console.error
Warning: Internal React error: Attempted to capture a commit phase error inside a detached tree. This indicates a bug in React. Likely causes include deleting the same fiber more than once, committing an already-finished tree, or an inconsistent return pointer.
Error message:

TypeError: _reactNative.AppState.removeEventListener is not a function
    at children (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\native-base\lib\commonjs\core\hybrid-overlay\HybridProvider.tsx:9:3)
    at disableCSSMediaQueries (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\native-base\lib\commonjs\utils\useResponsiveQuery\ResponsiveQueryProvider.tsx:20:12)
    at RNCSafeAreaProvider
    at RNCSafeAreaProvider (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\react-native\jest\mockNativeComponent.js:17:18)
    at children (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\react-native-safe-area-context\lib\commonjs\SafeAreaContext.tsx:35:3)
    at children (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\native-base\lib\commonjs\utils\createContext.tsx:9:13)
    at colorModeManager (C:\Users\duder\Desktop\School\Fall 22\CSE 485\MobileAppRepo\mobile-app\node_modules\native-base\lib\commonjs\core\NativeBaseProvider.tsx:52:5)`

I'm using native-base version 3.4.19 and still have the issue! Any updates on this?

Yes, got the same error, please fix it 🙏

Hi
The problem is still there but it has moved to a different file? Using 3.4.25
%s%s, TypeError: react_native_1.Dimensions.removeEventListener is not a function. (In 'react_native_1.Dimensions.removeEventListener('change', onChange)', 'react_native_1.Dimensions.removeEventListener' is undefined),      in CheckinScreen (created by Screens)
node_modules/native-base/src/utils/useResponsiveQuery/common.ts
// The below implementation is taken from React Native's source and added a flag to conditionally attach/remove listeners
export const useDimensionsWithEnable = ({ enable }: { enable?: boolean }) => {
  const [dimensions, setDimensions] = React.useState(() =>
    Dimensions.get('window')
  );

  React.useEffect(() => {
    if (enable) {
      function handleChange({ window }: { window: ScaledSize }) {
        if (
          dimensions.width !== window.width ||
          dimensions.height !== window.height ||
          dimensions.scale !== window.scale ||
          dimensions.fontScale !== window.fontScale
        ) {
          setDimensions(window);
        }
      }
      Dimensions.addEventListener('change', handleChange);
      // We might have missed an update between calling `get` in render and
      // `addEventListener` in this handler, so we set it here. If there was
      // no change, React will filter out this update as a no-op.
      handleChange({ window: Dimensions.get('window') });

      return () => {
        Dimensions.removeEventListener('change', handleChange);  //  <- HERE! ####################
      };
    }
    return () => {};
  }, [dimensions, enable]);
  return dimensions;
};

Version 3.4.28 still has this bug:
 console.error
    Warning: Internal React error: Attempted to capture a commit phase error inside a detached tree. This indicates a bug in React. Likely causes include deleting the same fiber more than once, committ
ing an already-finished tree, or an inconsistent return pointer.

    Error message:

    TypeError: Cannot read properties of undefined (reading 'remove')
        at HybridProvider (C:\personal_projects\clean-gta-clients\node_modules\native-base\lib\commonjs\core\hybrid-overlay\HybridProvider.tsx:9:3)
        at ResponsiveQueryProvider (C:\personal_projects\clean-gta-clients\node_modules\native-base\lib\commonjs\utils\useResponsiveQuery\ResponsiveQueryProvider.tsx:20:12)
        at RNCSafeAreaProvider
        at Component (C:\personal_projects\clean-gta-clients\node_modules\react-native\jest\mockNativeComponent.js:17:18)
        at SafeAreaProvider (C:\personal_projects\clean-gta-clients\node_modules\react-native-safe-area-context\src\SafeAreaContext.tsx:35:3)
        at Provider (C:\personal_projects\clean-gta-clients\node_modules\native-base\lib\commonjs\utils\createContext.tsx:9:13)
        at NativeBaseProvider (C:\personal_projects\clean-gta-clients\node_modules\native-base\lib\commonjs\core\NativeBaseProvider.tsx:52:5)

Same here but with the badge component, im doing the following:
`
const TestComponent = (props: Record<string, unknown>) => <View {...props} />
jest.mock('native-base', () => ({
...jest.requireActual('native-base'),
HamburgerIcon: () => ,
AddIcon: () => ,
}))
`

Came across this and same issue as @joaosousafranco
jest.mock('native-base', () => {
  console.log(jest.requireActual('native-base'))
  return {
    ...jest.requireActual('native-base'),
    Collapse: (props: ICollapseProps) => {
      if (props.isOpen) {
        props.onAnimationEnd?.()
      }
      return props.isOpen ? props.children : null
    },
  }
})

TypeError: Cannot read properties of undefined (reading 'Badge')


feat: rum query api impl

implements https://github.com/adobe/spacecat-api-service/pull/325
Thanks for contributing!

no activity, closing


Nomadtrips integration

This would be super awesome if it had a nomadtrips integration. This is what nomadtrips looks like: https://nomadtrips.co/mxstbr
I'd love to for this (if you'll allow it, of course) and use it for my parents etc., but I don't want to have to add my trips to two separate pages...
Any chance that could happen?

Just to note, the way to get the data of a user is to to append .json at the end of the URL, so to get my data in json format just query https://nomadtrips.co/mxstbr.json.


Cannot delete item

I use FreeOTP on iPhone 7Plus, iOS 14.0.1
I have lot of duplicated item than I cannot remove

How please perform this delete? Thanks in advance

I use FreeOTP on iPhone 7Plus, iOS 14.0.1
I have lot of duplicated item than I cannot remove

How please perform this delete? Thanks in advance

Does a swipe right of the token bring up a 'Delete' confirmation?


CVE-2007-4559 Patch

Patching CVE-2007-4559
Hi, we are security researchers from the Advanced Research Center at Trellix. We have began a campaign to patch a widespread bug named CVE-2007-4559. CVE-2007-4559 is a 15 year old bug in the Python tarfile package. By using extract() or extractall() on a tarfile object without sanitizing input, a maliciously crafted .tar file could perform a directory path traversal attack. We found at least one unsantized extractall() in your codebase and are providing a patch for you via pull request. The patch essentially checks to see if all tarfile members will be extracted safely and throws an exception otherwise. We encourage you to use this patch or your own solution to secure against CVE-2007-4559. Further technical information about the vulnerability can be found in this blog.
If you have further questions you may contact us through this projects lead researcher Kasimir Schulz.

Thank you for submitting this.
Ad-hoc tar files aren't currently designed to be processed by txtai. It's an internal use only function for combining internal index files. Nonetheless compression logic is being refactored and will take this into account. Thank you once again.


Some example issue...

https://github.com/Raffson/Test/blob/62d4a56b318732e4adfd3eabbf63236da7ede66a/newfile#L3
this should be changed to something else..

fixed


Addition of comments

As requested by user comments has been updated to improve the structure of HTML of the document.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/4918

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

Rich diff: https://github.com/MicrosoftDocs/windows-itpro-docs/pull/6013/files?short_path=1d7137d#diff-1d7137dbf2fc5383b8d875663c617788
Please note that the comment lines directly before any MarkDown code lines, like the 8 [!TIP] notes, makes it look like the MarkDown code has become broken or at least deactivated.

Thank @illfated for pointing out these. Looks good to me now but give your opinion.

There are likely more tag issues breaking MarkDown in this document, but I need a couple of hours' break before pointing out more of them.

@illfated Thanks for your help. I fixed those suggested by you and made some more fixes. Give it a quick look to see how does it looks like.

Very good, well done. The resulting file can be viewed in Rich Text (with images) on this page:

https://github.com/MicrosoftDocs/windows-itpro-docs/blob/16ea53ec106378ef094b470f63ca820c8593e528/windows/client-management/mdm/bitlocker-csp.md

@Dansimp This PR is ready to go live.


Documentation for configuring proxy access to the OIDC provider

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
We have identified a feature gap in the current deliverable and release a patch. This documents the configuration required for the new functionality.
Why should this Pull Request be merged?
Necessary documentation for cluster setup.
What testing has been done?
N/A

@MelissaHilliard This wasn't tracked by the two bugs I sent yesterday, but will need to be duplicated in your new documentation repo.

@MelissaHilliard It looks like Mark merged this change in order to support the customer. Feel free to propose any changes either as a PR or directly to me.


hi i love what yu did just one thing please

i use aspnet with vb
and the properties file.error and file.type
"error" and "type"
are reserved words
and i cant use them as properties
can it be changed ?
i saw someone use "errorMSG" instead for the error message
i now trying to fix it but may be someone else will encounter this issue
tnx

Hey @rommilg - I think you may have the wrong repository!


Retry testnet tests

This will retry any testnet that failed the first time.  The hope is this is a quick fix to reduce chairman test flakiness before we are able to have proper investigation into why the chairman tests fail in the first place.

Looks like this is already merged as part of another PR.  Closing.


Swallowing errors in resolver.

If I throw an exception in a resolver the error is swallowed somewhere in the graphqlExpress => makeExecutableSchema stack. Indeed the latter's docs (http://dev.apollodata.com/tools/graphql-tools/generate-schema.html#makeExecutableSchema) state:

logger is an optional argument, which can be used to print errors to the server console that are usually swallowed by GraphQL.

What's the best practice to catch these errors and return, say, a 500 response (rather than just formatting the error via formatError)?

well, basiclly if you will look it architecture wise,
the 500 response should happen if transport fails,
in your case transport succeed, and then the resolver on the graphql layer failed,
so the exception is swollen and converted into graphql error in ExecutionResult,
so, TLDR: this is the correct approach, why would you want to do otherwise?

Hi @DxCx

TLDR: this is the correct approach, why would you want to do otherwise?

500 means "internal server error" (below); nothing to do with the transport.
So my interpretation is that if the server throws an exception (e.g., due to a programmatic error), then this is the appropriate response. Now the server code could choose to catch these exceptions and return a different error...
BUT I don't see where we could even catch these exceptions in the current API.
https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
10.5.1 500 Internal Server Error

HTTP Status code, means HTTP, again, thats the transport.
No matter which status it returns, thats the transport layer status.
On top of that, there is GraphQL, in your case, graphql resolver fails, which is the next layer on top of HTTP.
Then, the error is returned on GraphQL layer.. as errors..
GraphQL doesnt define HTTP as the transport, what do you expect to happen if your scheme will be running on top of another transport?

@richburdon ?

I think I see what @richburdon is trying to accomplish.  When using something like "apollo-errors" with schema stitching, errors don't work because it's eating the error up.


PSModule resource does not support installing side-by-side versioned modules

Because the the Name parameter is Key, the PSModule resource cannot be used to download two versions of the same module.  It would be nice if the Key parameter could also accept the module specification format for scenarios when more than one version of a module is required for a node.
Consider the following.
    PSModule 'tooling-<IP_ADDRESS>'
    {
        Name = 'tooling'
        Repository = 'InternalNuget'
        RequiredVersion = '<IP_ADDRESS>'
        Ensure = 'Present'
        InstallationPolicy = 'Trusted'
    }

    PSModule 'tooling-<IP_ADDRESS>'
    {
        Name = 'tooling'
        Repository = 'InternalNuget'
        RequiredVersion = '<IP_ADDRESS>'
        Ensure = 'Present'
        InstallationPolicy = 'Trusted'
    }

The resulting error:
Test-ConflictingResources : A conflict was detected between resources '[PSModule]tooling-<IP_ADDRESS>
(C:\Users\Bob\Documents\config.ps1::16::5::PSModule)' and '[PSModule]tooling-<IP_ADDRESS>
(C:\Users\Bob\Documents\config.ps1::26::5::PSModule)' in node 'localhost'.  Resources have identical key properties but there are differences in the following non-key properties: 'RequiredVersion'. Values '<IP_ADDRESS>' don't match values '<IP_ADDRESS>'. Please update these property values so that they are identical in both cases.

Thanks for taking a look at this.

This issue was moved to OneGet/oneget#257


Minimum OS Requirement

Please add a note to this page in regard to the requirement stated in https://docs.microsoft.com/en-us/windows/msix/packaging-tool/support-legacy-context-menus.  I think it should appear here as

AppXManifest file must have the Dependency MaxTested set to at least 10.0.21300.0
Support on OS builds of at least 10.0.21300.0

This requirement note should also be placed on the page https://docs.microsoft.com/en-us/uwp/schemas/appxpackage/uapmanifestschema/element-desktop9-fileexplorerclassicdragdropcontextmenuhandler

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

ID: f23e5b7b-3a2a-c122-9a0e-fd7a97e4cd20
Version Independent ID: baed8ba5-589a-916e-371a-f372dda39adb
Content: desktop9:FileExplorerClassicContextMenuHandler - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-desktop9-fileexplorerclassiccontextmenuhandler.md
Product: uwp
Technology: winrt-reference
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

These requirements appear to have been added at some point in the past.


Placeholders added using render_placeholder not shown in Structure board

Summary
If we use placeholder outside cms by setting PlaceholderField to our model and {% render_placeholder instance.field %} in our template, placeholder is not shown in Structure board.
We used same model and view with djangocms 3.4 and we were able to see placeholders and edit plugins using structure mode.
We can still see and edit plugins that were already added to placeholders before upgrade to djangocms 3.5 version, but no new plugins can't be added as structure board is empty.
Expected behaviour
Placeholders added by {% render_placeholder %} tag are shown in structure mode
Actual behaviour
Placeholders are not shown in structure mode
Environment

Python version: 2.7.6
Django version: 1.11.13
django CMS version:3.5.2

It seems that setting current_page attribute to request in my custom view resulted it this behavior.
The moment I removed it, my structure board is filled as expected.


Bug | Payment for warrant not being collected

Describe the bug
Payment for warrant not being collected by the system

Closing as no further clarity received on this ticket


[Bug]-[110]:Request timeout occurs when port number is incorrect

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Request_timeout occurs when port number is incorrect and also no error message appears for this
Steps To Reproduce

Create a mysql connection
Add incorrect port number
Hit Test and observe there is continuous loader running in Test button
Observe there is a timeout and no errors occur

Environment
Release
Version
Cloud

mysql 6m usage: 275
most people use the default port, so allowing for 10% of these who may use another port number: 27.5
Stats

Stat
Values

Reach
27.5

Effort (months)
<>

I am not able to reproduce the issue anymore, hence closing it


Exclude options not inherited from gem

This seems related to #3744.
In my Rails project, the .rubocop.yml file is just this:
inherit_gem:
  barsoom_utils: shared_rubocop.yml

The shared_rubocop.yml file is this: https://github.com/barsoom/barsoom_utils/blob/master/shared_rubocop.yml
And contains:
AllCops:
  DisabledByDefault: true
  Exclude:
    - db/schema.rb
    - db/migrate/*
    - vendor/**/*
    - node_modules/**/*

…

But Rubocop in my Rails app does not respect these ignores, after moving them to the shared gem. Is this by design? Can I change something to get the desired behaviour? Or is it a bug?
I have seen https://github.com/bbatsov/rubocop/pull/5488, but if I understood #3744 correctly, then the behaviour I'm expecting should already exist.

I get this with local inheritance as well. If my .rubocop.yml only contains
inherit_from:
  - .rubocop-parent.yml

And .rubocop-parent.yml is identical to the shared_rubocop.yml mentioned above, it does not e.g. exclude db/migrate/test_file.rb.
What am I missing?

When running bundle exec rubocop, it's fine.
This was all from a Git hook that runs Rubocop on files modified in Git, calling rubocop my_file.rb.
I guess that when an ignored file is explicitly given on the command line, Rubocop will inspect it, which is not unreasonable.
Is there any way to get Rubocop to still follow its exclude rules even when a file is given explicitly on the command line, or would I have to implement something myself? Don't see any command-line flag for this: http://rubocop.readthedocs.io/en/latest/basic_usage/

Flag
Description

--force-exclusion
Force excluding files specified in the configuration Exclude even if they are explicitly passed as arguments.


Codeium seems not working !

Hey guys, I am using  with Neovim v0.8.1.
I currently have installed codeium.vim using Lazy.nvim like this
  {
    "Exafunction/codeium.vim",
    config = function()
      vim.g.codeium_enabled = true -- Enable Codeium by default
      vim.g.codeium_idle_delay = 500 -- Set the idle delay (in milliseconds)
    end,
  },

But I am unable to see any suggestion as described in the introduction videos of codeium installation in neovim.
I am authenticated and did not get any errors reported via codeium or Lazy till now.

Please suggest if some more configuration is needed !

After installation did you do the :Codeium Auth ?

After installation did you do the :Codeium Auth ?

Yes I am authenticated. Codeium or nvim did not report any errors

I had the same problem with lunarvim：
__                          _    ___

/ /   __  ______  ____ | |  / () ___
/ /   / / / / __ / __ / ___/ | / / / __
/ // // / / / / // / /   | |/ / / / / / / /
/___/_,// //_,//    |/// // //
Buffer info

filetype:                lua
bufnr:                   3
treesitter status:       active

Active client(s)
Automatic LSP info

Skipped servers: [angularls, ansiblels, antlersls, azure_pipelines_ls, custom_elements_ls, omnisharp, cssmodules_ls, denols, docker_compose_language_service, ember, emmet_ls, eslint, glint, golangci_lint_ls, gradle_ls, graphql, java_language_server, jedi_language_server, ltex, neocmake, phpactor, psalm, pylsp, pylyzer, pyre, quick_lint_js, reason_ls, rnix, rome, ruby_ls, ruff_lsp, solang, solc, solidity_ls_nomicfoundation, sorbet, sourcery, spectral, sqlls, sqls, standardrb, stylelint_lsp, svlangserver, tflint, unocss, verible, vtsls, vuels, pyright, clangd]

Formatters info

Active:
Supported: [lua_format, stylua]

Linters info

Active:
Supported: [luacheck, selene]

Code actions info

Active:

I found that it was caused by proxy. When I set up the HTTP agent, codeium does not work anymore


Use 'url' config for the base public path

In our production environment, we use custom domains for the GCS buckets. Unfortunately, using the apiEndpoint configuration variable (only for the $file->url(), aka only the display part), it gives the following error:
<?xml version='1.0' encoding='UTF-8'?><Error>InvalidAuthentication<Message>The provided authentication header is invalid.</Message>Cannot use OAuth Authorization header with form POST.</Error>

Replacing it with url in the config allows to use a different setting when we only want to use a different domain for the public url, while still using the regular DEFAULT_API_ENDPOINT = 'https://storage.googleapis.com'; for the upload.
In this PR, I added a new configuration variable url which is only used for the generation of the link.

Closing due to inactivity.


remaining Studio changes

Remaining changes needed for running RTE in merch at scale studio
Test URLs:

Before: https://main--mas--adobecom.hlx.live/
After: https://studio--mas--adobecom.hlx.live/

@Axelcureno can we clean bundles from libs that have been moved to Milo?

@Axelcureno can we clean bundles from libs that have been moved to Milo?

@yesil yes of course, I have cleaned up those files.

Closing this PR since https://github.com/adobecom/mas/pull/56 contains this and more changes.


Migrate 'master' branch to 'main'

We should migrate the "master" branch to a branch called "main" to be in sync with industry and MSFT standards.

@aguywithcode - could you make this change?

Done...any existing clones will need an update.  But the branch has been renamed.

Done...any existing clones will need an update.  But the branch has been renamed.


Challenge EI2 andrste

Note: As there is no direct bluetooth connection between iOS & Android I had to send export file through email.

confirmed:EI2


[pytryfi.fiPet] Could Not Set Place Error in Home Assistant Log Files

I'm seeing a continuous error that is showing up every 10 seconds in the home assistant log files:
Example:
2021-02-28 13:00:26 WARNING (SyncWorker_14) [pytryfi.fiPet] Could not set place, defaulting to Unknown
2021-02-28 13:00:36 WARNING (SyncWorker_31) [pytryfi.fiPet] Could not set place, defaulting to Unknown
2021-02-28 13:00:46 WARNING (SyncWorker_36) [pytryfi.fiPet] Could not set place, defaulting to Unknown
2021-02-28 13:00:56 WARNING (SyncWorker_47) [pytryfi.fiPet] Could not set place, defaulting to Unknown
2021-02-28 13:01:06 WARNING (SyncWorker_20) [pytryfi.fiPet] Could not set place, defaulting to Unknown
I'm not sure if this is something coming from TryFI and something that was never set up on our account, or originally, I thought I needed to define a location for the devices in home assistant, which I did, and did not resolve.   I tried to delete the integration and re-add it, with no such luck.
I'm running 0.0.6 version currently.
Brian

Hi Brian,
This can be safely ignored. If I remember correctly, the Place option is defined when you create a "safe" zone and the pet is within the zone. I know when I take my dog out of the safe zone, the place option is not set anymore by the TryFi APIs.


On opened before heartbeat check connection is open

After connection Hearbeat is send:

/node_modules/ws/lib/WebSocket.js:160
throw new Error('not opened');
^
Error: not opened
at WebSocket.ping (/node_modules/ws/lib/WebSocket.js:160:11)
at Timeout._onTimeout (/node_modules/gdax/lib/clients/websocket.js:77:19)
at ontimeout (timers.js:469:11)
at tryOnTimeout (timers.js:304:5)
at Timer.listOnTimeout (timers.js:264:5)

With this implementation it's impossible to catch this error, exit process ....
To avoid check WS is connected before send hearbeat

ClosIng in favor of channels


Fixes #3150 - Ignores resource key when it isn't present

The new test fails with the KeyError experienced by customer.
Tests passing on Python 2.7 and 3.4 (I don't have 3.5 on my env).

I just confirmed that and made the change I suggested, now tossing to @dhermes for review.
CI will still fail until #3178 is merged, so it will have to go in before this can.

I merged in master (which includes #3178), merging once CI passes.


WIP APPS-26 -- x-notify add bull queue and Jobs

Hi @duboisp : Glen is looking for an update on this and Shiva is looking for a code review.  Would you like to review the code or shall I?


Create a pipeline to analyze a WAR-based Java app

The specifics of a WAR are that we typically have:

some lib or similar directory that contains JARs
some plain Java classes (typically under WEB-INF dir)
because this is for some web app, we often have JSP (possibly complied ahead-of-time to Java and .class) and JavaScript (possibly transpiled from TypeScript, webpacked and/or minified, sometimes with a companion .js.map), CSS, images, fonts, and other web-related files.

We are handling JavaScript separately:

#650
there is not much special here beyond JSPs


BoundedState: add deselect, last, and first methods

/// Clear the state's selection
pub fn deselect(&mut self) {
  self.inner.select(None);
}

/// Select the first index in the state
pub fn first(&mut self) {
  self.inner.select(Some(self.boundary.0));
}

/// Select the last index in the state
pub fn last(&mut self) {
  self.inner.select(Some(self.boundary.1));
}

Add examples to these in doc comments


Issue with Synaptic and Somatic Voltage traces using synapse_replay feature

Hi @anilbey,
I've encountered an issue where both the synaptic and somatic voltage traces are turning out empty after utilizing the synapse_replay feature. I have attached the code that I used to run the simulation below.
Could you please review it and let me know if I've initialized the synapses correctly or if there's anything I might have missed or done incorrectly?
Thank you in advance for your time and assistance!
Script:
import neuron
from bluecellulab import SSim

neuron.load_mechanisms("./circuit_sonata/")

cfg_json = "/project/ctb-emuller/projects/proj01/dhuruva_sims/2_cpre_cpost/simulation_config.json"
sim = SSim(cfg_json,print_cellstate=True)

cell_id = 170923
pre_gid = ('S1nonbarrel_neurons', 170983)

# Working snippet - without synapse_replay
# sim.instantiate_gids([('S1nonbarrel_neurons',cell_id)])

# With synapse replay
sim.instantiate_gids([('S1nonbarrel_neurons',cell_id)], add_synapses=True,
                          pre_spike_trains={pre_gid: [1000.]}, add_projections = False, add_minis = False)

duration = 200
dt = 0.025
sim.run(duration, cvode=False, v_init=-70, celsius=34, dt=dt)

Somatic Voltage trace
Empty Somatic Voltage trace

Simulation Config:
{
    "manifest": {
        "$CURRENT_DIR": "."
    },
    "run": {
        "dt": 0.025,
        "tstop": 2000.0,
        "random_seed": 12345,
        "using_cvode": 0
    },
    "conditions": {
        "extracellular_calcium": 2.0,
        "v_init": -80.0,
        "spike_location": "AIS",
        "mechanisms": {
            "ProbAMPANMDA_EMS": {
                "init_depleted": true,
                "minis_single_vesicle": true
            },
            "ProbGABAAB_EMS": {
                "init_depleted": true,
                "minis_single_vesicle": true
            },
            "GluSynapse": {
                "init_depleted": false,
                "minis_single_vesicle": true,
                "cao_CR": 2.0
            }
            }
    },
    "target_simulator": "CORENEURON",
    "network": "---/circuit_config.json",
    "node_sets_file": "./node_sets.json",
    "node_set": "test_circuit",
    "output": {
        "output_dir": "./reporting",
        "spikes_file": "out.h5"
    },
    "reports": {
        "soma": {
            "cells": "hex_O1ExcitatoryPlastic",
            "type": "compartment",
            "variable_name": "v",
            "unit": "mV",
            "dt": 1.0,
            "start_time": 0.0,
            "end_time": 122500.0
        },
        "rho": {
            "cells": "hex_O1ExcitatoryPlastic",
            "type": "synapse",
            "sections": "all",
            "variable_name": "GluSynapse.rho_GB",
            "unit": "nd",
            "dt": 0.025,
            "start_time": 0.0,
            "end_time": 122500.0
        },
        "effcai_GB": {
            "cells": "hex_O1ExcitatoryPlastic",
            "type": "synapse",
            "sections": "all",
            "variable_name": "GluSynapse.effcai_GB",
            "unit": "mM",
            "dt": 0.025,
            "start_time": 0.0,
            "end_time": 122500.0
        }
    },
    "inputs": {
        "s1_syns_spikes": {
            "node_set": "hex_O1ExcitatoryPlastic",
            "input_type": "spikes",
            "delay": 0.0,
            "duration": 2000.0,
            "module": "synapse_replay",
            "spike_file": "./input_spikes/s1_syn_spikes.dat",
            "source": "hex_O1"
            }
    },
    "connection_overrides": [
        {
            "name": "plasticity",
            "source": "hex_O1Excitatory",
            "target": "hex_O1Excitatory",
            "modoverride": "GluSynapse",
            "weight": 1.0
        },
        {
            "name": "init",
            "source": "hex_O1",
            "target": "hex_O1",
	    "spont_minis": 0.0,
            "weight": 1.0
        },
        {
            "name": "VPM_init",
            "source": "proj_Thalamocortical_VPM_Source",
            "target": "hex_O1",
            "spont_minis": 0.0,
            "weight": 1.0
        },
        {
            "name": "POm_init",
            "source": "proj_Thalamocortical_POM_Source",
            "target": "hex_O1",
            "spont_minis": 0.0,
            "weight": 1.0
        }
    ]
}

Hi @dhuruvapriyan thanks for reporting this.
Has this issue occurred in other cells as well?
Could you also share the contents of the spikes file also?
"./input_spikes/s1_syn_spikes.dat"
Thanks in advance

Hi @anilbey,
I get the same results with Layer 2/3 and Layer 6 as well. I have attached my spikes.dat with this.
spikes.dat:
/scatter
1000 170983
1100 170983

Hi @dhuruvapriyan I couldn't find time this week but I will try to address it early next week.

Hi @anilbey,
Thanks for updating me on the status!
Do you think this issue could be related to multinode population?

Hi @dhuruvapriyan,

Do you think this issue could be related to multinode population?

Yes you are right. I think you have identified a very important issue. I will work on providing the relevant change asap.

Hey @anilbey.
The synapse replay feature is working fine now. Thanks for fixing it!

Perfect! Please let me know if there are any other issues with bluecellulab that are blocking you.

A new version (v1.7.0) that includes this fix is released. You can now use bluecellulab>=1.7.0.


Disable Cosign experimental features by default

This disables Rekor transparency log validation and Fulcio CA support
by setting the default value of COSIGN_EXPERIMENTAL to 0, instead of
previous 1 - meaning enabled by default.
Seems that there was a change in how TUF is deployed at Sigstore so the
call to fetch the Fulcio CA certificate done by the code to verify
against the transparency log now results in a panic:
panic: error getting targets

goroutine 1 [running]:
github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioroots.Get.func1()
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/cli/fulcio/fulcioroots/fulcioroots.go:68 +0x74
sync.(*Once).doSlow(0xc00021be40, 0x1c)
	sync/once.go:68 +0xd2
sync.(*Once).Do(...)
	sync/once.go:59
github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioroots.Get()
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/cli/fulcio/fulcioroots/fulcioroots.go:64 +0x31
github.com/sigstore/cosign/cmd/cosign/cli/fulcio.GetRoots(...)
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/cli/fulcio/fulcio.go:157
github.com/sigstore/cosign/cmd/cosign/cli/verify.(*VerifyCommand).Exec(0xc000751c48, {0x2e75ee8, 0xc000052040}, {0xc0008eced0, 0x1, 0x6906e5})
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/cli/verify/verify.go:113 +0x2fc
github.com/sigstore/cosign/cmd/cosign/cli.Verify.func1(0xc00095d680, {0xc0008eced0, 0x1, 0x3})
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/cli/verify.go:115 +0x265
github.com/spf13/cobra.(*Command).execute(0xc00095d680, {0xc0008ecea0, 0x3, 0x3})
	github.com/spf13/cobra@v1.4.0/command.go:856 +0x60e
github.com/spf13/cobra.(*Command).ExecuteC(0xc0003c6280)
	github.com/spf13/cobra@v1.4.0/command.go:974 +0x3bc
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.4.0/command.go:902
main.main()
	github.com/sigstore/cosign@v1.8.0/cmd/cosign/main.go:51 +0x4c

This disables that Fulcio CA certificate lookup to avoid the panic.

@redhat-appstudio/enterprise-contract please take a look

Included in https://github.com/redhat-appstudio/infra-deployments/pull/503 for validation but e2e test is still failing -> https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_infra-deployments/503/pull-ci-redhat-appstudio-infra-deployments-main-appstudio-e2e-deployment/1549350136952393728

Very odd.
We were seeing this traceback from the ec-cli (v2) task yesterday, but not on the v1 task, in the staging environment. I ruled it out as the Rekor API being done via http instead of https. Maybe that wasn't the issue after all.
+1 for disabling Rekor for now.


Support error reporting via napari-error-reporter

Description
Resubmission of #4040 that got merged too soon.  (approved by @DragaDoncila and @alisterburt, waiting on @jni)
Summary of changes since then:

upstream package changed to napari-error-reporter
decision made to restrict bug report access to those explicitly listed in https://github.com/tlambert03/napari-error-reporter/blob/main/ADMINS
any issues created in the napari repo from those bug reports will be manually inspected for leaked personally identifiable info, and it will be removed prior to posting here.
if that file changes (i.e. admins are added) then users will be asked to re-opt in, and they'll see a list of admins and links to their github pages
clarified some things in the opt in widget, which now looks like this:

ready for review @jni with v0.2.1 of napari-error-reporter now the only one on pypi