Add document for Simple Serialization (SSZ)

What's wrong
No specification for SSZ for auditing or verification.
How can it be fixed
Add document on ssz/README.md

How much detail is desired for this? I think I will try making something with a decent bit of detail and then a reviewer can axe details as necessary.

This might be a useful resource: https://github.com/sigp/lighthouse/tree/ssz-readme/ssz
It's a work in progress from one of our team members as we build out a Rust impl.
Feel free to use anything you want from it!

@paulhauner That's awesome! What do you think of concentrating our resources on the "common" part? e.g., Design rationale and how SSZ works. We can add the common document under Ethereum wiki.
@Poseyy Thank you so much! I think Lighthouse has a great outline of the document. The things that we can append in a common document are (i) the design rationale - maybe recap what Vitalik said in the eth2.0 call last week. (ii) More example of the serialized result.
On the other hand, we want to have an instruction on how to use Python SSZ APIs especially.

Thanks @hwwhww, we'd be more than happy to collaborate.
I'm not exactly sure what you mean by "concentrating our resources", though. Would you mind elaborating please?

Ok, got it, thanks.
This issue relates to Python specifics and we can talk about the "common" document in another channel :)

@hwwhww Sounds great, will do!


Support for proxying PATCH requests

While Flexirest does support PATCH requests, they are not supported if responses need to be transformed via a proxy. This change set introduces a new ProxyBase#patch method as well as a new test.

Coverage increased (+0.07%) to 99.052% when pulling 0ae72bce81ac119f6f926cffb05b8e9922bd8d32 on aschaper:proxy-patch-support into f23df44745d216fcd0cc03c9bf895918c114396d on andyjeffries:master.

Thanks for this Andrew. Merged and pushed as v1.3.9.


Uncaught TypeError: Cannot convert undefined or null to object

[Enter steps to reproduce:]

created a new branch thru the web client while atom was open
restarting atom seems to work and shows the new branch

Atom: 1.40.1 x64
Electron: 3.1.10
OS: Microsoft Windows 7 Ultimate
Thrown From: screeps-ide package 0.1.2
Stack Trace
Uncaught TypeError: Cannot convert undefined or null to object
At C:\Users\Bartos\.atom\packages\screeps-ide\node_modules\rxjs\internal\util\hostReportError.js:4

TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at store_1.default.reducer (/packages/screeps-ide/dist/src/components/screeps-panel/reducers/update.js:9:29)
    at /packages/screeps-ide/dist/src/store/index.js:15:17)
    at MapSubscriber._next (/packages/screeps-ide/node_modules/rxjs/internal/operators/map.js:49:35)
    at MapSubscriber.Subscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:66:18)
    at Subject.next (/packages/screeps-ide/node_modules/rxjs/internal/Subject.js:60:25)
    at Object.dispatch (/packages/screeps-ide/dist/src/store/index.js:9:11)
    at /packages/screeps-ide/dist/src/components/screeps-panel/index.js:58:37)
    at TapSubscriber._next (/packages/screeps-ide/node_modules/rxjs/internal/operators/tap.js:59:27)
    at TapSubscriber.Subscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:66:18)
    at /packages/screeps-ide/dist/src/socket/index.js:67:26)
    at SafeSubscriber.__tryOrUnsub (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:205:16)
    at SafeSubscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:143:22)
    at Subscriber._next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:89:26)
    at Subscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:66:18)
    at FilterSubscriber._next (/packages/screeps-ide/node_modules/rxjs/internal/operators/filter.js:52:30)
    at FilterSubscriber.Subscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:66:18)
    at TakeUntilSubscriber.Subscriber._next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:89:26)
    at TakeUntilSubscriber.Subscriber.next (/packages/screeps-ide/node_modules/rxjs/internal/Subscriber.js:66:18)
    at Subject.next (/packages/screeps-ide/node_modules/rxjs/internal/Subject.js:60:25)
    at SockJS.Socket._socket.onmessage (/packages/screeps-ide/dist/src/socket/index.js:41:31)
    at SockJS.EventTarget.dispatchEvent (/packages/screeps-ide/node_modules/sockjs-client/lib/event/eventtarget.js:51:20)
    at /packages/screeps-ide/node_modules/sockjs-client/lib/main.js:279:16
    at Array.forEach (<anonymous>)
    at SockJS._transportMessage (/packages/screeps-ide/node_modules/sockjs-client/lib/main.js:277:17)
    at WebSocketTransport.emit (events.js:182:13)
    at Client.WebSocketTransport.ws.onmessage (/packages/screeps-ide/node_modules/sockjs-client/lib/transport/websocket.js:35:10)
    at Client.dispatchEvent (/packages/screeps-ide/node_modules/faye-websocket/lib/faye/websocket/api/event_target.js:24:30)
    at Client._receiveMessage (/packages/screeps-ide/node_modules/faye-websocket/lib/faye/websocket/api.js:152:10)
    at /packages/screeps-ide/node_modules/faye-websocket/lib/faye/websocket/api.js:36:49)

Commands

Non-Core Packages
atom-ternjs-screeps 1.21.0
screeps-ide 0.1.2

I was able to duplicate this error.

Hello! Is there are a way to solve this issue?


Tengo fails to install

macOS 10.14.6
Go 1.14.3
GO111MODULE=auto

go get github.com/d5/tengo/v2
gives
package github.com/d5/tengo/v2: cannot find package "github.com/d5/tengo/v2" in any of:
	/usr/local/go/src/github.com/d5/tengo/v2 (from $GOROOT)
	/Users/carlca/Code/go/src/github.com/d5/tengo/v2 (from $GOPATH)

No worries. I was able to run go get github.com/d5/tengo.


Create validation strategy for Worldcoin

Hey @bonustrack, could you provide the specs for the validation? :pray:


Scroll gesture shifts months

Hello, first of all thanks for your vue component.
When I try to scroll down the page in my mobile I can see the months changes. Why this happens? How to disable this feature?

I am not sure, since I was not the one developing this "feature" (or is it a bug?)
Is it possible that you may change horizontally unintentionally?...
Can you search the code responsible for this behavior?

@shinesoftware hello... coming back from ReactJS (My day work is on that technology now)
I'll try to fix it.


Support dicom image retrieval from API

Fix #193
Initial work for obtaining associated images for DICOM files. the /image endpoint for the API can now retrieve thumbnail, label, and overview (macro) images from DICOM files.
Follow-up work will be done to support redaction of overview (macro) and label images.

Can you rebase or merge this with master now that the Vue refactor has been merged?


Make Mesos healtchecks compatible with Marathon

https://github.com/mesosphere/marathon/issues/916

Could you please move this to reviewboard so we got this growing discussion in the expected place? @drexin @haosdent @brat002 @ConnorDoyle

May you mind close this pull request and let me
Yes, of course.

2016-07-07 9:04 GMT+03:00 Benjamin Bannier<EMAIL_ADDRESS>
Could you please move this to reviewboard so we got this growing
discussion in the expected place? @drexin https://github.com/drexin
@haosdent https://github.com/haosdent @brat002
https://github.com/brat002 @ConnorDoyle https://github.com/ConnorDoyle
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/mesos/pull/118#issuecomment-230986849, or mute
the thread
https://github.com/notifications/unsubscribe/ADyecDwP1haN5pVccLf-ksJ-mPXssuB7ks5qTJb7gaJpZM4JCPCQ
.

Thanks @bbannier I file the ticket at https://issues.apache.org/jira/browse/MESOS-5803 , let's continue the discussion there.

@brat002 I think this should be closed as it turns out it was a bug in Marathon.

Closing this PR as the Marathon bug has been fixed and we don't want to introduce this behaviour to Mesos.


Expose LocationAware type to extensions

So that they can benefit from automatic plugging in of location of objects with test yaml files.
Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage remained the same at 99.086% when pulling 5eb4aac5165fdfb714224d323801b1a1dd62af3c on location into 2f06407726e5576aa7d1c7e4f1263b254afb3eab on main.


testScalarProduct check for non numeric values

test for non numeric values.
this is a unit test for this commit https://github.com/php-ai/php-ml/pull/14

Thanks for first PR :D :+1:
Add slash to stdClass() and i will merge.
new \stdClass()

Sorry I thought I did in the first place.
https://github.com/php-ai/php-ml/pull/13/commits/8c8740af5aabfe3e97b7c72dd64ac4631a989e4b
I would also like to volunteer as a contributor to this project. It is very important for the reputation of PHP to keep this up.
I will begin like so, making incremental improvements on unit tests, then I would move on to performance improvements so I can familiarize myself with the code. Once I feel comfortable enough then move on to bigger things like documentation, feature requests and bug fixes

Next time please do one PR with tests and patch. I prepare more info about contributing and add it to readme.


Update dummy app javascripts

The dummy app calls a couple javascript files that aren't there.  Either we need to manually add rails-jquery and rails-ujs javascripts, or update the dummy app to use the asset pipeline.

What effect is this having on the engine at the moment?

None directly but my subscription branch has some POST and DELETE links that won't work without rails-ujs

Rightio, that's fixed now.


connection error

Hi
I am hitting to this error, thanks
> Traceback (most recent call last):
  File "finetune_t5_trainer.py", line 379, in <module>
    main()
  File "finetune_t5_trainer.py", line 208, in main
    if training_args.do_eval or training_args.evaluation_strategy != EvaluationStrategy.NO
  File "finetune_t5_trainer.py", line 207, in <dictcomp>
    for task in data_args.eval_tasks}
  File "/workdir/seq2seq/data/tasks.py", line 70, in get_dataset
    dataset = self.load_dataset(split=split)
  File "/workdir/seq2seq/data/tasks.py", line 66, in load_dataset
    return datasets.load_dataset(self.task.name, split=split, script_version="master")
  File "/usr/local/lib/python3.6/dist-packages/datasets/load.py", line 589, in load_dataset
    path, script_version=script_version, download_config=download_config, download_mode=download_mode, dataset=True
  File "/usr/local/lib/python3.6/dist-packages/datasets/load.py", line 267, in prepare_module
    local_path = cached_path(file_path, download_config=download_config)
  File "/usr/local/lib/python3.6/dist-packages/datasets/utils/file_utils.py", line 308, in cached_path
    use_etag=download_config.use_etag,
  File "/usr/local/lib/python3.6/dist-packages/datasets/utils/file_utils.py", line 487, in get_from_cache
    raise ConnectionError("Couldn't reach {}".format(url))
ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/master/datasets/boolq/boolq.py
el/0 I1217 01:11:33.898849 354161 main shadow.py:210 Current job status: FINISHED

This happen quite often when they are too many concurrent requests to github.
i can understand it’s a bit cumbersome to handle on the user side. Maybe we should try a few times in the lib (eg with timeout) before failing, what do you think @lhoestq ?

Yes currently there's no retry afaik. We should add retries

