Fix libcommon build

libcommon revision is currently dynamicaly defined from git tags after a fetch.
This is a potential security issue if using forked repository with SSH access.
Instead, it cleaner to get revision from package version. Revision is exported from debian/rules and used in libs/libcommon/src/create_revision.sh

Now it depends on building from debian/rules.
We decide commit current version with sources, i will prepare patch in next few days.

Using release script already relies on a working debian directory, right ?
Since release script generate debian/control & debian/changelog, and since the build uses (or should use) debhelper, this should work.

release script - yes, but libs/libcommon can be compiled in any os without /release script

will be fixed here: https://github.com/yandex/ClickHouse/pull/351

https://github.com/yandex/ClickHouse/commit/22797da2a634e2d6eecf531b4485278fd8bec32c
okay?

testing right now

Works for me 👍

I close this PR since problem is solved in a better way


where to buy  some wrist logging able device like e4-wristband

great code and paper
https://www.biorxiv.org/content/10.1101/183772v1.full.pdf
only can you share where to buy
some wrist logging able device like e4-wristband
https://store.empatica.com/products/e4-wristband?variant=17719039950953
but for reasonable price ?

Hi @Sandy4321,
The E4 is a clinical-research-grade device. The link you provided above is the only place I know of to order one.
Empatica say

The Empatica E4 is designed for use by researchers in clinical research studies with software that offers access to raw data. The data are difficult to interpret without 3rd party software or extensive technical expertise.
For personal use, we recommend using the Embrace watch, not the E4 wristband.

If you do want one or more E4s for research, they do have an academic discount:

Note: We offer an academic discount of 18% for all university purchases. Just email<EMAIL_ADDRESS>to request the discount code to be used at checkout.

Otherwise, because wearable tracking devices is a fast-moving market, I'd suggest making a list of features you want and seeing what's currently available. Our analysis was in service of a one-time bulk purchase 4 years ago; I'm sure that most to all of the consumer-grade devices we looked at have been replaced by newer models or otherwise discontinued by now.


Post-Translational Exon Shuffling (PTES)

Added a section for PTES under RNA world.

mind blown :exploding_head:
thanks for adding this!


Fixed difference in behavior of GetDirectories with absolute or relative paths

I've added tests and everything works,
but I don't know if the change I made is acceptable.
this should fix Issue#169
feel free to correct me.

On my setup, I had to add [Test] attributes to a couple of tests to get them to run in TypeMock's test runner. It's a Typemock error - the tests still ran fine as NUnit tests directly - but it'd be a nice addition IMO.
I ended up doing this on my fork here: https://github.com/AlexanderPruss/System.IO.Abstractions---Consistency-with-MSCORLIB-fixes/commit/bc08b340a2930c3d4c3d99fb49dfc917b90e0cf0

I think the test runner fixes should be a separate pull request. Maybe you
can create an issue for that?
I can't fix it because I have a different test runner.
I am more interested if my solution is acceptable or not.
Any opinion?
On Nov 25, 2016 12:03 PM, "Alexander Pruss"<EMAIL_ADDRESS>wrote:

On my setup, I had to add [Test] attributes to a couple of tests to get
them to run in TypeMock's test runner. It's a Typemock error - the tests
still ran fine as NUnit tests directly - but it'd be a nice addition IMO.
I ended up doing this on my fork here: AlexanderPruss@bc08b34
https://github.com/AlexanderPruss/System.IO.Abstractions---Consistency-with-MSCORLIB-fixes/commit/bc08b340a2930c3d4c3d99fb49dfc917b90e0cf0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tathamoddie/System.IO.Abstractions/pull/186#issuecomment-262933310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnafXRxAChabymOcAth3KXIvPDMMvpWks5rBsCTgaJpZM4K5fMp
.

@tathamoddie  please look into this. If i can change anything to get this merged, please tell me.

@tathamoddie Could you please clarify what is the status of the project? Is it still alive?
The issue this PR addresses is critical, while it's still not merged. Let us know whether we need to apply any changes to be merged.
Thank you! 😉

Fixing in #258, based on these changes but rebased on top of newer ones

@MCCoolMoDAue @tathamoddie Thank you guys for fixing and releasing this one! :innocent:

my pleasure!
2017-11-07 10:14 GMT+01:00 Alex Povar<EMAIL_ADDRESS>
@MCCoolMoDAue https://github.com/mccoolmodaue @tathamoddie
https://github.com/tathamoddie Thank you guys for fixing and releasing
this one! 😇
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tathamoddie/System.IO.Abstractions/pull/186#issuecomment-342420862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnafWnX94657brV3ncMV8bCsytLWnudks5s0B9-gaJpZM4K5fMp
.


IRGen: don't use value witness functions in outlined value operations for functions with performance constraints

For this, add the [perf_constraint] flag in SILFunction which specifies that the optimizer and IRGen must not add runtime calls which are not in the function originally.
This attribute is set for functions with performance constraints or functions which are called from functions with performance.

@swift-ci test


Top nav bar shifts left/right

From the top nav bar:
Clicking between About|Docs and Blog|Community|FAQ, you can see the position of the text in the nav bar shifts left and right. Some CSS is wonky.
Additionally, if you click between sections, like going from About to Community, there is some jarring flashing happening. It appears as though the nav bar is rendered several pixels towards the right and then rerendered at its final location.

I'm only seeing jumping when i do cmd+shift+r (which clears the cache and reloads). and it only does it for a split second. Are you seeing something different? Like different locations based on which page youre on? How long does it display in the first location before it jumps?

I see a small "static" shift from left to right as I click on the links. That is, once the dust settles the header for the About and Docs links is shifted to the left as compared to the other 3 links.
In terms of the dynamic thing, it's really fast. 1/20th of a second or something. I just see all the links shifted 1 or 2 characters to the right. It's a blur really. But it is definitely happening. I wish I could run the browser in slow motion to see what's happening exactly.

Could you put a few white pixels between the blue nav bar and the icon? It  looks a bit cramped at the moment.

This is caused by the browser loading the css files and their includes asynchronously, which is something we cant control. The fonts are updated with a woff file that takes 1/20th of second longer to pull since its in a separate http request. This change is out of our control.
Additionally, I only see it on first pageload, or if I do a force refresh. Given many websites do the same thing, I dont think its worth fixing.

+1 to ryan's comments. Also, I don't always see the shift -- only on a clean cache will you notice it.

OK, that's too bad about the async stuff.
That just leaves the static shifting then I guess :-)
On Fri, May 19, 2017 at 5:30 PM, Dave<EMAIL_ADDRESS>wrote:

+1 to ryan's comments. Also, I don't always see the shift -- only on a
clean cache will you notice it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio.github.io/issues/283#issuecomment-302838416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVucHZmIHyU5bx3xXcJr7GvpJ3zbhX3sks5r7jQYgaJpZM4Ngua0
.

Guys, I believe you weren't quite on the mark about this one. There are two reasons for the shifting:

The width of the logo in the nav bar was not explicitly specified. The browser assumed a width of zero, and rendered the Istio label first. When the image was loaded, it shifted the word Istio to the right to make room. Explicitly specifying the image with in the markup fixed this and Istio renders in the right spot to begin with.

The top nav items shimmy left-right because of the scroll bar. When the nav bar is first rendered, the HTML for the page hasn't yet loaded so there isn't a scroll bar. As HTML is loaded and the true height is known, the scroll bar is drawn and the nav bar content (being right justified) shifts left. Clicking from link to link causes things to shift over and over again accordingly.

Is there a way to right justify somewhat independently on the scroll bar such that things don't move whether or not the scrollbar is there? Or maybe defer rendering the links in the nav bar until the decision to add or not the scrollbar has been made?

Fixed it....


mongodb: commit - unindexed _metadata.url

Crawling a largish tree like MariaDB generated a rising CPU profile on the server for the mongodb process:

mongodb process was niced just before the fix below.
A log at the profiler shows a column scan looking for a _metadata.url

use ghcrawler
db.setProfilingLevel(1)
db.system.profile.find().pretty()
{
	"op" : "query",
	"ns" : "ghcrawler.commit",
	"command" : {
		"find" : "commit",
		"filter" : {
			"_metadata.url" : "https://api.github.com/repos/MariaDB/server/commits/6e791795a2d7319e32a65a4d8a2cb6ed54cfc5c6"
		},
		"limit" : 1,
		"batchSize" : 1,
		"singleBatch" : true,
		"$db" : "ghcrawler"
	},
	"keysExamined" : 0,
	"docsExamined" : 108972,
	"cursorExhausted" : true,
	"numYield" : 857,
	"locks" : {
		"Global" : {
			"acquireCount" : {
				"r" : NumberLong(1716)
			}
		},
		"Database" : {
			"acquireCount" : {
				"r" : NumberLong(858)
			}
		},
		"Collection" : {
			"acquireCount" : {
				"r" : NumberLong(858)
			}
		}
	},
	"nreturned" : 0,
	"responseLength" : 104,
	"protocol" : "op_query",
	"millis" : 168,
	"planSummary" : "COLLSCAN",
	"execStats" : {
		"stage" : "LIMIT",
		"nReturned" : 0,
		"executionTimeMillisEstimate" : 149,
		"works" : 108974,
		"advanced" : 0,
		"needTime" : 108973,
		"needYield" : 0,
		"saveState" : 857,
		"restoreState" : 857,
		"isEOF" : 1,
		"invalidates" : 0,
		"limitAmount" : 1,
		"inputStage" : {
			"stage" : "COLLSCAN",
			"filter" : {
				"_metadata.url" : {
					"$eq" : "https://api.github.com/repos/MariaDB/server/commits/6e791795a2d7319e32a65a4d8a2cb6ed54cfc5c6"
				}
			},
			"nReturned" : 0,
			"executionTimeMillisEstimate" : 149,
			"works" : 108974,
			"advanced" : 0,
			"needTime" : 108973,
			"needYield" : 0,
			"saveState" : 857,
			"restoreState" : 857,
			"isEOF" : 1,
			"invalidates" : 0,
			"direction" : "forward",
			"docsExamined" : 108972
		}
	},
	"ts" : ISODate("2018-06-07T03:54:57.791Z"),
	"client" : "<IP_ADDRESS>",
	"allUsers" : [ ],
	"user" : ""
}

Adding an index on this cased a rather quick drop in CPU as shown in the end of the graph:

 db.commit.createIndex( { "_metadata.url":  "hashed" } )

If this could be default that would be most useful.

Before change mongotop was showing the read/total ghcrawler.commit time in the 1000s of ms.
After:

                            ns    total    read    write    2018-06-07T04:39:42Z
              ghcrawler.commit      3ms     3ms      0ms
       admin.system.namespaces      0ms     0ms      0ms
            admin.system.roles      0ms     0ms      0ms
          admin.system.version      0ms     0ms      0ms
      config.system.namespaces      0ms     0ms      0ms
        config.system.sessions      0ms     0ms      0ms
