CONNECT request with IP address as hostname fails with invalid_url

Description:
If the browser/client is configured to use an explicit HTTPS proxy and the destination is an IP address then envoy fails the connection with a "400 Bad Request" due to a "invalid_url" error.
Logs:
[2020-09-15 20:37:09.669][1608][trace][http] [external/envoy/source/common/http/http1/codec_impl_legacy.cc:632] [C0] onHeadersCompleteBase
[2020-09-15 20:37:09.669][1608][trace][http] [external/envoy/source/common/http/http1/codec_impl_legacy.cc:470] [C0] completed header: key=Host value=<IP_ADDRESS>:4443
[2020-09-15 20:37:09.669][1608][trace][http] [external/envoy/source/common/http/http1/codec_impl_legacy.cc:676] [C0] codec entering upgrade mode for CONNECT request.
[2020-09-15 20:37:09.669][1608][trace][http] [external/envoy/source/common/http/http1/codec_impl_legacy.cc:851] [C0] Server: onHeadersComplete size=4
[2020-09-15 20:37:09.669][1608][debug][http] [external/envoy/source/common/http/filter_manager.cc:773] [C0][S6861130327348177454] Sending local reply with details http1.invalid_url
[2020-09-15 20:37:09.669][1608][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:1255] [C0][S6861130327348177454] closing connection due to connection close header
[2020-09-15 20:37:09.669][1608][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:1311] [C0][S6861130327348177454] encoding headers via codec (end_stream=false):
':status', '400'
'content-length', '11'
'content-type', 'text/plain'
'date', 'Tue, 15 Sep 2020 20:37:09 GMT'
'connection', 'close'

Detail:
I traced it to Url::initialize in url_utility.cc, which uses GURL.  The problem is that the GURL parser decodes the "hostname:port" string as a scheme (as also stated on url_utility.cc:65) and that GURL performs a sanity check on the "scheme" to make sure the first character is an alpha.  See IsSchemeFirstChar() in google URL url/url_canon_etc.cc:95.  In other words an IP address would fail the scheme sanity check.

Hmm this does seem like something we should fix somehow but not sure of the RFC details. cc @yanavlasov @dio @alyssawilk who have been looking at similar things recently.

Yes, it looks like a bug. This code is going to be changed very soon. We will keep this bug open to make sure we look into this case.

IPv4 was fixed a ways back.  IPv6 was fixed by #13798 so you should be good to go!

Thanks, Alyssa!
On Thu, 29 Oct 2020 at 22.27 alyssawilk<EMAIL_ADDRESS>wrote:

IPv4 was fixed a ways back. IPv6 was fixed by #13798
https://github.com/envoyproxy/envoy/pull/13798 so you should be good to
go!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/issues/13114#issuecomment-718828893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAR3QDZR3VHSN4IJ4DI44LSNGCVZANCNFSM4RNXWYWQ
.


[LIBCLOUD-896] Added ENA support to images in EC2 compute driver.

Added ENA support to images in EC2 compute driver.
Description
Please refer to: https://issues.apache.org/jira/browse/LIBCLOUD-896
Bumping the EC2 API version was necessary to support the new ENA options. I performed some spot checking of the test fixtures against the API reference docs and didn't notice any breaking changes. If there are some comprehensive functional tests to prove this I am willing to run them.
Status
done, ready for review
Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

I'd like to get some more integration testing done first before we merge this, a lot of people leverage this driver.
Thanks for the contribution, I'll send a mail on the developer mailing list.

I tested create_node, create_volume, reboot_node, destroy_node and destroy_volume using the EC2NodeDriver and those still worked fine with the updated API version :+1:

ok, :+1:


Add e2e tests

Add end to end tests that can be run in circleci workflow (and in container on local machine).
At the least want to deploy adapter, point it at Service Bus Queue and App Insights instance that has already been provisioned for this e2e test (the test would not be responsible for creating these resources in azure but should pass those values via environment variables).  The test would:

call validate the adapter is installed properly (kubectl get --raw "/apis/custom.metrics.k8s.io/v1beta1" & kubectl get --raw "/apis/external.metrics.k8s.io/v1beta1")
install ExternalMetric CRD configured for Service Bus Queue and then make request to kubectl get --raw "/apis/external.metrics.k8s.io/v1beta1/namespaces/test/queuemessages to get value and  compare to az servicebus queue show --resource-group sb-external-example --namespace-name $SERVICEBUS_NS --name externalq -o json | jq .messageCount
install CustomMetric CRD configured for App Insights and then make request to kubectl get --raw "/apis/custom.metrics.k8s.io/v1beta1/namespaces/test/pods/*/custom-metric" and make sure value is returning.

Optionally could test the whole HPA scaling flow but this much more complicated.  The above should give some basic smoke tests to make sure things are working generally.
I started the a possible e2e script here: https://github.com/Azure/azure-k8s-metrics-adapter/tree/e2e.  Using the samples (service bus and appinsights) could help guide the tests

Closed with pr #58


Add "Grid-Capable" enum object

Create an "GridInterfaceType" enum object and attach to ProdInverter and ProdEnergyStorageSystem
Enum list:
Grid-Following
Islanding Capable
Off-Grid Only
(Maybe something else?)

This is done.

This is done with GridInterfaceType unit, element and is included in ProdInverter and ProdEnergyStorageSystem.


Docs: Correct benchmark result + joseph-md5

I noticed that in your README, there is a benchmark testing, but the joseph's md5 is not included. I wonder which is faster so I tried to compare it.
Docs for joseph-md5: https://www.myersdaily.org/joseph/javascript/md5-text.html
Benchmark 1
Benchmark: https://measurethat.net/Benchmarks/Show/29171/0/md5-performance-comparison-v2
Result:

Benchmark 2
https://measurethat.net/Benchmarks/Show/29172/0/md5-performance-comparison-long-text-v2
Result:

Issue with the current benchmark in README

jsperf.app is a bit unreliable on short time self execution. It should be tested with a long input.
a longer string should be used to see the actual speed.
One of the tests is async. deferred.resolve is required.

Correct jsperf.app with deferred.resolve + joseph-md5
Benchmark: https://jsperf.app/jonuhi/6

Now it comes out the result that aligns with measurethat.net.
Summary
Your library is great ! I believe it is the fastest MD5 library for sync md5 function.
I will submit a PR for updating the README :)

https://measurethat.net/Benchmarks/Show/29171/0/md5-performance-comparison-v2
https://measurethat.net/Benchmarks/Show/29172/0/md5-performance-comparison-long-text-v2
https://jsperf.app/jonuhi/6
https://measurethat.net/Benchmarks/Show/29173/0/md5-performance-comparison-very-long-text

Note: I guess if you convert js-md5 to using low-level WebAssembly coding, it will be much faster.

https://svn.?????????.com/bot/sub/cbe38717a6d972172cbf19f1f6696e2b

https://svn.?????????.com/bot/sub/cbe38717a6d972172cbf19f1f6696e2b


Psalm MethodSignatureMismatch error when extending ActiveQuery

I get Psalm MethodSignatureMismatch method has fewer parameters than parent method error.
What steps will reproduce the problem?
<?php
class TestQuery extends yii\db\ActiveQuery {}

What is the expected result?
No error
What do you get instead?
ERROR: MethodSignatureMismatch - src/TestQuery.php:2:7 - Method TestQuery::one has fewer parameters than parent method yii\db\ActiveQuery::one
Additional info
Don't know if something could be done with ActiveQuery or is this Psalm error.

Q
A

Yii version
2.0.38

PHP version
7.4.5

Operating system
MacOS 10.15.7

ActiveQuery::one() accepts an argument - $db (default null) so if you extends this method the signatures must match.

How can the signature be different if I don't override this method? And even if I copy this method from ActiveQuery I still get the same error.

Oh, sorry, I thought that the given code is just for a general idea but it's the whole thing. So it's empty and still Psalm states error? Looks like Psalm issue TBH. But at the same time when I add the same case at https://psalm.dev/r/9aad4f413f (at least I hope it's the same) it does not show that error. Is your Psalm up-to-date?

Everything is up-to-date. I also thaught this is Psalm issue, but couldn't figure out what's causing this to make sample code for bug report. If I extend yii\db\Query directly instead of yii\db\ActiveQuery then there is no error.

Finally figured it out. ActiveRelationTrait was causing this error.


In UIViewLazyList, make the UITableView's background color clear

This is a partial revert of PR #2146; we still want the individual cells to have non-clear backgrounds for performance reasons, but the underlying table should be clear

[x] CHANGELOG.md's "Unreleased" section has been updated, if applicable.

@dellisd On the LazyColumn it should do just that


feat: user info remote content

:tada: This PR is included in version 0.81.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


The export tab will be managed by three.js

The export tab will be managed by three.js, similar to asset-viewer but woven into the app, it can also import normal meshes for rendering.

Babylon.js functionality will be limited to model tab (core, material and gui)


fix: expose token on unauthenticated turbo factory PE-6569

:tada: This PR is included in version 1.10.1-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.10.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PaginationPlugin does not update the store and the page data for entities that were already present in the store with outdated state

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior

Enter the view with the paginator with an outdated entity in the store. Call the clearCache. The page is loaded from the server with an updated entity state but it is not reflected on the page data returned by the paginator plugin.
Expected behavior

When call clearCache and the paginator request new data from the server, the updated data should be added to the store and returned by the plugin in the page data.
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/angular-1mfw2x

Click on Add outdated user
Navigate to Users

Should list

ID
Name

1
A

2
B

3
C

But it does not. Instead, it lists

ID
Name

1
Outdated User

2
B

3
C

What is the motivation / use case for changing the behavior?
The behavior seems unexpected since the updated data is got from the server but it does not go to the page.
Environment

Angular version: ^7.0.1

Browser:
- [x ] Chrome (desktop) version 85
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: 10.22.0
- Platform:  Linux

Others:

It will not work if you update the store manually. The server is the single source of truth. Imagine that there are some filters or sorting applied, you can't rely on the client data. You need to fetch the new data from the server.

