[CONTINT-3435][PLD] Forward every query to the leader

What does this PR do?
This PR forwards every request received by DCA followers to the leader.

Motivation
Will be useful for cleanup and simplicity.

Additional Notes
The cluster checks handler has a similar mechanism but it requires some work to extract the logic that handles the leader election from the logic that handles cluster checks.
For simplicity and to avoid unexpected breaking changes in the clc handlers, we chose a different approach.

Possible Drawbacks / Trade-offs
GetLeaderIP will not return an error if the leader is not listed in the endpoints object. This function is only used in CLC runners.

Describe how to test/QA your changes

Deploy multiple cluster agent with process language detection enabled and remote config enabled (for rbacs).

Helm:
datadog:
  remoteConfiguration:
    enabled: true
  env:
    - name: DD_TELEMETRY_ENABLED
      value: "true"
clusterAgent:
  env:
    - name: DD_LANGUAGE_DETECTION_ENABLED
      value: "true"
  replicas: 2
  admissionController:
    enabled: true
    mutateUnlabelled: true
    configMode: hostip
    remoteInstrumentation:
      enabled: true # important

Deploy a dummy python deployment

apiVersion: apps/v1
kind: Deployment
metadata:
  name: python-sleep-app
spec:
  replicas: 4
  selector:
    matchLabels:
      app: python-sleep-app
  template:
    metadata:
      labels:
        app: python-sleep-app
    spec:
      containers:
      - name: python-sleep
        image: python:3.9
        command: ["python"]
        args: ["-c", "import time; while True: time.sleep(5)"]

The telemetry can help to know how many requests were forwarded and how many were processed.

❯ k -n datadog-agent-helm exec datadog-agent-linux-cluster-agent-6cc67c8d54-hrngs -- datadog-cluster-agent telemetry | grep -i forwarded_request
# HELP forwarded_requests Counter of requests forwarded to the dca leader
# TYPE forwarded_requests counter
forwarded_requests{handler="postDetectedLanguages"} 1
❯ k -n datadog-agent-helm exec datadog-agent-linux-cluster-agent-6cc67c8d54-hrngs -- agent status | grep -i follower
  Status: Follower, redirecting to leader at <IP_ADDRESS>
  Status: Follower, cluster agent leader is: datadog-agent-linux-cluster-agent-6cc67c8d54-s9zkg

For the leader:
❯ k -n datadog-agent-helm exec datadog-agent-linux-cluster-agent-6cc67c8d54-s9zkg -- datadog-cluster-agent telemetry | grep -i forwarded_request
Defaulted container "cluster-agent" out of: cluster-agent, init-volume (init)
❯ k -n datadog-agent-helm exec datadog-agent-linux-cluster-agent-6cc67c8d54-s9zkg -- datadog-cluster-agent telemetry | grep -i dca
Defaulted container "cluster-agent" out of: cluster-agent, init-volume (init)
# HELP language_detection_dca_handler_ok_response Tracks the number the request was processed successfully
# TYPE language_detection_dca_handler_ok_response counter
language_detection_dca_handler_ok_response 1
# HELP language_detection_dca_handler_skipped_patch Tracks the number of times a patch was skipped because no new languages are detected
# TYPE language_detection_dca_handler_skipped_patch counter
language_detection_dca_handler_skipped_patch{namespace="default",owner_kind="Deployment",owner_name="python-sleep-app"} 1

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge

/merge


docs: reorganize the structure of dev guide

Summary
Reorganize the structure of dev guide, to make the structure more clear and intuitive.
before:

after:

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

related https://github.com/web-infra-dev/rspress/issues/1083
would be fixed in rspress


Documentation for the project.

I want to use this wrapper of webrtc in my project, but I cannot understand its structure and how to begin.
Can you help me understand the project and its docs, if possible
And how to import it into my project. Any jar file?

As mentioned in the README, I think the easiest approach is to integrate this in a Maven project:
<dependency>
    <groupId>dev.onvoid.webrtc</groupId>
    <artifactId>webrtc-java</artifactId>
    <version>0.5.0</version>
</dependency>

Hi, I have seen your work it's greater and it has given me a lot of insights to web RTC.
So I have seen also your contributions to the  https://github.com/lectureStudio/lectureStudio project .they have been helpful.
I see that there is an implementation of WEBRTC you used the Janus media server. I have been going through it for me to see how you did for a client-side using JavaFX for desktop.
Please I need help to understand how to do it, your implementation is veeeery very advanced and you using injects at some points so it has been hard to understand the flow, I am using Janus the same as you. Do you or can you make a sample repo or example on how to implement this with out the complex flows or libs like injections (DI) more of an imperative flow, if you wish you can use JavaFX or swing .
I have been at this or reviewing your code for a week now, and it's not just working, the SDP offer is being rejected by Janus as invalid i have tried to play around with the media hoping to Janus will accept but still nothing yet. The wiki doesn't have much to offer as much code and a lot of people seem to also need help with this judging by the issues people are posting here.
Thank you for your time.

@devopvoid


Doesn't Support TLS 1.1/1.2 on Android 4.x

When using an Android 4.x device to connect to a web server that only supports TLS 1.1 and greater, React-Native-Fetch-Blob is unable to download files and fails with an error.
This is similar to a problem faced (and fixed) by the core React Native team (https://github.com/facebook/react-native/issues/7192).
I've patched React-Native-Fetch-Blob in a manner similar to the fix for the core project (which is pretty straightforward since both projects use OKHttp and will open a PR soon to resolve this issue.

@jhellman , thanks for the heads up. It'd be grateful if you can make a PR 👍

k - I'm ready to create a PR, but the PR template wants me to direct at branch 0.10.7 but that branch doesn't yet exist.


Check what sites does not have a favicon

Fixed in https://github.com/NoPlagiarism/ShikiFlow/commit/dabf5a042a3024e3606616982709ed0d628cc269


Mysterious space under some form field components (e.g. Select, Input, Textarea)

Description
Mesop adds some (unwanted) space beneath components like the ones mentioned in the title. The behaviour does not occur with all components; e.g. Text, Button, and Slider don't display this. This can't be overridden by setting component margins to zero.
To Reproduce
Steps to visibly reproduce the behavior:

Add a box to your page and make its borders visible
Add an Input element inside the box
See the rendered page and spot the space between the input element and the bottom border of the box

Expected behavior
No space between the Input element and the bottom border.
Screenshots

The content above was generated like this:
@me.page(path="/")
def page():
  with me.box(style=me.Style(
    border=me.Border.all(me.BorderSide(width=2, color="black", style="solid")),
    padding=me.Padding.all(0),
  )):
    me.input(
      label="Test input",
      on_blur=on_test_input_blur,
      style=me.Style(margin=me.Margin.all(0)),
    )

Desktop System Info

OS: MacOS Sonoma 14.6.1
Browser: Chrome

Additional context
I went into the generated HTML in the browser, and found that the mysterious space is a div element with class "mat-mdc-form-field-subscript-wrapper mat-mdc-form-field-bottom-align ng-tns-c1534922977-1". No idea where it comes from, though.

This looks to be part of the default angular components styling. So in some ways it works as intended.
But I agree it is annoying that we cannot override it in Mesop. The hacky way to do it is to include your own stylesheet and see if you can override it. You can specify a stylesheet to import via the page decorator.
To host the stylesheet you can either mount a Fast API server with Mesop or host the stylesheet from a separate server. Right now we do not have a way to expose a static assets directory (though I think that is doable and we do have a github issue for it somewhere).

You can use subscript_sizing parameter and set to "dynamic" - see material docs for details


Add missing dep in Actions CI

Now that the action can actually fail, I realized it's missing a dependency.

Requires https://github.com/ignition-tooling/ubuntu-bionic-ci-action/pull/13


lackey not handling non-core-windows applications properly

Hi,
I am working on an open source project to automate FTK Imager application. I have started the work and found that Lackey is not allowing the script to press ALT button in the FTK imager application. Here, the FTK is focusing up as a new window.
from lackey import *
appname=App("FTK Imager")
appname.focus()
appWindow=App.focusedWindow
key1 = Key.ALT
p1 = PlatformManagerWindows()
p1.typeKeys(key1)
**Note: I am including only the logic to press ALT key here.
I have also found that the Lackey is not even focusing the MS word or MS Excel
Could you please guide me on this?

Hi nefsal22,
The PlatformManager is an internal class, you don't need to instantiate it. To type keys, use the following code:
from lackey import *
appname = App("FTK Imager")
appname.focus()
type(Key.ALT)

I usually find it's easier to prototype with SikuliX (since it has a handy IDE) and then run the created script with Lackey. But it certainly should work either way! For more details on Sikuli script, check out their documentation.
If FTK Imager launches a Word or Excel document and it isn't focusing properly, you can try focusing the Word window the same way you focused FTK Imager:
word_doc  = App("Microsoft Word")
word_doc.focus()

Does this help?

Thanks for the reply!
In fact, my code is working perfectly with Notepad and Skype applications. But I changed it as per your instruction and still getting the same response.
FTK Imager is focusing properly but the ALT key function is not happening in the screen. I am running it through my DOS window and getting the below out.
Z:>python test.py
[action] Typing '{ALT}' with modifiers 'None'
I have also checked the same code with MS Word and MS Excel, but the same issue is there.

Hmm. A couple followup questions:

Does FTK Imager receive any keypresses from Lackey?
Are you trying to use ALT as part of a key combo (e.g., Alt+F to select the File menu)?
What do you get if you call print(appname.isRunning()) on the App instance associated with FTK Imager, MS Word, and MS Excel respectively?

Sure!

The Lackey could only able to focus the FTM Imager App. I have tried F1(which is the help menu) too. But none of the clicks are working properly.

Yes, I wanted to try ALT + F to get the options in the File menu. So that I can navigate and find the right option.

For all the app, I am getting a TRUE response for the isRunning() function. That's why I got clueless about why its not allowing to press ALT or F1 keys.

Some applications hook keyboard events at a low level, possibly ignoring simulated keypresses.  Just in case, let's try a couple more things.
First: Instead of calling the low-level PlatformManager methods directly, you can pass a KeyModifier to the type() function:
type("f", KeyModifier.ALT)

This holds the ALT key while typing "f".
Second: Note that there's a difference between type("f") and type("F"). the first types the "f" key, while the second holds shift and types the "f" key (to make it uppercase). So what your code actually sends is ALT+SHIFT+F, which might give you unexpected results in FTK Imager.
Third: If all that doesn't work, does the file menu pop up in FTK Imager when you manually key Alt+F?
Thanks!

I have already tried the keyModifier before and didn't make any difference in FTK. The FTK is not responding to the anyof the keyboard or mouse events.
And yes, the manual ALT is working with FTK. In fact all the all other shortcuts are working if I am manually doing it.(Like F1 or other ALT combinations)

What version of Windows are you running?

Windows 10.

Hmm. #49 also reports that some applications in Windows 10 are spotty in receiving events. These may be related. Let me see what I can dig up.

Hi nefsal22,
Per the discussion in #49, can you try running your script as administrator and see if FTK receives the keyboard & mouse events properly?

Awesome! It is working  when I run Command Prompt as administrator :)
So If I have to deploy my code in another system, I have to include the code to elevate the privilege right?