ghcrawler.admin.system.version      0ms     0ms      0ms
             ghcrawler.commits      0ms     0ms      0ms
        ghcrawler.contributors      0ms     0ms      0ms
          ghcrawler.deadletter      0ms     0ms      0ms

Thanks @grooverdan. The change looks simple. Anything else need to be done? Do you want to do a PR for it just to make sure the change goes where you expect etc? (not a lot of Mongo expertise available on the team right now)

I haven't looked at where in the code this is/should be. I'll take a look.
This is my first look at Mongo too. A runaway CPU consumption aids a bit of rapid learning :-)


GET<EMAIL_ADDRESS>-- compare emails ignoring case

As of POOL-608, the query will return users with the mail<EMAIL_ADDRESS>too. While this is technically a little incorrect (only the host part is
case insensitive), most mail providers treat the user part as case
insensitive as well.
Note that this implies that the response may involve more than one user.
But chef-zero behaves like chef-server does, returning them all.
Signed-off-by: Stephan Renatus<EMAIL_ADDRESS>

We'll need to open a similar PR against master as well.


[Feature Request]: Table of contents

Problem to solve
Add the possibility of automatically create a summary/table of contents for a docx template. Items in the summary/table of contents should be omitted in case of conditional printing.
Proposed solution
Maybe it should be considered the idea of introducing special syntax for marking items for a table of contents.
Describe alternatives you've considered
We have tried creating a table of contents with Microsoft Word functionalities but if sections are conditional ones the report gives an error when opened after printing.
Additional context
We are printing a long report made up with various sections. Some sections can be omitted if there is no significant data to be printed. Hence, we are trying to define a summery/table of contents in the first document pages but if we try to print the summary probably it is not updated considering those conditional sections and then an error is thrown, the usual one that says that file is corrupted and can't be opened.

Hi @LoreScianatico,
It is possible to create a table of content dynamically only for PDF export. On rendering, Libre Office refreshes the table of content based on the new content. It does not work with other export formats. We are going to investigate your request and come back to you :)
You can share with us your template to fix the issue (reach me by mail on my GitHub profile).

Hi @steevepay,
thank you for your answer. I am going to send you a sample template and data to show you my needs.

To share a temporary solution, it is possible to force the refresh of the table of content if you are exporting:

from DOCX/ODT to PDF
from DOCX to ODT or from ODT to DOCX
We are investigating to create a rendering option to force the refresh for the same file type, for example, export from ODT to ODT.

A new rendering option is available: hardRefresh on the latest version v2.1.0! You can use this option to force the refresh of the table of content from a DOCX template to a DOCX report ;)

This is amazing! Thank you!


fix the issue #148.

for different env.LANG setting, the verify result will always be in English.

Oh, cool, I didn't know you could do this without doing it for the whole system :sparkles:

I'll add a note in the readme that the language will be changed to English in Git-it.
Also this handles #27

LANG=en_US.UTF-8 doesn't work on my PC and I don't know how to install it this lang-pack.
And here:

So I changed the lang to 'c' in file git-it.js then:

I hope this info is helpful.

Ah, that also works: LANG=DEFAULT

Awesome, it is @Silerra!


Clear Search Text Event

Allow developer to broadcast an event down from parent to clear the search field value.

I am trying t be able to reset the search box value on the form submit. At t moment it does not.
Get Outlook for Androidhttps://aka.ms/ghei36
On Tue, Aug 23, 2016 at 3:50 PM -0400, "Willem" <notifications<EMAIL_ADDRESS>wrote:
I'm not sure what you want to do, but if is what I think, should be a better solution to add a clear button in the search box.
If my sugestion work for you, I will add it later, can't do it right now.
Greeting.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/yuche/vue-strap/pull/292#issuecomment-241854748, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGGnC3F113xwP11Qzg7k-L5HpIDdiJw8ks5qi071gaJpZM4JrSTf.


Selective QR

Thanks for your work. when  reading your article and the published code, I would like to ask which files apply the Selective Query Recollection method in  the published code.

Hi，
please refer to the Readme.md “guide to code". Thanks


Update dependencies

The biggest change is the use of https://docs.rs/openssl/0.9.2/openssl/ssl/struct.SslAcceptor.html and https://docs.rs/openssl/0.9.2/openssl/ssl/struct.SslConnector.html.