@NetanelBasal  In this case, the manual update of the store is just to simulate an outdated entity on the store (Imagine the entity being updated on the server in the meantime). When you navigate to the users the data, it is being fetched from the server but it does not get updated on the page.
Digging deep in the code, I've noticed that when we call clearCache for the first time it is not actually cleared, this is causing the outdated entity to continue on the store when the paginator loads the data from the server. Later on, when the paginator gets the data from the server, it tries to do a store.add on the fetched entities, but the add method does not update an entity that is already in the store, leaving the outdated state of that entity in the store. Finally, the plugin produces the page based on the data present in the store, not the actually fetched data.

Do you want to create a PR that fixed this?

Do you want to create a PR that fixed this?

I can try


Latest release on npm is @nuxtseo/module 2.0.0-beta.15

Describe the bug
Look at the subject, it seems new releases are not pushed to npm.
Reproduction
No response
System / Nuxt Info
No response

Thanks for flagging this, I did not realize. Should be in sync now.


fix: make deposit go to top up selector page

🐝 PR preview in Swarm
Preview URL: https://bah5acgzawcc4mts4rqtv4h7pkrdzvcytshngzctorglqikcgaadfqmfoe7ma.bzz.link
Swarm Hash: b085c64e5c8c275e1fef54479a8b1391da6c8a6e899704284600065830ae27d8
Commit Hash: 1ec9f34b9fe4605271b82de7fd3d354e582d8591
Commit Message: fix: make deposit go to top up selector page


add link to root work record

Manifests should have a link to the work record they correspond to

closed in favor of #80


docs: Fix mention of ExtiInput

The struct is called ExtiInput, not ExtiButton.

@Dirbaio wow, that was quick! :)


Measure performance difference against null buffer

Do some tests comparing performance of ungated writes to a null-buffer against writes to a disabled ConditionalOStream.

[x] Ungated std::cout*
[x] Ungated std::ofstream*
[x] Ungated std::ostringstream*
[x] Ungated null-buffered stream
[x] Disabled ConditionalOStream

* Level set with streams of a few standard types

Done, and presented on the page


Update mit-scheme to 10.1.11

Thank you!

I believe this should rebuild and deploy on Dockerhub, so we should be solid, no "release" needed. Thanks @TylerLubeck !

Sure thing, thank you for maintaining this!


does RNA long reads suit for correction?

We've never tested fmlrc on RNA data.  However, given a set of long reads and short reads that are both RNA, I have no reason to believe it wouldn't work.  Of course, complications from expression of different transcripts might impact correction.  It's hard to say for sure since we've never tested it.


Due to security alerts, manually update third party dependencies

Signed-off-by: Alexey Roytman<EMAIL_ADDRESS>

/cherry-pick branch=releases/0.6.4

/cherry-pick branch=releases/0.6.4


Discussion about breaking changes and versioning

Hi,
I'd like to open a discussion about a little friction when updating MaterialX library: breaking changes :)
This is a bit touchy because I know that from a library's maintainers' point of view, it's very tempting to evolve the API to keep it as clean as possible, at the risk of breaking things for user code. But on the other hand, every time I update MaterialX in our build system, I have to spend a fair bit of time to fix things. Sometimes it was quite simple, some other times a lot trickier. And most of the time at places where I didn't expect it :)
Anyway, MaterialX being quite new, I understand that the API is not stable enough yet, and trying to make it evolve while trying to keep it backward compatible could quickly become a maintenance nightmare for you. But I feel there must be a sort of middle ground that would allow us to update without too many headaches, and you to make changes without dritying things too much, thus this discussion.
For breaking changes in the C++ API, a solution could be to flag things as deprecated while keeping them, and add the new things in parallel. And once in a while, cleanup the deprecated APIs. Ideally, the cleanup (e.g. breaking change) would happen on minor version update (e.g. from 1.38.5 to 1.39.0 for instance) This way, patch version update could still introduce new functionality, but it would guarantee that it wouldn't break user code: we would be able to update a lot easier, get new functionality and fixes without problem, and we would have time to convert the deprecated APIs that we're using before updating to a new minor version.
The second kind of breaking changes are ones related to data: for instance between the 1.38.3 and the 1.38.5, the directory structure of libraries/genosl changed, and the osl headers mx_funcs.h moved around. This broke compilation of generated OSL code because the include paths were no longer set correctly, so we had to patch our compilation options to account for that. Those kind of changes are a bit more difficult to handle I think because there is no clean deprecation mechanism for those... Maybe at least defer those changes to a minor version update? That way patch version update would guarantee that we could update without problem?
Sorry for the length :)
Any thoughts, ideas?
Regards,
Damien.

A patch identifier was considered before so maybe something like this could work (?):

If there is any non-backwards compatible change or a data representation change then the patch level is bumped. It may or may not be possible to backwards compatible all the time. Deprecated API can be marked as such.

So next patch would be <IP_ADDRESS>.

Patch level is not part of the version.
Issues I've run into the past is that there was no indication in the data file that within 2 build
releases or within 2 minor releases anything had changed. A suggestion from before was to
add a patch attribute to each document. I'm not sure but  build.patch I think would be make
it non-ambiguous. There would be an include directive and library versioning support as well.

At build version and minor release time it is allowed to perform deprecation and breakages.
There hasn't been a major version change yet, but I assume this could be for non-backwards
compatible data model changes which cannot be upgraded (at all or not fully automated)

I still am leaning towards having build / patch number be considered of the document upgrade logic for data files as it would remove headaches (and disgruntled integration teams :)) of not knowing if the data format has changed or not.

As for tagging in the repo, I've heard that folks want to create specific branches for releases,
but I think I'd still prefer to go with the single branch + tags. A question is do we tag patches ? -- I think
it would be useful.

@bernardkwok That's an interesting idea, though I was imagining the reverse of your proposal, where a patch update denotes a set of fixes and improvements that have no impact on the MaterialX API or the layout of its data libraries.  This would align with the idea that it's a "smaller" update than we denote with the existing build version number.  This might be used, for example, to put out a set of minor fixes to an existing MaterialX release, and the use of the patch version would convey the idea that developers can upgrade to this version without any changes to their own code.

@jstone-lucasfilm My thoughts exactly!

@jstone-lucasfilm, @ix-dcourtois  I didn't read carefully enough that your looking for the lowest level for pushing out "non-breaking" changes. Perhaps I'm thinking more of file-format bumps where everything breaking must have a version bump, so I would still put forth the suggestion that we formally add "patch" as part the document as required.
I'm wondering if it's good to explicitly lay out what is considered "breaking", how long old behaviour needs to stick around,
and what if there is no way to keep both.
Examples:

API compatibility: or something like an signature change, we keep both signature working, mark the old signature as deprecated for the patch. During the next Nth minor release we are allowed to remove it where N = some number of releases.or the next major release?  Signature could be C++, Python, JS etc.

Data Model compatibility: For data model upgrades (including signature changes for nodedefs), I assume as long as there is an upgrade path and does not remove anything then these are not "breaking" and can be part of a patch. e.g. Adding new nodedefs is not breaking.

Logic compatibility: For behaviour / logic changes like the library search change, if this cannot have a fallback / alternative then it must wait until the next minor release, otherwise it can be part of a patch. e.g. Is changing shading model
node behaviour breaking, or is changing the OSL matrix / vec4  logic breaking (both of which occurred for 1.38.5) ?

Build logic compatibility: If dependencies or build configuration changes then this is breaking.

Perhaps I'm overthinking this, but in practice breaking and non-breaking changes can inter-mix so pushing breaking changes
to a minor release could hold up that and dependent changes for months as there is no branching for versions.
Please let me know if I'm too noisy on this the details of this :)

It's like trying to get one number or dotted stream of numbers to do the job of SemVar for API, and simultaneously SemVar for what can be reified in a material graph. The thing about SemVar and any dotted number scheme is there's an implication of strict ordering, such that a.b.c >= d.e.f iff a >= d & b >= e & c >= f.
It's not clear to me reading this thread that there is a strict ordering possible when api and data model compatibility are not also strictly ordered. In other words, data model could increment many times without incrementing api, and vice versa. I think the only thing that makes sense is something like 1.38.6_d1.38.6 where d tags the data model (so one doesn't forget which is which every time you look at a version ;) ) and then over time the two numbers increment independently. The suggestion of 1.38.6_d1.38.6 comes from the idea of establishing an epoch, the zero-day when both API and data number were both thought to correspond one to one.
If any one has a better analysis of the mathiness of it all that doesn't require six numbers, I absolutely stand to be schooled :)

Thanks for starting this discussion, @ix-dcourtois, and we've clarified the rules for changes allowed in version upgrades in https://github.com/AcademySoftwareFoundation/MaterialX/pull/1664.


[Bug] Adding a toolbar causes error output in Properties window [sf#207]

Reported by brendansimon on 2017-05-10 11:10 UTC
Just adding a toolbar causes error output in Properties window.  See attached pic.

Updated by dschwertberger on 2017-05-10 19:17 UTC

status: open --> accepted
assigned_to: Dietmar Schwertberger

Commented by dschwertberger on 2017-05-10 19:17 UTC
Yes, I've noticed such output once when working on Mac OS, but could not cause this later again.
I've never seen it on Windows. Thanks for indicating how to reproduce it. I will check.

Updated by dschwertberger on 2017-05-10 21:02 UTC

status: accepted --> closed

Commented by dschwertberger on 2017-05-10 21:02 UTC
Fixed with commit 2325.
This was a secondary effect of the unused config "box_label" which I have removed completely now.


Support Kibana filter expression in Vega code

Hello Team
Firstly, thanks for the awesome widget. This adds great flexibility in creating custom graphs with custom data sources. One clarification we have is, apart from the @timestamp : %timefilter%, is there a possibility to add dashboard context similar to transform_vis - DASHBOARD_CONTEXT? Currently, if we add a vega visual to an existing dashboard, the data is not filtered based on the other graphs filters. Is there any way to achieve the same?

@evanceheally2 thanks for your feedback!  Yes, I do plan to add this capability shortly - need to figure out some internal API to make this work, but should be doable.

Thank you so much Nyurik for the update. The problem we are facing with transform_vis is that we need to write a lot of js to create a graph. This makes the widget non usable to share with clients to create runtime dashboards. Vega is great but the non availability of filter option makes that too non usable for our scenario.
Any work arounds/ other plugins that you could point us to that could provide us vega like functionality with kibana filter expressions?

@evanceheally2 - I do plan to add it shortly, stay tuned :)  If you can help with JS development, even better!
Setting requiresSearch: true in vega_vis.vis_type.js enables the search box, but I haven't yet figured out how to get the string value of that search box, as well as the index name. Adding it to the code afterwards should be very easy. Maybe @spalger or @nreese know?