Yes. If FTK Imager is running with admin rights, your script will have to run with admin rights as well.


pass-in term metadata in lexicon

Try nlp("modulus").nouns().toSingular().out("text")
The result is "modulu", but actually modulus is singular and its plural is moduli
Try nlp("moduli").nouns().toSingular().out("text")
Result is "moduli" but it should be modulus.
Is there a way to specify to the engine these kind of rare/peculiar words? Like in a set of json document where each document would be structured as
{
    "word":"modulus",
    "type":"noun",
    "plural":"moduli",
    "singular":"modulus"
    // In case word was of type verb then other fields like past-tense etc will be present
}

hi Faizan, thanks! Yeah, I'll fix that case today. I agree there should be a way to pass-in irregular forms in the api, and I like that format. That could work for verb conjugation too.
let me stew on one this for a little while.

hey @leoseccia you have any opinions on this idea?

Hello,
In English, the plural form of words ending in -us is not straight forward... The general rule should be that if the singular ends in -us then the plural should be -i and all the (many) exceptions should go in data/nouns/irregular_plurals.js
Note: in some ways, the word "modulus" should actually be considered to be regular, as in it follows the main rule.
I'm not sure what would be expected for octopus, which has three plural forms: octopuses, octopi, and octopodes...
There is a whole wikipedia article about these: https://en.wikipedia.org/wiki/Plural_form_of_words_ending_in_-us
From the point of view of passing data to the API, I have been thinking about this quite a lot. In the example that @faizanahemad provided, there is quite a bit of omittable info (ie. singular is the same as the word, the plural should not be considered irregular and type is actually a tag) however the idea is a good one.
I would describe it like this instead
{"modulus": {
tags: ["Latin", "Noun"] //plural not included as it should be catered by the generic rule -us => -i
},
{"campus": {
tags: ["Latin", "Noun"],
plural: "campuses" //included here as it is irregular / an exception to the rule
},
{"campuses": {
tags: ["Latin", "Noun"],
singular: "campus" //included here as it is irregular / an exception to the rule
},
..etc..
I'm not sure whether the end in -i is a good enough sign of a noun being plural... What do you think?
On a side note, do you think there is any way to leverage on tries/dawg (or any other data structure) to make the lookup quicker on complex data type like the above? I can't think of a real way... I was toying with the idea of nested tries for a moment but my head started hurting very quickly.

man. exactly. That's exactly what I've been thinking about. craazy.
so the tries rn compress only a list of flat words. Maybe the solution involves getting them to hold an object somehow.
yeah, the applications include:

pass-in flat lexicon with a tag - {word: 'Noun'}
pass-in lexicon with meta-data - { word : { tag:'Noun', plural:'words' } }
pass-in new tags with metadata - {tags: {Conductor: {parent: 'Person' }}}

just thinking aloud:
{
  lexicon:{
    word:{
      meta : 'data'
    }
  },
  tags:{
    Conductor: {
      meta : 'data'
    }
  }
}

yeah leo, where i was going with your lexicon-as-trie idea was this:
var nlp=require('compromise')
var myLexicon={blah:'whatever}
let shareableContext = nlp.pack(myLexicon)
...
nlp('say my name say my name, when no one is around you, say baby..', shareableContext)

yes, I like what you are saying and using the same object it would also be good to be able to pass regex/pseudo regex (as in containing nlp tags) rules somehow. ho and how about tag hierarchy?

yeah. so it becomes an all-in-one custom interpreter/parser thing. Love it

And on a side note (I am really side-tracking now), I love the nlp pseudo regex tag match but it would be great if we could make it work better and in harmony with real regex stuff, like XRegExp and the way it extends the POSIX...

:+1: on this and would like to add that being able to include more metadata that would allow for custom taxonomy-like relationships (synonyms, broader terms/parents) is key.

straight-up nlp.addWords() method is working on dev,
gonna look at adding a nlp.addRegex() for v11
right now it's pretty weird to do

hey guys, this stuff is deployed on v11:
nlp.addWords()
nlp.addTags()
nlp.addRegex()
nlp.addPatterns()
nlp.addPlurals()
nlp.addConjugations()
//or just combine them in:
nlp.plugin()

please check it out,
https://github.com/nlp-compromise/compromise/wiki/Plugins
https://github.com/nlp-compromise/compromise-plugin
cheers!


zIndex thoughts?

Right now the overlay doesn't have a zIndex defined. This has caused issues for us where components with a zIndex style appearing on top of the overlay.
Do you think it might make sense to either allow optionally defining a zIndex, or use the max value for zIndex for the overlay?

Hit the same issue here. The overlay containers need class names to be able to control this. I'll look at this later today.

After some internal discussion with my team (thanks @majapw!), I think the best way to accomplish this would be to wrap the children in a div with position: relative; and z-index: 0. This will "trap" all of the child content so that the overlay will always appear on top.

@kesne That's a great fix. Since I've already got this PR open, I'll change it over to that.

Merged the PR sent by @eschaefer with release v3.1.1. Tested and it seems to work fine. I'll close this issue now.


Only mark timers to update

Timers are updated on every single packet call, but their accuracy is limited to 250ms where currentTime is updated. So just mark timers to update, and update them only once in update_timers.
This PR is part 1 of several PR's to revamp the neptun noise logic. The next PR would remove the lock from the peer.tunnel and introduce interior locks to tunnel members instead. In the hotpath, we would only be reading the session so no need to lock the entire tunnel for that. Other members of tunnel are either lockable or can be made atomic. Locking timers internally is not possible, otherwise it can lead to a deadlock as update_timers also tries to lock() the timers then.

Please add some more detail to the second part:

This is also needed in order to have a lock free tunnel, because locking timers internally is not possible, otherwise it can lead to a deadlock as update_timers also tries to lock() the timers then.

I don't fully understand this.

ID
Role
Interval
Transfer
Bitrate
Retransmissions

5
Sender
0.00-120.00 sec
5.17 GBytes
370 Mbits/sec
40,779

5
Receiver
0.00-120.00 sec
5.17 GBytes
370 Mbits/sec
N/A

7
Sender
0.00-120.00 sec
5.21 GBytes
373 Mbits/sec
42,817

7
Receiver
0.00-120.00 sec
5.21 GBytes
373 Mbits/sec
N/A

Performance test from base

ID
Role
Interval
Transfer
Bitrate
Retransmissions

5
Sender
0.00-120.00 sec
5.24 GBytes
375 Mbits/sec
43,217

5
Receiver
0.00-120.00 sec
5.24 GBytes
375 Mbits/sec
N/A

7
Sender
0.00-120.00 sec
5.08 GBytes
364 Mbits/sec
41,078

7
Receiver
0.00-120.00 sec
5.08 GBytes
364 Mbits/sec
N/A

I've done a ton of testing yesterday on my setup (two physical machines connected directly via cable). Unsurprisingly the most consistent results I get when measuring only one way UDP traffic. On multiple 6 minute runs I got an average of 684 Mbps with this branch vs 662 Mbps with the main branch. Looks like 3% improvement which is nice.

@jjanowsk  I think this amount is good enough because I would expect only a marginal gain, if any, by this change. Either way this change is needed for future improvements.

More updates from the measurements on the CI. When run in the CI on 15 minute runs of single way UDP traffic the gains are not significant. There is less then 0.5% of the difference between main and this branch, sometimes main being faster and sometimes this branch. In total I've run 4 pipelines like this.

Traffic the gains are not significant. There is less then 0.5% of the difference between main and this branch, sometimes main being faster and sometimes this branch.

In that case, maybe we can close this PR?


Wish: Upgrade react-popper to version 2

tl;dr; Please upgrade react-popper from version 1 to version 2, since popper.js is deprecated.
Details
Currently reactstrap relies on react-popper version 1:
https://github.com/reactstrap/reactstrap/blob/c6a790c7f4bdbfac0540d53896834638edf9c793/package.json#L260
react-popper version 1 itself relies on popper.js version 1:
https://github.com/popperjs/react-popper/blob/b451e968fc28b9ab59775742d82c219e92d11f20/package.json#L66
popper.js in the current form (version 1) is now deprecated. https://www.npmjs.com/package/popper.js reads:

This package has been deprecated
Author message:
You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1

You can also see this in your terminal, if you do a fresh npm install:
npm WARN deprecated<EMAIL_ADDRESS>You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1

It would be awesome, if reactstrap upgrades to react-popper version 2, so that we no longer see the deprecation warning for popper.js version 1.

@eddywashere can I take this process?

This is a breaking change.
Popper modifiers have also been changed in v2, so this would need to be adjusted as well.  It's worth noting that bootstrap 4 still uses popper v1, so I'm not sure I see the benefit to moving to v2 at this point?

Bootstrap 5 is final and uses v2. See the benefit now?

Would it be possible to backport #2104 to V8?
Even though it is a breaking change it is often not possible for projects to update from Bootstrap 4 to Bootstrap 5.
Popper 1 makes testing with JSDOM quite difficult and sometimes even impossible.

I'm going to close this as reactstrap v9 uses popper v2 with the bootstrap 5 update. I don't see too much of a benefit of backporting popper to V8 as bootstrap 4 still uses popper v1 and popper v2 has some breaking changes.


Missing UndefinedVariable error when expected

Hi all, Thanks for the help in advance.
I searched in the issue history but couldn't find anything, if it was already asked please forgive me.
I was coding and expecting a UndefinedVariable report from pylance. I was wondering if this behaviour is expected and it is intended as it is or not, and if yes could you explain to me the reasons. Thanks a lot.

Environment data

Language Server version: v2023.6.10
OS and version: Ubuntu 22.04.2 LTS
Python version (& distribution if applicable, e.g. Anaconda): 3.11.3

Code Snippet

Expected behavior
Expecting report error in both cases.
Actual behavior
When declaring in a function it is not reporting it.

Here's the text of the sample so others don't need to retype it:
def prova():
    try:
        c = 4
    except Exception as e:
        a = 4 + c + a
        return a

I'm able to repro the problem on pylance but not on pyright. Normally that would lead me to believe that it's a bug related to evaluation ordering, since pylance's semantic token feature often causes statements to be evaluated in a different order than they would otherwise be evaluated. But on initial inspection, I'm not able to repro the problem even if I change the evaluation ordering to match pylance. More investigation is needed.

Ah, the issue is that pyright is using two different diagnostic rules for these two cases. For the global scope, it's reporting the problem using the reportUndefinedVariable diagnostic rule. In pylance, this is a warning by default, so you will see a yellow squiggly. For the function case, pyright is using the reportUnboundVariable diagnostic rule. In pylace, this is disabled by default, so you won't see any diagnostic. If you set python.analysis.typeCheckingMode to "basic", you will receive errors for both cases.
The two different cases may sound inconsistent, but the Python interpreter distinguishes between "undefined" and "unbound" variables in the same manner.
a = a + 1 # NameError: name 'a' is not defined

def foo():
    a = a + 1 # UnboundLocalError: cannot access local variable 'a' where it is not associated with a value
foo()

The reason that reportUnboundVariable is disabled by default in Pylance is that it can occasionally produce false positive errors in unannotated code.
So, I think this is all working as designed. No bug to fix.


Project setup script

If we want to have the whole development of a node project doable from NoDock, we a way to init the project, maybe provide scaffolding?

Adding a workspace container should fix this... That might be a better idea.

#37 added the workspace container


[Talos] - metrics-server app didn't get installed

This issue is a:

[x] Bug Report
[ ] Enhancement suggestion
[ ] Feature request

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
@Rancher_Labs (reference)
Enhancement suggestions / feature requests
If you are requesting an enhancement to a feature already present, or a new feature, please outline below the feature you would like to see, and how it would benefit you and the wider Civo Kubernetes userbase:
N/A
Bug Reports
If the issue is a Bug Report, please detail the following:

Expected behaviour: metrics-server app get installed in Talos cluster
Encountered behaviour: metrics-server app didn't get installed
The steps to recreate the behaviour that you are reporting:

Launch a Talos cluster using web UI (make sure the metrics-server app is selected)
When the cluster is ready, check if there's any metrics-server pod in the cluster

If you have logs or screenshots, please include them below.
$ kgpo -A
NAMESPACE     NAME                                 READY   STATUS      RESTARTS   AGE
default       install-redis-zulh-test-2023-gq7vj   0/1     Completed   0          15m
default       install-traefik2-nodeport-zu-kpfs6   0/1     Error       0          15m
default       redis-0                              1/1     Running     0          14m
kube-system   civo-ccm-bd46857ff-k8q4j             1/1     Running     0          15m
kube-system   civo-csi-controller-0                4/4     Running     0          15m
kube-system   civo-csi-node-8s92p                  2/2     Running     0          14m
kube-system   civo-csi-node-jwvc4                  2/2     Running     0          14m
kube-system   civo-csi-node-tvdt2                  2/2     Running     0          14m
kube-system   coredns-67cbdd6d9d-bhgrb             1/1     Running     0          15m
kube-system   coredns-67cbdd6d9d-d8ns7             1/1     Running     0          15m
kube-system   konnectivity-agent-fsls4             1/1     Running     0          14m
kube-system   konnectivity-agent-jlptg             1/1     Running     0          14m
kube-system   konnectivity-agent-rpbdg             1/1     Running     0          14m
kube-system   kube-flannel-4krvf                   1/1     Running     0          15m
kube-system   kube-flannel-m57wj                   1/1     Running     0          15m
kube-system   kube-flannel-w52xt                   1/1     Running     0          15m
kube-system   kube-proxy-494kz                     1/1     Running     0          15m
kube-system   kube-proxy-c8drn                     1/1     Running     0          15m
kube-system   kube-proxy-rjxvj                     1/1     Running     0          15m

Just to add some details on this one I tested with version 1.27.0 and it acts like everything installed fine, but in reality no metrics-server was scheduled.

civo k8s create talosmetrics01 --region NYC1 --save --merge --nodes 1 --size g4s.kube.small --cluster-type talos --switch --wait -a civo-cluster-autoscaler --version 1.27.0
# this is the current stable image
civo k8s versions | grep -i talos
| talos-v1.2.8 | 1.25.5       | talos        | deprecated  | false   |
| talos-v1.5.0 | 1.27.0       | talos        | stable      | true    |

It might be better if this can error out in the interim for the --cluster-type talos from the cli experience or just not even make the attempt to install till a workaround can be implemented for deploying apps against talos. The current message is  The application was installed giving the impression it was successful.
civo version && date && civo k8s apps add metrics-server -c talosmetrics01
Civo CLI v1.0.83
Tue May 14 10:08:21 PM EDT 2024
The application was installed in the Kubernetes cluster talosmetrics01

 k get pods -A | grep -i metrics
~$


Parsing error '=>' expected when parsing typed snippets

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
9.12.0
What version of eslint-plugin-svelte are you using?
2.44.1
What did you do?

Welcome to SvelteKit

Visit kit.svelte.dev to read the documentation

{@render editor_button({ onclick: hello })}

{#snippet editor_button(foo: {
    onclick: () => void,
})}
  <button on:click={foo.onclick}>Click me</button>
{/snippet}

What did you expect to happen?
I expect no errors
What actually happened?
Received this error:
  12:15  error  Parsing error: '=>' expected

Link to GitHub Repo with Minimal Reproducible Example
https://github.com/bradyisom/eslint-typed-snippets
Additional comments
To repro with GitHub repo, yarn install and yarn lint

Perfect! Thank you for the quick response. I can verify that this bug is resolved with the latest version.


[automated] Update Rector Dependencies

Automated changes by create-pull-request GitHub action

:clap:


feat: add verification step of sumcheck (PROOF-913)

Rationale for this change
Adds the verification step of sumcheck.
What changes are included in this PR?

Adds a component verification for verifying sumcheck
Makes a fix to polynomial_utility to correctly compute 0-1 sum.

Are these changes tested?
Yes

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


Check failed: params_->device->RequiresRecordingAccessedTensors() == record_tensor_accesses_

The code used to run ok on both CPU or  GPU mode, but I don't know what happend , now it will fail whether cpu or gpu version
python -c "import tensorflow; print(tensorflow.version)"
F /usr/local/lib/python2.7/dist-packages/tensorflow/include/tensorflow/core/framework/op_kernel.h:1087] Check failed: params_->device->RequiresRecordingAccessedTensors() == record_tensor_accesses_

Jeff, it looks like this check was recently added. Can you add some background on why it might be triggered? Thanks!

I find the problem is associate with the user defined op(I write one auc.cc -> auc.so), which used to work.
If remove running auc op, then the script can run.

Hi, now after rebuilding auc.so on centos platform, now everything is fine, might due to mismatch of platform(auc.so was generated on another ubuntu platform before).
@jeffreyadean Close this if not an issue.

Thanks for the update! I'll close this.


How can I hide left numbers of Datepicker and open Datepicker after focused input

Thanks, what I want works fine in 0.10.0. What I want to hide is below:

https://github.com/angular-ui/bootstrap#need-help


APNs auth key production is not working

I can fire the push notification during the development using my own local devices.
But when I deploy the app to the app store and then download it. the push notification does not show up. Is there anything i missed ?
Hopefully someone will be able to guide me. thanks

Are you using certificates or auth key?
Make sure your entitlements have the push notification permission enabled.

Please comment with more information and I'll reopen this ticket.


feat(FN-972): file type validation on re upload

Introduction
We need the file type validation on the check the report page
Resolution
The file type validation was already there but we were missing the frontend logic of the upload errors on the check-the-report page.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main-stbr@543fb5e). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main-stbr   #2155   +/-   ##
===========================================
  Coverage             ?   2.18%