Retries were added in #1603 :)
It will be available in the next release

Hi @lhoestq thank you for the modification, to my experience, also setting timeout to a larger number like 3*60 which I normally use helps a lot on this.


fix(core, monitor, feeder): data races conditions

Purpose of PR?:
The changes proposed in this PR fixes #842 , Found 14 data races during my run and the changes handles all those conditions. No more data races found in subsequent trials/test with the proposed changes.
Fixes #842
Does this PR introduce a breaking change?
No
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[x] Bug fix. Fixes #842
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Commit has unit tests

Codecov Report

Merging #884 (4817e4a) into main (e65a5d5) will decrease coverage by 25.60%.
The diff coverage is 64.70%.

@@             Coverage Diff             @@
##             main     #884       +/-   ##
===========================================
- Coverage   38.81%   13.20%   -25.61%
===========================================
  Files          31       13       -18
  Lines        9703     5293     -4410
===========================================
- Hits         3766      699     -3067
+ Misses       5430     4455      -975
+ Partials      507      139      -368

Impacted Files
Coverage Δ

KubeArmor/feeder/policyMatcher.go
0.00% <0.00%> (-40.92%)
:arrow_down:

KubeArmor/monitor/processTree.go
55.08% <0.00%> (-40.62%)
:arrow_down:

KubeArmor/feeder/feeder.go
14.67% <100.00%> (-37.82%)
:arrow_down:

KubeArmor/monitor/systemMonitor.go
45.90% <100.00%> (-4.23%)
:arrow_down:

KubeArmor/enforcer/appArmorTemplate.go
0.00% <0.00%> (-100.00%)
:arrow_down:

KubeArmor/monitor/logUpdate.go
9.19% <0.00%> (-60.45%)
:arrow_down:

KubeArmor/enforcer/appArmorProfile.go
0.00% <0.00%> (-57.95%)
:arrow_down:

... and 22 more

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

multicontainer ginkgo tests are passing locally on my system
Ran 4 of 4 Specs in 16.802 seconds
SUCCESS! -- 4 Passed | 0 Failed | 1 Flaked | 0 Pending | 0 Skipped
PASS

@rksharma95 can you please resolve the conflicts? : )


⚠ Bofrost.be has degraded performance

In 5579172, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1655 ms

Resolved: Bofrost.be performance has improved in 1cea845 after 1 hour, 8 minutes.


ToDo: Add Large File Download Functionality

Currently Cloud-ShareSync has limited b2 download functionality. I need to add support for "Large File" downloads to be able to download files over 5gb.
b2 download doc
This is needed to get the simplerestore functionality implemented.

Functionality added in latest develop branch commit.


memory leak

depends on https://github.com/v4vendeta/horizon/issues/31, https://github.com/v4vendeta/horizon/commit/5804fe61cdc6e33d26c9e28b2274d7e0d9287d1a pushed a fix to create descriptorpool when intialize and free descriptorset before each descriptorset allocation.


enhanced toClipboard

Instead of calling toClipboard and displaying toast twice, enhanced into one with showData option.

okay


Update Custom Commands

Description of Your Pull Request
Consolidates custom commands to come under 1 command in the Admin section over 6 commands that
Changes Proposed and or Files Changed
deleted:    commands/Custom Cmds Management/createcmd.js
deleted:    commands/Custom Cmds Management/deletecmd.js
deleted:    commands/Custom Cmds Management/listcmds.js
deleted:    commands/Custom Cmds Management/togglecustomcmds.js
deleted:    commands/Custom Cmds Management/updatecmd.js
created:    commands/Admin/customcmd.js
Label Your Pull Request

[x] This PR modifies existing code to enhance the bot
[ ] This PR fixes a bug within the bot
[x] This PR removes/renames methods or properties
[ ] This PR adds methods, features, commands or properties

Closed due to age of requested changes.


500 when trying to embed vocabulary entries

Describe the bug
If you retrieve a vocabulary, and ask to embed the entries link the server returns a 500
To Reproduce
Steps to reproduce the behavior:

Authenticate as an admin
Retrieve a vocabulary and verify that it works, e.g. common_types
Retrieve its entries link and verify that it works, e.g. common_types/entries
Try to embed the entries in the original request, and verify that it fails, e.g. common_types?embed=entries

Expected behavior
The request should succeed and any entries should be embedded

@artlowel : I agree this is a bug that needs resolving.  However, for scheduling/prioritization purposes, do you have an immediate use case for this?  Just trying to determine if this is a must fix in beta 5 / 7.0, or if this is something that could be slotted for 7.1.  (I'm also a little worried the implementation here might be more complex than it seems, since these vocabularies are driven by XML configuration they may need a custom embed solution....but I haven't looked at the code deeply here.)

No it's not urgent, it just means that some extra requests will need to happen. I've disabled the embeds in https://github.com/DSpace/dspace-angular/pull/975 and added TODO's to turn them back on with a link back to this ticket.

Bug still exists in 7.5.  While it's low priority, moving to 7.6 board for tracking.


Add docs on running kore-rpc-booster on a bug report

This PR adds a tutorial on how to run kore-rpc-booster manually to dissect a bug_report.tar produced by Kontrol.
~Blocked on this PR to pyk, which implements pretty-printing of RPC request and responses.~
~Blocked on this PR to pyk, which implements converting responses to new requests~
Needs pyk version v0.1.482. Blocked on dependency update in evm-semantics.

kontrol currently points to kevm 1.0.337, which points to pyk 1.0.486.


Tooltip on marked element?

Hi, can be implemented tooltip on marked keyword? I see only is doing embed but without option to custom that.

Hi @dmitrirussu,
You can of course make the <mark> element a tooltip. Just create it within the each callback (see documentation). For example:
$(".context").mark("lorem", {
  each: function(element){
    // do whatever you like with $(element), e.g. create a tooltip
  }
});

I'm giving you an example with Bootstrap tooltips:
Example
Did this answer your question?

thank you very much

@julmot Would you mind to share a mui or how I can do it without jQuery? Thanks :heart:


Problem with deep object and a shallow value

Sorry to be the bearer of the bad news, but I may have stumbled upon another bug. This is the payload:
  return typedjson(
    { data: [{ greeting: "hello", today: new Date() }], counter: 1 },
  );

And the StackBlitz link: https://stackblitz.com/edit/github-gbcwza-6tcdt7?file=app%2Froutes%2Ftest.tsx
counter is null and its meta is "undefined". Doesn't change when it's a string either.

Thanks, I've fixed the issue, but broke something else. I should have this working soon.

Fixed in v0.0.8


Add telegram to social-element defaults

Is your feature request related to a problem? Please describe.
I'm currently using a custom social-elements tag to get telegram (https://telegram.org) included in our social elements list. The platform is big enough now that I think inclusion into the defaults is warranted.
Describe the solution you'd like
I'd like telegram added as a default and usable just like facebook, twitter etc. social elements.
<mj-social-element name="telegram" href="https://t.me/example"</mj-social-element>
It should likely be using this icon as the default https://fontawesome.com/icons/telegram-plane?style=brands
Describe alternatives you've considered
The custom social element as described here is workable. I would still argue that telegram should be added to the defaults.

We don't want to extend this feature anymore (eg named social network) as you should use custom social element instead.
It's more flexible, portable, and not tied to MJML version.


Listen to textDidChange