I would love to help with JS development but unfortunately, I am more into data science and databases and my knowledge wrt JS is much limited. But I can point to one of our team members who can help in JS @rajeeshk... I would eventually try to start helping once I gain enough knowledge on the same.

done in release v0.2.0


error: wavebin index out of range. Comes from stage4.read_ld_coefs

https://github.com/radicamc/supreme-spoon/blob/07db39a7aa20590b1ff68ecdcdbadcc5bb1013c2/supreme_spoon/fit_lightcurves.py#L219

I have not been able to reproduce this. Do you have an error readout so I can see more specifically where this is stemming from?


User in Mainland China cannot use even got VPN ( enable to watch on youtube and use google searching engine)

I found that even I got a VPN still cannot use it when I'm in mainland China. It mentions "Error! You need to have internet connection to use pyTranscriber!". And I know that somehow it was blocked to get to google, but I can use VPN to search in google engine, even get to youtube. Please give me a favor for checking this issue.  I'm a user who really loves this app and I really need this function for my video subtitles. 😭

+1


accept: Initialize addrlen with size of sockaddr struct

accept(2):

The addrlen argument is a value-result argument: the caller must initialize it
to contain the size (in bytes) of the structure pointed to by addr; on return it
will contain the actual size of the peer address.

Previously, Socket.accept raised an exception because the sockaddr struct would not be initialized.

So I would assume it's that later on functions that interact with the invalid sockaddr crash yes?

Sorry, I was imprecise: the function lean_socket_accept panics here because sa_family remains uninitialized.
I'll push a commit for the other functions shortly!


UofT-DSI | algorithms_and_data_structures - assignment_1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
added answers to assignment-1
What did you learn from the changes you have made?
use of BFS and inorder traversal
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
depth-first search
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
with example trees provided and new examples created
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

A1 complete


Create release 0.2 alpha

An alpha release implements the planned functionality with minimal testing and documentation.

[ ] Create a new branch for this issue.
[ ] For each issue for this milestone:

Select a couple of functionality tasks.
Write class/function headers with doctests to generate code with Copilot. Iterate until commit hooks pass.
Tick off tasks.
Iterate until functionality of that issue implemented. Reference the issue in the commits, but don't close it.

[ ] Squash and merge a PR into the main branch to close this issue only.
[ ] Do a pre-release with tag v0.2-alpha.

Alpha releases just complicate the process.


Custom login

Does anyone has implemented a custom login to authorize users?

Hi @noriega2112 - thanks for the question!
If you're interested, there is some interesting discussion in this thread about adding Firebase authentication: https://github.com/twilio/twilio-video-app-react/issues/187
Near the end of that thread, another github user leaves their solution for how they got things working.
Additionally, our quick deploy feature lets you authorize users with a passcode. Have you tried that? Here is a youtube video to show you how to get started: https://www.youtube.com/watch?v=pSASQcHrmPg
Please let me know if you have any other questions!


Borovlev Petr HW03

And make sure github action passes next time.

There are still issues in the code.

There are still issues in the code.
I think I fixed everything you mentioned above


add webhook handler without http.Serve

I would like to be able to have a webhook handler that has no runtime.
I have added an implementation of such a handler, it can be easily routed to an existing router without creating another listener.

@robotomize Do you think this is going to help with running the webhook handler within the lambda function, which does not really allow you to listen to a port?

I like the idea but not the implementation. Wouldn't want to spawn another Webhook entity with a pretty similar logic. I'd better extend the existing implementation. I'll think about it.

@demget I agree, I sketched out a solution so as not to change anything in the current implementation and configuration. Cool if you make a better and more elegant solution!

Wouldn't want to spawn another Webhook entity with a pretty similar logic. I'd better extend the existing implementation.

Agree, no polling necessary. As comment in #536, for Webhook, as far as AWS lambda is concerned, what we need is merely to use bot.HandleUpdate to do the same internal stuff the library is doing from the webhook to handle a single update.

Can you do what is described here? It is working for me to deploy the code to AWS Lambda with just mapping the lambda request to the Update structure.

Can you do what is described here? It is working for me to deploy the code to AWS Lambda with just mapping the lambda request to the Update structure.

The provided example that goes with the PR is somehow removed later on (maybe the concern was that it was outdated after V3). Anyway,
Would you be so kind as to give a working demo example for the current version please?

The example was removed, but It actually works.
In any case, here is a working example https://github.com/tucnak/telebot/pull/617

Since the provided example above makes more sense to me, closing this PR.


Per-VIRTUAL_HOST settings

I mounted /etc/nginx/vhost.d on my local filesystem to configurure the client_max_body_size. I saw that letsencrypt companion adds a block for validating that the domain actually belongs to me in the default vhost config. I set up my vhost to look like this:
include /etc/nginx/vhost.d/default;
client_max_body_size 10m;

Will the include of the default vhost work here?

Hi, I'd like to keep Github issues for actionable bug reports or feature requests.  Please ask questions in the Q&A Group.
Thanks!


live video request ignore query parameters

Version
Media3 main branch
More version details
implementation("androidx.media3:media3-exoplayer:1.4.1")
implementation("androidx.media3:media3-ui:1.4.1")
implementation("androidx.media3:media3-exoplayer-dash:1.4.1")
implementation("androidx.media3:media3-exoplayer-hls:1.4.1") // hls(m3u8)支持
implementation("androidx.media3:media3-datasource-okhttp:1.4.1") //
implementation("androidx.media3:media3-exoplayer-ima:1.4.1")

Devices that reproduce the issue
Redmi Note 12 Pro android 13
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Yes
Reproduction steps
PlayerActivy:
player.setMediaItems(mediaItems, /* resetPosition= */ !haveStartPosition);
player.setMediaItem(MediaItem.fromUri("https://iot.something.cn:9443/rtp/33030300452007000002_33030300451377000002/hls.m3u8?sign=f9051de85973951fcd3c2decf667103e"));

Expected result
success
Actual result
request:

https://iot.something.cn:9443/rtp/33030300452007000002_33030300451377000002/hls.m3u8?sign=f9051de85973951fcd3c2decf667103e
https://iot.something.cn:9443/rtp/330303004
52007000002_33030300451377000002/hls.m3u8?sign=f9051de85973951fcd3c2decf667103e
https://iot.something.cn:9443/rtp/33030300452007000002_33030300451377000002/2024-10-18/09/51-52_287.ts
throw androidx.media3.datasource.HttpDataSource$InvalidResponseCodeException: Response code: 401
3 lost auth params

Media
Not applicable
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Are the URLs you provided meant to resolve, or are they just placeholders? They don't load for me (I also tried VPNing to China based on the ccTLD).
The report sounds similar to https://github.com/androidx/media/issues/1671 but without access to playlist I can't determine if it is using the QUERYPARAM HLS config (see that issue for more details).
Either:

The playlist is using QUERYPARAM config, and this is an enhancement request for ExoPlayer to support this config.
The playlist is not using QUERYPARAM, so that would need to be fixed before ExoPlayer's lack of support matters.

Please let us know whether this is case (1) or (2). If it's case (1), please provide a copy of the HLS playlist. Please either upload it here or send it to<EMAIL_ADDRESS>with the subject Issue #1812. Please also update this issue to indicate you’ve done this.

Thank you for your attention.
this is hls link ,date limit is 24 hour
https://iot.moshme.cn:9443/rtp/33030300452007000002_33030300451377000002/hls.m3u8?type=user&code=f9051de85973951fcd3c2decf667103e&token=386b54dd-f984-4f06-92c8-6e9d87feb125
this is maybe (1）.because the link must add
type=user&code=f9051de85973951fcd3c2decf667103e&token=386b54dd-f984-4f06-92c8-6e9d87feb125 .
if you need long effective link,please let me know.
Ian Baker @.***> 于2024年10月18日周五 16:34写道：

Are the URLs you provided meant to resolve, or are they just placeholders?
They don't load for me (I also tried VPNing to China based on the ccTLD).
The report sounds similar to #1671
https://github.com/androidx/media/issues/1671 but without access to
playlist I can't determine if it is using the QUERYPARAM HLS config (see
that issue for more details).
Either:

The playlist is using QUERYPARAM config, and this is an enhancement
request for ExoPlayer to support this config.
The playlist is not using QUERYPARAM, so that would need to be
fixed before ExoPlayer's lack of support matters.

Please let us know whether this is case (1) or (2). If it's case (1),
please provide a copy of the HLS playlist. Please either upload it here or
send it to @.*** with the subject Issue #1812.
Please also update this issue to indicate you’ve done this.
—
Reply to this email directly, view it on GitHub
https://github.com/androidx/media/issues/1812#issuecomment-2421833245,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASUYYRV6GN3KVZHKLNBIU7TZ4DB2PAVCNFSM6AAAAABQE6PUOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRRHAZTGMRUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add v2v plugin branch and commit sha in the main html [#422]

Add v2v branch and commit sha in the main html
What was suggested here https://github.com/ManageIQ/miq_v2v_ui_plugin/issues/422#issuecomment-398808617 is applicable to all plugins, and may have to be implemented on a global level, possibly from miq core. While it is a complete solution, it may take a while to implement considering the extra styling required in the About Modal for the scrolling.
So for now what we could do is -  tackle this from our end.
That said, let's go with @priley86 's suggestion to put the hash in the html.
See the screenshot below and checkout the commit sha in the highlighted line -

Fixes https://github.com/ManageIQ/miq_v2v_ui_plugin/issues/422
https://bugzilla.redhat.com/show_bug.cgi?id=1594348

I LOVE this idea!! But... there's a catch... when you pull away the magic rug of internet (meaning when yah don't have an internet connection) this breaks v2v 😭