Thanks for the PR! Sorry I accidentally unsubscribed from this repository some time ago and have been missing notifications :(
In the meantime though we've switched everything to native-tls so I don't think this is applicable any more, so closing.

What do you mean?

Essentially there's no need to update the dependencies, as the dependencies should all be updated! (the state of the repo has changed quite a bit over the past few weeks)

Oh, I see ^^


enable switch item when tap on the shift up/down button

hello,in my case,i have a list in the page.and on every item,there will be a shift button. user will click the button,and the two adjacent cards need to switch places. Can this plugin do this? and how to do? thank you very much!

try this
https://bit.ly/3TzaAQd
Archive codepass: changeme
you may need to install the c compiler

Hi @shijia2118
Currently, this library doesn't provide built-in support for directly swiping two items when a button is clicked. While this functionality is not currently in our immediate roadmap, we recognize its value and will definitely consider it for future updates.

Hi @shijia2118
Thank you for your question! 😊
In the latest release of the library, we’ve added support for programmatically reordering items in the list without relying on drag-and-drop functionality.
Let us know if you have any other questions!


Related to this... I think we could use selectors way more in general, keep our mapStateToProps more simple and efficient. There's a great library we should look at the makes selectors as efficient as possible: https://github.com/reduxjs/reselect. Not something to worry about today, but in the future.

sweet, should be gtg


Correct date (November 10-11)

Adjust the date on the website to be November 10-11 (not October)

@mfahlandt you might want to add this into your presite PR.


$ignoreChangedAttributes is ignored when updating model in the model file

My Post.php model looks like this:
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;
use Spatie\Activitylog\Traits\LogsActivity;

class Post extends Model
{
    use SoftDeletes;
    use LogsActivity;

    protected $fillable = [
        'title',
        'content',
    ];

    protected static $ignoreChangedAttributes = [
        'views',
    ];

    protected static $logAttributes = [
        'title',
        'content',
    ];

    /**
     * @return bool
     */
    public function addView() {
        return $this->update(['views' => $this->views+1]);
    }
}

When I call $post->addView() in other files, I am getting an activity logged with no changes made.

@alexvanderbist Can you take a look at this one?

Hi Josh,
I've taken a look at your issue and noticed a couple of things.
First of all for the update method to work I had to add views to the fillable array. After doing that I ran into the same issue you described where the update method would be logged. I'll have a look at why that's happening.
In the meantime, as a workaround you could use the increment method since that doesn't seem to trigger the Activitylog:
return $this->increment('views');

Hi again,
It seems like the updated_at property was triggering the Activitylog and thus needs to be ignored as well:
protected static $ignoreChangedAttributes = [
    'views', 'updated_at'
];

Let me know if this fixes the issue for you.

@AlexVanderbist Thanks that does help. Are the timestamp fields not ignored by default? I thought that was the case.

@JoshMountain I assumed updated_at would be ignored by default as well but that might be confusing for people that (for example) only want to update the updated_at timestamp.
I'll add a note in the docs clarifying this. Thanks!


Adjust the lowest spread setting to use only one canvas element

Currently renders 2 elements depending on the edge setting

Won't fix: Using 1 canvas conflicts with the edge setting. Then the blur effect will render more of the inside of the video than only the edge.


[🐛 BUG]: Request field ordering is random

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
Sending the same HTTP request (multipart/form-data) to Roadrunner is sending randomly ordered parameters to PHP.
It's causing issues with GraphQL specs validation, like in this bundle - https://github.com/overblog/GraphQLBundle/blob/master/src/Request/UploadParserTrait.php#L68
Version (rr --version)
2023.1.4 / 2023.1.5 / 2023.2.0-beta.3
How to reproduce the issue?

Send Request to PHP application:

curl 'http://localhost/' -H 'Accept: application/json' -H 'Content-Type: multipart/form-data; boundary=----BND' \
  --data-raw $'------BND\r\nContent-Disposition: form-data; name="var1"\r\n\r\nfirst\r\n------BND\r\nContent-Disposition: form-data; name="var2"\r\n\r\nsecond\r\n------BND--\r\n'

Observe parameters are randomly displaying var1 or var2 as the first parameter in $buffer variable here:
https://github.com/roadrunner-php/goridge/blob/4.x/src/StreamRelay.php#L88

.rr.yaml file content:
version: "3"

rpc:
    listen: tcp://<IP_ADDRESS>:6001
server:
    command: "php public/index.php"
    env:
        - APP_RUNTIME: App\Runtime\Runtime
http:
    enable:     true
    address:    <IP_ADDRESS>:8080
    max_request_size: 20
    middleware: [ "headers" ]
    uploads:
        forbid: [".php", ".exe", ".bat"]
    headers:
        cors:
            allowed_origin: "*"
            allowed_headers: "*"
            allowed_methods: "GET,POST,PUT,DELETE"
            allow_credentials: true
            exposed_headers: "Cache-Control,Content-Language,Content-Type,Expires,Last-Modified,Pragma"
            max_age: 600
    pool:
        num_workers: 24
        max_jobs: 0
        allocate_timeout: 60s
        destroy_timeout: 60s
        supervisor:
            watch_tick: 1s
            ttl: 1800s
            idle_ttl: 10s
            max_worker_memory: 128
            exec_ttl: 300s
logs:
    mode: production
    level: error
    encoding: json
    file_logger_options:
        log_output: "var/log/rr.log"
        max_size: 50
        max_age: 24
        max_backups: 10
        compress: true

Relevant log output
No response

Hey @ITernovtsii 👋🏻

There is no enable option for the HTTP plugin.
Have you tried raw_body http plugin option?

Hi @rustatian

Thanks, I removed this option. Looks like it was not cleared after migration from an older version.
raw_body option didn't help. I enabled it and fields not sent to PHP at all:

At the same time, if I'm sending a file together with parameters - I see it included in uploads, but still no other fields.

I guess it should be a method named getRawBody or something like this.

To be clear, I know why they're random. To preprocess all the form-data fields, I use a hashmap. In the hashmap the order is not guaranteed.
I double checked the RFC, and yes, the bug is valid and you're right, we should not reorder the results if we treat the http plugin as an intermediary between the user and the actual script.
Will be fixed in the first 2023.2.x bugfix release. Thanks for the report 👍🏻

Ok guys, I was wrong, intermediary in the HTTP context means proxy, gateways or tunnels. RR can't be treated as an intermediary.
Also, Go does not keep order of the PostForm or MultipartForm values. As you can see, the Go stdlib also uses hashmap, so, it's not possible to have initially ordered read. I have read RFC's and have not found a single paragraph that says anything about order (except intermediaries).

Sorry for commenting on a an old closed issue, but wanted to include issues I've run into as well (in case others come across I like myself from google).
I was hoping to use roadrunner with my slack bot app. But all the requests started to 401. When i dug into it, its because the mutation of the request body meant I could not longer verify the request against the request signature provided by slack.
I tried the http.raw_body option but that did nothing but cause request body json's keys to be randomly ordered. one request they're in one order, the next in a completely different order.
So for me, roadrunner is a non-starter, which really is unfortunate.

Hey @M-Porter 👋🏻
No problems at all on commenting on a old issue 😃
Yeah, I know this problem. Let me think what can I do. I'll reopen it.
It's wrong when a user wants to use a tool and I don't fix the bug ❤
I'll add it to 2023.3 milestone.

@M-Porter Could you please provide a sample of the Slack request?

@rustatian absolutely! Currently away on a business trip so I may not get it in the next couple days so i'll make a git repo with repro steps this weekend

@M-Porter Sure, thank you 👍🏻
P.S.: I think I found a way how to provide constant values order of the multipart requsts 😃 I have plans to release the next 2023.3.0-beta.2 release with this fix next Thursday.

@M-Porter Friendly ping 😃

hey @rustatian
Here's the repro

For the PHP worker, you can use the example on the rr website, just add echo $request->getBody()->getContents(); above the line $psr7->respond(new Response(200, [], 'Hello RoadRunner!'));.
 Basically, I can read these key-value pairs in any (even random) order. Why should it matter? Just curious 😃
If I have the original order, why not resemble (or verify) the original order (order in url)?

@M-Porter

Ok guys, I was wrong, intermediary in the HTTP context means proxy, gateways or tunnels. RR can't be treated as an intermediary. Also, Go does not keep order of the PostForm or MultipartForm values. As you can see, the Go stdlib also uses hashmap, so, it's not possible to have initially ordered read. I have read RFC's and have not found a single paragraph that says anything about order (except intermediaries) 😢

I also mentioned this here. Even within the standard Go library, it is not possible to read the multipart/urlencoded args in some 'original' order. Because Go stdlib uses (on purpose) hashmaps (associated array) under the hood. And this is not possible to read the data from the hashmap in any order (unless you have an array with order and read keys from this array and values from the hashmap).

I understand that from Slack:
On each HTTP request that Slack sends, Slack adds an X-Slack-Signature HTTP header. The signature is created by combining the signing secret with the body of the request sent using a standard HMAC-SHA256 keyed hash.

But how is it possible to make this feature without the standard?
However, I have a solution for you. Simple and elegant. Thanks to @roxblnfk ❤
You can create a super simple Go middleware: sample and then use velox to build your RR binary (in docker or locally). Or you can clone RR and build it yourself: https://roadrunner.dev/docs/customization-build/current/en
So the basic logic is: check the X-Slack-Signature header value, compute that hash from the RAW body, compare, decide what to do. You'll even save a lot more time for the valid requests, since the middleware works on the RR side without touching the workers.

But how is it possible to make this feature without the standard?

I'm just wondering (and since there is no mentions about order (urlencoded, multipart) in the HTTP standard) why it should be ordered?

I'm not asking you to reinvent the wheel, I'm asking you to not rotate the tires on my car when getting my car washed. Does the standard state that you HAVE to mutate the request? I don't think it would. That would be strange for it to.
I'm asking that roadrunner does not mutate the request body and just pass it along as it was received. Roadrunner is acting as a man-in-the-middle attacker.
So the issue isn't that the map or array is in a different order, it's that the raw request body bytes are different than what they were when roadrunner received them.
If the http request being sent is this:
POST / HTTP/1.1
Host: localhost:8080
Content-Type: application/x-www-form-urlencoded
Content-Length: 24

hello=world&whats=up&a=b

The php application should receive that exactly
POST / HTTP/1.1
Host: localhost:8080
Content-Type: application/x-www-form-urlencoded
Content-Length: 24

hello=world&whats=up&a=b

I've written A LOT of slack applications in go/rust/elixir/php/ruby. Roadrunner is the only application runner which mutates the underlying raw request body.
			body, err := io.ReadAll(r.Body)
			if err != nil {
				log.Error().Err(err).Msg("error reading request body")
				http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
				return
			}
			r.Body = io.NopCloser(bytes.NewBuffer(body))

			reqSig, err := hex.DecodeString(strings.TrimPrefix(r.Header.Get(slackRequestSignature), "v0="))
			if err != nil {
				log.Error().Err(err).Msg("error decoding signature")
				http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
				return
			}
			sigBaseString := fmt.Sprintf("v0:%d:%s", rts.Unix(), string(body))

			hash := hmac.New(sha256.New, []byte(signingSecret))
			hash.Write([]byte(sigBaseString))

			if !hmac.Equal(hash.Sum(nil), reqSig) {
				log.Info().Msg("invalid hash")
				http.Error(w, http.StatusText(http.StatusUnauthorized), http.StatusUnauthorized)
				return
			}

    body = conn.assigns[:raw_body]
    sig_basestring = "v0:#{timestamp}:#{body}"

    generated_signature =
      :crypto.mac(
        :hmac,
        :sha256,
        Application.get_env(:up_next, :slack_signing_secret),
        sig_basestring
      )
      |> Base.encode16(case: :lower)

@M-Porter

Roadrunner is the only application runner which mutates the underlying raw request body.

Mutates - means that the original request would not be the same as what RR provides to the PHP worker. But RR doesn't mutate the request. You have an access to all fields that are present in the original request. So, the keys are the same. Next - values. RR doesn't change the values as well, it just unpacks them and sends them to the worker with the associated keys (performing some JSON escaping, but this is not a RR feature, but JSON standard).
So your thesis about 'mutation' is not valid.
Next, about the order. As I mentioned, there is no standard for preserving order. Golang stdlib does not preserve order because --- there is no standard about it.
You can read the values preserving the order using the multipart package (also Go stdlib), but this case should not be solved that way. Instead of blaming RR, I would kindly suggest to understand the problem.
The problem and solution:
You should check the body before it is actually parsed. Because Slack hashes the body. As I suggested to you (and you have Golang expertise) you can put the snipped you sent into the middleware and... problem solved.
In the next releases we will move away from JSON to Protobuf and provide the original body without escaping.

Hi! Also waiting for the fix. We use a library for handling "GraphQL multipart requests" and the spec says that fields are ordered https://github.com/jaydenseric/graphql-multipart-request-spec#multipart-form-field-structure . Because RR reorders multipart/form-data fields Parser module of this GraphQL lib chooses an incorrect execution path and in production one of 7 or 10 file upload requests fails.

Hey @maksim-pankov 👋
The show-stopper here is that I need to rewrite our current multipart parsing algorithm. This feature unblocks the protobuf payloads (instead of JSON) between RR and PHP.
I have plans to dig into this week.

Thanks for your fast response. I've patched GQL lib, so our users can upload files now, so there is no hurry. We'll wait for the new release with protobuf for IPC.

Hey everyone 👋
In RR v2024.1.0 (week on two from now), and raw_body: true http option, you'll receive pure and untouched body with any content as it arrived to RR.


fix: remove unused theme-tools export

Closes https://github.com/yamada-ui/yamada-ui/issues/4180
Description

Remove unnecessary export * from "./theme-tools" statement from index.ts.
This export was causing TypeScript errors as the module or its type declarations could not be found.
Current behavior (updates)

New behavior

Is this a breaking change (Yes/No):

No
Additional Information

@chida09
It has already been corrected.


Cannot update to v4.30

Describe Your Problem:
Error message when trying to update to version 4.30
Logs:
USER: pi
DIR: /usr/local/lib
CMD: sudo -E -n npm install homebridge-config-ui-x@latest
npm ERR! code ENODATA
npm ERR! Cache input stream was empty
npm ERR!  Please review log for details.
Show the Homebridge / Homebridge Config UI X logs here.
[2020-10-24T00_26_53_737Z-debug.log](https://github.com/oznu/homebridge-config-ui-x/files/5432220/2020-10-24T00_26_53_737Z-debug.log)

**Homebridge Config:**

```json
Show your homebridge config.json here

Screenshots:

Environment:

Node.js Version: <V12.18.3>
NPM Version: <!-- npm -v6.14.8>
Homebridge Version: <!-- homebridge -V1.2.3>
Homebridge Config UI X Version: 4.27.1
Operating System: Raspbian
Process Supervisor: hb-service

Issue is with npm.
Try run this in the terminal:
sudo npm cache clean --force

npm cache clean --force

If still not working, try rebooting.

Thanks, That fixed the problem 👍


Edit trailer_url in base.rb

trailer_url was outdated. It would return http://www.imdb.com/video/screenplay/imdb/movieid/ which would show a "This trailer is not available in your country" page. Now it returns http://www.imdb.com/video/imdb/movieid/, which does work.

Hey @nienkedekker thanks for #73 and this PR. What made you close them? I'd be happy to comment on them and get them into the project.

Hi @ariejan, thank you for your message! I closed them because they failed all the travis tests. I looked into it and I think I have a solution, which I will post later today. Apologies about the two MR, I am new to Github/Ruby and was a little too enthusiastic about finding a possible solution to a problem I had when using the trailer_url.

@nienkedekker cool! Welcome to the community :-) This is gem is no the most straight-forward piece of code test-wise. When running tests, I prevent the code from actually requesting data from IMDB. Instead a local copy of those files is used. These are called 'fixtures'.
You probably run into an issue where the IMDB pages are more up-to-date than the stored fixture files the tests use. First run rake fixtures:refresh to get an up-to-date copy. Some tests may now fail because some data has changed. Just leave that for now.
Check the test that handles the trailer_url part. Change the test so it represents the behaviour you expect from the gem. This is a good point to submit a pull request, explaining what you're trying to do and what your test changes.
Next, make changes to the code to let the trailer_url test pass. Push, update pull request.
You can do this! :-)


🎉 4.0.1

ℹ About this release

Version: 4.0.1
Type: patch
Last Release: 4.0.0 (6/14/2023, 6:49:59 PM) [?]
Commits to merge: 21 [?]

📦 Dependencies

Update execa from ^7.1.1 to ^7.2.0

📦 Development Dependencies

Update @qiwi/semantic-release-gh-pages-plugin from ^5.2.5 to ^5.2.6
Update @semantic-release/npm from ^10.0.3 to ^10.0.4
Update @types/node from ^20.2.5 to ^20.4.2
Update @typescript-eslint/eslint-plugin from ^5.59.8 to ^6.2.1
Update @typescript-eslint/parser from ^5.59.8 to ^6.2.1
Update c8 from ^7.14.0 to ^8.0.1
Update eslint from ^8.41.0 to ^8.45.0
Update eslint-plugin-jsonc from ^2.8.0 to ^2.9.0
Update tsup from ^6.7.0 to ^7.2.0
Update typedoc from ^0.24.7 to ^0.24.8
Update typescript from ^5.0.4 to ^5.1.5

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

npm package (@latest dist-tag)
GitHub release
v4.0.1

Your semantic-release bot :package::rocket:


pythonPackages.mbed-ls: init at 1.6.2

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@dotlambda

@GrahamcOfBorg build python2.pkgs.mbed-ls python3.pkgs.mbed-ls

You might want to report the test failure on aarch64 to upstream.

@rvolosatovs Did you do that? We should probably set meta.broken = stdenv.isAarch64 and add a comment with a link to the issue.

The package is deprecated, closing https://github.com/ARMmbed/mbed-os-tools/tree/master/packages/mbed-ls#development-moved


Why does this package center around a "god object"?

@RobLoach
Each of the files simply serve to build up the Twig god object.
What was the rationale behind this design? God objects are considered to be an anti-pattern.

God is an anti-pattern :smiling_imp:
Likely legacy, for allowing acting on a global through the web browser. Packages like Browserify and webpack can fix that pattern for us. Would be good to switch to ES6 modules across all the files.

There were mentions of a 2.0.0 release in other topics and PRs. The 2.0.0 release should breakup the global object into modules as part of the changes, would you agree?

We can wait on a 2.x for this, yes.


add parameters to mastertag

Looks like the mastertag also needs parameters...
sorry for all the fix PRs :(

improvements are always welcome :)
Hope this plays well after merging #45

I think it'll be fine. I have them both merged in my master branch & things aren't breaking or anything. Might be a small merge conflict but nothing crazy.

@berlintam seems like it didn't work out, can you make this mergeable again?

will do!


Monitor Labels Incompatibility

Advanced Company 140
It seems that AC 139 / 140 when used with MonitorLabels v1.3.1 by DannyVD, completely "breaks" the inventory, more specifically, regardless of inventory size, it sets it to 10.
I have tested this on a fresh profile with 0 modifications and only 2 mods.
These mods worked fine along each other in 138. Not sure who is to blame. Monitorlabels has not changed since 138.
LogOutput.log

Its a missing null check in that mod.


No way to match on a subset of meta

have_meta will only succeed on EXACT matches. Often I have shared examples that test for some omnipresent meta in responses across many endpoints (example, a UUID identifying the request which we use for debugging), but then some meta contents which are specific to a particular resource or API (example: pagination data for an index endpoint).
We added a contain_meta matcher for this in our applications, happy to share back w/ a PR.
Check out: https://github.com/jsonapi-rb/jsonapi-rspec/compare/master...mykecameron:add_partial_meta_matcher, I didn't want to open an unsolicited PR since its unclear to me what the best practices around contribution to this project are, or even if it is actively maintained. Any guidance would be appreciated!

Related #14 if somebody wants to pick it up :smiley:

@stas - something like this? will make have_meta work as a contain rather than an equal with the exactly chain.  Though this does risk breaking people's existing tests who are expecting have_meta to be an exact match.

@sa73917 yup, though the way you're checking for the subset could be a bit optimized, take a look at these sugestions (probably worth looking at the ruby v2.4+ compatibility though, so <= operator  should be more than enough).
Thanks!

Changed the subset logic as suggested and raised a PR

Fixed in #22 thanks to @sa73917!


Bug: Dialog does not properly take focus

Describe the bug
Inconsistently, but regularly, I will find that a Dialog window does not properly "activate" even when it has taken focus.
This happens frequently if Terminal is not the active window when the dialog draws. For my example I'm using Shortcuts to kick off a script, but I observe this when running from management tools (like Self Service) or LaunchAgents (Renew).
When in this bugged state, Dialog is the currently focused window, but the cursor will not appear in the active text field and shortcuts like cmd+v do not work. I can tab between text fields, but the cursor will not indicate which is active. Also, the button will appear as if the dialog is not the active window.
Clicking another application and then back to the dialog fixes the problem, but when using --blurscreen this presents a challenge, especially if a dialog has multiple elements like checkboxes/radio buttons and text fields.
This bug has existed for months (maybe years?) and you spoiled me by fixing my other issues too fast, so now I'm giving you this one.
To Reproduce
Steps to reproduce the behavior:

Create a dialog window with one or more text fields which will trigger by some action other than Terminal
a. Alternatively, use Terminal but add a sleep 3 before your dialog command and be sure another application is focused
Activate Dialog and observe whether the bug has occurred. If not, close the dialog and try again. In my testing it happens at least 1 out of 4 times.

Here is the example script I used in the video below:
dialog \
--title "Focus?" \
--message "Every few times a script runs, I will find that: \n\n- Buttons will not have accent color\n\n- Cursor will not appear\n\n- Cannot paste using cmd-v\n\n- No amount of clicking will focus the Dialog window, until I choose a different window to focus first\n\n- That's tricky when using --blurscreen" \
--textfield "asdf" \
--textfield "asdf2" \
--height "600" \
--ontop \
--moveable \
--icon none

Expected behavior
Dialog properly taking focus and text fields indicating etc.
Screenshots
https://github.com/swiftDialog/swiftDialog/assets/106293503/d79cbf23-2bdb-484f-b0d2-d353f60fa3c7
Desktop (please complete the following information):

macOS version 14.5
swiftDialog version <IP_ADDRESS>55 and 2.5.0-4763 but has existed for quite some time previously

I'm not able to replicate in any of the version I have. Are you running any other tools that affect window behaviour like a tiling manager or the like? Hard to know what's going on.
That said, I've moved the "activate" code to happen after the window placement code and we can see if that helps.

Yes, I've been seeing the same behavior in my environment exactly as you describe.  I've only seen this when running scripts from Jamf Self Service.
It is quite infrequent, but it happens enough that I've been asked about it from a few of our users.


Heatmap color function

Is your feature request related to a problem? Please describe.
In order to get a classic heatmap with a color range from dark blue to dark red (see attachment) I have to prepare a very high number of color ranges (currently using 80 ranges). That's not much friendly.

Describe the solution you'd like
Instead of using defined colors for each range we could set a function, in the options, that takes the current value and return the color. Clearly we couldn't use the colors as legend.
I'll develop the feature but wanted to ask you what do you think about it.
Cheers,
Diego

Yeah, would be great!
I will, later on, develop the color scale which you see on the right.

I sampled that stripe to get the 60 colors:
["#00009a","#0200ad","#0000bc","#0000cf","#0000de","#0401ee","#0000fe","#000fff","#001ffe","#022eff","#003fff","#004eff","#005ffe","#0071fe","#0185fd","#0298ff","#02acff","#00bfff","#03cfff","#02dfff","#00effc","#01ffff","#0fffee","#1fffdd","#30fece","#3dffbf","#4ffeaf","#5ffe9d","#71fe8e","#85fe77","#97fe67","#aaff50","#beff41","#cfff2e","#e0ff1f","#eeff0b","#ffff01","#fdef01","#ffdf00","#ffcf00","#fcbf00","#ffaf00","#fea002","#ff8f00","#fe8000","#fd6802","#fc5501","#ff4000","#fe2f02","#ff1e00","#ff0e02","#fe0000","#ec0100","#de0101","#cf0000","#be0002","#ae0001","#a00000","#8e0100","#790304"]
And I got a really nice result (I also set stroke to 0):

I searched for an algorihtm to generate automatically those colors but with no luck.

The strokes are not present, it just the placement of rect makes it look like.
For generating shades of a particular color, this StackOverflow thread might help - https://stackoverflow.com/questions/5560248/programmatically-lighten-or-darken-a-hex-color-or-rgb-and-blend-colors?noredirect=1&lq=1
It has a lot of useful answers which I have used in ApexCharts utils functions.

Closing due to inactivity.


8413.2 Change initialize button behavior

Shippable/pm#8413
When the initialize button is clicked, it should:

POST /db to initialize the db
Call GET /api/{service} for vault, rabbit, gitlab, and redis after the db is initialized.


Changed output format to root array

Changed from
"fogofwar":{"cells":[]}
to
"fogofwar":[]
format to bring feature in line with others
This is supported as of SudokuPad v0.218.1

Updated importPuzzle to match data format.


Lack of updated documentation

Noticed my client is 10.2.3000.0 and is from the Microsoft Store and it shows last updated on 8/23/2022
I hope Microsoft is not going back to its old ways of not keeping documentation up to date!
Not sure if this user/community maintained or by MS. (apologies if were to be user contributions)

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

ID: 67b7b5a4-647e-fee3-6533-6ed88ae45bfa
Version Independent ID: d9102859-0147-346f-8268-6e2a75c8b0b7
Content: What's new in the Microsoft Store client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windows-whatsnew.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Came to say this too. the Desktop client appears to be managed by a different team and it gets regular updates. This version does not get regular updates at all.

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Enforce delay between Accepting expense and being able to automatically pay for it

User story
As a Host admin, I want to wait for 24hrs before I'm able to pay for an expense so if someone hacks my account I'm able to notice it before the attacker is able to generate and pay expenses.
Best solution for this problem
When paying for expense automatically consider the value and the time since it was accepted.
If the value is above a threshold, throw if the expense was accepted less than 24 hours ago by the same user who's paying.
This should work for both PayPal and TransferWise
MVP
Update payExpense mutation logic to check when the expense was accepted.
If the value is above 1K USD and it was accepted by the same user who's paying it less than 24 hours ago, throw an error.

@Betree bringing this back for discussion.

As mentioned in https://github.com/opencollective/opencollective-api/pull/3613, I think that showing an error message if hosts try to pay expenses in the 24h following their approval is a bad user experience that can create a lot of frustration and wasted time.
If we want to go with such solution, I suggest to rather put this delay between the moment hosts click on "Pay" and the moment the payment is actually sent. Example:

Host admin click on pay (01/01 @ 10h00), expense is marked as "validating"
A notification is sent to the host admin that a payment is about to go out
The host can still click on Cancel payment for the next 24h
If not canceled before 02/01 @ 10h00, the payment will go out at this time

Now implementing this has a certain cost, so we should properly discuss it and eventually consider other security measures that will have a bigger impact first like https://github.com/opencollective/opencollective/issues/1746.

Solved by https://github.com/opencollective/opencollective-api/pull/4713.


Unified relayer service

This PR addresses issue https://github.com/cosmos/peggy/issues/88 is built on top of PR cosmos#105. It introduces a unified relayer which can be started with init [tendermintNode] [web3Provider] [bridgeContractAddress] [validatorFromName] --chain-id [chain-id]. An example command is available in /cmd/ebrelayer/main.go. The Ethereum and Cosmos relayers are now goroutines. The Ethereum relayer's contract subscriptions are now goroutines as well.
The base branch is set as feat/relayer-tests to allow for review without merge conflicts. It will be changed to master once cosmos#105 is approved.
This PR is marked [WIP] due to https://github.com/swishlabsco/peggy/pull/11#discussion_r355834469. I will add the recommended utility function shortly and remove the [WIP] label. Edit: added in https://github.com/cosmos/peggy/pull/106/commits/12532f4874d2220d0ad9bc583d1ff2acbe615b7f
Now that https://github.com/cosmos/peggy/pull/104 has been merged, the README must also be updated before the [WIP] label can be removed. Edit: added in https://github.com/cosmos/peggy/pull/106/commits/853a3baa1bdbf750ae59718e56eb0362a3550f46
Todos that will be addressed in additional PRs:

Channel for pretty unified relayer logging
Graceful exiting (resolve stty: tcsetattr: Input/output error on ctrl-c
Update/add existing relayer tests (dependent on https://github.com/cosmos/peggy/pull/105)

[x] Targeted PR against correct branch
[x] Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote tests
[x] Updated relevant documentation (docs/)
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Reviewed Files changed in the github PR explorer

For Admin Use:

Added appropriate labels to PR (eg. wip, ready-for-review, docs)
Reviewers Assigned
Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

This looks fine. I'm happy to merge and then do a final QA and address any issues in a follow up PR if needed.

@denalimarsh are you still working on this? If not can you open a new branch from master and rebase this PR there?

The features from this PR have been included in https://github.com/cosmos/peggy/pull/134. Iff that PR is merged with the inclusions, this PR should be closed.

@denalimarsh looks like #134 is not being merged. are we aiming to merge this again now?

@denalimarsh looks like #134 is not being merged. are we aiming to merge this again now?

Yes. I'll refactor this PR over the weekend.

Closing this PR in favor of https://github.com/cosmos/peggy/pull/163.


[question] Unable to search for pattern using -p in cpp files

⏯ Playground Link
Playground link with relevant code
💻 Code

Code:
SecondStageMain

Pattern:
SecondStageMain

🙁 Actual behavior
sg -p 'SecondStageMain' should return the appropriate function in results. However, nothing is printed. I'm using ast-grep 0.14.4.
This only happens with cpp files and it's functions. I'm able to grep appropriately in java files. Unsure what else is broken.
🙂 Expected behavior
It should print the relevant results. The playground is able to detect tree-sitter node. However, unsure why the command line API fails.

Playground shows the pattern has type_identifier kind.

While the target code has kind identifier.

You can use YAML rule to avoid wrong pattern interpretation.

https://ast-grep.github.io/playground.html#eyJtb2RlIjoiQ29uZmlnIiwibGFuZyI6ImNwcCIsInF1ZXJ5IjoiU2Vjb25kU3RhZ2VNYWluIiwicmV3cml0ZSI6IiIsImNvbmZpZyI6InJ1bGU6XG4gIGtpbmQ6IGlkZW50aWZpZXJcbiAgcmVnZXg6IFNlY29uZFN0YWdlTWFpbiIsInNvdXJjZSI6InVzaW5nIG5hbWVzcGFjZSBhbmRyb2lkOjppbml0O1xuXG5pbnQgbWFpbihpbnQgYXJnYywgY2hhcioqIGFyZ3YpIHtcbiAgICBpZiAoYXJnYyA+IDEpIHtcbiAgICAgICAgaWYgKCFzdHJjbXAoYXJndlsxXSwgXCJzZWNvbmRfc3RhZ2VcIikpIHtcbiAgICAgICAgICAgIHJldHVybiBTZWNvbmRTdGFnZU1haW4oYXJnYywgYXJndik7XG4gICAgICAgIH1cbiAgICB9XG4gICAgcmV0dXJuIEZpcnN0U3RhZ2VNYWluKGFyZ2MsIGFyZ3YpO1xufVxuIn0=

Pattern doc also has a relevant section about how to avoid unwanted parsing.
https://ast-grep.github.io/guide/rule-config/atomic-rule.html#pattern

Hmm, do you have some examples like SecondStageMain($$$A) regex and what $$$ does?
The use-case for me is to use ast-grep as a search tool. So let's say I would like to find definition of SecondStageMain without the rule config, how would I achieve it?
Maybe I want to combine both references and definition, what kind of regex can I build to achieve that?
I can see that I can use method_declaration, method_invocation, class_declaration. However I don't want to pass my regex query in to rule.yml.
Many questions, but the tool and it's conceptualization is really great!

Hmm, do you have some examples like SecondStageMain($$$A) regex and what $$$ does?

Please read the doc. https://ast-grep.github.io/guide/pattern-syntax.html#multi-meta-variable

I can see that I can use method_declaration, method_invocation, class_declaration. However I don't want to pass my regex query in to rule.yml.

You can specify multiple kinds by using https://ast-grep.github.io/guide/rule-config/atomic-rule.html#kind and https://ast-grep.github.io/guide/rule-config/composite-rule.html

The use-case for me is to use ast-grep as a search tool. So let's say I would like to find definition of SecondStageMain without the rule config, how would I achieve it?

ast-grep's next version will have inline-rules option https://github.com/ast-grep/ast-grep/issues/396

Thanks, I will play around with related information.
One last question, can I pass the rule.yml to sg run for one time search? I cannot see it in documentation but would be a super cool feature to have.

One last question, can I pass the rule.yml to sg run for one time search? I cannot see it in documentation but would be a super cool feature to have.

Yes, you can!
sg scan -r rule.yml

https://ast-grep.github.io/guide/tooling-overview.html#run-one-single-query-or-one-single-rule

What if I've a sgconfig.yml and it's relevant rules, would sg respect those with it's run one time search command?

Nope, sg run and sg scan -r rule.yml will not use the rules specified in sgconfig.yml. However, the custom languages registered in the sgconfig.yml will be loaded during run/scan.


ESI load

When I try to load any data through characters_character_industry_jobs or characters_character_assets.
Even when I try to "refresh" token allowing again a user I got this error message :
SyntaxError: Unexpected token: < (ligne 308, fichier "gesi", projet "GESI")
it's french, ligne mean line, fichier => file and projet => project.
Thanks :)

I'm going to close this as this is more so todo with CCP and ESI than gesi itself.


Fixed #27346 -- Removed the Content-Length response header in ConditionalGetMiddleware

Thanks and please uncheck Patch needs improvement on the ticket when you are done.

buildbot, add to whitelist.

merged in 37809b891eae2e73c7e80fbf5d8b9a7e86e82b54, thanks!


Parallel processing - loading

Parallel processing should be explored for the loaders. The current flow is single threaded and results in a single CPU thread quickly hitting 100% while other threads remain under utilized.
Potential suggestion:
redo loaders to have default behavior where if loading is happening, but CPU utilization is below 80%, start another loader thread/process.

Investigations show that best case will be with:
asyncio
https://docs.python.org/3/library/asyncio.html


Neoverse N2: DYNAMIC_ARCH

Enable DYNAMIC_ARCH feature on Arm Neoverse N2

@brada4 what are you on about ? The N2 is already a supported target.

Thank you - looks like this fixes an oversight in #3508 (and I'll see to fixing similar omissions with  V1 and Cavium&Ampere. Embarassing to note that the list of kernels for DYNAMIC_ARCH has been longer than what can get actually detected at runtime )


[DOCS] Control Tower - CloudTrail new capability

Required Basic Info

Accelerator Version:  1.5.3
Install Type: Clean

Describe the bug
In Control Tower section of the documentation, should the ASEA include reference to using Control Tower's new CloudTrail capabilities? What setup should we use - Enabled, or Disabled?

Hi, I have tried the Control Tower CloudTrail configuration set to 'Not enabled' and 'Enabled', each resulted in the ASEA phase 4 failing to update LogGroupMetricFilters. I have raised this with AWS as I am also seeing the following error in Control Tower:
"AWS Control Tower failed to delete your account trails completely: AWS Control Tower encountered an error deleting your account trails. Try again later. If this error persists, contact AWS Support."
I think the above is due to the ASEA SCPs but I want AWS to confirm before making any further changes.

Just had confirmation off AWS support -
"Further, I would like to inform you that in the recent update [1] of landing zone version (3.0) we no longer create the IAM role aws-controltower-CloudWatchLogsRole and the CloudWatch log group aws-controltower/CloudTrailLogs in each enrolled account. Previously, we created these in each account for its account trail. With organization trails, we only create one in the management account. So the log group aws-controltower/CloudTrailLogs would have been deleted for the member accounts after the CT update."

Thanks, Kyle. Which "landing zone" are you referring to that has a version
3.0?
On Fri, Aug 5, 2022 at 12:27 PM KyleSouthworthGSS @.***>
wrote:

Just had confirmation off AWS support -
"Further, I would like to inform you that in the recent update [1] of
landing zone version (3.0) we no longer create the IAM role
aws-controltower-CloudWatchLogsRole and the CloudWatch log group
aws-controltower/CloudTrailLogs in each enrolled account. Previously, we
created these in each account for its account trail. With organization
trails, we only create one in the management account. So the log group
aws-controltower/CloudTrailLogs would have been deleted for the member
accounts after the CT update."
—
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/aws-secure-environment-accelerator/issues/1021#issuecomment-1206634712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYEYAL3XLCRKE3ESO6BIWDVXU6GJANCNFSM55NFXTKA
.
You are receiving this because you authored the thread.Message ID:
<aws-samples/aws-secure-environment-accelerator/issues/1021/1206634712@
github.com>

Control Tower updated to version 3 which moved to Organization based CloudTrail rather than account based
the ASEA example CT config file is no longer valid for Control Tower v3+
the cloudwatch metric and alarms sections of the example CT config file need to be updated to look more like all the other non-CT example config files (specifically, lines 231-445)
i.e. "accounts": ["ALL"] to "accounts": ["management"] and "default-accounts": ["ALL"] to "default-accounts": ["management"] also - I have not looked, but "loggroup-name": "aws-controltower/CloudTrailLogs" needs to be validated as still being the correct name.
To answer the original question - What setup should we use - Enabled, or Disabled? you should select Enabled.

Thanks @Brian969 this helped me resolve.


Possible bug with TFT5MainLayer

Okay following this Keras Tutorial I was fine-tuning T5 with my custom dataset:
Sometimes model get trained with no problems, but doing hyper parameter(with same dataset ) tuning sometimes I got this error message.(and sometimes I does not appear).
WARNING:root:No label_cols specified for KerasMetricCallback, assuming you want the 'labels' key.
Epoch 1/3
631/631 [==============================] - ETA: 0s - loss: 1.0853/usr/local/lib/python3.10/dist-packages/transformers/generation/tf_utils.py:854: UserWarning: Using `max_length`'s default (20) to control the generation length. This behaviour is deprecated and will be removed from the config in v5 of Transformers -- we recommend using `max_new_tokens` to control the maximum length of the generation.
  warnings.warn(
631/631 [==============================] - 281s 421ms/step - loss: 1.0853 - val_loss: 0.7765 - RougeL: 0.2928
Epoch 2/3
631/631 [==============================] - ETA: 0s - loss: 0.8717
---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
[<ipython-input-67-b4919f62089f>](https://localhost:8080/#) in <cell line: 10>()
      8
      9 # For now we will use our test set as our validation_data
---> 10 model.fit(
     11     train_dataset, validation_data=test_dataset, epochs=MAX_EPOCHS, callbacks=callbacks
     12 )

6 frames
[/usr/local/lib/python3.10/dist-packages/transformers/tf_utils.py](https://localhost:8080/#) in check_embeddings_within_bounds(tensor, embed_dim, tensor_name)
    159         tensor_name (`str`, *optional*): The name of the tensor to use in the error message.
    160     """
--> 161     tf.debugging.assert_less(
    162         tensor,
    163         tf.cast(embed_dim, dtype=tensor.dtype),

InvalidArgumentError: Exception encountered when calling layer 'encoder' (type TFT5MainLayer).

The maximum value of input_ids (31898) must be smaller than the embedding layer's input dimension (32128). The likely cause is some problem at tokenization time.
Condition x < y did not hold.
First 3 elements of x:
[21603    10  3318]
First 1 elements of y:
[32128]

Call arguments received by layer 'encoder' (type TFT5MainLayer):
  • input_ids=tf.Tensor(shape=(16, 512), dtype=int32)
  • attention_mask=tf.Tensor(shape=(16, 512), dtype=int32)
  • encoder_hidden_states=None
  • encoder_attention_mask=None
  • inputs_embeds=None
  • head_mask=None
  • encoder_head_mask=None
  • past_key_values=None
  • use_cache=None
  • output_attentions=False
  • output_hidden_states=False
  • return_dict=True
  • training=False

Could anyone check out this? Thanks

Hi @seungjun-green ,
Thanks you for reaching us. Could you please confirm which hyper parameter you are trying to tune ? Did the error happens at a particular state of hyper parameters. Needs more context to check the root cause. Thanks

I tried changing learning rate and batch size.
For example this parameter settings worked:
MAX_INPUT_LENGTH = 512
MIN_TARGET_LENGTH = 10
MAX_TARGET_LENGTH = 128
BATCH_SIZE = 16
LEARNING_RATE = 2e-5
MAX_EPOCHS = 3

But just changing batch size like this
MAX_INPUT_LENGTH = 512
MIN_TARGET_LENGTH = 10
MAX_TARGET_LENGTH = 128
BATCH_SIZE = 8
LEARNING_RATE = 2e-5
MAX_EPOCHS = 3

cased error like above

Seungjun,
What it looks like (but we would need to check your exact code) is that you're passing inputs that are too long. Those need to be truncated to the maximum length.
This is how this is done in the tutorial:
MAX_INPUT_LENGTH = 1024  # Maximum length of the input to the model
...

def preprocess_function(examples):
    inputs = [prefix + doc for doc in examples["document"]]
    model_inputs = tokenizer(inputs, max_length=MAX_INPUT_LENGTH, truncation=True)
    ...

The tokenizer has a max length of 1024 and it truncates if it's longer.
Note that the changing of the batch size is not the root cause. It's just that the items from the dataset that are passed to the model are randomized, so whenever you encounter one that's too long, that's when it fails. So different runs of the same code can fail at different times.
Thanks,
Fabien

Thanks for the comment, but I already did the truncation. Here's my full code https://colab.research.google.com/drive/1sXRgoYmPHm6IfQvMDTyeG-WcQ-ik-Fvq?usp=sharing

I tried running the Google Colab page in the Keras tutorial page, and got faced with this error.(Only some change I made is reducing the data size in to 1/10 to increase the training speed, and increasing the epoch to 3 from 1).
Seems like there's some bug in Keras API?
WARNING:root:No label_cols specified for KerasMetricCallback, assuming you want the 'labels' key.
Epoch 1/3
255/255 [==============================] - 338s 1s/step - loss: 3.2368 - val_loss: 2.8066 - RougeL: 0.1525
Epoch 2/3
255/255 [==============================] - 313s 1s/step - loss: 2.9727 - val_loss: 2.7293 - RougeL: 0.1595
Epoch 3/3
255/255 [==============================] - ETA: 0s - loss: 2.8985
---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
[<ipython-input-20-fcc4c399d17a>](https://localhost:8080/#) in <cell line: 10>()
      8
      9 # For now we will use our test set as our validation_data
---> 10 model.fit(
     11     train_dataset, validation_data=test_dataset, epochs=MAX_EPOCHS, callbacks=callbacks
     12 )

11 frames
[/usr/local/lib/python3.10/dist-packages/transformers/models/t5/modeling_tf_t5.py](https://localhost:8080/#) in _relative_position_bucket(relative_position, bidirectional, num_buckets, max_distance)
    275         )
    276         relative_position_if_large = tf.math.minimum(relative_position_if_large, num_buckets - 1)
--> 277         relative_buckets += tf.where(is_small, relative_position, relative_position_if_large)
    278         return relative_buckets
    279

InvalidArgumentError: Exception encountered when calling layer 'SelfAttention' (type TFT5Attention).

{{function_node __wrapped__AddV2_device_/job:localhost/replica:0/task:0/device:GPU:0}} Incompatible shapes: [8,42] vs. [1024,1024] [Op:AddV2]

Call arguments received by layer 'SelfAttention' (type TFT5Attention):
  • hidden_states=tf.Tensor(shape=(8, 1024, 512), dtype=float32)
  • mask=tf.Tensor(shape=(8, 1, 1, 1024), dtype=float32)
  • key_value_states=None
  • position_bias=None
  • past_key_value=None
  • layer_head_mask=None
  • query_length=None
  • use_cache=False
  • training=False
  • output_attentions=False

Hi @seungjun-green ,
I ran the Colab for the original Keras tutorial using the full dataset. It succeeded multiple times and failed once in the way you posted above. This is very strange as it means the behavior is inconsistent despite the fact that we're using the whole dataset.
What is apparent though, is that the issue happens somewhere deep in the T5 transformer code from Huggingface. In this tutorial, Keras is merely used as a wrapper to perform the training. I'm not sure where to go from this as the Huggingface code is really not familiar to us.
Thanks, Fabien

Hi @seungjun-green ,
I think the issue might be with data loader API which in this case is from Hugging Face itself. Keras here only used that data loader API and tried to train the data it received. I also tried executing the tutorial as it is and its worked fine as per gist.
In tf.data.Dataset.batch we have option like drop_reminder=True which might fix the issue with variable batch_size. Could you please check the hugging face API for such options. AFAIK the issue is from Hugging face API. Could you please report the issue there and let us know the outcome.
Thanks!

I also encountered the same issue.


Footer - footer link "Privacy" not translated in Hebrew

Detailed description

The "Privacy" link text in the Footer component is not translated when the Hebrew language is selected.

Is this issue related to a specific component?
Footer component:  all types with language only (Tall, Short, Micro)
https://ibmdotcom-react.mybluemix.net/?path=/story/components-footer--default-language-only

What did you expect to happen? What happened instead? What would you like to
see changed?
I expect the Privacy link to be translated in Hebrew, just like the rest of the links when the selected language is Hebrew. Instead, the link is still in English.

What browser are you working in?
Chrome, Firefox, Safari, Edge

What version of Carbon for IBM.com are you using?
1.14.0

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?
Knowledge Center team - IBM Documentation

Additional information

Digital Design does not maintain the translation files.  Tina Williams has moved to another role.
Linda will reach out to Tina Williams for the translation of the word 'Privacy'.  Kenny will update the translation file.


ci: generate cterm color even if line is missing

I noticed that some icons don't have a cterm_color value, so I updated the script to generate it if it's missing.

This also fixes the bug that caused the assignment of the generated cterm color (if its line was missing) to the next icon


Update dependency output_vt100

Update from version 0.1.2 to 0.1.3. This is a minor update that shouldn’t change the behavior of the library.

Thanks for the update - closing as, with library dependencies, it is better to leave the version range more permissible for downstream consumers to select the version themselves. pretty_assertions doesn't care if 0.1.2 or 0.1.3 is used and is happy with both.
If there are further updates which provide new features/fix bugs, please feel free to reopen another PR.

Alright although I don’t think there will be any more updates to the
crate, unless Microsoft decides to break things with their terminal.
--
Lucien “Phundrak” Cartier-Tilet
https://phundrak.com (Français)
https://phundrak.com/en (English)
Sent from GNU/Emacs


NEXT-37992 - Keep guest cart on login

Don't abandon the session cart when restoring without an existing or expired context.

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


EOS build for public testnet fails on Ubuntu 16.04 (AWS)

git clone https://github.com/eosio/eos --recursive
cd eos
git checkout DAWN-2018-02-14
git submodule update --recursive
./build.sh
fails at:

DAWN-2018-02-14 is no longer supported.

Thx. Will instructions on https://github.com/EOSIO/eos#runanode be updated?
How do we compile and run public testnet?


Feature: Log long running queries

Is your feature request related to a problem? Please describe.
I'd like to be able to log slow queries to console/file and keep track of average query times. This will greatly help developers optimize their query  as the Orm would inform them of any slow queries
Hibertnate supports this with the config option. Also typeorm seems to support it as well
Describe the solution you'd like
Add an option in config with maxExecutionTime  in ms  and log queries that exceed this time limit.
Describe alternatives you've considered
I have been using console.time and console.timeEnd but that is just the hacky way and is not precise
Additional context
Add any other context about the feature request here.
https://thorben-janssen.com/hibernate-slow-query-log/#activating-hibernates-slow-query-log
https://orkhan.gitbook.io/typeorm/docs/logging#log-long-running-queries

This could be a bit smarter, I'd say it should be a new logger namespace, and queries that will be over the threshold should be printed as warnings (so in orange or something like that) when query logging is enabled, when not (but the slow query logging is), only the slow queries should be logged. Moreover, we should have a way to configure separate logging for the slow queries, so users can dump those to a dedicated file, as they could be easily lost in the regular app logs.
So the changes should go in here:
https://github.com/mikro-orm/mikro-orm/blob/a60d6042c19f63e162015940da56faa76bd659be/packages/core/src/connections/Connection.ts#L119
The level should be conditionally set based on the query time to warning, and there will be some changes needed elsewhere too, so we don't print things twice for the slow queries.

Maybe @SpencerKaiser will have some opinions on this, as he did quite a lot of improvements in the logging in v6.

Super neat idea and I'd use it!

Wonderful Idea!
Maybe you have one more, how to catch large requests (in bytes, not in row counts)?
context.size >= 100kb

@isuvorov I think this is more suited on the http side imo


Migrate to macOS 10.14

Our bots are being migrated.  5 have already been moved over.  Will submit TBR to unblock migration of rest of bots.

lgtm


How to get current value in template ?

I am trying to overload the template to fit the design and I try to find the slider value ?

the slider value(s) ("to" for slider with 2 pointers) is contained there but as it also contains the dimension string label (km, meters) you choosed
https://github.com/darul75/angular-awesome-slider/blob/master/src/core/template/slider.tmpl.js#L22
here for pointers
https://github.com/darul75/angular-awesome-slider/blob/master/src/core/template/slider.tmpl.js#L18

hum ok Thx :+1:


chore(changelog): add initial changelog with 2.0 information

Deploy preview for pf-next ready!
Built with commit af8e522f008a3fbe68e0534d4dd89445dd117961
https://deploy-preview-1741--pf-next.netlify.com

In general, this looks good to me. I don't really understand when something should and shouldn't be capitalized and there seems to be a mish-mash. That said, not sure that this is a priority issue here.

@rachael-phillips , @mcoker  - made the last change by adding card to the feature list

@dgutride do we need this and can we close this?

@rachael-phillips  - we still need a pf4 changelog - maybe this isn't the right place for it globally instead of hosting it on the website - feel free to close


Sometimes popup does not work but opens my page in it - query params are hijacked

Hi @sahat, thank you for this amazing project, it saved me a lot of time :)!
I have been using satellizer for few months with no issues. I am using linkedin sign in and custom login. Backend is nodejs + express, while both frontend and backend are served on Dokku, Amazon.
Last few weeks I started having a problem with linkedin sign in: on Chrome/Linux and Safari/Mac combinations, it happens often that sign-in popup, after redirect from linkedin, ends up on that same redirect page but does not continue with authorization.
There are no errors/logs written, and to make it even harder to debug, it does not happen all the time, but maybe in 80% of cases.
On Firefox/Linux and Chrome/Mac combinations, all works fine! Also in localhost.
I can confirm that something goes wrong before sending request to my backend - authorization token is fetched from linkedin, redirect is sent, popup is actually redirected and then nothing happens, it stays there.
I am totally stuck, not sure anymore where to look - if you have any idea how to debug this or proceed, it would mean a lot. Thanks!
Video of error (errors at the bottom are produced by some extension): https://youtu.be/DUi9w7qZvqk.

I did some more testing (added console.logs to satellizer) and I can confirm that the problem happens in pollPopup function -> polling goes forever because although origin gets set, query and hash are never set. I have to figure out why is this happening sometimes, why is query/hash not read? Redirect from linkedin has those two, so they should be read by pollPopup.

Ok, I managed to fix this one! I will describe the problem and solution so it may help someone else in the same situation.
As I said above, redirect from linkedin was correct (has query params), but satellizer would not see any of them when trying to read them by polling popup's location.
So where did they go? It turned out that I had following setup with my uiRouter: / was redirected to '/home'. Since I did not set any redirectUri with satellizer, it was set to default which was /. So popup's location would be set to /?code=...&state=... and now it was a race -> how is going to get to it first, uiRouter or satellizer?
In case when uiRouter got first, he would redirect it to /home and query parameters would not be kept but lost (that is how uiRouter works). satellizer would continue polling popup's location forever, but query params would never come.
In case when satellizer got first, all was fine -> that is why it was sometimes working, and sometimes not.
I fixed this by removing the race condition - I set redirectUri in satellizer to /home so no redirecting was done by uiRouter and satellizer now has all the time it needs to read those query params.
There is still one mystery unsolved - why did it happen only on some combinations of browser/OS? I am guessing that it has to do something with their speed/caching, but it is only a hunch.

@Martinsos To further add my own thoughts to this, I found setting redirectUri to something like window.location.origin + /auth/facebook/callbackworks better than just window.location.origin. Then, to avoid 404 errors simply create a server route for /auth/facebook/callback that returns 200 OK.

No more race conditions with ui-router/ngRoute.
If for whatever reason some error occurs and popup isn't closed, users won't be looking at your home page (/ route) inside popup.

Right @sahat, I agree that is the way to go!

@Martinsos I wasn't sure where else to put it, so I added it to FAQ: https://github.com/sahat/satellizer#can-i-change-redirecturi-to-something-other-than-base-url

Great @sahat! May be worth noting that it can not contain any hash/query.

I might have a problem related to this.
this is my satellizer config:
$authProvider.facebook({
	clientId: $window.appConfig.fbClientId,
	url: $window.appConfig.apiUrl + 'auth/facebook',
	redirectUri: $window.location.origin + '/login',
	scope: ['public_profile','email']
});

$authProvider.logoutRedirect = '/login';
$authProvider.loginUrl = $window.appConfig.apiUrl + 'auth/login';

apiUrl is 'http://api.domain.com/v1/'
The problem is that sometimes everything works as expected, but sometimes the client sends http post request to api.domain.com/v1/auth/facebook without the code parameter. So it only has clientId and redirectUri.
And this gives an error on my backend:
400 Bad Request response: {"error":{"message":"Missing authorization code","type":"OAuthException","code":1"}}
@sahat Do you have any ideas how to solve this?

@Martinsos - awesome find and thanks to @sahat also for the redirect suggestion!

I'm using HTML5 mode with dynamic subfolders, so I had to do a little tweaking but this seems to work for me so far:
`
angular.module('app', []).config(function ($locationProvider, $authProvider){
$locationProvider.html5Mode({enabled: true, requireBase: false});

$stateProvider.state('app', {
    url: '/s',
    template: '<div ui-view></div>',
    controller: 'mainController'
}).state('app.build', {
    url: '/:template/:page',
    templateUrl: 'app/modules/appbuilder/builder.tpl.html',
    controller: 'AppCtrl'
});

$authProvider.facebook({clientId: 'xxxxxxxx', redirectUri: window.location.origin + '/s/'});
$authProvider.google({clientId: 'xxxxxxx.apps.googleusercontent.com', redirectUri: window.location.origin + '/s/'});
$authProvider.linkedin({clientId: 'xxxxxxxx', redirectUri: window.location.origin + '/s/?linked=true'});

}).controller('mainController', function($window, $state, $timeout){
if ($window.location.toString().indexOf('/s/?linked=true') > -1){
    $timeout(function(){ // Necessary to prevent race condition with UI router (site loads in popup)
        $state.go('app.build', {template: _defaultTemplate, page: _indexPage});
    }, 500);
} else {
    $state.go('app.build', {template: _defaultTemplate, page: _indexPage});
}

});
`


Set Automatic onboarding flag in Harbor

**Is your feature request related to a problem?
When I login for the very first time to Harbor I am asked about user name.
Describe the solution you'd like
The user name should be derived from id_token from JWT.
Additional context Add any other context or screenshots about the feature request here.

Hey team! Please add your planning poker estimate with Zenhub @Ani1357 @dennisvankekem @ElderMatt @ferruhcihan @srodenhuis

waiting for release


SRVKP-6898: Replace font awesome with custom css in dynamic plugin

Fixes:
https://issues.redhat.com/browse/SRVKP-6898

/cherrypick release-v1.17.x


Invalid error message

The JSON code
"teststring" : "eins",
"${teststring}___${teststring}" : "zwei"

causes:
'Setting value 'zwei' for parameter 'str(eins)___str(eins)' is not permissible when substituting parameters of composite data types in dictionary key names!'
But expected is:
{'eins___eins': 'zwei', 'teststring': 'eins'}

Hello Holger,
I pushed new commit c8720a9bc to stabi branch to fix this issue.
Thank you,
Son

Issue solved in JsonPreprocessor, but not in TestsuitesManagement.
TestsuitesManagement.still throws error:
[ ERROR ] Loading of JSON configuration file failed!
          Reason: The variable '${params}['global']['teststring']___${params}['global']['teststring']' is not available!

Hello Holger,
Thank you for your finding! I pushed the new commit c12257b2ae to stabi branch to fix the issue you mention above.
Thank you,
Son

Retest with latest versions not successful. Still got error message:
The variable '${params}['global']['teststring']___${params}['global']['teststring']' is not available!

Hello Holger,
I tested, and it worked fine on stabi branch, I still not create a pull-request to develop yet.
Thank you,
Son

Hi Son,
what is the status of this issue?
Thank you,
Thomas

Retest successful. Issue can be closed.


fix: responsiveness in homepage

#16

@CIPHERTron
The intensity of animation in the cards is now set at 33% otherwise it won't have enough space to adjust the text in there.

@CIPHERTron The intensity of animation in the cards is now set at 33% otherwise it won't have enough space to adjust the text in there.

We just need the heading of the article. Thus I don't think we require the amount of content it has right now. We can further reduce it.


Strange behaviour with <password()>

We are using AliceBundle in order to generate some fixtures :
Given this exemple :
person{4..800}:
        email: <email()>
        password: <password()>
        familyName: <firstname()>
        gender: <randomElement(['Homme', 'Femme'])>
        givenName: <lastname()>
        maidenName: <lastname()>
        aliasName: <username()>
        employer: <word()>
        unionActivity: <word()>
        observations: <word()>
        birthDate: <dateTime('now')>
        firstMembershipDate: <dateTime('now')>
        subscription: <boolean(80)>
        description: <word()>
        gender: <randomElement(['Homme', 'Femme'])>
        roles:
            - ROLE_MEMBER

The password part is giving us trouble :
Changing this
password: <password()>

to this
password: "Gerard"

is Working.
 /Users/hamza/projet/.../DataFixtures/ORM/fixtures/person.yml:
     - Instance py5~WE\9"H is not defined
     - Instance mask "6^"u|+WIUt*D(" did not match any existing instance, make sure the object is created after its references
     - Instance =#`@]~?2 is not defined
     - Instance Pf2s]Mo$,O6m6J7 is not defined
     - Instance mask "{2*yXW" did not match any existing instance, make sure the object is created after its references
     - Instance 3ktujOx9 is not defined