===========================================
  Files                ?     159
  Lines                ?    3021
  Branches             ?     475
===========================================
  Hits                 ?      66
  Misses               ?    2592
  Partials             ?     363

Flag
Coverage Δ

unit-test
2.18% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Bug]

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
Hi, I am currently working on zendesk__ticket_metrics Table in Snowflake. However, when we tried to get some of the tickets, all of these shown as zero for business metrics but for calendar date (mertics) we do have value. When we tracked back to what source (back to int_zendesk__schedule_spine) that created the date we found that we have bunch of gaps (we do not have value from 2024-07-07 to 2024-08-04 and also 2024-08-11 to 2024-2024-11-03 ) for the date so that's why the ticket created between the those periods do not have any value
<img width="1141" alt="Screen Shot 2024-08-16 at 10 45 51 AM" src="https://github.com/user-attachments/assets/c00e358c-55f7-45dd-8984-b4146d2ab03d">

Relevant error log or model output
zendesk__ticket_metrics
int_zendesk__schedule_spine

Expected behavior
The expected thing should be all the gaps need to be filled in appropriately so that the ticket that create in these date do have value
dbt Project configurations
int_zendesk__schedule_spine: Until_From to Until_End
Package versions
zendesk 0.16.0
What database are you using dbt with?
snowflake
dbt Version
////
Additional Context
No response
Are you willing to open a PR to help address this issue?

