Could not rewrite resource in-place because URL is not in cache on static resources

I'm having a problem with ngx_pagespeed where it's working like a dream for any proxied or cached resources, but I can't figure out how to get it to do in-place rewrites of static HTML content served by nginx.
For a plain-Jane html file, this is the first hit:
[Sat, 14 Feb 2015 03:17:09 GMT] [Info] [13515] Trying to serve rewritten resource in-place: https://neosmart.net/EasyBCD/easybcd.html
[Sat, 14 Feb 2015 03:17:09 GMT] [Info] [13515] Cache entry is expired: https://neosmart.net/EasyBCD/easybcd.html (fragment=neosmart.net)
[Sat, 14 Feb 2015 03:17:09 GMT] [Info] [13515] Could not rewrite resource in-place because URL is not in cache: https://neosmart.net/EasyBCD/easybcd.html

And this is the second:
[Sat, 14 Feb 2015 03:17:23 GMT] [Info] [13515] Trying to serve rewritten resource in-place: https://neosmart.net/EasyBCD/easybcd.html
[Sat, 14 Feb 2015 03:17:23 GMT] [Info] [13515] Cache entry is expired: https://neosmart.net/EasyBCD/easybcd.html (fragment=neosmart.net)
[Sat, 14 Feb 2015 03:17:23 GMT] [Info] [13515] Could not rewrite resource in-place because URL is not in cache: https://neosmart.net/EasyBCD/easybcd.html

The headers for the file in question:
HTTP/1.1 200 OK
Server: nginx/1.7.9
Date: Sat, 14 Feb 2015 03:21:19 GMT
Content-Type: text/html
Content-Length: 13252
Last-Modified: Sat, 07 Feb 2015 15:34:50 GMT
Connection: keep-alive
Vary: Accept-Encoding
ETag: "54d6309a-33c4"
Expires: Sat, 14 Feb 2015 03:22:19 GMT
Cache-Control: max-age=60
Strict-Transport-Security: max-age=604800
Cache-Control: no-transform
X-UA-Compatible: IE=Edge,chrome=1
Accept-Ranges: bytes

Do you have any idea why pagespeed isn't caching the file correctly? There are no messages about file size, origin, etc. that would indicate something's wrong.

I'm guessing this is due to the inability to force ngx_pagespeed to ignore the no-transform header on html files in any way, shape, or form.

@mqudsi This looks like expected behaviour to me.
In-place rewriting does not optimize and/or cache html by itself.
For caching of html, you could have a look at https://developers.google.com/speed/pagespeed/module/downstream-caching
For that to work, the html should be served as cacheable in the first place, and you might have to fix the date header (yours looks odd, does it look like that without ngx_pagespeed?: Date: Tue, 27 Jan 1970 01:20:08 GMT)
Closing this, feel free to re-open if you think there's still a problem.
@jeffkaufman as a side note, it looks like these html requests are never marked as "ipro not being applicable" in cache, resulting in a costly lookup all the way to the slowest cache backend in the property cache each time? Should we look into that?

Thanks. Opened as #913

@oschaaf This is odd, but pagespeed is now rewriting the pages in question? As you know, I've been playing around with the ngx_pagespeed config to get things the way I want them, but the HTML is still being served locally, the date header is still wrong, but now ngx_pagespeed is rewriting the static HTML as I wish it to. As an FYI, I did upgrade to 1.7.10.
This is the new response:
HTTP/1.1 200 OK
Server: nginx/1.7.10
Content-Type: text/html; charset=utf-8
Connection: keep-alive
Vary: Accept-Encoding
Strict-Transport-Security: max-age=604800
X-UA-Compatible: IE=Edge,chrome=1
Date: Wed, 28 Jan 1970 13:54:19 GMT
X-Page-Speed: <IP_ADDRESS>-4448
Cache-Control: max-age=0, no-cache

No real changes to speak of.

@mqudsi I was busy trying to reproduce (no luck yet) and had another look at the two response headers you posted. The first and second example of the Date: header value you posted show that it's value actually is being set, but incorrectly. That is remarkable, and worth a closer look.

@oschaaf Yeah I wanted to comment on that at the time, I was going to say the date is wrong but the time isn't but that was of course not the case.
I'm guessing it actually shows nginx uptime added to epoch?

@mqudsi Possibly, I'm not sure. Are you on a 32 bits platform?

@oschaaf I am indeed.

On Feb 16, 2015, at 3:54 PM, Otto van der Schaaf<EMAIL_ADDRESS>wrote:
@mqudsi Possibly, I'm not sure. Are you on a 32 bits platform?
—
Reply to this email directly or view it on GitHub.

@mqudsi That confirms my suspicion that using ngx_current_msec here was not the best of ideas:
https://github.com/pagespeed/ngx_pagespeed/pull/605/files#diff-8fb776418fb480e5f084e8500b8f0b0cR251
I'll post a patch for this tomorrow.


Tile loading color

I'm using google maps in a dark themed app and when the map is loading it shows (or flashes, since it loads quickly) a white background which clearly doesn't fit in with the rest of the design and is actually quite aggressive on the users experience. I've found no way to style it and make it work with a dark themed app
It would be great if we could change this to be some other color as is possible with js gmaps, but i don't know if this is even possible with android or ios sdks.
I've tried changing the background color, but that has no effect.

i am having same issue..

I'm using google maps in a dark themed app and when the map is loading it shows (or flashes, since it loads quickly) a white background which clearly doesn't fit in with the rest of the design and is actually quite aggressive on the users experience. I've found no way to style it and make it work with a dark themed app
It would be great if we could change this to be some other color as is possible with js gmaps, but i don't know if this is even possible with android or ios sdks.
I've tried changing the background color, but that has no effect.
for styling map use this site : https://mapstyle.withgoogle.com/
and insert exported code here
mapView.setStyle(JSON);


UI/UX improvements

Clean up the interface so that it looks beautiful on both desktop and so that it is more mobile friendly. Right now, the fonts are too big and the padding/margin is too much on mobile. Add some media query magic.
Also, the site could be even faster with more pre-loading and caching.

For the most part this is fine. AdSense implemented.


how to select the item in  Listbox

I tyring to use the pywinauto to test the software and the backend of the app is uia ,so how can i  to select the  item in a  Listbox?
wait for you help

You may get some ideas by having a glance at unit tests for ListView and ListViewItems https://github.com/pywinauto/pywinauto/blob/55f31621557a3884bbbda4eccbd015c6d1d58c33/pywinauto/unittests/test_uiawrapper.py#L796.
As for example in:
https://github.com/pywinauto/pywinauto/blob/55f31621557a3884bbbda4eccbd015c6d1d58c33/pywinauto/unittests/test_uiawrapper.py#L1108


return a file object instead of the path in case the strip option is used

prevents a call to file.path returns undefined

https://github.com/bower/decompress-zip/pull/31


Chocolatey 0.9.10 and exit codes - cChoco is now reporting successful package installs as failures

Related to #19, but better context.
I mentioned this before in April - https://github.com/chocolatey/choco/issues/512. 0.9.10 just released today and folks are going to start reporting breakages even though we noted it in our release notes and tried to give plenty of notice to folks that it was coming (2 months notice in this case).
Here is the relevant portion of the ChangeLog (under breaking changes):

Exit with the same exit code as the software being installed - see #512

There are more exit codes from Chocolatey now that indicate success -0, 1605, 1614, 1641, and 3010. You may need to adjust anything you were using that would only check for 0 and nonzero.
If you need the previous behavior, be sure to disable the feature usePackageExitCodes or use the --ignore-package-exit-codes switch in your choco commands.

Morning,
I reviewed this when it was raised, as we rely on the Test-Resource call to validate the install of the package rather than checking an exit code from the choco install command I don't believe this will affect the DSC Resource.
Installer invoked here: https://github.com/PowerShellOrg/cChoco/blob/master/DSCResources/cChocoPackageInstall/cChocoPackageInstall.psm1#L145
Check install run here, using choco list -lo: https://github.com/PowerShellOrg/cChoco/blob/master/DSCResources/cChocoPackageInstall/cChocoPackageInstall.psm1#L182
May not be the ideal approach for the resource longer term but has worked well for me in the past.
On a separate note there are a fair few oustanding issues in the repo and over recent months I've not been able to give time to it, if you know any community members who'd be interesting in assisting reviewing and updating the resource could you point them this way?

#31


Check output of rpmqa command and retry if needed

Updated release notes draft:

Output of rpmqa command is checked and retried if empty


Querstion about cache invalidation

Hi, If I run multiple varnish instances is there a vay to purge the caches so the purge command is distributed between the varnish pods?