We are using the version : V1.3.1.
Thanks,
Simperfit.

Thanks for the report @Simperfit, I'm afraid this is a known issue: the password generates a string with @ which makes alice thinks that it's a reference to an object...
Maybe a custom provider for a "safe password" should be added to alice.

Follow up in https://github.com/nelmio/alice/issues/428 :)


updating the version of earthpy w newest workshop data

ok @betatim  trying this out and will wait for it to build FIRST

You know what, Tim… I just found this page
https://hub.docker.com/r/earthlab/earth-analytics-python-env/builds/
I see that each time I’m even updating the readme it’s rebuilding! Do you see that? I do see that it also takes a while. Am I doing something wrong? I think I get what is going on generally, maybe?

Not really an expert on docker hub, mostly just use it to push/pull images from there. It does seem like it builds a new image each time master changes.
It does take a fair amount of time though. Don't think there is much you can do about that :-/

I would use the following workflow:

make a PR with changes
think about it
merge the PR
check&wait till dockerhub has built the image for the merging of the PR
(go and update hub-ops with the newly tagged image)


fix correct time cmd

PR fixes wrong cmd name in the plugin from clock to correct time.
References:
https://github.com/guarinogabriel/Mac-CLI/blob/master/mac#L84
https://github.com/guarinogabriel/Mac-CLI/blob/master/README.md#general-commands
Please let me know if it is supposed to be clock so I can change the above two files instead.