[X] Yes.
[X] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @zztkitty! Could you provide more context into this issue you're seeing? For example, do you have a schedule where you see the gap? Are there holidays in that schedule and what's the time zone?

Hi, taking this one as an example, we missed a lot of dates like:

2024-08-11 to 2024-11-03
2024-07-07 to 2024-08-04
2024-06-23 to 2024-07-07

These schedule_id is corresponding to business_hour but miss a lot of date for business hours.

These are all Eastern Time Zone

Thanks @zztkitty-- a few followups:

I noticed that every schedule period (every row) has is_holiday_week = true. Is that so, where each week has a holiday?
As for the missing weeks-- any chance those are completely holidays?

I checked it, all the missing periods are all business hours (it's impossible for the whole week to be holiday)
I mean for those is_holiday_week = true, between those date, there are a lot of missing periods that are not holiday.

@fivetran-reneeli

@fivetran-reneeli
The first picture is the date that I added 2024-06-23 to 2024-07-07 while the second picture is the date from the fivetran dbt package which are missing 2024-06-23 to 2024-07-07 and it's not a holiday week

Thanks @zztkitty! I see your issue, however am unable to reproduce it with our own data. If you could open a support ticket, this would allow us to see into your data so we can investigate this further.

Marking as blocked for the time being; let us know when you've been able to create that support ticket and we'll take a look from there!

Hi @fivetran-reneeli, I have submitted the ticket support request with ID #219240, please let me know any update after you guys take a look

Hi @zztkitty to give you an update on this one. Thank you for opening the ticket! I was able to recreate the issue with int_zendesk__schedule_spine you were seeing. I am working on identifying the cause and fix and will keep you posted.

Hi @zztkitty we were able to identify the issue that you saw in int_zendesk__schedule_spine. The fix is a bit complex since we will need to update the window logic used in that model, but we plan to address this in the next couple weeks.

Hi @zztkitty I have reworked the int_zendesk__schedule_spine that should fix the issues your seeing. I have created a test branch of this package with the updates. If you would like to install it, you can add the following code in place of the existing zendesk in your packages.yml.
- git: https://github.com/fivetran/dbt_zendesk.git
  revision: feature/historical-schedules
  warn-unpinned: false

Let me how this works for you!

I has this exact issue and this git branch worked for me. What would be the next step? Should I continue using this git branch in my package.yml going forward?

@fivetran-catfritz
Thank you for working on resolving the issue. Could you please confirm if the changes to the package were only made to the int_zendesk__schedule_spine model? Were there any updates made to its dependent sources as well?
I've been manually testing it on my end today, but I'm still seeing a lot of gaps.
Thank you!

@zztkitty Thanks for trying it out! I made changes to int_zendesk__schedule_spine and upstream as well. Here are all my changes and had made some changes just recently. I'm hoping you didn't catch the test branch during one of my mistake pushes. Would you be able to re-install the test branch, re-run and share an example of the gaps you're still seeing? You will still see gaps where there are holidays, but the larger gaps were looking closed when I tested.

Hi @fivetran-catfritz
We are unable to install it because of the version (v1.0 instance). That's why I manually test it from the source . As I manually tested this, it still show the gaps that are exactly the same as before.

@zztkitty Yes, your understanding is correct. When you run manually, how are you compiling the code? When last I tested on your data, it was looking good, but our access expired today so I didn't get to test with today's changes. I will ask customer support to re-establish access, and I'll check it out again.

@fivetran-reneeli
Thanks for confirming! When I run manually, I compile the code by executing SQL scripts directly in the Snowflake environment. I usually ensure that all dependencies are correctly set up and test the query against our current dataset.
Do you have any plans to address the gaps between holidays?
Also, could you please keep me updated once access is re-established and everything is good from your end. Let me know if there’s anything I need to do on my end to help re-enable access.
Thanks!

@zztkitty Sorry I should have mentioned--what I meant for the holiday gaps, if for example you have an entire week that is a holiday, then you will see a gap for that entire week. If you have at least 1 non-holiday in a week, then you will have schedule records for that entire week.
I will keep you updated once we get access back!

@fivetran-catfritz Yes, this would be very helpful!!!! Thank you so much for offering to send it over! I appreciate your support and assistance in resolving this.

@zztkitty I just sent the compiled model to you. Also it looks like the access request was sent this morning as well!

v0.18.0, which addresses this issue has been released. I will close this issue, but feel free to comment here with any additional feedback or questions!


神凪之杜 五月雨缀[PCSG01268]

App summary

App name: 神凪之杜 五月雨缀
App serial: PCSG01268
App version: 1.00

Vita3K summary

Version: v1.0.5
Build number: 2793
Commit hash: https://github.com/vita3k/vita3k/commit/[054a8119]
CPU backend: Dynarmic

Test environment summary

Tested by: ASUS Laptop
OS: Windows 11
CPU: AMD Ryzen 7 5800H with Radeon Graphics
GPU: RTX3050
RAM: 16 GB

Issues
Initial interface does not respond.
Screenshots

Log
Recommended labels

Intro
crash

Fix the title Kannagi no mori(神凪ノ杜 五月雨綴り)  [PCSG01268]

as of https://github.com/Vita3K/Vita3K/commit/2101d1ea3faf8bb4c01fe3b6853724125169daf8, the game now is playable (its ingame + for me since i have a low-end laptop). Also add the unity label since its a unity game.
Screenshots

Log
vita3k.log
Recommended Labels

Playable
unity


Cannot read values after I use UpdateLinkedValue

Open an xlsx file, update one or more values. If excel has functions you should use UpdateLinkedValue function. But If call UpdateLinkedValue all calls using GetCellValue returns "".
When I update excel and do not use UpdateLinkedValue values are updated and I can use excelize to read data without any issues. I tested this case.
Using Open XML SDK Tools I have found differences in all sheet xmls.
Example diff from sheet xml file:
   <c r="F13" s="53" t="str">
    <f>
     IF(WEEKDAY($E$9)=1,$E$9,"N/A")
    </f>
    <v>
     N/A
    </v>
   </c>

vs
   <c r="F13" s="53">
    <f>
     IF(WEEKDAY($E$9)=1,$E$9,"N/A")
    </f>
   </c>

or
   <c r="I13" s="54">
    <f>
     IF(H13="N/A", IF(WEEKDAY($E$9)=4,$E$9,"N/A"), H13+1)
    </f>
    <v>
     43040
    </v>
   </c>

vs
   <c r="I13" s="54">
    <f>
     IF(H13="N/A", IF(WEEKDAY($E$9)=4,$E$9,"N/A"), H13+1)
    </f>
   </c>

@xuri, relevant files sent via email.

Thanks for your issue. The function UpdateLinkedValue actually removes the <v> tags from the XML. It be used to store  the "cached results" generated by the Excel application when it is saved. After removes the <v> tags, it will be recalculated when the file is open again in Excel application. Recalculation requires Excel application. Excelize library doesn't implement the formula calculation engine currently (reference #65). So we will get a blank value using the function GetCellValue after call UpdateLinkedValue.

OK, I this case I am closing an issue as a duplicate to https://github.com/360EntSecGroup-Skylar/excelize/issues/65.

Big thank for answer.


Error in curl recipe

I'm using this recipe in Vagrant, and I'm getting this error:
apt-get -q -y install curl=7.19.7-1ubuntu1.1 returned 100, expected 0
Is it something that got deprecated?

Closing, if this is something you still need, please reopen and provide some more information.


Support GPIO for SPI Chip Selection?

Possible to support an/several GPIO for SPI Chip Selection?

This could be supported relatively easily by passing a GPIO object as a cs_gpio keyword argument in the SPI constructor. The SPI object would assert and deassert the chip select in transfer(). Inverted / non-inverted would be up to the GPIO.


Connection Refused using an Arduino MKR1000 board

First of all thank you for the really useful library!
I have tried to run the Basic.ino example to see if my firebase database and the MKR1000 were working properly, but form the serial monitor of arduino I keep receiving a "connection refused" message.
I did check my firebase host and authorization. I have tried to use the hotspot on my phone as well to change wifi, but nothing worked so far.
Any suggestion on why it refuse to work ?
(the code I'm using is just the basic.ino example with my host, auth, ssid and password)

This may be the SSL certificate issue.
Please try to update firmware in Arduino IDE, go to Tools -> WiFi101 / WiFiNINA Firmware Updater

Sorry for my late response, I couldn't try earlier.
Anyway it worked, thank you so much !!


Missing ? energy_sites/grid_import_export

Is it possible to add the following to READ and SET
https://owner-api.teslamotors.com/api/1/energy_sites/{}/grid_import_export
Payload options:
"customer_preferred_export_rule": "battery_ok"
"customer_preferred_export_rule": "pv_only"
"disallow_charge_from_grid_with_solar_installed": false
"disallow_charge_from_grid_with_solar_installed": true

Hi, @avecezar , I assume you are talking about the tools - Yes, these could be added. I often thought it would be good to have a single command line tool to do all these updates or polls. I'll flag that as a TODO and add these.
If anyone wants to take this on and combine/expand any of the set-mode or set-reserve tools, please feel free and submit a PR. I won't have time to work on it for a while.

Dear Jason,
The “tools” concept and execution worked for me 100% (I am new to python and linux).
What I would like to achieve is to stop charging at 70% if I set it this way using cron .
So battery is charging from 23:30 cron is checking it every 10min and when the battery reached 70% it will disable "grid charging" making space for solar. At 6AM it would enable “grid charging” back again
Do you know any other way to do it ?
Cezar

On 28 Jan 2024, at 19:25, Jason Cox @.***> wrote:
Hi, @avecezar https://github.com/avecezar , I assume you are talking about the tools https://github.com/jasonacox/pypowerwall/tree/main/tools - Yes, these could be added. I often thought it would be good to have a single command line tool to do all these updates or polls. I'll flag that as a TODO and add these.
If anyone wants to take this on and combine/expand any of the set-mode or set-reserve tools, please feel free and submit a PR. I won't have time to work on it for a while.
—
Reply to this email directly, view it on GitHub https://github.com/jasonacox/pypowerwall/issues/69#issuecomment-1913697007, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIYIXCL7O6OEGILDQ652YTYQ2QZHAVCNFSM6AAAAABCNITWUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGY4TOMBQG4.
You are receiving this because you were mentioned.

Yes, that would be a new tool (to turn on/off grid charging). I suspect much of the code is already in one of the existing tools, but someone needs to investigate how to poll and change the "grid charing" setting.


Update SQL parser to parse all shared tests

This parser should parse the following test cases:
https://github.com/elastic/apm/blob/master/tests/agents/json-specs/sql_signature_examples.json
https://github.com/elastic/apm/blob/master/tests/agents/json-specs/sql_token_examples.json
Refers to https://github.com/elastic/apm/issues/13

@felixbarny should we move those test cases to https://github.com/elastic/apm/tree/master/tests/agents/json-specs? I can open a PR

Whoops, that was a copy/paste error. I've updated the links.

One could argue that the token examples are an implementation detail and not strictly a requirement?

Unfortunately, our current parser fails about half of the signature tests. I'll look into adapting the parser from the Go agent.

The original Ruby implementation was also very far from living up to the examples. FWIW copying Andrew's Go implementation to Ruby was a fun challenge and the examples really helped.
https://github.com/elastic/apm-agent-ruby/pull/640
When it was done, we released it as an opt-in feature. Then after a while we flipped the defaults, making it opt-in to use the old implementation. Finally we removed the old one. Never heard anything from users which I take as a great success.
The new implementation was slower than the original but actually usable with anything but the simplest queries.


Allow PHP 8.3

Description
Allow PHP 8.3 and drop PHP 8.1
Wait for https://github.com/kreait/firebase-php/pull/847
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have made corresponding changes to the documentation

There is no need to drop support for php 8.1.
@jeromegamez

Merged with 7cd191ac9a3cf229f4c9c6073c69c3e2a536c6e7, thank you @L3o-pold!


EPROTO Error when connecting to the server using https inside docker container

Situation:
I developed a Node.js program which works fine on my system but behaves strange inside docker container!
In this program I'm trying to connect to a server using https. On my system It always connects successfully but inside docker container it just can connect first time and then exits with error.  So I'm sure this problem is related to docker but I'm new to docker and don't know how to fix it!
NOTE: Inside docker container, If it was never connecting then it made sense but now it's really strange that it can connect one time but no more.
Files:
The content of mentioned program is as below:
const express = require('express');
const debug = require('debug');
const https = require('https')
const log = debug('app::main-Interface');
const args = process.argv.slice(2);
const app = express();
const port = args[0] || process.env.port || 3000;

function sleep(toSleep){
    return new Promise((resolve, reject)=>{
        setTimeout(() => {
            resolve(true)
        }, toSleep);
    })
}

async function initializeRemotely(lengthOfOrder = 4096){
    return new Promise((resolve, reject)=>{

    https.get(`https://2ton.com.au/getprimes/random/${lengthOfOrder}`,
                (res)=>{
                    res.on('data', async (data)=>{
                        log('Data received!')
                        resolve(true);
                    })
                }
    )
    })
}

async function DEBUG(){
    let breakTime = 5000;
    while(true){
        await initializeRemotely()
        log('First operation succeed')
        await sleep(breakTime);
        breakTime *= 2;
    }
}

app.listen(port, async () => {
    log(`Server started listening on port : ${port}`);
    //schedulerPool.MSRulesWatcher(config.get('Times.schedulers'));
    DEBUG()
});

Content of ./deploy/Dockerfile:
FROM node:alpine

EXPOSE 3000
WORKDIR /interface

COPY package.json .
RUN npm install

COPY . .

Content of docker-compose.yml:
version: "3"

services:
  interface:
    image: interface
    container_name: interface
    build:
      context: .
      dockerfile: ./deploy/Dockerfile
    entrypoint: ["npm", "run", "development"]

Execution
When I run this program on my system:
$ npm run development

><EMAIL_ADDRESS>development /home/sysadmin/test
> export NODE_ENV=development; export DEBUG=app:*; node server.js

  app::main-Interface Server started listening on port : 8086 +0ms
  app::main-Interface Data received! +5s
  app::main-Interface First operation succeed +3ms
  app::main-Interface Data received! +6s
  app::main-Interface First operation succeed +1ms
  app::main-Interface Data received! +11s
  app::main-Interface First operation succeed +0ms
^C

As you can see it works normally because it connected to the server three times without any problem.
Now I run the docker container using below commands:
$ sudo docker-compose build
Building interface
Step 1/6 : FROM node:alpine
 ---> 025c3cbb849f
Step 2/6 : EXPOSE 3000
 ---> Using cache
 ---> 01c2a8f41688
Step 3/6 : WORKDIR /interface
 ---> Using cache
 ---> a8323f19dd04
Step 4/6 : COPY package.json .
 ---> Using cache
 ---> bccbb39147bd
Step 5/6 : RUN npm install
 ---> Using cache
 ---> 573aa7b45092
Step 6/6 : COPY . .
 ---> Using cache
 ---> a2f41c692811
Successfully built a2f41c692811
Successfully tagged interface:latest

$ sudo docker-compose up -d
Starting interface ... done

Then if I get logs from docker:
$ sudo docker logs interface
><EMAIL_ADDRESS>development
> export NODE_ENV=development; export DEBUG=app:*; node server.js

2022-02-26T05:58:48.550Z app::main-Interface Server started listening on port : 3000
2022-02-26T05:58:49.818Z app::main-Interface Data received!
2022-02-26T05:58:49.824Z app::main-Interface First operation succeed
node:events:498
      throw er; // Unhandled 'error' event
      ^

Error: write EPROTO 80B9D562707F0000:error:0A000152:SSL routines:final_renegotiate:unsafe legacy renegotiation disabled:../deps/openssl/openssl/ssl/statem/extensions.c:907:

    at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)
Emitted 'error' event on ClientRequest instance at:
    at TLSSocket.socketErrorListener (node:_http_client:442:9)
    at TLSSocket.emit (node:events:520:28)
    at emitErrorNT (node:internal/streams/destroy:164:8)
    at emitErrorCloseNT (node:internal/streams/destroy:129:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -71,
  code: 'EPROTO',
  syscall: 'write'
}

Node.js v17.4.0

Which indicates it succeed to connect first time but then it gets below error:
Error: write EPROTO 80B9D562707F0000:error:0A000152:SSL routines:final_renegotiate:unsafe legacy renegotiation disabled:../deps/openssl/openssl/ssl/statem/extensions.c:907

Questions:

Why this problem happens?
How can I solve it? Is it relate to using node:alpine?

System Information
I'm using a VPS(Virtual Private Server) with below properties:
$ uname -a
Linux vote 5.4.0-26-generic #30-Ubuntu SMP Mon Apr 20 16:58:30 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Requested Outputs:
Docker Version:
$ sudo docker version
Client:
 Version:           20.10.7
 API version:       1.41
 Go version:        go1.13.8
 Git commit:        20.10.7-0ubuntu5~20.04.2
 Built:             Mon Nov  1 00:34:17 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server:
 Engine:
  Version:          20.10.7
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.8
  Git commit:       20.10.7-0ubuntu5~20.04.2
  Built:            Fri Oct 22 00:45:53 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.5.5-0ubuntu3~20.04.1
  GitCommit:
 runc:
  Version:          1.0.1-0ubuntu2~20.04.1
  GitCommit:
 docker-init:
  Version:          0.19.0
  GitCommit:

Docker Info:
Client:
 Context:    default
 Debug Mode: false

Server:
 Containers: 7
  Running: 6
  Paused: 0
  Stopped: 1
 Images: 64
 Server Version: 20.10.7
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version:
 runc version:
 init version:
 Security Options:
  apparmor
  seccomp
   Profile: default
 Kernel Version: 5.4.0-26-generic
 Operating System: Ubuntu 20.04 LTS
 OSType: linux
 Architecture: x86_64
 CPUs: 4
 Total Memory: 7.775GiB
 Name: vote
 ID: PSSV:DVNJ:J67S:HEFK:U42A:XSNI:XDWQ:U3ZO:MFIS:5CDJ:LXFH:XYTO
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 Username: dayannilisani
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false
 Default Address Pools:
   Base: <IP_ADDRESS>/16, Size: 24

WARNING: No swap limit support

This is a wrong repo to report this issue in. Please, at the very least read the repo description. Ask this question in moby/moby.


Drop Node 4 support?

done, the writable stuff relied on destructuring, and I am not porting that back


Improve logging

https://github.com/spring-projects/spring-authorization-server/issues/1240

@jgrandja I made logging enhancements in some key places.

@leshalv Apologies I didn't have time to review the changes.
I'm off for the holidays and back Jan 8. I will review it the week I'm back.
Enjoy the holidays!

Thanks for the updates @leshalv. This is now merged. FYI, I added a polish commit with some minor updates.


Messages missing from logs captured for a job

Everything is logged correctly on worker but in logs captured for a job, some messages are missing.
See this diff for example: https://gist.github.com/nforro/e5bc12c0b84212535c7293da4deb2e98

May be related: #1953


Formatting of dates and times in a DRY manner

May make it easier to make changes to the format, i.e: to match the default format suggested
"YYYY-MM-DD HH:MM:SS +/-TTTT" as from https://jekyllrb.com/docs/frontmatter/
or to make these configurable values.

@jekyllbot: merge +dev


fix some greek translations

I merged it, but it's a mistake from me.
@vkarampinis we manage translations for core and the Admin plugin using Crowdin.
So before each release, we overwrite the language files and we merge the contributed translations.
Please apply the same edits on https://crowdin.com/project/grav-core so they will not be lost in the next Crowdin translations merge, let me know once you have done so I can clear this task from my list.

No problem. I just edited there too!

:+1: thanks!


python: vim.vars nested dictionary assignment not propagated to vim

nvim --version:

ben@BeniMac2020 vimspector % nvim --version
NVIM v0.4.4
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/tmp/neovim-20200808-89283-1us0ytm/neovim-0.4.4/build/config -I/tmp/neovim-20200808-89283-1us0ytm/neovim-0.4.4/src -I/usr/local/include -I/tmp/neovim-20200808-89283-1us0ytm/neovim-0.4.4/deps-build/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include -I/usr/local/opt/gettext/include -I/tmp/neovim-20200808-89283-1us0ytm/neovim-0.4.4/build/src/nvim/auto -I/tmp/neovim-20200808-89283-1us0ytm/neovim-0.4.4/build/include
Compiled by brew@Catalina

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.4.4/share/nvim"

Run :checkhealth for more info

vim -u DEFAULTS (version: ) behaves differently? YES
Operating system/version: macOS catalina
Terminal name/version: iTerm2
$TERM: xterm-256color

Steps to reproduce using nvim -u NORC

cat test.vim

py3 import vim
py3 vim.vars[ 'test' ] = {}
py3 vim.vars[ 'test' ][ 'toast' ] = 1

echo g:test.toast

vim -u NORC test.vim --cmd source test.vim

Actual behaviour
Error detected while processing /Users/ben/.vim/bundle/vimspector/nvim_python_dict.vim:
line    5:
E716: Key not present in Dictionary: toast
Press ENTER or type command to continue

Expected behaviour
1
Vim returns 1.

py3 import vim
py3 vim.vars[ 'test' ] = {}
py3 vim.vars[ 'test' ][ 'toast' ] = 1
echo g:test.toast

This can't work because the container (map/list) is serialized and copied over RPC. References (bindeval) are not supported.

What a surprise.

@puremourning contributions welcome. (I want to work on this, but soo much to do already)


update CNI plugins (1.5.0), containerd (1.7.1), slirp4netns (1.3.1)

Great Job. 👍👍👍
As a volunteer developer, if there is an issue ( dump the CNI version with help wanted label) .
It's a chance for me to contribute :-)

Great Job. 👍👍👍
As a volunteer developer, if there is an issue ( dump the CNI version with help wanted label) . It's a chance like me to contribute :-)

Thanks, feel free to open PRs if you find something out of date.
We also welcome PRs to automate these updates 🙏


Building mobile apps fails

Hi there,
I have latest saltcorn installed on debian 11. When I try to build android or ios package I receive this error

spawn saltcorn ENOENT
Error: spawn saltcorn ENOENT
at Process.ChildProcess._handle.onexit (internal/child_process.js:277:19)
at onErrorNT (internal/child_process.js:472:16)
at processTicksAndRejections (internal/process/task_queues.js:82:21)

What do I need to build mobile packages

Please see https://github.com/saltcorn/saltcorn/issues/1343

Please see #1343

Thank you. I will try it.

Hi, can you run a "saltcorn info" and send me the "cliPath" output?
And if it is a systemctl service the output from
sudo systemctl cat  saltcorn.service

Hi, can you run a "saltcorn info" and send me the "cliPath" output?
And if it is a systemctl service the output from
sudo systemctl cat saltcorn.service

Hi Christian,
Here it is:
cliPath: /home/marius/.local/lib/node_modules/@saltcorn/cli/src/commands
and
# /lib/systemd/system/saltcorn.service
[Unit]
Description=saltcorn
Documentation=https://saltcorn.com
After=network.target

[Service]
Type=notify
WatchdogSec=5
User=marius
WorkingDirectory=/home/marius
ExecStart=/home/marius/.local/bin/saltcorn serve -p 8080
Restart=always
Environment="NODE_ENV=production"

[Install]
WantedBy=multi-user.target

I will also like to mention that I tried to build the docker image on my own following the instructions from #1343 but it didn't work, although it seems that the image was build successfully. This is the error I got when trying to build android app from the backend:
spawn saltcorn ENOENT
Error: spawn saltcorn ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:277:19)
    at onErrorNT (internal/child_process.js:472:16)
    at processTicksAndRejections (internal/process/task_queues.js:82:21)