Unfortunately, there is not a built in way to do this with the varnish operator. Are you wanting to run the purge command across all varnish instances at once or serially? If the former, perhaps [kubectl-parallel-exec](https://github.com/cin/kubectl-parallel-exec) could help. If this is a feature you'd like added, feel free to contribute a PR. I have been maintaining a fork of this repo here. I need to do some maintenance on this project in general.

Yes, in general I would like to control the cache with a single request, like on this project: https://github.com/mittwald/kube-httpcache


LUCENE-10584: Properly support #getSpecificValue for hierarchical dims in SSDV faceting

Description (or a Jira issue link if you have one)
Please see: LUCENE-10584

@mdmarshmallow - might be a good one for you to have a look at when you get a minute. Pretty simple change but you've got a lot of context since you added hierarchical dim support to SSDV facets (thanks again!).


How do I change the window size?

Hello, nice plugin.
The location list window takes half my screen on MacVim (haven't tried on normal Vim cause it's broken hehehe), any way I can make it like 25% or 10% of my screen?

This is described in the manual:

:h 'syntastic_loc_list_height'
:h syntastic-loclist-callback


update build task

What it does:

updates the build task to use dependsOn rather than run-commands.

I ended up using this for @tanstack/table, and I think I like this better

The splitting out of the rollup step to its own cacheable task makes the caching a bit better too (I believe prior, we were stashing everything in the build directory for both build and build:types steps)

adding rollup to cacheable operations

is this a thing worth doing? I just fear that if we get the cache config wrong, we might be shipping cached packages that are not made from the latest source ...

also, codesandbox preview build is now failing 🤔

adding rollup to cacheable operations

is this a thing worth doing? I just fear that if we get the cache config wrong, we might be shipping cached packages that are not made from the latest source ...

Prior to this PR, we are still caching the build. This change is really to split out rollup so that the rollup command is "atomically cacheable", meaning we can run nx rollup and cache the result of that command alone. Without this change, we can cache the result of pnpm build, but not any of the individual steps.
Let me know if we want to remove the build from the list of cacheable operations! As we've mentioned, caching is hard, so if we're not confident yet we can keep caching only for the pr workflows or only for the "validation" / "test:*" tasks!
I do think ideally we'll want to eventually cache the build though!

also, codesandbox preview build is now failing 🤔

Hrmm, not able to recreate the same failure locally while following along with the commands in the log here: https://ci.codesandbox.io/status/TanStack/query/pr/5166/builds/358582
I'll try to dig deeper tomorrow. Do we know how to find/set the node version for code sandbox?

RE: code sandbox failures - leaving this for future me:
csb's node version: 16.18.1
setup steps:

clones repo
npm run install:csb
npm run build < this doesn't complete successfully

Let me know if we want to remove the build from the list of cacheable operations! As we've mentioned, caching is hard, so if we're not confident yet we can keep caching only for the pr workflows or only for the "validation" / "test:*" tasks!

I do think ideally we'll want to eventually cache the build though!

given that this is a caching library, we believe in caching 😆 . If you're confident it'll work, I'm fine with it :)

it might be worth noticing that we usually only release packages that have affected changes in the src directory, but there are ways to bypass this by having RELEASE_ALL in the commit message. This will right now release all packages. I just want to make sure that in those cases, we won't get cached output released ...

Codecov Report
Patch coverage: 97.67% and project coverage change: +0.31 :tada:

Comparison is base (ed7d9f8) 91.90% compared to head (fdc4a27) 92.22%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5166      +/-   ##
==========================================
+ Coverage   91.90%   92.22%   +0.31%
==========================================
  Files         111      111
  Lines        4188     4229      +41
  Branches     1083     1099      +16
==========================================
+ Hits         3849     3900      +51
+ Misses        318      308      -10
  Partials       21       21

Impacted Files
Coverage Δ

packages/query-core/src/query.ts
99.03% <ø> (ø)

...kages/react-query-devtools/src/styledComponents.ts
92.85% <ø> (ø)

packages/react-query/src/errorBoundaryUtils.ts
100.00% <ø> (ø)

packages/react-query/src/reactBatchedUpdates.ts
100.00% <ø> (ø)

packages/react-query/src/useBaseQuery.ts
95.45% <ø> (ø)

packages/react-query/src/useInfiniteQuery.ts
100.00% <ø> (ø)

packages/react-query/src/useIsFetching.ts
92.30% <ø> (ø)

packages/react-query/src/useIsMutating.ts
92.30% <ø> (ø)

packages/react-query/src/useMutation.ts
96.15% <ø> (ø)

packages/react-query/src/useQueries.ts
94.64% <ø> (ø)

... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

closing this PR as I've pulled these changes into: #5143
let's continue the conversation there @TkDodo


C# 8 and 9 Pattern Matching

I don't see any examples for C# 8 and 9-style pattern matching on our pattern matching page. No recursive patterns, positional patterns, property patterns, etc. We likely need to copy in the examples from https://docs.microsoft.com/en-us/dotnet/csharp/whats-new/csharp-8#more-patterns-in-more-places.

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

ID: d1f2a2fe-8279-e6b3-93e8-33398167655f
Version Independent ID: 35191215-988f-3700-1f85-4751711f692e
Content: Pattern Matching - C# Guide
Content Source: docs/csharp/pattern-matching.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks @333fred
I've added this to the list of .NET 5 / C# 9 updates.

@pkulikov Are there tasks remaining for this issue? Or did the recent PRs address everything?

@BillWagner the tasks I'm going to work further are about updating the reference pages (of switch and is).
However, it's not related to this issue. This issue is about the concept article:
https://docs.microsoft.com/en-us/dotnet/csharp/pattern-matching
Looks like that article was written in the C# 7.0 time and has not been updated since then. Nowadays, it's out-of-date. Also, it has big overlap with the reference about the switch statement. That is not necessary anymore. That article should either be completely rewritten or removed. For example, the following tutorial provides a better concept overview:
https://docs.microsoft.com/en-us/dotnet/csharp/tutorials/pattern-matching


feat: Add fine grained cache and topic level permissions to tokens

Summary

This PR adds the ability to scope tokens to certain topics or caches per the scenarios in this doc.
See related PR for proto changes: https://github.com/momentohq/client-protos/pull/179
Note: Build jobs are expected to fail till the protos PR is merged.

Testing

Added unit tests
Integration tests are being worked on and will be tested once server is ready

Looks good. Will approve once build failures are fixed.

about to review this again but just curious what happens if this gets merged before the server-side changes?

about to review this again but just curious what happens if this gets merged before the server-side changes?

about to review this again but just curious what happens if this gets merged before the server-side changes?

Things would break if anyone tried to generate a AllDataReadWrite token because it will now try to create the token with the new permission fields. But I don't plan to merge this before server side changes are deployed.

about to review this again but just curious what happens if this gets merged before the server-side changes?

about to review this again but just curious what happens if this gets merged before the server-side changes?

Things would break if anyone tried to generate a AllDataReadWrite token because it will now try to create the token with the new permission fields. But I don't plan to merge this before server side changes are deployed.

ok cool, in that case probably safest to convert this to a "draft" PR until then.

@krispraws based on our call the other day and the thing you showed me where you were able to pass in an object literal to the function, I think you and I are on the same page, but please feel free to re-request review whenever you are ready for another pass.

about to review this again but just curious what happens if this gets merged before the server-side changes?

about to review this again but just curious what happens if this gets merged before the server-side changes?

Things would break if anyone tried to generate a AllDataReadWrite token because it will now try to create the token with the new permission fields. But I don't plan to merge this before server side changes are deployed.

ok cool, in that case probably safest to convert this to a "draft" PR until then.

I was actually wrong about things breaking. Control plane would just have ignored the new fields in the proto, so nothing would break.

I'm good with this PR at this point; my only note:
I don't want to do a release of the JS SDK until we have spent some time bug-bashing this with an eye on the API, just in case we come up with any other usability changes.
and it would be great if we can add an example in the examples dir, and possibly add web sdk support, prior to release.
we don't have any urgent things that I know of that need to get out in the JS SDK so I think it's okay to merge and block release for a few days or whatever, but we just need to make sure everyone's aware of that plan once we merge this.

Added an example in https://github.com/momentohq/client-sdk-javascript/pull/598/commits/8480434030e88fee8d8fb67964104614b471131d
It is very basic and works on my machine.
Re: web-sdk support, the work is likely minimal but I would need to scope it. @danielamiao - do we want to add web sdk support right now?

@krispraws @cprice404 let's not make web sdk support a release blocker, but a fast follow post release.

I'm good with this PR at this point; my only note:
I don't want to do a release of the JS SDK until we have spent some time bug-bashing this with an eye on the API, just in case we come up with any other usability changes.
and it would be great if we can add an example in the examples dir, and possibly add web sdk support, prior to release.
we don't have any urgent things that I know of that need to get out in the JS SDK so I think it's okay to merge and block release for a few days or whatever, but we just need to make sure everyone's aware of that plan once we merge this.

Added an example in 8480434 It is very basic and works on my machine. Re: web-sdk support, the work is likely minimal but I would need to scope it. @danielamiao - do we want to add web sdk support right now?

web-sdk changes are now part of this PR - thanks to @rishtigupta !


Profiling tool: I/O metrics in separate section

This PR contributes to https://github.com/NVIDIA/spark-rapids/issues/4897 .
It extends the current data_source_information.csv with additional columns -  readCount, accumId, buffer_time, scan_time, data_size and decode_time. I am not certain if we can get Previous read time as mentioned in the issue.
A new output file io_analysis.csv will be generated if there is any IO in the SQL query.
All the columns except numberOfWrites and duplicateReads is included in this PR.
numberOfWrites seems easy to add in the followup PR but I am not sure about the duplicateReads
Update current tests to reflect the changes and added one new test for ioMetrics.
data_source_information.csv output:
appIndex,sqlID,format,buffer_time,scan_time,data_size,decode_time,location,pushedFilters,schema
1,0,Text,NA,NA,NA,NA,InMemoryFileIndex[file:/home/user1/workspace/spark-rapids-another/integration_tests/src/test/resources/people.csv],[],value:string
1,1,CSV(GPU),1180511,26,"",8465370,file:/home/user1/workspace/spark-rapids-another/integration_tests/src/test/re...,],_c0:string;_c1:string;_c2:string
1,2,JSON(GPU),305515,22,"",2703739,file:/home/user1/workspace/spark-rapids-another/integration_tests/target/test...,],number:double
1,3,ORC(GPU),40994352,58,"",2636708,file:/home/user1/workspace/spark-rapids-another/tests/src/test/resources/file...,],loan_id:bigint;orig_channel:int;orig_interest_rate:double;orig_upb:int;orig_loan_term:int;...
1,4,Parquet(GPU),24043071,38,"",8232238,file:/home/user1/workspace/spark-rapids-another/integration_tests/target/test...,],loan_id:bigint;monthly_reporting_period:string;servicer:string;interest_rate:double;curren...

io_metrics.csv output:
appIndex,appID,sqlID,input_bytesRead_sum,input_recordsRead_sum,output_bytesWritten_sum,output_recordsWritten_sum,diskBytesSpilled_sum,memoryBytesSpilled_sum,sr_totalBytesRead_sum,sw_bytesWritten_sum
1,local-1675128111910,0,49954174,9999999,86518,19999,0,0,0,0
1,local-1675128111910,1,49954174,9999999,45296,9999,0,0,0,0

please add what the output looks like to desciprtion

Python unit test failure doesn't look due to this PR.
pylint: FAIL ✖ in 35.89 seconds
flake8: install_deps> python -I -m pip install flake8
flake8: install_package_deps> python -I -m pip install fire==0.4.0 importlib-resources pandas==1.4.3 pyYAML==6.0 tabulate==0.8.10
flake8: install_package> python -I -m pip install --force-reinstall --no-deps /home/runner/work/spark-rapids-tools/spark-rapids-tools/user_tools/.tox/.tmp/package/2/spark-rapids-user-tools-22.12.0.tar.gz
flake8: freeze> python -m pip freeze --all
flake8: fire==0.4.0,flake8==6.0.0,importlib-resources==5.10.2,mccabe==0.7.0,numpy==1.24.1,pandas==1.4.3,pip==22.3.1,pycodestyle==2.10.0,pyflakes==3.0.1,python-dateutil==2.8.2,pytz==2022.7.1,PyYAML==6.0,setuptools==65.6.3,six==1.16.0,spark-rapids-user-tools @ file:///home/runner/work/spark-rapids-tools/spark-rapids-tools/user_tools/.tox/.tmp/package/2/spark-rapids-user-tools-22.12.0.tar.gz,tabulate==0.8.10,termcolor==2.2.0,wheel==0.38.4,zipp==3.12.0
tox: flake8
.pkg: _exit> python /opt/hostedtoolcache/Python/3.8.16/x64/lib/python3.8/site-packages/pyproject_api/_backend.py True setuptools.build_meta
  pylint: FAIL code 20 (35.89=setup[24.06]+cmd[11.83] seconds)

After running this I think I would get rid of the SQL descrption from the IO Metrics table... I don't think it adds anything.

Please update the linked issue with what has been implemented and what is left and why


[QA] Uploads from iOS clients with some issues

The scenario is uploading a big file (~1.5GB) from the iOS app (via Files App) to oCIS with EOS storage.

iOS client selects to upload the file
File is greyed out in the file list of iOS app. There is no progress update, it appears like "pending"
Some seconds later, a POST request appears with the info related to the file to upload (checked with mitmproxy). Headers include Tus-Resumable
Some minutes later (~3), request times out in iOS client, showing the proper notification.
Then, server responses 500 (no payload) to the request in 3.
A bit later, file appears as uploaded in file list of iOS and other clients.