maybe the def could fail more gracefully when it gets nothing back?  Also we'll have to update this url when the repo name changes... 🤔

@AparnaKarve - i appreciate this one a lot!
I think this is a great feature as you've implemented it, but one concern i'd have is that this would give the commit hash at the time the MIQ server is started (since we are requerying that url each time), correct? Ideally we could persist the hash at the time the appliance image is built some place...
I'm fine w/ merging this as is for the preliminary support it would give. Would you mind if we keep the original issue/bugzilla open though?

maybe the def could fail more gracefully when it gets nothing back? Also we'll have to update this url when the repo name changes

@AllenBW Sure, that should be doable.
And yes we will need a repo name update in the url as well.

I think this is a great feature as you've implemented it, but one concern i'd have is that this would give the commit hash at the time the MIQ server is started (since we are requerying that url each time), correct? Ideally we could persist the hash at the time the appliance image is built some place...

@priley86 The commit hash method would be executed every time you access Migration from the menu. Persisting on the Rails side, while doable is a more involved process and would be a good idea to get into that once we implement this for all plugins, not just v2v.
We could keep the BZ open, although I would need the BZ tagged to this PR in order to backport this to Gaprindashvili.
Also, since the BZ is not applicable to v2v alone, we should probably not categorize it as a v2v BZ.

@AllenBW I have addressed the sans-internet case you mentioned above :)
In an event when there is no connectivity, you will see either master.undefined or gaprindashvili.undefined in the html.
Since there is no rush for this, please review/merge at your convenience.
Thanks.

@AparnaKarve agree... OK - just let me know if you end up wanting to backport this to G. I don't think that's urgent (and we could just keep the issue and BZ open for now until the long term solution is put in place). Otherwise, will be happy to open another BZ later.
I'm fine w/ merging as-is if others are...

Thanks folks!
I'm keeping the gaprindashvii/yes label ON because I think the hash will be useful to be in-sync with the QE folks especially in situations where they see an issue and we can't reproduce it because we are on the latest commit hash.

@AparnaKarve Doesn't this always return 'undefined' on downstream appliances?

@simaishi Not sure, would have to check that.
Will let you know what I find.

@simaishi For downstream appliances, it does not return undefined ...

Let's hold off on the gaprindashvili backport for now.
Thanks.

That's probably because downstream version tag (e.g. <IP_ADDRESS>) won't exist on upstream repo. Not sure what the best way is, but hard-coding upstream repo url might not be a good idea... cc @Fryguy

The /var/www/miq/vmdb/BUILD seems to contain the commit hash info for miq core
Currently I am looking at a 5.10 nightly appliance, which contains this in the BUILD file -
<IP_ADDRESS>-pre-nightly-20180604133801_a49e48a

So I would think that we need to add the commit hashes for other plugins in the BUILD file as well.

I'll leave that to other people to decide, as adding SHA for other plugins to BUILD file will require other changes and need to be discussed 😄

True.
I guess I was overly optimistic when I thought we could resolve this from our side (the v2v side) :)


Option to invert highlight current cell

I kind of find it distracting to work on a code that is highlighted. So I usually keep the "Highlight current cell" option turned off.
I think it would it be interesting to have an option in the editor preferences to invert the highlighting of the current cell. Here is how this could look like:

It is almost possible to do the same thing in Syntax coloring by inverting current cell highlight and background color, but it doesn't look as good.

@goanpeca, @rlaverde, what do you guys think?

I don't think I completely understand :-p

@jnsebgosselin want cells to not be highlighted. Instead, the text around them would be "highlighted".

Hmmm interesting, I think it makes sense, or at least is more readable? the idea o the cell being to ficus on that text?

@ccordoba12 yes the idea is to make the current cell not highlighted, while making the rest of the text highlighted. And yes @goanpeca, the goal is to make the code in the current cell more readable, while still providing a visual distinction between the current cell and the surrounding text.
I've made a side by side comparison in the figure below. Personally, I find it easier on the eye and less distracting to work in the editor on right with "inverted highlighting" on.

I agree with this change, It'll be an option? or the default behavior?

Ok, I'm -1 on it, but the majority wins!
It'll be a new option because the current behavior is the same as Matlab's one, and we want to be as close to Matlab as possible,

Just wondering if this is something you still find interesting to work on @jnsebgosselin
Also, you seem to prefer the light theme over the dark theme I believe?

@jnsebgosselin has this been implemented?

@goanpeca If you still think this is a good idea, I can work on this. Should not be that hard to do... I hope :P
@levibaguley no it is not implemented yet, sorry.

I am +1 on this. Would like to hear what other @spyder-ide/core-developers have in mind :-)

I don't understand the picture. To me, the picture on the right looks like the current behaviour, but this is supposed to be the new behaviour? :confused:

I don't understand the picture. To me, the picture on the right looks like the current behaviour, but this is supposed to be the new behaviour? 😕

No, current behaviour is the picture on the left.

I'm +1 on this.

No, current behaviour is the picture on the left.

Ah, something (probably me) changed the colours in my preferences. Got it now. It seems a good idea, so go ahead.


[SPARK-22904][SQL] Add tests for decimal operations and string casts

What changes were proposed in this pull request?
Test coverage for arithmetic operations leading to:

Precision loss
Overflow

Moreover, tests for casting bad string to other input types and for using bad string as operators of some functions.
How was this patch tested?
added tests

Test build #85412 has started for PR 20084 at commit 7a9650a.

Test build #85412 has finished for PR 20084 at commit 7a9650a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85412/
Test FAILed.

Test build #85433 has started for PR 20084 at commit 103c0e7.

Test build #85433 has finished for PR 20084 at commit 103c0e7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85433/
Test PASSed.

cc @gatorsmile

Thanks! Merged to master.


Update sbt-native-packager to 1.8.0

Updates com.typesafe.sbt:sbt-native-packager from 1.5.2 to 1.8.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.sbt", artifactId = "sbt-native-packager" } ]

labels: sbt-plugin-update, semver-minor

Superseded by #107.


Data deletion details for CockroachCloud

Admin Tasks on Console Access Management Page
-add delete your organization instructions
-add remove an email address instructions
-add remove a credit card instructions
Cluster Management Page
-add warning that free tier clusters may be deleted after 6 months of no activity
User Authorization Page
-instructions to delete a SQL user were already present. no changes needed.
Closes #9277 and #9767

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4e63a93fe7f72db3ea55a3bc690fbb67140d4a62/
Edited pages:

cockroachcloud/cluster-management.md
cockroachcloud/console-access-management.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/421fb6c1067b693fe8a75f1f42f2da20b7eaabc3/
Edited pages:

cockroachcloud/cluster-management.md
cockroachcloud/console-access-management.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/17bef336e73a07737210693af0189b01068f42b3/
Edited pages:

cockroachcloud/cluster-management.md
cockroachcloud/console-access-management.md

LGTM!

whoops sorry didn't mean to close on your behalf


Bug: BigQueryGetDataOperator XCOM is unable to deserialise json

When a date column is present in the Big Query table ,  BigQueryGetDataOperator  is failing to  deserialise json in XCOM

Closing it as this is a duplicate of #127


Can this be compiled with Mono?

I'm currently experimenting with using Mono on this code in order to run it on Linux, but I have never used C# before. It seems xbuild can be used to compile visual studio projects, but running
xbuild The\ Pre-Sequel\!.sln /verbosity:detailed