Encryption button display bug

As you can see, a serious encryption button display bug took place.

I just created a new blog folder and find it normal. That's quite strange.
So, it seems that there is a problem in config update process.

Note some other bugs of the encryption feature.

When switching to system editor, the content won't be decrypted automatically.
This problem also exists when switching back to builtin editor.
In the site, after decrypting the content, it won't be rendered as markdown. It just displays as plain text.

All fixed in aeb62b5.
If you find any other bugs, you can reopen this issue.

When an article is pinned to the top of the article list, the problem comes up again.


Add run-in-container make target

The run-in-container make target will execute the given ARGS="" in a
special development container. This enables some make targets and commadns,
like running container builds on outdated or otherwise strange environments.
This is not working as a development environment replacement as things
like permissions (chown), network access and kind need to be sorted out
seperatly.
Signed-off-by: Nico Schieder<EMAIL_ADDRESS>

/lgtm


Negate unsupported warning caused by appimaged

On login, the above notification appears, which is coming from appimaged as the OS is "not supported" (even though it is). This warning is superfluous and confusing to new users, so we need to stop it occuring.
This error is currently not disable-able. See:

https://github.com/probonopd/go-appimage/issues/178
https://github.com/probonopd/go-appimage/issues/133

Since this will become a major annoyance, removing the autostart file until this issue can be resolved.