Backend: EOS storage via via https://github.com/owncloud-docker/compose-playground/blob/master/examples/hetzner-deploy/make_ocis_eos_compose_test.sh
Version: v1.0.0-rc1
Client: iOS App d323c29e over iOS13
Is this an issue of client, backend or both?
CC @felix-schwarz

@jesmrec I'd need to see the client log for that to really tell what's going on, but step 6 suggests this:
Unless the server provides a chunk size, the iOS client will try to upload using the create-with-upload extension, meaning the entire file is sent in the body of the POST request. If the file eventually shows up in other clients, too, this to me suggests that request completes correctly - but the ocis server doesn't respond in time.
The only explanation why the server and client would time out that I can see right now would be if the client wouldn't send a Content-Length header with that POST request (step 3), so that the server might wait for additional content after the request's body was already sent - and the client waits for the server's response that never comes. What are the headers of the request from step 3 in mitmproxy?

Here you have logs from iOS. File name is Video_test.zip
ownCloud_16_Sep_2020_at_13_19_05.log.txt
Headers of the request done in 3. :
Host:             ...
Upload-Offset:    0
Accept:           */*
Authorization:    Bearer ...
Accept-Language:  en-gb, en
Accept-Encoding:  gzip, deflate, br
Content-Type:     application/offset+octet-stream
X-Request-ID:     A1157DDB-D859-4A0C-AB1B-C16D67BCF4E1
Content-Length:<PHONE_NUMBER>
Tus-Resumable:    1.0.0
User-Agent:       ownCloudApp/11.4 (App/169; iOS/13.7; iPhone)
Connection:       keep-alive
Upload-Length:<PHONE_NUMBER>
Upload-Metadata:  filename VmlkZW9fdGVzdC56aXA=,checksum U0hBMSAxMjBlMDliZDU0M2ZjMjRjNTg5NzEyOGFiMTkzODRhNjYyMWI5NjMy

in which the Content-Lenght is completed
Headers of the 500 response:
Access-Control-Allow-Headers:   Tus-Resumable, Upload-Length, Upload-Metadata, If-Match
Access-Control-Allow-Origin:    *
Access-Control-Expose-Headers:  Tus-Resumable, Location
Content-Length:                 0
Content-Security-Policy:        default-src 'none';
Date:                           Wed, 16 Sep 2020 11:30:57 GMT
Location:      ...
Tus-Resumable:                  1.0.0
Upload-Offset:<PHONE_NUMBER>
Vary:                           Origin
X-Content-Type-Options:         nosniff
X-Download-Options:             noopen
X-Frame-Options:                SAMEORIGIN
X-Permitted-Cross-Domain-Polic  none
ies:
X-Robots-Tag:                   none
X-Xss-Protection:               1; mode=block

i hope it helps.

@jesmrec Thanks for the logs as well as the request and response headers. I can't find any issue from the app side from the logs or the headers:

on the app side, the 500 response does not arrive, but the connection times out
on the server side, I see Upload-Offset:<PHONE_NUMBER> in the header, confirming the receipt of the full file (request was Content-Length:<PHONE_NUMBER>). Not sure why/how that info comes as part of a 500 response, though.

thanks @felix-schwarz for your effort. The issue is in the correct place then.

So the ocis server is sending a 500 response. Can you find that related error in ocis logs? It would be great to have the server logs here and an update in the top post. Thanks

@jesmrec @felix-schwarz this still an issue?

Not reproducible with ocis-2.0.0. Any way to check with EOS storage?

Please open again if the ticket is still relevant


Range filter not applied if just one value entered

Bug report
Using the new range filter, makes it possible to set a minimum and a maximum value. See the documentation
The problem is that the minimum and maximum values are only appended to the AJAX call as query parameter if both the minimum and maximum values are entered.
I propose that the values are added to the query even if only one of the fields is entered, as it will be possible to limits results by a single parameter or require both in the filter it self (the documentation has a check that both values should be set). That way the filter can be used for searches like:

numbers greater than X
numbers less than Y
numbers between X and Y

Right now, we can only use the last search using this filter.
What I did:
Create a range filter like this:
$this->crud->addFilter([
  'name' => 'number',
  'type' => 'range',
  'label'=> 'Range',
  'label_from' => 'min value',
  'label_to' => 'max value'
],
false,
function($value) { // if the filter is active
  $range = json_decode($value);
  if ($range->from && $range->to) {
    $this->crud->addClause('where', 'number', '>=', (float)$range->from);
    $this->crud->addClause('where', 'number', '<=', (float)$range->to);
  }
});

I then only filled out the minimum field (min value) with a value of 2
What I expected to happen:
I expected that the AJAX call should be made as `POST: /search?number={"from":"2","to":""}
What happened:
An AJAX call was made without query parameter `POST: /search
What I've already tried to fix it:
I looked at the documentation, which implements a check for both fiels presence, but apparantly they will always be there:
  if ($range->from && $range->to) {
    $this->crud->addClause('where', 'number', '>=', (float)$range->from);
    $this->crud->addClause('where', 'number', '<=', (float)$range->to);
  }

Backpack, Laravel, PHP, DB version:
backpack/base: 0.9.6
backpack/crud: 3.4.13
laravel/framework: 5.6.22

Seems like an easy enough PR to me (and probably should add to one of the unit tests too, though).

You're totally right. Thank you @bilfeldt - it makes perfect sense for this filter to also support min-only and max-only filtering. Just merged @pxpm 's PR which allows this. It'll be tagged in a few hours/days along with a few more improvements.
Cheers!

@bilfeldt do note that this to take advantage of this, you should change you query altering to:
            $range = json_decode($value);
            if ($range->from) {
                $this->crud->addClause('where', 'number', '>=', (float) $range->from);
            }
            if ($range->to) {
                $this->crud->addClause('where', 'number', '<=', (float) $range->to);
            }

Just updated the docs with that too.

how make limits number just 1 until 3, if more then can't


rewrite 02 ,96 ,35 ,39 api , test configuration.ValidateCheck is false

#29 test configuration.ValidateCheck is false
Signed-off-by: Bruce<EMAIL_ADDRESS>

@tsconn23 I had rewrite test name and description , test assertion  . please review ,thx

@weichou1229 @cloudxxx8 This is approved. I'll let you guys merge on your own timeline.


feat(headers): Internal rework and introduce HTTP Headers API

Closes gravitee-io/issues#6772

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

1.3.0
GitHub release

Your semantic-release bot :package::rocket:


Add assert to nil node on self:get_node

It will help to better debug and find issues earlier

Solved


Fix typo in link

Oops!

Sorry, I was editing this on my phone and didn't realize i messed up some of them until I went on my laptop.

No worries. Good spotting it second time!


not issue, but question

is there anyway you can show me how i can edit this or the openvpn.conf to make the samba files be seen from all clients on the vpn
as if they were in a lan

How can I add password and username every client? please help, Thank you.

This is not the OpenVPN support forum and spamming multiple issues without even caring to read isn't appreciated either.


Home page

Could we please have a home page that Franz can be configured to open to, this so that, when Franz opens, it doesn't expose some sensitive information to whomever may be around at the time. The home page could be any of blank or a valid URL or even a list of all the services as clickable icons.

It should be possible to configure Franz to have a Home icon showing at the top of the services pane on the left so that one can activate this view whenever. It should also be possible to configure if Franz always open to the Home screen or only on initial start-up.


getaddrinfo ENOTFOUND connct

We have written the needed data into your clipboard because it was too large to send. Please paste.

Error while starting Sync: request to https://vscode-auth.github.com/token?code=1bc6e71be94697ff138b&state=6672313d-1bfb-4575-b5a7-99ba51e93c65 failed, reason: getaddrinfo ENOTFOUND connect

The error message doesn't really help us troubleshooting, it even seems unrelated to this extension?

when check for VS updates net::ERR_PROXY_CONNECTION_FAILED it problem started from npm
npm ERR! code ENOTFOUND
npm ERR! errno ENOTFOUND
npm ERR! network request to https://registry.npmjs.org/gatsby-cli failed, reason: getaddrinfo ENOTFOUND proxyhostname
npm ERR! network This is a problem related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly. See: 'npm help config'

npm ERR! network This is a problem related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly. See: 'npm help config'

npm clearly indicates what your problem is, a misconfiguration of your HTTP(S) proxy

Another thing I like to add is: This is NOT the repository for the vscode editor.


Incompatible Versions - Client vs 3D Secure

General information

SDK version: Braintree Web (3.107.1), @paypal/react-paypal-js (8.6.0), @google-pay/button-react (3.1.0)
Environment: Sandbox
Browser and OS: Chrome 128.0.6613.121 (64-bit) on MacOS 14.6.1

Issue description
I'm upgrading versions of Braintree-Web (and other payment packages) from 3.84.0 to the latest. However when I toggle between payment methods I'm consistently getting back the following: "Client (version 3.84.0) and 3D Secure (version 3.107.1) components must be from the same SDK version.". However I do not see anything in my code which specifies 3.84.0?
Is this a known bug? The documentation does not make it clear to me what Client is and where I would find this.

I have downgraded versions to 3.92.1 and have logged out the braintree version we the application is creating the hostedFields. It is using 3.92.1 yet I am still getting theincompatible versions error

This was caused due to React-Paypal-JS


Feature/change gameboard completion meaning

The definition of a completed gameboard or assignment is changing from having correctly answered every question on the gameboard to having attempted every question on the gameboard.
As completion is used for filtering and sorting of some paginated endpoints we need to compute this in the back-end.

Pull Request Check List

[ ] ~Unit Tests & Regression Tests Added (Optional)~
[x] Removed Unnecessary Logs/System.Outs/Comments/TODOs
[ ] ~Added enough Logging to monitor expected behaviour change~
[x] Security - Injection - everything run by an interpreter (SQL, OS...) is either validated or escaped
[ ] ~Security - Data Exposure - PII is not stored or sent unencrypted~
[x] Security - Data Exposure - Test any altered or created endpoints using swagger
[ ] ~Security - Access Control - Check authorisation on every new endpoint~
[ ] ~Updated Release Procedure & Documentation (& Considered Implications to Previous Versions)~
[x] Peer-Reviewed

In rebasing my changes on such a long-running branch I have created a merging headache so the meaningful changes have been cherry picked onto the linked PR to avoid the conflicts: https://github.com/isaacphysics/isaac-api/pull/620.
I will close this PR to record comments.


Fix CI workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (6aa834d) to head (ae8d231).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #152   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           48        48
  Lines         1117      1117
  Branches       127       127
=========================================
  Hits          1117      1117

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Npx Prisma Generate Fails with Orm 4.0.0

> prisma_generate
> npx prisma generate

Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Error:
Could not format because the source could not be parsed:

line 2633, column 59 of .: Unterminated string literal.
     ╷
2633 │ final engine = _i4.BinaryEngine(schema: 'generator client {
     │                                                           ^
     ╵
line 2636, column 3 of .: Expected to find ')'.
     ╷
2636 │ \n}
     │   ^
     ╵
line 3357, column 33 of .: Unterminated string literal.
     ╷
3357 │ \n', datasources: datasources, );
     │                                 ^
     ╵
line 2633, column 41 of .: Expected to find ';'.
     ╷
2633 │ final engine = _i4.BinaryEngine(schema: 'generator client {
     │                                         ^^^^^^^^^^^^^^^^^^^
     ╵
line 2634, column 1 of .: Expected an identifier.
     ╷
2634 │ \n  provider        = "dart run orm"
     │ ^
     ╵
line 2634, column 1 of .: Expected to find ';'.
     ╷
2634 │ \n  provider        = "dart run orm"
     │ ^
     ╵
line 2634, column 23 of .: Expected to find ';'.
     ╷
2634 │ \n  provider        = "dart run orm"
     │                       ^^^^^^^^^^^^^^
     ╵
line 2635, column 1 of .: Expected an identifier.
     ╷
2635 │ \n  previewFeatures = ["fullTextSearch", "fullTextIndex"]
     │ ^
     ╵
line 2635, column 1 of .: Expected to find ';'.
     ╷
2635 │ \n  previewFeatures = ["fullTextSearch", "fullTextIndex"]
     │ ^
     ╵
line 2635, column 57 of .: Expected to find ';'.
     ╷
2635 │ \n  previewFeatures = ["fullTextSearch", "fullTextIndex"]
     │                                                         ^
     ╵
(2801 more errors...)

The exact prisma.schema works with 3.x

@AndryHTC Can you share the contents of schema.prisma where this error occurs? So that I can troubleshoot this issue.

Duplicate，See #344


Added biglake-catalog module

Added biglake-catalog module

Checklist

I applicable, I acknowledge that I have:

[X] Read the contributing guide
[X] Ran terraform fmt on all modified files
[X] Regenerated the relevant README.md files using tools/tfdoc.py
[X] Made sure all relevant tests pass

changes done

@apichick is this still planned for merge or should we close it?


Attempt at fixing circleci failure

@geeknoid I got it to pass now (had to change html-proofer, I have a PR for that change upstream but in the meantime the gem lock points to my version https://github.com/ldemailly/html-proofer/commit/a9fd7130ddf4bffc54ef938783d09a110e46574b )


Updated NGS tutorial for GTN Smörgåsbord

Completely redone NGS tutorial based on @tnabtaf @blankenberg and @mvdbeek SARS-CoV-2 material

This is almost done. Just waiting on Jupyter section. Will remove work-in-progress once done.

Don't understand why lint is failing. The images are there and I do not have images without alt text

I see some alt text changes in the latest commit. I'm guessing it was that, hopefully it passes how :)

It seems the failing images were in the introduction topic. I'm guessing you copied and pasted text from the introduction into this variant analysis topic tutorial? I've updated the links, hopefully they work now :)

It seems the failing images were in the introduction topic. I'm guessing you copied and pasted text from the introduction into this variant analysis topic tutorial? I've updated the links, hopefully they work now :)

Ahh . .. of course. Thanks

Oh, my paths are wrong still. Ok, I can fix this tomorrow, and we'll get this merged.

Thanks for the update @nekrut, this is great to see :)

Just ping me when it's done @nekrut and I'll have a look :)

done
Anton Nekrutenko
Professor of Biochemistry
and Molecular Biology
Penn State University
http://nekrut.bx.psu.edu http://nekrut.bx.psu.edu
(814) 826-9628
On Fri, Feb 12, 2021 at 8:57 AM Helena<EMAIL_ADDRESS>wrote:

Just ping me when it's done @nekrut https://github.com/nekrut and I'll
have a look :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/galaxyproject/training-material/pull/2339#issuecomment-778210548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAXYNEURM3FU666QY2IIHLS6UXVFANCNFSM4XPLNRZA
.


Fix raspberrypi image generation

First commit fixes the image generation failure, but it still didn't boot during my test.
More work coming up soon, hopefully.

Can confirm that the image builds successfully with this.

Any reason not to rebase this on the master so that the HSM test will pass?

I've changed it again, it should not mess with INITRAMFS_IMAGE_NAME actually...

Ok, finally made it to work. No more change here, but it needed these two PRs in meta-updater-raspberrypi:
https://github.com/advancedtelematic/meta-updater-raspberrypi/pull/25
https://github.com/advancedtelematic/meta-updater-raspberrypi/pull/26

@lbonn this should be caused by recent changes to master - maybe it is time to use kernel-artifact-names.bbclass (also introduced recently).

@Jmennius It's already using kernel-artifact-names (${KERNEL_FIT_LINK_NAME} comes from there). Did you think of a way to make it cleaner?


Modloader menu not showing

========================== Mod Loader 0.3.7 ==========================
Game version: GTA III 1.0 UNK_REGION
Loading basic config file modloader/.data/config.ini
Parsing command line
No aScreen address, not injecting a menu
Looking for plugins...
Reading plugins.ini
Loading plugin module "gta3\std.asi.dll"
Plugin module "gta3\std.asi.dll" loaded as gta3.std.asi Jan 16 2018 17:44:17 by LINK/2012
Starting up plugin "gta3.std.asi"
CLEO library version 20005 found at "scripts\III.CLEO.asi"
Loading plugin module "gta3\std.bank.dll"
Plugin module "gta3\std.bank.dll" loaded as gta3.std.bank Nov 12 2016 18:52:34 by LINK/2012
Starting up plugin "gta3.std.bank"
Warning: Failed to startup plugin "gta3.std.bank", unloading it.
Unloading plugin "gta3.std.bank"
Loading plugin module "gta3\std.data.dll"
Plugin module "gta3\std.data.dll" loaded as gta3.std.data Nov 12 2016 18:52:35 by LINK/2012
Starting up plugin "gta3.std.data"
Loading plugin module "gta3\std.fx.dll"
Plugin module "gta3\std.fx.dll" loaded as gta3.std.fx Nov 12 2016 18:52:38 by LINK/2012
Starting up plugin "gta3.std.fx"
Loading plugin module "gta3\std.movies.dll"
Plugin module "gta3\std.movies.dll" loaded as gta3.std.movies Nov 12 2016 18:52:50 by LINK/2012
Starting up plugin "gta3.std.movies"
Loading plugin module "gta3\std.scm.dll"
Plugin module "gta3\std.scm.dll" loaded as gta3.std.scm Nov 12 2016 18:53:09 by LINK/2012
Starting up plugin "gta3.std.scm"
Loading plugin module "gta3\std.sprites.dll"
Plugin module "gta3\std.sprites.dll" loaded as gta3.std.sprites Nov 12 2016 18:53:10 by LINK/2012
Starting up plugin "gta3.std.sprites"
Loading plugin module "gta3\std.stream.dll"
Plugin module "gta3\std.stream.dll" loaded as gta3.std.stream Jan 16 2018 17:44:17 by LINK/2012
Starting up plugin "gta3.std.stream"
Loading plugin module "gta3\std.text.dll"
Plugin module "gta3\std.text.dll" loaded as gta3.std.text Nov 12 2016 18:52:34 by LINK/2012
Starting up plugin "gta3.std.text"
Loading plugin module "gta3\std.tracks.dll"
Plugin module "gta3\std.tracks.dll" loaded as gta3.std.tracks Nov 12 2016 18:52:33 by LINK/2012
Starting up plugin "gta3.std.tracks"
Warning: Failed to startup plugin "gta3.std.tracks", unloading it.
Unloading plugin "gta3.std.tracks"
Reading profile named "Default" at "modloader.ini".
Using profile named "Default".
Using profile named "Default"
Scanning mods at "modloader"...
Updating gta3.std.data state...
Done updating gta3.std.data state.
Starting up filesystem watcher...
Mod Loader has started up!
Opening file for streaming "MODELS\GTA3.IMG"
Loading splash sprite "mainsc1.txd"
Loading default vehicles handling "HANDLING.CFG"
Loading default surface adhesion limits "DATA\SURFACE.DAT"
Loading default ped stats "PEDSTATS.DAT"
Loading default time cycle properties "TIMECYC.DAT"
Shutting down Mod Loader...
Shutting down filesystem watcher...
Shutting down menu...
Unloading plugin "gta3.std.fx"
Unloading plugin "gta3.std.asi"
Unloading plugin "gta3.std.data"
Unloading plugin "gta3.std.movies"
Unloading plugin "gta3.std.scm"
Unloading plugin "gta3.std.text"
Unloading plugin "gta3.std.sprites"
Unloading plugin "gta3.std.stream"
Mod Loader has been shutdown.

Hi. Unfortunately GTA III doesn't have an in-game menu.


Refactor Dockerfile

Thank you for providing the awesome project!
There are 2 updates in this PR

Update the apt key installation step to the latest.

ref: https://github.com/ros2/ros2_documentation/pull/1189

Somehow, the ROS installation using ros2-setup.bash is being executed twice, so make it once.

It seems to be added by this commit: https://github.com/Unity-Technologies/Robotics-Nav2-SLAM-Example/pull/7/commits/a5bb1878c8a92238a1335469ee8c8ba434c0c2d4#diff-c937b4b19f91982f17fbe3301b5aacb92dd64b712dc4beb2ba50ec53be2702a1R58-R59

By the way, the ros2_docker/Dockerfile appears to be a fork of the Dockerfile that I have published at docker-ros2-desktop-vnc under the Apache-2.0 license. Please consider noting the license instead of just writing "adapted from."
I know that some projects (to avoid any misunderstanding, I'm not talking about the project published at Unity-Technologies) are using code from my projects without any license notation regarding the use, but since most of my personal projects are not business, I pretended didn't see it.
However, since this project is a tutorial and will be used by many beginners, I think it is desirable to be polite about noting OSS use.

Thanks for the contribution, @Tiryoh, especially for bringing the licensing issue to my attention! We always want to make sure we're providing full attribution to open source creators whose work we're building off of.  I'll add your licensing information to our Third Party Notices file immediately and make sure the header on that file more directly references the license and license-holder. Please always feel free to call this out if you ever see it on another Unity repository; proper attribution of OSS creators is mandatory here.

@mrpropellers I saw the PR #38. Thank you for your quick and polite response!


Fix Coverage::isAvailable for case with both pcov and xdebug enabled

It's possible when both xdebug and pcov extensions enabled, but for different purposes: xdebug for debugging only, pcov for coverage. In such configuration ini_get('xdebug.mode') will not include coverage and thus pcov should be prioritised.

Can you re-open this pull request, but against Collision v7.x?


javax.print.PrintService

class file for javax.print.PrintService not found

That class (interface) is part of  Java API...

how to print pdf files with TCP IP

hi @Harisul44 ,
in my point of view, you can utilize the https://github.com/anastaciocintra/escpos-coffee-samples/tree/master/usual/tcpip-stream
and
https://github.com/anastaciocintra/escpos-coffee-samples/tree/master/miscellaneous/PdfPrinting
or
https://github.com/anastaciocintra/escpos-coffee-samples/tree/master/miscellaneous/PdfPrinting


fixes broken pycamhd imports

fixes broken pycamhd imports:
old:
import pycamhd as camhd

new:
import pycamhd.pycamhd as camhd

One of your commits has a diff that is too large to preview, which probably means that you changed one of the data fields (aka one of the images). I have switched to using the script/setup described here for my Jupyter notebooks which strips the images among other things:
http://timstaley.co.uk/posts/making-git-and-jupyter-notebooks-play-nice/
If you want to remove 783de8f I will pull these. I will also run everything through the stripper so that we don't run into this again.

Strange.  I just forked your repo and used the edit tool on github to change those lines.  But something went wrong somewhere, OBVI.
So you are going to strip the images from pycamhd.ipynb, right?


dualRXTX fails for LimeNET micro

The command fails with the following error
$ sudo ./dualRXTX
Devices found: 1
Reference clock 30.72 MHz
Number of RX channels: 1
Number of TX channels: 1
Invalid channel number.

Running on a Raspberry Pi CM3 with ubuntu 20.04 LTS installed.
LimeSuite compiled with no errors from source.

It's probably because you're trying to run a MIMO example on a SISO platform; LimeNET Micro does not have a second channel.


Handle enabled flag per row for table actions properly (as a function) on extension API

We should handle enabled flag per row for table actions properly (as a function) on extension API, because right now it only handles a boolean.
https://github.com/aalves08/dashboard/blob/extension-enhancements/shell/core/plugin-helpers.js#L95-L98

Updated milestone to v2.7.next3 as the work is already done on a PR and up for review


Enable Power Tools for  VS 2017 and VS 2015 (also removes support for VS 2010)

Addresses #152
This PR shows what is required to move PowerTools to VS 2017 and VSIX 2/3.
Currently pressing F5 to build and deploy in VS 2015 fails, but it is possible to build a release VSIX using the following command line:
 msbuild PowerTools.sln  /p:configuration=Release /p:DeployExtension=false /v:m

Info about some of the changes is available here: https://docs.microsoft.com/da-dk/visualstudio/extensibility/how-to-migrate-extensibility-projects-to-visual-studio-2017
I have successfully installed the built release VSIX and tested with VS 2017 RC, see screenshots below:

@ErikEJ Erik, that's really great and thanks very much. However at the same time we need to keep providing the old VSIX format so that people working on older versions of VS will be able to install it there. We're not quite sure how we plan to do that yet (a lot of people are already off for the holidays). I'll discuss with @divega and we'll get back to you.

@lajones The produced VSIX is installable on VS 2012 and higher! (VSIX 3 is "backwards compatible") - it just contains a couple of new json files in addition to the files in a VSIX2 package - I attach my test one for your information (or even test?)

Cool. I did not think that would be the case. Let me have a chat with @divega and we'll let you know.

@divega Let me know what concrete action you would like me to take on your comments ? 😄

@ErikEJ @divega - you reference the RC2 version of Microsoft.VSSDK.BuildTools in a few places. The release of RC3 is already out and RTW should be coming soon.  Can we wait on this, update the references to Microsoft.VSSDK.BuildTools to the RTW version once it's available and then check in? (The rest of it all looks fine to me).

@lajones  Yes, let us just wait for VS 2017 RTW

@lajones @bricelam Updated to RTW build tools!

@ErikEJ @bricelam Brice should check that the new version numbers are good, but assuming so it all looks good to me. Thanks, once again, Erik for your contributions.

@divega @lajones @bricelam :shipit:  ???

Sorry. I will do this.

@lajones That sounds great, at lot of users have been waiting for this! :smile:

I am building on a VS 2015 machine without any issues ( see my top comment re msbuild command line). I worry about requiring 4.6

I can get that to work on a VS2015-only machine if I change the references to Microsoft.VisualStudio.ComponentModelHost, Microsoft.VisualStudio.Shell.10.0 and Microsoft.VisualStudio.Shell.Design in both the src and test .csproj files from 10.0 to version 14.0. Were you building on a machine which had the 10.0 versions of those DLLs in the GAC?
In any case if you're OK with those changes then I can go ahead with that.

@lajones If you upgrade to version 14.0, the extension will only run on VS 2015 and later - in that case we need to update the target version range in the vsixmanifest - ideally, you should build on a machine with the lowest VS version that will be supported installed (and yes, I built on a machine with VS 2010 installed) - VS has binding redirects for the older versions of the references in later versions.
I think it would be  a good goal to support VS 2012 and onwards, but on the other hand.... If you decide on only supporting VS 2015 and later,  this line need to change from:
 <InstallationTarget Id="Microsoft.VisualStudio.Pro" Version="[11.0,16.0)" />

to
<InstallationTarget Id="Microsoft.VisualStudio.Pro" Version="[14.0,16.0)" />

Thought about it (went for a walk with the 🐩 ) - I am fine with those csproj changes + the manifest version range change 😄

We thought about it too :-). And we think you were right i.e. we'd rather inconvenience developers of this code than users of the VSIX it produces. But not sure we want to require the 10.0 DLLs. I've installed VS2012 and that works if I update the above to the 11.0 versions instead of the 14.0 ones. So a developer would have to install both versions of VS but we think that's preferable to not providing a VSIX for VS2012. Also that means I will not need to change the InstallationTarget. Does that sound good?

Excellent :applause:

Checked in with commit ee7c5220b.

Thanks, happy to make my first contribution to the Github  EF6 repo - it is complex, magic stuff those extensions

Is the plan to release this primo March?

There is a bit of a compliance process to go thru on our end before publishing, I'm working thru that now. I hope to get it out in Feb.

Great news, thanks

@ErikEJ We were testing this out with the latest bits and it was working on VS2012, but we are seeing the failures below on VS2015 and VS2017 (RTM). Are you seeing this? (You can make the first error go away by manually installing the EntityFramework package but the second one still persists).
System.TypeLoadException: Could not load type 'NuGet.NuGetConstants' from assembly 'NuGet.VisualStudio'.
   at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMarkHandle stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName, ObjectHandleOnStack type)
   at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName)
   at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark)
   at System.Type.GetType(String typeName, Boolean throwOnError)
   at Microsoft.DbContextPackage.Extensions.ProjectExtensions.InstallPackage(Project project, String packageId)
   at Microsoft.DbContextPackage.Handlers.ReverseEngineerCodeFirstHandler.ReverseEngineerCodeFirst(Project project)

One or more errors occurred while processing template 'Entity.tt'.
error : An exception was thrown while trying to compile the transformation code. The following Exception was thrown:
System.ArgumentException: Empty path name is not legal.
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at System.IO.File.OpenRead(String path)
   at Roslyn.Utilities.FileUtilities.OpenFileStream(String path)
   at Microsoft.CodeAnalysis.MetadataReference.CreateFromFile(String path, MetadataReferenceProperties properties, DocumentationProvider documentation)
   at Microsoft.VisualStudio.TextTemplating.CompilerBridge.<>c.<.ctor>b__15_0(String x)
   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.<UnionIterator>d__66`1.MoveNext()
   at System.Linq.Enumerable.<UnionIterator>d__66`1.MoveNext()
   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at System.Collections.Immutable.ImmutableArray.CreateRange[T](IEnumerable`1 items)
   at Microsoft.CodeAnalysis.ImmutableArrayExtensions.AsImmutableOrEmpty[T](IEnumerable`1 items)
   at Microsoft.CodeAnalysis.Compilation.ValidateReferences[T](IEnumerable`1 references)
   at Microsoft.CodeAnalysis.CSharp.CSharpCompilation.WithReferences(IEnumerable`1 references)
   at Microsoft.CodeAnalysis.CSharp.CSharpCompilation.CommonWithReferences(IEnumerable`1 newReferences)
   at Microsoft.VisualStudio.TextTemplating.CompilerBridge.PrepareNewCompilation()
   at Microsoft.VisualStudio.TextTemplating.CompilerBridge.Compile()
   at Microsoft.VisualStudio.TextTemplating.TransformationRunner.Compile(String source, String inputFile, IEnumerable`1 references, Boolean debug, SupportedLanguage language, String compilerOptions)

@lajones @divega  How about removing this obsolete feature? (replaced by latest EF Tools and/or the EF Reverse POCO Template) - happy to do a new PR to that effect

@ErikEJ Other than making only the minimal necessary changes, the main reason to keep the feature was that some customers had expressed a preference for the code PowerTools generate, e.g. using EntityTypeConfiguration<T> classes. I don't think this is a sure strong argument though.
Unless we can fix the feature easily, I agree removing it (or even disabling it selectively for VS 2017 and VS 2015) is a valid option.
I would like to hear what @bricelam and @ajcvickers think about this.

I've been wanting to get rid of it ever since we productized the feature as part of the designer. For those who prefer it, they'll probably find the EF Reverse POCO Template just as satisfying...

@divega I think customers wanting Configuration classes will benefit from EF Reverse POCO, not only does it support this, but you can also switch to DataAnnotations (in addition to 100s of other features, like stored procedure and function mappings)

Sounds good to me.

@divega on the other hand, just disabling for vs2015/2017 would cause minimal disruption/breaking changes (as these versions were not supported before) !?

I think at this point I would opt for removing it completely. It's been obsolete for a long time now and it's  one less thing to maintain going forward.

@ajcvickers @divega PR coming up!

I have submitted PR #213 - hope that this can be released at some point in time?

@ErikEJ Hi, we can I download the vsix file. I cannot find it.

@rudolph2907 See my earlier comment in this thread with a link to the file.

Thanks found it.

Hello,
When is it planned for the new version of the EF Power Tools (supporting VS17) to be released to the VS Marketplace?
Thanks and regards,
Antoni

@antoni-marasek If you need it now, you can download: https://github.com/aspnet/EntityFramework6/pull/154#issuecomment-267394589

@ErikEJ @antoni-marasek I'm slightly surprised this isn't being built and pushed to the myget nightly feed along with the nuget package.

@CZEMacLeod It is not a Nuget package, so it does not belong in a Nuget feed (it is a VSIX - VS extension)

@ErikEJ Thank you for your response.
Is the version referenced in #154 stable and ready to be used in the commercial projects or is it still a beta version?
In Visual Studio prior to 2017 (e.g. 2015) Power Tools used to be downloadable from the VS Marketplace (i.e. using the Tools -> Extensions menu item available directly in VS). Is it going to be the case also for VS17 and the new version of the plugin?

@antoni-marasek It is still at beta, and a couple of rev-eng features have been removed in the release version- I am sure the team intend to release to the MarketPlace, as stated here: https://blogs.msdn.microsoft.com/dotnet/2017/05/23/announcing-ef-6-2-beta-1/ - not sure what is holding them back, as no public test/beta effort is taking place for the PowerTools @divega  ?

@ErikEJ I am well aware that the VSIX is not a NuGet package. However MyGet has a special 'feed' section for vsix packages which can be used in Visual Studio as an extension source.
See https://www.myget.org/vsix and http://docs.myget.org/docs/walkthrough/getting-started-with-vsix
It has had this feature since 2016.
I had already added https://www.myget.org/F/aspnetwebstacknightly/vsix/ as a source for Visual Studio assuming you were pushing your nightly builds of both the nuget package and the vsix package from your CI.

Sorry, misunderstood... I am not on the EF team, but doing this sounds like a good idea - I do the same for my VS extension: https://github.com/ErikEJ/SqlCeToolbox/wiki/Subscribing-to-latest-"daily"-build

@ErikEJ the main blocker for this the last time we tried was to find a certificate that we could sign the VSIX with which would work across all the supported versions of Visual Studio.
We will try again (when we are not heads down with EF Core 2.0 work), but ultimately we may leave this as source code only that you can build and install in your machine as you did.

Is the version referenced in #154 stable and ready to be used in the commercial projects or is it still a beta version?

We don't have any plans to ever move EF PowerTools out of the beta phase.

@divega My extension is not signed - could "someone" else host a "homebuilt" version on MarketPlace? Or on their web site? Or would that break too many rules?

I don’t know the exact rules for the market place, but we can’t ship something without signing it.


Make bool's deserialization non-malleable

Description
Makes the deserialization of bool non-malleable. Now the only strings that can be deserialized as a boolean are the encodings of 0u8, 1u8.
closes: #95
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (main)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[x] Updated relevant documentation in the code
[x] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

Small comment: instead of arbitrarily mutating the data, why not try mutating it to every possible byte value? That’ll remove the dependence on ark-ff

That works for testing bool, but wouldn't generalize if we wanted to test this for any data type with a larger serialization.
We have no other data type implemented in serialize (AFAIK) which has a non-1-to-1 encoding to test for atm, so maybe that point is moot. (Also for the current API to generalize well, we'd probably need to change valid_mutation to return type T instead of a valid/invalid boolean)
Maybe it'd make sense to refactor this method out of tests, and do the above change, so other types (e.g. Fields) can check this?


<EMAIL_ADDRESS>doesn't work (EUNSUPPORTEDPROTOCOL, Unsupported URL Type "catalog:")

As per https://stackoverflow.com/q/78755320/51685:
~ $ docker run -it node:20 bash
root@767680ea83e2:/# node --version
v20.11.1
root@767680ea83e2:/# npx npx codemod react/findDOMNode --target sr^C
root@767680ea83e2:/# npx codemod react/findDOMNode --target src
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
npm ERR! code EUNSUPPORTEDPROTOCOL
npm ERR! Unsupported URL Type "catalog:": catalog:
npm ERR! A complete log of this run can be found in: /root/.npm/_logs/2024-07-16T15_16_05_184Z-debug-0.log

root@767680ea83e2:/# npx<EMAIL_ADDRESS>react/findDOMNode --target src
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
{"message":"libsecret-1.so.0: cannot open shared object file: No such file or directory"}
root@767680ea83e2:/#

thanks for reporting this issue. here is the link to our npm release and i dont know where 0.11.29 even come from! :D
feel free to chat more in slack if you have more questions.

Hi there.
If you don't know where 0.11.29 came from, I would recommend contacting NPM immediately to ask for audit logs, because that would indicate someone else has your NPM publishing credentials, published that version, and then yanked it.
Extracted from the container where I reproduced the above bug, here's an extract of the NPM registry response that mentions 0.11.29.
{
  "name": "codemod",
  "dist-tags": {
    "latest": "0.11.29"
  },
  // ...
  {
    "0.11.28": {
      "name": "codemod",
      "version": "0.11.28",
      "dependencies": {
        "keytar": "^7.9.0",
        "blessed": "^0.1.81",
        "esbuild": "^0.23.0",
        "prettier": "^3.2.5",
        "prettyjson": "^1.2.5",
        "@ast-grep/cli": "^0.24.0",
        "@ast-grep/napi": "^0.24.0"
      },
      "devDependencies": {
        "diff": "^5.1.0",
        "glob": "^10.4.1",
        "open": "^8.4.2",
        "axios": "^1.6.8",
        "memfs": "^4.6.0",
        "yargs": "^17.6.2",
        "semver": "^7.6.2",
        "vitest": "^1.0.1",
        "valibot": "^0.24.1",
        "inquirer": "^9.2.16",
        "prettier": "^3.2.5",
        "ts-morph": "18.0.0",
        "unzipper": "^0.11.6",
        "columnify": "^1.6.0",
        "form-data": "^4.0.0",
        "@types/diff": "^5.0.3",
        "@types/node": "18.11.9",
        "cosmiconfig": "^8.3.6",
        "@types/yargs": "^17.0.13",
        "@types/semver": "^7.5.8",
        "terminal-link": "^3.0.0",
        "@types/blessed": "^0.1.25",
        "@types/inquirer": "^9.0.7",
        "@types/unzipper": "^0.10.9",
        "@types/columnify": "^1.5.4",
        "@types/prettyjson": "^0.0.33",
        "@codemod-com/runner": "1.0.6",
        "@types/cli-progress": "^3.11.5",
        "@vitest/coverage-v8": "^1.0.1",
        "exponential-backoff": "^3.1.1",
        "@codemod-com/filemod": "2.0.3",
        "@codemod-com/printer": "1.0.1",
        "@codemod-com/telemetry": "1.1.0",
        "@codemod-com/utilities": "1.1.6"
      },
      "bin": {
        "codemod": "dist/index.cjs"
      },
      "dist": {
        "shasum": "d1cd2ef4f0734ed35a44794dce7fd62a40562249",
        "tarball": "https://registry.npmjs.org/codemod/-/codemod-0.11.28.tgz",
        "fileCount": 4,
        "integrity": "sha512-ugmN3vh6XYv2KvP3jCiT3xtuAPd76Q5fd6V8vhRHgWHeuZc1ltW3evn0AEbIfXvAFvZxXtT5ZcDvTJOfH2MtcA==",
        "signatures": [
          {
            "sig": "MEUCIFurJeo3uRoAMUMLt7MyxBZZG6bmtsSSxQo96TDa8GoiAiEA5TmLN3WDbDIdosDImApTLe+glb272IKrbRFrw8j43fs=",
            "keyid": "SHA256:jl3bwswu80PjjokCgh0o2w5c2U4LhQAE57gj9cz1kzA"
          }
        ],
        "unpackedSize": 20970251
      },
      "engines": {
        "node": ">=18.5.0"
      }
    },
    "0.11.29": {
      "name": "codemod",
      "version": "0.11.29",
      "dependencies": {
        "@ast-grep/cli": "catalog:",
        "@ast-grep/napi": "catalog:",
        "esbuild": "^0.23.0",
        "blessed": "catalog:",
        "keytar": "catalog:",
        "prettier": "^3.2.5",
        "prettyjson": "catalog:"
      },
      "devDependencies": {
        "@types/blessed": "catalog:",
        "@types/cli-progress": "catalog:",
        "@types/columnify": "catalog:",
        "@types/diff": "catalog:",
        "@types/inquirer": "catalog:",
        "@types/node": "18.11.9",
        "@types/prettyjson": "catalog:",
        "@types/semver": "^7.5.8",
        "@types/unzipper": "catalog:",
        "@types/yargs": "catalog:",
        "@vitest/coverage-v8": "catalog:",
        "axios": "catalog:",
        "columnify": "catalog:",
        "cosmiconfig": "catalog:",
        "diff": "catalog:",
        "exponential-backoff": "catalog:",
        "form-data": "catalog:",
        "glob": "catalog:",
        "inquirer": "catalog:",
        "memfs": "^4.6.0",
        "open": "catalog:",
        "prettier": "^3.2.5",
        "semver": "^7.6.2",
        "terminal-link": "catalog:",
        "ts-morph": "18.0.0",
        "unzipper": "catalog:",
        "valibot": "catalog:",
        "vitest": "^1.0.1",
        "yargs": "catalog:",
        "@codemod-com/filemod": "2.0.3",
        "@codemod-com/printer": "1.0.1",
        "@codemod-com/runner": "1.0.6",
        "@codemod-com/utilities": "1.1.6",
        "@codemod-com/telemetry": "1.1.0"
      },
      "bin": {
        "codemod": "dist/index.cjs"
      },
      "dist": {
        "integrity": "sha512-AxLrjNwxXhe3bVaf3yq+0KzLKksGZXjColBkgxx4vDGPbppVNZlyi/SqAvIPM/KS1k8KKmzlYz6DB5jEwoaxNw==",
        "shasum": "81e4b50c0c8693b724fcc00bef24b0862f361f4d",
        "tarball": "https://registry.npmjs.org/codemod/-/codemod-0.11.29.tgz",
        "fileCount": 4,
        "unpackedSize": 20971037,
        "signatures": [
          {
            "keyid": "SHA256:jl3bwswu80PjjokCgh0o2w5c2U4LhQAE57gj9cz1kzA",
            "sig": "MEQCIFyMYrH4ZNYnV82HT4zCRZ3ZjtUkm3fbu8Gkhl4jkzbwAiAlFVrzfDq/m+ffQApzcvUkHtFmGu8gj5f8zjFehYnV+g=="
          }
        ]
      },
      "engines": {
        "node": ">=18.5.0"
      }
    }
  },
  "modified": "2024-07-16T11:51:50.403Z"
}

Thanks. Mostly likely it's been my team. Might have unpublished it right after publishing the last version. Will double check and confirm. Thanks again.
Does the current version now work for you?

Might have unpublished it right after publishing the last version.

For what it's worth, the version had been up for 3 hours at the time the question was asked on Stack Overflow.

Does the current version now work for you?

I don't know, I'm not an user of the tool, I just wanted to let you know that this version was broken as I found that out via the SO question 😄

Thank you so much for reporting this. Will get to the bottom of this soon.
(Btw, hope we can have you as our user sometime soon too 😀 for any large refactoring, migrations, cleanups, and even code mining keep us in mind 😉 🍻)

(Btw, hope we can have you as our user sometime soon too 😀 for any large refactoring, migrations, cleanups, and even code mining keep us in mind 😉 🍻)

👍 I'll definitely keep you in mind the next time I need to bulldoze some code!  Thanks for reaching out. Accidentally, during publishing v0.11.29 of the CLI, pnpm did not replace catalog: version specifiers with their appropriate versions which led to this release being broken. We did not notice that right away, so we indeed unpublished it in couple hours.
As for the libsecret error I can see in your logs, we have seemingly fixed this issue in one of our previous releases, the intended behaviour is that the software should now throw an explanatory error for Linux users when the program attempts to use keytar package that relies on libsecret being available in the system and fails (see https://github.com/codemod-com/codemod/blob/284166bb7202069e827f7d4be4d25126b4f6248d/apps/cli/src/utils.ts#L18). One explanation for that would be that it fails literally at the moment of resolving keytar module (meaning if the node process ever goes into the file where this library is imported, it will panic because keytar has certain instantiation logic (?)), but we have not extensively tested that behaviour, since most of our team are macOS users and only one of our users ever reported a similar error related to libsecret.
We will debug this issue and see how we can resolve it. It would help if you provide us with your OS specs for debugging purposes. Thanks.

@akx fair enough, linux image with node:20 should do. we will put some effort to debug this so that users see the instructions on how to act when libsecret is not present in the system. i'll keep this open for reference until the issue is fixed.

Fixed in https://github.com/codemod-com/codemod/pull/1160


UI getting crash in  component having  with custom icons

Describe the bug
When user tries to filter data from clr-datagrid component, having clr-dg-placeholder with custom icon then,  while filtering somehow UI is getting crash with  below error.
Uncaught TypeError: Cannot read property 'indexOf' of undefined at HTMLElement._shapeTemplateSubscription (scripts.ed758b85cef043972290.js:65) at HTMLElement.s.attributeChangedCallback (scripts.ed758b85cef043972290.js:9) at l.invoke (polyfills.a75a54c75e112fdb669a.js:13) at a.runGuarded (polyfills.a75a54c75e112fdb669a.js:13)
How to reproduce
Please check with https://stackblitz.com/edit/clarity-light-theme-v5-ggw3as
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/clarity-light-theme-v5-ggw3as
Click on filter text box, search anything  then clear search field
Repeat step 2 for few times.
See error UI is getting crash with error Error in /turbo_modules/<EMAIL_ADDRESS>(614:55)
Cannot read property 'indexOf' of undefined

Expected behavior
There should not be any UI crash or console exception while filtering data-grid.
Versions
Clarity project:

[] Clarity Core 3.1.5
[] Clarity Angular/UI 4.0.3

Clarity version:

[] v4.x 4.0.3

Framework:

[] Angular

Framework version:

Angular 11

Device:

Reproducible on any device with any OS.

there's a lot going on in that stackblitz and right now stackblitz is crashing everytime i fork and try to edit.
so i rebuilt the app in Angular 11 like you had. I think the biggest difference is how the custom icons are being loaded in. there's no need to make that a service.
the demo in the link above works and i was able to activate the toggle button you had commented out.
closing because this isn't a clarity issue. but you can fork that repository, run it locally, and get where you need to go from there.


Feauture Adding Linelist_Appointments

@Marymary-dev please rem to add the new table and columns to the Dictionaries


Preview: upgrade to ocamlformat.0.18.0

This is a preview of the not-yet-released ocamlformat.0.18.0, please wait until the package is published in opam to merge this PR.
The diff is composed of:

empty lines between one-liner class items are removed (now consistently with structure items and signature items) when module-item-spacing = compact

Rebased to prod the CI. Will merge once it's happy.


Create now.json

Awesome, thanks!

For reference on what a now.json is for: https://zeit.co/docs/v2/advanced/configuration


Recipients are broken since 1.19.1

I believe 1.19.1, which featured https://github.com/messagebird/php-rest-api/pull/103 should not be released as a minor change, but rather in a new major version release, as it's not backwards compatible:

Since the types changed, all code using recipients as arrays broke.
Previously, this worked fine:

$messageBirdMessage->recipients = [$number];

But that now breaks.
2. The Recipient class defines the $recipient property as integer (https://github.com/messagebird/php-rest-api/blob/master/src/MessageBird/Objects/Recipient.php#L16), while that formerly accepted a string, which can still be seen in the examples (https://github.com/messagebird/php-rest-api/blob/master/examples/message-create.php#L9). The original documentation also shows that string values are allowed (https://developers.messagebird.com/quickstarts/sms/send-sms-curl/), so why this changed is unclear. This doesn't break code as much, as it's not strictly enforced by the compiler, but does break PHPStan checking my code.
3. Examples have not been updated, so documentation is now very confusing for new users.
All-in-all, I think the best course of action now would be to:

Revert the changes in 1.19.1, releasing 1.19.2 as a fix
Fix the documentation, examples and types
Release 2.0 with this change again

Hi @pascaldevink! You are right, that was my oversight 🤦  I have reverted the PR and released v1.19.2 with the fix.
@JohnstonCode fyi. I think your code was good, but we both missed the assignment side of this. Please feel free to submit an improved version of your PR with no changes to BC again 🙏

@modprobe Thanks! The change is certainly good, no argument there 🙂


protractor hangs when accessing elementFinder in async / await

Node Version: v8.9.1
Protractor Version: ~5.4.0
Angular Version: ^6.1.2
Browser(s): Chrome
Operating System and Version Windows 10

I have a problem where protractor will hang and give a 30s timeout error whenever I try to access any elements. This happens if I enable signalR, or if any http requests are made while the test is running.
The option to prevent the hang is to set
browser.ignoreSynchronization = true;
I tried to add the workaround described in this issue (comment from Oct17 2017), but with no luck
https://github.com/angular/protractor/issues/2227
This is the error I see on a
		let elm = element(by.tagName(tagname));
		if (await elm.isPresent()) {
			...
		}
		...

{ ScriptTimeoutError: script timeout: result was not received in 30 seconds
  (Session info: chrome=70.0.3538.102)
  (Driver info: chromedriver=2.43.600210 (68dcf5eebde37173d4027fa8635e332711d2874a),platform=Windows NT 10.0.14393 x86_64)
    at Object.checkLegacyResponse (C:\my-app\node_modules\selenium-webdriver\lib\error.js:546:15)
    at parseHttpResponse (C:\my-app\node_modules\selenium-webdriver\lib\http.js:509:13)
    at doSend.then.response (C:\my-app\node_modules\selenium-webdriver\lib\http.js:441:30)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
From: Task: Protractor.waitForAngular() - Locator: By(css selector, back-to-home)
    at Driver.schedule (C:\my-app\node_modules\selenium-webdriver\lib\webdriver.js:807:17)
    at ProtractorBrowser.executeAsyncScript_ (C:\my-app\node_modules\protractor\built\browser.js:425:28)
    at angularAppRoot.then (C:\my-app\node_modules\protractor\built\browser.js:456:33)
    at ManagedPromise.invokeCallback_ (C:\my-app\node_modules\selenium-webdriver\lib\promise.js:1376:14)
    at TaskQueue.execute_ (C:\my-app\node_modules\selenium-webdriver\lib\promise.js:3084:14)
    at TaskQueue.executeNext_ (C:\my-app\node_modules\selenium-webdriver\lib\promise.js:3067:27)
    at asyncRun (C:\my-app\node_modules\selenium-webdriver\lib\promise.js:2927:27)
    at C:\my-app\node_modules\selenium-webdriver\lib\promise.js:668:7
    at <anonymous> name: 'ScriptTimeoutError', remoteStacktrace: '' }

Are you using the SELENIUM_PROMISE_MANAGER: false, in your configuration file?

No, I'm using the default config.
Should I add this in onPrepare ?
promise.USE_PROMISE_MANAGER = false as described here ? https://stackoverflow.com/questions/46213650/protractor-disable-selenium-promise-manager-in-one-suite
let SpecReporter = require('jasmine-spec-reporter').SpecReporter;

exports.config = {
	allScriptsTimeout: 30000,
	specs: [
		'./e2e/**/*.e2e-spec.ts'
	],
	capabilities: {
		'browserName': 'chrome'
	},
	directConnect: true,
	baseUrl: 'http://localhost:4200/',
	framework: 'jasmine',
	jasmineNodeOpts: {
		showColors: true,
		defaultTimeoutInterval: 60000,
		print: function () { }
	},
	useAllAngular2AppRoots: true,
	beforeLaunch: function () {
		require('ts-node').register({
			project: 'e2e'
		});
	},
	onPrepare: function () {
		jasmine.getEnv().addReporter(new SpecReporter({ spec: { displayStacktrace: true } }));
	},
	logLevel: 'DEBUG'
};