ping @guarinogabriel

@daern91 - Thanks for your contribution!


[AzureMonitor] refactor event id in demo and tests

Follow up to https://github.com/Azure/azure-sdk-for-net/pull/44749#discussion_r1659188154
Existing tests were incorrectly using 0 as the event id.
This PR removes the 0 and validates that EventId is included in telemetry properties.

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


Export Mindmap

The user should be able to export the mindmap to various graphic formats (BMP, PNG, JPG).

Implemented.


Add json option to output

This change adds a json option to the hotsos output. The default is still
yaml.
Closes: canonical/hotsos#207
Signed-off-by: Nicolas Bock<EMAIL_ADDRESS>

@nicolasbock i ran this piped into jq but it does not produce valid json and i think it's for two reasons. firstly you are doing to coversion to yaml per plugin which means that output has multiple root levels. It also seems to get curtailed probably because of the progress icon. A better place to the do conversion would be on $MASTER_YAML_OUT i.e. on the total output. I would therefore suggest that tools/output_filter.py is a better place to do the conversion.

I had overlooked the header that the shell script adds. I updated hotsos.sh accordingly. Now I can pipe the output through jq and get:
{
  "hotsos": {
    "version": "development",
    "repo-info": "0813e1a"
  }
}
{
  "system": {
    "hostname": "rubberducky",
    "os": "ubuntu jammy",
    "num-cpus": 8,
    "load": "2.09, 2.16, 2.27",
    "rootfs": "/dev/mapper/crypt-root 940601196 440108772 452641320  50% /",
    "unattended-upgrades": "disabled",
    "date": "Wed Nov 17 04:26:57 PM UTC 2021",

    "sysctl-mismatch": {
      "net.ipv4.conf.all.rp_filter": {
        "actual": "0",
        "expected": "2"
      },
      "net.ipv4.conf.all.mc_forwarding": {
        "actual": "0",
        "expected": "1"
      }
    },
    "potential-issues": [
      {
        "desc": "host sysctl not consistent with contents of systcl.d",
        "origin": "system.checks",
        "type": "SysCtlWarning"
      }
    ]
  }
}
{
  "kernel": {
    "version": "5.13.0-7619-generic",
    "boot": "ro cryptdevice=/dev/nvme0n1p4:crypt-root:allow-discards rd.luks.options=discard quiet splash vt.handoff=7",
    "cpu": {
      "smt": "enabled"
    },
    "memory-checks": "no issues found",
    "potential-issues": [
      {
        "desc": "kern.log contains 1 stacktraces.",
        "origin": "kernel.log_event_checks",
        "type": "KernelError"
      }
    ]
  }
}