[FALSE-POSITIVE]

Domains or links
https://immigrationsolutionusa.com/
Please review the above domain, as we believe it to be a false positive.
More information
We discovered that our website was listed here through a cybersecurity audit. We have not been hacked, but we have received a few attack attempts. The status is marked as "undefined," but our domain appears as a phishing site.
The website has not been hacked.
Our domain has been incorrectly marked as phishing, but we are not listed on Phishtank or OpenPhish to our knowledge.
Have you requested removal from other sources?
Not applicable. This is the first time we have been flagged as phishing, and we have not requested removal from any other sources, as we are not listed on Phishtank or OpenPhish.
Additional context
Our website has never been compromised or involved in phishing activities. We have implemented strong security measures to protect against hacking attempts, and so far, there have been no security breaches. We believe that our listing as a phishing site is a mistake, likely a false positive, and we request a review for removal from the blacklist. We are committed to maintaining the highest security standards and can provide further details if necessary.

please update your url or delete cookies. sorry for the inconvenience.

please update your url or delete cookies. sorry for the inconvenience.

Cookies 🍪 I don't do cookies, I have diabetic 😉
And now you tries to fingerprint my browser, for what? collecting PII data? without my explicit approval?
Logger output

+5
||cloudfront.net^
--
immigrationsolutionusa.com
3
get
image
https://d1rozh26tys225.cloudfront.net/loader.svg