I would also use the waitForAngularEnabled method instead of ignoreSynchronization property. For the next version of Protractor, we are probably going to remove ignoreSynchronization.
Also in your config:
exports.config = {
  SELENIUM_PROMISE_MANAGER: false,
  ...
}


hub asking for username and password when used with ssh protocol

Command attempted:
hub clone dot-files
What happened:
promts for username
More info:
git config --global hub.protocol
ssh
hub --version
git version 2.37.0 (Apple Git-136)
hub version refs/heads/master
OS
macOS 12.6 (21G115)
Giving full URL works fine for e.g. hub clone git@github.com:metapraveen/dot-files.git clones the repo via ssh auth

hub is failing for me like this, too, anytime I try something that requires authentication. git clone and git push work, but not the hub versions of these commands.
(master) % hub create
github.com username: chaimleib
github.com password for chaimleib (never stored):
Error creating repository: Not Found (HTTP 404)
Not Found

% hub --version
git version 2.38.2
hub version 2.14.2

I've tried both of these:
% git config --global hub.protocol https
% git config --global hub.protocol ssh

I have URL replacement on to force ssh instead of https on github, but both above configs also fail, even if I turn replacement off:
[url "git@github.com:"]
	insteadOf = https://github.com/

macOS 10.15.7

Yeah, hub has not been usable for anything for the last six months. Only auth errors and no prompting to authenticate using browser or similar.