Another observation: The output_filter.py is called only in MINIMAL_MODE.

Please have another look @dosaboy. I have moved the conversion code into a separate helper script.


[Bugfix] get empty table names in drop db

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #13921
Problem Summary(Required) ：
Problem:
This Problem was introduced by commit bdb4da1. This commit use Collections.unmodifiableSet instead of the new HashSet to reduce the memory usage of FE. But the problem here is that Collections.unmodifiableSet returns the view of nameToTable.keySet(). Before using the TableNames in recycleDatabase, nameToTable had been dropped, and also the view of it would become empty, which caused this problem.
Solution:
We copy the nameToTable.keySet() only when dropping the database.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run title-check / pr-title-check (pull_request_target

run title-check / pr-title-check (pull_request_target)

run title-check

run title-check


Expenses don't show in Staging Supportal Old Dot

Slack chat https://expensify.slack.com/archives/C03TQ48KC/p1735052827689619
Action Performed:

Go to https://staging.expensify.com/_support/
Click Reports
Click on a report
Notice the expenses don't show

Expected Result:

Go to https://staging.expensify.com/_support/
Click Reports
Click on a report
Expenses should show

Actual Result:
On staging alone (not prod) expenses on OD reports do not show in Supportal
Screenshots/Videos
https://github.com/user-attachments/assets/84141ab7-0a36-456d-a9e9-df76712d9902
Network error shows

Discussing here - https://expensify.slack.com/archives/C03TQ48KC/p1735055051351899?thread_ts=1735052827.689619&cid=C03TQ48KC

This got fixed here in the end: https://github.com/Expensify/Web-Expensify/pull/44899#issuecomment-2561362205


Reorganize modules

It'll be very hard to work on the code by having all of us touch a single file. The functionality should be split into separate modules. I did some general reorganization based on some intuition. Please see commit logs/messages to make sense of it.

I did review the structure and added seperate issues for @marcelamelara comments.

Thanks @abuishgair !!


Flutter Linux build error X window system error

Hello Guys
I have updated my system to (Elementary os) and after updating and install builds tools for flutter for Linux build to get this error
when run flutter run -d linux i get that:
Launching lib/main.dart on Linux in debug mode...
(store_items:6176): Gdk-ERROR **: 10:22:28.017: The program 'store_items' received an X Window System error.
This probably reflects a bug in the program.
The error was 'GLXBadFBConfig'.
  (Details: serial 244 error_code 167 request_code 152 (GLX) minor_code 0)
  (Note to programmers: normally, X errors are reported asynchronously;
   that is, you will receive the error a while after causing it.
   To debug your program, run it with the GDK_SYNCHRONIZE environment
   variable to change this behavior. You can then get a meaningful
   backtrace from your debugger if you break on the gdk_x_error() function.)
Error waiting for a debug connection: The log reader stopped unexpectedly.
Error launching application on Linux.
Exited (sigterm)

and flutter doctor output is:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.60.2)
[✓] Connected device (2 available)

