fix: component suspended while responding to input

I am opening this temporary PR to see what kinds of tests fail when I introduce a small change to the useSaveArtworkToArtworkLists hook that is used to give artwork grids save icons 💜.
While fixing failing tests for #11098 I noticed the following error in a variety of test suites, and I was surprised that most of them were not related to the act of saving an artwork.
A component suspended while responding to synchronous input. This will cause the UI to be replaced with a loading indicator. To fix, updates that suspend should be wrapped with startTransition.

      110 |     const view = renderWithWrappers(<TestRenderer />)
      111 |
    > 112 |     act(() => {
          |        ^
      113 |       console.log("FOO")
      114 |       const resolve = preloaded
      115 |         ? env.mock.queueOperationResolver

      at throwException (node_modules/react-test-renderer/cjs/react-test-renderer.development.js:8929:35)
      at throwException (node_modules/react-test-renderer/cjs/react-test-renderer.development.js:15604:7)
      at handleError (node_modules/react-test-renderer/cjs/react-test-renderer.development.js:15710:7)
      at renderRootSync (node_modules/react-test-renderer/cjs/react-test-renderer.development.js:15412:20)
      at callback (node_modules/react-test-renderer/cjs/react-test-renderer.development.js:2594:22)
      at callback (node_modules/react/cjs/react.development.js:2667:24)
      at flushActQueue (node_modules/react/cjs/react.development.js:2521:11)
      at actImplementation (node_modules/@testing-library/react-native/src/act.ts:30:25)
      at renderWithRelay (src/app/utils/tests/setupTestWrapper.tsx:112:8)
      at Object.renderWithRelay (src/app/Components/ArtworkGrids/ArtworkGridItem.tests.tsx:68:7)

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against b3c60b7c554220f38daf46fb5bcf4a58012ef3af


manual sha and slug specification for openshift prow ci

In the prow environment
make submit-coverage would fail with error
Unable to detect service, please specify sha and slug manually.

This PR ensures that sha and slug information are passed in manually.

Codecov Report

Merging #317 (a2c35d8) into master (9871610) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #317   +/-   ##
=======================================
  Coverage   41.36%   41.36%
=======================================
  Files           9        9
  Lines        1697     1697
=======================================
  Hits          702      702
  Misses        958      958
  Partials       37       37

Continue to review full report at Codecov.

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

/retest

/retest


Move request slotting to the spider middleware

With ScrapyZyteAPIDownloaderMiddleware the slot names are not changed now - I've checked with the Articles+DownloaderAwarePriorityQueue - all good.
Thanks @Gallaecio! Please let me know when it will be merged and ready to use.


[Feature] Cron jobs

Summary
As a user, I want to be able to run & manage cron jobs on Appsmith. I would like to define a cron schedule on which an action/workflow will get executed.
Motivation
A lot of tools require actions such as importing of data, executing workflows on a regular basis. Cron jobs are integral to powering such flows on a predictable basis.
Additional Context

Closing as it's a duplicate of #738


Always-active mouse move events are used for no good reason

From<EMAIL_ADDRESS>on December 28, 2009 16:08:51
What steps will reproduce the problem? Move the mouse - SQL.Table.move and SQL.Map.move fire while moving the mouse
around even though there are no tables, and nothing is being dragged (neither a table
nor a map item). This may be a contributor to dnd sluggishness in Opera. What is the expected output? What do you see instead? SQL.Table.move should be invoked only when dragging a table.
SQL.Map.move should be invoked only when dragging a map item. What version of the product are you using? On what operating system? svn rev 71 Please provide any additional information below. Patch is attached.
Attachment: events.patch
Original issue: http://code.google.com/p/wwwsqldesigner/issues/detail?id=54

From ondrej.zara on January 03, 2010 11:19:45
Commited in r72 , thanks!
Status: Fixed


Switching from let_it_rain to game_of_life causes error.

Maybe that's not a common usecase - but the second thing I tried after letting it rain was making the fallen rain turn in to new life (game_of_life), which failed with some error.

Hey, thanks for reporting that. I'm on holidays now, I'm going to look into that next week.

@felixniemeyer What was the error, as I failed to reproduced the issue

Hi,
Let me clarify: you still have to be in the letting_it_rain buffer and execute :CellularAutomaton game_of_life there and not switch back to the original buffer with :q in between.
I expect that what was rain to come to life according to the game of life rules then. Instead, I get this error:
Error executing Lua callback: ...llular-automaton.nvim/lua/cellular-automaton/manager.lua:71: ...llular-automaton.nvim/lua/cellular-automaton/manager.lua:52: Nested animations are forbidden
stack traceback:
        [C]: in function 'error'
        ...llular-automaton.nvim/lua/cellular-automaton/manager.lua:71: in function 'execute_animation'
        .../cellular-automaton.nvim/lua/cellular-automaton/init.lua:43: in function 'start_animation'
        ...rt/cellular-automaton.nvim/plugin/cellular-automaton.lua:16: in function <...rt/cellular-automaton.nvim/plugin/cellular-automaton.lua:15>```

Nested animations are forbidden. It's even stated in the error you pasted ;)

Ah that's a nested animation. Makes sense ;)
Thank you


Silence whitespace-only diffs.

Merging because as mentioned on #956 this is failing due to an unrelated issue.


Edit data source  warning error

I'm submitting a ...

[ ] Bug report
[ ] Feature requestdd
[1 ] Question / Support request

If you don't send bug report, remove sections below
For bug report please include this information:
Software Versions

Grafana
Zabbix
Grafana-Zabbix Plugin

4.4.3
3.2.6
3.6.1

What did you do?
hello,
Now that I am configuring the data source on the grafana page, the mysql data source has been added successfully, and when I add zabbix data, I am prompted to connect the error. Do I need to change the configuration file?

Please help me, thanks

Try to use Proxy mode instead of Direct.


Organism Name in right panel is not linked

Steps to Reproduce

Click on a Dataset to opened Dataset Single page
Click on More info button
Pay attention to Organism Name in Sample and Experiment Conditions section

Expected Results
Organism Name data should be linked
Evidence

@lauramarcos could you retest please?

Organism Name is still missing the link! Keeping this one opened

duplicate of #150. closing this as not all organism fields are meant to have a link.


Chapter 1 - Aside - Consider adding teardown / cleanup instructions

If the reader were to follow the instructions in Chapter 1 and then decide to no longer follow the book, what actions (if any) would be required by the reader to restore their system to the pre-book state?
If actions were required, consider listing them or adding a teardown script.
If no actions are required, consider highlighting that point and explaining why no further actions would be required.

actually this whole project runs on linux, in the video jonhoo mentioned that he's not sure whether this can run on win, macos or other unix-like system
I think you can consider buy an ecs / run a virtual machine to run it, if anything goes wrong, re-install the system image or just delete it then buy a new one.
I'm not reading this book but following the video, I've finished the first 2 parts and not planning to go through the 3rd part immediately, cause as jonhoo said, his videos are so-damn-long...
what I recommend (my learning process):

check the repo(https://github.com/jonhoo/rust-tcp) first, search the timeline to find the commit where jonhoo accomplished at the end of each video, read the code
try figure out the main idea of the code at current commit
you can write down what each variable means in comments, you know the project's var naming is so difficult to read... at least for me it's hard to tell that a connections actually means quad_to_connection hashmap
watch the video, I recommend you mainly follow the design idea demonstrating part, you can 10x speed go through the debugging part (follow the debugging part and try copy every movement is such a stupid thing, and miserable... just denote what test you can make)
"if you find yourself tired, you can get away and go back whenever you've restored power and feel interested again" --- jonhoo in "why my vids are so damn long"


live video doesn't play, but local .ts files play fine

Describe the bug
loading https://server/video.ts plays, however loading wss://server:444/video/ doesn't play live video.
<VirtualHost *:444>
ProxyPass "/video/" "ws://localhost:8082/"
ProxyPassReverse "/video/" "ws://localhost:8082/"
</VirtualHost
Error: WebSocket connection to 'wss://server:444/video/' failed: An SSL error has occurred and a secure connection to the server cannot be made.
To Reproduce
Run the jsmpeg node example, and proxy from apache in front of it to satisfy the SSL.
Any way to make live work? given the wss:// requirement?

See https://github.com/phoboslab/jsmpeg#streaming-via-websockets


Update numpy

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 86.203%

Files with Coverage Reduction
New Missed Lines
%

../../../../../opt/hostedtoolcache/Python/3.9.17/x64/lib/python3.9/site-packages/matid/classification/periodicfinder.py
8
98.23%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
2468

Relevant Lines:
2863

💛  - Coveralls


Firefox 61 beta breaks some effects

I'm using the beta channel of Firefox as my main browser at the minute, and I noticed that since I upgraded to version 61 (which came out in the last few days, I think?), a bunch of Tone.js effects no longer seem to work (Freeverb, JCReverb, PingPongDelay, etc.).
Here's a reproduction of the issue in JSFiddle - it's just the code example from the Freeverb docs page: https://jsfiddle.net/em6c13nr/
Not a huge issue for me, as I can use other browsers to test my code in the meantime, but I thought I should point it out before Firefox 61 makes its way to stable :)
EDIT: D'oh, it'd be helpful if I provided the error message, wouldn't it:
InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable

Good to know. will investigate! lmk if you find the exact spot where that's being thrown from.

Looks like simply commenting out these two lines fixes it. I guess the spec has been updated so that channelCount and channelCountMode are both already defined in this manner and throw an error when set.
i will try committing that change.

Aha, I fixed my project's dodgy sourcemaps and that is indeed where the error is coming from 😄
Thank you for the quick response!

Brilliant, thank you so much!

Hi. I'm still seeing this issue in the latest release. A patched version with the two lines commented out works fine though.

@joegaffey are you using a version which is greater than<EMAIL_ADDRESS>So far this fix hasn't made its way into a release it's still on the dev branch.

Sorry no, I thought that the fix was in the 13.1.6 release based on a previous comment.
Thanks for the clarification and great work on the library!

This just saved me, was having total destruction in firefox, the 13.1.6 works great.  Thanks for the support and the awesome library @tambien


Add trading_calendars to Quantopian conda channel

Hello, I am working on building a conda package for our library, and it uses trading_calendars. I'm having a hard time building my own version of the conda package for trading_calendars due to some odd dependency issues. Do you have any plans to add this project to Quantopian's conda channel? I'm hoping that you'll be able to avoid some of the issues I was running into and that I'll just be able to pull your package when it's on there.

@ttt733 we've actually had trading-calendars on the conda channel for a while, but we hadn't promoted any builds from our ci label to main (we build all our conda packages as part of the zipline build, so the ci builds get pushed once the zipline test suite passes with a newly-released version of this repo). I've just gone through and promoted releases for 1.2, 1.3.1, 1.4.2, and 1.5.1.

Oh, you know what, I just wasn't seeing it because I was searching with the _ instead of the -. But thank you for updating it!


fix: Handle notification icon retrieval error

Description
Closes: #15278
P.S. This repo is massive (couldn't clone the first few trials 😅)
Checklist

[x] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

You can try the Fork action next time instead which only checks out what you need 🙂


JWT token generate！ ‘key is of invalid type’

GUIDES->Business Developing->Demo project download
I downloaded the demo and configured the parameters, but running getJwtToken () returns an error :' Key is of invalid type'. Why is that?
user-api.yaml:
Name: user-api
Host: <IP_ADDRESS>
Port: 8888
Mysql:
  Datasource: root:@tcp(<IP_ADDRESS>:3306)/test?charset=utf8mb4&parseTime=true&loc=Asia%2FShanghai
CacheRedis:
  - Host: <IP_ADDRESS>:6379
    Pass:
    Type: node
Auth:
  AccessSecret: hello231asdf
  AccessExpire: 3600

Debug:Add fmt.Println(secretKey)
func (l *LoginLogic) getJwtToken(secretKey string, iat, seconds, userId int64) (string, error) {
	claims := make(jwt.MapClaims)
	claims["exp"] = iat + seconds
	claims["iat"] = iat
	claims["userId"] = userId
	token := jwt.New(jwt.SigningMethodES256)
	fmt.Println(secretKey)
	token.Claims = claims
	return token.SignedString([]byte(secretKey))
}

then:
 curl -i -X POST \
  http://<IP_ADDRESS>:8888/user/login \
  -H 'Content-Type: application/json' \
  -d '{
    "username":"666",
    "password":"123456"
}'

The result is consistent with Auth in user-api.yaml
Starting server at <IP_ADDRESS>:8888...
hello231asdf
......

Where did I go wrong?

I don't get what's the problem. I work fine in my code.
Can you give a demo?

change secretKey type to []byte , don't use string

change secretKey type to []byte , don't use string

In fact, secretKey has already been changed.
return token.SignedString([]byte(secretKey))


gh-copilot: 1.0.1 -> 1.0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gh-copilot is: Ask for assistance right in your terminal
meta.homepage for gh-copilot is: https://github.com/github/gh-copilot
meta.changelog for gh-copilot is: https://github.com/github/gh-copilot/releases/tag/v1.0.5
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.5 with grep in /nix/store/3gskvla7154r3qbbl6pwgcs58bqqkk78-gh-copilot-1.0.5
found 1.0.5 in filename of file in /nix/store/3gskvla7154r3qbbl6pwgcs58bqqkk78-gh-copilot-1.0.5

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

gh-copilot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3gskvla7154r3qbbl6pwgcs58bqqkk78-gh-copilot-1.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gh-copilot https://github.com/r-ryantm/nixpkgs/archive/ced9c85be3043b152f67df28d342da782972f6b9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ced9c85be3043b152f67df28d342da782972f6b9#gh-copilot

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3gskvla7154r3qbbl6pwgcs58bqqkk78-gh-copilot-1.0.5
ls -la /nix/store/3gskvla7154r3qbbl6pwgcs58bqqkk78-gh-copilot-1.0.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

gh-copilot

Maintainer pings
cc @PerchunPak for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362565

x86_64-linux

:white_check_mark: 1 package built:

gh-copilot

aarch64-linux

:white_check_mark: 1 package built:

gh-copilot

x86_64-darwin

:white_check_mark: 1 package built:

gh-copilot

aarch64-darwin

:white_check_mark: 1 package built:

gh-copilot

@NixOS/nixpkgs-merge-bot merge


License File Missing

Doesn't the MIT license require you to have the license file?
I would love to credit you in my fork, but you provided no such license file.

Thanks @simplymathematics.
Honestly, I am not so aware with what MIT Licence implies. I just wanted to emphasise that repo is open-source, but maybe it is already obvious by having the repo public.
I removed the MIT Licence statement from repo.

well, now it's not open sourced at all. Just because you publish cover doesn't mean that other people are legally allowed to use it .
An MiT license just says "do whatever, but you cannot delete this file, which informs you and all future users of these rights."
right now, nobody (but you) can legally use your code  (though github does allow me to "fork" you code, which gives me little to no rights off platform).
the solution is to simply add the license file to the repo and problem solved.

Thanks for the explanation @simplymathematics. I will do that then as soon as I can 👍.

I added the License with that PR https://github.com/damienmarlier51/JupyterNotebookImageExporter/pull/21/files. Let me know if that works 👍


Make ResolutionStorer implement the Factory API

I've done some preliminary work on this in https://dart-review.googlesource.com/c/sdk/+/61420, but I'm going to put the rest of the work on hold until #33502 is finished, to avoid creating needless merge conflicts.

I don't have any outstanding changes, and likely won't until tomorrow at the earliest. If you want to go ahead you can.

Fair enough.  I've sent the CL to you for review.

Completed in ee19f2804a54e526ff318d4c07fa4b580b7b7ce2


bug :: 63 add proguard rules to keep serialized annotated members

Fixes #63
Add proguard rules in app level to keep annotated class members.
R8 strips annotated members of DTO - could be fixed in concrete DTO - MarketChartDto with @Keep annotation, but issue could happen with other DTOs as well

Thanks for the PR! You have been able to verify that the crash no longer happens on your side?

Yes, I was able to simulate crash  in release version and after adding rules it no longer happens on my side.

Yes, I was able to simulate crash in release version and after adding rules it no longer happens on my side.

Awesome ! I'll merge then confirm. Thanks a whole lot! This bug blocked its release for a while.


Secure Storage in http (insecure) websites

Hello,
Since the Web Crypto API only works secure contexts (HTTPS), I get this error TypeError: Cannot read properties of undefined (reading 'importKey') in http websites. This could also result in throttling errors like Error: This request exceeds the MAX_WRITE_OPERATIONS_PER_MINUTE quota.. What's the best approach to store keys in such contexts? 1. Just use regular storage, 2. Use regular storage with crypto-js encryption, 3. Drop support for insecure websites? 4. Other.
Thanks

You can also detect whether it's HTTP or HTTPS, and swap your storage instance accordingly - if it's HTTPS, use SecuredStorage, if it's HTTP, use plain Storage.


⚠ Bofrost.de has degraded performance

In 42f3082, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 943 ms

Resolved: Bofrost.de performance has improved in 68253c6 after 7 minutes.


header in a line different than line 1

What would be a way to set a line that is not the first one as header?

No.
But you can specify the header with headers parameter.
https://github.com/Keyang/node-csvtojson#header-row

Ok, thank!


Getting Startify to work with nvim-web-devicons

Hi, vim-devicons has function WebDevIconsGetFileTypeSymbol, which allows startify to use devicons using:
function! StartifyEntryFormat()
    return 'WebDevIconsGetFileTypeSymbol(absolute_path) ." ". entry_path'
endfunction

How can I accomplish the same thing with this plugin?
I know that there is luaeval and tried to do the following:
return luaeval("require('nvim-web-devicons').get_icon_by_filetype( absolute_path ) .. ' entry_path'")

but that doesn't seem to work.
I don't want to use vim-devicons for that since it doesn't support colours by default and carrying two plugins doing the same thing seems ugly.

The ..by_filetype functions depend on the &filetype option of buffers.
Also the concatenation seems wrong on your example.
For absolute paths you can try something like this instead:
luaeval("require('nvim-web-devicons').get_icon(absolute_path) .. ' ' .. entry_path")

There is an answer on reddit:
Web Devicons wrapper in lua:
function _G.webDevIcons(path)
  local filename = vim.fn.fnamemodify(path, ':t')
  local extension = vim.fn.fnamemodify(path, ':e')
  return require'nvim-web-devicons'.get_icon(filename, extension, { default = true })
end

This is the vim function for startify
function! StartifyEntryFormat() abort
  return 'v:lua.webDevIcons(absolute_path) . " " . entry_path'
endfunction


DDP as a transform

What does this PR do?
Adds DDP as a Transform (previously Early Transform), similar to what is done in https://github.com/Lightning-AI/lightning-thunder/pull/359.
Can be used by passing to thunder.ddp a ThunderModule (if passed a torch.nn.Module, previous behaviour is kept). If a ThunderModule is passed, a transform is added to the module.
The transform modifies the traces to insert the necessary syncs.
For testing, I modified two existing tests to test this new way of DDPing a module, and added a new one that tests weight sharing works efficiently (similar to the one in test_fsdp).
Fixes #795
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun? yes!

Marking it ready to review, any feedback is welcome.
For the tests, currently the basic cases + weight sharing case is tested. Should I try to change every test to use the 'new' way? Or is it better to change gradually?

Hi @davidgonmar ! Looks like an awesome start. So between back when we did FSDP and now, our view on transformations has evolved:

we want to move ddp_transform_module to DDPTransform.transform_module,
it might be interesting to move the materialization to the MaterializationTransform instead of having it here (allow tracing models on meta device #862 has a bit more).

I think we should do the first in this PR, for the second, we should probably do this separately, but could do this before or after merging this one. WDYT?

I agree with the first one, working on it. As for the second one, I'm currently not even materializing the params on this transform (which I am now realizing seems wrong?). But yeah, I agree, moving common logic to other transforms is nice, but better to do on other PR. Should I still materialize them in this transform for this PR?

it might be interesting to move the materialization to the MaterializationTransform instead of having it here (allow tracing models on meta device #862 has a bit more).

Since DDP doesn't do parameter sharding, what would be the benefits of meta initialization for ddp?

we want to move ddp_transform_module to DDPTransform.transform_module,

is it safe to assume that transform_module will always be called before transform_traces_pre_prologue? I moved the logic that tracks some things (needed for the trace transform) to transform_module.

Since DDP doesn't do parameter sharding, what would be the benefits of meta initialization for ddp?

Right. But would we know that nothing follows DDP that might benefit?


index.md: Clarify image tagging

Changing "and" to "or" ...
The label com.github.debarshiray.toolbox="true" is deprecated, but the page wording makes it seem as though both the current tag and the deprecated tag are required.
Please double-check this, but it would make sense if either tag were required, but not both the current and deprecated tags are required.

Based on changes to the homepage, this requested change is no longer relevant. Closing the PR.


cloudscale metrics collector with k8s api support

fixes #30
Checklist

[x] PR contains a single logical change (to build a better changelog).
[ ] Update the documentation.
[ ] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[ ] Link this PR to related issues or PRs.

Also you can remove the whoie kubernetes.crossplane.io from https://github.com/vshn/component-appcat/blob/66c044a777dfe638c5711c98a12190e145daf580/packages/tests/golden/composition-objectstorage-cloudscale/appcat/appcat/compositions.yaml#L122 which is probably managed by: https://github.com/vshn/component-appcat/blob/2a589d3ddd248aec8bccc97bd77831181cf58ec4/packages/composition/objectstorage/cloudscale.yml#L17

ah, right. That's the one :) Removed.


The scrolling speed of ScrollView in Flutter Desktop is very slow

How to modify the scrolling speed appropriately?

@xiao-cao-x
Are you using a mouse to scroll or machine's trackpad?

@darshankawar

Are you using a mouse to scroll or machine's trackpad?

Mouse wheel

@xiao-cao-x
There's a similar open issue describing your case, https://github.com/flutter/flutter/issues/67985
Please follow-up there for updates.
Closing this as duplicate. If you disagree, write in comments and I'll reopen it.

Can I force the scroll speed to be modified through code at present?

@darshankawar
Can I force the scroll speed to be modified through code at present?

@xiao-cao-x please check if this helps https://stackoverflow.com/a/63989215/8253662


Always mount sysfs as rw for privileged container.

Similar with https://github.com/moby/moby/pull/36808.
In cri, we don't change mount ro/rw based on ReadonlyRootfs, and strictly respect whatever CRI specifies. https://github.com/Random-Liu/cri-containerd/blob/42a22ae1991dc4ef354d81b6b48a1efb6387c339/pkg/server/container_create.go#L644-L648
Default mounts won't be affected by ReadonlyRootfs, so we should be able to safely set sysfs as rw for Privileged=true regardless of ReadonlyRootfs.
I don't think we need to cherrypick this, because this is not a regression.
/cc @justincormack
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

Yes this is correct (we changed Docker recently to match this too).


Chrome in headless mode

I try create a crawl for whatsapp with puppeteer in headless mode but whatsapp only support Chrome. (Chrome not work in headless mode).

@nicobytes

if you think WhatsApp should support Chromium, I'd reach out to them right away :)
if the question is "how do I run puppeteer with stable chrome", I've just answered similar question here

haha @aslushnikov  ok I get it, thanks a lot. =)

@nicobytes did you manage to make it work?


file_serving example error in creating ctrl socket (112) (IDFGH-725)

Hi,
i get this error with fileserver example. but the example works.
I (4667) event: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (4667) TEST_WIFI: SYSTEM_EVENT_STA_GOT_IP
I (4667) TEST_WIFI: Got IP: '<IP_ADDRESS>'
E (4667) httpd: httpd_server_init: error in creating ctrl socket (112)
used sample v3.3\examples\protocols\http_server\file_serving
windows 10
visual gdb current version
esp32 wroom

@arnold-b Please clarify

i get this error with fileserver example. but the example works.

What do you mean by example works? Is the file server accessible via a browser (eg. try opening  http://<IP_ADDRESS>/)? Also, it will help if you can paste the full log

@arnold-b Hi, would you help provide any updates for the issue? Thanks.

Sorry,
i didnt see this.
I just mean i get the error message in the log, but i still can access the files. later i will post the full log

@arnold-b  I'm going to close the issue for now. Please feel free to open it later.


Files: image bytes are loaded twice

Tested on webui.ipfs.io and http://<IP_ADDRESS>:5001/webui/, Firefox 124.0.2 and Google Chrome 124.0.6367.18.
Opening image in Files triggers fetch twice

once via Kubo RPC and /api/v0/cat
once via subdomain gateway at cid.ipfs.localhost:port

I guess we had code somewhere which should try one, and then fallback to other, but instead does both?
There is also unrelated(?) bug in Chrome where the gateway GET listed here fails (https://github.com/ipfs/ipfs-webui/issues/2218)

https://github.com/ipfs/ipfs-webui/issues/2218 is resolved, but this one is still the case in  v4.3.0


Use angular2-modal on a non-top level component

All examples I've seen is using this library on a top level component (for example app-component). But I want to use it in a nested component. For example, I want to open a login modal inside the navbar component.
How would I do this? It gives an error every time I try to do it in a nested component:
EXCEPTION: Uncaught (in promise): Error: Default view container not set. Add the "defaultOverlayTarget" directive to the application root component template (e.g: . You can also set it manually using the "Overlay" service "defaultViewContainer" property.

Just add this to your html on the component container

@gedarufi There will be some grammatical errors with your instruction. Any updates?

In you root template add this:
Set defaultViewContainer property is on the modal.overlay object in the root component constructor:
constructor(vcRef: ViewContainerRef, public modal: Modal) {
    modal.overlay.defaultViewContainer = vcRef;
}


Accept filters from request arguments and not only metadata

In DSG metadata is what used to replace the query parameters systems. It is very flexible as it's not in the proto file.
The main inconvenients are:

The metadata are not binary serialized so passing a lot of data as filters may result in poor performance
They not exported in the proto so not documented by default.

To contourn that we want to look about best practices and implement it.
Think to update doc after.

@AMontagu,
I fully support this reature request.
Right now, I am using a "decorator" workaround, e.g.:
# filter_decorator.py
import functools
import json

import lara_django_samples_grpc.v1.lara_django_samples_pb2 as lara_django_samp_pb2

def dict_filter(func):
    @functools.wraps(func)
    def wrapper_filter(filter_dict : dict={}, **kwargs):
        metadata = (("filters", (json.dumps(filter_dict))),)
        return func.List(lara_django_samp_pb2.SampleListRequest(), metadata=metadata, **kwargs)
    return wrapper_filter

and then I can do something like:
from filter_decorator import dict_filter

channel = grpc.insecure_channel(f'{grpc_server_host}:{grpc_server_port}')
sample_client = lara_django_samp_pb2_grpc.SampleControllerStub(channel)
list_filter = dict_filter(sample_client)

filter_dict = { "name__contains": "term-to-search"}
res = list_filter(filter_dict=filter_dict)
print(res)

Which is more readable, but still has the problem of using metadata to send the filter.

The first part of this issue is completed.
The second part is to precisely detailled the filters and pagination fields names and types instead of using a struct.
This improvement may happen in late 2024.


Wrap public methods in try/catch

Description
Adds a try/catch around our public API so that errors are not fatal to the containing application, only log an error.
Check List

[x] Are you changing anything with the public API?  Sort of, our public API will no longer throw KlaviyoExceptions
[x] Are your changes backwards compatible with previous SDK Versions? Yes
[x] Have you tested this change on real device? n/a
[x] Have you added unit test coverage for your changes? YES
[x] Have you verified that your changes are compatible with all Android versions the SDK currently supports?

Changelog / Code Overview

Added private inline functions that wrap a block in try/catch
Wrapped all the public methods in Klaviyo in those try/catch methods
Added tests for each public API method when called without initializing

Test Plan

Unit tests, test app.

Related Issues/Tickets
CHNL-4688
CHNL-5324

This is a slight change between iOS and Android where in iOS we crash in debug builds. We'd discussed if crashing is required but does seem like a effective way to get folks to setup correctly. Does the log bubble up and avoid being missed?

This is a slight change between iOS and Android where in iOS we crash in debug builds. We'd discussed if crashing is required but does seem like a effective way to get folks to setup correctly. Does the log bubble up and avoid being missed?

I thought we'd come to the conclusion that crashing on debug and not release was confusing, but I'm open to continuing that discussion. I don't know of any other options for android other than crashing or logging, like the XCode thing you showed me.

Also, QQ - does this also show up on RN if they don't open the IDE but build from terminal. I'm guessing yes but just confirming.

Also, QQ - does this also show up on RN if they don't open the IDE but build from terminal. I'm guessing yes but just confirming.

Very glad we checked this. No it does not, and I don't think it even logs in a regular android application, because we turn logging off in the "release" flavor of the Android SDK. Gonna have to address this in the Android SDK.


Add the option to install packages like the hlint linter

I'd like to add another option which I'll do in a separate PR, but with this change the Haskell feature can install an arbitrary list of packages through Haskell to support various VS Code Extensions (see https://github.com/devcontainers-contrib/features/issues/135).
I'm defaulting to no packages, since it adds so much to the build time, but hlint is necessary for the linting extension I've mentioned in https://github.com/devcontainers-contrib/features/issues/135.

ch seems like it's completely independent of this chan

yeah ill check whatever happened to tdlr in different issue


Attempts to set content_synchronisation on every terraform apply

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version
Terraform v0.13.5
+ provider registry.terraform.io/jfrog/artifactory v2.2.4

Affected Resource(s)

artifactory_remote_repository

Terraform Configuration Files
resource "artifactory_remote_repository" "read-only-mirror_rubygems_org" {
  key             = "read-only-mirror.rubygems.org"
  package_type    = "gems"
  repo_layout_ref = "simple-default"
  url             = "https://rubygems.org/"
  property_sets   = ["artifactory"]
  notes           = "managed by terraform"
}

On every terraform apply, it will attempt to set
    - content_synchronisation {
          - enabled = false -> null
        }

This makes for a lot of noise in the terraform apply (notice how 40 resources are going to be changed)

  # artifactory_remote_repository.read-only-mirror_rubygems_org will be updated in-place
  ~ resource "artifactory_remote_repository" "read-only-mirror_rubygems_org" {
        allow_any_host_auth                   = false
        blacked_out                           = false
        block_mismatching_mime_types          = true
        bypass_head_requests                  = false
        description                           = " (local file cache)"
        enable_cookie_management              = false
        enable_token_authentication           = false
        fetch_jars_eagerly                    = false
        fetch_sources_eagerly                 = false
        force_nuget_authentication            = false
        handle_releases                       = true
        handle_snapshots                      = true
        hard_fail                             = false
        id                                    = "read-only-mirror.rubygems.org"
        includes_pattern                      = "**/*"
        key                                   = "read-only-mirror.rubygems.org"
        max_unique_snapshots                  = 0
        missed_cache_period_seconds           = 1800
        notes                                 = "managed by terraform"
        offline                               = false
        package_type                          = "gems"
        property_sets                         = [
            "artifactory",
        ]
        remote_repo_checksum_policy_type      = "generate-if-absent"
        repo_layout_ref                       = "simple-default"
        retrieval_cache_period_seconds        = 7200
        share_configuration                   = false
        socket_timeout_millis                 = 15000
        store_artifacts_locally               = true
        suppress_pom_consistency_checks       = true
        synchronize_properties                = false
        unused_artifacts_cleanup_period_hours = 0
        url                                   = "https://rubygems.org/"
        xray_index                            = false

      - content_synchronisation {
          - enabled = false -> null
        }
    }

Plan: 0 to add, 40 to change, 0 to destroy.

Duplicate of https://github.com/jfrog/terraform-provider-artifactory/issues/13


[Fedora 39] Outline client frequently interrupts the connection after distribution update.

Describe the bug
After updating Fedora from 37 to 39 the client hops between 2 states:

multiple re-connection attempts for 10-30 seconds followed by;
30-60 seconds of working fine (websites are browse-able etc...).

To reproduce

How can the broken behavior be reproduced?

Install Fedora 37;
Install Outline Client and add working server connection;
Update the distro to newer version.

Expected behavior

What behavior did you expect?

The connection to be established  right away and won't be interrupted.
At least more continuous  operation.
Screenshots

If applicable, add screenshots to help explain your problem.

Sure, I hope it will help. Here is a recording I have made that illustrates the behavior.
Client system:

Outline Client Version [e.g. iOS 1.3.1]
Your Operating System: [e.g. iOS 14.2, Windows 10]

Outline Client Version: Version 1.11.0 (468189)
Operating System:  Linux fedora 6.5.12-300.fc39.x86_64 #1 SMP PREEMPT_DYNAMIC Mon Nov 20 22:44:24 UTC 2023 x86_64 GNU/Linux
Submit feedback

Please submit feedback through the app and label it with this issue number.

List debugging tools for networking? That could help..? Per OS of course.
Additional context

Add any other context about the problem here.

I am using Fedora 39 on two computers and the behavior is the same. I also upgraded the second pc and got the exact same problem (from 37 to 39).
The issue is not with the Outline Server because the android app works just fine with exact same host. There are no such weird shenanigans and the web is browse-able...
Btw, if I ran the app it complained about a missing connection store file. The googled proposals did not helped.

I have tried updating to the latest version - no positive change.

Here is the output from the app:
Outline is starting
loading web app from file:///tmp/.mount_Outling8rDqZ/resources/app.asar/www/index_electron.html?appName=Outline
Could not load active tunnel:  [Error: ENOENT: no such file or directory, open '/home/sheezzarr/.config/Outline/connection_store'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/home/sheezzarr/.config/Outline/connection_store'
}
Checking for update
Update for version 1.12.1 is not available (latest version: 1.12.1, downgrade is disallowed).
connecting to *******...
using tun2socks to check connectivity
[7797:1209/200113.675273:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 1 times!
[EXIT - tun2socks]: Exited with code 0
UDP support: true
starting routing daemon
tun2socks started
connected to *******
reconnecting to *******
reconnected to *******
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
reconnecting to *******
reconnected to *******
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
reconnecting to *******
reconnected to *******
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
[7797:1209/200121.460970:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 2 times!
[7797:1209/200125.022959:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 3 times!
reconnecting to *******
reconnected to *******
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
reconnecting to *******
reconnected to *******
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 0
[EXIT - tun2socks]: Exited with code 0
tun2socks exited with no errors
disconnected from *******

Hi @SheezZarR , sorry about the inconvenience. Due to limited workforce, Outline Client only supports Ubuntu for now. But we have made a simpler Outline CLI that should be compatible to more Linux distros. Would you mind trying that to see whether it works on Fedora? If you are a developer, please also feel free to contribute to it if you meet any problems. Thanks.

@jyyi1. Thanks for reaching out!
I have tested the CLI utility and it works just fine but one caveat.
I have to run the go run github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli@latest -transport "ss://<outline-server-access-key>" under the root otherwise it fails with this error message: [ERROR] 2023/12/14 13:15:51 failed to create tun device: failed to create TUN/TAP device: ioctl: operation not permitted

@SheezZarR , yes the root account is required in order to configure system routing tables. Outline Client uses some tricks to avoid that, but they're causing issues on Linux distros other than Ubuntu.

Hi @jyyi1 @SheezZarR
If I run go run github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli@latest -transport "ss://" under the root. I get error<EMAIL_ADDRESS>invalid version: unknown revision 6cd87709dc1e
What could be a problem?
Maybe the repository doesn't contain a commit with that hash?

@Sergei-Nikiforov Hello
Does specifying the commit hash work? Like so:
go run github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli@7294484 -transport "ss://<outline-server-access-key>"

Maybe some sort of a caching issue, idk... Latest tag has always been working for me.
Additionally, you could try to update your go installation or change to 1.20.2 which i currently use and seem to work.

Hi @SheezZarR , it sounds like the CLI is referencing to an invalid version of Outline SDK. Would you please create a new issue here? https://github.com/Jigsaw-Code/outline-sdk/issues , so we will be able to track and fix it in the correct repository. Thanks.

Hi @SheezZarR
Thanks a lot.
The commit hash works:
"go run github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli@7294484 -transport "ss://<outline-server-access-key>"
However https://www.dnsleaktest.com/ shows the dns leaks.

@Sergei-Nikiforov, glad it has helped. Unfortunately, I can't follow up on the dns issue tho. Would you mind opening an issue there (https://github.com/Jigsaw-Code/outline-sdk/issues)? In regards to the incorrect version that CLI pulls in your case.
@jyyi1 the CLI works fine for me. It references the correct hash/tag. It is @Sergei-Nikiforov that has the issue.

Hi @SheezZarR , @jyyi1
Done!
https://github.com/Jigsaw-Code/outline-sdk/issues/182

Adding my 2 cents here on Fedora support. The DNS issue can be worked around through resovlectl.
First type resolvectl to identify your outline-tun0 interface number. On my system it is Link 3.
Set a DNS for the interface
resolvectl dns 3 <IP_ADDRESS>

Then set the default route flag to false for all your interfaces except outline-tun0 (I only have another Link 2)
resolvectl default-route 2 false
resolvectl default-route 3 true

I am also experiencing the continuous reconnection issue every few seconds using the client appimage. Note that it was not happening on Fedora 36, but it does since I updated to Fedora 40:
[EXIT - tun2socks]: Exited with code 4
connectivity check error: Error: Process terminated by non-zero exit code: 4
reconnecting to 5ffc3dd2-73b2-4197-a7d4-ae434dd1a76e
reconnected to 5ffc3dd2-73b2-4197-a7d4-ae434dd1a76e
using tun2socks to check connectivity
[EXIT - tun2socks]: Exited with code 4
connectivity check error: Error: Process terminated by non-zero exit code: 4
reconnecting to 5ffc3dd2-73b2-4197-a7d4-ae434dd1a76e
reconnected to 5ffc3dd2-73b2-4197-a7d4-ae434dd1a76e

I am available for troubleshooting and patching under some guidance.

I also faced this issue, and after installing CLI via go run, i  have another error "failed to create tun device: failed to create TUN/TAP device: ioctl: operation not permitted".
I find this project, which provides script for Outline cli installation, and it works! All installation process in readme:
https://github.com/Kir-Antipov/outline-cli


Feature Request (UI Multilanguage)

Hi 4minitz Team,
you made an awesome open source project, thanks for that. I would like to contribute to your work by  adding multi-language to the app.
As this is quite an intensive work I would only take this effort if it's possible to merge the multilang files into your main branch afterwards.
Is this OK for you?
Regards
Wolfgang

Hey coldib,
thanks for the positive feedback.
We would be more than happy, if you could take over this big job.
We actually have "translation of UI" on our wishlist for quite some time.
See here: https://github.com/4minitz/4minitz/issues/30
Maybe you have a look, if this matches your plans roughly?
If yes, I would suggest that we do a Slack session this week to synchronize on where to start...
If you see this helpful, please open https://demo.4minitz.com/  >> About >> Impressum
and drop an eMail with your contact data to the EMail address specified there. Than we can arrange things.

Hi derwork,
will reach out to you as soon as I find some time :)

Hi, I would be interrested in translating the GUI in french .
How could I participate in doing this?

I am willing to do a Dutch translation

@ludovic-durand @jseverinod @vmario89 Feel free to support the translation on https://crwd.in/4minitz


Document IExceptionHandler

See https://devblogs.microsoft.com/dotnet/asp-net-core-updates-in-dotnet-8-preview-5/#iexceptionhandler for detail about this feature.
Should be added to https://learn.microsoft.com/en-us/aspnet/core/fundamentals/error-handling?view=aspnetcore-8.0#exception-handler-page
@Tratcher @adityamandaleeka @joperezr
@tdykstra edit to add document details block

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

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: riande

Associated WorkItem - 115465

@tdykstra, just a heads up, I think that is your doc.

See related #29600


Fixes 4054: privatize repos without trailing slash

Summary
This marks all public repos that are missing a trailing slash as not public.  This should cause them to be deleted if there are no repo configs pointing to them.
Testing steps
Create a duplicate:
make repos-import
make db-cli-connect
insert into repositories (uuid, url, public) values (gen_random_uuid(), 'https://cdn.redhat.com/content/dist/rhel8/8.8/x86_64/appstream/os', true);

Now run the up migration.  It should no longer be public:
select count(*) from repositories where url = 'https://cdn.redhat.com/content/dist/rhel8/8.8/x86_64/appstream/os'
 and public = true;

0
select count(*) from repositories where url = 'https://cdn.redhat.com/content/dist/rhel8/8.8/x86_64/appstream/os'
 and public = false;

1
Checklist

[ ] Tested with snapshotting feature disabled and pulp server URL not configured if appropriate

https://issues.redhat.com/browse/HMS-4054

/retest


Add privilege needed for Queue up Schedules to run now

I need this privilege to be added, so https://github.com/ManageIQ/manageiq-ui-classic/pull/5923 can work properly

@miq-bot assign @himdel

We already have a miq_report_schedule_run_now.
Should this not be using the same feature?

We already have a miq_report_schedule_run_now.
Should this not be using the same feature?

miq_report_schedule_run_now is just for reports, this is for schedules for any type.

@miq-bot assign @kbrock


[SMALLFIX] Ignore flaky PermissionCheckTest

https://alluxio.atlassian.net/browse/ALLUXIO-2426 is created.
Also make the mode output more readable when permission check failed.

@apc999 @aaudiber

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12282/
Test PASSed.


Add install rule for make

You can also change the installation directory by running
$ make install --prefix=my/custom/installation/dir
This closes #6

Merged. Thanks for contributing.


CU-8692kkax6: Bump medcat to 1.10

Let's see if this is compatible.
PS:
This will run GHA. But some other things don't really get run.

This is non-functional. Some of the scripts need to change. Logging etc will need to be added


cmd/upgrade: Change build.build_bottle? to build.bottle?

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example
[x] Have you successfully run brew style with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

I received the following error when attempting to use the upgrade
command on a tap:
$ brew tap teddywing/passextract https://github.com/teddywing/Passextract.git
$ brew upgrade --verbose teddywing/passextract/passextract
Updating Homebrew...

==> Upgrading 1 outdated package, with result:
teddywing/passextract/passextract 0.4.0
Error: Calling build.build_bottle? is disabled!
Use build.bottle? instead.
/usr/local/Homebrew/Library/Homebrew/cmd/upgrade.rb:121:in `upgrade_formula'
Or, even better, submit a PR to fix it!

Change the method call to use the (presumably) newer version.

Nice catch, thanks!

Thanks so much for your contribution!  You rock, @teddywing!

Thanks!


Text Size

Thanks for making and sharing Devdoc!
I have difficulty reading for long small text size of articles/reference pages and unfortunately, Devdoc comes with very small text size.
Please make text size adjustable so that it can be customized by user.

Checkout v0.3.0 release: https://github.com/egoist/devdocs-app/releases/tag/v0.3.0
You can add following CSS to your custom CSS file:
._list-item, ._content {
  font-size: 1rem !important;
}

@egoist: Subject: to custom increase content font. I followed your instructions for custom.css and it is not working for me. Any idea's why? I have upgraded to v0.5.1 and added to custom.css:

._list-item, ._content {
font-size: 1rem !important;
}
Next, I tweeted different sizes 1rem, 5rem, 10rem - but none increase the size of font content. The small font size that is used by default in devdocs.io is prohibitive so effectively i end up not using devdocs at all. It would be really nice to have this feature work.


Add set-simulator-location by @lyft to Location section

Project URL

https://github.com/lyft/set-simulator-location
Description

Add set-simulator-location by @lyft to Location section
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

1847
301
https://github.com/uacaps/PageMenu redirects tohttps://github.com/PageMenu/PageMenu

2534
522
http://rypress.com/tutorials/objective-c/index

  Generated by :no_entry_sign: Danger

Thx!


Set up coveralls.io integration

So now we know exactly how terrible we are by this metric :+1:

:+1:


Strange Behavior with MVC WebApi Routes

Hi,
We're trying to support multiple API versions with controllers that differ by namespace only. We've noticed that some routes appear to work fine, while other nearly identical routes return a 404 (see below):
Working controller route:
    [Authorize]
    [ApiVersion("2.0")]
    [Route("{version:apiVersion}/Filing")]
    public class FilingController : ApiController
    {
        private Context db = new Context ();

        [ResponseType(typeof(Filing))]
        public IHttpActionResult Get(int? id = null)
        {
            var result = new List<FormGroup>();

            if (id.HasValue)
            {

                result = new List<FormGroup>() { User.GetFormGroup(db, id.Value) };

            }
            else
            {
                result = User.AppUser(db).Customer.FormGroups.Where(o => o.IsActive).ToList();
            }

            return Ok(result.Select(o => new Filing(o)).ToArray());
        }

Works fine (note the 200 OK):

Non-working controller route:
  [Authorize]
    [ApiVersion("2.0")]
    [Route("{version:apiVersion}/FilingErrors")]
    public class FilingErrorsController : ApiController
    {
        private Context db = new Context();

        [ResponseType(typeof(FilingErrors))]
        public async Task<IHttpActionResult> Get(int id)
        {
            FormGroupRepository repo = new FormGroupRepository(db);
            var result = await repo.GetErrors(User, id);
            return Ok(new FilingErrors(result));
        }

Non-Working Route Result

Our Startup.cs file:
`
[assembly: Microsoft.Owin.OwinStartup(typeof(Removed.Api.Startup))]
namespace Removed.Api
{
using Removed.Api.Models;
using global::Owin;
using Microsoft.Web.Http.Routing;
using Swashbuckle.Application;
using System.IO;
using System.Reflection;
using System.Web.Http;
using System.Web.Http.Description;
using System.Web.Http.Routing;
using System.Diagnostics;
using System.Web.Http.Dispatcher;
using Removed.Api.Utilities;
using System.Collections.Generic;
/// <summary>
/// Represents the startup process for the application.
/// </summary>
public partial class Startup
{
    /// <summary>
    /// Configures the application using the provided builder.
    /// </summary>
    /// <param name="builder">The current application builder.</param>
    public void Configuration(IAppBuilder builder)
    {
        // we only need to change the default constraint resolver for services that want urls with versioning like: ~/v{version}/{controller}
       var constraintResolver = new DefaultInlineConstraintResolver() { ConstraintMap = { ["apiVersion"] = typeof(ApiVersionRouteConstraint) } };

        //IMPORTANT keep this here, order is important to make sure auth works.
        ConfigureAuth(builder);
        var configuration = new HttpConfiguration();
        var httpServer = new HttpServer(configuration);

        configuration.MapHttpAttributeRoutes(constraintResolver);

        // reporting api versions will return the headers "api-supported-versions" and "api-deprecated-versions"
        configuration.AddApiVersioning(o =>
        {
            o.ReportApiVersions = true;
            o.AssumeDefaultVersionWhenUnspecified = true;
            o.DefaultApiVersion = new Microsoft.Web.Http.ApiVersion(2,0);
        });

        // add the versioned IApiExplorer and capture the strongly-typed implementation (e.g. VersionedApiExplorer vs IApiExplorer)
        // note: the specified format code will format the version as "'v'major[.minor][-status]"
        var apiExplorer = configuration.AddVersionedApiExplorer(o => o.GroupNameFormat = "'v'VVV");

        configuration.EnableSwagger(
                        "{apiVersion}/swagger",
                        swagger =>
                        {
                            // build a swagger document and endpoint for each discovered API version
                            swagger.MultipleApiVersions(
                                    (apiDescription, version) => VersionSupportResolver(apiDescription.GetGroupName(), version),
                                    info =>
                                    {
                                        foreach (var group in apiExplorer.ApiDescriptions)
                                        {
                                            var description = "Removed API";

                                            if (group.IsDeprecated)
                                            {
                                                description += " This API version has been deprecated.";
                                            }

                                            Debug.WriteLine(group.Name);

                                            info.Version(group.Name, $"Removed.Api.v{group.ApiVersion}")
                                                    .Contact(c => c.Name("Removed Support").Email("support@removed.com"))
                                                    .Description(description)
                                                    .TermsOfService("https://removed.com/Terms");

                                            foreach(var apiDescription in group.ApiDescriptions)
                                            {
                                                apiDescription.RelativePath = apiDescription.RelativePath.Replace("{version}", group.Name);
                                            }
                                        }

                                    });

                            // add a custom operation filter which sets default values
                            swagger.OperationFilter<SwaggerDefaultValues>();

                            // integrate xml comments
                            swagger.IncludeXmlComments(XmlCommentsFilePath);
                            //swagger.UseFullTypeNameInSchemaIds();
                            swagger.ResolveConflictingActions((apiDescriptions) =>
                            {
                                var result = new ApiDescription();
                                var apiEnum = apiDescriptions.GetEnumerator();
                                while (apiEnum.MoveNext())
                                {
                                    result = apiEnum.Current;
                                }
                                return result;
                            });

                        })
                     .EnableSwaggerUi(swagger => { swagger.EnableDiscoveryUrlSelector(); });

        configuration.Routes.MapHttpRoute(
            "VersionedUrl",
            "v{apiVersion}/{controller}/{id}",
            defaults: new { id = RouteParameter.Optional },
            constraints: new { apiVersion = new ApiVersionRouteConstraint() });

        //configuration.Routes.MapHttpRoute(
        //    "VersionedQueryString",
        //    "api/{controller}/{id}",
        //    defaults: new { id = RouteParameter.Optional }
        //   );

        builder.UseWebApi(httpServer);
    }

    private bool VersionSupportResolver(string apiDescription, string version)
    {
        return apiDescription == version;
    }

    static string XmlCommentsFilePath
    {
        get
        {
            var basePath = System.AppDomain.CurrentDomain.RelativeSearchPath;
            var fileName = "XmlDocument.xml";
            return Path.Combine(basePath, fileName);
        }
    }
}

}
`
I noticed that commenting out
configuration.MapHttpAttributeRoutes(constraintResolver);
will allow the erred controller route to work, but of course our Swagger documentation at v2/swagger breaks:

We downloaded the source for aspnet-api-versioning and including this in the solution to set breakpoints and determine where the root cause of the issue is, and we found the following:
The non-working controller route returns 'true' for controllerDescriptor.IsAttributeRouted() in ActionSelectorCacheItem.cs - Line 85. Our working controller returns 'false' at this point:
  if ( controllerDescriptor.IsAttributeRouted() )
                {
                    selectionCache.StandardCandidateActions = new CandidateAction[0];
                }
                else
                {
                    var standardCandidateActions = new List<CandidateAction>();

If we set a breakpoint in the controllerDescriptor.IsAttributeRouted() function and force it to return a false for the erred controller, the route will work.
Does anyone know what could possibly be wrong here? From what we can tell the 2 controller routes are formatted identically (except for the name) where one works and one doesn't.  Any help is greatly appreciated.

Thanks for all of the detailed information. It's dense, but it does help track down things quickly. ;)
It seems that you have a simple typo in your ApiController routes. The template [Route("{version:apiVersion}/Filing")] should be [Route("v{version:apiVersion}/Filing")]. You've missed the leading v in the attribute templates, but you have it on the convention-based route.  That's why convention routes are working, but attribute routes are not.
I strongly discourage mixing routing styles, unless you have legacy stuff that you're afraid to touch. Having attribute and convention-based routes together are supported, but can be very confusing to understand and troubleshoot. Just a suggestion. Anything you may have seen in an example is purely for illustration (e.g. not a best practice).
I would recommend deleting the configuration.Routes.MapHttpRoute for "VersionedUrl" and "VersionedQueryString". It looks like you may have been just running through some trial and error scenarios.
The following should get you want you want:
[Authorize]
[ApiVersion("2.0")]
[RoutePrefix("v{version:apiVersion}/Filing")]
public class FilingController : ApiController
{
    private Context db = new Context ();

    [Route("{id:int?}")]
    [ResponseType(typeof(Filing))]
    public IHttpActionResult Get(int? id = null)
    {
        var result = new List<FormGroup>();

        if (id.HasValue)
        {
            result = new List<FormGroup>() { User.GetFormGroup(db, id.Value) };
        }
        else
        {
            result = User.AppUser(db).Customer.FormGroups.Where(o => o.IsActive).ToList();
        }

        return Ok(result.Select(o => new Filing(o)).ToArray());
    }
}

[Authorize]
[ApiVersion("2.0")]
[RoutePrefix("v{version:apiVersion}/FilingErrors")]
public class FilingErrorsController : ApiController
{
    private Context db = new Context();

    [Route("{id:int}")]
    [ResponseType(typeof(FilingErrors))]
    public async Task<IHttpActionResult> Get(int id)
    {
        var repo = new FormGroupRepository(db);
        var result = await repo.GetErrors(User, id);
        return Ok(new FilingErrors(result));
    }
}

I hope that helps. Let me know if you have more questions.

Thank you for this!
We had tried this fix earlier and it worked, but our Swagger documentation started to generate routes for this as '/vv2/FilingErrors/{id}'. Is there a simple override for the Swagger URL generation to correct this?

Thank you for this!
I just wanted to expand on this a little more.
'FilingErrorsController' from above was a new controller that only exists in the v2 namespace, but 'FilingController' was in both v1 and v2. When I add a FilingErrorsController to the v1 namespace with the ApiVersion("1.0") this route seems to work as expected. Is there a trick to get this to work when the controller is new for v2 without having to add a controller for prior versions?

It seems that you might have updated or otherwise retracted one of your last comments. The reason the Swagger routes are not being generated correctly is because your are mucking with the URLs. Possible solutions:

Don't change the URLs; the API explorer generates the correct URLs and supplies default values for the corresponding route parameter so that the user doesn't have to enter it
Change the GroupNameFormat to the format VVV without the literal 'v'
Account for the literal v in your string replacement (e.g. Replace("v{version}", group.Name))

The API explorer bucketizes the controllers by API version. Namespaces and other organization techniques are purely implementation details and of no importance to the API explorer.
Assuming the following:
namespace Services.V1
{
    [ApiVersion("1.0")]
    [RoutePrefix("v{version:apiVersion}/Filing")]
    public class FilingController : ApiController { }
}
namespace Services.V2
{
    [ApiVersion("2.0")]
    [RoutePrefix("v{version:apiVersion}/Filing")]
    public class FilingController : ApiController { }

    [ApiVersion("2.0")]
    [RoutePrefix("v{version:apiVersion}/FilingErrors")]
    public class FilingErrorsController : ApiController { }
}

Then I would expect the API explorer to generate API descriptions grouped the following way:

API Version 1.0

Services.V1.FilingController

API Version 2.0

Services.V2.FilingController
Services.V2.FilingErrorsController

If that isn't the case, let me know. There may be a bug in the API explorer. I'm not sure what effect ResolveConflictingActions has on the Swagger document generation, but this configuration should not be necessary.

I also noticed that you've enabled AssumeDefaultVersionWhenUnspecified. This isn't doing what you think it does since you are implementing API version by URL segment and the default API version is 2.0. In order for that to work, you have to have default routes for without the API version segment. It is not possible to have a default or otherwise assume value in the middle of the URL. See the wiki topic URL Path Segment Routing with a Default API Version for more information and possible solutions.

Right, sorry about that!
After I had posted, I notice that tweaking the ApiExplorer formatting seemed to correct the Swagger urls.
Thank you so much for your help, it is sincerely appreciated!

Glad to help. Feel free to close this out if all of your questions have been answered and you're in business. Thanks!

Seems like you're good to go. Feel free to come back or reopen this issue if your issue has not been solved.


oc-id: Source gems from rubygems when we can

There's no reason to grab these from git anymore and it makes the bundle
install / update much slower.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/2925#ff8e330c-4e6e-459d-9ed6-d239acc3f0f5


[JSON RPC] extract out JSON RPC payload handling logic as generic helpers

Motivation

Extract out generic JSON RPC payload handling logic into shared code used by CLI and JSON RPC async client
use get_block_metadata instead of get_state_proof in CLI test_validator_connection

Will refactor for json-rpc unit tests in later PR

@bors-libra: r+

:pushpin: Commit 180ad67 has been approved by phoenix-antigravity

:hourglass: Testing commit 180ad67d1b4830904803e51f86628bfba5ccfff9 with merge a1cd35c23e52fc808a4c796b333d8c0fb6f75a12...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit 180ad67d1b4830904803e51f86628bfba5ccfff9 with merge d9aea4bfe1007953343b77118f36d336c2afced2...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: phoenix-antigravity
Pushing d9aea4bfe1007953343b77118f36d336c2afced2 to master...


[FEATURE] toggle claim chests config to decrease consuming resource

Is your feature request related to a problem? Please describe.
If my players builds many machines with many hoppers, the onHopper listener will consume resources. But i don't allow my players claim chests.
Describe the solution you'd like
Add a config enableClaimChests: false to enable/disable claiming chests. If disabled, don't register ChestManagement and disable related commands.
Describe alternatives you've considered
Additional context

You can disable chest claim commands by setting enabled to false under the relevant commands in teampermissions.yml and chestpermissions.yml
are you noticing any performance impact from the onHopper method?

@booksaw not too heavy. Although only a few hoppers will cause apparently resource consumption. Maybe someone can testing for many hoppers.
It's a report from lag monitor. Sampling time is 25.797 seconds.

Yeah thats higher than is acceptable, ill make some improvements

Instead of adding a config option, the chest checks will be disabled automatically when the chest command is disabled.
This bug has been fixed and the patch will be included in the next update. Thank you for reporting.


Math magicians testing

We did the following in Math magicians testing branch:

Set up React Testing Library.
Write unit tests for the files operate.js and calculate.js using Jest.
Create unit tests for all React components:
Use Jest snapshots to test the components.
Use React Testing Library to simulate user interaction.
Run your full tests suite using Jest. All tests should pass.

@omarsalem7
Request  README.md file Update

@ichala

Approved.


radix sort added

please check my request #22

great work @riya244  pls follow me and star this repo i am merging ur pr


feat!: migrate to constructs v10 (take 2)

Closes https://github.com/cdk8s-team/cdk8s/issues/773
Signed-off-by: Christopher Rybicki<EMAIL_ADDRESS>

Before merging this, we need to rename the main branch to "1.x" and change the default branch to "2.x".


Update in regards to Autopilot objects

Michael Niehaus has written about the consequences about breaking the Azure AD device association with the Windows Autopilot object.
see here on of the last paragraphs: https://oofhours.com/2019/07/07/what-happens-when-you-register-a-device-with-windows-autopilot/
I've taken this and incorporated it into the article.

@okieselbach : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@simorjay Thoughts?

Good info. I think the error code page is very useful. Thanks for the idea, I think it's good to include in the guidance.

#sign-off


Firefox Extension Broken

The Firefox extension is currently unsigned. You must install it using the  Load Temporary Add-on button on about:debugging#/runtime/this-firefox.

firefox no have hcapcha solver


feat(gateway): Add GraphQL Codegen to support type definitions in TypeScript

Add graphql-codegen package and configurations
Add types to user queries and mutations
Update tsconfig.json, .gitignore and package.json
Add mock delay

Add graphql-codegen package and configurations
Add types to user queries and mutations
Update tsconfig.json, .gitignore and package.json
Add mock delay


Bump version to 1.2.4

Also updates RELEASE-CHECKLIST.md to fix #2428
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

Ping @dongluochen @abronan @vieux

LGTM


To mask basic authcode while documenting the records to EATL

This is specific to AccessAdministration, the following change shall be made for /v1/approve-oam-request and /v1/approve-basic-auth-request (for both schema)

While req.body is being sent to executionAndTraceService.recordServiceRequest(), the attribute "Authorization" in request body shall be removed. Since it is sensitive information, it shall not be stored in ExecutionAndTraceLog

Close this issue , raised PR


Prior Parameter Distribution and Transform Visuals + Adstock Weighting and Contribution Visuals (Draft)

Description
Additional functionalities serving to help the users specify their priors. This includes functions to:

Explore the priors across channels
See the effect of the adstock- and saturation transforms across channels
Compare the adstocked contribution with the direct contribution curve

Checklist

[x] Checked that the pre-commit linting/style checks pass
(some functions in delayed_saturated_mmm.py might need to be checked.
[ ] Included tests that prove the fix is effective or that the new feature works
(have done some here, but not an exhaustive list of tests)
[ ] Added necessary documentation (docstrings and/or example notebooks)
(only modified the mmm_example.ipynb file to show functionality. Would either need a prior predictive notebook or create an additional section in the file with more detailed description and appriopriate headers + placement.
[ ] If you are a pro: each commit corresponds to a relevant logical change

Sorry, but the commits are a bit all over the place 🙈
Modules affected

[x] MMM
[ ] CLV

Type of change

[x] New feature / enhancement
[ ] Bug fix
[x] Documentation (A maybe here)
[ ] Maintenance
[ ] Other (please specify):

📚 Documentation preview 📚: https://pymc-marketing--477.org.readthedocs.build/en/477/

Thank you @eirikbaekkelund ! I will check this one in the upcoming weeks :)

Why do we have so many commits from previous PRs/ Maybe you try rebasing?

Hi @eirikbaekkelund
Is this something that you are still interested in?

sure, can get it done within the next 2-3 weeks i think with current availability

still of interest for you guys @wd60622 @nialloulton @juanitorduz ?

still of interest for you guys @wd60622 @nialloulton @juanitorduz ?

Hi @eirikbaekkelund
If you find it useful then I think it will be a great addition!
We did do a large refactor of the media transformations to each be a class themselves and no longer coupled with the MMM instance. i.e. media function agnostic MMM.
Because of these large changes, you will at least have to do a rebase or restart with these changes.
Here is an overview of the new class functionality which might be related to your edits. I am not familiar with your changes and would have to take a deeper look. Maybe you can make sense of what might still be missing.
If you share the produced curves from these edits, it might be easier to see and compare

Hi @eirikbaekkelund, still interested in this issue? If you are short on time, would you mind creating an issue for this feature? If you have a visual of what you are looking for, that would be helpful.

Hey @wd60622,
Apologies for the delayed response. I've been juggling a lot at work for some months, but I'm hoping to revisit this PR over the weekend. I'll need to familiarize myself with the latest updates to the library and clean up the boilerplate code.
I'm curious about your thoughts on creating a Streamlit app (similar to Desmos). If you're interested, let's schedule a quick call once I've made some progress.

I will close this PR as it has become stale. Thanks for the work here. I will create some issues from it if others are interested!


Add option to fix_permalink to dump the yaml

Signed-off-by: Jason Lowe-Power<EMAIL_ADDRESS>

@BobbyRBruce Here's a script to fix the permalinks and to output (some) yaml stuff. It's not quite hierarchical, but I think it's going to have to be up to the user anyway how to add hierarchy.


Add authentication and TLS support to Fluentd

Hi,
It would be great to be able to stream dummy data to secure Fluentd instances.
Do you have this feature in your roadmap?

l have no roadmap. PR is welcome.


Add maintenance task for syncing OS hostname with AMT hostname

Describe the bug 🪲
I need a way to sync the current OS hostname with the AMT hostname so that AMT will respond to network requests to the hostname when the OS is not available.
Expected behavior
When I run a maintenance task with RPC, I can specify that I want to sync the hostname.  RPC and RPS will then get the hostname from the OS and put that information into AMT

linked now


”Breznița-Motru”, ”Cosovăț”, ”Deleni”, ”Făuroaia” are listed in the drop-down menu

Description:
When user writes ”Breznița-Motru”, ”Cosovăț”, ”Deleni”, ”Făuroaia” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Breznița-Motru"

Expected results
The location appears in the drop-down menu as "Sat Breznița-Motru, MEHEDINȚI (Breznița-Motru)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Cosovăț"

Expected results
The location appears in the drop-down menu as "Sat Cosovăț, MEHEDINȚI (Breznița-Motru)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Deleni"

Expected results
The location appears in the drop-down menu as "Sat Deleni, MEHEDINȚI (Breznița-Motru)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Făuroaia"

Expected results
The location appears in the drop-down menu as "Sat Făuroaia, MEHEDINȚI (Breznița-Motru)".


TypeError: Cannot read property 'query' of null - react-router/node_modules/history/lib/useQueries.js:43:22

Hi,
We tried to migrate our app from react-router 0.13 to 1.0 and we got some big errors.
{"error":"TypeError: Cannot read property 'query' of null\n    at .../node_modules/react-router/node_modules/history/lib/useQueries.js:43:22\n    at .../node_modules/react-router/lib/useRoutes.js:252:15\n    at .../node_modules/react-router/lib/useRoutes.js:88:15\n    at .../node_modules/react-router/lib/useRoutes.js:124:15\n    at done (.../node_modules/react-router/lib/AsyncUtils.js:49:19)\n    at .../node_modules/react-router/lib/AsyncUtils.js:55:7\n    at getComponentsForRoute (.../node_modules/react-router/lib/getComponents.js:9:5)\n    at .../node_modules/react-router/lib/getComponents.js:28:5\n    at .../node_modules/react-router/lib/AsyncUtils.js:54:5\n    at Array.forEach (native)"}
We added the History module, but it looks like the react-router use his own dependencies.
It's quite an easy isomorphic router we use;
export default function isomorphicRouter(location, history, store) {
  let routes = createRoutes(store);

  return new Promise((resolve, reject) => {
    match({ routes, location }, (error, redirectLocation, renderProps) => {

      let component;

      if (error) {
        return reject(error);
      }

      if (redirectLocation) {
        return resolve({
          redirectLocation,
          isRedirect: true
        });
      }

      if (history) {
        renderProps.history = history;
      }

      component = (
        <Provider store={store} key="provider">
          {() => <Router {...renderProps} children={routes} />}
        </Provider>
      );

      return resolve({
        component,
        isRedirect: false
      });
    });
  });
}

And we use it in this server.js file;
import express from 'express';
import cookieParser from 'cookie-parser';
import bodyParser from 'body-parser';
import csrf from 'csurf';
import createStore from './redux/create';
import ApiClient from './api-client';
import createLocation from 'history/lib/createLocation';
import React from 'react';
import Html from './html';
import PrettyError from 'pretty-error';
import isomorphicRouter from './isomorphic-router';
import {API_PORT} from './utils/consts';
import httpProxy from 'http-proxy';

let server = express();
let port = process.env.PORT || 4000;
let env = server.get('env');
let csrfProtection = csrf({
  cookie: true
});
let pretty = new PrettyError();

let proxy = httpProxy.createProxyServer({
  target: 'http://localhost:' + API_PORT
});

proxy.on('error', (error) => {
  log('proxy error', error);
});

server.use('/', express.static(__dirname + '/../build'));
server.use(cookieParser());

// Proxy to the API service gateway.
server.use('/api', (req, res) => {
  proxy.web(req, res);
});

server.use(bodyParser.json());

server.use(csrfProtection, (req, res) => {
  let client = new ApiClient(req);
  let store = createStore(client);
  let location = createLocation(req.url);

  if (IS_SSR_DISABLED) {
    res.send('<!DOCTYPE html>\n' + React.renderToString());
  } else {
    isomorphicRouter(location, undefined, store)
      .then(({component, redirectLocation, isRedirect}) => {
        if (isRedirect) {
          res.redirect(redirectLocation.pathname);

          return;
        }

        res.send('<!DOCTYPE html>\n' + React.renderToString());
      })
      .catch((error) => {
        if (error.redirect) {
          res.redirect(error.redirect);

          return;
        }

        log('ROUTER ERROR:', pretty.render(error));
        res.status(500).send({
          error: error.stack
        });
      });
  }
});

server.listen(port, () => {
  if (process.send) {
    process.send('online');
  } else {
    log('listening on http://localhost:' + port + ' in ' + env + ' mode');
  }
});

export default server;

It it the let location = createLocation(req.url); who is not initialized correctly? I'm very lost in migrating the new changes. Thank you for your help!

Does anyone know what cause this? I'm very lost right now

I would dig into node_modules and add logs at all the locations in that stack trace to figure out why location is null in the useQueries module. Hard to tell if it's a problem with your code or a bug.

Happens to me too (node v4.1.1) It seems like the history module which react-router depends on doesn't install its own dependencies. It does fine when it's installed separately.

Addendum: this is only the case for react-router v1.0.0-rc2. rc1 works.

The history module is installed separately in my project and i'm on rc1. I just can't where the actual problem occurs in the code and why. What's the best way to log traces in a module?

Getting this as well when upgrading from v1.0.0-rc1.

I just updated to v1.0.0-rc2 and also getting this error. I also installed the history module separately.

v1.0.0-rc3 solves this problem.

Got this error instead
}```

I'm getting this error with RC3:
function addQuery(location) {
	      if (location.query == null)

I was having the error because of using <Route history={history} ... before match() on the server.
I fixed the error by removing the history attribute from the root Route and by moving history to rendered_props in
match({ routes, history, location }, (error, redirect_location, render_props) =>
{
			// used only on client for <Router history={history} ...>
			if (history)
			{
				render_props.history = history
			}
}

v1.0.0-rc3 doesn't solve the problem :S

This is caused by a disconnect between the internal history lib dependency in react-router and your app history lib dependency version. It would imagine it should work if the versions match.

I don't have history installed as a dependency (only as a sub-dependency) and i'm still getting:
Cannot read property 'apply' of undefined
when a user visits a url with an onEnterHook. I'm also using redux-router but i'm struggling to find out what's up... @aackerman

Hi.
I have also this error with
TypeError: Cannot read property 'query' of undefined
I am using react-router on server side and I have checked all examples here and somewhere else and it seems i am getting this error with query. Is there any solution to this problem?
server.js

app.get('*', function(req, res) {
    // Note that req.url here should be the full URL path from
    // the original request, including the query string.
    var location = createLocation(req.url);

    match({routes, location: location}, function(error, redirectLocation, renderProps) {

        // used only on client for <Router history={history} ...>
        if (history) {
            renderProps.history = history;
        }

        if (error) {
            res.status(500).send(error.message);
        } else if (redirectLocation) {
            res.redirect(302, redirectLocation.pathname + redirectLocation.search);
        } else if (renderProps) {
            var RoutingContextFactory = React.createFactory(RoutingContext, renderProps);
            res.status(200).render('index.ejs', { reactOutput: ReactDOMServer.renderToString(RoutingContextFactory()) });
        } else {
            res.status(404).send('Not found');
        }
    });
});

routes.js

var createLocation = require('history').createLocation;
var useBasename = require('history').useBasename;

var history = useBasename(createLocation)({
    basename: '/'
});

module.exports = [
    <Router history={history}>
        <Route path="/" component={App}>
            <Route path="login" component={Login} />
            <Route path="logout" component={Logout} />
            <Route path="review" component={ReviewHandler} onEnter={requireAuth} />
        </Route>
    </Router>
];

I am having this exact issue as everyone else. Any real fix for this?

same issue here :/

I recently ran into the same issue during migration.
It wasn't caused by any missing modules or dependencies, but because of the deprecation of pushState.
Previously we'd write the programmatic route changes like this:
History.pushState(null, '/path');

Now they have to done as:
History.push('/path');

The lack of a string in the push method causes this error to be thrown.
After some refactoring the error seems to have gone away.

I was having the same issue and figured out that the link I was creating had an invalid to property.
var permalink = null
<Link to={ permalink }>This will throw "Cannot read property 'query' of null"</Link>

IMHO, react-router should consider "" as default if to is falsy.

Same error while trying to run Telescope. I didn't enter my own code. It was just a fresh install of Telescope. It first asked me to install<EMAIL_ADDRESS>a as a dependency and I did meteor npm install --save<EMAIL_ADDRESS>The app worked fine for a couple of times. I created a user and a post. And now I am getting this error. If the issue is related to the version of react-router not matching with whatever package requires it, what version should I install. Below is the error I am getting:
W20160728-11:41:45.801(3)? (STDERR) Thu Jul 28 2016 11:41:45 GMT+0300 (EAT) 'error while server-rendering' 'TypeError: Cannot read property \'query\' of undefined\n    at CategoriesList.render (packages/nova:base-components/lib/categories/CategoriesList.jsx:83:34)\n    at [object Object].ReactCompositeComponentMixin._renderValidatedComponentWithoutOwnerOrContext (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:687:34)\n    at [object Object].ReactCompositeComponentMixin._renderValidatedComponent (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:707:32)\n    at [object Object].wrapper [as _renderValidatedComponent] (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactPerf.js:66:21)\n    at [object Object].ReactCompositeComponentMixin.performInitialMount (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:291:30)\n    at [object Object].ReactCompositeComponentMixin.mountComponent (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:222:21)\n    at [object Object].wrapper [as mountComponent] (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactPerf.js:66:21)\n    at Object.ReactReconciler.mountComponent (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactReconciler.js:39:35)\n    at [object Object].ReactCompositeComponentMixin.performInitialMount (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:297:34)\n    at [object Object].ReactCompositeComponentMixin.mountComponent (/Users/merhawi/Documents/hubs/codehub/meteor/Telescope/node_modules/react/lib/ReactCompositeComponent.js:222:21)'

Same error for me, it was also the Link having an invalid 'To' property.

<Link to={ s.slug }>{ s.title }</Link>

In this case s.slug was undefined. Could improve the error?


Setting "Words per dot" ignores cyrillic text

I make daily notes both on English and Russian, however "Words per dot" counts words only made in English.

This should be fixed in #55


Generate tasks from diffs for exporting

Part of #534.
Reviewers:

@guy-har for everything
@nopcoder for changes to diff format, and for Go, and I hope also everything.

Codecov Report

Merging #803 into master will increase coverage by 0.97%.
The diff coverage is 89.40%.

@@            Coverage Diff             @@
##           master     #803      +/-   ##
==========================================
+ Coverage   41.60%   42.57%   +0.97%
==========================================
  Files         133      134       +1
  Lines       10384    10583     +199
==========================================
+ Hits         4320     4506     +186
- Misses       5484     5491       +7
- Partials      580      586       +6

Impacted Files
Coverage Δ

catalog/diff.go
18.51% <ø> (ø)

metastore/diff.go
0.00% <0.00%> (ø)

parade/ddl.go
72.50% <87.50%> (+1.14%)
:arrow_up:

export/tasks_generator.go
90.97% <90.97%> (ø)

catalog/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

onboard/import.go
62.74% <0.00%> (-2.57%)
:arrow_down:

onboard/catalog_actions.go
67.85% <0.00%> (-2.28%)
:arrow_down:

cloud/aws/s3inventory/orc_reader.go
80.39% <0.00%> (+2.97%)
:arrow_up:

block/s3/inventory_iterator.go
85.50% <0.00%> (+5.50%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Simplify implementation of kernels

I know I pushed for it, but I admit I was wrong.
I think having two codes (the first using for loops, the second using broadcast) is error-prone, and long-term will just increase the burden on maintenance.
This PR removes the kernels using for loops, and keep only the kernels using broadcasting. Hence removing all duplicates code. That fixes the error I made.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (9a4ce89) 73.21% compared to head (304aca9) 75.57%.

Files
Patch %
Lines

src/IPM/kernels.jl
97.22%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   73.21%   75.57%   +2.36%
==========================================
  Files          41       41
  Lines        4756     4529     -227
==========================================
- Hits         3482     3423      -59
+ Misses       1274     1106     -168

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


[E2E] Remove e2e test for skip-check-hosts-file-permissions

This options was recently removed on #2282.
This PR fixes the e2e which was testing that property.

Can one of the admins verify this patch?

/lgtm


Ability to clear output

Recently you made a change for the command line output to remember previous output history.
Personally, I appreciated it when the command line was reset per run. It would be very great if there was a toggle for this kind of option that would be saved. Or even just a button to clear it, or even a keyboard shortcut like Cmd K in Xcode. The current workaround is to refresh the page (and remember to copy the code).

Oh I see. I'll add a clear button soon. Thank you for the feedback! 👍

Fixed https://github.com/kishikawakatsumi/swift-playground/commit/dc552ff4d44ba904f9ffc5ec89b37dfa83b0e066

Thank you so much! This is great!


Can't use Sharepoint online as a private gallery

I am trying to create a private gallery pointing to a sharepoint site. I uploaded vsix file on to the sharepoint site but when I try to add the link to my gallery in Visual Studio & try to access my extension, I get
"remote server returned 403"

No, you need to use a ATOM feed, but that along with the .vsix files can be hosted anywhere - including on SharePoint. See this https://github.com/madskristensen/PrivateGalleryCreator

Closing the issue due to inactivity, please comment below if there are further issues.


No 'Resource type' setting displays in the Summary dialog when SAS attaching one ADLS Gen2 blob container

Storage Explorer Version: 1.26.0-dev
Build Number: 20220914.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Monterey 12.5.1 (Apple M1 Pro)
Architecture ia32\x64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Right click the blob container.
Click 'Get Shared Access Signature...' -> Click 'Create' -> Copy the URL -> Close the dialog.
Open connect dialog -> Click 'ADLS Gen2 container or directory' -> Select 'Shared access signature URL (SAS)' -> Click 'Next'.
Paste the copied URL -> Click 'Next'.
Check whether the 'Resource type' setting displays in the Summary dialog.

Expected Experience
The 'Resource type' setting displays in the Summary dialog.
Actual Experience
No 'Resource type' setting displays in the Summary dialog.

Additional Context
This issue doesn't reproduce for regular blob container.

Verified this issue on the build main/20220921.1. Fixed. So, we are going to close it.


Could not find artifact blowfish:blowfish:jar:1.0 in central (http://repo.maven.apache.org/maven2) -> [Help 1]

FYI it's missing dependenicies and not working for me....
[INFO] Building Burgershop Maven Webapp 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
Downloading: http://repo.maven.apache.org/maven2/blowfish/blowfish/1.0/blowfish-1.0.pom
[WARNING] The POM for blowfish:blowfish:jar:1.0 is missing, no dependency information available
Downloading: http://repo.maven.apache.org/maven2/blowfish/blowfish/1.0/blowfish-1.0.jar
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.427s
[INFO] Finished at: Fri Jan 17 08:30:48 EST 2014
[INFO] Final Memory: 5M/10M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project burgershop: Could not resolve dependencies for project de.zaunberg:burgershop:war:1.0-SNAPSHOT: Could not find artifact blowfish:blowfish:jar:1.0 in central (http://repo.maven.apache.org/maven2) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[root@tc01 burgershop]#

I ran into the same problem.
Could you add your repository to the pom.xml here:
https://github.com/dvayanu/burgershop
Thanks,
--Erik

Hi Eric,
ok will do, but are you aware that there is a better, newer version of the demo here:
https://github.com/anotheria/moskito-demo
subprobject burgershop/burgershop-spring-example ?
It uses/shows new moskito features like tagging, tracers or error catching.
regards
Leon