OK It's not just me. Same issue.

That is a workaround, not a fix. A better workaround is to just ditch Hub for the official GitHub CLI, as Hub is essentially deprecated: https://mislav.net/2020/01/github-cli. All of Mislav's efforts go into that these days and I have been able to transition from all current Hub commands to gh very easily (i.e.  gh pr checkout <issue>, gh browse, ...)


Cache dependency file checksums

Fixes Issue
https://github.com/jeremylong/DependencyCheck/issues/2485
Description of Change
Naive solution to avoid recomputing file checksums. Open to suggestions about a better place for the cache to sit. Possibly Checksums?
Have test cases been added to cover the new functionality?
No

Done. If/when you're happy with the change @jeremylong, I can rebase out the first two commits.


feat(base-modal): prevent content shift when hiding the scrollbar

Motivation and context
Showing the layer hides the main scrollbar causing a content shift:
https://user-images.githubusercontent.com/194784/190706233-b7ac7614-0149-4d54-98ee-7c2e98916b89.mov
I haven't been able to test this PR, but the expected result is like this:
https://user-images.githubusercontent.com/194784/190706551-3043f46c-eea1-45ba-97e5-48d2eed326e1.mov
Also, there was a line that confuses me:
https://github.com/empathyco/x/blob/980f3e6628cfedad38f5f5ceeefbb24544b38cb6/packages/x-components/src/components/modals/base-modal.vue#L122-L124
The last line overrides all logic regarding previousBodyOverflow and previousHTMLOverflow.

