CUPS driver usage kind of broken on v1.3

Description
When attempting to use CUPS in v1.3, it seems that it's usage is broken in a few ways:

Just installing the package results in errors (Issue #11, partially fixed by #14 )
When applying the fix from #14, then there is a deprecation notice when just even attempting to list printers.

<warning>PHP Deprecated:  Passing a Psr\Http\Message\ResponseFactoryInterface and a Psr\Http\Message\StreamFactoryInterface to SocketClient is deprecated, and will be removed in 3.0, you should only pass config options. in /var/www/html/project/vendor/php-http/socket-client/src/Client.php on line 59</warning>

Beyond the deprecation notice, listing printers also fails due to the following exception:

TypeError: Argument 2 passed to Rawilk\Printing\Drivers\Cups\Cups::remoteServer() must be of the type string, null given, called in /var/www/html/project/vendor/rawilk/laravel-printing/src/Factory.php on line 46

This error cryptically is trying to say that a username/password are required, but the defaults for the username and password in the published config don't have a default and become null which generates the error.
Once I set the username/password to be blank strings, I then got the following error:
Rawilk\Printing\Exceptions\InvalidDriverConfig with message 'Remote CUPS server requires a username and password.'

This seems to be due to a arbitrary requirement of needing a username and password for a remote cups server.
While not great security wise to have no username/password when I remove the conditional that validates for requiring username/password, the CUPS calls work just fine (albeit with the deprecation notices).
Steps to reproduce
Install the package, configure only a valid CUPS_SERVER_IP, run Printing::printers()
Additionally, setting a username/password results in similar errors minus the error on requiring a username/password
Context

Laravel Printing version: 1.3
Laravel version: 8.35.1

Description
When attempting to use CUPS in v1.3, it seems that it's usage is broken in a few ways:

Just installing the package results in errors (Issue #11, partially fixed by #14 )
When applying the fix from #14, then there is a deprecation notice when just even attempting to list printers.

<warning>PHP Deprecated:  Passing a Psr\Http\Message\ResponseFactoryInterface and a Psr\Http\Message\StreamFactoryInterface to SocketClient is deprecated, and will be removed in 3.0, you should only pass config options. in /var/www/html/project/vendor/php-http/socket-client/src/Client.php on line 59</warning>

Beyond the deprecation notice, listing printers also fails due to the following exception:

TypeError: Argument 2 passed to Rawilk\Printing\Drivers\Cups\Cups::remoteServer() must be of the type string, null given, called in /var/www/html/project/vendor/rawilk/laravel-printing/src/Factory.php on line 46

This error cryptically is trying to say that a username/password are required, but the defaults for the username and password in the published config don't have a default and become null which generates the error.
Once I set the username/password to be blank strings, I then got the following error:
Rawilk\Printing\Exceptions\InvalidDriverConfig with message 'Remote CUPS server requires a username and password.'

~This seems to be due to a arbitrary requirement of needing a username and password for a remote cups server. While not great security wise to have no username/password when I remove the conditional that validates for requiring username/password, the CUPS calls work just fine (albeit with the deprecation notices).~
Steps to reproduce
Install the package, configure only a valid CUPS_SERVER_IP, run Printing::printers()
Additionally, setting a username/password results in similar errors minus the error on requiring a username/password
Context

Laravel Printing version: 1.3
Laravel version: 8.35.1

i was getting  Rawilk\Printing\Exceptions\InvalidDriverConfig with message 'Remote CUPS server requires a username and password.' too.
The  way I've done is I've initially set CUPS_SERVER_IP=unset,  CUPS_SERVER_USERNAME=unset & CUPS_SERVER_PASSWORD=unset.
This doesn't generate any errors and I don't call any functions if these are unset.
When they are properly configured, i.e: the value is not unset then I call the relevant functions.
Hope this helps you :slightly_smiling_face:

I'm not sure if this would be an issue on v2 but feel free to submit a PR that fixes it on v1.x or v2 if still an issue.


Add changelog and/or tagged releases

This is a must if we're expected to keep our dependency up to date (of course we are).
It can probably be automatically created from commit messages. Ideally it would allow things like Dependabot/Renovate/Depfu to source this changelog to let maintainers know what changed between versions.

Thanks for the suggestion. We'll do it soon.

Awesome 🙌  Thank you!

@hacknug we've created our first release with tag: https://github.com/mailersend/mailersend-nodejs/releases/tag/v1.1.1 from the current state of the repository.
From now on, we'll create a tagged release with changelogs for our next releases.


Credit Calculation | Editing or updating an EER value does not save for fuels with only one fuel class |1|

Location:

IDIR user
Administration section, Compliance Reporting sub-tab, Energy Effectiveness Ratio table

Issue:
TFRS is not saving a new/edited/updated EER value for the affected fuels (see below). When a user edits the EER value, including putting in a new effective period, after saving and confirming the change within the confirmation alert, the value does not update.
Affected Fuels:
This issue only affects those fuels with an EER value for a single fuel class (either gasoline or diesel), including LNG, Petroleum-based diesel..., and Petroleum-based gasoline. https://trello.com/c/DKxoKHeb/1465-credit-calculation-editing-or-updating-an-eer-value-does-not-save-for-fuels-with-only-one-fuel-class-1

 Credit Calculation | Editing or updating an EER value does not save for fuels with only one fuel class |1|


[BUG] - DecoderErrorDeserialiseFailure "Address" when query stake-address-info

Internal/Exernal
Internal   if an IOHK staff member.
Summary
I am getting a DecoderErrorDeserialiseFailure "Address" error while trying to interrogate a stake address using the  cardano-cli shelley query stake-address-info command.
Steps to reproduce
Steps to reproduce the behavior:

create a payment and a stake address (+ key pairs)

delegate the stake address to one stake pool

query the stake address using:
cardano-cli shelley query stake-address-info --address STAKE_ADDRESS --testnet-magic 42

See error

Expected behavior
No error; stake address info is displayed
System info (please complete the following information):

OS:  Ubuntu 18.04.4 LTS
node version

~# cardano-cli version
cardano-cli 1.14.0 - linux-x86_64 - ghc-8.6
git rev 9cfe6fc4236ca38cfa7245ffd70feb161badd79a

Screenshots and attachments
# CARDANO_NODE_SOCKET_PATH=example/node-bft1/node.sock \
> cardano-cli shelley query stake-address-info \
> --address e058bcb673c7acb99076482cb4617fcd650639ab7870cd7de0353ddb9d \
> --testnet-magic 42
option --address: DecoderErrorDeserialiseFailure "Address" (DeserialiseFailure 0 "expected list len")
Usage: cardano-cli shelley query stake-address-info --address ADDRESS
                                                    (--mainnet |
                                                      --testnet-magic NATURAL)
                                                    [--out-file FILE]
  Get the current delegations and reward accounts filtered by stake address.

Are you sure it's not a payment address ? My stake addresses are 4 characters longer.

The address is created with cardano-cli shelley stake-address build

This is still not working in 1.15.0 - different error
CARDANO_NODE_SOCKET_PATH=example/node-bft1/node.sock \
	cardano-cli shelley address info \
	--address e03a251e8f0ed7b010912f4ca24fd0e079af4aa5bccbdc8526acd448c0

option --address: Failed reading: invalid address

Usage: cardano-cli shelley query stake-address-info --address ADDRESS
                                                    (--mainnet |
                                                      --testnet-magic NATURAL)
                                                    [--out-file FILE]
  Get the current delegations and reward accounts filtered by stake address.

# cardano-cli --version
cardano-cli 1.15.0 - linux-x86_64 - ghc-8.6
git rev 97b3e95c67940608f5acda929cf861e8ebfeddd1


Changes to Lab 8

The only place I added a chunk of text was in the sensitivity analysis section. Everywhere else, I fixed small typos and/or shifted around sentences to make things a bit clearer. I did not change any code. In some places I changed prompts to be more explicit, but the underlying questions are the same (though look at the text I proposed for sensitivity analysis.)

Didn't actually intend for commits from Lab 2 to be in here too but those are tiny typo fixes - I didn't make a separate branch for this and have just been working off master


Updating documents for v1.1

Updated AboutUs
Updated Glossary and NFR in developer guide

Codecov Report

Merging #53 (e870f75) into master (6ecc988) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #53   +/-   ##
=========================================
  Coverage     71.48%   71.48%
  Complexity      407      407
=========================================
  Files            73       73
  Lines          1287     1287
  Branches        130      130
=========================================
  Hits            920      920
  Misses          332      332
  Partials         35       35

Continue to review full report at Codecov.

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


new links for UMIL

We want umil.simssa.ca, umil.linkedmusic.ca, and https://whatinstrumentisthis.ca to point to current vim.simssa.ca.
Domains are managed on AWS. We need to figure out how to port new url with nginx (currently everything goes to prod edge)

Do you want them to redirect to vim.simssa.ca? Or all be acceptable host names for the same site?

What's the difference between them? I guess we just want all those urls to go to the same VIM server.

Either way, as long as the DNS set-up in AWS is pointing to the prod edge server, the rest can be done in the operations repo with ansible and then with a few modifications to the current VIM django/nginx configuration.

Great! This part I have some idea.

The operations bit may require a bit of finagling because I think this will be the first time using the linkedmusic.ca domain.

This I don't know. I will read some documentations and maybe I will need some help later.

Do you want them to redirect to vim.simssa.ca? Or all be acceptable host names for the same site?

What's the difference between them? I guess we just want all those urls to go to the same VIM server.

In the first case, when someone requests umil.linkedmusic.ca we send them back a 30X with location: vim.simssa.ca (in other words, their browser will them open vim.simssa.ca and it will show "vim.simssa.ca" in the address bar.
In the latter, it'll show the same site no matter which of those you have in your address bar, but it'll keep the address bar the same.

The operations bit may require a bit of finagling because I think this will be the first time using the linkedmusic.ca domain.

This I don't know. I will read some documentations and maybe I will need some help later.

I suspect that we'll want to refactor our process so that we can specify domains and subdomains for each service in the playbooks/vars files.

Actually, I would like the reverse. I would like to slowly phase out the use of VIM and replace it with UMIL. So vim.simssa.ca should show up as umil.linkedmusic.ca. Whatinstrumentisthis.ca can also show up as UMIL.linkedmusic.ca


Cyclical References

https://github.com/saiichihashimoto/sanity-typed-schema-builder/issues/206
https://github.com/saiichihashimoto/sanity-typed-schema-builder/issues/114
https://github.com/saiichihashimoto/sanity-typed-schema-builder/issues/138
I'm going to leave this open because I believe it's generally unsolvable but will continue to be brought up.
There's no sane way to derive cyclical types (a tweet by @mattpocock), at least with our current pattern. There might be ways to accomplish a heuristic (deriving up to a certain level or allowing a next level derivation on demand, etc) but it's not going to "just work". Even finding a way to allow manual typing to mix-in has been a challenge. Until we can find a creative proof of concept, we'll have to leave this open as a "sorry this really sucks".

Maybe this:
export const programSchema = s.document({
  name: 'program',
  title: 'Program',
  fields: [
    {
      name: 'parentProgram',
      title: 'Parent Program',
      type: s.array({
-       of: [s.reference({ to: [programSchema] })],
+       of: [s.reference({ to: [programSchema as DocumentType<any, any, any, any>] })],
      }),
    },
  ],
});

Unfortunately, the only way I can see right now of dealing with cyclical types is to break the cycle somewhere.

Apparently it's not even a type problem at this point. It's giving me an error about referencing the schema before it is created.

Block-scoped variable 'programSchema' used before its declaration

This appears to be possible in Sanity.. Is there any way achieve this with typed schema builder?

Ah, I get it. Ignoring TS or sanity, this isn't possible in JS:
const foo = { referencingFooInItsOwnDeclaration: foo };

Which is what's happening here. One thing I was thinking of is the fact that this is possible:
const foo = { referencingFooInItsOwnDeclarationInAFunction: () => foo };

I could change the s.reference params to also allow () => DocumentType as well, to allow this:
export const programSchema = s.document({
  name: 'program',
  title: 'Program',
  fields: [
    {
      name: 'parentProgram',
      title: 'Parent Program',
      type: s.array({
-       of: [s.reference({ to: [programSchema] })],
+       of: [s.reference({ to: [() => programSchema] })],
      }),
    },
  ],
});

That might fix this runtime issue, but you might still be stuck with the TS issue. I haven't tested that, so you can't do this right now. A VERY hacky thing you could do now is this:
export const programSchema = s.document({
  name: 'program',
  title: 'Program',
  fields: [
    {
      name: 'parentProgram',
      title: 'Parent Program',
      type: s.array({
-       of: [s.reference({ to: [programSchema] })],
+       of: [s.reference({ to: [s.document({ name: 'program' })] })],
      }),
    },
  ],
});

All s.reference is getting from the document schemas is the name (unless you're using s.resolved) so this would work. A bit messy.

As of now, cycles work in @sanity-typed/types, so that's pretty cool.


增加edit 和 edit_to方法,但是由于receipt没有完成,暂时未能给出使用实例

增加edit 和 edit_to方法,但是由于receipt没有完成,暂时未能给出使用实例
实现discord,telegram,kaiheila的编辑信息功能
其中由于tg官方api的问题, telegram的编辑信息只能由纯Text信息编辑到纯Text信息,由Text和Image混合信息编辑到Text和Image混合信息

Codecov Report

Merging #74 (bbd0bba) into main (b6323da) will decrease coverage by 8.30%.
The diff coverage is 54.48%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
- Coverage   94.92%   86.62%   -8.30%
==========================================
  Files          18       19       +1
  Lines        1222     1406     +184
==========================================
+ Hits         1160     1218      +58
- Misses         62      188     +126

Files Changed
Coverage Δ

nonebot_plugin_saa/adapters/onebot_v11.py
96.63% <ø> (ø)

nonebot_plugin_saa/adapters/discord.py
14.13% <14.13%> (ø)

nonebot_plugin_saa/utils/helpers.py
60.00% <28.57%> (-28.89%)
:arrow_down:

nonebot_plugin_saa/utils/types.py
90.47% <45.83%> (-4.47%)
:arrow_down:

nonebot_plugin_saa/adapters/kaiheila.py
79.13% <47.36%> (-16.43%)
:arrow_down:

nonebot_plugin_saa/adapters/telegram.py
80.85% <74.28%> (-7.47%)
:arrow_down:

nonebot_plugin_saa/utils/platform_send_target.py
98.57% <93.33%> (+0.13%)
:arrow_up:

nonebot_plugin_saa/__init__.py
100.00% <100.00%> (ø)

nonebot_plugin_saa/adapters/__init__.py
100.00% <100.00%> (ø)

nonebot_plugin_saa/adapters/feishu.py
93.82% <100.00%> (ø)

... and 4 more

关了,这部分最后转移到了receipt的edit方法中去了


The code appears to build fine. The certificates are all loaded. But I cannot connect.  There has to be a step missing. Do we need to create client certificate on the cluster? When I try to open the service fabric explorer I get a not authorized message. Not sure why?

The code appears to build fine. The certificates are all loaded. But I cannot connect. There has to be a step missing. Do we need to create client certificate on the cluster? When I try to open the service fabric explorer I get a not authorized message. Not sure why?

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

ID: 3ac102a5-e9ad-0046-f217-446f1ad263c5
Version Independent ID: a47523e2-0c30-dee4-0037-5f58836fa37f
Content: Add an HTTPS endpoint using Kestrel to a Service Fabric app in Azure
Content Source: articles/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint.md
Service: service-fabric
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@tshinkle @MicahMcKittrick-MSFT @aljo-microsoft   I recently refreshed this article.  At one point, I remember seeing the "There was an error during CodePackage activation.The service host terminated with exit code:1" error in SFX as well.  In the https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint#configure-kestrel-to-use-https step, did you replace the "<your_CN_value>" value with "mytestcert" (or your test cert subject) in the GetCertificateFromStore method?

Thanks @rwike77 I did replace the value both times I ran through it. I even kept the naming convention the same using "mytestcert"
Although, looking back at it, I am wondering if I just put "mytestcert" instead of "CN=mytestcert"
Is it expected to keep the CN= part? If so that might be the issue and I can rerun through the doc.

@MicahMcKittrick-MSFT  You should keep the "CN=" part.  So if you're following the doc, the full value should be "CN=mytestcert".

@rwike77 got it. I don't remember if I did or not so I will try it again and get back to you on it.

Revisited the code again, but this part doesn't work. Getting the same error again.  "The target process exited without raising a CoreCLR started event. Ensure that the target process is configured to use .Net Core. This may be expected if the target process did not run on .net core." and https://localhost shows same above blank screen "This Site Cannot be reached".

@tshinkle @MicahMcKittrick-MSFT @its-saurabhjain
The idea of a local secured clusters doesn't make sense; what is your use case for this?
@dragav
To provide additional context.

Following the tutorial (
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-tutorial-create-dotnet-app).
The tutorial creates a local service, then walks through moving it to a
cluster. Then walks through using https and certificates in part 3, first
locally and then in a cluster. None of the part 3 htts steps appear to
work, locally or in a cluster. (
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint).
I'm pretty sure this was discussed previously in the thread.
On Mon, Feb 4, 2019 at 12:42 PM aljo-microsoft<EMAIL_ADDRESS>wrote:

@tshinkle https://github.com/tshinkle @MicahMcKittrick-MSFT
https://github.com/MicahMcKittrick-MSFT @its-saurabhjain
https://github.com/its-saurabhjain
The idea of a local secured clusters doesn't make sense; what is your use
case for this?
@dragav https://github.com/dragav
To provide additional context.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/23408#issuecomment-460341730,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXuvPCvqvgutiFU8lEccHWBCIBS2sAvCks5vKHETgaJpZM4aUzme
.

@MicahMcKittrick-MSFT if you have a local repro, can you send me the logs please? dragosav @ msft.
On the surface, this does appear to be caused by missing permissions to the cert's private key. Please check in the cert mgmt UX that the private key is available and acl'd accordingly. (For context, self-signed certs generated with PowerShell are by default CNG certs, unless a 'legacy' crypto provider is specified explicitly. Our runtime ACLing code doesn't handle CNG certs.)
The failure to connect to the cluster could be explained by the client (browser) rejecting the server's untrusted cert. This check can be bypassed in Chrome.

@tshinkle I just spun this up on my local work PC. So I dont have the app deployed at the moment. I am happy to deploy it again and send you any logs needed. That being said, I did just follow the steps in the doc exactly as written so you should be able to run through them and get the same results as we are all seeing.
I also checked to ensure the cert has the right permissions and all appears well. I also am using chrome to connect to the local cluster.

@tshinkle may I ask you to zip the traces in the SFLogs\traces directory, and share them with me? I'm dragosav at microsoft dot com. The exact path to the logs directory is listed in the cluster manifest. Thank you.

@dragav if you like I can also publish my code to an Azure Cluster and give you the subscription and cluster information offline if that helps.

Just FYI, I have given access to @dragav to my Azure Cluster seeing this error. Hopefully we can find some problems and get the doc updated.

I am also followed the same steps mentioned above and also getting same 'Site can't be reached error'

@dragav any luck on this? Seems we have multiple users who get the same issue with this doc

The enable https part don't work, just as what    @MicahMcKittrick-MSFT hits. Any update on this please?

We narrowed it down to a failure to either find or ACL the certificate. Highly recommend using the SF CertSetup.ps1 script to generate a certificate (in dev/test environments), as that is proven to work. I suspect whoever is hitting this may be generating CNG self-signed certs, whose private key is not accessible directly in PSh (and so the ACLing code fails). We're working on this.

@MicahMcKittrick-MSFT
#please-close

Was this issue resolved?

@dapathy
https://docs.microsoft.com/azure/service-fabric/service-fabric-best-practices-security
Please adopt documented best practices to mitigate issues like this.

I ran into the same problems as others and here is a few hints that might help others.
Firstly: If you added the .bat file like the tutorial says "right-click VotingWeb and select Add->New Item and add a new file named "Setup.bat". Visual studio might encode it wrong. In my case there was unwanted symbols in the beginning of the file (I might have done it incorrectly), check it by running the .bat file directly.
Secondly: When i ran the SetCertAccess.ps1 from PS, I got an error on line 34-37. regarding "$accessRule", because my windows is in german, so line two needed another user group from "NETWORK SERVICE" to "Netzwerkdienst".
$userGroup="Netzwerkdienst".
Thirdly: Also related to windows language. System group in ApplicationManifest.xml needs to reflect your windows language, in my case i changed from "Administrator"  to "Administratoren".

That did it for me.

@dapathy
https://docs.microsoft.com/azure/service-fabric/service-fabric-best-practices-security
Please adopt documented best practices to mitigate issues like this.

I have the same problem, I read this document but i don't know how to fix my problem. Please help me?


Fix test cases after httpbin.org changes

Haha, the point of httpbin was that we shouldn't have to make these changes :)

I probably should have changed the test cases to use one of the documented test endpoints rather than the main page


[Bug] Rest request can not handle custom header's value correctly

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
3.2.15
Steps to reproduce this issue
Start a dubbo provider service register by rest protocol with config version or group
In my case the register log is on below
2024-08-13T17:03:56.633+08:00  INFO 31907 --- [springboot-dubbo-provider] [           main] o.a.d.r.zookeeper.ZookeeperRegistry      :  [DUBBO] Register: rest://<IP_ADDRESS>:8080/com.doodl6.springboot.dubbo.api.FirstDubboService?application=springboot-dubbo-service-local&deprecated=false&dubbo=2.0.2&dynamic=true&generic=false&interface=com.doodl6.springboot.dubbo.api.FirstDubboService&methods=getDubboInfo&prefer.serialization=fastjson2,hessian2&release=3.2.15&revision=1.0-local&server=netty&service-name-mapping=true&side=provider&timeout=10000&timestamp=1723539836485&version=1.0-local, dubbo version: 3.2.15, current host: <IP_ADDRESS>

Then, start a another consumer service to invoke the service use rest protocol.
By default, dubbo will use okhttp to send http request,
After send the request, here is the provider service's log
POST /dubbo-rest/getDubboInfo HTTP/1.1
Host: <IP_ADDRESS>:8080
User-Agent: okhttp/4.12.0
Content-Length: 8
Accept: application/json; charset=UTF-8
Accept-Encoding: gzip
Content-Type: application/json
Rest-Service-Path: com.doodl6.springboot.dubbo.api.FirstDubboService
Rest-Service-Remote.application: springboot-web-local
Rest-Service-Version: 1.0-local, dubbo version: 3.2.15, current host: <IP_ADDRESS>, error code: . This may be caused by rest service Path no found, current path info:PathMatcher{path='/dubbo-rest/getDubboInfo', version='null', group='null', port=null, hasPathVariable=false, contextPath='null', httpMethod='POST'}, go to  to find instructions.

org.apache.dubbo.rpc.protocol.rest.exception.PathNoFoundException: rest service Path no found, current path info:PathMatcher{path='/dubbo-rest/getDubboInfo', version='null', group='null', port=null, hasPathVariable=false, contextPath='null', httpMethod='POST'}
	at org.apache.dubbo.rpc.protocol.rest.filter.ServiceInvokeRestFilter.doHandler(ServiceInvokeRestFilter.java:90) ~[dubbo-3.2.15.jar:3.2.15]
	at org.apache.dubbo.rpc.protocol.rest.filter.ServiceInvokeRestFilter.filter(ServiceInvokeRestFilter.java:69) ~[dubbo-3.2.15.jar:3.2.15]
	at org.apache.dubbo.rpc.protocol.rest.handler.NettyHttpHandler.executeFilters(NettyHttpHandler.java:152) ~[dubbo-3.2.15.jar:3.2.15]
	at org.apache.dubbo.rpc.protocol.rest.handler.NettyHttpHandler.handle(NettyHttpHandler.java:84) ~[dubbo-3.2.15.jar:3.2.15]
	at org.apache.dubbo.rpc.protocol.rest.netty.RestHttpRequestDecoder.lambda$decode$0(RestHttpRequestDecoder.java:69) ~[dubbo-3.2.15.jar:3.2.15]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[na:na]
	at org.apache.dubbo.common.threadlocal.InternalRunnable.run(InternalRunnable.java:39) ~[dubbo-common-3.2.15.jar:3.2.15]
	at java.base/java.lang.Thread.run(Thread.java:840) ~[na:na]

The reason is okhttp formatted the request header's name to Upper Camel Case, but in provider side, it only use the original header's name to get value

What you expected to happen
Provider side service can handle the custom header's value correctly
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

it is ok in my test,  i don't see the operate about "okhttp formatted the request header's name to Upper Camel Case". you can upload the demo to further discuss.

I can't reproduce it now either. Maybe it's affected by other configurations. This issue can be closed


Typed import/export definitions

How to categorize this PR?

/area quality
/kind api-change
/priority 2
What this PR does / why we need it:
Currently, data and target import definitions in a blueprint are distinguished by whether the schema or targetType field is specified. However, we plan on adding a few more types and the implicit distinction gets uglier with every new one, therefore we decided to add a type field which explicitly states the type of the import.
For backwards compatibility, we cannot make the field mandatory, as this would break all existing blueprints. Thus, the type field is optional for now, but will likely become mandatory with the next big api-change.
Release note:

TODO

/test

/test

/test

/test

/test

/help

/touch

/test

/test

/test

/test

@schrodit Please have a look again.


Initial commit for Module 15

do we really need the outline slide (#3)?? Perhaps we should organized it same way we organized l14. i.e., in modules, all in one slide deck. Say modules 15.1, 2, 3 .... and insert empty slide before each section.
I actually liked the public-private-key image we had used (with some updates), amazon one.
Formatting nitpick: different headings have different font sizes
There is a lot of material but I will have to present in a cohesive manner. Can't think of one right now.
We start by taking about 5 vulnerabilities but end up with 4.
3rd item is outline is discussed where?
I will have to review these again in the morning with clearer head.

On second read .... I think lecture looks good. Needs some minor updates:
Slide #7: Suggest adding the picture of "house with glass windows" back in. That way we can discuss some options to make the house secure.
Slide 35: Suggest adding the slide with Amazon/CA public private key animation here before slide that discusses CA compromise.
Still need to make sure that we correct the numbers (of vulnerabilities) in the last slide (says 4), and Slide 14 (list of vulnerabilities we will discuss, says 5) and Slide 2 (learning objectives, says 5)
Rest lrtm
I use this on Monday for lecture.

On second read .... I think lecture looks good. Needs some minor updates:
Slide https://github.com/neu-se/CS4530-Fall-2023/pull/7: Suggest adding the picture of "house with glass windows" back in. That way we can discuss some options to make the house secure.

Done.  I always found that slide to be a distraction, but I've put it in so you can have it.

Slide 35: Suggest adding the slide with Amazon/CA public private key animation here before slide that discusses CA compromise.

Restored. I always found the animation confusing--  the text on the slide before is intended to be a clearer version.

Still need to make sure that we correct the numbers (of vulnerabilities) in the last slide (says 4), and Slide 14 (list of vulnerabilities we will discuss, says 5) and Slide 2 (learning objectives, says 5)

I think I got it this time.   Also put in a title slide introducing Outline item # 3.

Link to OWASP website is useful We should add it in slide # 14

It's already on the module page.  I added some text in the speaker's notes to slide 14.   Also updated the list from 2017 to 2021 (the most recent version).

Rest lrtm
I use this on Monday for lecture.
I hope it's now in usable shape for you.

apart from two minor issues (1) different font sizes for headings, (2) last slide still says 4, lgtm

ok,ok.   Fixed the last slide.   Fixed some of the bad headings.  (Stupid Keynote formats).  That's all I can do tonight.
G'nite.


CoreMessagingXP.dll crash

Describe the bug
I am maintaining a .NET application with a WinUI3 UI which sometime crashes. In the event viewer I see that the crash is caused by CoreMessagingXP.dll but I have no idea about the real cause of the problem since I see the application crashing without any exception an I see the event shown below in the event viewer. At the moment I have no idea about how I could fix this or where to start searching for a solution. The OS is Windows 10 Enterprise (ver 1809). WindowsAppSDK 1.2.230118.102
Steps to reproduce the bug
It happens randomly. I have not found a procedure to reproduce the bug.
Expected behavior
No response
Screenshots
No response
NuGet package version
None
Windows version
Windows 10 (1809): Build 17763
Additional context
<Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
- <System>
  <Provider Name="Application Error" />
  <EventID Qualifiers="0">1000</EventID>
  <Level>2</Level>
  <Task>100</Task>
  <Keywords>0x80000000000000</Keywords>
  <TimeCreated SystemTime="2024-04-16T22:29:53.7299382Z" />
  <EventRecordID>69252</EventRecordID>
  <Channel>Application</Channel>
  <Computer>HAWKAGLE4PRO0265</Computer>
  <Security />
  </System>
- <EventData>
  <Data>HAWKManager.exe</Data>
  <Data><IP_ADDRESS></Data>
  <Data>65ab0000</Data>
  <Data>CoreMessagingXP.dll</Data>
  <Data>10.0.22622.1036</Data>
  <Data>d9ffb8dc</Data>
  <Data>c000027b</Data>
  <Data>000000000009418c</Data>
  <Data>21e4</Data>
  <Data>01da904d905aaf91</Data>
  <Data>C:\Program Files\PLL\HAWK Manager X\HAWKManager.exe</Data>
  <Data>C:\Program Files\PLL\HAWK Manager X\CoreMessagingXP.dll</Data>
  <Data>c063be60-6ef1-456e-a02e-ee33adaf34be</Data>
  <Data />
  <Data />
  </EventData>
  </Event>

Just wanted to corroborate that I have seen this issue on Windows App SDK 1.5.7 when running (debugging) my packaged .NET 8 WinUI 3 app for the second time after cloning my project onto a workstation. This crash happened after approximately one second of the app window spawning (with no XAML elements showing). I couldn't reproduce it after the crash.
Event Viewer log:
Faulting application name: safesurfer.exe, version: <IP_ADDRESS>, time stamp: 0x66e80000
Faulting module name: CoreMessagingXP.dll, version: 10.0.26105.1010, time stamp: 0x3cb81cec
Exception code: 0xe0464645
Fault offset: 0x000000000000a387
Faulting process id: 0x21E0
Faulting application start time: 0x1DB322D535375FB
Faulting application path: C:\Users\[User]\Source\Repos\SafeSurfer-Windows\src\SafeSurferWindows\bin\x64\Debug\net8.0-windows10.0.26100.0\win-x64\AppX\safesurfer.exe
Faulting module path: C:\Program Files\WindowsApps\Microsoft.WindowsAppRuntime.1.5_5001.275.500.0_x64__8wekyb3d8bbwe\CoreMessagingXP.dll
Report Id: 7dbcc311-2f5b-4354-a4e2-c1a6b767917f
Faulting package full name: SafeSurferLimited.SafeSurfer_<IP_ADDRESS>_x64__g9f2k15qvwspg
Faulting package-relative application ID: App


Check whether there exists file conflict on both scheduler and judger

If there is a file conflict, the behaviour depends on which file is prepared later than others. This is REALLY strange because it will cause the result to be random.
如果发生文件冲突，评测行为将依赖于评测程序中哪个文件后被准备好。这个行为会导致评测结果是随机的。
Example:
src.hpp is mentioned in the supportedFiles in config.json, and the spj version is 4. Either the file in the problem package or the file submitted will be use depending on which one has been prepared later.
在 config.json 的 supportedFiles 里有 src.hpp，评测点为 spj 4。评测机可能使用题目数据包中的 src.hpp，也可能使用用户提交的 src.hpp。
Solution:
解决方案：

[x] (Quick fix) The judger should detect the file conflict before fetching. 评测机应当在抓取数据前检测文件冲突。
[ ] (Better fix) The scheduler should detect the file conflict when generating the judger plan. Scheduler 应当在生成评测计划的时候检测文件冲突。

Note: The quick fix should be done even if the better fix is done because there are some judger plan that has already got a file conflict.
注意：即使已经完成了 better fix，也需要实现 quick fix，因为已经有了错误的数据包。

59d4a272db548aafe61437a86c54024a60e4a5a2 implements the quick fix.


[POC] ARROW-10671: [FlightRPC] Bearer Token refresh design with retry

8780

 Quality gate: Failed
2 Issues

 2 Code issues

Details


Add ToolboxItemContainer Bridge Gadget

Hi everybody,
as promised here is another bridge gadget for you I found several days ago as a by-product.
Not very exciting but at least allowed to bypass some filters for a custom app.

Awesome, thanks for the contribution!

Thanks!


Support for tvOS (Apple TV)

It seems to be supported already: https://github.com/jdg/MBProgressHUD/blob/59a8240da67a8c98ac7e77da2fa198d21b1ab03c/MBProgressHUD.podspec#L16 (#357). Or have you found issues with it?

There is even a small demo app in 1.0. If you find any issues, please open additional tickets.


[core] Prepare for Feb 2024 releases

bump minor versions for @azure/abort-controller dependency version upgrade
update CHANGELOG

API change check
API changes are not detected in this pull request.


Add support for additional J4U compatible supermarkets

This includes Albertson's and Vons as they use the same user repository database.

This is an "in-progress" issue.  Currently, the following markets have been validated and are supported:

Albertsons
Vons
Pavilions
Carrs

More to follow

Added support for Tom Thumb


Allow passing command into container definition

@byujol has a use-case where he'd like to use a single docker image in multiple ways, which docker supports by passing in a command.
I'd normally call this a minor-version update, but since we can't have default values for attributes on an object like primary_container_definition (until this is no longer experimental), users would need to explicitly pass in command = null. Are we calling that a major-version update?

☝ The test worked, BTW.

☝ The test worked, BTW.

I guess we'll err on the side of caution and call it a major update.

I guess we'll err on the side of caution and call it a major update.


3 versions of collectResults.m

What are the different collectResults.m files? Which one should we use?
collectResults.m
collectResults.m-d1552287-482b-4c88-a3c7-e5c52d475cf7
collectResults.m-e232cfd4-c3d6-40d3-9086-3379c1d58234

Good question. I seems that collectResults.m is the right one and the rest can be deleted.

On 28 Sep 2016, at 14:22, Daniel Weindl<EMAIL_ADDRESS>wrote:
What are the different collectResults.m files? Which one should we use?
collectResults.m
collectResults.m-d1552287-482b-4c88-a3c7-e5c52d475cf7
collectResults.m-e232cfd4-c3d6-40d3-9086-3379c1d58234
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ICB-DCM/PESTO/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/ALuj_gVb3lKoKR7SFUVjRc2lG_XrB-bAks5qulwggaJpZM4KIwRk.

Okay, removed the additional ones in 0f9b915472fc34b0801fe9bbf4ce08587d1bd3e7


[AIO] Make CallbackWrapper the wrapper for functors in client-side

As title.

This issue should also propagate debugging information into the callback, so if the callback failed, our framework can log something that is helpful for debugging.


naming convention

CNCC
Customizable Naming Convention Checker can report on identifiers in your code that do not follow certain naming conventions.

Looks like CNCC is not maintained anymore.
There is similar functionality in clang-tidy: https://clang.llvm.org/extra/clang-tidy/checks/readability/identifier-naming.html


Preserve empty lines in wordWrap

At the moment, word wrap inputs which contain multiple newlines in a row lose empty lines in the output, this PR fixes such behavior.

@fogleman , please check this one merge request


[Feature Request] Add isNil function

Hi!
I just discover radash and is awesome!
It only left one mising function for me:
isNil

https://github.com/lodash/lodash/blob/4.17.15/lodash.js#L11973
Maybe I would make a double check with === undefined || === null, but the interesting part is the types, something like this
isNullOrUndefined = (value: T | null | undefined): value is null | undefined => {
  return value === null || value === undefined;
}

https://stackoverflow.com/questions/65901270/typescript-check-variable-null-or-undefined
You thing is a request that can be aligned with radash principles? Thanks!

Hey @Nisgrak 👋 This might be a fit for radash I just personally wouldn't use it. With es5+ features like nullish coalescing ?? and optional chaining ?. I haven't had any need for an IsNil or isNullOrUndefined function (though I remembering wanting and using it years ago).
These are small functions and easy additions I'm just not sure they provide much value. I also try not to overlap radash features solutions that exist natively in javascript now. I'm open to it though!

Closing this due to lack of activity. If anyone feels strongly about it comment, would be happy to add it (or merge a PR with it) if others agree it's useful.

For one obvious case where this is useful, shake(someObject, isNil) to remove both undefined and null values instead of just undefined values.

Hi! I would love to have this for readibility (I know is maybe not a huge point), but I like to do:
// assuming 0 and false are acceptable values
if ( isNil(someVar) ) throw new Error('missing property someVar');
// instead of
if ( someVar === undefined || someVar === null ) throw new Error('missing property someVar');

At least for me is faster and easier to undersant


documentMode=string is undocumented

Describe the bug
This feature was added in d0acf100e750d9a3cbad3aebaa7e705fdcb9627f and released in 1.8.3 but is not documented.

Website and docs includes docs for it now :)


DSC_Disk: Add support for creating Dev Drive volume

Problem description
Windows has released a new feature called Dev Drive that is currently being used by developers to increase their performance workloads. One
thing that has come up is the need to be able to create a Dev Drive through a DSC configuration. Dev Drives are ReFS drives with special features under the hood. Currently to create a Dev Drive one can use the Format-Volume  powershell cmdlet with the -DevDrive flag. The Dev Drive feature is now in GA with 10.0.22621.2338 or later.
The StorageDsc appears to be a good place to add this ability. The Disk resource already has the ability to format volumes among other volume related scenarios like:

Ability to change the volume label
Ability to add a partition style either GPT  or MBR  to an uninitialized disk.
Ability to change the partition style from GPT to MBR  and vice versa.
Ability to rename the volume label.
Ability to resize a partition.
Ability to create a new partition from unallocated space and format a volume into this new partition. Users can use the size parameter to say how much space they want to use or not use the size parameter to use all the space available.
Ability to set a disk online,
Ability to mark a disk from readonly  to write.
Ability to clear a disk.

Note: The Disk Dsc currently does not have a parameter for 'Present' and 'Absent', so the configuration is mostly "if x drive letter on y disk matches all the parameters inputted, do nothing. But if a parameter does not match what was inputted update it so matches the inputted parameters".
We just would like to add one more parameter to the disk resource called DevDrive that will allow users to create a Dev Drive volume using the Disk dsc resource.  Note: Dev Drive volumes should only be created using 50 Gb of space. The disk resource will need to be aware of this when this new Flag is used.

A flag that when set allows a new partition to be created from unallocated space and a new Dev Drive volume formatted onto this new partition.
A flag that when set, will check if the volume with the drive letter from the disk that the user inputted, is a Dev Drive volume or not. If it is a Dev Drive volume no new Dev Drive related work from this change is needed. This will help with idempotence.
If no unallocated space exists, attempt to safely shrink an existing volume to create enough contiguous unallocated space to create a new partition for the new Dev Drive volume. For this scenario the AllowDestructive flag would be used.
Format an existing drive that holds the drive letter that the user inputted as a parameter to be a Dev Drive volume if drive has enough space to hold a Dev Drive volume. Note: both NTFS and ReFS volumes can be reformatted as Dev Drive volumes assuming they have enough space. However Dev Drive volumes from a storage perspective are just like regular ReFS volumes on a Windows machine. The difference However, is that most of the filter drivers except the antivirus filter will not attach to the volume at boot time by default. This is a low-level concept that most users will never need to interact with but for further reading, see the documentation here for further reading.

prototype in forked branch
example config could be:
<#
    .DESCRIPTION
        For this scenario we want to create two 60 Gb Dev Drive volumes. We know that disk 2 has 3 existing
        NTFS volumes and we prefer not to remove them. At most we only want the disk DSC resource to shrink any
        of them should there not be enough space for any of the Dev Drive volumes to be created. We also know that the
        the 3 existing volumes are 100Gb, 200Gb and 300Gb in size and disk 2 is 600 Gb in size. Since all the space
        is being used by the existing volumes, The Disk Dsc resource will resize the existing volumes to create
        space for our new Dev Drive volumes. An example of what could happen is the Disk resource could resize the
        300Gb volume to 240Gb for the first Dev Drive volume and then resize the 240Gb volume again to 180Gb for the second.
        Thats just one combination, the disk Dsc resource uses the Get-PartitionSupportedSize cmdlet to know which volume
        can be be resized to a safe size to create enough unallocated space for the Dev Drive volume to be created. Note:
        ReFS volumes cannot be resized, so if the existing volumes were all ReFS volumes, the Disk Dsc resource would not be able
        to resize any volumes and would instead throw an exception.

        This configuration below will wait for disk 2 to become available, and then create two new 60 Gb Dev Drive volumes,
        'E' and 'F'. The volumes will be formatted as ReFS volumes and labeled 'Dev Drive 1' and 'Dev Drive 2' respectively.
        Note: setting 'AllowDestructive' to $true will not cause the disk to be cleared, as the flag is only used when there
        is a need to resize an existing partition. It is used as confirmation that you agree to the resizing which will
        create the necessary space for the Dev Drive volume. This flag is **NOT** needed if you already know there is enough
        unallocated space on the disk to create the Dev Drive volume. If this flag is not used and there is not enough space
        to create the Dev Drive volume an error will be thrown and the Dev Drive will not be created. Its important to be very
        careful not to add the 'ClearDisk' flag while using the 'AllowDestructive' flag, as this will cause the disk to be cleared,
        and all data lost on the disk (even existing volumes).
#>
Configuration Disk_CreateDevDriveOnDiskWithExistingPartitions
{
    Import-DSCResource -ModuleName StorageDsc

    Node localhost
    {
        WaitForDisk Disk2
        {
            DiskId = '5E1E50A401000000001517FFFF0AEB84' # Disk 2
            DiskIdType = 'UniqueId'
            RetryIntervalSec = 60
            RetryCount = 60
        }

        # Will create a Dev Drive volume of 60 Gb called Dev Drive 1.
        Disk DevDrive1
        {
            DiskId = '5E1E50A401000000001517FFFF0AEB84'
            DiskIdType = 'UniqueId'
            DriveLetter = 'E'
            FSFormat = 'ReFS'
            FSLabel = 'DevDrive 1'
            DevDrive = $true
            AllowDestructive = $true
            Size = 60Gb
            DependsOn = '[WaitForDisk]Disk2'
        }

        # Will create a Dev Drive volume of 60 Gb called Dev Drive 2.
        Disk DevDrive2
        {
            DiskId = '5E1E50A401000000001517FFFF0AEB84'
            DiskIdType = 'UniqueId'
            DriveLetter = 'F'
            FSFormat = 'ReFS'
            FSLabel = 'DevDrive 2'
            DevDrive = $true
            AllowDestructive = $true
            Size = 60Gb
            DependsOn = '[Disk]DevDrive1'
        }
    }
}

Verbose logs
nothing here

DSC configuration
nothing here

Suggested solution
nothing here
Operating system the target node is running
nothing here

PowerShell version and build the target node is running
nothing here

StorageDsc version
nothing here

FYI @PlagueHO

Looks good! Thanks @bbonaby !

@PlagueHO FYI PR is out, added tests and examples to confirm unit test functionality. The Dev Drive feature isn't available in Windows server 2019 and as far as I know won't be in Server 2022 either at least for now. I'll have the virtual hard disk PR out hopefully today as well.

updated the issue statement to better reflect what's in the PR today.

The deploy fails because there is a space in the tag that was added in this PR. Dev Drive should be changed to DevDrive in the module manifest (in a new PR): https://github.com/dsccommunity/StorageDsc/pull/278/files#diff-8eee5c09560a979f88c375c8c47f71c0d85a219e61956b96f31efc5988fa2b02R55
See error: https://dev.azure.com/dsccommunity/StorageDsc/_build/results?buildId=8223&view=logs&j=5d0a9c4e-8741-5118-af45-406a30fe661c&t=bda0b158-a237-516f-c1d0-31339587c5b5&l=78

Created a new issue to track it https://github.com/dsccommunity/StorageDsc/issues/280.


Support specifying what commands to use for compiling stuff

Is your feature request related to a problem? Please describe.
Right now, (as far as I can tell) compilers are hard-coded to gcc, g++, go, etc. In some cases, this means that someone would need to install a different compiler than the one they're already using for this plugin to work.
This could also allow for using a compiler cache like ccache or sccache so things run a little faster (but that's not as important)
Describe the solution you'd like
An option to specify what compilers to use.
Describe alternatives you've considered
All I've got right now is changing the hard-coded values in the source code, and then building from scratch.
For C & C++ specifically, the default compilers could be changed to respect the $CC & $CXX environment variables respectively, and default to cc and c++ respectively. Those are arguably safer if it stays hard-coded.

You are totally right, that should be an option,   but it's not as simple as you would think:

all compiler don't have feature parity (for example, only gcc allows functions definition inside of another function)
the output on different compiler/interpreters (on compilation or runtime errors) might be too different to properly display a message: I rely heavily on the fact that gcc provide a meaningfull message on a particular line of its otherwise undereadably long output

That being said, gcc/clang is a bit of a corner case and most other languages will benefit from this idea at no cost. I see some way of doing that, and it could have been complicated but boy I'm happy to have thought ahead: I see a relatively straightforward way of achieving something satisfying


[Android] [BackAndroid] New listeners registered while back event is handled also receive the current event

I just faced for me unexpected behavior when (indirectly) registering a listener to BackAndroid while an event is currently handled.
In my project I use a Root component type, which calls a function provided to it via the onBackPress prop. It looks something like this:
export default class Root extends React.Component {

  _handleBackButton = () => {
    if (this.props.onBackPress) {
      this.props.onBackPress();
      return true;
    }
    return false;
  };

  componentDidMount() {
    BackAndroid.addEventListener('hardwareBackPress', this._handleBackButton);
  }

  componentWillUnmount() {
    BackAndroid.removeEventListener('hardwareBackPress', this._handleBackButton);
  }

  render() {
    return (
      …
  }
}

Now if the back button is pressed and, as part of the onBackPress function, the current Root element is replaced by a new one, the new Root element's componentDidMount is executed before onBackPress returns, adding the new event listener to BackAndroid.
Problem is: BackAndroid will add the new listener to the Set of listeners it loops over and eventually call the newly added listener for the same event it was created by.
I think the better behavior would be not to modify the Set of listeners while looping over them to dispatch an event. In case the Set needs to be modified create a copy, apply the modification there and continue to use the original Set to complete handling of the current event.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/android-backandroid-new-listeners-registered-while-back-event-is-handled-also-receive-the-current-event
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Cm 25  partial classes in c#

Created example Console project with unit tests for [CM-25] (https://content.code-maze.com/browse/CM-25)
Partial Classes In C# task

Aborting because of some trouble with gitignore


[question] How to use the option when initialize the middleware

Is this a BUG REPORT or FEATURE REQUEST?:
/kind question
I implement a customized middleware to add the DB interface to the context, and the middleware supports different DB backends (e.g. SQLite3, MySQL and so on), thus I use the option to let users define the backend.
But, I found that we cannot use the option before cmd.ExecuteWithConfig, since the values will all be empty before the command is actually run. While the middleware should be configured before the cmd is run, I am not sure if we could use the option in middleware in the current design. Do you have any suggestion?
I'd appreciate it if you could answer.

Middleware is a concept of router. You need to implement a plugin witch install a db middleware to router.
Here is the document which describes how to implement plugins: https://caicloud.github.io/nirvana/zh-hans/concepts/plugin.html
Also you can find an example at: https://github.com/caicloud/nirvana/blob/master/plugins/reqlog/reqlog.go

Thanks, gotcha. I implemented a middleware directly. It seems that I misunderstand the concept of middleware. I will take a look.


Add CommonName to the self signed certificate

Without a CN the self signed certificate is considered
invalid by chrome.
You can check with: openssl x509 -in cert.pem -subject -noout
Fixes issue #1231

Thanks!


⚠ Avatar API Endpoint has degraded performance

In 0b5ca9c, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4938 ms

Resolved: Avatar API Endpoint performance has improved in fc0ad43 after 3 minutes.


Exception when using AES GCM on Android API 19

Currently encrypting on Android API 19 using AesGcmKeyManager.aes256GcmTemplate() results in an InvalidAlgorithmParameterException:
java.security.InvalidAlgorithmParameterException: unknown parameter type.
    at com.android.org.bouncycastle.jcajce.provider.symmetric.util.BaseBlockCipher.engineInit(BaseBlockCipher.java:559)
    at javax.crypto.Cipher.init(Cipher.java:616)
    at javax.crypto.Cipher.init(Cipher.java:566)
    at com.google.crypto.tink.subtle.AesGcmJce.encrypt(AesGcmJce.java:75)
    at com.google.crypto.tink.aead.AeadWrapper$WrappedAead.encrypt(AeadWrapper.java:52)
    at com.appmattus.layercache.Encryption.encryptValue(Encryption.kt:87)
    at com.appmattus.layercache.Encryption.access$encryptValue(Encryption.kt:47)

This fails on API 19 because AesGcmJce.getParams will generate a GCMParameterSpec as the class exists, however, the cipher doesn't support it. Instead the logic in getParams on Android should use IvParameterSpec when Build.VERSION.SDK_INT <= Build.VERSION_CODES.KITKAT even though GCMParameterSpec appears "available".
Additionally, once the above is resolved, another error will be returned instead due to the call(s) to updateAAD:
java.lang.UnsupportedOperationException: This cipher does not support Authenticated Encryption with Additional Data
    at javax.crypto.CipherSpi.engineUpdateAAD(CipherSpi.java:393)
    at javax.crypto.Cipher.updateAAD(Cipher.java:1056)
    at com.google.crypto.tink.subtle.AesGcmJce.encrypt(AesGcmJce.java:77)
    at com.google.crypto.tink.aead.AeadWrapper$WrappedAead.encrypt(AeadWrapper.java:52)
    at com.appmattus.layercache.Encryption.encryptValue(Encryption.kt:87)
    at com.appmattus.layercache.Encryption.access$encryptValue(Encryption.kt:47)

The only way to resolve this is, again, specifically on API 19 ensuring that the associatedData to Aead.encrypt and Aead.decrypt is always null. Would it be better for the associatedData to not be used when running API 19 (or less) rather than relying on the user stumbling upon the comment in the source code and needing to guard against it themselves in their call to encrypt/decrypt?
Once those two issues are resolved then AES GCM encryption works on API 19.

Good catch! I'll try to reproduce and fix it.

This fails on API 19 because AesGcmJce.getParams will generate a GCMParameterSpec as the class exists, however, the cipher doesn't support it. Instead the logic in getParams on Android should use IvParameterSpec when Build.VERSION.SDK_INT <= Build.VERSION_CODES.KITKAT even though GCMParameterSpec appears "available".

I'm fixing this. The change will be included in 1.5.0.

The only way to resolve this is, again, specifically on API 19 ensuring that the associatedData to Aead.encrypt and Aead.decrypt is always null. Would it be better for the associatedData to not be used when running API 19 (or less) rather than relying on the user stumbling upon the comment in the source code and needing to guard against it themselves in their call to encrypt/decrypt?

This is a known issue, see the last bullet in https://github.com/google/tink/blob/master/docs/KNOWN-ISSUES.md#android. If a user sets an associated data, we can't silently ignore it, but we have to throw an exception to notify the user that what they wanted to do is not supported by Android.


Harvest articles from Trove list

Recipe request from @MichWatsonOz -- harvest newspaper articles from a Trove list.

41c6f86b39c1e657c81790dd42eaaa062562b035

Added: 41c6f86b39c1e657c81790dd42eaaa062562b035


firefox: node states not displayed correctly

works in FF 121 or later: https://caniuse.com/?search=%3Ahas


Images either pseudotransparent or have no margin padding

What browsers are you seeing the problem on?
Firefox
Bug report for the website
Probably as a byproduct of enabling dark mode (thanks a lot for that, helps the eyes!) site-wide, certain webpages now feature images that
Expected Formatting

Source: https://docs.substrate.io/tutorials/v3/kitties/pt1/
Buggy Formatting

Source: https://docs.substrate.io/tutorials/v3/proof-of-existence/

Source: https://docs.substrate.io/tutorials/v3/create-your-first-substrate-chain/
Steps to reproduce the problem
Visiting the webpage.
It is an problem within static assets so it isn't really a "bug" in the technical sense but regardless still a visual inconsistency.

Thank you for flagging this @kittywhiskers. This is something we definitely plan on doing soon! Adding the design label too.


cannot convert from 'System.Linq.Expressions.Expression<System.Func>' to 'System.Func' [Tailspin.SpaceGame.Web.Tests]csharp(CS1503)

@tpetchel, I'm triaging two Customer Feedback work items related to mslearn-tailspin-spacegame-web:

CUSTOMER FEEDBACK 535277 | Errors in Run quality tests in your build pipeline by using Azure Pipelines
CUSTOMER FEEDBACK 535628 | Build failure in module - Run quality tests in your build pipeline by using Azure Pipelines

Both work items are related to the same issue. I've confirmed the issue locally:
$ dotnet build --configuration Release
Microsoft (R) Build Engine version 17.0.0+c9eb9dd64 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  Restored C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web\Tailspin.SpaceGame.Web.csproj (in 122 ms).
  Restored C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web.Tests\Tailspin.SpaceGame.Web.Tests.csproj (in 502 ms).
  Tailspin.SpaceGame.Web -> C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web\bin\Release\net5.0\Tailspin.SpaceGame.Web.dll
  Tailspin.SpaceGame.Web -> C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web\bin\Release\net5.0\Tailspin.SpaceGame.Web.Views.dll
C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web.Tests\DocumentDBRepository_GetItemsAsyncShould.cs(43,17): error CS1503: Argument 1: cannot convert from 'System.Linq.Expressions.Expression<System.Func<TailSpin.SpaceGame.Web.Models.Score, bool>>' to 'System.Func<TailSpin.SpaceGame.Web.Models.Score, bool>' [C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web.Tests\Tailspin.SpaceGame.Web.Tests.csproj]

Build FAILED.

C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web.Tests\DocumentDBRepository_GetItemsAsyncShould.cs(43,17): error CS1503: Argument 1: cannot convert from 'System.Linq.Expressions.Expression<System.Func<TailSpin.SpaceGame.Web.Models.Score, bool>>' to 'System.Func<TailSpin.SpaceGame.Web.Models.Score, bool>' [C:\Users\v-jopar\mslearn-tailspin-spacegame-web\Tailspin.SpaceGame.Web.Tests\Tailspin.SpaceGame.Web.Tests.csproj]
    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:05.81

I checked the history of mslearn-tailspin-spacegame-web/Tailspin.SpaceGame.Web.Tests/DocumentDBRepository_GetItemsAsyncShould.cs and I'm not seeing any recent changes that might be causing the issue. That's as far as my level of expertise can take me. Please have a look at the file when you get a moment. Thanks.

I'm completing this course right now and came to this issue to find the solution, but I ended up fixing it myself by putting .Compile() after the call to queryPredicate.

@john-par - thanks for reporting. I made some changes the other week, and apparently forgot to apply them to the unit-tests branch. (The fix was to remove the Expression wrapper from the query expression.)
I pushed up a fix just now. If you fetch the latest unit-tests branch, dotnet build and dotnet test should now succeed.
Sorry for trouble, and thanks again for reporting!


Multi-version testing

Not sure if this is going to work yet, but Azure Pipelines offers 10 parallel jobs, so why not give it a try.
This almost works right now, but the standard PostgreSQL image (obviously) doesn't include PostGIS, so we need to work out how to add that to it.
Related
#646
#928
#934

I'll give this a review when I have time, but I still think this would add more trouble than it would save. See https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/issues/710 and https://github.com/npgsql/npgsql/pull/2477#issuecomment-495538881

TODO:

Setup releases to NuGet.
Investigate containerizing the Windows build.

Planning to merge this in the next hour. There's a few files that conflict with #943, but I've attempted to cluster those changes to the last commit (e.g. 2cf729b3159ba14d68db3e217b577931a5f1fff6).

One more idea: why don't we starting running tests on PostgreSQL 12 beta1 as well? It will allow us to start working on new PG 12 features (I'm thinking about generated columns in particular), and would in theory flush out any breaking changes on the PG side.

Testing against the betas sounds like an excellent idea. I'll look into wiring that up as part of phase 3.


Ecological tournament variant

I'm not sure if it was organized by Axelrod, but there was a subsequent "ecological" tournament where each strategy was represented by some number of copies. The tournament consisted of ~1000 generations, and at the beginning of each generation the population of a strategy was set based on its score in the previous round.
This interesting variant could be easily built on top of the current code.

Of course, this tournament can be based on the average payoff matrix determined in the round robin phase.

I'd not heard of this subsequent tournament: sounds like a great idea though. I wrote some code that allowed for a genetic algorithm to be applied to any normal form game (I used it for this screencast: https://www.youtube.com/watch?v=Tz-lZy0AKRI)... I like the idea of this :+1:

Not sure if the PR is still coming: just a thought that it could be nice to start seperating things. So this could live in it's own file under axelrod/? Similarly I might pull the players out in to their own file... Just to keep things tidy. Any reason not to do this?

BOOM @langner killed this one with 062be93e19c19594832744806d23f24f32bb2a8b :D


Support retake history and other grades report-related adjustments

now it's possible to keep the register of the grade before the retake process when needed. closes #174
grade calculation functions where adjusted to support entries with student self-assessment but no teacher assessment. fixes #175

forgot to mention: added some UI adjustments for grades report grid in report card's "Grades" tab, preventing content from being (aggressively) clipped


Moved statement methods to cached properties

Caching improves performance by approximately 15%.
This accomplishes what was intended to be the goal of _utils.memoize_weak mentioned in #47.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

39 of 40 (97.5%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 98.207%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ssort/_statements.py
17
18
94.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
1260

Relevant Lines:
1283

💛  - Coveralls


cross-platform inconsistency in I2C bus speeds

The I2C bus specification in section 5 specifies these named speeds:

Standard up to 100 kHz
Fast up to 400 kHz
Fast+ up to 1 MHz
High up to 3.4 MHz
Ultra up to 5 MHz (unidirectional only)

Evaluation in https://github.com/zephyrproject-rtos/zephyr/pull/23971#pullrequestreview-387200780 plus code review show that NXP (mcux, mcux_lpi2c, imx) and SiLabs (gecko) I2C drivers use 1 MHz for Fast, and do not implement Fast+.
The interpretation of named I2C bus speeds in Zephyr should be (a) documented and (b) consistent.

@mnkp and @MaureenHelm could you send a patch to correct Fast so it becomes Fast+?

@mnkp and @MaureenHelm could you send a patch to correct Fast so it becomes Fast+?

yes


bugfix: travis-ci exceeded the maximum log length

Signed-off-by: slievrly<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
bugfix: travis-ci exceeded the maximum log length
Ⅱ. Does this pull request fix one issue?
fix #1962
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1961 into develop will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             develop   #1961      +/-   ##
============================================
+ Coverage      55.29%   55.3%   +<.01%
- Complexity      2408    2410       +2
============================================
  Files            429     429
  Lines          14279   14279
  Branches        1676    1676
============================================
+ Hits            7896    7897       +1
- Misses          5675    5676       +1
+ Partials         708     706       -2

Impacted Files
Coverage Δ
Complexity Δ

...very/registry/zk/ZookeeperRegisterServiceImpl.java
63.77% <0%> (-0.79%)
25% <0%> (ø)

...o/seata/server/coordinator/DefaultCoordinator.java
48.41% <0%> (+0.39%)
28% <0%> (+1%)
:arrow_up:

...in/java/io/seata/server/session/GlobalSession.java
85.02% <0%> (+0.48%)
68% <0%> (+1%)
:arrow_up:


Missing data references

Version and Platform (required):

Binary Ninja Version: 3.4.4242-dev, 2e3a4286
OS: macos
OS Version: 13.3
CPU Architecture: arm64

Bug Description:
A function is referring to a memory address but the location is not being cross-referenced. The reference is indirect and the references appear as soon as I re-type the variable holding the base address as a pointer.
Shouldn't BN be doing this for me automatically, though? I would have to have to go through each and every function to change types.

Additional Information:

Related to the above. All of the ints here hold function pointers. BN has created the functions for me. Why hasn't BN adjusted these to be pointers, though?

I think you might be running into the same thing I did in #3746.

V35 should search for "Encourage Salesman Prompt Delay" to find the database.

This was complete in 3.5.4376 along with pointer sweep. There does still seem to be issue due to some missing lifting.


Path Parameter annotations in interface are parsed as body param

When the @PathParam is annotated only to the interface method it is parsed as a body parameter in the swagger.json.
I have a interface for my endpoint which describes the JAX-RS endpoint. I want to have the JAX-RS annotations in the interface and the swagger annotations in the implementation
@Path("/myep")
@Produces(MediaType.APPLICATION_JSON)
public interface MyEndpointInterface{
 @GET
 @Path("/{id}")
public Response getMyData(@PathParam("id") Long id);
}

@Api(value = "/myep")
public class MyEndpointImpl extends MyEndpointInterface{
@ApiOperation(value = "Blah")
@Override
public Response getMyData(Long id){
 //My Impl
}
}

In the case above the id parameter is parsed as a body parameter.
I think the reason for this is the way parameter annotations of a method are read in the Reader.java (line 838).
Annotation[][] paramAnnotations = method.getParameterAnnotations();

Because of this only the annotations on the child method get parsed and the ones defined in the interface are not parsed.
I have created a workaround in ReflectionUtils.java to parse the parameter annotations in the base classes and interface as well. Please let me know if the same is acceptable
//Change in Reader.java
Annotation[][] paramAnnotations = ReflectionUtils.getParameterAnnotations(method);

//Method in ReflectionUtils.java
public static Annotation[][] getParameterAnnotations(Method method) {
  Annotation[][] methodAnnotations = method.getParameterAnnotations();
  Method overriddenmethod = getOverriddenMethod(method);

  if (overriddenmethod != null) {
   Annotation[][] overriddenAnnotations = overriddenmethod.getParameterAnnotations();

   for (int i = 0; i < methodAnnotations.length; i++) {
     List<Type> types = new ArrayList();
    for(int j = 0; j < methodAnnotations[i].length; j++){
     types.add(methodAnnotations[i][j].annotationType());
    }
    for(int j = 0; j < overriddenAnnotations[i].length; j++){
     if(!types.contains(overriddenAnnotations[i][j].annotationType())){
      methodAnnotations[i] = ArrayUtils.add(methodAnnotations[i], overriddenAnnotations[i][j]);
     }
    }

   }
  }
  return methodAnnotations;
 }

This is using which version?

1.5.4-SNAPSHOT

same problem for me
version 1.5.5

@ssoltanid can you create a pull request?  Looks like a proposed solution is above.

I have the same issue where swagger won't look for jax-rs annotations in interface.
Here is the sample code.
Interface:
package com.example.sampleapi;

import javax.ws.rs.Consumes;
import javax.ws.rs.GET;
import javax.ws.rs.Path;
import javax.ws.rs.Produces;
import javax.ws.rs.QueryParam;
import javax.ws.rs.core.MediaType;
import javax.ws.rs.core.Response;

@Path("authenticationStuff")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_JSON)
public interface ISampleRest {

    @Path("getMethod")
    @GET
    public Response demoGetMethod(@QueryParam("welcomeWho") final String welcomeWho);
}

Implementation:
package com.example.webapi.sample;

import javax.ws.rs.Path;
import javax.ws.rs.Produces;
import javax.ws.rs.core.CacheControl;
import javax.ws.rs.core.MediaType;
import javax.ws.rs.core.Response;

import com.fasterxml.jackson.databind.node.JsonNodeFactory;
import com.fasterxml.jackson.databind.node.TextNode;
import com.tutelatechnologies.dashboard.DashboardInterfaceEngine.server.sampleapi.ISampleRest;

import io.swagger.annotations.Api;
import io.swagger.annotations.ApiParam;

@Path("authenticationStuff")
@Produces(MediaType.APPLICATION_JSON)
@Api(value = "/authenticationStuff", tags = "authenticationStuff")
public class SampleRest implements ISampleRest {

    final JsonNodeFactory factory = JsonNodeFactory.instance;

    @Override
    public Response demoGetMethod(@ApiParam("welcomeWho") final String welcomWho) {
        TextNode textNode = factory.textNode("called get method by: " + welcomWho);
        return getNoCacheResponseBuilder(Response.Status.OK).entity(textNode).build();
    }

    private Response.ResponseBuilder getNoCacheResponseBuilder(Response.Status status) {
        CacheControl cc = new CacheControl();
        cc.setNoCache(true);
        cc.setMaxAge(-1);
        cc.setMustRevalidate(true);

        return Response.status(status).cacheControl(cc);
    }
}

@mohitmutha, @ssoltanid, @nooruddin - 1.5.9-SNAPSHOT has been pushed with the merged PRs that are meant to fix the issue. Can you give it a try and confirm it resolves the case?

I pulled out latest snapshot and plugged it in.  It's seamlessly working now.
Thanks for the update.

In jersey, I guess annotation inheritance works for method level.  But one does need to specify the produces and path annotations at implementation class level.  So in that sense, every other annotations are inherited from interface properly now.

Great! Works as expected now.


Handling system settings, like max_user_watches

Brief
Both ubuntu xenial and docker for mac has the following setting by default:
fs.inotify.max_user_watches = 8192

Which is too little for some projects (like the one of my employer)
Various tools react to exceeding limit (error ENOSPC) differently:

webpack --watch silently crashes
nodemon shows errno but doesn't say how to fix it
Some tools just fallback to slower stat calls (and they are super-slow on this number of files)

In perfect world, the tool will print a hint which is easy to follow. But sorry...
Solution
Add a setting to a command:
  run: !Command
    expect-inotify-limit: 16384

Then we can:

By default: check and print warning if max-user-watches less than expected
When some user-level setting is enabled run sudo sysctl fs.inotify.max_user_watches=16384
In vagga-docker tweak sysctl by default

In General
The bigger questions are:

Should we add system checks to vagga config?
Should we help user to tune the system to suit project? Or maybe it must be done in the shell script in the project itself?

My feel is that vagga is a tool for setting up development environments. So we should make them as easy as possible. And more important point is that, if you have discovered that max_user_watches is required for your program to function correctly, you should communicate it as simple as possible to other users.
The downsides are obviously: more code and config to learn. And sometimes maybe non-obvious attack vectors (i.e. making max_user_watches enormous)
/cc @warik, @popravich, @anti-social, @alxpy

We can set reasonable limit for expect-inotify-limit setting.

Should we add system checks to vagga config?

This sounds reasonable.

Should we help user to tune the system to suit project?

I think this should be done in context of point 1, ie vagga can check system settings and report
what is wrong providing either http link to FAQ page explaining problem and fix or showing
steps need to be run to fix/change setting.


Only set target-specific CC, CXX, and AR for cross-compile

Checking for "eabi" in $TARGET causes failure to build natively on ARM systems.

@bors-servo: r+

:pushpin: Commit 447e509 has been approved by jdm

:hourglass: Testing commit 447e509 with merge 5427341...

:broken_heart: Test failed - travis

@bors-servo: retry

exploration! experimentation! the quest for knowledge!

:hourglass: Testing commit 447e509 with merge 1415114...

:sunny: Test successful - travis


Indexing and slicing of peekables

Re: Issue #4, this PR adds indexing and slicing support for peekables.
As described in the issue, you can think of this as analogous to calling list() on the remaining items in the iterable and indexing/slicing the result. The peekable will not be advanced, however.
You can even index and slice infinite iterables:
>>> from itertools import count
... from more_itertools import peekable
...
... p = peekable(count())
... p[10:21:2]
0: [10, 12, 14, 16, 18, 20]
>>> next(p)
0
>>> next(p)
1

Not sure whether it actually saves any overhead, but islice does look much nicer!

In general, staying out of the interpreter loop helps. Using itertools primitives help a LOT, because they turn straight into C stack frames.

Let me know if you want any more opinions on this. I don't see any showstoppers. It'll be exciting to have a well-maintained version of this instead of a bunch of copy pasta on Stack Overflow.
I'm not sure what the right decision is about the datatype to return from __getitem__(slice). Lists are convenient but use a ptr's worth of mem for every item, while https://docs.python.org/2/reference/datamodel.html?highlight=getitem#object.getitem suggests slices should be "of the same type" as the parent container but may be inapplicable since it's talking about sequences. Having typed that, I find I've just argued myself into returning an islice, since I bet it performs faster and smaller and it can be transformed into a list by the caller if needed. Whatever you decide, I trust your judgment.

I almost buy the argument for returning an islice iterable, but (a) I'm using peekable in the first place because I don't want my iterator-y to be so iterator-y; I asked for a slice and now I have to iterate over it to materialize it? (b) I've never encountered something that I've used slice notation on that returned an iterator.
So I think I'll stick with list and appreciate your trust of my judgement. I like this addition, and am pleased to solve the oldest open issue.

Wait, I checked what range does, and you're right - this should return an iterator. I was so sure of myself 5 minutes ago.

Wait, scratch that again - if this returns an islice of self._cache, then self._cache could change by the time you finish iterating over it.
Back to list, and I'm merging before I change my mind again.

Ha! Sounds good. We can always flop around more before the next release.


fix: configuring resource 'local_file kubeconfig' as sensitive content

PR o'clock
Description
The problem occurs when you run the pipeline in an CI tool, the secret of kubectl appears in the output, this PR hide the content of the file to be taken as a secret.
Checklist

[x] CI tests are passing
[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

this file is not sensitive


200806_540P

Summary
Provide a general description of the code changes in your pull
request... were there any bugs you had fixed? If so, mention them. If
these bugs have open GitHub issues, be sure to tag them here as well,
to keep the conversation linked together.
Other Information
If there's anything else that's important and relevant to your pull
request, mention that information here.
Thanks for contributing to the Watson Developer Cloud!

 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.MBA seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@usr065794 can you provide some info for the motivation behind these changes? I am not familiar with the .ibm-project config file.

I don't have any idea what these changes are intended to do.  You'll need to provide some explanation for these changes or we will probably just close this PR.

Given there doesn't seem to be much context here, I am going to close this PR. If you'd like to discuss any proposed changes to the SDK, please feel free to open an issue so that we can get a better idea of what you'd like to change.


setRouteLeaveHook and 4.0's NavigationPrompt doesn't work on iOS Safari at all

Version
3.0.0 + 4.0.0
For 3.0.0, I used this:
   componentDidMount(){
    this.props.router.setRouteLeaveHook(
      this.props.route,
      this.routerWillLeave
    )

    routerWillLeave(nextLocation){
      return 'Are you sure you want to leave?';
    }

For 4.0.0, I simply tested with the example in https://react-router.now.sh/
I tested on iOS 10, with Chrome, Firefox, UC Browser, Opera Mini all working well.
The reason is most likely that Safari doesn't support beforeunload and pagehide is used instead.
https://developer.apple.com/library/content/documentation/AppleApplications/Reference/SafariWebContent/HandlingEvents/HandlingEvents.html
However, it's worst when I used it anyway on my 3.0.0 react SPA with browserHistory or hashHistory, as trying to navigate away by clicking links or back button not only doesn't trigger a leave prompt, the back button and any links on the page becomes malfunctioned as clicking on it has no effect at all, just like being trapped in those spam sites that forces you to stay.
I wonder what could be the solution? Would it be better to have something as follows implemented?
if ('onpagehide' in window) {
//if(window.onpagehide || window.onpagehide === null){
   window.addEventListener('pagehide', exitFunction, false);
} else {
   window.addEventListener('unload', exitFunction, false);
}

Source:
http://stackoverflow.com/questions/6906146/how-to-detect-browser-support-for-pageshow-and-pagehide

This is something that would need to be supported on history most likely.

@kdenz This is something that would need to be fixed in the history library. Let's follow up in https://github.com/mjackson/history/issues/406

Ah, I just realized that this issue is talking about beforeunload. We don't actually currently support beforeunload in 4.x due to inconsistencies between the ways that browsers handle it. If you'd like support for it, I'd suggest you open a new issue in the history lib. As for now, it works as intended.


Use tput reset to clear screen.

Use tput reset instead of clear so that scrollback is cleared in
addition to the currently visible portion of the terminal.
(Should this fallback to clear if tput reset fails? tput reset is mandated by POSIX and seems to be available on basically everything even remotely unixy (except Android), so it probably isn't worth it.)

Thanks for the PR!
I did some research, and I think tput reset is more applicable here than clear. I used clear mainly because I didn't know tput reset is a better fit for this. Going to accept this and defer worrying about tput not being present for now.


DPVP: Investigate use of one-shot timers

DynamicPVP 4.2.20 calls timer.Once() in several places without calling Destroy(). This seemed to cause a lot of heap churn in Zone Manager, so may be worth fixing here?

May not need to worry about this for now - per Lorenzo:

timer.Every  need to be destroyed.   The other will auto destroy when they expire.  (after execution or repeated execution).


Adjust website graphics to make clear this is an expanded version of a preexisting generator and library

Right now it will look exactly the same, which is not ideal - we should make it clear that most of the credit still goes to spatial illusions of course

TODO:

Logo with link
Favicon
Disclaimer

TODO:

Logo with link
Favicon
Disclaimer


TSP100 Print error: Checked block is not available for this printer

Hello,
I encountered an error while using print, portDiscovery() and connect() are both working. Any idea what is causing this?

@craidencool13 Also experiencing this — did you find a reason/solution?


Deploying

I am planning to merge Smart Home with Google IoTCore and Firestore,
Can I ask your professional opinion for the architecture ?
In [Case 3], I've lost my way, I am so poor for that.
Is there any suggested method for in "[Case 3]  [Some function or send a request to my server]" ?
My device already has pun/sub MQTT function for Google IoTCore.
Thank you for your noble opinion :)
-Device has Sub/Pub Google IoTCore MQTT(JWT) for GoogleIoTCore.
-MyServer uses Firestore for storing users and devices and its states.
[Case 1]
Control with,
* Google Home App
* Google Assistant ------------>  MyServer Webhook (Already in the savior fleker example)
   .                          -> Additional function to update Firestore
                              -> Firestore triggered to update the
                                     Google IOT Core Device config.
                               -> Google IoT core pub MQTT to Control Device
[Case 2]
Control with,
* MyServerPage   -------------> execute smart-home-api
  .                          -> Additional function to update Firestore
                              -> Firestore triggered to update the
                                     Google IOT Core Device config.
                               -> Google IoT core pub MQTT to Control Device
[Case 3]
Control at,
* My Device  ------>   <Some function or send a request to my server>

   to run above [Case 2] sequences for update Google Homegraph and
    Update my server and IoTCore states without controling the device.

This is great reference to me,

https://cloud.google.com/community/tutorials/cloud-iot-firestore-config

Thank you Fleker,

"Send telemetry data to the Cloud" means pub message to IoTCore ?
And deploy firestore cloud function to Google IoT core pub/sub? right?

It only update IoTCore device status, config and DB which is not connected to Homegraph API.
How can I updates the device status on Google Home App(Mobile) .
Can I deploy some firestore cloud function to update homegraph api status, sync?

Thank you, Fleker :)


Add Ozone Chain and Lobster Swap

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Blockchain Submissions:

[x] At least one AMM on your chain with a daily volume over $100k. Do you include it in your PR?
https://lobsterswap.com/

[x] Your chain logo
https://ozonechain.io/static/images/ozone-256x256-bm.png

[x] Your native token logo
https://ozonechain.io/static/images/ozone-256x256-bm.png

[x] Block explorer that could open account and transactions pages
https://ozonescan.io/

[x] High-performance RPC endpoint. Raise the issue in this repo or reach out our team at (https://discord.gg/SeP2KEacBv) with access details.
https://node1.ozonechain.io
https://node2.ozonechain.io
https://node3.ozonechain.io
https://node4.ozonechain.io

New AMM Submissions:
We'll need the following to integrate your AMM:

[x] Do you have a daily trading volume over $20k?

[x] Does "type" at AMM config match your code base?

[x] Your AMM logo. Please add a link to your logo or attach it to a new issue in this repo.
Lobster Swap Logo : https://lobsterswap.com/assets/img-lobsterswap/lobsterswap-brandmark-512x512.png

Please @evahteev review


DNS Challenge default and token save.

I'm running this container on an UNRAID server. It works great, i've been using it for years. My question is, is there a way to set cloudflare as the default DNS challenge provider? And is there a way to save my api key so I don't have to put it in every time?

But I think it already works like that, with npm, you save the ssl certificate settings as dns challenge verification method and save your cloudflare api key, it is then saved and you don't have to re-enter it again!

It seems to have saved for that certificate. Which is fine since it’s a wildcard cert. but if I add a new host and want to issue another cert I have to check dns challenge. Then I select cloudflare and the default token is there. I have to enter my token in everytime I want fetch a new cert.


feat: bump version 7.0.0-dev

Platforms affected
none
Motivation and Context

Prepare to merge in some breaking changes.
Description

Bump version to 7.0.0-dev
Testing

Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

Codecov Report

Merging #258 (ebf65e8) into master (ce5be26) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
- Coverage   83.62%   83.55%   -0.07%
==========================================
  Files          14       12       -2
  Lines         519      517       -2
==========================================
- Hits          434      432       -2
  Misses         85       85

Impacted Files
Coverage Δ

test/test-platform-modules/exec.js

test/test-platform-modules/platform.js

Continue to review full report at Codecov.

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

What did you think with Prepare to merge in some breaking changes.? Any specific Issue or PR already merged or not yet since last year?
And here to ask if a release and publish would be possible?
Merges for #242 and #243 are really useful but still not available since v6.1.0 two years ago.
Both cordova-android and cordova-ios still rely on "cordova-js": "^6.1.0" even on their current dev branches.
So should it be a v6 or v7 release I don't know, but it would be appreciated ; with following version update, builds and publish for both android and ios please, so that we can take advantage of these improvements and try to prevent some app crashes.


Conventions

@jyemin @rozza @craiggwilson
This should cover most of the conventions stuff we've discussed so far.  The biggest outstanding pieces are probably generics and polymorphism support.

bump

Closing this, as we're not ready to pull the trigger on this work.


Site-specific config missing page title option

The content page title looks to be hardcoded to "OxRSE Training" in components/Header.tsx (https://github.com/steve-crouch/gutenberg/blob/main/components/Header.tsx). Could this be made a configurable option?

we were just saying that this morning :) Agree it should be configurable. We also were talking about putting a logo somewhere obvious on each page (ie.. either oxford of southampton logo), and in the longer term a configurable theme so we can style things differently. At the moment our two pages look very similar!


[# 1870] Override MSSQLDbmsDialect to use varchar(max) for String

Description
Overridden MSSQLDbmsDialect String equivalent in sql type to varchar(max).
Related Issue
Fixes : #1870
Motivation and Context
SQL Server has default length of 30 for varchar which will fall short for most of the users for doing string operations like CONCAT , CAST_STRING etc.

I have followed the same way its being done in DB2DbmsDialect . My personal Preference would be to make this behaviour change in other constructor as well like below to make it consistent until user specifically overrides String type with help of this constructor :

public DB2DbmsDialect(Map<Class<?>, String> childSqlTypes) {
        Map<Class<?>, String> types = getSqlTypes();
        if(childSqlTypes!=null)
           types.putAll(childSqlTypes)
        super(types);
    }

I don't see any dialect specific test cases in existing code. So, I have not added the same. Also I have verified that existing test cases pass by using a sql server instance and below command:

mvn test -Pmssql,hibernate-5.6,deltaspike-1.9,spring-data-2.7.x -Djdbc.password=<my-local-pass>

@beikov let me know if we want to add an explicit TC for this. Currently I believe it is being covered with other Test Cases.

Thanks!


Improve doctor output on incomplete Visual Studio installation

Flutter doctor warns if the user is updating Visual Studio or has paused a Visual Studio update. The current output recommends reinstalling Visual Studio. This is unnecessary as the user should be able to use the Visual Studio Installer app to complete the installation:

This message is covered by this existing test:
https://github.com/flutter/flutter/blob/229b74d987720a0437039b6e44e13f20fb94ad51/packages/flutter_tools/test/general.shard/windows/visual_studio_validator_test.dart#L68-L81
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Ahh, I see now the context of your question. In this case, I would say copy pasting the relevant error message to your test, and either renaming the existing test or creating a new one with a name that makes it clear this is an expectation will help document this behavior and prevent regression from clueless tool maintainers :)

I updated the tests, please take another look @andrewkolos @christopherfujino. Thanks!

The customer_testing failure is actively being worked on upstream. See discord #tree-status for more context.

Filed b/298053054 and manually overrode Google testing

@christopherfujino Thanks! Looks like Google testing is now hitting a second issue, filed b/298076936

:'(


New Website Logo: TypeScript CodeDOM

Please fill the following out:

Name: TypeScript CodeDOM
Homepage url: https://github.com/zijianhuang/webapiclientgen/wiki/TypeScript-CodeDOM
Brand Guidelines/Licensing: MIT
Logo: https://raw.githubusercontent.com/zijianhuang/webapiclientgen/master/Doc/icons/TypeScriptCodeDOM.svg

Hey folks -we're looking at a new website and it's likely that the friends of TypeScript page isn't going to make it to the new site. This is mainly because close to everyone is a friend of TypeScript now and listing folks kinda feels a bit un-inclusive to everyone else.
So I'm going to close this issue - thanks for posting to apply


support integer-1.1 and ndarray-1.1

fixes an issue with integer where IntegerType instead of cls was used during class instantiation which resulted in objects being created with the incorrect version. Prior code (that only had one integer version, 1.0) did not suffer from issues because of this bug.
fixes #1245 by removing the use of super in NDArrayType and adding an exception to check for incompatible use of super in classes of type ExtensionTypeMeta.
This should be merged with asdf-standard PR https://github.com/asdf-format/asdf-standard/pull/350

Now that asdf-format/asdf-standard#350 is merged can you Remove the skips added in #1251 and @1252?

It looks like this will require either a release of asdf-standard or refactoring test_builtin_extension (and readding one of the xfails for defunit).


Bug: declaration for multiple entry points are included inside an index folder

Describe the bug
When bundling with rollup using multiple entry points and preserveModules output option the declaration file of the entry point file is located inside an index folder.
This breaks resolution of the import / export reference which are based on the parent folder:
dist/a
dist/a/index.js
dist/a/ComponentA
dist/a/ComponentA/ComponentA
dist/a/ComponentA/ComponentA/index.js
dist/a/ComponentA/ComponentA/index.d.ts
dist/a/index
dist/a/index/index.d.ts <---- should be dist/a/index.d.ts

dist/b
dist/b/ComponentB
dist/b/ComponentB/ComponentB
dist/b/ComponentB/ComponentB/index.js
dist/b/ComponentB/ComponentB/index.d.ts
dist/b/index.js
dist/b/index
dist/b/index/index.d.ts <---- should be dist/b/index.d.ts

This issues seems only affect the entry points. Other declaration files are located correctly alongside their js counterpart.
Reproduction
https://github.com/marcalexiei/unplugin-isolated-decl-bug-index
System Info
System:
    OS: macOS 14.6.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 15.88 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.17.0 - ~/.nvm/versions/node/v20.17.0/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v20.17.0/bin/npm

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Fixed via c724430a3700b357c7133d9efe9741646071b8cf

Thank you very much for the quick fix and release!
I just tried it and it works.
I was tackling with the code to try to fix the issue myself and I was about to write some unit tests when you resolved the issue.
Would you like I open a PR to add test for this scenario?

Sure, feel free to open!


🗃 CRUD for Social Connections Entity

SocialConnection Entity CRUD

[x] createSocialConnection(socialConnection: SocialConnection)
[x] getSocialConnection(id: string)
[x] updateSocialConnection(id: string, socialConnection: Partial<SocialConnection>)
[x] deleteSocialConnection(id: string)
[x] associateSocialConnectionWithPerson(socialConnectionId: string, personId: string)

Closed by #214


v2.9.16

This is an automated message from CI:
Docker Image for build 3 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1864
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.