+5
||cloudfront.net^
--
immigrationsolutionusa.com
3
get
image
https://d1rozh26tys225.cloudfront.net/robot-suspicion.svg

+5
||cloudfront.net^
--
immigrationsolutionusa.com
3
get
image
https://d1rozh26tys225.cloudfront.net/loader.svg

+5
||cloudfront.net^
--
immigrationsolutionusa.com
3
get
image
https://d1rozh26tys225.cloudfront.net/robot-suspicion.svg

Maybe this is why you are marked for phishing, as you are phishing for PII on behalf on crimeflair!! Just saying, this is for sure degrading the trust in your domain, and since the domain ain't running properly secured by SSL and the fact the domain ain't public available, but only available from cloudflare's walled garden intranet, I can't help you, as I can't access the closed circle.
You'll have to wait for @mitchellkrogza or @funilrys to pick up this issue. Opening a PR or Issue in https://github.com/mitchellkrogza/Phishing would make any different, as it is @mitchellkrogza or @funilrys and me who hold the merge power.

How can I solve this problem?
I would appreciate if you can help me, we really are an honest company that we have suffered attacks, I am the person in charge, we have fought denouncing pages that create with our brand and image and are dedicated to defraud our customers. It turns out that in recent weeks Google has shown us this.
I would be very grateful if you can help me, if this work also has a value, I would also be willing to assume it.

you need to hardened the Nginx setup

Get rid of WordPress, it is to commonly used, and therefore a obviously target, with a well known base of exploits

Ditch any BigTech (Alphabet (Google), Meta, Cloudflare etc), they serves you no good, at all. They degrade the trust in your server (domain)

Make sure you Nginx is configured correctly ⏫ getting SSL domains with no valid certs...  Here is a fresh screenshot...

As a beginner package for Nginx, somewhat, security, you can implant @mitchellkrogza Nginx bad bots, but be picky, he have left out some hosts, as they serves him a personal purpose.

Make your own deny configuration for Nginx to block unwanted hosts, IE, I block CloudFlare and Alphabet + some others in a script like this
if ($http_user_agent ~ "(heritrix|Googlebot|Google-Extended|Googlebot-Image|Googlebot-News|Googlebot-Video|Googlebot-Image|Storebot-Google|Google-InspectionTool|GoogleOther|Google-Extended|APIs-Google|AdsBot-Google-Mobile|AdsBot-Google|Mediapartners-Google|FeedFetcher-Google|GoogleProducer|google-speakr|Google-Site-Verification|ChatGPT|ChatGPT-User|CCBot)/"){return 444;}

This can be done in other ways to, this is only meant as an example

Hide your server application + version server: nginx

Enable listen 443 quic; in nginx

Add also add_header Alt-Svc 'h3=":443"; ma=86400'; to the nginx.conf

delete <link rel="preconnect" href="https://fonts.gstatic.com" .* and any other third party crap, if you need some scripts, host them inside your own domain, it is usual only a few (m)bytes anyway,

https://fonts.gstatic.com/s/montserrat/v26/JTUFjIg1_i6t8kCHKm459Wx7xQYXK0vOoz6jq6R8WXZ0oA.woff host you fonts your self, no reason to sell all your visitors to Alphabet, and it is violating GDPR and the Californian Privacy act, as you haven's asked for permission to share your visitors data with third party (See ad 10), This do not only concerns cookies...
If you have issues, figuring out how to import and intergrate the fonts, at least use a privacy aware domain: https://googledonts.private.coffee/

Again <link rel='dns-prefetch' href='//js.hs-scripts.com' /> Host it your self... no reason to loos the fight before it have begun...

Your Caching doesn't seems to work... do you have any visitors to keep it hot?

I do not notice the use of add_header Strict-Transport-Security "max-age=63072000" always; & ssl_stapling on; ssl_stapling_verify on;

Disconnect everything from cloudflare.

If you want more... Then I'll suggest you feeds the Marsupilami (@spirillen)

This explanation is much more than I expected! Thank you so much for taking the time and trouble to answer my questions, I appreciate having found you. It would be interesting to know the possibility of contracting these solutions with you. I want my work to be good, unfortunately I don't know how to develop, I make a living with WordPress, and I don't want to look bad to my bosses, I like what I do and I want to be better! I want to learn, but right now I have this problem. I would be willing to pay out of my pocket if necessary, as long as I can solve it and nothing happens to the company, since it's my responsibility.

Do you think we can get in touch?
I would like to please give me your price to hire and develop these solutions for the domain to be perfect!. in the url of donation that you pass me, it is like a subscription. and I do not know how much it would cost this work on your part.

Not listed anymore. Closing.


FlightReview: need a way to edit or delete uploaded flight reports

I just discovered that my description for upload #6 is very wrong. It turns out the the parameter file load did not work correctly, and I wasn't even running the right attitude estimator.
Simplest option is probably just to delete that flight report.

Moved this here: https://github.com/PX4/flight_review/issues/10


feat: Update 941160.ra

Added events from PortSwigger XSS CheatSheet
https://portswigger.net/web-security/cross-site-scripting/cheat-sheet

@Xhoenix There have been some problems this afternoon which broke all recent PRs. The other PRs were okay after merging the fixed v4.0/dev branch into the PRs, but for some reason that has not fixed this PR.

@Xhoenix please run crs-toolchain regex update -a to update the rules from the assembly files.

@RedXanadu @theseion fixed

For the record, this is an update to a keyword list that was supposed to be automated in #2701 based on a known source for the keywords. Somehow this was skipped, maybe because the overhaul was very early.
Either way, we will need to get this done sooner or later.
The PR by @Xhoenix is very welcome.

@Xhoenix I made the fix but messed up the force push. I had to open a new PR with your changes: #3315.