[ ] Dependencies. If any, specify:
[ ] Open issue. If applicable, link:

Type of change

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

What is the destination branch of this PR?

[x] Main
[ ] Other. Specify:

How has this been tested?

Tests performed according to testing guidelines:
Checklist:

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

Thanks @javieri-empathy for your time.
I close this PR because it needs a deeper investigation and define an expected behavior.

Some websites, with custom CSS changed the scrolling container from the html to the body:

Sadly this is possible and True
Also, I think the scrollbar gutter property should not be removed on enableScroll.


add: cmake-deb

Did you see this coming?

@Henryws
CMake Error at CMakeLists.txt:1 (cmake_minimum_required):
  CMake 3.25.2 or higher is required.  You are running version 3.25.1

😂
I guess I will wait for the patch update then 😅

Ok, make a new PR, and add incompatible to it.

No, I mean I wanted to use this updated cmake to update obs-backgroundremoval-git, but it requires an ever so slightly newer cmake 😅
It's ok, I guess debian and ubuntu will catch up in some time. then I'll update these pacscripts to 3.25.2


Work on dockerfile, clean, fix some errors

This doesn't solve all of the issues, but it gets you most of the way there, builds without errors.
Changes:

remove duplication of the copy
move copy up, ordering error
Deleted the line that made it build for production (it was skipping over all the dev dependencies, which you need)
remove moving node_modules up a directory