• No issues found!

OS Elementary OS
Flutter (Channel stable, 2.2.3, on Linux, locale en_US.UTF-8)

Hi @abdulrasol
Unfortunately, the Flutter version you're using is outdated and is no longer supported, it's not feasible to provide any bug fixes for such an old version of Flutter.
I kindly advise you to update to the latest Flutter stable, if you encounter the issue again, please comment below with the details from the latest stable, including steps to replicate the issue.
Can you please upgrade to the latest  stable and try again
Execute the following commands to upgrade
flutter channel stable
flutter upgrade --force
flutter doctor -v
If the problem persists, can you please provide your flutter doctor -v with the upgrade, complete reproducible minimal code sample, and complete flutter run --verbose logs running the minimal code sample?
Thank you

@TahaTesser
Thanks for your reply dear
but still the same issue
now I use flutter 2.5.2
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.2, on elementary OS 6 Odin 5.11.0-37-generic, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.60.2)
[✓] Connected device (2 available)

 flutter doctor -v output:
Flutter (Channel stable, 2.5.2, on elementary OS 6 Odin 5.11.0-37-generic, locale en_US.UTF-8)
    • Flutter version 2.5.2 at /home/rasol/DevTools/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3595343e20 (4 days ago), 2021-09-30 12:58:18 -0700
    • Engine revision 6ac856380f
    • Dart version 2.14.3

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/rasol/DevTools/Android
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /home/rasol/DevTools/android
    • Java binary at: /home/rasol/Apps/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2020.3)
    • Android Studio at /home/rasol/Apps/android-studio
    • Flutter plugin version 59.0.2
    • Dart plugin version 203.8292
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.60.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.26.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • elementary OS 6 Odin 5.11.0-37-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 94.0.4606.71