Add support for graphql-codegen plugin level config (resolves #692)

This ensures knip finds references to graphql-codegen plugins with plugin-level configuration, and can detect these as used, unused or unlisted dependencies.

Thanks @CensoredUser! Short 'n sweet 👌


Honor Content-Type in .headers files.

This change is

Current coverage is 83.72% (diff: 100%)

Merging #103 into master will decrease coverage by <.01%

@@             master       #103   diff @@
==========================================
  Files            25         25
  Lines          2029       2028     -1
  Methods           0          0
  Messages          0          0
  Branches        281        282     +1
==========================================
- Hits           1699       1698     -1
  Misses          219        219
  Partials        111        111

Powered by Codecov. Last update 4432c5f...a5fbea7


Receiving custom props with payload.

Instead of going through form reducers and adding/removing validations that could be passed as props e.g.
<Field ... required/> or passing a match pattern...
It would be easier to have a set of predefined validators that would execute if a properties exists since some of them do the same thing. This could enhance recommended approach and DRY up code.

Are you on the most recent version of redux-form? The current Field API lets you pass a validate function as a prop which can do synchronous validation of that field. If you want to reuse validators across fields or compose them, it isn't too hard to do that with some mapping and reducing.


Lazy {}-style logging everywhere

Part of the logging overhaul (see issue #1044), especially in relation with issue #916. The diff is long and mostly boring. I nearly always respected pep8 (including the 80 columns part).

beets.logging is a drop-in replacement for logging, the loggers it creates (though getLogger()) use {}-style formatting. beets/logging.py is therefore the most interesting file of the diff. It has basic tests in test/test_logging.py.
all other files import beets.logging instead of logging, and have switched from eager to lazy formatting.
a format string error has been fixed in beetsplug.duplicates.
a logging error has been fixed in beets.library, fixing 4 tests.

The next step will be using a different logger for each plugin (and maybe for each module of the core) and cleaning some format strings (that prepend the log message with their name).

Wow! This is awesome. Let's merge this posthaste; I'm really excited to have lazy log formatting back.
Travis reports that something is wrong on 2.6. Unless it's an easy fix (haven't looked closely, but looks like it's still trying to use % formatting for some reason?), maybe we should call this a good reason to drop 2.6 support.
Now that we have a logging replacement, I'd love to do some stuff to fix up Unicode pitfalls too (see note in #1198). I'll do that next.

The 2.6 fix was a bit hacky (but is well-commented). The current (logging-related) drawback of supporting 2.6 will be the inability to do "{}".format("bar"): one has to do {0}.format("bar").
I'll now move the from beets import logging statements. I initially changed them by a simple sed, I'm afraid automating this one will be a bit harder!

This looks great; I'm glad there were some evident 2.6 fixes. I plan on merging this soon (later today, I hope?) after pushing out a release.

Okay, this is in! I'm also going to add you as a repo collaborator so you can make fixes directly. (Feel free to keep opening PRs for larger stuff, of course.)

That's great! Thank you.


Possible to use local package in development and CDN in production?

The Get Started instructions give two installation options:

Package Manager
CDN (no bundler)

Is there any way to use the Package Manager for local (offline) development and the CDN for other (production) environments?

@jghaines yes, it's possible, but I don't recommend doing this nowadays.
It will improve load performance due missing tree-shaking and a fact that CDN cache is not shared anymore (https://www.jefftk.com/p/shared-cache-is-going-away). It also could create hard to catch bugs.

However, if you using Webpack it should be pretty simple to do. Modify:
index.ejs or index.html to include CDN build
<%if (__PROD__) {%>
<script src="CDN_URL.js"></script>
<%}%>

webpack.config.js
module.exports = {
  // ---
  externals: {
    ...(process.env.NODE_ENV === "production" && {
      "semantic-ui-react": "semanticUIReact"
    })
  }
  // ---
};


Build on Windows

Maybe remove directories older than 7 days so prevent its overfilling? Something like this (AI generated, untested):
# Specify the directory you want to clean up
$targetDirectory = "C:\"
$mask = "gh*"

# Specify the age threshold (e.g., 7 days)
$ageThreshold = 7

# Get all directories in the target directory
$directories = Get-ChildItem -Path $targetDirectory -Directory -Filter $mask

# Filter directories older than the threshold and delete them
$directories | Where-Object { $_.LastWriteTime -lt (Get-Date).AddDays(-$ageThreshold) } | Remove-Item -Recurse -Force

Maybe remove directories older than 7 days so prevent its overfilling?

Yes, or better use ephemeral runners, which we planning soon.


Finalize v1.0.0 Release with Workflow Enhancements and Code Refactoring

Purpose
Finalizing the release of version 1.0.0 by incorporating necessary changes such as adding GitHub workflows, renaming the module directory, and applying code refactoring to align with the release requirements.
Goals
The primary objective is to ensure that the project is ready for the v1.0.0 release, with workflows in place to streamline automation and a codebase optimized for deployment.

 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.1 out of 2 committers have signed the CLA.:white_check_mark: Ajai-Suvendran:x: imesh94You have signed the CLA already but the status is still pending? Let us recheck it.


Add com.snowplowanalytics.snowplow.storage/redshift_config/jsonschema/2-1-0

This schema contains nullable tunnelConfig object with SSH tunnel configuration https://github.com/snowplow/snowplow-rdb-loader/issues/22

Renamed ticket to reflect that this is revision bump, not addition (due password change)


Update JsonApiResponseTrait.php

Fixed some error responses that were not working.

+1 on this, similar issue with resource not found exception as well, see #26

PR ignored so closing for personal GitHub maintenance.


[modify] able to load config without default

Coverage: 82.926%. Remained the same when pulling 7ea50cb74eb69e5f5c693a456483c9a2baea2705 on pr-config-without-default into 796a52f9cd58240fce7dd1046cd70023dd25a3d6 on master.


failed to execute command gulp on samples/angular_contacts

I am having error when running gulp command. Look like the pre-requisite is to install the sp command?
[10:58:22] Error in plugin 'gulp-shell'
Message:
    Command `sp push -mainHtmlFile release/index.html` failed with exit code 127
Error running task sequence: { [Error: Command `sp push -mainHtmlFile release/index.html` failed with exit code 127]
  message: 'Command `sp push -mainHtmlFile release/index.html` failed with exit code 127',
  showStack: false,
  showProperties: true,
  plugin: 'gulp-shell',
  __safety: { toString: [Function] } }

Hi,
Yes, that's correct. The default gulp task defined in gulpfile.js invokes sp push so that the application is pushed to the server.  The Script Portlet command line client support must be installed on your workstation for this functionality
This article has more information about this set of samples: https://developer.ibm.com/digexp/docs/docs/script-portlet/using-node-js-build-tools-with-script-portlet/


'Apparel' misspelled on CC Store

Description
The word 'apparel' is misspelled on the CC Store.
Reproduction

Visit the CC Store.
See error.

Expectation
The correct spelling 'apparel' should be used. It is actually used in the subsequent button.
Screenshots

Resolution

[ ] I would be interested in resolving this bug.

Hi @dhruvkb  can I work on this?

Sure @pranavvp10, please do

Hi @dhruvkb  actually I'm a starter to open source. I couldn't find the html file (../about/store)

@pranavvp10 I don't know how the site is put together either. Maybe @TimidRobot knows and can help you out.

Thanks - done!


wrong trackingCode field Value

It becomes an incorrect string on SetFulfillmentInformation
{"carrier":"DHL","trackingCode":"["00340434287034152770"]"}
Result is ErrorCode 400. Wrong Fieldvalue on Field: trackingCode
The Problem is the Double Encoding in HasParameters at addParameter:
After deactivate json_encode
{"carrier":"DHL","trackingCode":["00340434287034152770"]}

Thank you for reporting. The problem has been fixed.


setContent removing data attributes

If I do something like...
 editor.setContents('
TEXT

');
I get the p and the span and its class, but no data attributes.

Can you confirm if this is a bug or am I doing something wrong?
Thanks

@sdraper69 You must be define the "attributesWhitelist" option.
attributesWhitelist: {
 all: "data-propA|data-propB"
}

Great !! Sorry i missed that in the docs

@sdraper69 You must be define the "attributesWhitelist" option.
attributesWhitelist: {
 all: "data-propA|data-propB"
}

Hi Your work is best out there --- after experimenting with every big and famous editor in last one month, Only your editor looks promising for my project ---- My project has big future prospect ---- so before i bet on it , help me in following aspect --- and be honest ---
1.  Are you serious about your this editor? because i see less frequent update and most of discussion , issue are unresolved on git.
2. What are your future plan about this editor. Are you going to maintain and develop it seriously?
3.  Have you designed your editor for editing or rendering/publishing also? Can i render 100 Suneditor instance on a page for public user to read content? Do you recomment this? Or i need to just save html from Suneditor to database and then render it separately in some other component?
4. Although this is best editor i have seen, Why this is not getting more attention and starts?

@roker15 Sorry, for the late reply.

Yeah sure,
Previously, it was updated very frequently.
However, lately I've been too busy to have time to update the editor.
Future plan is to release v3.0.0 version.
As you can see from the previous commit history, I do my best to update.
Many instances can be rendered, but not recommended.
It will be improved to be able to render all in one instance in "3.0.0" version.
It still lacks in many areas.
You need to pay more attention to the design or the details.


Improve docs

We should add more documentation to this project. First, public headers should be fully documented. Then, we should update the instructions in the README, and potentially have more pages with explanations.
TODO:

[x] include/kernel
[x] include/libc
[x] README

I refactored the docs infra a bit (see #454).

It isn't perfect yet but it is better than before.


Section 508 502.3.1: (devenv/custom '') An element must not have the same Name and LocalizedControlType as its parent.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:
#A11yMAS; #A11yTCS; #Win32; #DesktopApp; #Feature-ML.NETModelBuilder-Win32-Nov2020; #A11ySev2; #Benchmark; #Visual Studio; #A11yAuto; #WCAG4.2.1;
Environment Details:
Microsoft Visual Studio 2019 preview Version 16.8.0 Preview 6.0
Repro Steps:
1.Open Visual Studio Enterprise 2019 Preview
2. Create a new .Net Core Console App (C#)
3. Tab to "Solution Explorer" and right click on the project (Shift + F10) and then from the expanded list Select "Add" and then from the list items select "Machine learning"
4.Tab till "Text Classification" scenario and select it
5.tab till "next" button and press enter
6.Now run accessibility on the screen
The following accessibility issue needs investigation.
App:
devenv
Element path:
custom ''
Issue Details:
An element must not have the same Name and LocalizedControlType as its parent. Section 508 502.3.1
How To Fix:
Provide unique names for controls that have a parent/child relationship and the same ControlType property.
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.

Fixed by previous release

Fixed by previous release

Verified the bug in the latest build and the issue doesn't re-pro.
Hence closing the bug
Please find the screenshot below for reference
Microsoft Visual Studio Enterprise 2019 Preview
Version 16.9.0 Preview 4.0 [30917.4.main]

Github tag: #Closed;