Feel free to close it out if it doesn't suit you! That said, it might be cleaner this way :) I think there might be changes needed to the index.js file as well, but I ran out of time to work on this tonight.

thanks for the help. i came to a similar conclusion. to recap our chat ... i'd prefer to have a multi-stage file that builds the app first, then in stage 2, just copies the prod files to the actual image. but for simplicity's sake, I'm going to stick with the smaller dockerfile as it's easier to grok for everyone.
So yeah --- what you did :)
This PR got smashed by a git force push i had to do, to clean up some other git unrelated git messes i had. Sorry it got closed without a comment originally. I'm fighting a few fires.
Thanks for the help!


Create color scheme that can be used as backgrounds

We need colors that match the scheme that can be used as backgrounds for various things.

Here's an attempt: http://www.colourlovers.com/palette/4285558/IndieWeb_Backgrounds

for what it's worth, I only see four colours in your sample above but I can tell it should be five because of the size of the first three boxes


Smoketest failure: Editor finds 'All References' to 'app'

Error: Timeout: get elements '.monaco-editor .zone-widget .zone-widget-container.peekview-widget.reference-zone-widget.results-loaded .body .ref-tree.inline .monaco-tree-row .reference' after 20 seconds.
      at /Users/roblou/code/vscode/test/smoke/out/vscode/code.js:142:23
      at Generator.next (<anonymous>)
      at fulfilled (out/vscode/code.js:8:58)
      at <anonymous>