Thanks for making this library! It's really cool
How can I listen to changes happening to the text document? Or listen to any key pressed? Something like .onTextDidChange(perform: {}) or something for any key pressed
Edited: I want to do it in the ViewModel, like
  override func viewWillSetupKeyboard() {
        super.viewWillSetupKeyboard()
        setup { DemoKeyboardView(vm: .init(controller: $0) } //how to do it inside of the vm?
    }

Hi @pixelbitx
Thank you 🙏
To detect changes to the document, the controller itself has willChange() and didChange() events, which KK also uses. KeyboardKit however also has observable context types that you can use as well, such as the controller's keyboardTextContext which continously syncs the document text. You can for instance use onReceive(...) with its text properties to perform actions when the text changes.

Thanks a lot for the quick response! Using keyboardTextContext helps with my use case!

Great, happy to hear it!

Hey  @danielsaidi ,
can I ask you a related question, please: when I want to observe selectedText in my SwiftUI toolbar view... what is the most reliable way to do so? This won't register all changes. (e.g. in a textfield: selecting a word, changing the selection etc)
struct MyToolbar: View {
 @EnvironmentObject private var context: KeyboardContext
 @State var selectedText: String?
...
 var body: some View {
        content()
            .onChange(of: context.textDocumentProxy.selectedText) { selectedText in
                self.selectedText = selectedText
            }
    }
}

Hi @ondrejkorol
I would suggest overriding the controller's selectionDidChange and update some observable state:
open override func selectionDidChange(_ textInput: UITextInput?) {
    super.selectionDidChange(textInput)
    // Here
}

Makes sense! Thank you. I got the best results by observing textDidChangeAsync method and passing it via ObservableObject to views.
But I noticed: when you select a word, and delete it, none of these methods will be called, meaning the app still thinks there is an active selection.

Yeah...selectionWill/DidChange doesn't seem to be called at all, and textWill/DidChange is only called when the position or selection changes.
In that case, I'd suggest using textDidChange and a custom action handler to handle this. Just update whenever the action handler handles an action, and you should be covered.


XSS vulnerability in https://select2.github.io/examples.html

Under "Loading remote data", the data that is retrieved remotely is not sanitized before displaying it, and this should be done to prevent XSS issues. For example, when I enter je1 in the input field, one of the retrieved GitHub repo's data contains malicious JS code that is executed and that simulates a fake PayPal page which asks for your credit card info (seems like phishing).

We're having this issue with 3.4.8 but it seems to be present in newer versions too. I even get this on the example page...

I should note that when this was reported, I did get in contact with GitHub about that repository. The repository has since been disabled: https://github.com/accountupdates/paypal
The problem here which we still need to fix is that the example formatter does not properly use jQuery to sanitize the data, and it's something we're planning on fixing in the next release.

@ Kevin
Viewing the release log at https://github.com/select2/select2 it appears you have not yet attended to this.  The 4.1 milestone shows only 14% complete and this matter still open.
As it appears the 4.1 milestone is at least a year away, is it perhaps warranted to have a commit with this vulnerability plus whatever else you have closed?

@TheThemeBuilders I'm in the process of looking for new maintainers for Select2, and that's been a slow process. If you're interested in joining the (currently small) team of maintainers, get in contact with me and we can chat.
Basically, until that happens I can't guarantee when the next release will be.

Hi all - running 4.0.3 here, and still seeing this issue. Can anyone tell me if there's a fix available in future releases?
And thanks again for a great library!

If this project is seeking active maintainers, can you disable this bot?
This unresolved issue is referenced in a CVE for 2016....
https://nvd.nist.gov/vuln/detail/CVE-2016-10744
On Fri, May 17, 2019, 8:26 PM stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you
for your contributions.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/select2/select2/issues/4587?email_source=notifications&email_token=AAAEBYXV3FTS2FSUR4XDQCLPV5EK5A5CNFSM4CPXP3XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVWD4FA#issuecomment-493633044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAEBYT5RZVAUHVAPUHGN43PV5EK5ANCNFSM4CPXP3XA
.

Closing this off as a duplicate of https://github.com/select2/select2/issues/5448.
I thought I did that earlier...

I'm using select2 version 3.5.4 in my project and a security report mentioned this very issue as possible risk for an XSS attack.
This issue is marked as a duplicate of https://github.com/select2/select2/issues/5448. The commits mentioned there fixing the issue solely refer to the documentation.
Early in this issue someone stated they are having this issue with 3.4.8.
Simply put: is there a risk in the library as such that needs a correction in file select2.js?

Simply put: is there a risk in the library as such that needs a correction in file select2.js?

No.

There was an XSS vulnerability that existed because of a misconfiguation within the Select2 documentation website. Thanks to the fact that people blindly copy/paste from the source code on the documentation website (as well as the examples, some of which had an issue as well), this was referenced in a CVE where someone has misconfigured Select2 to enable XSS within their application.
I am willing to say this though: if you are turning escapeMarkup into a no-op, then you shouldn't be surprised when you are allowing XSS through. The escapeMarkup option exists to automatically escape the text and content within options to mitigate any XSS issues.

I am willing to say this though: if you are turning escapeMarkup into a no-op, then you shouldn't be surprised when you are allowing XSS through

If you're using custom formatters with rich menus, there isn't really another way to handle this though. It loads up originally escaped properly using the escaping code we added, and then if you select something else and re-select, it was overriding our mitigation. So I'd still say this is a bug or at least a misfeature.
We've resolved this on our end, but it's still broken in the library according to my definition.

If you're using custom formatters with rich menus, there isn't really another way to handle this though.

I fully disagree. Since 4.0.0, we have supported returning actual DOM or jQuery elements which contain the escaped markup, and that's what we switched the documentation to recommend. I acknowledge that having the documentation recommend an insecure way was a poor choice and one which was easily corrected, but that doesn't mean that there was no way to do it securely.

We've resolved this on our end,

I would highly recommend switching to returning DOM elements from your templates, which will allow you to safely inject text (and only text) into the DOM elements, instead of returning HTML as text directly from your templates.
If you're looking for an example, you can reference the new docs: https://select2.org/selections#templating

For anyone coming to this maintain old/legacy code, the documentation in https://select2.github.io/select2/ might be a bit off in that it has the declaration of formatResult and formatSelection as formatResult(object, container, query) and formatSelection(object, container) but the arguments list contains a fourth/third argument of escapeMarkup. That function is a library supported way to escape markup in the result.
Using that function seems like a more comprehensive approach than the commit linked to above. For example:
formatResult(object, container, query, escapeMarkup) {
  return escapeMarkup(object.foo);
}


Live view - Camera not found.

I have installed roflcoopter / amd64-cuda-viseron: dev v2.0.0b2. After launching, I cannot access the live view. In the container log I get:
[2022-11-01 12:35:40] [INFO    ] [viseron.core] - -------------------------------------------
[2022-11-01 12:35:40] [INFO    ] [viseron.core] - Initializing Viseron
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setting up component data_stream
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setup of component data_stream took 0.0 seconds
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setting up component webserver
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setup of component webserver took 0.0 seconds
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setting up component mqtt
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setting up component ffmpeg
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setup of component ffmpeg took 0.0 seconds
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setup of component mqtt took 0.0 seconds
[2022-11-01 12:35:40] [INFO    ] [viseron.components] - Setting up domain camera for component ffmpeg with identifier camera_one
[2022-11-01 12:35:42] [INFO    ] [viseron.components] - Setup of domain camera for component ffmpeg with identifier camera_one took 1.6 seconds
[2022-11-01 12:35:42] [INFO    ] [viseron.core] - Viseron initialized in 2.3 seconds
[2022-11-01 12:36:23] [WARNING ] [tornado.access] - 404 GET /api/v1/camera/camera_one/snapshot?rand=2qm54&width=568 (<IP_ADDRESS>) 8.69ms
[2022-11-01 12:36:33] [WARNING ] [tornado.access] - 404 GET /api/v1/camera/camera_one/snapshot?rand=bkub&width=568 (<IP_ADDRESS>) 1.92ms
[2022-11-01 12:36:43] [WARNING ] [tornado.access] - 404 GET /api/v1/camera/camera_one/snapshot?rand=irwdp&width=568 (<IP_ADDRESS>) 1.91ms
[2022-11-01 12:37:05] [WARNING ] [tornado.access] - 404 GET /camera_one/mjpeg-stream (<IP_ADDRESS>) 60089.59ms

Looks like you are missing the nvr component, it is required to get all functionality to work.
It's not very intuitive, sorry about that. Will try to make it more clear, and possibly add it automatically

Thank you. Now it is working. Component nvr and motion detector are required

Great!
Motion detector is not required, but you need to set `scan_on_motion_only: false‘ for the object detector that you are using in order for it to work.
Also not very intuitive, will try to make it more obvious.

Ok, great project. How can I get a camera snapshot from Viseron? (.jpg, .png). As a result of some event, I would like to take a camera shot in HA to Telegram.

You can fetch a snapshot from the camera using the API path: /api/v1/camera/<camera identifier>/snapshot

Thank you. It works. Is there an API description in the documentation? I have not found it here.

Perfect, no there is no documentation of the API yet, it is still quite new.
There really isnt much to fetch using the API atm tho, will make sure to create some docs for it when i expand upon it.


Option to strip ansi color output when using --push-results

Thanks for spin, work great. But I see ugly ansi escape sequences in my RubyTest console in Sublime. I recognized that it`s not a problem with RubyTest but with the raw output from spin. So I tried to fix it in spin- with success. A simple regex just before sending the results back will clear the escape sequences. Available through a new CLI option "--strip-ansi-color".
BUT: See the problem description at the end, option is not passed yet.
spin serve --push-results --strip-ansi-color

Here the working fix
#spin.rb line 74
while line = socket.readpartial(100)
  break if line[-1,1] == "\0"
  if strip_ansi_color
    #clear ansi escape sequences
    line = line.gsub(/\e\[[^m]*m/, '')
  end
  print line
end

Only one problem: I was not able to figure out how to pass options to from spin server to spin push when forked. All options are lost after forking and only the default parameter preload:true is left in method #push.
Maybe somebody can look into this ? The fix itself is working. Screens before and after attached.

use this package: https://github.com/aziz/SublimeANSI


[Snyk] Security upgrade org.fugerit.java:fj-doc-mod-fop from 8.8.9 to 8.9.1

Snyk has created this PR to fix 1 vulnerabilities in the maven dependencies of this project.
Snyk changed the following file(s):

pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

XML External Entity (XXE) Injection SNYK-JAVA-ORGAPACHEXMLGRAPHICS-8179276
  631
org.fugerit.java:fj-doc-mod-fop:  8.8.9 -> 8.9.1  No Known Exploit

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 XML External Entity (XXE) Injection

Alread upgraded to 8.10.2


Spelling mistake in README.md File.

Describe the bug
Spelling mistake in README.md file
Under Checkout dependencies:
"For developers who will be working on Blocky within the context of App Inventor"

Affects
Creates confusion among new contributors & ruins the aesthetics of Read-Me file.

[ ] Designer
[ ] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[x] Other... (please describe)

Expected behavior
"For developers who will be working on Blockly within the context of App Inventor"

Steps to reproduce
Correct the spelling.
Blocky --> Blockly

please review & approve it if the issue raised is correct, so i can link the pr.


Fix draggable markers rotation

Fixes https://github.com/hyperknot/Leaflet/issues/12

shadowAnchor property related code is now cleaner. Draggable needs to stay that way, see my comment

Nice code now, only one check is missing before merge.

There you go

Great!


Azure Firewall Manager Policy update using Azure Powershell

Azure Firewall Manager Policy update using Azure Powershell

@kumudd Could you review this new article proposal for Firewall Manager? If it's accepted, it will need to be submitted through the private repository; the PR review team can move the commit. Please let us know. Thanks.

@vhorne Please can you review this new article proposal. Thanks!

@ktoliver I think this can proceed. It needs some editing and independent testing before publication, but it's a good start.

@ktoliver I think this can proceed. It needs some editing and independent testing before publication, but it's a good start.

@vhorne Could you add a metadata block or post one here in the comments and I'll add it? Then, I'll move the commit to the private repo so the automation checks for quality assurance items. Thanks.

@vidhoonv the title should be "Azure Firewall Manager Policy update using Azure Powershell"
'manager ' is missing in the title section "title: 'Quickstart: Create an Azure Firewall policy using Azure PowerShell'"

We'll close this pull request. I've captured the required revisions and will ask @vhorne to review the PR in the private repo before I sign off.


docs: add make and g++ packages

Ubuntu doesn't come with make or g++ installed by default
dpkg is mentioned in the package list but is not present in the command. I don't remember if I added it previously on my machine or not, so I didn't touch it.

thanks!

dpkg is mentioned in the package list but is not present in the command. I don't remember if I added it previously on my machine or not, so I didn't touch it.

dpkg is the underlying package manager of Ubuntu and Debian used by apt so it will almost always be present in Ubuntu


Add review delete option for admins and review authors

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 31 (16.13%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.9%) to 38.868%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/src/pages/WriteReviewPage/index.tsx
0
1
0.0%

client/src/pages/LocationPage/ReviewSection/Review.tsx
4
6
66.67%

client/src/pages/LocationPage/ReviewSection/index.tsx
0
9
0.0%

src/routes/reviews.ts
1
15
6.67%

Files with Coverage Reduction
New Missed Lines
%

client/src/pages/LocationPage/ReviewSection/Review.tsx
3
39.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.9%

Covered Lines:
375

Relevant Lines:
749

💛  - Coveralls


[BUG] You can delete the vlan in use

Steps to reproduce

start a vm and add a vlan network
delete the vlan network

Result
The vlan network is deleted without any hint

Expect
The vlan network cannot be deleted, and there is a response prompt

validated on master 45e2593


Fix crash in rb_vm_raise

Version: 0.12 (321) | be.michotte.hstracker
Stacktrace
-;rb_vm_raise;;
-;rb_exc_raise;;
-;rb_raise;;
-;rb_cmperr;;
-;rb_cmpint;;
-;sort_bang;;
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__sort_cards!__;ns_array.rb;15
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__deck_discard:__;opponent_tracker.rb;209
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__opponent_deck_discard:__;game.rb;439
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__tag_change:__;log_observer.rb;522
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__analyze:__;log_observer.rb;366
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__changes_in_file__block__3;log_observer.rb;121
-;vm_block_eval(RoxorVM*, rb_vm_block*, objc_selector*, unsigned long, int, unsigned long const*);;
-;rb_vm_yield_args;;
-;rb_yield;;
-;rary_each;;
-;rb_vm_dispatch;;
-;vm_dispatch;;
-;rb_scope__changes_in_file__block__2;log_observer.rb;120
-;vm_block_eval(RoxorVM*, rb_vm_block*, objc_selector*, unsigned long, int, unsigned long const*);;
-;rb_gcd_block_dispatcher;;
Reason
*** Terminating app due to uncaught exception 'ArgumentError', reason: 'in sort_cards!': comparison of PlayCard with PlayCard failed (ArgumentError) 	from in deck_discard:'
from in opponent_deck_discard:' 	from in tag_change:'
from in `analyze:'
'
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/219389/crash_reasons/43104614

dup of #123


fix: different weird things with empty task descriptions

Should fix that nothing shows up when the task description is not set at an Agora position.
Also deletes the category from the body overview, if it was set in the past but now empty again

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


-stdlib=libc++ appears to be ignored

From logs
V[10:40:54.061] Ignored diagnostic. argument unused during compilation: '-stdlib=libc++'
But this should surely affect the frontend.

I can't reproduce this. Are we sure the TU in question is a c++ file? I think this error would only show up for non-c++ TUs, e.g. if one passes -xc or so.

Yes, you're right - this was in combination with #555.


uboot: 2020.10 -> 2021.01

Motivation for this change

https://lists.denx.de/pipermail/u-boot/2021-January/437056.html

(Sadly they stopped doing actual changelogs)
Things done
I built every uboot packages:
 $ nix-build --keep-going --no-out-link \<nixpkgs\> -A pkgsCross.aarch64-multiplatform.ubootBananaPim64 -A pkgsCross.aarch64-multiplatform.ubootOdroidC2 -A pkgsCross.aarch64-multiplatform.ubootOrangePiZeroPlus2H5 -A pkgsCross.aarch64-multiplatform.ubootPine64 -A pkgsCross.aarch64-multiplatform.ubootPine64LTS -A pkgsCross.aarch64-multiplatform.ubootPinebook -A pkgsCross.aarch64-multiplatform.ubootPinebookPro -A pkgsCross.aarch64-multiplatform.ubootQemuAarch64 -A pkgsCross.aarch64-multiplatform.ubootROCPCRK3399 -A pkgsCross.aarch64-multiplatform.ubootRaspberryPi3_64bit -A pkgsCross.aarch64-multiplatform.ubootRaspberryPi4_64bit -A pkgsCross.aarch64-multiplatform.ubootRock64 -A pkgsCross.aarch64-multiplatform.ubootRockPi4 -A pkgsCross.aarch64-multiplatform.ubootRockPro64 -A pkgsCross.aarch64-multiplatform.ubootSopine
 $ nix-build --keep-going --no-out-link \<nixpkgs\> -A pkgsCross.armv7l-hf-multiplatform.ubootA20OlinuxinoLime -A pkgsCross.armv7l-hf-multiplatform.ubootAmx335xEVM -A pkgsCross.armv7l-hf-multiplatform.ubootBananaPi -A pkgsCross.armv7l-hf-multiplatform.ubootBananaPim3 -A pkgsCross.armv7l-hf-multiplatform.ubootBeagleboneBlack -A pkgsCross.armv7l-hf-multiplatform.ubootClearfog -A pkgsCross.armv7l-hf-multiplatform.ubootJetsonTK1 -A pkgsCross.armv7l-hf-multiplatform.ubootNovena -A pkgsCross.armv7l-hf-multiplatform.ubootOdroidXU3 -A pkgsCross.armv7l-hf-multiplatform.ubootOrangePiPc -A pkgsCross.armv7l-hf-multiplatform.ubootPcduino3Nano -A pkgsCross.armv7l-hf-multiplatform.ubootQemuArm -A pkgsCross.armv7l-hf-multiplatform.ubootRaspberryPi2 -A pkgsCross.armv7l-hf-multiplatform.ubootRaspberryPi3_32bit -A pkgsCross.armv7l-hf-multiplatform.ubootRaspberryPi4_32bit -A pkgsCross.armv7l-hf-multiplatform.ubootUtilite -A pkgsCross.armv7l-hf-multiplatform.ubootWandboard

All are building (as they did just before).
I have not tested it yet on hardware.

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

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

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

cc @petabyteboy @Mic92 @andir (recently showed interest in u-boot)

diff LGTM

diff LGTM

Just in case: do not merge until it was verified to work on hardware for at least one board.
(It'd be nice to package and prep a test for QEMU.)

Just in case: do not merge until it was verified to work on hardware for at least one board.
(It'd be nice to package and prep a test for QEMU.)

I got this message while building:
===================== WARNING ======================
CONFIG_OF_EMBED is enabled. This option should only
be used for debugging purposes. Please use
CONFIG_OF_SEPARATE for boards in mainline.
See doc/README.fdt-control for more info.
====================================================

But I'll ignore it if everything works, since I don't know if it's related to the update.

I got this message while building:
===================== WARNING ======================
CONFIG_OF_EMBED is enabled. This option should only
be used for debugging purposes. Please use
CONFIG_OF_SEPARATE for boards in mainline.
See doc/README.fdt-control for more info.
====================================================

But I'll ignore it if everything works, since I don't know if it's related to the update.

Booted on the rpi 3 and rpi 4, each with the device-specific and latest kernel :+1:

Booted on the rpi 3 and rpi 4, each with the device-specific and latest kernel :+1:

I got this message while building:

Yeah, their defconfig has the feature enabled.
Thanks for testing and merging!

I got this message while building:

Yeah, their defconfig has the feature enabled.
Thanks for testing and merging!


—
Reply to this email directly, view it on GitHub
https://github.com/NebulaMC-GG/Support/issues/1426#issuecomment-1154697457,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZTYV5263TVSSI2PVH5ZONDVPADL7ANCNFSM5YWB4P2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

@ytskylerplayz - how did you get these modifiers so I can fix it for the future please?

The crates
Sent from my T-Mobile 5G Device
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: HackoJacko @.>
Sent: Sunday, August 7, 2022 8:38:38 AM
To: NebulaMC-GG/Support @.>
Cc: ytskylerplayz @.>; Mention @.>
Subject: Re: [NebulaMC-GG/Support] only gave me 1x ball modifier when i should have gotten 3x (Issue #1426)
@ytskylerplayzhttps://github.com/ytskylerplayz - how did you get these modifiers so I can fix it for the future please?
—
Reply to this email directly, view it on GitHubhttps://github.com/NebulaMC-GG/Support/issues/1426#issuecomment-1207410476, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZTYV5YLCU2YEANOX6BQTCTVX6355ANCNFSM5YWB4P2Q.
You are receiving this because you were mentioned.Message ID: @.***>

Which one please? Vote?

i believe cosmic
On Thu, Aug 11, 2022 at 8:17 AM HackoJacko @.***> wrote:

Which one please? Vote?
—
Reply to this email directly, view it on GitHub
https://github.com/NebulaMC-GG/Support/issues/1426#issuecomment-1212133553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZTYV5YHUK2QPLAKMEHR7KDVYUKSFANCNFSM5YWB4P2Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ytskylerplayz - I have added an additional two Growth Modifiers to your inventory. Sorry for the delay - I've given you an additional one (3) as compensation.


Replicated msmarco-doc and msmarco-passage experiments for initial screening, Matthew Yang

Environment:
OS: macOS 11.2.1 (Big Sur)
Java: <IP_ADDRESS> (openjdk)
Python: 3.9.1
Problems encountered:

Initially used Java 8, had to switch to Java 11

Just to confirm, the results you got are exactly the same as the numbers in the documents?

Yes.


Fixes #1742 (emoji support)

The problem is not Android itself in this case, but normal UTF-8 received from other platforms.  The reverse would also be true (modified UTF-8 from Android would cause other platforms problems) so the answer is to convert to UTF-8 at write time and convert to modified at read time.  I think it might be better to actually convert to UTF-16 at read time since it will look like this:
UTF-8 -> UTF-16
Instead of this
UTF-8 -> modified UTF-8 -> UTF-16 (note that only required characters are converted, the rest are copied as is for the first step)
However, I'm going to leave it as the second since I'm unsure of the benefit, and the algorithm is more complicated than the one I have now.

Also be sure to take care of the submodule commits when merging.


Fixes for pod failover automation run

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

pkg/azdiskschedulerextender/azdiskschedulerextender.go
7
57.58%

Totals

Change from base Build<PHONE_NUMBER>:
-1.5%

Covered Lines:
205

Relevant Lines:
452

💛  - Coveralls


Use spotlight without supplying a DSN

Problem Statement
I want to be able to use spotlight without having to add a DSN.
Solution Brainstorm
Don't throw when spotlight is enabled, but no DSN is configured.
I have no clue whether this is an intended use case, but it would be nice if it's possible.
Are you willing to submit a PR?
Yes

Yeah sounds good, afaik other sdks do it as well
e.g https://github.com/getsentry/sentry-ruby/issues/2226 and same feature request on .net https://github.com/getsentry/sentry-dotnet/issues/3481

From the Dart SDK perspective it's pretty much just a change in the init assertion, would there be more needed on the native SDKs to make it possible?

would there be more needed on the native SDKs to make it possible?
probably the same, I don't think native currently allows using spotlight without dsn as well.

but now that you mention it, prolly makes sense to also enable spotlight on native when we enable spotlight in the flutter sdk (currently it doesn't)

would there be more needed on the native SDKs to make it possible?

probably the same, I don't think the native sdks currently allow using spotlight without dsn as well.
@buenaflor
It looks like the Android SDK needs a dsn: sentry.java

    if (!options.isEnabled() || (dsn != null && dsn.isEmpty())) {
      close();
      return false;
    } else if (dsn == null) {
      throw new IllegalArgumentException(
          "DSN is required. Use empty string or set enabled to false in SentryOptions to disable SDK.");
    }

@buenaflor I think this is unblocked as #2282 has been fixed with https://github.com/getsentry/sentry-dart/pull/2285

@kahest this is not fully unblocked yet because the cocoa and android sdk doesn't support this feature yet (we use the native sdk spotlight option there)
I'll open an issue for that in the cocoa and  android repo


Migrate ConvertPrestoServlet to JAX-RS

Hmm, to be honest, I don't think there is a big advantage to migrate to JAX-RS because it increases dependency and it might cause regression like https://github.com/yanagishima/yanagishima/pull/82
Of course, if there is direct merit for end-users we should consider but refactoring doesn't offer value to the user directly.
I think this job(https://github.com/yanagishima/yanagishima/issues/81) does not pay.
I'm not against refactoring but the merit per cost/risk is important.

The reason we should migrate is that it will decrease the development cost (≒time to implement new features) and removes redundant codes. Also, we can add tests easily after migration.
I don't think introducing dependencies aren't a problem in this project since this is a server-side application and not a library.
Anyway, I'm going to close this PR. I'm not sure why you didn't leave the comment on the issue. It looks just a waste of time to me.

I'm sorry for making you spend wasted time.


New-SFTPSession fails on macOS with Windows Registry error

Platform is macOS 10.13.4 with PowerShell 6 and .NET Core 2.1
When trying to create an SFTP session:
New-SFTPSession -ComputerName $server -Credential $creds -AcceptKey -Verbose
An exception is thrown seeming to indicate this isn't compatible with macOS, as it's referencing the Windows Registry:
New-SFTPSession : The type initializer for 'Microsoft.Win32.Registry' threw an exception.
At line:1 char:2
+  New-SFTPSession -ComputerName redacted -Credenti ...
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [New-SFTPSession], TypeInitializationException
+ FullyQualifiedErrorId : System.TypeInitializationException,SSH.NewSftpSession

$PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.2
PSEdition                      Core
GitCommitId                    v6.0.2
OS                             Darwin 17.5.0 Darwin Kernel Version 17.5.0: Mon Mar  5 22:24:32 PST 2018; root:xnu...
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The module is for Windows PowerShell it is compiled against full .Net and not for .Net Core

On Jun 26, 2018, at 1:30 PM, Paul<EMAIL_ADDRESS>wrote:
When trying to create an SFTP session:
New-SFTPSession -ComputerName $server -Credential $creds -AcceptKey -Verbose
An exception is thrown seeming to indicate this isn't compatible with macOS, as it's referencing the Windows Registry:
`New-SFTPSession : The type initializer for 'Microsoft.Win32.Registry' threw an exception.
At line:1 char:2
New-SFTPSession -ComputerName redacted -Credenti ...
CategoryInfo : NotSpecified: (:) [New-SFTPSession], TypeInitializationException
FullyQualifiedErrorId : System.TypeInitializationException,SSH.NewSftpSession
$PSVersionTable
Name Value
PSVersion 6.0.2
PSEdition Core
GitCommitId v6.0.2
OS Darwin 17.5.0 Darwin Kernel Version 17.5.0: Mon Mar 5 22:24:32 PST 2018; root:xnu...
Platform Unix
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion 2.3
SerializationVersion <IP_ADDRESS>
WSManStackVersion 3.0 `
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/218, or mute the thread https://github.com/notifications/unsubscribe-auth/AAf0HqJ3sRCG5InF8j13QPn9GbBbYt6fks5uAm_CgaJpZM4U4X4t.


feat(Image): Render children appropriately

Previously you could render <Image> with children but they  #would be passed to the , which is not technically valid markup.
Here's debug output from the rendersChildren test:

  Children are rendered here :|

This updates the Image component to render the .ui.image wrapper if children are present, which brings this more inline with the Input component. Both are wrapping elements around an HTML element which technically can't take children. However, since there could be other elements inside the wrapper this gives the user more control over how it renders.

Current coverage is 99.49% (diff: 100%)

Merging #784 into master will increase coverage by <.01%

@@             master       #784   diff @@
==========================================
  Files           136        136
  Lines          2185       2187     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2174       2176     +2
  Misses           11         11
  Partials          0          0

Powered by Codecov. Last update f4e1495...fdbf936

@jcarbo May be we can add example in usage section that explains markup?

Added an example for rendering children.

I also wound up:

pulling in those wireframe images into this repo so we can serve them locally instead of hotlinking to SUI-core
updating the title/descriptions of the Image examples to match SUI-core.

Just one comment regarding the circular warning.

@levithomason - I added the message about circular images back as requested. This should be g2g I think 👍

@jcarbo I aggre with @levithomason, __BASE__ looks horrible. Do we really need it there?
I looked on Dropdown examples:
image: { avatar: true, src: `${__BASE__}assets/images/avatar/small/jenny.jpg` },

It will be converted /assets/images/avatar/small/jenny.jpg in docs site and in local, too. I'm think can simply replace ${__BASE__} with '/' there.

Also, looks there is missing main, test for children rendering.

@layershifter - There was already a test for rendering children: https://github.com/Semantic-Org/Semantic-UI-React/blob/master/test/specs/elements/Image/Image-test.js#L13. It wasn't failing because it was just rendering children within the  tag:

  Children are rendered here :|

I would've thought React would complain since that's not valid markup but maybe that's an issue withe enzyme?
I'll give a shot at removing __BASE__. FWIW, it was something introduced before my time so blame @levithomason 😆

Just removed usage of __BASE__.
I realized what it was for - the docs for stardust were served from the /stardust path at technologyadvice.github.io while the local docs were served from the / path. __BASE__ was used to account for the difference. After switching to SUI-react, both the prod and local docs are served from / so we can just use the root absolute path for images (e.g. /assets/images/...).
Also, I don't think the images are the culprit with slow build times:

on first build: they may a little but it's just a raw copy/paste: https://github.com/Semantic-Org/Semantic-UI-React/blob/master/gulp/tasks/docs.js#L56-L59. I can't see how that would drastically decrease performance.
for rebuilds: they will only be rebuilt if an image file was touched: https://github.com/Semantic-Org/Semantic-UI-React/blob/master/gulp/tasks/docs.js#L139-L148 so the images should have no effect on rebuild time.

I think this is g2g. @levithomason - when you're ready 👍

@layershifter - just updated the src to put them inline. There were a few instances where the same src were used many times in the same example, so in those instances I kept it as a variable.

To be clear, we can/should use the globals (BASE, DEV, etc) in private code such as doc site components themselves, I just don't want them to show up in example code snippets (docs/app/Examples/*).


Fix small lint issues

This fixes some issues currently reported by Go Report Card.
https://goreportcard.com/report/github.com/gdamore/tcell

These changes look good.  I'm going to squash them all before I commit though.

Squashed and merged here:
commit 44772c121bb7838819d3ba4a7e84c0c2d617328e
Author: Jaime Piña<EMAIL_ADDRESS>Date:   Mon Dec 5 15:37:37 2016 -0800
Fix small lint & spelling issues


codegen statements no longer generating graphql files

Amplify CLI Version
10.8.1
Question
I am upgrading CLI from 6.1.1 to 10.8.1 (it's quite a leap!) and running into issues with with queries.graphql, mutations.graphql and subscriptions.graphql file generation.
Configuration
Amplify CLI 10.8.1
.graphqlconfig.yml
projects:
  acme-project:
    schemaPath: amplify/backend/api/acme-project/build/schema.graphql
    includes:
      - amplify-generated/graphql/**/*.ts
    excludes:
      - ./amplify/**
    extensions:
      amplify:
        codeGenTarget: typescript
        docsFilePath: amplify-generated/graphql
        region: us-west-2
        apiId: abcdefg
        maxDepth: 11
extensions:
  amplify:
    version: 5

Problem
When codeGenTarget: typescript is specified in .graphqlconfig.yml, none of these work to generate .graphql files:

amplify codegen generates only .ts files
amplify codegen statements generates only .ts files

queries.graphql, mutations.graphql and subscriptions.graphql files are not generated/updated in the path specified by docsFilePath or in any other path in the system.
I was able to confirm that with amplify CLI version 6.1.1 running
amplify codegen statements
works as intended. With 6.1.1 all three .graphql files are being generated and put in amplify-generated/graphql alongside mutations.ts, queries.ts and subscriptions.ts files.
However, if I change codeGenTarget to say
codeGenTarget: graphql
queries.graphql, mutations.graphql and subscriptions.graphql files are correctly generated. But then mutations.ts, queries.ts and subscriptions.ts are not generated.
And I need both .graphql and .ts files. That's how greedy I am!
The only way to currently work around this problem for me was creating a script and swapping codeGenTarget values in the .graphqlconfig.yml file on the fly so that it says codeGenTarget: graphql before running amplify codegen statements and then change it back to codeGenTarget: typescript before running amplify codegen again.
That way I had all six files created/updated correctly, but it's a nasty workaround.
Question
Is it an expected behavior in the current CLI version or am I missing something obvious? The documentation was not particularly helpful in this case.

I have had the same problem since forever and also the same solution of using scripts as a workaround..


Trailing characters

Hi everyone,
When I try and run the following:
ChatGPT test

I get the following error:

Trailing characters: test

I've attached a screenshot.

I've followed some feasible solutions @terror, but none of them seem to work.

Hmm strange, running this exact command on the following neovim version seems to work for me:
NVIM v0.8.3
Build type: Release
LuaJIT 2.1.0-beta3

Could it be that I'm using Lazy and it should look like this:
use({
  'terror/chatgpt.nvim',
     event = "VeryLazy",
     config = function()
       run = 'pip3 install -r requirements.txt'
    end
  }
}

As it turns out @terror, this seems to work (still working out somethings, might be useful to add to your documentation to your folks using Lazy instead of Packer.) Since I already have @jackMort's ChatGPT.nvim plugin installed, I'm getting a duplicate tag error, maybe we can work on this together so the widely used other ChatGPT.nvim has synergy. I could see foresee this being a problem for many people. Here's my core core.lua file:
{
"jackMort/ChatGPT.nvim",
    config = function()
      require("chatgpt").setup()
    end,
},

{
'terror/chatgpt.nvim',
     event = "VeryLazy",
     config = function()
       run = 'pip3 install -r requirements.txt'
    end
  }
}

Here's a screenshot for when I run Lazy install all:

same issue

It's worth noting - I'm running this instance of AstoNVim on Docker. I run it via:
docker run -it alpine:edge
apk add git curl NeoVim

Then to the rest of the configuration, I don't know if that would effect it, (I don't see how it would.)
Cheers,
Montana Mendy

Hi @Montana
I too am getting the "E488 Trailing Characters" issue, the issue for me is i'm not sure how to edit the python file. there's not that much documentation. the video showed was only about 30 seconds and it looked like author had it already installed but I am getting "E488 Trailing Characters" . I even tried doing in vim :%s/\/apps/log_dir/g this is  where string to replace=/apps and replaced string=log_dir

Hi @EvilOtto2,
There is a question, on some imports in this Python file, that don't exist in the Python file in the really cool video @terror  posted. I'll attach screenshot. In particular Enum.

I've debugged a little, the only thing I can think of is since @jackMort's plugin is called ChatGPT.nvim and this one is called chatgpt.nvim there might be some naming collision.

Theoretically it could be just a weird setting, but I'm wondering if there Is a naming collision issue.
Cheers,
Montana Mendy

Hi @EvilOtto2,
There is a question, on some imports in this Python file, that don't exist in the Python file in the really cool video @terror posted. I'll attach screenshot. In particular Enum.

I've debugged a little, the only thing I can think of is since @jackMort's plugin is called ChatGPT.nvim and this one in particular @terror's is called chatgpt.nvim, is it feasible that there might be some naming collision?

Theoretically it could be just a weird setting in nvim that needs to be changed, but I'm wondering if there is a naming collision issue here at hand? I honestly even question that though given the casing of each plugin.
Cheers, Montana Mendy

It seems like you're running an outdated version of the plugin, I'd try either updating or re-installing the plugin entirely -- I've since switched to using the official OpenAI python bindings instead of the reverse-engineered one. Let me know if this works!

Very interesting @terror,
I also have the Enum library being called to be imported in the chatgpt_nvim.py, I'll run:
rm -rf chatgpt.nvim
git clone https://github.com/terror/chatgpt.nvim

Then re-install. This is probably the fix. I really appreciate it @terror.

Hi @terror,
To confirm, is the one you show in asciinema the current chat_nvim.py? I only ask that because looking at the current repo, both still have the library Enum being imported to the headers, on both branches that exist.

In the asciinema video, these don't exist. Just wondering, and if so is there anyway I could see that configuration instead of the ones pushed up (that is if that's the most current build.) Thanks again @terror.

Hey everyone,
I've finally got it working, the issue is things like NeoVim, AstroNVim, LunarVim, I could go on are moving away from Packer. I've yet to find a good way to install this through Lazy, which seems to be the new package standard and not Packer.
Query:

Response:

Cheers,
Montana Mendy


Sync prometheus

I upstreamed ShardedPostings() (PR). In this PR I'm re-synching upstream Prometheus fixing the related conflicts due to small changes introduced upstream.
Note: I've re-organized the sorting of a couple of struct fields to reduce future conflicts.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 15 committers have signed the CLA.:white_check_mark: bboreham:white_check_mark: ptodev:white_check_mark: aknuds1:white_check_mark: beorn7:white_check_mark: gouthamve:white_check_mark: roidelapluie:white_check_mark: pracucci:x: kalpadiptyaroy:x: bwplotka:x: Nexucis:x: fatsheep9146:x: SuperQ:x: Leegin-darknight:x: Loori-R:x: fpetkovskiYou have signed the CLA already but the status is still pending? Let us recheck it.


[probes.grpc] Add validators to gRPC probe

We can pass gRPC probe's response to validators as json and run jq validator on it.

Note: we already run validator for gRPC probes but we don't use response body as json.


Package and Configuration management system

I'll make a Configuration management system which handles

Constant string such as Package name of this plugin
Scripting Define Symbols to add when this package is installed

https://github.com/styly-dev/STYLY-visionOS-Plugin/issues/6#issuecomment-1950250266


How to make an alarm following the principles of Clean Architecture?

Hi, I have a question, if I have to implement something like alarm, that will trigger at specific time and run my use case, setting an alarm is business logic of my app, so it should be located in domain(direct me if I wrong) and it's use case should set up my alarm, so it should has the android classes. At this moment I have domain module like android library, and one use case set up an alarm that starts a broadcast, ideally that broadcast should run the same use case (and set up another alarm). I would like to know a valid approach, how to do that.
Logic:
(implemented) 1. User add cycle transaction and add set alarm. AddCycleOperationUseCase --> write to db
(implemented) 2. AddCycleOperationUseCase set the AlarmManager with intent with CycleOperation model
(want) triggering broadcast that ideally has injected AddCycleOperationUseCase and it will do same operation again.
P.S. I saw few examples of CA by Uncle Bob and it seems domain module it's something like a wrapper that encapsulates CRUD, it isn't?

I have a similar logic in my study project. I defined a interface in the domain layer(https://github.com/enuoCM/DE-MVP-Clean/blob/master/app/src/main/java/com/xixicm/de/domain/interactor/SentenceFetchScheduler.java), and a use case(https://github.com/enuoCM/DE-MVP-Clean/blob/master/app/src/main/java/com/xixicm/de/domain/interactor/ScheduleFetchTodaysSentenceUC.java) to call it. The alarm implementation of the SentenceFetchScheduler interface is in a separate  infrastructure layer(https://github.com/enuoCM/DE-MVP-Clean/blob/master/app/src/main/java/com/xixicm/de/infrastructure/alarm/AlarmSentenceFetchScheduler.java).


Fail to send alerts to alertmanager due to EOF

What did you do?
We configured multiple(1000+) prometheus instance to send alerts to one alertmanager, at the beginning, everything is fine, after a few minutes, we found some prometheus failed to send alerts due to the error:
level=error ts=2021-07-01T12:33:16.623Z caller=notifier.go:527 component=notifier alertmanager=https://alertmanager/api/v2/alerts count=1 msg="Error sending alert" err="Post \"https://alertmanager/api/v2/alerts\": EOF"

What did you expect to see?
All prometheus instances send alerts to Alertmanager successfully.
What did you see instead? Under which circumstances?
We found some prometheus failed to send alerts due to the error:
level=error ts=2021-07-01T12:33:16.623Z caller=notifier.go:527 component=notifier alertmanager=https://alertmanager/api/v2/alerts count=1 msg="Error sending alert" err="Post \"https://alertmanager/api/v2/alerts\": EOF"

Environment
Prometheus v2.4.2
Alertmanager 0.21.0
Checked the source code for the client http transport settings, we found that each prometheus keeps large max idle connection(20000 in total and 1000 per host) to send alerts to Alertmanager.
See: https://github.com/prometheus/common/blob/a1b6ede20323252d2b99a0f57178a4b7d364d0ca/config/http_config.go#L370-L380
I wonder why we hard code these settings, they should be customized by end users. For my case, we have 1000+ Prometheus to send alerts to one Alertmanager, the client connection numbers will be *1000 for the upstream Alertmanager.
If there are some rate limit in upstream, the connection will be closed. In any case, we need to make sure the connection from each Prometheus instance can be customized.

Prometheus should not use that many connections. I suggest you to update
prometheus and alertmanager to the latest release to benefit for all the
bugfixes, including the go bugfixes. Please also check the number of file
descriptors (limit) of each process.
Le mar. 6 juil. 2021 à 04:39, Morven Cao @.***> a
écrit :

What did you do?
We configured multiple(1000+) prometheus instance to send alerts to one
alertmanager, at the beginning, everything is fine, after a few minutes, we
found some prometheus failed to send alerts due to the error:
level=error ts=2021-07-01T12:33:16.623Z caller=notifier.go:527 component=notifier alertmanager=https://alertmanager/api/v2/alerts count=1 msg="Error sending alert" err="Post "https://alertmanager/api/v2/alerts\": EOF"
What did you expect to see?
All prometheus instances send alerts to Alertmanager successfully.
What did you see instead? Under which circumstances?
We found some prometheus failed to send alerts due to the error:
level=error ts=2021-07-01T12:33:16.623Z caller=notifier.go:527 component=notifier alertmanager=https://alertmanager/api/v2/alerts count=1 msg="Error sending alert" err="Post "https://alertmanager/api/v2/alerts\": EOF"
Environment
Prometheus v2.4.2
Alertmanager 0.21.0
Checked the source code for the client http transport settings, we found
that each prometheus keeps large max idle connection(20000 in total and
1000 per host) to send alerts to Alertmanager.
See:
https://github.com/prometheus/common/blob/a1b6ede20323252d2b99a0f57178a4b7d364d0ca/config/http_config.go#L370-L380
I wonder why we hard code these settings, they should be customized by end
users. For my case, we have 1000+ Prometheus to send alerts to one
Alertmanager, the client connection numbers will be *1000 for the upstream
Alertmanager.
If there are some rate limit in upstream, the connection will be closed.
In any case, we need to make sure the connection from each Prometheus
instance can be customized.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/9057, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACHHJR4QJ6B4QQNVTPHMRLTWJUEZANCNFSM473U2KDA
.

thanks @roidelapluie for the quick reply.
I tried to upgrade to latest version, it didn't fix the issue.
I checked the source code, and the root cause should that the http connection is not closed proactively after each alert send request is done.
see: https://github.com/prometheus/prometheus/blob/62598878dd080d0488457e2238fb6e5a847a8247/notifier/notifier.go#L580-L590
actually resp.Body.Close() is not enough in my case, because the connection is till there until maxIdleConnectionTimeout, I add the following code:
req.Close = true

to make sure http connection is closed after the request is done, and then I rebuild it and didn't see EOF any more.
So IMO, prometheus should probably provide an option in settings to close the http connection, or customize the http connection settings, there are many scenarios similar to mine, there is rate limit in the upstream alertmanager, which will make the situation worse.

I did more experiments, if I change the http connection settings as follows, the issue is also gone.
diff --git a/config/http_config.go b/config/http_config.go
index 07de306..0fde8f9 100644
--- a/config/http_config.go
+++ b/config/http_config.go
@@ -223,14 +223,14 @@ func NewRoundTripperFromConfig(cfg HTTPClientConfig, name string, disableKeepAli
                // It is applied on request. So we leave out any timings here.
                var rt http.RoundTripper = &http.Transport{
                        Proxy:               http.ProxyURL(cfg.ProxyURL.URL),
-                       MaxIdleConns:        20000,
-                       MaxIdleConnsPerHost: 1000, // see https://github.com/golang/go/issues/13801
+                       MaxIdleConns:        200,
+                       MaxIdleConnsPerHost: 100, // see https://github.com/golang/go/issues/13801
                        DisableKeepAlives:   disableKeepAlives,
                        TLSClientConfig:     tlsConfig,
                        DisableCompression:  true,
                        // 5 minutes is typically above the maximum sane scrape interval. So we can
                        // use keepalive for all configurations.
-                       IdleConnTimeout:       5 * time.Minute,
+                       IdleConnTimeout:       5 * time.Second,
                        TLSHandshakeTimeout:   10 * time.Second,
                        ExpectContinueTimeout: 1 * time.Second,
                        DialContext: conntrack.NewDialContextFunc(

This would effectively disable keepalive, which is not desired.
Did you look at the actual bottleneck? It could be a bottleneck on the network or on the alertmanager, how many file descriptors are open and how many are allowed?

@roidelapluie
We have a gateway in front of the Alertmanager, and the gateway is implemented HAProxy, which has client-connection timeout less than the IdleConnTimeout of the prometheus. I checked the prometheus code, the IdleConnTimeout of the prometheus is 5 minutes and it can be customized for sending alerts.
If I understanding it correctly, this will results in broken http connections that the prometheus doesn't know.

The maxConnection for the gateway is 20000, equal to the value of MaxIdleConns for single prometheus instance. But we have 1k+ prometheus instances sending alert via the same gateway, the connections will not be sufficient in gateway.
But if we can customize the maxIdleConnection at the prometheus side, we can make it to a more reasonable value to make sure each prometheus instance can have active connections.

Do you have evidence that Prometheus is actually using those? This should be a connection pool and Prometheus should use one or two connections. However, it could be a misconfiguration of HAProxy. HAProxy client-connection timeout probably has nothing to do with this situation.
note: you could also disable the keepalive in haproxy: option httpclose

@roidelapluie thanks, we currently don't have option httpclose, in HAProxy settings, but we have option http-pretend-keepalive.
Anyway, let me give it a try.

thanks @roidelapluie for your suggestion.
We figured out the issue is actually caused by the connection closed by the proxy end, but the fix is not that easy, because the change to the proxy may impact may services. We wonder if it possible to add retry logic for alert sending.

option httpclose can be used with an ACL to only match prometheus traffic. Also, this is doing the same as you have done with resp.Close. I think you might have a bottleneck somewhere in your network, unrelated to this setting that has not changed or caused issues in years.
I also recommend you to upgrade prometheus to benefit from 3 years of improvements and bugfixes in the golang http libraries.

hi:
I also had a similar error, but it happened in the process of prometheus notifying alertmanager, please move: https://github.com/prometheus/prometheus/issues/9176
Do you have a solution to the error, what should I do, thank you very much

option httpclose can be used with an ACL to only match prometheus traffic. Also, this is doing the same as you have done with resp.Close. I think you might have a bottleneck somewhere in your network, unrelated to this setting that has not changed or caused issues in years.
I also recommend you to upgrade prometheus to benefit from 3 years of improvements and bugfixes in the golang http libraries.

hi:
I also had a similar error, but it happened in the process of prometheus notifying alertmanager, please move: https://github.com/prometheus/prometheus/issues/9176
It is urgent, do you have a solution to the error, what should I do, thank you very much

option httpclose can be used with an ACL to only match prometheus traffic. Also, this is doing the same as you have done with resp.Close. I think you might have a bottleneck somewhere in your network, unrelated to this setting that has not changed or caused issues in years.
I also recommend you to upgrade prometheus to benefit from 3 years of improvements and bugfixes in the golang http libraries.

hi:
I have a similar error, please move: https://github.com/prometheus/prometheus/issues/9176
It's urgent, do you have a solution to the error, what should I do, thank you very much

I have the same problem. Is the configuration file or startup command incorrect? Please go to https://github.com/prometheus/prometheus/issues/9176 to view the detailed information. I have built a set of prometheus and alertmanager environments without involving the introduction of any code layer. At the beginning, both prometheus and alertmanager are running normally, and the alarm is also normal. The error is reported after a short while, please give some solutions or suggestions

In my case, I have proxy between prometheus and alertmanager. I made some changes to the proxy settings to timeout of closing idle connection, it mitigated the issue but not resolve this, still looking for a better solution.

In my case, I have proxy between prometheus and alertmanager. I made some changes to the proxy settings to timeout of closing idle connection, it mitigated the issue but not resolve this, still looking for a better solution.

Can you leave the WeChat id contact information, we seem to be closer，  happy

@morvencao - have you found a better solution since your last comment ?

no better solution found @AlexandreRoux

If we make maxIdleConnectionTimeout configurable, it would solve the problem right? I see a lot of discussion around closing connection per request, but looks to me like allow configurable maxIdleConnectionTimeout  is the way to go here?

Also, looks like this issue and https://github.com/prometheus/prometheus/issues/9176 are dups :)

Retries for alert sends implicitly happen on the next rule evaluation interval. We should be careful not to pile up too much in-flight or pending work, so for that reason I am wary of adding another layer of retries. They have the potential to make a slightly overloaded Alertmanager very overloaded very quickly. We could retry specific errors that definitely mean the request did not go out, but I am not sure EOF guarantees that.
However, for the same reason, I do agree that up to 20000 open connections per Prometheus, with no way to reduce that, is excessive. Real world installations with hundreds of servers exist, and I don't think it helps anyone if Alertmanager has to juggle millions of TCP connections. "Check file handle limits" is easier said than done esp. for containers, and you always find out that you missed a spot at the worst of times.

Hello from the bug-scrub!
Over 18 months has gone by with no movement.  We will leave the issue open since this and #9176 are unsolved.
If anyone has an idea how to investigate it, please drop a comment below.


New API: setSharedThemeColorProvider(_:)

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Adds a new API for changing the ColorProviding instance for the shared theme.
Usage:
// Swift
let colorProvider: ColorProviding = /* create your own instance of ColorProviding */
FluentTheme.setSharedThemeColorProvider(colorProvider)

// Now all calls to `FluentTheme.shared` will route through your color provider

// Objective-C
id<MSFColorProviding> colorProvider = /* create your own instance of ColorProviding */
[MSFFluentTheme setSharedThemeColorProvider:colorProvider];

This makes it possible to change the color scheme of an entire app at once, instead of solely per-window.
In order to test this, I overhauled the themes used by the FluentUI Demo app, removing the default theme (itself just a copy of the shared default theme) and replaced it with a new "purple" theme.
There's also a new picker in the appearance customization dropdown for changing the theme of the entire app, and the existing "Theme" picker is now called "Window Theme".
Binary change
Total increase: 49,160 bytes
Total decrease: -2,424 bytes

File
Before
After
Delta

Total
30,935,680 bytes
30,982,416 bytes
⚠ 46,736 bytes

 Full breakdown

File
Before
After
Delta

FluentTheme.o
123,424 bytes
155,144 bytes
⚠ 31,720 bytes

DateTimePickerController.o
138,800 bytes
144,016 bytes
⚠ 5,216 bytes

__.SYMDEF
4,817,896 bytes
4,821,272 bytes
⚠ 3,376 bytes

ColorProviding.o
41,480 bytes
43,416 bytes
⚠ 1,936 bytes

HUD.o
257,784 bytes
259,672 bytes
⚠ 1,888 bytes

FocusRingView.o
817,288 bytes
818,984 bytes
⚠ 1,696 bytes

PopupMenuController.o
382,680 bytes
383,856 bytes
⚠ 1,176 bytes

MultilineCommandBar.o
140,592 bytes
141,272 bytes
⚠ 680 bytes

FluentUIFramework.o
104,320 bytes
104,816 bytes
⚠ 496 bytes

AvatarGroup.o
420,360 bytes
420,632 bytes
⚠ 272 bytes

BottomCommandingController.o
848,112 bytes
848,376 bytes
⚠ 264 bytes

DateTimePickerViewComponentTableView.o
67,800 bytes
67,992 bytes
⚠ 192 bytes

DateTimePickerViewComponent.o
138,048 bytes
138,152 bytes
⚠ 104 bytes

DateTimePickerView.o
207,920 bytes
207,968 bytes
⚠ 48 bytes

DrawerController.o
482,952 bytes
482,992 bytes
⚠ 40 bytes

BadgeView.o
625,304 bytes
625,328 bytes
⚠ 24 bytes

NavigationBarTokenSet.o
72,328 bytes
72,344 bytes
⚠ 16 bytes

PopupMenuItem.o
125,672 bytes
125,688 bytes
⚠ 16 bytes

DateTimePickerViewLayout.o
59,784 bytes
57,360 bytes
🎉 -2,424 bytes

Verification

Visual Verification

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[x] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[x] iPad Pointer interaction
[x] SwiftUI consumption (validation or new demo scenarios needed)
[x] Objective-C exposure (provide it only if needed)

Is there anyway to update the app-wide theme picker in both windows? At the end of your gif it's on purple on the left window but default on the right, and since default was chosen more recently everything is blue. But it'd be better if the two app-wide pickers were in sync.

Is there anyway to update the app-wide theme picker in both windows? At the end of your gif it's on purple on the left window but default on the right, and since default was chosen more recently everything is blue. But it'd be better if the two app-wide pickers were in sync.

Good point! I'll see if there's a trivial fix, otherwise it may come in a future PR.

Is there anyway to update the app-wide theme picker in both windows? At the end of your gif it's on purple on the left window but default on the right, and since default was chosen more recently everything is blue. But it'd be better if the two app-wide pickers were in sync.

Good point! I'll see if there's a trivial fix, otherwise it may come in a future PR.

@huwilkes done! Uploaded a new verification GIF to show the fixed behavior.


Creating a closing quote when typing a contraction in a comment

VSCode Version: 1.2.1
OS Version: Windows 10.0.10586

Steps to Reproduce:

Be editing python document
Be typing in a comment
Type a contraction such as isn't
Notice that the editor creates a closing apostrophe that is inappropriate in this context.

Not sure if this is inside the extension or core. But I have seen issues with wrong auto closing often in selfhost too.

Well... this has been there since day 1.
@alexandrudima @jrieken @aeschli Does anyone know how JavaScript solves this, since it's solved there?


Publish new versions to Github container registry (ghcr.io)

📑 I have found these related issues/pull requests
did a search, didn't find anything related
🏷 Feature Request Type
Deployment
🔖 Feature description
I hope you're doing well! I wanted to suggest publishing container packages to this repository (e.g., ghcr.io/) as an alternative to Docker Hub for pulling updates. Docker Hub’s daily rate limits for polling containers can be quite restrictive, so having another option would be incredibly helpful for users.
Thanks so much for considering this, and keep up the great work!
✔ Solution
publish container versions to ghcr.io so users can pull container updates using an image path of ghcr.io/louislam/uptime-kuma:
documentation reference: https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry
❓ Alternatives
n/a
📝 Additional Context
n/a

We already do that
https://github.com/louislam/uptime-kuma/pkgs/container/uptime-kuma

We already do that https://github.com/louislam/uptime-kuma/pkgs/container/uptime-kuma

Yeah but there are no actual versions there - just nightlies and hash commits. Are there plans for adding versioned packages?

We will release future versions (anything after the V2 beta) there as well


24hr reset

How do I reset the energy reading to zero at midnight, and also is it normal for power reading to be zero occasionally.
Thanks for the yaml code, I tried to code it myself and I got very close to your solution, its the lambda bits I don't understand, more reading required I think

Hi,
You should not have to reset anything. What you need to do is to implement an utility meter in HomeAssistant that will track the usage.
The Utility Meter will count the energy used (the pulses of your meter) and you can have the energy meter automatically reset every day, week, month, year etc.
See info here: https://www.home-assistant.io/integrations/utility_meter/
Not sure about the power reading be zero. I think documentation says that if no pulse is detected for 5 minutes (default unconfigured value) it will then show zero?

Thanks for that, I am already using the utility meter for my consumption and export, I just didn't think to use it with this.
As regards the zero reading this was solved by reintroducing 100ms filter that was commented out.


here's bug?

package src;
import src.*;
public class Test{
public static void main(String[] args){
A a = new A();
String s = a.id;
s.length();
}
}
Here s maybe a nullpointer.
But infer said "No issues found"

What is the code of class A?
What command did you use to run infer?
What version of infer are you using?
Can you paste the full output?

Thanks!

Sorry for my mistake. I got the result because I didn't clean my .class files.
But it really dosn't work when I check my java project,  the code is like this:
String docvrid = doc.getClassid();
if (! docvrid.equals("10000103")) {
...
}
doc.classid is a private member and initialized from a xml in the constructed function, but [classid] in the xml maybe null. So, the code 'docvrid.equals("10000103")' maybe call a NullPointerException.
The version of infer I'm using is 0.8.0.
The command I ran infer is infer -- ant.

Hi @chen881129. Infer does not parse the xml files, so the information whether doc.getClassid() may  return null is not known. In this case, Infer will make optimistic assumptions about the code. Here, docvrid being dereferenced in docvrid.equals(...), Infer will assumes that docvrid != null. If you know that docvrid can be null, you can annotate the code accordingly:
@Nullable String docvrid = doc.getClassid();

And Infer will then make the opposite assumption: docvrid will be considered as potentially null unless there is enough information to state otherwise. For instance:
  if (docvrid != null && !docvrid.equals("10000103")) {
   ...
  }

Hello @jeremydubreil, very nice to receive your message.
You are right that Infer dose not know docvrid is null. Actually docvrid is not always null, it's potentially null. So, docvrid != null && !docvrid.equals("10000103") will be safe and docvrid.equals("10000103") without docvrid != null is dangerous.
In my opinion the dangerous code should be print. Can Infer print the potentially NullPointerException?

In theory, Infer could report every time there is not enough information to prove that a variable being dereference is not null. But in practice, this will generate way too many reports and make the tool unusable.

If you want more demonic behavior w.r.t nullability, you might want to give eradicate a try (infer -a eradicate -- <your_compile_command>). Eradicate requires more annotations than Infer, but is also more exhaustive in reporting NPE warnings.

Check out my answer to https://github.com/facebook/infer/issues/383 to see another way to make Infer's NPE reporting more aggressive.


Gardening: Address some new warnings

There are bunch of new warnings emitted when compiling the Swift compiler and runtime since rebranch was merged. This PR addresses a couple that are trivial to address:

Fix -Wunsued-but-set-variable warnings by not setting a superfluous variable.
Suppress -Winvalid-offsetof warnings in static_asserts.

@swift-ci please smoke test

@swift-ci please smoke test Linux

@swift-ci please smoke test


Add cBagger's update-swank...

It's simple and so very useful. I can also remove the threading.

I have a fix for sdl2. The other systems work fine.

Update-Swank forces CLinch to require Swank. I'm not adding it, it should be part of the user's project. I'll add it as an example.


[AIRFLOW-416] Use ordinals in README's company list

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-416

Current coverage is 64.53% (diff: 100%)

Merging #1726 into master will not change coverage

@@             master      #1726   diff @@
==========================================
  Files           127        127
  Lines          9450       9450
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           6099       6099
  Misses         3351       3351
  Partials          0          0

Powered by Codecov. Last update 9aa7da5...e30b2f7

With the numbered list we will have to update the rest of the numbers whenever a company asks to be removed or we add a new company (since we keep the list alphabetized), how about putting "over X companies use airflow:" before the list, or something similar?

I agree with @aoen, this will be very difficult to maintain. However, if you number everything as "1.", Markdown will automatically increment the numbers.

1. my
1. auto
1. incremented
1. list

renders as:

my
auto
incremented
list

@aoen @jlowin thx. Both good suggestions.. I'll go with making everything 1... good solution.

@aoen @jlowin fixed.

LGTM


please make boot process "synchronous" + provide simple logging with "log_file", "log_level" options.

the default boot process is asynchronous and does not allow developers to point ports to volumes like this;

7001 -> volume_1
7002 -> volume_2
7003 -> volume_3
7004 -> volume_4
so on..

I handle file routes in backends , so I need to do this programmatically. I've also tried with "& wait & " in shell but the result is same.
As I point also here; https://github.com/chrislusf/seaweedfs/issues/107
program waits for volumes to be ready, and logs info. until they are ready. You said, it's ok, but it's not ok for real world apps.
devops are not always logging "infos", for high traffic websites, devops prefer to disable logs for infos , and activate them for more critical things like warnings, fatal errors, so on...
And they trigger that logs with sms, email...
Another thing that I want to discuss is; logging operation itself. I've looked into the default log option, and found that "not productive". Why not we can't use simple logging like another daemons ?
log_level (warning, info, error) , log_file (path_to_file) and that's all. So, we can integrate it into centralized logs like logstash ...
In short, please;
1. Make boot process "synchronous".
2. Provide simple logging with "log_file", "log_level" options.

Is it only me who think about the above improvements or we are more ?
+1 if you agree.

Thanks for this open source super file system.

volume_size_in_mb="$((${volume_size} * 1000))"

command="./weed master -mdir=\"${directory_storage}/seaweedfs/meta/\" -port=${port_master} -volumeSizeLimitMB=${volume_size_in_mb}"

i="0"
while [ "${i}" -lt "${volume_number}" ]; do
	i=$[$i+1]

	port_volume=$[$port_volume+1]

	command+=" & wait & ./weed volume -dir=\"${directory_storage}/seaweedfs/vol_${i}/\" -max=1 -mserver=\"localhost:${port_master}\" -port=${port_volume} -images.fix.orientation=false -leveldb=${leveldb}"
done

eval $(echo $command)

Seaweedfs uses glog. Try "weed -h" to see the options. Does it help?

Sorry, I didn't understand. can you clarify a little big ? yes, I've checked "weed -h" help options, but didn't/can't find anything useful for my use case. And may it's already there the solution of my problem, but I just can't see that. You might help for this. :+1:

If you talk about the logging option, I know it exists, what I mean is, making it simple as my explanation above. keeping logging boot process is not something useful. anyway I'm just sharing my ideas.

You can use -v=4 to change log levels.

Regarding "making boot synchronous":

When starting master server, if you can clean the master meta folder specified by "-mdir", it will be much faster to startup.
It's better to have multiple volumes for one volume server. So you do not need to have such a complicated startup script.

2. It's better to have multiple volumes for one volume server. So you do not need to have such a complicated startup script.
Well, thanks for your response. It's not that complicated. However, it's how I create port-volume relationship. Do you have any suggestion how to do it better? Let's say I have 20 volumes and then
port 7000 - as a namespace for volume 1-5,  7100 - as a namespace for volume 6-10, and so on...
7000 - volume_server_1 - 1,2,3,4,5
7100 - volume_server_2 - 6,7,8,9,10

The problem is, in backends, I'm not asking SeaweedFS for a file location, I'm directly sending a file to the specific volume server. And I didn't find any solution yet. May, you can put another flag/argument where we start volumes or something else option to allow user to create port-volume-volume-server relationship. Any ideas? :)

I think, volume id;s should not be randomized at startup, that's the core of the problem. So, if they keep starting asynchronously, then they will be randomized always. correct me, if I'm wrong.

How about something like this?
./weed volume -dir="/data/seaweedfs/vol_1/" -max=7 -mserver="localhost:7000" -port=7001 -images.fix.orientation=false -leveldb=true" volume_ids="1,2,3,4,5,6,7"
so, we can be sure that; volume_ 7001 have 1,2,3,4,5,6,7. I think, I should not be hard to do this. You can also add some validations, for the new arg -volume_ids with -max arg.

Use this HTTP API to create a specific volume:
https://github.com/chrislusf/seaweedfs/wiki/API#create-а-specific-volume-on-a-specific-volume-server

It seems OK to me, but how then I can restart master-slaves?

the same way as I start volumes? some options are confused :S

https://github.com/chrislusf/seaweedfs/wiki/Failover-Master-Server#startup-multiple-master-servers ?

I removed the second section "when-you-need-to-restart". It is confusing. Just follow this section:
https://github.com/chrislusf/seaweedfs/wiki/Failover-Master-Server#cheat-sheet-startup-multiple-servers

Thanks ! I'll re-code something new today to see that my expectation solves the problem. So, I can back here to close this issue asap :) Thanks.

right now, I'm back on the issue. Can you explain how can I use the followings together:
https://github.com/chrislusf/seaweedfs/wiki/Failover-Master-Server#cheat-sheet-startup-multiple-servers
https://github.com/chrislusf/seaweedfs/wiki/API#create-а-specific-volume-on-a-specific-volume-server
help says: /weed volume starts volume, and that's fine. If we dont use -max= it automatically starts up with 7 volumes. Let's say we have 1 master 1 volume,  so we have 1,2,3,4,5,6,7 parts in volume server, so
how this helps in this scenario?
https://github.com/chrislusf/seaweedfs/wiki/API#create-а-specific-volume-on-a-specific-volume-server
as we already have 1,2,3,4,5,6,7..
Also, I've looked at glog doc and found that it does not allow to use same file for logging, so it's possible to have thousands of log files..

Your daemon is very good! I respect your effort, but I feel like I'm wasting my time. SeaweedFs is behind a reverse proxy, so I need something that is not randomized for each start-up.  port -> volume ->parts .. if I can't this map, it's just useless for me.

If you would like help me at this point, please provide me pseudo code, not doc please. I'm reading them over and over again, and I can't solve my problem with the exist docs. I don't know, may the solution is already there but I can't catch that. :S Any help is welcome..

Yeah, finally, I've skipped the docs, and just applied what you said. It works!
curl "http://localhost:8080/admin/assign_volume?replication=000&volume=1" however, why returns {"error":""} ? I think, {"success":"Your command has been run successfully"} is more preferable. Anyway, Thanks, your help saved my day :+1:


Added 'HotReload' in Meta section in README.md

Hi Amis92,
I've added a new link about enabling hot reload of source generator projects. This should be useful for people building their own generators. And I hope that the formatting is OK ;-)
Best regards,
Andrzej

Sure, fixed it now, please take a look.


[Android] Keyboard accessibility follow up

Summary
This is a follow up PR to #24359. There's a good thread in the mentioned PR for more background for why I'm doing this change. Essentially focusable makes more sense since it is about whether a view can receive user-initiated focus from a pointer or keyboard.

@cpojer @necolas FYI


No placeholder shows on 'Search' box for one blob container

Storage Explorer Version: 1.10.0
Platform/OS: Windows 10
Architecture: x64
Language: Chinese (zh-CN)/ Russian(ru)/ German(de)
Regression From: Not a regression
Steps to reproduce:

Run the command 'StorageExplorer.exe --lang=zh-CN' to launch Storage Explorer.
Expand a Non-ADLS Gen2 account -> Blob Containers.
Create a new blob container then check the search box on the editor.

Expect Experience:
The 'Search' box is filled with 'Search by prefix (case-sensitive)'

Actual Experience:
The 'Search' box is empty.

This issue has been fixed on the build 20190914.1. So we close it.