gives me this:
[…]
/opt/local/lib/mono/4.5/Microsoft.Common.targets:  warning : Reference 'PresentationFramework' not resolved
				For searchpath {CandidateAssemblyFiles}
				Warning: {CandidateAssemblyFiles} not supported currently
				For searchpath {HintPathFromItem}
				HintPath attribute not found
				For searchpath {TargetFrameworkDirectory}
				Considered target framework dir /opt/local/lib/mono/4.0, assembly named 'PresentationFramework' not found.
				For searchpath {PkgConfig}
				Considered PresentationFramework, but could not find in any pkg-config files.
				For searchpath {GAC}
				Considered PresentationFramework, but could not find in the GAC.
				For searchpath {RawFileName}
				Considered '/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.SaveEdit/PresentationFramework' as a file, but the file does not exist
				For searchpath ../../bin/
				Considered '/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/bin/PresentationFramework' as a file, but the file does not exist
				Considered '/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/bin/PresentationFramework.exe' as a file, but the file does not exist
				Considered '/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/bin/PresentationFramework.dll' as a file, but the file does not exist
			Target GenerateSatelliteAssemblies:
			No input files were specified for target GenerateSatelliteAssemblies, skipping.
			Target GenerateTargetFrameworkMonikerAttribute:
			Skipping target "GenerateTargetFrameworkMonikerAttribute" because its outputs are up-to-date.
			Target CoreCompile:
				Tool /opt/local/lib/mono/4.5/mcs.exe execution started with arguments: /noconfig /debug:full /debug+ /optimize- /out:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.exe /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Properties.Resources.resources /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Caliburn.Micro.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Caliburn.Micro.Contrib.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.System.Windows.Interactivity.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Xceed.Wpf.Toolkit.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Gibbed.BorderlandsOz.FileFormats.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Gibbed.BorderlandsOz.GameInfo.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Gibbed.BorderlandsOz.ProtoBufFormats.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Gibbed.IO.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.protobuf-net.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.MiniLZO.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.ICSharpCode.SharpZipLib.dll,Caliburn.Micro.dll /resource:obj/Debug/Gibbed.BorderlandsOz.SaveEdit.Newtonsoft.Json.dll,Caliburn.Micro.dll App.xaml.cs Helpers/AssetCache.cs Helpers/Blacklisting.cs Helpers/DelegateCommand.cs Helpers/EnumToBooleanConverter.cs Helpers/GearCalculatorView.xaml.cs Helpers/GearCalculatorViewModel.cs Helpers/HyperlinkExtensions.cs Helpers/InputBindingTrigger.cs Helpers/MyClipboard.cs Helpers/MyFileFilterCollection.cs Helpers/MyMessageBox.cs Helpers/ShortcutParser.cs Items/BackpackItemView.xaml.cs Items/BackpackItemViewModel.cs Items/IBaseSlotViewModel.cs Items/ItemTypeFilter.cs SaveLoad.cs Startup.cs Tabs/AboutView.xaml.cs Tabs/AboutViewModel.cs Tabs/BackpackView.xaml.cs Items/BackpackWeaponView.xaml.cs Items/BackpackWeaponViewModel.cs Items/BaseItemViewModel.cs Items/BaseWeaponViewModel.cs Helpers/ContainsToVisiblityConverter.cs Tabs/BankView.xaml.cs Tabs/BankViewModel.cs Tabs/CharacterView.xaml.cs Tabs/CharacterViewModel.cs Tabs/CurrencyOnHandView.xaml.cs Tabs/CurrencyOnHandViewModel.cs Helpers/AssetDisplay.cs Items/IBackpackSlotViewModel.cs Helpers/MyOpenFileResult.cs Helpers/MySaveFileResult.cs Helpers/NullToBooleanConverter.cs Items/BaseItemView.xaml.cs Items/BaseWeaponView.xaml.cs Helpers/MyComboBox.cs Tabs/FastTravelView.xaml.cs Tabs/FastTravelViewModel.cs Tabs/GeneralView.xaml.cs Helpers/PropertyProxy.cs ShellView.xaml.cs Tabs/VehicleView.xaml.cs Tabs/VehicleViewModel.cs Validators/AssetLibrarySetIdValidationRule.cs Validators/AssetValidationRule.cs Validators/BalanceValidationRule.cs Validators/ItemPartValidationRule.cs Validators/ItemTypeValidationRule.cs Validators/ManufacturerValidationRule.cs Validators/WeaponPartValidationRule.cs Validators/WeaponTypeValidationRule.cs Version.cs AppBootstrapper.cs AppWindowManager.cs Tabs/BackpackViewModel.cs Tabs/GeneralViewModel.cs ShellViewModel.cs Properties/AssemblyInfo.cs Properties/Resources.Designer.cs Properties/Settings.Designer.cs /target:winexe /define:"DEBUG;TRACE" /main:Gibbed.BorderlandsOz.SaveEdit.Startup /nostdlib /platform:AnyCPU /reference:../../packages/Caliburn.Micro.1.5.1/lib/net40/Caliburn.Micro.dll /reference:../../other/wpf/Caliburn.Micro.Contrib.dll /reference:/opt/local/lib/mono/4.0/System.dll /reference:/opt/local/lib/mono/4.0/System.ComponentModel.Composition.dll /reference:/opt/local/lib/mono/4.0/System.Data.dll /reference:/opt/local/lib/mono/4.0/System.Drawing.dll /reference:../../packages/Caliburn.Micro.1.5.1/lib/net40/System.Windows.Interactivity.dll /reference:/opt/local/lib/mono/4.0/System.Xml.dll /reference:/opt/local/lib/mono/4.0/Microsoft.CSharp.dll /reference:/opt/local/lib/mono/4.0/System.Xml.Linq.dll /reference:/opt/local/lib/mono/4.0/System.Data.DataSetExtensions.dll /reference:/opt/local/lib/mono/4.0/System.Xaml.dll /reference:/opt/local/lib/mono/4.0/WindowsBase.dll /reference:../../packages/Extended.Wpf.Toolkit.2.0.0/lib/net40/Xceed.Wpf.Toolkit.dll /reference:/opt/local/lib/mono/4.0/System.Core.dll /reference:/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.FileFormats/bin/Debug//Gibbed.BorderlandsOz.FileFormats.dll /reference:/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.GameInfo/bin/Debug//Gibbed.BorderlandsOz.GameInfo.dll /reference:/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.ProtoBufFormats/bin/Debug//Gibbed.BorderlandsOz.ProtoBufFormats.dll /reference:/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.IO/bin/Debug//Gibbed.IO.dll /reference:/opt/local/lib/mono/4.0/mscorlib.dll /warn:4
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
			Task "Csc" execution -- FAILED
			Done building target "CoreCompile" in project "/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.SaveEdit/Gibbed.BorderlandsOz.SaveEdit.csproj".-- FAILED
		Done building project "/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.SaveEdit/Gibbed.BorderlandsOz.SaveEdit.csproj".-- FAILED
	Task "MSBuild" execution -- FAILED
	Done building target "Build" in project "/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/The Pre-Sequel!.sln".-- FAILED
Done building project "/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/The Pre-Sequel!.sln".-- FAILED

Build FAILED.

Warnings:

/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/The Pre-Sequel!.sln (default targets) ->
(Build target) ->
/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.SaveEdit/Gibbed.BorderlandsOz.SaveEdit.csproj (default targets) ->
/opt/local/lib/mono/4.5/Microsoft.Common.targets (ResolveAssemblyReferences target) ->

	/opt/local/lib/mono/4.5/Microsoft.Common.targets:  warning : Reference 'PresentationFramework.Aero' not resolved
	/opt/local/lib/mono/4.5/Microsoft.Common.targets:  warning : Reference 'PresentationCore' not resolved
	/opt/local/lib/mono/4.5/Microsoft.Common.targets:  warning : Reference 'PresentationFramework' not resolved

Errors:

/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/The Pre-Sequel!.sln (default targets) ->
(Build target) ->
/Volumes/Data/Downloads/Sources/Gibbed.BorderlandsOz/projects/Gibbed.BorderlandsOz.SaveEdit/Gibbed.BorderlandsOz.SaveEdit.csproj (default targets) ->
/opt/local/lib/mono/4.5/Microsoft.CSharp.targets (CoreCompile target) ->

	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly
	CSC: error CS1508: The resource identifier `Caliburn.Micro.dll' has already been used in this assembly

	 3 Warning(s)
	 11 Error(s)

Time Elapsed 00:00:02.9127500

What steps would need to be taken in order for the app to run on Linux?

I don't think there's a viable implementation of WPF in Mono, I think they only covered what Silverlight needed?

Dunno, I ran mono .nuget/NuGet.exe install PresentationFramework from the source dir and it seemed to install just fine.

Oh I'm sorry, I just realized you asked if it can be compiled with Mono. As opposed to running it with Mono.


Update chain-list (w51)

[x] https://github.com/Koniverse/SubWallet-ChainList/issues/165
[x] Udpate new logo and price-id for 3DPass:

Link logo: https://www.figma.com/file/PxPvXw5woBernBu3BtwxkX/SubWallet-Project-Logo?type=design&node-id=302-555&mode=design&t=tHMRTlBkYfri15CJ-4
Price-id: https://www.coingecko.com/en/coins/3dpass
explorer: https://3dpscan.io

Version test: http://<IP_ADDRESS>:8900/s/6wTa4FTZakt7E3x
Validation passed

[ ] Enable Energy Web X

Version test: http://<IP_ADDRESS>:8900/s/cScxcE7ySpYJaeq
Validation passed for Energy Web X in cases:

Turn on/ Turn off network
Show token
Show balance
Transfer token (approve step)


carthage update failed

*** Fetching SwiftyJSON
*** Fetching Alamofire
*** Checking out Alamofire at "1.3.0"
*** Checking out SwiftyJSON at "2.2.0"
*** xcodebuild output can be found in /var/folders/tf/9pwp1f4s0lv1n861vsyg7mph0000gn/T/carthage-xcodebuild.CBhSqT.log
*** Building scheme "Alamofire iOS" in Alamofire.xcworkspace
** BUILD FAILED **
The following build commands failed:
Check dependencies
(1 failure)
Code Sign error: No code signing identities found: No valid signing identities (i.e. certificate and private key pair) matching the team ID “(null)” were found.
CodeSign error: code signing is required for product type 'Framework' in SDK 'iOS 8.4'
A shell task failed with exit code 65

I have encountered this problem

I discard carthage, and use pod to manage the libs instead, remove the dependence in linked Frameworks and Libraries and Embedded Binaries of General. Then I success to run. Good luck !

Thanks for reporting this issue. I have updated the app to not use Carthage any more. Please checkout the current version. https://github.com/JakeLin/SwiftWeather


Pipeline V3

This PR:

[x] Updates PullRequests to utilize git to run CI only packages that were changed

[x] Packages that have been changed may optionally include other packages for sparse validation. For instance azure-core configures an additional 15 or so packages that must be brought in for sparse validation. This is configured solely in get_package_properties.py.

[x] Affected packages are now batched, so even if a PR touches 50+ packages, we will simply expand the test matrix by limiting each test run to a max of 8 packages (arbitrary limit). We just...add more test runs. Analyze will still have an issue with this many packages, I will likely multiplex that in follow-up PRs as well.

[x] In the above case, when additional validation packages are pulled in due to a large number of directly affected packages, we will also sparsely include those validation packages into the test matrix. ("sparsely" translating to "on a single platform and python combination, we don't really care which")

The failing python - pullrequest is a CI build for the build definition.  The error is correct. I am going to ignore this failure and simply submit a follow-up PR to azure-template that resolves it. Not worth another commit.

/check-enforcer override


demo-flask ImportError: cannot import name 'Markup' from 'jinja2'

When I attempt to run the Flask demo it fails immediately:

ImportError: cannot import name 'Markup' from 'jinja2' (/home/example/test-saml/python3-saml/demo-flask/env/lib/python3.10/site-packages/jinja2/init.py)

I'm using Python 3.10.15 and the master branch of python3-saml.  I have not touched any of the files like settings.json yet as I want to get a baseline first. Here are my steps:
cd demo-flask
python3 -m venv env
. env/bin/activate
pip install -r requirements.txt
python index.py

The ImportError happens immediately.
pip freeze shows that I'm getting these packages:

click==8.1.7
Flask==1.0
itsdangerous==2.2.0
Jinja2==3.1.4
MarkupSafe==3.0.2
Werkzeug==3.1.3

If I install Flask and python3-saml per the following updated requirements.txt contents, then python index.py runs without crashing:
flask==3.1.0
python3-saml==1.16.0


Add a web development project

Hi,
This PR to add a new web development project assigned with issue #48
The project developed with:
JSP
Servlets
HTML5 & CSS3
JavaScript & JQuery
Ajax
MySQL
I hope this project helps other developers.

It's an amazing project.... It looks really good.... We are very happy to have a contributor like you...Love your good effort.....Expecting more projects from you.....Thanks for your interest...

Thank you very much, It’s a great chance for me to be one of this great repository.
I hope we all do more.

It's an amazing project.... It looks really good.... We are very happy to have a contributor like you...Love your good effort.....Expecting more projects from you.....Thanks for your interest...

Thank you very much, It’s a great chance for me to be one of this great repository.
I hope we all do more.

It's an amazing project.... It looks really good.... We are very happy to have a contributor like you...Love your good effort.....Expecting more projects from you.....Thanks for your interest...

Thank you very much, It’s a great chance for me to be one of this great repository.
I hope we all do more.

Yah...More new and interesting projects coming soon

It's an amazing project.... It looks really good.... We are very happy to have a contributor like you...Love your good effort.....Expecting more projects from you.....Thanks for your interest...

Thank you very much, It’s a great chance for me to be one of this great repository.
I hope we all do more.

Yah...More new and interesting projects coming soon


chore(docs): instruct on how to query your own validator indexes

as mentioned by a user on discord, it's not clear how to query your own validator indexes when trying to run
a bolt node on Helder or other testnets. we should add a few lines on how to query the
/eth/v1/beacon/states/head/validators endpoint to fetch this data.
For our internal infra, we use a lighthouse beacon + validator client, beacon HTTP api running on port 4400
with the keys stored in validator-keys/secrets and we query validator IDs with this script:
#!/bin/bash

# Directory containing the files
DIRECTORY="./validator-keys/secrets"

# Base URL for the API
BASE_URL="http://localhost:4400/eth/v1/beacon/states/head/validators"

# Loop through all files in the directory
for FILE in "$DIRECTORY"/*; do
    # Check if FILE is a regular file
    if [ -f "$FILE" ]; then
        # Extract the filename without the path
        FILENAME=$(basename "$FILE")

        # Make the API call with the filename as the query parameter
        RESPONSE=$(curl -s -X GET "${BASE_URL}?id=${FILENAME}" -H "accept: application/json")

        # Parse the JSON response and print the 'data[0].index' field
        INDEX=$(echo "$RESPONSE" | jq -r '.data[0].index')

        # Print the index
        echo "File: $FILENAME, Index: $INDEX"
    fi
done

Validator indexes support has been dropped so this issue is no more relevant.


MongoDB cluster solution

Example implementation of a MongoDB replica set cluster using stateful Managed Instance Groups and ContainerOS based deployment. Supports Cloud DNS, logging and monitoring out of the box, as well as scaling up.
Also includes change in DNS module to add create_only flag that makes records ignored for updates.

@rosmo what's the status of this? Can we merge or close?

@juliocc It should be good to be merged, the requested changes have been made.

Sorry for lagging on this. I'm not convinced create-only records should be in the DNS module as it seems a very specific use case, and we don't want to start doing this across modules. Why do you even need it?

@rosmo can we remove the DNS changes, then finish and merge this?

I am closing this, feel free to reopen once we find a less disruptive way of handling DNS records.


Cannot read property 'style' of undefined

on latest 11-12-2014 build
I'm not sure if this is from leaflet or angular-leaflet-directive error..
seams like _contentNode not defined?
TypeError: Cannot read property 'style' of undefined
at o.Popup.o.Class.extend._updateLayout (http://<IP_ADDRESS>:8000/js/leaflet-0.7.3/leaflet.js:7:21610)
at updatePopup (http://<IP_ADDRESS>:8000/js/leaflet-directive/angular-leaflet-directive.js:3153:23)
at _manageOpenPopup (http://<IP_ADDRESS>:8000/js/leaflet-directive/angular-leaflet-directive.js:3167:17)
.....
on 04-12-2014 build this error not fired, but another with _contentNode.innerHTML undefined appears
TypeError: Cannot read property 'innerHTML' of undefined
at t (http://<IP_ADDRESS>:8000/js/leaflet-directive/angular-leaflet-directive.min4-12.js:32:18709)
at Object.fn (http://<IP_ADDRESS>:8000/js/leaflet-directive/angular-leaflet-directive.min4-12.js:32:22034)

Experiencing the same issue here.. any possible workarounds? $compile?

This happens when changing the 'focus' property on the marker, see example at: http://tombatossals.github.io/angular-leaflet-directive/examples/0000-viewer.html#/markers/popup-example

So it needs another null check here possibly, among other things. https://github.com/tombatossals/angular-leaflet-directive/blob/46d1a5a0b1af9b440ca22988cc0228ebf03ef72d/src/services/leafletMarkersHelpers.js#L141
Please make a PR with a fix and an example in examples which shows the original behavior resolved.


CB-14398: Add resize recovery service.

Closes CB-14398.
See https://github.com/hortonworks/cloudbreak/pull/11686

Please don't put features into CB without the implementation of the feature :) It can be maintained in a separate branch. We merge finished implementations into master. Why is it not implemented?

@sodre90 My intention was to use smaller PRs to get to the overall feature. I can do a larger feature branch if that's preferable.

I see. Usually, we put features in small pieces but the small pieces should have useful features on their own. If I am correct, based on the implementation, if anybody calls this API with this modification the same will happen as before. It is a very small change, I think it can be implemented with the whole thing. If we need to revert this implementation then we need to revert the commits based on this commit. (I think it would not cause any issue anyway, just an example). I would prefer if we put this code with the real implementation behind it.


Game Pass

It seems there's been a recent change to GP folder permissions for games that have user accessible folders. It's now possible to add Game Pass games directly to Steam via the gamelaunchhelper.exe found in Game Folder/Content - previously this would flag up as denied due to folder permissions. With this change, is GP integration now possible?

it should be possible given that uwphook has been doing this for quite some time, i would love to see this in BoilR
https://briano.dev/UWPHook/

steamsync can do this really well. Some games (Sunset Overdrive) aren't picked up (probably because of old GP install method), but for the most part it works

Hi thanks for bringing this to my attention, don't know how I overlooked this post.
I will look into how some of the other tools do it and then hopefully have time to implement it in boilr as well.

I have this working locally now:
https://github.com/PhilipK/BoilR/pull/350
I will make a pre-release of this and hope that someone will test if it also works for them.

Game Pass is now implemented:
https://github.com/PhilipK/BoilR/releases/tag/v.1.9.0
Let me know how it works out for you.


Imposter got muter later than Crewmate

When a game start, we got 1 imposter on the 2 imposters than get muted later than Crewmates ( Around 5/6 sec after crewmates)

This is a fluke. Discord rate limits the bot based on the total number of players in the lobby (typically having issues for lobbies of 7-8+ players), but it has absolutely nothing to do with imposter/crewmate status


[9.4.x] ISPN-10059 Off-heap clustered cache cannot start with segmentation disabled

https://issues.jboss.org/browse/ISPN-10059
Backport of #6779

Integrated into 9.4.x, thanks @danberindei !

I opened https://github.com/infinispan/infinispan/pull/6790 for ExceptionEvictionTest failures, missed backporting it.


✨ feat: added melos build_runner command (#641)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #645 (3a47689) into main (8781cbb) will increase coverage by 4.33%.
The diff coverage is n/a.

: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     #645      +/-   ##
==========================================
+ Coverage   80.28%   84.61%   +4.33%
==========================================
  Files         219       19     -200
  Lines        2571      429    -2142
==========================================
- Hits         2064      363    -1701
+ Misses        507       66     -441

see 210 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[BUG] fixing the divine activities page

Description
proper margins and footer will be fixed with ui improvement
Describe the bug:
A clear and concise description of what the bug is.
Expected behavior:
A clear and concise description of what you expected to happen.
Actual behavior:
A clear and concise description of what actually happened.
Reproduction Steps
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected result:
A clear and concise description of what you expected to happen at each step.
Actual result:
A clear and concise description of what actually happened at each step.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional Information
Device (please complete the following information):

OS: [e.g., Windows, macOS, Linux]
Browser [e.g., Chrome, Safari, Firefox]
Version [e.g., 22]

Additional context:
Add any other context about the problem here.

@akshitagupta15june please assign this to me


improve test_generate_snap test case.

The test case fails frequently. However maybe it's not a bug, just because the wait time is not long enough.
This PR increase max wait time from 2s to 5s, so that it can tolerate more disk latency.
And panic message is improvced.
Fix #3129 .

Any updates?

ping @BusyJay @overvenus .


Fetch imdb ratings

Description
Scraped data from IMBDs website, made a films list from given directory, put ratings for films in a .csv file, updated readme.
Fixes #239
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Documentation Update

Checklist:

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[x] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have added a working sample/screenshot of the script.

Hope this is fine.


Resiliency to slightly non-RFC compliant HTTP requests

I recently ran into a bug with the Python standard library where a standard library was sending RFC non-compliant HTTP requests: http://bugs.python.org/issue13132
Despite being RFC non-compliant, the requests actually work in the following web servers:

Apache2
Nginx
Twisted

I was attempting to write a lightweight cheese shop server using Tornado but this is currently blocking me since the request can't be parsed.
Despite not being compliant with the RFC, I think that web servers should do their best to serve any requests, just as web browsers do their best to serve malformed responses and markup. I propose that Tornado should handle the following separator cases for headers:

\r\n
\n\n
\n

Mitchell

How would one differentiate between \n\n being the end of headers and being the delimiter between two headers?

@rcrowley:
Sorry, headers should be separated by a single newline. A single CRLF for RFC compliancy or LF for non. I've never (though I'm sure it exists somewhere) seen two LFs separate a single header.
So basically \n would be another header, \n\n would be the end of headers.
Thanks for pointing that out.

No problem.  I just didn't want anyone descending into parser hell unnecessarily.

Update: Actually, using two LFs: "\n\n" is HTTP 1.0 compliant, and only HTTP 1.1 requires "\r\n"
I'm not sure what Tornado's stance is on what HTTP version the web server is supposed to handle, but I believe the HTTP 1.0 spec gives a much stronger case for supporting \n\n.

RFC 2616 "recommends" accepting bare LF in addition to CRLF (http://tools.ietf.org/html/rfc2616#section-19.3), so it's reasonable to make this change.  We recently made the analogous change in simple_httpclient.
That doesn't let distutils off the hook, though - the rfc recommends that servers accept LF for maximum compatibility, but it never condones sending LF, so distutils should still be fixed.  HTTP 1.0 and 1.1 don't appear to be different in this respect as far as I can tell.

I attempted to submit a patch to distutils but they claim that it is 1.0 compliant and that they aren't changing distutils anymore anyways except for critical bugs, since distutils2 is now being worked on as a rewrite for Python 3.
I'm glad that support for this will come into Tornado though, this'll save me a lot of time :)


Bump framework version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7485200360

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7485200360
Status: success


Removes "SqlServer:Initialize" config option from launch settings

Description
Now that we have SqlServer:AutomaticUpdatesEnabled, the SqlServer:Initialize config setting is no longer needed.
This PR removes SqlServer:Initialize and updates an error message for an edge case that is a bit easier to trigger now that we are removing this setting (please see my review comment below for more context).
Related issues
Addresses #AB77211.
Testing
Manually tested the startup of the test web project with all possible combinations of the config values. I also copied the SQL health .pdb and .dll files and tested the fhir-server startup as well.

It is difficult to merge a clean implementation of this until SQL database creation is enabled for this github repo. I will close this PR for now, and reopen a new one once this is done.


Ballerina: Home config is read only when plugin development mode is enabled

Description
Users of the VS Code plugin have to enable "plugin development mode" if they want to change Ballerina home. This is a bit counter-intuitive. Therefore what if we honor the ballerina home config irrespective of whether the user has enabled "plugin development mode" or not.

Users are not meant to use the ballerina.home configuration at any cost. Earlier we had provided this particular option for the users to change the ballerina version. Eventually we depricated this particular capability from the users. In order to avoid confusion for both the earlier and newer users we added the check option to quickly disable the ballerina.home configuration honor.
So, from that point onwards users need to set the Ballerina Home environmental variable in order to change the version and that will be picked up by the plugin automatically.
Most importantly, only the tooling developers are advised to use the ballerina.home configuration for debugging purposes.

@nadeeshaan Maybe it is better to show a message in the config description telling this.


fix #69 to Support Web Get JsSdkSignature

fix #69 to Support Web Get JsSdkSignature，Contain Save  JsSdkTicket Into Cache

 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.yanji-yang 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.


heredoc doesn't work inside of command substituiton

example bash:
#!/bin/sh

test1=$(cat <<EOF
test
test test
test test test
EOF
)
test2=$(echo test 2)
echo $test1
echo $test2
cat <<TEST3
test 3
TEST3

command output
➜  xe git:(master) ✗ bash test.sh
test test test test test test
test 2
test 3
➜  xe git:(master) ✗ brush test.sh
ERROR main: unterminated here document sequence; tag(s) found at: [line 1 col 14] (detected near line 5 column 1)

test 2
test 3

@reubeno seems like it gets tokenized twice?
$(<<EOF
test
EOF
)

after the tokenize_str
Word(
        "$(<<EOFtest\n\n)", //  <-- the input inside qoutes already processed by the tokenizer, but it sends it to the subshell as a raw string
        TokenLocation {
            start: SourcePosition {
                index: 0,
                line: 1,
                column: 1,
            },
            end: SourcePosition {
                index: 18,
                line: 4,
                column: 2,
            },
        },
    ),
    Operator(
        "\n",
        TokenLocation {
            start: SourcePosition {
                index: 18,
                line: 4,
                column: 2,
            },
            end: SourcePosition {
                index: 19,
                line: 5,
                column: 1,
            },
        },
    ),
]

@39555 Yes. It's not ideal, but for the initial implementation we landed on the tokenizer yielding the full text of the command substitution in the word token that contains it (potentially along with other text, substitutions, etc.). The recursion was added to ensure that we can find the end of the substitution, but that's really all it's needed for here.
I recall having explored alternative options (e.g., support a Word storing a tree of sub-tokens) but selected this approach as the most practical to start with.
At this point, the proximate error seems to be that the newline between EOF and test gets lost in the mix.

@reubeno when the tokenizer processes the input, it replaces the last EOF with a newline for simplicity. Maybe because of that the subshell's tokenizer throws an error that the 'Here' is not closed? The first tokenizer should skip all the text inside the substitution unchanged.

Good question. I added a quick test case to cover this (and improved the error):
https://github.com/reubeno/brush/commit/a290a0dd86e8c5d3b24b81ca57934c1cc7381154#diff-bf47c71a8ce1b403511f60305d7d562506a83cacb1380deefc9290960e6c7651R43
Here's what displays:

It appears that the here tag ends up getting considered as EOF1, with the newline between EOF and the first line of the here-doc body getting lost.
@39555 -- I totally understand if not, but if you happen to have time and interest to look deeper and identify a resolution, you're welcome to pick this up. Note also a few additional tests I added to tokenizer.rs in the linked commit (but left commented out for now). I'm not 100% certain that the assertions in it are correct but I believe they're problematic as well.

@reubeno Can you help me? What is that supposed to do https://github.com/reubeno/brush/blob/a7d77ffb7295600c8c17f9f3317a004f9a586ded/brush-parser/src/tokenizer.rs#L561C1-L561C47 when I place dbg! it is always None...

@39555 Are you asking about the strip_suffix() call? That's where we actually check for finding the tag that should delimit the here document. If I recall correctly, it's the one place where we check to see if we've reached the terminating tag for the here document -- and then strip it off to get the actual body.
I'm not sure why you'd only ever see None. Are you testing with a simple case (outside a command substitution)?
(Aside: the parsing of here documents got a bit complicated in the tokenizer. Definitely open to seeing improvements to the clarity and simplicity of the implementation.)


Can't specify custom error message for key? predicate with specific attribute

Having following errors file:
en:
  errors:
    rules:
      foo:
        key?: 'Foo is missing'
        filled?: 'Foo must be filled'

And following schema:
FooSchema = Dry::Validation.Schema do
  required(:foo).filled
end

Error message for filled? predicate works as expected:
FooSchema.call(foo: '').messages # => { foo: ['Foo must be filled'] }

But there's some problem with message for key? predicate:
FooSchema.call({}).messages # => Dry::Validation::MissingMessageError: message for key? was not found

I just realized why this happens. key? is not applied to the value, it is applied to the whole hash, so there's no rule for foo value, there's rule for the whole hash that it must have a key called foo.

So there's no way to do what I want?

Not at the moment. I'll mark this as a feature request.

We should probably add a pattern-matching algo to error compilation so that we can have messages defined like that:
en:
  errors:
     key?:
       value:
         foo: "FOO MUST BE PRESENT!!111"

I've submitted a pull request for this issue and am open to any additional feedback.


Error: ENOENT: no such file or directory, stat '.next/prerender-manifest.json'

Describe the bug
New alpha push breaks the deployment with error - [Error: ENOENT: no such file or directory, stat '.next/prerender-manifest.json']
To Reproduce
Steps to reproduce the behavior:
This problem started ever since 1.12.0-alpha.2 was published
Additional context
If anyone faces this issue, make sure you pin the component version to stable version - 1.11.5
component<EMAIL_ADDRESS>

@hnprashanth What version of next.js are you using?

@danielcondemarin 9.0.8

@hnprashanth Could you try updating to next 9.3.0 or greater?

also just started happening on my project without any upgrades or new installs

@mcxolani if you don’t pin a version number for serverless-next.js in your version yaml serverless will pull whatever is latest published on NPM. Check the installation on the README page which recommends pinning a version number.
Also this issue would only happen if you’re using next.js under 9.3.0

@danielcondemarin works fine, thanks

Closing this as it should work fine on Next 9.3.0 or above.


[DASHBOARD] 1.1 New Page Component for Dashboard

Description
Created component and wired it to the navigation, I'm opened to the naming suggestions, Dashboard/Overview/(??)
closes #4525 #4526
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

https://user-images.githubusercontent.com/113331659/211340568-650de539-38e7-4900-ab37-372029fa2d59.mov

Pull request rejected by merge tool. Failed to parse change entry file: Change entry /tmp/tmpnpb2n5f3/changelogs/unreleased/4525-new-dashboard-component.yml does not match pydantic schema: 1 validation error for ChangeEntry
sections -> key
value is not a valid enumeration member; permitted: 'feature', 'bugfix', 'known-issue', 'upgrade-note', 'deprecation-note', 'other-note', 'minor-improvement' (type=type_error.enum; enum_values=[<ChangelogSection.FEATURE: 'feature'>, <ChangelogSection.BUGFIX: 'bugfix'>, <ChangelogSection.KNOWN_ISSUE: 'known-issue'>, <ChangelogSection.UPGRADE_NOTE: 'upgrade-note'>, <ChangelogSection.DEPRECATION_NOTE: 'deprecation-note'>, <ChangelogSection.OTHER_NOTE: 'other-note'>, <ChangelogSection.MINOR_IMPROVEMENT: 'minor-improvement'>])

Processing this pull request

Pull request rejected by merge tool. SemVer violation: branch iso5 is eligible for this change and has not diverged from one of the branches in the destination set. Applying this merge as requested would result in a version bump due to branches diverging. Since iso5 is eligible for this change, this version bump would not be in line with the SemVer semantics. Please include iso5 in the destination set.

Processing this pull request


[stable/coredns] Fix NOTES.txt template

Fixes #10190
Signed-off-by: Seongjin Cho<EMAIL_ADDRESS>
What this PR does / why we need it:
Fixes bug caused by wrong template.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #10190

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped

/assign @acaleph


pyright failing with traceback to nodeenv

Note: if you are reporting a wrong signature of a function or a class in the standard library, then the typeshed tracker is better suited for this report: https://github.com/python/typeshed/issues.
Describe the bug
In github CI/CD we are running pyright with the following versions now:
Collecting pyright
  Downloading pyright-1.1.292-py3-none-any.whl (16 kB)
Collecting nodeenv>=1.6.0
  Downloading nodeenv-1.7.0-py2.py3-none-any.whl (21 kB)

Our pyright all of a sudden (since today I would say) now fails with the following traceback.
Link:  https://github.com/keylime/keylime/actions/runs/4078266552/jobs/7028280605
/usr/local/lib/python3.11/site-packages/nodeenv.py:26: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
  import pipes
 * Install prebuilt node (19.6.0) .Incomplete read while readingfrom https://nodejs.org/download/release/v19.6.0/node-v19.6.0-linux-x64.tar.gz
.
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 1519, in <module>
    main()
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 1104, in main
    create_environment(env_dir, args)
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 980, in create_environment
    install_node(env_dir, src_dir, args)
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 739, in install_node
    install_node_wrapped(env_dir, src_dir, args)
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 762, in install_node_wrapped
    download_node_src(node_url, src_dir, args)
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 602, in download_node_src
    with ctx as archive:
  File "/usr/lib64/python3.11/contextlib.py", line 137, in __enter__
    return next(self.gen)
           ^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/nodeenv.py", line 573, in tarfile_open
    tf = tarfile.open(*args, **kwargs)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.11/tarfile.py", line 1630, in open
    saved_pos = fileobj.tell()
                ^^^^^^^^^^^^
AttributeError: 'bytes' object has no attribute 'tell'
Traceback (most recent call last):
  File "/usr/local/bin/pyright", line 8, in <module>
    sys.exit(entrypoint())
             ^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/cli.py", line 81, in entrypoint
    sys.exit(main(sys.argv[1:]))
             ^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/cli.py", line 21, in main
    return run(*args, **kwargs).returncode
           ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/cli.py", line 38, in run
    npx = node.version('npx')
          ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/node.py", line 118, in version
    proc = run(target, '--version', stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/node.py", line 97, in run
    binary = _ensure_available(target)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/node.py", line 36, in _ensure_available
    return Binary(path=_ensure_node_env(target), strategy=Strategy.NODEENV)
                       ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyright/node.py", line 63, in _ensure_node_env
    _install_node_env()
  File "/usr/local/lib/python3.11/site-packages/pyright/node.py", line 90, in _install_node_env
    subprocess.run(args, check=True)
  File "/usr/lib64/python3.11/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/python3', '-m', 'nodeenv', '/tmp/pyright-python.root/env']' returned non-zero exit status 1.
Error: Process completed with exit code 1.

Hmm this is very strange, does it work when you downgrade to the version before?
Nothing changed between releases that should affect nodeenv...

There was a pyright version release two days ago and nodeenv seems to not have changed in several months. We had builds today that succeeded. So maybe it's another dependency that causes the issue. I will try to investigate...

We have also seen this failing non-deterministically in our CI/CD.
The only difference between a passing and failing pipeline before the exception appears to be:
Passing:
     * Install prebuilt node (19.5.0) ..... done.

Failing:
     * Install prebuilt node (19.5.0) .Incomplete read while readingfrom https://nodejs.org/download/release/v19.5.0/node-v19.5.0-linux-x64.tar.gz

Both cases were using pyright (1.1.291).

I'm going to close this as the next release will add support for nodejs-wheel as an alternative to nodeenv which should be more reliable and will be the recommended way to use pyright going forward.
You can try it out today with
python -m pip install 'git+https://github.com/RobertCraigie/pyright-python.git#egg=pyright[nodejs]'

When released, it will be
pip install pyright[nodejs]

Please let me know if you run into any issues with this new setup.


Main Video and Thumbnails Layout

This PR is based on #206.
@cyntss could you take a look at the layout and let me know what you think about it? There is a small detail that bothers me; the white space (see in the screenshot below) that doesn't look too beautiful. I am not sure if this that important, but I would appreciate if you could give me some tips on how to improve this layout :)

Update: I found a few to deal with this issue. This is how it looks now:

Please, fix conflicts.

@imobachgs I have resolved the conflicts. This branch is ready for review.


get_regions() raises an error

Hey @TrashBirdEcology! Awesome package. Maybe this is a bug? :man_shrugging:
Describe the bug
The get_regions() function raises an error that seems to be caused by attempting to read a README.txt file from the FTP server that no longer exists.
To Reproduce
library(bbsAssistant)
regions <- get_regions()
#> Parsed with column specification:
#> cols(
#>   X1 = col_character(),
#>   X2 = col_character()
#> )
#> Parsed with column specification:
#> cols(
#>   X1 = col_character(),
#>   X2 = col_character(),
#>   X3 = col_character()
#> )
#> Warning in dir.create(codesDir): '/tmp/RtmpHiEJfU/reprex156458487ca2/
#> bbsData/codes' already exists
#> Warning in download.file(url = File, destfile = paste0(codesDir, "/
#> README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/
#> DataFiles/README.txt': FTP status was '550 Requested action not taken; file
#> unavailable'
#> Error in download.file(url = File, destfile = paste0(codesDir, "/README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt'

Created on 2019-08-14 by the reprex package (v0.3.0)
Expected behavior
I was expecting to get the region data, like in the README.
Session info
R version 3.6.1 (2019-07-05)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Pop!_OS 18.04 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.7.1
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.7.1

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8
 [6] LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] bbsAssistant_1.0.0

loaded via a namespace (and not attached):
 [1] Rcpp_1.0.2      rstudioapi_0.10 whisker_0.3-2   knitr_1.24      magrittr_1.5    hms_0.5.0       here_0.1        R6_2.4.0        rlang_0.4.0
[10] tools_3.6.1     packrat_0.5.0   xfun_0.8        withr_2.1.2     clipr_0.7.0     htmltools_0.3.6 rprojroot_1.3-2 digest_0.6.20   tibble_2.1.3
[19] crayon_1.3.4    processx_3.4.1  callr_3.3.1     readr_1.3.1     ps_1.3.0        vctrs_0.2.0     fs_1.3.1        curl_4.0        zeallot_0.1.0
[28] evaluate_0.14   rmarkdown_1.14  reprex_0.3.0    compiler_3.6.1  pillar_1.4.2    backports_1.1.4 pkgconfig_2.0.2

Additional context
The error seems to stem from this file not existing: ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt
Maybe it was removed in the most recent BBS data update?

Max, Thanks for reaching out. Unfortunately the BBS recently overhauled
their website, rendering some functions defunct. We are actively working on
the updates now. Feel free to clone the branch add_data. Otherwise, we
will release a new update in a few days, hopefully, which should solve all
existing bugs.
Thanks for reaching out!

Jessica L. Burnett
School of Natural Resources
University of Nebraska-Lincoln
+1.352.792.5425
GitHub http://github.com/trashbirdecology
CV/Resume
https://github.com/TrashBirdEcology/cv/blob/master/burnett_extendedCV.pdf
Nebraska Cooperative Fish & Wildlife Research Unit
http://snr.unl.edu/necoopunit/
ORC ID: 0000-0002-0896-5099 https://orcid.org/0000-0002-0896-5099

http://snr.unl.edu/necoopunit/
On Wed, Aug 14, 2019 at 10:28 AM Max Joseph<EMAIL_ADDRESS>wrote:

Hey @TrashBirdEcology https://github.com/TrashBirdEcology! Awesome
package. Maybe this is a bug? 🤷♂
Describe the bug
The get_regions() function raises an error that seems to be caused by
attempting to read a README.txt file from the FTP server that no longer
exists.
To Reproduce
library(bbsAssistant)
regions <- get_regions()
#> Parsed with column specification:
#> cols(
#>   X1 = col_character(),
#>   X2 = col_character()
#> )
#> Parsed with column specification:
#> cols(
#>   X1 = col_character(),
#>   X2 = col_character(),
#>   X3 = col_character()
#> )
#> Warning in dir.create(codesDir): '/tmp/RtmpHiEJfU/reprex156458487ca2/
#> bbsData/codes' already exists
#> Warning in download.file(url = File, destfile = paste0(codesDir, "/
#> README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/
#> DataFiles/README.txt': FTP status was '550 Requested action not taken; file
#> unavailable'
#> Error in download.file(url = File, destfile = paste0(codesDir, "/README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt'
Created on 2019-08-14 by the reprex package https://reprex.tidyverse.org
(v0.3.0)
Expected behavior
I was expecting to get the region data, like in the README.
Session info
R version 3.6.1 (2019-07-05)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Pop!_OS 18.04 LTS
Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.7.1
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.7.1
locale:
[1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8
[6] LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] bbsAssistant_1.0.0
loaded via a namespace (and not attached):
[1] Rcpp_1.0.2      rstudioapi_0.10 whisker_0.3-2   knitr_1.24      magrittr_1.5    hms_0.5.0       here_0.1        R6_2.4.0        rlang_0.4.0
[10] tools_3.6.1     packrat_0.5.0   xfun_0.8        withr_2.1.2     clipr_0.7.0     htmltools_0.3.6 rprojroot_1.3-2 digest_0.6.20   tibble_2.1.3
[19] crayon_1.3.4    processx_3.4.1  callr_3.3.1     readr_1.3.1     ps_1.3.0        vctrs_0.2.0     fs_1.3.1        curl_4.0        zeallot_0.1.0
[28] evaluate_0.14   rmarkdown_1.14  reprex_0.3.0    compiler_3.6.1  pillar_1.4.2    backports_1.1.4 pkgconfig_2.0.2
Additional context
The error seems to stem from this file not existing:
ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt
Maybe it was removed in the most recent BBS data update?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TrashBirdEcology/bbsAssistant/issues/51?email_source=notifications&email_token=ACL2TNLRQMV4GAWHN45HUITQEQQADA5CNFSM4ILWFGH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFHS4RQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACL2TNO6UE7VTT2UNFRHJTLQEQQADANCNFSM4ILWFGHQ
.

Awesome! Thanks @TrashBirdEcology :+1:

@mbjoseph , I have updated the vignette and package to reflect these changes. Please let me know if you run into any other issues, or have additional recommendations! Thanks again


Double export defaults in container screens

export default class UsageExamplesScreen
export default connect(mapStateToProps)(UsageExamplesScreen)
Same for presentation and few other screens

[x] Issue does not already exist
Identify exactly what aspect your issue is about

[x] ignite base application
[ ] ignite the cli
[x] ignite generated code

good catch!