Like https://github.com/Microsoft/vscode/issues/64829, another sneaky CSS selector change that comes with the new tree.


Typo in pubspec.yaml fonts leads to TypeError instead of helpful error message.

I can't run my app on my device,when i click the run button, there was no gradle build info tips. and
no error on my terminal,but only some info in my logs, floowing is the log
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter --no-color run --machine --track-widget-creation --device-id=GSL0216A23005048 lib\main.dart
exception
_TypeError: type 'YamlMap' is not a subtype of type 'Iterable'
#0      _validateFonts (package:flutter_tools/src/flutter_manifest.dart:409:47)
#1      _validateFlutter (package:flutter_tools/src/flutter_manifest.dart:354:9)
#2      _validate (package:flutter_tools/src/flutter_manifest.dart:312:9)
<asynchronous suspension>
#3      FlutterManifest._createFromYaml (package:flutter_tools/src/flutter_manifest.dart:45:40)
<asynchronous suspension>
#4      FlutterManifest.createFromString (package:flutter_tools/src/flutter_manifest.dart:40:12)
<asynchronous suspension>
#5      FlutterManifest.createFromPath (package:flutter_tools/src/flutter_manifest.dart:34:12)
<asynchronous suspension>
#6      FlutterProject._readManifest (package:flutter_tools/src/project.dart:144:60)
<asynchronous suspension>
#7      FlutterProject.fromDirectory (package:flutter_tools/src/project.dart:44:44)
<asynchronous suspension>
#8      FlutterProject.current (package:flutter_tools/src/project.dart:55:46)
#9      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:542:59)
<asynchronous suspension>
#10     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:489:33)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#12     _rootRun (dart:async/zone.dart:1124:13)
#13     _CustomZone.run (dart:async/zone.dart:1021:19)
#14     _runZoned (dart:async/zone.dart:1516:10)
#15     runZoned (dart:async/zone.dart:1463:12)
#16     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#17     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:480:20)
#18     CommandRunner.runCommand (package:args/command_runner.dart:196:27)
<asynchronous suspension>
#19     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#20     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#21     _rootRun (dart:async/zone.dart:1124:13)
#22     _CustomZone.run (dart:async/zone.dart:1021:19)
#23     _runZoned (dart:async/zone.dart:1516:10)
#24     runZoned (dart:async/zone.dart:1463:12)
#25     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#26     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#27     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:111:29)
#28     new Future.sync (dart:async/future.dart:224:31)
#29     CommandRunner.run (package:args/command_runner.dart:111:11)
#30     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#31     run.<anonymous closure> (package:flutter_tools/runner.dart:60:20)
<asynchronous suspension>
#32     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#33     _rootRun (dart:async/zone.dart:1124:13)
#34     _CustomZone.run (dart:async/zone.dart:1021:19)
#35     _runZoned (dart:async/zone.dart:1516:10)
#36     runZoned (dart:async/zone.dart:1463:12)
#37     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#38     runInContext (package:flutter_tools/src/context_runner.dart:50:24)
<asynchronous suspension>
#39     run (package:flutter_tools/runner.dart:51:10)
#40     main (package:flutter_tools/executable.dart:60:9)
<asynchronous suspension>
#41     main (file:///D:/FlutterSdk/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#42     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:298:32)
#43     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:171:12)

flutter doctor
[✓] Flutter (Channel beta, v1.4.9-hotfix.1, on Microsoft Windows [Version 6.1.7601], locale zh-CN)
    • Flutter version 1.4.9-hotfix.1 at D:\FlutterSdk\flutter
    • Framework revision 88fa7ea403 (2 weeks ago), 2019-04-11 14:01:46 -0700
    • Engine revision 4737fc5cd8
    • Dart version 2.2.1 (build 2.2.1-dev.4.1 None)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\dell\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: D:\AndroidStudio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] Android Studio (version 3.2)
    • Android Studio at D:\AndroidStudio
    • Flutter plugin version 31.3.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] Connected device (1 available)
    • FRD AL10 • GSL0216A23005048 • android-arm64 • Android 8.0.0 (API 26)

• No issues found!

Can you post your pubspec.yaml? it looks like a format error is triggering a type error (which we should catch anyway)

yeah, of course ,firstly, i check my pubspec.yaml and repair some error, so there is no red alert in my pubspec.file, but my project still report the same error. and  i'm sure my environment is correctly, because when i run demo app,it's normal.
pubspec.yaml:
name: flutter01
description: A new Flutter application.
The following defines the version and build number for your application.
A version number is three numbers separated by dots, like 1.2.43
followed by an optional build number separated by a +.
Both the version and the builder number may be overridden in flutter
build by specifying --build-name and --build-number, respectively.
In Android, build-name is used as versionName while build-number used as versionCode.
Read more about Android versioning at https://developer.android.com/studio/publish/versioning
In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
Read more about iOS versioning at
https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
dependencies:
flutter:
sdk: flutter
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^0.1.2
shared_preferences: ^0.5.1+1
fluttertoast: ^3.0.1
json_annotation: ^2.0.0
flutter_redux: ^0.5.3
device_info: ^0.4.0+1
connectivity: ^0.4.2
url_launcher: ^5.0.1
flutter_markdown: ^0.2.0
share: ^0.6.0+1
flutter_spinkit: ^3.1.0
flutter_webview_plugin: ^0.2.1+2
sqflite: ^1.1.1
pub_semver: ^1.4.2
flutter_statusbar: ^0.0.1
flutter_svg: ^0.11.0+1
photo_view: ^0.2.1
flutter_slidable: ^0.4.9
dio: ^2.0.15
flutter_localizations:
sdk: flutter
event_bus:  ^1.0.3
flutter_cache_manager: ^0.3.1
path_provider: ^0.5.0+1
permission_handler: 3.0.0
scoped_model: ^1.0.1
package_info: ^0.4.0+1
rxdart: ^0.21.0
android_intent: ^0.3.0
test: ^1.5.1
dev_dependencies:
build_runner: ^1.2.8
json_serializable: ^2.0.0
flutter_test:
sdk: flutter
For information on the generic Dart part of this file, see the
following page: https://www.dartlang.org/tools/pub/pubspec
The following section is specific to Flutter.
flutter:
The following line ensures that the Material Icons font is
included with your application, so that you can use the icons in
the material Icons class.
uses-material-design: true
To add assets to your application, add an assets section, like this:
assets:
- images/a_dot_burr.jpeg
- images/a_dot_ham.jpeg
An image asset can refer to one or more resolution-specific "variants", see
https://flutter.io/assets-and-images/#resolution-aware.
For details regarding adding assets from package dependencies, see
https://flutter.io/assets-and-images/#from-packages
To add custom fonts to your application, add a fonts section here,
in this "flutter" section. Each entry in this list should have a
"family" key with the font family name, and a "fonts" key with a
list giving the asset and other descriptors for the font. For
example:
fonts:
- family: Schyler
fonts:
- asset: fonts/Schyler-Regular.ttf
- asset: fonts/Schyler-Italic.ttf
style: italic
- family: Trajan Pro
fonts:
- asset: fonts/TrajanPro.ttf
- asset: fonts/TrajanPro_Bold.ttf
weight: 700

For details regarding fonts from package dependencies,
see https://flutter.io/custom-fonts/#from-packages
assets:
- static/images/logo.png
- static/images/default_img.png
- static/images/welcome.png
fonts:
- family: buttomIconStyle
fonts:
-asset: static/font/iconfont.ttf
------------------ 原始邮件 ------------------
发件人: "Jonah<EMAIL_ADDRESS>发送时间: 2019年4月29日(星期一) 上午10:33
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [flutter/flutter] I can't install my app  (#31764)
Can you post your pubspec.yaml? it looks like a format error is triggering a type error (which we should catch anyway)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

that's is my new log
------------------ 原始邮件 ------------------
发件人: "Jonah<EMAIL_ADDRESS>发送时间: 2019年4月29日(星期一) 上午10:33
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [flutter/flutter] I can't install my app  (#31764)
Can you post your pubspec.yaml? it looks like a format error is triggering a type error (which we should catch anyway)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

------------------ 原始邮件 ------------------
发件人: "Jonah<EMAIL_ADDRESS>发送时间: 2019年4月29日(星期一) 中午11:24
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [flutter/flutter] I can't install my app  (#31764)
This line:
-asset: static/font/iconfont.ttf
Should be:

asset: static/font/iconfont.ttf

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yay, I will repurpose this bug so we can add a better error message to the tool

thanks,I‘m look forward a better flutter tool
happy weekend.
------------------ 原始邮件 ------------------
发件人: "Jonah<EMAIL_ADDRESS>发送时间: 2019年4月29日(星期一) 中午11:37
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [flutter/flutter] I can't install my app  (#31764)
Yay, I will repurpose this bug so we can add a better error message to the tool
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

The bug is that on package:flutter_tools/src/flutter_manifest.dart:409:47 we do an unsafe cast from Object/dynamic to Iterable. We should perform an is check first and throw an error message instead.


Add GPU tests

This PR adds a simple gpu test, where the 2D diffusion test problem is solved with CuArrays.

bors try

Closes #114.


Improve CAPZ upstream VNet Peering

Motivation
In https://github.com/giantswarm/roadmap/issues/2011 we found out, that we can improve the VNet Peering in upstream CAPZ by enabling transit gateway.
Todo

[x] Implement change in our CAPZ fork https://github.com/kubernetes-sigs/cluster-api-provider-azure/issues/3183
[x] Submit a pull request to upstream CAPZ with the same change
[x] Release new CAPZ version from our fork (based on the latest CAPZ v1.8.2 and our VNet peering improvement)
[x] Release CAPZ app based on our fork

Outcome

We can easily and securely access the protected private network workload clusters through the management cluster.
We enabled to do this in an upstream CAPZ Release

Created upstream CAPZ issue here https://github.com/kubernetes-sigs/cluster-api-provider-azure/issues/3183

CAPZ app released from our fork, upstream PR merged and added to v1.9 milestone.