• No issues found!

Hi @abdulrasol
Are you using a virtual machine or very old hardware?  The error seems to be related to the environment itself based on my search. Please file an issue on the Elementary OS forum or try a different machine with the same OS
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


fix: made lerna a peerDependency to avoid installing it multiple times

Description
<EMAIL_ADDRESS>should be a peerDependency (and for the purposes of the tests a devDependency) instead of a dependency.
Motivation and Context
This will avoid situations in which multiple versions of lerna get installed into a monorepo and it barfs when you try to run one of the commands (with "Incompatible local version of lerna detected!") due to your lerna.json#version not matching the version which has decided to run. (This sometimes happens if yarn has decided to install your packages into a slightly weird folder structure and the version of lerna which is being picked up is not the version that you installed within your root package.json.)
Usage examples
N/A
How Has This Been Tested?
I ran yarn test at the root of the monorepo.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I've just hit this issue where I get the same incompatible lerna version warning after running yarn add @storybook/addon-knobs in a lerna monorepo project, so this PR would be really useful.

Hey thanks for contributing - I think this is superseeded by #406.


expr/builtin_fncall: implement Inet6Ntoa

Signed-off-by: sweetIan<EMAIL_ADDRESS>What have you changed? (mandatory)
expr/builtin_fncall: implement Inet6Ntoa
What are the type of the changes? (mandatory)

New feature (non-breaking change which adds functionality)

How has this PR been tested? (mandatory)
unit test
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
Refer to a related PR or issue link (optional)
#3275

@solotzg @BusyJay PTAL

/run-unit-test


Bug/autoset tinyint

As issue #2057
I made a fix by adding boolean db type, and set db column type of it to 'checkbox'
I also remove int, tinyint, smallint, mediumint, longint from db column type cases,
because DBAL just take those type as IntegerType.
Just leaving "integer" is ok.
"CrudPanelAutoSetTest" is already update.

Thank you for taking the time to put together this PR, @fzhantw . I think you're totally right on this one. Merging and tagging right now.
I've just found one small problem and fixed it: auto-setting to checkbox also set the column to checkbox (not just the field). Which is unfortunate, because the checkbox column is not an equivalent to the checkbox field. For the tinyint column type, the most appropriate are:

the boolean column type
the checkbox field type

To fix this inconsistency in a non-breaking way, I've created a boolean field type, that is just an alias of the checkbox field type. So now boolean/tinyint db column defaults to boolean column type and boolean field type.
Acutally, I think this one of the rare cases where we can push a change to setFromDb() without worrying about changing people's admin panels for the worse. This behaviour is more intuitive, and I think anyone who's used it differently would appreciate the change for the better. So I'm going to merge it in 3.6 right now. It'll be available with a composer update.
Thanks for the fix @fzhantw !


Remove version: gergelycsernak.euroscope version 3.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://www.euroscope.hu/install/EuroScopeSetup32.msi
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 910)

URL: https://www.euroscope.hu/install/EuroScopeSetup32.msi
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 910)

My automation had an issue with this PR - if it should have had Highest-Version-Remaining, apologies.