text:
NetBird fails to forward traffic from Oracle Cloud VM to internal peer, works fine with Tailscale

Describe the problem
I'm facing an issue where NetBird does not properly forward traffic from an Oracle Cloud VM to a peer in my internal network. I followed the exact same steps with Tailscale, and everything works fine under the same conditions. However, with NetBird, the traffic does not reach its destination.
I have an Oracle Cloud VM running Ubuntu (Ampere ARM64, 4 cores, 24 GB RAM) with a static public IP. On this VM, I installed NetBird from scratch, logged into my cloud account, and have a policy that allows all peers to connect to each other.
In my homelab, I have an LXC container running uptimekuma service on port 3001. The internal NetBird IP of this peer is 100.88.192.94. If I run a curl command from the Oracle VM, I can retrieve the HTML response from the NGINX service, so the internal connection between peers is working.
The issue arises when I try to forward the traffic coming to the VM on port 80 to the internal NetBird peer. I configured traffic forwarding using iptables, but it doesn't seem to work with NetBird, whereas with Tailscale, the traffic is correctly forwarded without issues.
To Reproduce
Steps to reproduce the behavior:

Create an Oracle Cloud VM (Ampere ARM64, Ubuntu minimal).
Install NetBird and set up a peer in the internal network (in this case, an LXC container running an NGINX service).
On the Oracle VM, enable traffic forwarding and configure iptables to forward traffic on port 80 to the NetBird peer on port 3001.
Try to access the web service through the Oracle Cloud public IP.

Expected behavior
I expected the incoming traffic on port 80 of the Oracle Cloud VM to be correctly forwarded to the internal NetBird peer, as it happens with Tailscale under the same conditions.
Are you using NetBird Cloud?
Yes, I am using NetBird Cloud.
NetBird version
0.28.7
NetBird status -dA output
NetBird status -dA output
ubuntu@oracle:~$ netbird status -dA output
Peers detail:
iphone-fede.netbird.cloud:
NetBird IP: 100.88.5.103
Public key: WKZknsTPHEZQREt75Kmaz/HV2jwcNehnpt0S91PKH1c=
Status: Connected
-- detail --
Connection type: P2P
Direct: true
ICE candidate (Local/Remote): srflx/srflx
ICE candidate endpoints (Local/Remote): 198.51.100.0:51820/198.51.100.1:57557
Last connection update: 22 minutes, 50 seconds ago
Last WireGuard handshake: 1 minute, 48 seconds ago
Transfer status (received/sent) 3.8 KiB/1.6 KiB
Quantum resistance: false
Routes: -
Latency: 22.578577ms
oracle.netbird.cloud:
NetBird IP: 100.88.61.146
Public key: PGT03R1EDb1cAf+uZymgvGYFXKHnnOP3/0ccKPInfBI=
Status: Disconnected
-- detail --
Connection type:
Direct: false
ICE candidate (Local/Remote): -/-
ICE candidate endpoints (Local/Remote): -/-
Last connection update: -
Last WireGuard handshake: -
Transfer status (received/sent) 0 B/0 B
Quantum resistance: false
Routes: -
Latency: 0s
4da8810ea346.netbird.cloud:
NetBird IP: 100.88.110.131
Public key: +AawwZqEzKesGbPISiXgeU0yfTfmtkGKXGs0v7U152s=
Status: Disconnected
-- detail --
Connection type:
Direct: false
ICE candidate (Local/Remote): -/-
ICE candidate endpoints (Local/Remote): -/-
Last connection update: -
Last WireGuard handshake: -
Transfer status (received/sent) 0 B/0 B
Quantum resistance: false
Routes: -
Latency: 0s
macbook-fede.netbird.cloud:
NetBird IP: 100.88.180.38
Public key: bQZuQnpveGtcU45nr7DTJtlWbhqi6O7rj/UwkNcB1iA=
Status: Connected
-- detail --
Connection type: P2P
Direct: true
ICE candidate (Local/Remote): srflx/srflx
ICE candidate endpoints (Local/Remote): 198.51.100.0:51820/198.51.100.1:57431
Last connection update: 22 minutes, 53 seconds ago
Last WireGuard handshake: 2 minutes, 3 seconds ago
Transfer status (received/sent) 3.6 KiB/1.2 KiB
Quantum resistance: false
Routes: -
Latency: 14.533849ms
uptimekuma.netbird.cloud:
NetBird IP: 100.88.192.94
Public key: hA3DLsvCah9Gz6YeWWWRGEBtCusBhadDopDOom0a2Qs=
Status: Connected
-- detail --
Connection type: P2P
Direct: true
ICE candidate (Local/Remote): srflx/srflx
ICE candidate endpoints (Local/Remote): 198.51.100.0:51820/198.51.100.1:58213
Last connection update: 22 minutes, 53 seconds ago
Last WireGuard handshake: 2 minutes, 25 seconds ago
Transfer status (received/sent) 3.0 KiB/2.1 KiB
Quantum resistance: false
Routes: -
Latency: 20.015482ms
nginx.netbird.cloud:
NetBird IP: 100.88.211.126
Public key: 9kpbqwyghDtjVPX92Ds0EF054+RzOMMw8/+efConO2I=
Status: Disconnected
-- detail --
Connection type: P2P
Direct: false
ICE candidate (Local/Remote): srflx/srflx
ICE candidate endpoints (Local/Remote): 198.51.100.0:51820/198.51.100.1:57104
Last connection update: -
Last WireGuard handshake: 1 minute, 48 seconds ago
Transfer status (received/sent) 3.8 KiB/1.3 KiB
Quantum resistance: false
Routes: -
Latency: 0s
pihole.netbird.cloud:
NetBird IP: 100.88.212.19
Public key: gmSdBrNEI2j5fNwAWHRGyYhMAJUjTizb2HVqHEhq20Y=
Status: Connected
-- detail --
Connection type: P2P
Direct: true
ICE candidate (Local/Remote): srflx/srflx
ICE candidate endpoints (Local/Remote): 198.51.100.0:51820/198.51.100.1:57104
Last connection update: 22 minutes, 53 seconds ago
Last WireGuard handshake: 1 minute, 48 seconds ago
Transfer status (received/sent) 3.8 KiB/1.3 KiB
Quantum resistance: false
Routes: -
Latency: 9.531658ms
OS: linux/arm64
Daemon version: 0.28.7
CLI version: 0.28.7
Management: Connected to https://api.netbird.io:443
Signal: Connected to https://signal.netbird.io:443
Relays:
[stun:stun.netbird.io:5555] is Available
[turns:turn.netbird.io:443?transport=tcp] is Available
Nameservers:
[100.88.212.19:53] for [local.anon-are26.domain] is Available
FQDN: oracle-1.netbird.cloud
NetBird IP: 100.88.223.175/16
Interface type: Kernel
Quantum resistance: false
Routes: -
Peers count: 4/7 Connected
Do you face any client issues on desktop?
No desktop client is involved, only the Oracle VM client and the peer in my homelab (LXC).
Screenshots

Additional context
Here are the commands I used to configure port forwarding:

enable port forwarding
sudo sed -i 's/^#net.ipv4.ip_forward=1/net.ipv4.ip_forward=1/' /etc/sysctl.conf
sudo sysctl -p

add iptables rules
sudo iptables -t nat -A PREROUTING -p tcp --dport 80 -j DNAT --to-destination 100.88.192.94:3001
sudo iptables -t nat -A POSTROUTING -j MASQUERADE

I have ensured that port 80 is open and accessible from the Oracle Cloud dashboard. As I mentioned before, this exact setup works fine with Tailscale, which makes me think this could either be a bug in NetBird or a misconfiguration on my side.

Hi @fedeiglesias,
could you test if this build fixes the issue? https://github.com/netbirdio/netbird/actions/runs/10419607061/artifacts/1836477728
You can replace /usr/bin/netbird with the new binary

I'm having thr same issue. As soon as the peer is connected, i cannot access the web server hosted on it via the public IP

Same problem here Netbird but working with Tailscale on same setup (Oracle Cloud VM and a VM on my local network)

authors: fedeiglesias ghaisasadvait lixmal raegedoc

repo: netbirdio/netbird


text:
fix: xauthority file needs to be available in x11

this restores the check that I removed yesterday, and correctly passes down the xauthority cookie to the container, I tested this and it is finally working on my system 😅
@hobeone I'm guessing you are running wayland?

who needs apt-get, yum, npm, or all the other package managers when we clearly can just use containers instead :rofl:
We should consider creating our own OS based upon a container os (i.e coreos)  and then simply have all the things in containers :laughing:

source: gharchive/pull-request

repo: netbrain/zwift

license: Unlicense

license_type: permissive

license_source: github-api


text:
deploy netdata using the nodiscc.xsrv.monitoring_netdata ansible role

fixes https://gitlab.com/nodiscc/xsrv/-/issues/1
README.md: fix typo, whitespace

/cc @odyslam

@ilyam8 ok to merge?

@nodiscc sorry for the delay. @Ferroin and @maneamarius will take a look (I hope) soon.

As far as remember, it was decided that we will not offer the community automation solutions. We had a meeting. Why we talking about it again?

If the pr might help some future users why would we not merge it?

Going back to that discussion, the reasons for not doing it are roughly the following.
This way we sort of take responsibility for this code. We would be more comfortable giving links to the repositories of those who developed the relevant playbook/role.
This particular role is (from memory) well written, but its usefulness is extremely limited by the fact that it only works with
Debian 11. Since there are no organizations that use a single version of a single operating system, more generic roles will be preferred.
Also, Ansible is not the only automation tool.
I would prefer to see this role in Ansible Galaxy. And we will, of course, point out its existence in the documentation.

Thanks @nodiscc !

Hi @andrewm4894 @iigorkarpov

This way we sort of take responsibility for this code. We would be more comfortable giving links to the repositories of those who developed the relevant playbook/role.

I assumed (see https://gitlab.com/nodiscc/xsrv/-/issues/1) this repository was for listing community-supported projects, which implies netdata does not officially support/take responsibility for the content.
If you feel more comfortable only linking to external projects I suggest converting the repo to an "awesome"-style list (see https://gitea.com/gitea/awesome-gitea for example) which links to community projects or articles on netdata. I can move the article to a Github gist or similar then we can simply link to it from here (then we should also move the other ansible howto).
All things considered I think the "awesome list" format would be more suitable for this repo.

This particular role is (from memory) well written,

Thanks :ok_hand: I try to keep it clean

its usefulness is extremely limited by the fact that it only works with Debian 11. Since there are no organizations that use a single version of a single operating system, more generic roles will be preferred.

It should work with Ubuntu 18/20 with minor modifications, which would be maintainable for me.
But you are right that it will not support other distributions/RHEL family because it is out of scope for me. It has other limitations, it only supports the APT/.deb installation method, not all netdata settings are configurable through it... and that is by design. Config management is about standardizing environments and having too many config toggles defeats this purpose (and as the number of configuration variables increases, so does the number of non-working configurations).
If you see another use case for which it could be adapted, please let me know (for example I recently added the possibility to configure streaming in a simplified way).

Ansible is not the only automation tool

This repo could point to other deployment methods/examples/howtos... I think the description/structure should be clarified in this way.

I would prefer to see this role in Ansible Galaxy. And we will, of course, point out its existence in the documentation.

It's there: https://galaxy.ansible.com/nodiscc/xsrv (as a collection):
$ ansible-galaxy collection install nodiscc.xsrv

But I suggest installing from git instead:
# $ cat requirements.yml
collections:
  - name: https://gitlab.com/nodiscc/xsrv.git # from git
    type: git
    version: release
# $ ansible-galaxy collection install --force -r requirements.yml

repo: netdata/community

license: MIT


text:
老Demo项目，怎么升级duilib库

NetEase IM Demo
NetEase Education Demo

这几个项目，好像还在用比较老的duilib版本。
想知道如何能升级到最新的duilib。
我自己把duilib目录和duilib_d.lib拷到相应的目录，编译是报错的。
感觉这个升级不太好升，因为样式xml中属性，跟最新的duilib格式也不太一样了

升级中改动较大的就是字体相关的属性，需要在 XML 中批量替换已经写好的字体标签、个别 UI 组件中修改设置字体的代码。
如果你着急使用，请先尝试自己替换编译逐步解决错误，无法自己解决的错误可以提供详细的错误内容一起来排查。
如果你不是很着急升级 Duilib，还请耐心等待我们后续的 Demo 更新，我们会逐步替换到 Demo 中的 Duilib 库。

感谢！

source: gharchive/issue

authors: nmgwddj zywj


text:
implement Idd.eval and add tests for remaining 4 reductions

No need to merge this, just thought it'd be a good way for you to take a look at what I've been up to!

LGTM!

repo: netkat-lang/idd


text:
use consistent hostname for live tunnel URL

- Do you want to request a feature or report a bug?
feature.
- What is the current behavior?
Currently, a random suffix is appended to your netlify live hostname at each start of the tunnel, meaning the URL you'd share changes every time you use it even against an identical codebase.
Proposed solution is to either:

create a flag to not do this
or just not do it automagically.

@calavera mentioned some related things:

related code is here:  https://github.com/netlify/bitballoon/blob/master/app/models/site.rb#L1745 (sorry non-Netlify folks, you won't be able to see that)
we'll need to handle session cleanup differently if we change the naming conventions.
slack convo on the topic: https://netlify.slack.com/archives/CBC2U9MMG/p1565384911057700

This would also be awesome for testing OAuth flows, as you are usually required to configure the allowed domains ahead of time with the provider, and some (ahem Google) you can't provide localhost

I am building something that integrates with two different services using webhooks, and every time I start netlify dev --live I have to update the URLs in two places. One of them can be set via a curl request so I'm pulling the live URL out of stdout and setting it but the other can't be done that way so I have to copy paste it every time. I'm also using Auth0 and have to set the URL there every time as well.
Having a static URL would be amazing.

As a workaround, I made the following change locally to generate an env file.
+++ node_modules/netlify-cli/src/commands/dev/index.js
@@ -2,2 +2,3 @@
 const path = require('path')
 const process = require('process')
+const fs = require('fs')
@@ -273,4 +274,5 @@
     process.env.URL = url
     process.env.DEPLOY_URL = url
+    fs.writeFileSync('.netlify/.env.live', "URL=" + url + "\nDEPLOY_URL=" + url)

     printBanner({ url, log })

The generated file, .netlify/.env.live, looks like:
URL=https://scollup-309a73.netlify.live
DEPLOY_URL=https://scollup-309a73.netlify.live

Of course, this change is not permanent and will disappear on the next netlify-cli update.  I'll likely create fork and submit a PR at some point, if people want it.

As a workaround, I made the following change locally to generate an env file.
+++ node_modules/netlify-cli/src/commands/dev/index.js
@@ -2,2 +2,3 @@
 const path = require('path')
 const process = require('process')
+const fs = require('fs')
@@ -273,4 +274,5 @@
     process.env.URL = url
     process.env.DEPLOY_URL = url
+    fs.writeFileSync('.netlify/.env.live', "URL=" + url + "\nDEPLOY_URL=" + url)

     printBanner({ url, log })

The generated file, .netlify/.env.live, looks like:
URL=https://vandalay-309a73.netlify.live
DEPLOY_URL=https://vandalay-309a73.netlify.live

Of course, this change is not permanent and will disappear on the next netlify-cli update.  I'll likely create a fork and submit a PR at some point, if people want it.

For what it's worth, you can create a development page in your site and have that call a function with the current domain in it. That function could do the webhooks updating part by calling the appropriate api's.
Dev page:
---
title: Development tools
layout: layouts/html.njk
permalink: "dev/"
---
<form action="/.netlify/functions/update-webhooks" method="post">
    <input name="domain" size="100" id="update-webhooks" type="text" />
    <button>Update webhooks</button>
</form>
<script>
    document.querySelector('#update-webhooks').value = `${window.location.protocol}//${window.location.host}`
</script>

update-webhooks.ts:
import type { Handler, HandlerContext, HandlerEvent, HandlerResponse } from '@netlify/functions';

export const handler: Handler = async (event: HandlerEvent, context: HandlerContext): Promise<HandlerResponse> => {
    const domain = unescape(event.body).split('=')[1];

    //Update your webhooks here

    return {
        statusCode: 200
    };
};

I refined my workaround (see above) so I don't need to patch the plugin.  I put this in my project.  I run this instead of netlify dev.
#!/usr/bin/env node

// netlify dev

const fs = require('fs')
const cli = require('netlify-cli')

process.argv.push('dev')
cli.run().then(() => {
  // export the live URL
  fs.writeFileSync(
    '.netlify/.env.live',
    'URL=' + process.env.URL + '\nDEPLOY_URL=' + process.env.DEPLOY_URL
  )
})

The .netlify/.env.live file won't be updated until after compiling is complete.  You can modify the then() above if you want to do any post-processing with javascript APIs (e.g. auth0, stripe, etc)
Or here's a bash script you could use that does some post-processing
#!/bin/bash
set -eu   # strict mode

# Launch netlify dev in background, and wait for file update.  (My script above.)
rm .netlify/.env.live
./netlifydev.js &
while ! [[ -f .netlify/.env.live ]]; do sleep 1; done

# Load results
source .netlify/.env.live

# Put any other commands here to publish your URL to wherever
# This is an example setting stripe webhook.  First, delete existing webhook
stripe webhook_endpoints list | jq '.data[].id' -r | xargs -r stripe stripe webhook_endpoints delete
# add new hook
stripe webhook_endpoints create --url="$DEPLOY_URL"

# block on netlify dev
wait -n

any update here? currently still needing to use ngrok alongside netlify dev

This is now available with CLI v15.4.0.
When you use Netlify Dev with the --live flag, a random URL will be generated and used for all live sessions started in that machine.
Additionally, if you choose to specify your own subdomain with --live=<subdomain>, we'll generate a custom URL like https://<subdomaiin>--<site name>.netlify.live.
If anything isn't working as expected, please let us know on this issue and we'll reopen.
I know it took us a while to get this one out — thanks for your patience, and I hope you find it useful!

authors: eduardoboucas fool gamebox isaac-martin mikeslattery redmagic terrarum

repo: netlify/cli


text:
Posts not fully loaded when using the back button

Describe the bug
When you visit a post and press the back button, it doesn't load all of the posts that were previously there.
To Reproduce

Go to list of posts
Click on a post
Click the back button in the browser
See error

Expected behavior
Expecting to see all the posts loaded
Screenshots

Applicable Versions:

Netlify CMS version: 2.1.1
Git provider: GitLab
OS: macOS
Browser version chrome 68.0.3440.106

@Benaiah pagination thang?

@erquhart my guess is that this is most likely another instance of the react waypoint we use to load more posts on scroll not firing its visibility event despite the fact that it's visible. Perhaps we should add a "load more" button that manually triggers pagination so users can manually fire it when the waypoint fails.
What I think is happening:

The "Doc" collection is being filtered somehow so that each page of API results is only loading a couple entries.
The react waypoint frequently fails to fire its visibility event if you load a page that  doesn't have enough entries to trigger scrolling.
This doesn't normally occur, since most collections load enough entries to trigger scrolling.

There's a variety of ways we can work around the bug, but the best way forward is likely to decouple API pagination from UI pagination, fix the instances where waypoint isn't indicating we need more entries, and add a manual fallback to load more entries just in case. The pagination already needs some UI work to indicate that it's loading more pages, so the manual trigger could probably be as simple as clicking the loading indicator.

I found a similar bug:

The problem here is that react-waypoint has already been mounted once, and this instance is reused across collections.
react-waypoint trigger the onEnter, when initially visible, only at mount.

Sounds like we need a PR that repositions the waypoint so that it's remounted when a new collection is selected.

@erquhart  I'm experiencing the same issue.
I tried passing a child to the Waypoint component, and this change triggered the collection to re-fetch entries like the first time it's opened.
Could we utilize this to show e.g. a loading indicator in the Waypoint component? I'm not sure whether that's considered a hack or not, but it seams to be a feature of Waypoint component and it does seem to have the desired behavior.
On a different, but somewhat related, note - are the collections supposed to be re-fetched when moving between them in the UI (like the gif above)? It makes the UI jitter a little bit.

Not sure why it's completely refetching when you pass Waypoint a child, but we shouldn't need to do that. I'd expect to render a new Waypoint when we render a collection, whether the collection was previously navigated to or not. Superfluous fetching should be avoided at a higher level - and I don't see any in the gif above, it only shows fresh fetching (with the loading indicator) when a collection is first loaded. No spinner when navigating back to a collection that was previously loaded.

..., it only shows fresh fetching (with the loading indicator) when a collection is first loaded. No spinner
when navigating back to a collection that was previously loaded.

Maybe I'm mistaken, but this is what I see in the gif above:

Reload page
Select the Pages collection
Go back to the Translations collection
Observe that all previously fetched translations are visible for ~0.5s before they get re-fetched (and then we encounter the bug reported in this issue).

@LoicMahieu  we now have a PR to fix your related issue.
@erquhart I believe this issue is still related to unnecessary loadEntries calls,  which replaces the already loaded paginated id entries. Here is another gif that duplicates the issue.

Notice after hitting the editor screen and back to the collection list page, the already loaded paginated entries are gone and you would need to scroll down for them to load again. PR  #1881 and #1964 should  fixed the issue.

Merged both, will release soon.

authors: Benaiah LoicMahieu barthc erquhart mryanb sondrele

repo: netlify/netlify-cms


text:
feat(editor): add field option when to be able to show fields conditionally

Summary
Add field option when to be able to show fields conditionally.
It could be useful to conditionally show/hide some fields depending on the data. This PR allows to specify a new option when which could be:

string: use of conscript.js for simple language for writing potentially-complex sets of conditions.
function: write own condition directly in javascript.

Test plan
WIP...

Deploy preview for netlify-cms-www ready!
Built with commit 56a10581d1664eb9887be64241635d06593bcc9b
https://deploy-preview-1894--netlify-cms-www.netlify.com

Deploy preview for cms-demo ready!
Built with commit 3b182b1246d1755fa451a5617a706b8785b805b6
https://deploy-preview-1894--cms-demo.netlify.com

I found conscript by change. It seems pretty new and so not battle tested. I think it is promising.
Anyway, in my use-case, I don't use this syntax but only the function style. In fact, I configure only by JS, see #1866

Yeah - I'm not saying it isn't functional, but I'd like to avoid bringing outside specs into our API. For example, Immutable.js is a part of our API right now, meaning folks need to understand how that library works to do any customization in the CMS, and that's a decision I regret (planning to remove it in an upcoming major release).
Totally agree on the goal of this PR, we just need to find a different approach.

Closing as stale due to lack of activity and/or interest. Please comment if you feel this should be reopened.

authors: LoicMahieu erquhart verythorough


text:
Broken builds due to netlify/plugin-lighthouse internal error.

Describe the bug
Broken builds with the following error:
Plugin "@netlify/plugin-lighthouse" internal error
Could not import plugin:
Error [ERR_REQUIRE_ESM]: require() of ES Module /opt/build/repo/src/.netlify/plugins/node_modules/lighthouse-logger/index.js from /opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/gather/fetcher.js not supported.
Instead change the require of index.js in /opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/gather/fetcher.js to a dynamic import() which is available in all CommonJS modules.
While loading "@netlify/plugin-lighthouse" from Netlify app at Object. (/opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/gather/fetcher.js:18:13) at Object. (/opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/gather/driver.js:8:17) at Object. (/opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/runner.js:9:16) at Object. (/opt/build/repo/src/.netlify/plugins/node_modules/lighthouse/lighthouse-core/index.js:8:16) at async Promise.all (index 0) at async importLogic (file:///opt/buildhome/node-deps/node_modules/@netlify/build/lib/plugins/child/logic.js:25:29) at async getLogic (file:///opt/buildhome/node-deps/node_modules/@netlify/build/lib/plugins/child/logic.js:10:19) at async Object.load (file:///opt/buildhome/node-deps/node_modules/@netlify/build/lib/plugins/child/load.js:12:19) at async handleEvent (file:///opt/buildhome/node-deps/node_modules/@netlify/build/lib/plugins/child/main.js:28:42) at async process. (file:///opt/buildhome/node-deps/node_modules/@netlify/build/lib/plugins/ipc.js:91:24)
Error properties: { code: 'ERR_REQUIRE_ESM' }
To Reproduce
Steps to reproduce the behavior:

Deploy Netlify site
Build error occurs
Deploy fails

Expected behavior
A working deploy and lighthouse scoring of the build.
This may be related to a release by the Chrome team of lighthouse-logger: https://github.com/GoogleChrome/lighthouse/issues/15085
https://www.npmjs.com/package/lighthouse-logger

Thanks for the detailed report and for jumping into the Lighthouse issues too! I've done some testing this morning and all seems ok again. Along with your comment, I think we can close this ☺

authors: hardyjohnson jackbrewer

repo: netlify/netlify-plugin-lighthouse


text:
initial congestion window configuration

Motivation:
Some environments, with a high bandwidth, may require an increased congestion window upon start
Modifications:
Quiche config method for setting initial congestion window is exposed
Result:
Initial congestion window may be increased optionally

@poisonriver can you sign our icla and let me know once done ? https://netty.io/s/icla ?

@poisonriver can you sign our icla and let me know once done ? https://netty.io/s/icla ?

@normanmaurer just signed

@poisonriver thanks a lot!

@poisonriver thanks a lot!

@normanmaurer if I have some other ideas where should throw them? the discussions board looks dead...

@poisonriver thanks a lot!

@normanmaurer if I have some other ideas where should I throw them in? the discussions board looks dead...

You can create new issues or just use the board

authors: normanmaurer poisonriver

repo: netty/netty-incubator-codec-quic


text:
Direct memory leakage，when the connection is closed, data written to direct memory cannot be released

Direct memory leakage, slow network transmission, large write data, and when the connection is closed, data written to direct memory cannot be released

Netty version
4.1.107.Final
JVM version (e.g. java -version)
1.8.0_191
OS version (centos6)

repo: netty/netty


text:
Allow to create Unsafe ByteBuf implementations that not use a Cleaner…

… to clean the native memory.
Motivation:
Using the Cleaner to release the native memory has a few drawbacks:

Cleaner.clean() uses static synchronized internally which means it can be a performance bottleneck
It put more load on the GC

Modifications:
Add new buffer implementations that can be enabled with a system flag as optimizations. In this case no Cleaner is used at all and the user must ensure everything is always released.
Result:
Less performance impact by direct buffers when need to be allocated and released.

@nmittler @trustin @Scottmitch PTAL ... Also @snazy please check as this may be what you want for cassandra :)

From a brief look, this looks good 👍

@normanmaurer LGTM

@snazy PTAL again buddy.

+1

I've tried this patch and it works really great!
I'd be +1 on making this the default (instead of requiring -Dio.netty.noDirectBufferNoCleaner=false).
Nice side effect seems to be, that memory allocation (including "free") is faster.

Some suggestions/wishes:

can you make the pool size configurable via a separate system property and just fall back to JVM config?
having (static) methods to query the current and maximum size of the pool would be useful to expose some metrics
introduce a new exception to be thrown instead of OutOfMemoryError that clarifies that there's not enough memory from Netty's direct memory pool. This is different from a "normal" OutOfMemoryError because it (usually?/in my case?) affects the current operation/request but not the whole system.

@snazy like an OutOfMemoryError subclass?

Am 29.05.2016 um 20:42 schrieb Robert Stupp notifications@github.com:
Some suggestions/wishes:
can you make the pool size configurable via a separate system property and just fall back to JVM config?
having (static) methods to query the current and maximum size of the pool would be useful to expose some metrics
introduce a new exception to be thrown instead of OutOfMemoryError that clarifies that there's not enough memory from Netty's direct memory pool. This is different from a "normal" OutOfMemoryError because it (usually?/in my case?) affects the current operation/request but not the whole system.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@trustin wdyt about this change and also about making it default?

Impact of this change for Apache Cassandra:
https://issues.apache.org/jira/plugins/servlet/mobile#issue/CASSANDRA-11818/comment/15306030
Quite some improvement ;)

+1 for making this default.

@snazy can you explain why you would need an extra system property ?

Re: system-property to configure direct-memory-pool size: The memory pool managed by this patch is different from the one that's managed by the Java. Giving Java's direct memory pool 2g would mean that the application could allocate 4g in total (Java + Netty), which might be an unexpected behavior. There might be also valid reasons to configure both pools differently.

@snazy I see your point but still not sure if the other is any better as this may also confuse the user.

@trustin @nmittler ^^ WDYT ?

@normanmaurer I don't feel terribly strongly here. Does it hurt to add an extra property? On the other hand, it may be unexpected behavior if the user adjusts the existing property and it doesn't work in this case. I can see issues either way, I guess.

I think @snazy's suggestion makes sense. An alternative to io.netty.noDirectBufferNoCleaner could be something like io.netty.maxDirectMemory which behaves as follows:

<0 - Don't use cleaner, and inherit max direct memory from java. In this case the "practical max direct memory" would be 2*max memory as currently defined.
0 - Use cleaner, Netty will not enforce max memory, and instead will defer to JDK.
>0 - Don't use cleaner. This will limit Netty's total direct memory (note: that jdks' memory is independent of this).

Sounds like a good idea as well @snazy wdyt?

Am 01.06.2016 um 22:13 schrieb Scott Mitchell notifications@github.com:
I think @snazy's suggestion makes sense. An alternative to io.netty.noDirectBufferNoCleaner could be something like io.netty.maxDirectMemory which behaves as follows:
<0 - Don't use cleaner, and inherit max direct memory from java. In this case the "practical max direct memory" would be 2*max memory as currently defined.
0 - Use cleaner, Netty will not enforce max memory, and instead will defer to JDK.

0 - Don't use cleaner. This will limit Netty's total direct memory (note: that jdks' memory is independent of this).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

+1 on this

great work @normanmaurer ... lgtm!

Cherry-picked into 4.1 (3d29bcfc8ddeda50e0bff0234d0c5240f404ddb5) and 4.0 (62655c00a93a3c4c9e421fabbf5991c19dfb4161)

authors: Scottmitch nmittler normanmaurer snazy trustin


text:
Fixed|SimpleChannelPool.close() should only return after complete.

Motivation:
We need to ensure we only return from close() after all work is done as otherwise we may close the EventExecutor before we dispatched everything.
Modifications:
Correctly wait on operations to complete before return.
Result:
Fixes https://github.com/netty/netty/issues/7901.

@trustin ping


text:
Documentation update to MessageSizeEstimator.java

Motivation:
Did not understand the context of "ca".
Modification:
Clarified "CA" to "approximately".
Result:
Fixes #9031

@netty-bot test this please

@Mr00Anderson please fix the check style error:
12:01:02 [ERROR] /code/transport/src/main/java/io/netty/channel/MessageSizeEstimator.java:19: Line is longer than 120 characters (found 127). [LineLength]
12:01:02 Audit done.

That is what i get for using the web editor.

@netty-bot test this please


text:
[question] How messages are distributed across cluster of services?

Do you have any kind of paper on clustering of eventuate services?
Thx for your time.

We are using Hazelcast to ensure each message will be processed once and only once.

Ahhh, that is exactly what I was looking for :-)))

The pre-condition would be all the services must be idempotent.

authors: archenroot stevehu


text:
Artifacts are not created with new actions on Kind

Build: https://github.com/networkservicemesh/integration-k8s-kind/actions/runs/8155999145/job/22292695463?pr=986
Action update: https://github.com/networkservicemesh/integration-k8s-kind/pull/975/files

Seems like done.

authors: VitalyGushin denis-tingaikin


text:
jdiff not ble to catch reference key in dict of dicts

Environment

Python version:  3.7
jdiff version:  0.0.2

Expected Behavior
As it stands now, jdiff can catch a reference key only in a list of dictionary. There are case where NAPALM returns dict of dicts. In the example below, we want .local. and .local..0to be the ref key ofis_up`
{
  ".local.": {
    "description": "",
    "is_enabled": true,
    "is_up": true,
    "last_flapped": -1,
    "mac_address": "Unspecified",
    "mtu": 0,
    "speed": -1
  },
  ".local..0": {
    "description": "",
    "is_enabled": true,
    "is_up": true,
    "last_flapped": -1,
    "mac_address": "Unspecified",
    "mtu": 0,
    "speed": -1
  },
}

@pszulczewski FYI

I believe this could be a limitation from JMSPATH. The most similar case I found is this one:https://jmespath.org/tutorial.html#object-projections
As we can see, the our hypothetical reference key is lost using *.  I will drop a message on jmspath channel support but I feel that the user will have to do some post parsing using indexes

Looks like there is keys(@) which catches all dict keys. So in jdiff the path should be something like keys(@).is_up and should work. @pszulczewski FYI

keys(@) produce list of keys, similar to python dict.keys(), they don't have access to values.
Thus keys(@).is_up or $keys(@)$.is_up return None.

At this point we have 3 options:

Let the user use index to create a mapping between key and value got from jdiff - this will not work out of the box with pre defined checks
Make jdiff dynamically discovery the data type and act accordingly - for this case the expression would be something like $*$.is_up
Create some sort of flag

As discussed with @pszulczewski we will support $*$.is_up and interpolate the list of keys with the list of values returned.

repo: networktocode/jdiff


text:
generated serializer does not support deserialize.

I'm getting the error below, however, with other serializers there's no problem.
generated serializer for DbDescriptor does not support deserialize.
at Utf8Json.Formatters.ThinServerDatabase_Model_DbDescriptorFormatter1.Deserialize(JsonReader& , IJsonFormatterResolver )
at Utf8Json.JsonSerializer.Deserialize[T](Byte[] bytes, Int32 offset, IJsonFormatterResolver resolver)
at BinarySerializer.Serializers.UTF8.Deserialize[T](String jSon) in I:\OneDrive\ThinDatabase\BinarySerializer\Serializers\UTF8.cs:line 13
at BinarySerializer.Serializers.SerailizerFileExtenstions.ObjectStringDeSerializerETX[T](ISerializer serializer, Stream fileSource, List`1 filePostions) in I:\OneDrive\ThinDatabase\BinarySerializer\Serializers\SerailizerFileExtenstions.cs:line 254

at Utf8Json.Formatters.ThinServerDatabase_Model_DbDescriptorFormatter1.Deserialize(JsonReader& , IJsonFormatterResolver )
at Utf8Json.JsonSerializer.Deserialize[T](Byte[] bytes, Int32 offset, IJsonFormatterResolver resolver)
at BinarySerializer.Serializers.UTF8.Deserialize[T](String jSon) in I:\OneDrive\ThinDatabase\BinarySerializer\Serializers\UTF8.cs:line 13
at BinarySerializer.Serializers.SerailizerFileExtenstions.ObjectStringDeSerializerETX[T](ISerializer serializer, Stream fileSource, List`1 filePostions) in I:\OneDrive\ThinDatabase\BinarySerializer\Serializers\SerailizerFileExtenstions.cs:line 254
[Serializable]
public class IndexEntry
{
[DataMember]
public Guid ID { set; get; }
[DataMember]
public DbAddress Address { set; get; }
[DataMember]
public String ObjectType { set; get; }
    [IgnoreDataMember]
    public long FilePosition {
        set
        {
            FilePointer = BitConverter.GetBytes(value);
        }
        get
        {
            return BitConverter.ToInt64(FilePointer, 0);
        }
    }

    [DataMember]
    public byte[] FilePointer { set; get; }

    public IndexEntry(Guid id, DbAddress addr, String objType, long filePos)
    {
        ID = id;
        Address = addr;
        ObjectType = objType;
        FilePosition = filePos;
    }
}`

`

IndexEntry class need a parameterless constructor to be deserialized

authors: Boomx frankinstien


text:
7 drop food with click

Added a sprite sheet with various fruit
Currently only uses 1 fruit
Drops fruit with left click

You can check out the PR, but I noticed I need to setup the gpg signing for this git account still. Also some issues with the integration test is happening.

repo: neuragene/neuragene


text:
Upload data tar to tag and download upon integration install

This PR adds a path for automated maintenance and pulldown of integration-specific data as tag assets.
The full lifecycle:
Data upload

Trigger upon push to a release branch, only if data files are modified (currently, only yolov5 data files are checked)
Determine the appropriate tag to upload to (e.g. push to release/0.12 - > tag = v.0.12.0)
Check if the tag already exists. If it doesn't, skip the steps below
Create a tar.gz file of the data files (right now only targets yolov5 data)
Push the asset to the tag

Data download

Upon installation of integration (again, only yolov5 supported), download data archive
Uncompress it into the sparseml install directory

Notes:

Data will only be stored on and downloaded from major.minor releases. Patches will not have data assets attached to them
This PR includes a minor comment change to one of the data files, so that when it's cherry picked to the release branch, it will trigger the data upload

Closing this PR, as there is no good way to test it prior to release. Reverting to including yolov5 data files in the wheel installable.

repo: neuralmagic/sparseml


text:
Chat doesn't work

Chat doesn't work
{   "error": {     "message": "Нет доступных каналов для модели default в текущей группе gpt-4",     "type": "one_api_error"   } }
`

`

Also.

repo: neurogen-dev/NeuroAPI


text:
Event throw exception when the assembly version is updated.

Describe the bug
When I update an assembly who defined an event if a change the assembly version the code will throw an exception System.IO.FileNotFoundException: Could not load file or assembly 'assembly, Version=x.x.x, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified..
And your assembly is now at version y.y.y
Expected behavior
Do not try to load a specific assembly version, but only try to load the assembly.
Screenshots

 Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HMLALECPS4R7", Request id "0HMLALECPS4R7:0000009A": An unhandled exception was thrown by the application.
      System.InvalidOperationException: The exception handler configured on ExceptionHandlerOptions produced a 404 status response. This InvalidOperationException containing the original exception was thrown since this is often due to a misconfigured ExceptionHandlingPath. If the exception handler is expected to return 404 status responses then set AllowStatusCode404Response to true.
       ---> System.IO.FileNotFoundException: Could not load file or assembly 'Souche.Domain, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

      File name: 'Souche.Domain, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null'
         at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, StackCrawlMarkHandle stackMark, ObjectHandleOnStack assemblyLoadContext, ObjectHandleOnStack type, ObjectHandleOnStack keepalive)
         at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, StackCrawlMark& stackMark, AssemblyLoadContext assemblyLoadContext)
         at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, StackCrawlMark& stackMark)
         at System.Type.GetType(String typeName, Boolean throwOnError)
         at Neuroglia.Data.EventSourcing.ESEventStore.UnwrapsStoredEventAsync(ResolvedEvent resolvedEvent, CancellationToken cancellationToken)
         at Neuroglia.Data.EventSourcing.ESEventStore.ReadEventsForwardAsync(String streamId, Int64 offset, Int64 length, CancellationToken cancellationToken)
         at Neuroglia.Data.EventSourcing.ESEventStore.ReadAllEventsForwardAsync(String streamId, CancellationToken cancellationToken)
         at Neuroglia.Data.EventSourcing.Services.EventSourcingRepository`2.FindAsync(TKey key, CancellationToken cancellationToken)
         at Souche.Application.Commands.Projects.CreateProjectCommandHandler.HandleAsync(CreateProjectCommand command, CancellationToken cancellationToken) in /src/src/Souche.Application/Commands/Projects/CreateProjectCommand.cs:line 184
         at Neuroglia.Mediation.FluentValidationMiddleware`2.HandleAsync(TRequest request, RequestHandlerDelegate`1 next, CancellationToken cancellationToken)
         at Neuroglia.Mediation.DomainExceptionHandlingMiddleware`2.HandleAsync(TRequest request, RequestHandlerDelegate`1 next, CancellationToken cancellationToken)
         at Neuroglia.Mediation.RequestPipeline`2.HandleAsync(IRequest`1 request, IServiceProvider serviceProvider, CancellationToken cancellationToken)
         at Neuroglia.Mediation.Mediator.ExecuteAsync[TResult](IRequest`1 request, CancellationToken cancellationToken)
         at Souche.Api.Controllers.ProjectsController.Create(CreateProjectCommandDto command) in /src/src/Souche.Api/Controllers/ProjetsController.cs:line 54
         at Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.TaskOfIActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeActionMethodAsync>g__Awaited|12_0(ControllerActionInvoker invoker, ValueTask`1 actionResultValueTask)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeNextActionFilterAsync>g__Awaited|10_0(ControllerActionInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Rethrow(ActionExecutedContextSealed context)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeInnerFilterAsync()
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeNextResourceFilter>g__Awaited|25_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Rethrow(ResourceExecutedContextSealed context)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.InvokeFilterPipelineAsync()
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Logged|17_1(ResourceInvoker invoker)
         at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Logged|17_1(ResourceInvoker invoker)
         at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
         at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
         at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
         at Microsoft.AspNetCore.Authorization.Policy.AuthorizationMiddlewareResultHandler.HandleAsync(RequestDelegate next, HttpContext context, AuthorizationPolicy policy, PolicyAuthorizationResult authorizeResult)
         at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
         at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
         at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)
         --- End of inner exception stack trace ---
         at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.HandleException(HttpContext context, ExceptionDispatchInfo edi)
         at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)
         at Microsoft.AspNetCore.ResponseCompression.ResponseCompressionMiddleware.InvokeCore(HttpContext context)
         at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)

@cdavernas

Due to the way the ESEventStore references serialized events

See at https://github.com/neuroglia-io/framework/blob/main/src/Data/Neuroglia.Data.EventSourcing.EventStore/Services/ESEventStore.cs#L395

The assembly version update is probably not the cause of this issue

Closed as stale

repo: neuroglia-io/framework


text:
Known tissue parameters for Models in documentation?

I was wondering today if we should start creating tables of known tissue values for all of the Models to help new users with simulating data or for validation when fitting their first sets of data? All values could be sourced from published work, which would be referenced in the tables.
If so, would the documentation section be an appropriate location to list those values?

We can put this info in model files header to be sure to fill this info for future models and to have access to it in the qMRLab help (Gabriel is currently parsing the header).
We can provide values for WM and GM in the brain.
Shall we look in the literature or do we use values extracted from the example datasets?

authors: mathieuboudreau tanguyduval

repo: neuropoly/qMRLab

license: mit

license_source: bigquery


text:
bash code should be in a %%bash cell to be run-able

@tsalo , I see that you made the bash code for downloading the data part of a markdown cell in the tutorial notebook. Did you have a particular reason? If we want it to be run-able, I think it needs to be a code cell.
The %%bash on the top line of the cell makes the cell able to execute bash code (instead of the python code, which is the default).

I added some instructions for copying to a runnable cell, but I wanted to make sure that folks wouldn't accidentally run the cell in Binder. I figured Markdown + instructions would be the best compromise.

I see, sorry, I didn't read the instructions carefully. Sounds good!
Just curious - why would it be a problem in binder? I tried to make it so it would only download the data if it detected that it wasn't already there. But I wasn't able to test it in binder yet.

I saw the if statement, so I don't think it would be a problem to run the cell in Binder, but I feel like I would have to explain the code a bit if we were running it, and since it's not necessary for the Binder-based tutorial...
That said, I will probably reuse this notebook for Neurohackademy, and I don't know if we'll use Binder for that, so I may switch it back to a code cell after tomorrow.

Cool, okay, then I'll let you close this now or when you've decided what to do.

authors: koudyk tsalo


text:
getting error during initial install 'The super role could not be found. Please perform a complete install.'

The query is not executing successfully.
Also, we need to update the commands to match the new schema.
For example the user table has fewer columns.

move the install into the hooks. We have a new method for executing a hook on a single module.

repo: neutrinog/atomar


text:
xorgxrdp segfaults Xorg on reconnect: OsLookupColor+0x139

I experienced two crashes within the past month when trying to reconnect to an Xrdp session that had been running for many days/weeks, with many successful reconnects. One day however, upon reconnect, my remote desktop client showed a black screen. I wouldn't be able to say what was different in the failed reconnect attempts (in contrast to all the successful ones before). After closing the client window and another connection retry, a fresh desktop is spawned, and all open apps from the previous session are lost.
The relevant ~/.xorgxrdp.?.log log file contains the following error:
[2168320.699] rdpRROutputGetProperty:
[2168320.726] (EE)
[2168320.726] (EE) Backtrace:
[2168320.728] (EE) 0: /usr/lib/xorg/Xorg (OsLookupColor+0x139) [0x559c10550459]
[2168320.735] (EE) 1: /lib/x86_64-linux-gnu/libpthread.so.0 (funlockfile+0x50) [0x7fbff07358df]
[2168320.743] (EE) 2: /usr/lib/xorg/modules/libxorgxrdp.so (a8r8g8b8_to_a8b8g8r8_box+0xd7) [0x7fbff3281da7]
[2168320.746] (EE) 3: /usr/lib/xorg/modules/libxorgxrdp.so (rdpCapture+0xb7b) [0x7fbff3282c8b]
[2168320.749] (EE) 4: /usr/lib/xorg/modules/libxorgxrdp.so (rdpClientConEndUpdate+0x145) [0x7fbff327fbd5]
[2168320.752] (EE) 5: /usr/lib/xorg/modules/libxorgxrdp.so (rdpClientConGetScreenImageRect+0x30c) [0x7fbff32815dc]
[2168320.752] (EE) 6: /usr/lib/xorg/Xorg (TimerSet+0x180) [0x559c10549d50]
[2168320.752] (EE) 7: /usr/lib/xorg/Xorg (TimerSet+0x1e7) [0x559c10549e07]
[2168320.752] (EE) 8: /usr/lib/xorg/Xorg (WaitForSomething+0x277) [0x559c10549ef7]
[2168320.752] (EE) 9: /usr/lib/xorg/Xorg (SendErrorToClient+0x113) [0x559c103f3243]
[2168320.752] (EE) 10: /usr/lib/xorg/Xorg (InitFonts+0x3c8) [0x559c103f7478]
[2168320.755] (EE) 11: /lib/x86_64-linux-gnu/libc.so.6 (__libc_start_main+0xe7) [0x7fbff0353b97]
[2168320.755] (EE) 12: /usr/lib/xorg/Xorg (_start+0x2a) [0x559c103e113a]
[2168320.755] (EE)
[2168320.755] (EE) Segmentation fault at address 0xffffffffffffffff
[2168320.755] (EE)
Fatal server error:
[2168320.755] (EE) Caught signal 11 (Segmentation fault). Server aborting

Both times, I was running Matlab R2019a in the crashed session, as well as standard Ubuntu apps (terminal, file browser) and the Fiji/ImageJ image analysis software (Java-based). Either Matlab (the GUI of which is based on Java AWT) or Fiji/ImageJ also leave behind a crash log file that contains the following:
FATAL ERROR in native method: Nativewindow X11 IOError: Display 0x7fbd343d2a30 (:10.0): Resource temporarily unavailable
        at sun.awt.X11.XlibWrapper.XEventsQueued(Native Method)
        at sun.awt.X11.XToolkit.run(XToolkit.java:566)
        at sun.awt.X11.XToolkit.run(XToolkit.java:532)
        at java.lang.Thread.run(Thread.java:748)

I'm running packages xrdp 0.9.5-2 and xorgxrdp 0.2.6-1build1 on an up-to-date Ubuntu 18.04 LTS. I'm using MS Remote Desktop Client 10.2.13 (1593) to connect from MacOS 10.14.5. The client was configured for 2450 x 1340 px, 16 bit color, clipboard and sound forwarding, in a non-fullscreen window.
I'm attaching my relevant snippets from my /var/log/xrdp.log, /var/log/xrdp-sesman.log, as well as ~/.xorgxrdp.10.log and ~/.xsession-errors (slightly redacted for privacy). Timestamp of crash: July 5, 14h44: xorgxrdp.10.log.txt, xrdp-sesman.log.txt, xrdp.log.txt, xsession-errors.txt
The only related issue I was able to find is #447. In contrast to what is reported there, I definitely did not alter the original xrdp.ini from the package.
Any hints appreciated! Please let me know if I can provide further debugging information. Thank you.

I believe to also see this issue after upgrading to Debian 10 (Buster).

@bantu What about Buster clean installation?

@metalefty I am afraid, I will not be performing a clean install of Debian 10. Anything else I can look out for in particular?

This is actually xorgxrdp issue, not xrdp. See neutrinolabs/xorgxrdp#121.

For the record: I purged and reinstalled the xrdp package, then customized the xrdp.ini to my use case. This solved the problem for me.

authors: bantu metalefty nobias

repo: neutrinolabs/xrdp


text:
mark count with unused attribute

This may throw a warning when devel logs are disabled
Fixed
../../../xrdp-0.9.19/sesman/chansrv/chansrv.c:198:9: error: variable 'count' set but not used [-Werror,-Wunused-but-set-variable]                                                                                int count;                                                                                ^                                                                                1 error generated.
Upstream-Status: Pending
Signed-off-by: Khem Raj raj.khem@gmail.com

Hi @kraj
I can't reproduce this. Can you let me know what your compiler version and any flags are?
Also, there's a macro to do this called UNUSED_VAR(). It's not as pretty as the __attribute__ but it works on a wider range of compilers.
I'll have a look at the FreeBSD CI fail. That's nothing at all to do with you.

@kraj - I've just noticed your branch is extremely out of date - you're 367 commits behind the development branch.
This is why the FreeBSD CI check is failing.

@kraj - I've just noticed your branch is extremely out of date - you're 367 commits behind the development branch.
This is why the FreeBSD CI check is failing.

its latest  clang-15 ( to be released soon)

@kraj - thanks for your contribution


text:
ci: Add Open Source Policy Workflow

Hello from the Open Source and Developer Advocacy team at New Relic! This PR contains a GitHub actions workflow which runs repolinter, an automated tool to help us find and fix inconsistencies in our community files and public repositories. This PR registers a GitHub Actions workflow (repolinter.yml) running repolinter-action. The goal of this workflow is to make our Open Source Policy enforcement more consistent and transparent for everyone.
Repolinter Action does:

Scans the repository for Open Source policy issues using Repolinter.

The issues this workflow scans for are determined by the ruleset specified by with.config-url: this ruleset will be maintained by the Open Source and Developer Advocacy team and will not change frequently.

Runs only on push to the default branch, ignoring PRs and other events.
If any policy issues are found, this workflow will open a GitHub issue titled "Open Source Policy Issues" and with the repolinter label. This issue will contain detailed information on which policies generated errors and what changes are suggested. An example issue is here.
As policy issues are resolved, this workflow will update the open issue to reflect the changes. If all policy issues are resolved, the issue will automatically be closed.

This workflow is designed to cause the least amount of friction as possible:

This means workflow will only run on push, and as a result will not participate in status checks for PRs. Furthermore, this workflow will always exit with a passing status unless a fatal internal error occurs.
The goal of this workflow was not to prevent work: this means that it is up to your team to backlog the items found by this tool appropriately.
This workflow will only notify you when an issue is created. In other words, you will only be notified when new problems occur, but not when existing problems change.

What you should do:

Review this PR and workflow. Make sure that the correct base branch is selected.

If this workflow conflicts with your team's infrastructure, let us know in #help-opensource! We're happy to work with individual project needs.

If the PR looks good, merge the PR and check to see if the repolinter-action runs successfully in the actions console. You can remove the branch created by this PR as it is no longer needed.
The action may open a GitHub issue on this repository (it will have the repolinter label). If an issue is opened, look it over and make sure all the problems seem correct. Shoot us a ping in #help-opensource if anything looks suspicious.
In the future, this workflow may open issues as our open source policies are updated. Keep an eye on the GitHub issues for these updates (it may help to watch the repository).

This workflow is still in beta! If you have feedback, please let us know in #help-opensource. Thanks!

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


text:
Increase size of flushExecutor thread pool size

Patch by marcuse; reviewed by Branimir Lambov for CASSANDRA-12071

🍻

sidekick:approved

authors: derekroller jfernandez

repo: newrelic-forks/cassandra


text:
EpochTime UnmarshalJSON Error

Description
UnmarhsalJSON method of EpochTime struct return an error in case we unmarshal an empty object
Go Version
go version go1.19.5 darwin/amd64
Current behavior
When I try to call UnmarshalJSON method of EpochTime struct's object I retrieve the next error

Expected behavior
Everything needs to be successfully unmarshaled
Steps To Reproduce
Steps to reproduce the behavior:

As EpochTime struct declared in internal package we can't access it but we are going to access it via entities.DashboardEntity object (it has a field of EpochTime type dashboard.NRDBQuery.Metadata.TimeWindow.Begin)
First I marshal empty object to json
Then I unmarshal it back
Retrieve the error
Gist here https://gist.github.com/diabeticMike/d3abfd5b98b6c70b900cace344424442

Debug Output (if applicable)
strconv.ParseInt: parsing "-62135596800,": invalid syntax
Additional Context
As i saw there is even no unit tests for MarshalJSON func - i suppose thats the main reason

References or Related Issues

Thanks @diabeticMike for detailed ticket and reproducible case
This seems to be a known go issue and expected behaviour: https://github.com/golang/protobuf/issues/710
I will double check with team to see if there's something we can do about this.
If I can ask, what are you trying to do with the above? Or what's the use-case? Are you getting this error when talking to our API?

Hi @kidk , thanks for replying, the case is when I retrieve a list of dashboards then marshal them to JSON and then unmarshal back to the same struct - so EpochTime object is empty in the moment of marshaling but when i unmarshal i get an error
At this moment i created a fix for my case but test cases that are present are failing, so i'm trying to be backward compatible.
Also as i said you don't have any unit tests for MarshalJSON method.

I spent a few days digging into the problem and the last thing I want to propose (in case it is an emergency case) is this
https://gist.github.com/diabeticMike/6c27c4128a4f2d06bdcf449bc000c444
Ping me again and in case you are ok to go with it - i'll make PR

I've asked the team to review your PR @diabeticMike Thanks for making the change.

Merged and released. Thanks @diabeticMike

authors: diabeticMike kidk sanderblue

repo: newrelic/newrelic-client-go


text:
NR-147981: Add redis otel quickstart

Summary
Adds a quickstart for the OpenTelemtry Redis receiver.

Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you include a Data source and Documentation reference?
[x] Are all documentation links publicly accessible?
[x] Did you check your descriptive content for voice and tone?
[x] Did you check your descriptive content for spelling and grammar errors?
[x] Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

[x] Does the PR contain a screenshot for each of your dashboards?
[x] Do your screenshots show data?
[x] Has the sanitization script been run on each dashboard?

Alerts

[x] Did you check that your alerts actually work?

DO NOT MERGE: We're moving the PR to Ready for review in order to be reviewed and foresee any issues. Our PM, @mangulonr, will coordinate the merge along with this docs PR.

@seemantk Can we have this PR reviewed by your team and ready to be merged?once we have the docs and the entity definition ready?

Hi @mdumpati,
I addressed all the changes you proposed along with the warning alert, thanks.

Hi @mdumpati
Can you help us review the changes done by @marcsanmi?
We're still struggling with some problems with the entity synthesis, the entity platform team is on it.
But we would like this PR to be ready to merge and merge it as soon as the problems with the entity are solved.
cc @seemantk

Hi @mdumpati and @seemantk
Problems with entity synthesis are solved.
We need your review of this PR, once reviewed we will coordinate the release of the docs linked in this Quickstart.
Thanks

Thanks @zstix
OTel Redis docs are live https://docs.newrelic.com/docs/more-integrations/open-source-telemetry-integrations/opentelemetry/collector/collector-configuration-examples/opentelemetry-collector-redis/

Hi @zstix @seemantk
Do you have an estimation on when this Quickstart will be published?

authors: mangulonr marcsanmi

repo: newrelic/newrelic-quickstarts


text:
Updated infrastructure-bundle to v1.6.0

Bump the infra bundle with to last version.

CI is complaining:
  Error: Unable to process command '::set-env name=GOROOT::/opt/hostedtoolcache/go/1.15.5/x64' successfully.
  Error: The `set-env` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
  Error: Unable to process command '::add-path::/opt/hostedtoolcache/go/1.15.5/x64/bin' successfully.
  Error: The `add-path` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
  Added go to the path
  Error: Unable to process command '::add-path::/home/runner/go/bin' successfully.
  Error: The `add-path` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

authors: gsanchezgavier paologallinaharbur

repo: newrelic/nri-kubernetes


text:
chore: bumps theme

This PR bumps the theme which removes UTM params from the sign up button clicks, which should resolve some analytics problems.

Relates to newrelic/docs-website#1816

Something is definitely off with the new version of the theme. Looks like this is also failing the build for the same reason as the dev site.

I'm looking into this error:
SiteSiteMetadata.utmSource provided incorrect OutputType: 'Object({
resolve: [function wrappedTracingResolver], extensions: Object({
createdFrom: "createResolvers" }) })'

  Error: SiteSiteMetadata.utmSource provided incorrect OutputType: 'Ob
  ject({ resolve: [function wrappedTracingResolve

Closing for this PR https://github.com/newrelic/opensource-website/pull/751

authors: jerelmiller jpvajda

repo: newrelic/opensource-website


text:
Readeck bookmark scripts

Hi,
This adds two scripts to allow bookmarking to your Readeck installation.
They are Clojure scripts which need the added dependency of Babashka to run, but using babashka allows for much faster startup times and memory usage than would be possible with JVM Clojure.
See the readme for more details :)

coverage: 60.454%. remained the same
when pulling 4c54ef7744ebd05a442670ddde0faff7ba0d80e3 on rainbow-bamboo:readeck-bookmarks
into 455c70185aea58f1fde0a32e57a2c22d39625be7 on newsboat:master.

authors: coveralls rainbow-bamboo

repo: newsboat/newsboat


text:
adding markdown file

Here's my test stuff @ebeshero

Hey Grason Nice job with your test. This is Test 5 of GitHub

@A-Little-Coconut I'm finally merging in old branches from the Git Test. Good work with round 3! You can safely delete your tefftBranchTest3 on your local copy of the textAnalysis-Hub repo if you haven't done so already.

authors: A-Little-Coconut SCD5363 ebeshero

repo: newtfire/textAnalysis-Hub


text:
mergify: Expand the types of PRs that can be merged

commit 7fdecb47e1db5af7c3e4a07b1008ea2c82f8bd22
Author: Russell Bryant russell.bryant@gmail.com
Date:   Thu Apr 20 15:27:47 2023 -0400
mergify: Expand the types of PRs that can be merged

This PR defines 3 sets of rules for what CI must pass based on the
files changed in the PR.

1) If a PR only runs the docs workflow, only check for `markdown-lint`
   passing. This is based on checking the file types changed in the
   PR. The regex must stay in sync with the file types listed in the
   github workflows, so I added a comment across the 3 files as a
   reminder.

2) If a PR has no markdown file changes, it only needs to pass the
   build workflow.

3) If a PR also includes markdown changes, it must pass the build
   workflow and markdown-lint.

Also add a requirement that the DCO check passes.

Signed-off-by: Russell Bryant <rbryant@redhat.com>

commit 16d0167d3c681d003c14c55e6062ce050f1c243f
Author: Russell Bryant russell.bryant@gmail.com
Date:   Thu Apr 20 17:22:31 2023 -0400
ci: Drop enforce-labels workflow

The functionality here has been replaced by using mergify to enforce
merge policy.

Signed-off-by: Russell Bryant <rbryant@redhat.com>

Oh cool, looks like mergify won't merge changes to it's config unless you explicitly allow it 🎉

authors: dave-tucker russellb

repo: nexodus-io/nexodus


text:
[클린코드 리액트 2기 김성찬] 계산기 구현

안녕하세요!
이번 TDD, 클린 코드 with React 2기 수강생 김성찬입니다!
react-testing-library와 jest가 익숙하지 않은 채로 테스트 코드를 작성하면서 진행해보니 시간이 오래걸리더라구요 ㅠㅠ..
시간은 오래걸렸지만 재미있게 열심히 작업했습니다!
리뷰 잘 부탁드립니다!

🎯 기능 요구사항

[x] 2개의 숫자에 대해 덧셈이 가능하다.
[x] 2개의 숫자에 대해 뺄셈이 가능하다.
[x] 2개의 숫자에 대해 곱셈이 가능하다.
[x] 2개의 숫자에 대해 나눗셈이 가능하다.
[x] AC(All Clear)버튼을 누르면 0으로 초기화 한다.
[x] 숫자는 한번에 최대 3자리 수까지 입력 가능하다.
[x] 계산 결과를 표현할 때 소수점 이하는 버림한다.
[x] 연산의 결과값이 Infinity일 경우 오류라는 문자열을 보여준다. (아이폰 참고)

🤚 질문있습니다!
style에 대한 테스트 코드가 필요할까요?

해당 프로젝트에서는 style에 대한 테스트 코드를 작성하지 않았는데 만약 state에 따라 background-color가 변하는 동작이 있을 경우에는 테스트 코드를 작성해도 괜찮지 않을까 생각하게 되었고 리뷰어께선 어떤 의견이 있으신지 궁금해서 질문 남깁니다!

alert에 대한 테스트 코드는 어떻게 작성하는게 좋을까요?

jest에서 alert에 대한 테스트 코드를 작성하는 방법을 찾아보니 window.alert = jest.fn() 같이 작성하여 jest.fn()을 임시로 window.alert에 할당하고 window.alert가 몇번 호출되었는지로 테스트 진행하더라구요.

그런데 위 방법으로 테스트를 진행하게 되면 alert에 들어가 있는 문구를 확인할 수 있는 방법이 없었습니다.

혹시 위 방법을 사용했을 때 문구를 확인할 수 있는 방법이 있는지 궁금합니다! 또는 더 좋은 방법이 있을지 궁금합니다! 계속 찾아봤는데...안보이더라구요

else와 중첩 if 사용에 대하여...
// 변경 전 if...else
if (isNaN(buttonValue)) {
  operation.push(buttonValue);
  counter = counter + 1;
} else {
  number[counter] = number[counter] ? number[counter] + buttonValue : buttonValue;
}

// 변경 후 if, if
if (isNaN(buttonValue)) {
  operation.push(buttonValue);
  counter = counter + 1;
}

// ...

if (!isNaN(buttonValue)) {
  number[counter] = number[counter] ? number[counter] + buttonValue : buttonValue;
}

많은 클론 코딩 관련 글을 찾아보면 else의 사용을 지양하라는 글을 많이 발견합니다.
저 또한 그래서 else를 사용하지 않으려고 위와 같이 수정을 진행하였습니다.
코드를 변경하고 보니 else를 사용함으로써 반대되는 케이스를 잡는 것이 의도를 전달하는데 더 유리해 보였습니다.
리뷰어께서는 else의 사용에 대해서 어떤 의견이 있으신지 궁금합니다!

추가로 중첩 if에 대해서도 질문이 있습니다!

// 변경 전
if (isNaN(buttonValue)) {
  if (!number[counter]) {
    alert(INPUT_NUMBER_FIRST_MESSAGE);
    return totalText;
  }
  operation.push(buttonValue);
  counter = counter + 1;
}

// 변경 후
if (isNaN(buttonValue) && !number[counter]) {
  alert(INPUT_NUMBER_FIRST_MESSAGE);
  return totalText;
}

if (isNaN(buttonValue)) {
  operation.push(buttonValue);
  counter = counter + 1;
}

중첩 if문 사용을 지양하기 위해 위와같이 중첩 if문을 단일 if문으로 분리하여 작성하였습니다.
중첩 if문을 사용하면 !number[counter]만 체크하면 되는데 단일if문으로 변경하고 보니 isNaN(buttonValue)라는 동일한 조건을 2번 체크하게 되었습니다.
리뷰어께서는 위에 상황에서 중첩 if문 사용에 대해서 어떻게 생각하는지 궁금합니다.

validation 로직을 어떻게 분리하는게 좋을까요?

계산기에 입력된 식을 계산하는 로직을 커스텀 훅으로 옮겨주는 작업을 진행하였습니다.
옮겨주고 보니 로직의 반이 validation 로직이 차지하고 있었습니다.
validation 처리를 하는 부분을 하나의 함수로 만들어서 boolean 값을 리턴하는 식으로 바꿔보면 어떨까하는 생각을 해봤습니다.
validation 체크를 하는 로직을 분리하는게 좋을지, 아니면 그대로 유지해도 좋을지 궁금합니다.

expect(window.alert).toHaveBeenCalledWith('잘못된 기호 또는 문자가 포함되어있습니다.');

안녕하세요. 동희님!
피드백 주신 부분들 하나씩 확인해보며 적용해보았습니다.
그런데 함수의 역할을 분리하는 부분에서 의문이 생겼는데
함수를 분리하면서 커스텀 훅 내부에 UI 측으로 노출되지 않는 함수가 여러개 생성 되었는데 추가적으로 이러한 함수들을 다른 파일이나 커스텀 훅으로 분리하는게 좋을지 궁금합니다.
또 totalText를 커스텀 훅 내부로 옮기면서 setTotalText를 통해 데이터가 변화하는 부분을 insert~ 네이밍의 함수들로 분리를 진행했는데 괜찮은 방법인지 궁금합니다!
감사합니다!

repo: next-step/react-calculator


text:
Update documentation

There is a lot of old outdated documentation in this pipeline. This needs resolving before the v1.0 release.

Fixed in PR https://github.com/nf-core/chipseq/pull/38

authors: chuan-wang ewels

repo: nf-core/chipseq


text:
TEMPLATE branch is dirty

At some point, commits from dev have been merged into the TEMPLATE branch. This makes the TEMPLATE branch "dirty" and will make the automated template synchronisation pull requests super difficult to merge.
Docs on how to fix this are here: https://nf-co.re/developers/sync#fixing-a-broken-template-branch - please do in conversation with me / other core-team members on Slack though. If I find time I will try to sort this out myself in the coming days.

Yeah, it was here: https://github.com/nf-core/dualrnaseq/pull/2
This is the naughy commit that merged into TEMPLATE:

We didn't know about this problem until fairly recently. Shouldn't happen again as now automated PRs come from dedicated branches so it is safe to do this ☝🏻

Fixed: https://github.com/nf-core/dualrnaseq/commits/TEMPLATE

repo: nf-core/dualrnaseq


text:
switch to dark scroll bars in dark mode

While trying to fix the ToC, I got too annoyed by the white scroll bars 🤷♂
Firefox:

Webkit:

This makes the scroll bars more obvious and annoying for me (Chrome, OSX).
Before the change the scrollbars are darker and automatically disappear when not being scrolled:

After the change they are there all the time and are more obvious:

Similar behaviour in Firefox and Safari. I guess this is an operating-system thing?

Ah, yes, I didn't test for cases when the OS hides scrollbars. Scrap this PR then. But interesting that your chrome bars are automatically dark. Which OS version and are you using a chrome theme?

macOS Catalina 10.15.6 (19G73)
Chrome 84.0.4147.89

No theme..

okay, maybe I should finally update to Catalina (or Big Sur soon) then 🙂

authors: ewels mashehu


text:
iControl/Xfinity Communication Error

Hello! Very new to GitHub and code so please excuse the lack of technical knowledge...
I have Xfinity Home and I was able to get HomeBridge running and recognized in MyTouchHome.app. However, Current System State is wrong and I cannot change it. Siri also does not recognize any commands. Any help would be greatly appreciated!
Here's what I get in terminal:
iControl Hidden input lang = en +0ms
[Xfinity Home] There was an error while communicating with iControl. Status code was 200 and error was: Error: Bad status code 200
Stack:Error
at iControl._notifyError (/Users/john/homebridge/node_modules/node-icontrol/index.js:476:181)
at iControl. (/Users/john/homebridge/node_modules/node-icontrol/index.js:239:12)
at Request.self.callback (/Users/john/homebridge/node_modules/request/request.js:373:22)
at emitTwo (events.js:87:13)
at Request.emit (events.js:172:7)
at Request. (/Users/john/homebridge/node_modules/request/request.js:1318:14)
at emitOne (events.js:82:20)
at Request.emit (events.js:169:7)
at IncomingMessage. (/Users/john/homebridge/node_modules/request/request.js:1266:12)
at emitNone (events.js:72:20)
Response:
Then it shows the Comcast website in html...

In the HTML response, the text says "The username and password entered do not match. Please try again."
Are you positive your email/password combination is correct? Can you login at their website here: https://login.comcast.net/login

I can't believe I missed that. Thanks @nfarina! I'm up and running now.

authors: nfarina tropicalisurf

repo: nfarina/homebridge


text:
dependency of react-helmet uses legacy componentWillMount lifecycle method

The dependency react-side-effect uses componentWillMount, so react-helmet will not work in concurrentMode.

https://github.com/gaearon/react-side-effect/issues/54

Looks like react-helmet-async is a useful alternative:
https://github.com/staylor/react-helmet-async

Looks like react-side-effect v2.0.0 now uses UNSAFE_componentWillMount.
Thus, no more warning in the console.

I confirm, after manually upgrading  react-side-effect to v2.0.0, the warning is gone.

Looks like react-helmet-async is a useful alternative:
https://github.com/staylor/react-helmet-async

No, it's not. Creating another package just to undeprecate something so trivial is not an alternative, it's a fallback.

before react-helmet update its dependency, a temporary workaround in package.json
  "resolutions": {
    "react-side-effect": "^2.1.0"
  }

so react-helmet will not work in concurrentMode

It doesn't sound like the issue as originally opened was actually fixed. The only difference is a switch from componentWillMount (which emits a depreciation notice and won't work in concurrent mode)  was replaced with UNSAFE_componentWillMount (that doesn't emit a deprecation notice; but still does not work in concurrent mode).

Why is this closed? Seems that still not working properly (https://github.com/bbc/simorgh/issues/3893#issuecomment-730702092) since we are getting the warning with ^6.1.0.

Why is this closed? Seems that still not working properly (https://github.com/bbc/simorgh/issues/3893#issuecomment-730702092) since we are getting the warning with ^6.1.0.

@cwelch5 its still the same issue.

authors: adamnorbacker danielweck dantman eduard-malakhov goldmont jfrolich kevr miquelvir samuraime

repo: nfl/react-helmet


text:
Refactor MrsPyramidRecordReader

After the switch to spark, it looks like most of the MrsPyramidRecordReaders can be simplified  to eliminate a lot of the custom code to do thinks like blank tile filling.
The only thing the record reader should do any more is read the data from the appropriate data source.

To expand on @ttislerdg comments, the MrsPyramidSimpleInputFormat has two inner classes that used to fill in blank tiles prior to the transition from map/reduce to Spark. That capability is no longer used, and should be removed - including the includeEmptyTiles flag in TiledInputFormatContext.
All of static "setup" functions in MrsImageDataProvider should be removed because they are not used any more.
MrsPyramidInputFormat, MrsImagePyramidInput, MrsPyramidRecordReader and MrsImagePyramidRecordReader can all be removed since they are no longer used.
Get rid of the two AllTiles inner classes from MrsImagePyramidSimpleRecordReader.
Rename MrsPyramidSimpleInputFormat to MrsPyramidInputFormat. Rename MrsImagePyramidSimpleInputFormat to MrsPyramidInputFormat. Rename MrsPyramidSimpleRecordReader to MrsPyramidRecordReader. Rename MrsImagePyramidSimpleRecordReader to MrsImagePyramidRecordReader.

We also believe that these changes will fix a problem that we are currently seeing when building a pyramid on an image stored in S3 within EMR.

Potential impact on performance?

I don't anticipate much of any impact on performance. Most of this is removing unused code.

And possibly increased stability.

Yes, and simplified code, and most importantly for the short-term, we expect it will address our current problem with build pyramid.

authors: djohnson729 drew-bower ttislerdg

repo: ngageoint/mrgeo


text:
chore: Address issues identified by linter in src/core/environment.go

Proposed changes

Replace deprecated io/ioutils with os read/write function calls
Rename files variable in func ReadDirectory() due to identical name in another library
Appropriately handle file close
Appropriately deferred handle file close
Tested: make clean; make build; make deps; make unit-test

Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[ ] I have run make install-tools and have attached any dependency changes to this pull request
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Running make install-tools yields minor version upgrades to about 10 modules.
Is it expected that one includes these as well eventhough this commit is only addressing linter warnings?

repo: nginx/agent


text:
Nginx ingress controller and Nginx Gateway Api

Is your feature request related to a problem? Please describe.
Trying to install both Nginx ingress controller and Nginx Gateway Api at the same time on AKS, but seems like installing and/or configuring Nginx Gateway Api breaks the ingress controller.
Describe the solution you'd like
A way to have both Nginx ingress controller and Nginx Gateway Api at the same time

Hi @pfekrati, can you please provide me with the following information:

Version of the NGINX Kubernetes Gateway - release version or a specific commit. The first line of the nginx-gateway container logs includes the commit info.
Version of NGINX Ingress Controller
Version of Kubernetes
Steps to reproduce
Error messages or relevant logs

Thanks!

Hi @kate-osborn,
Thanks for your response.
Version of the NGINX Kubernetes Gateway - release version or a specific commit. The first line of the nginx-gateway container logs includes the commit info:
"version":"0.1.0-rc.1-SNAPSHOT-6befb1d","commit":"6befb1dffbf6bb32be9257f8a6f373766a1ad372","date":"2023-04-27T17:50:15Z"
Version of NGINX Ingress Controller:
4.6.0
Version of Kubernetes:
1.25.6 (AKS)
Steps to reproduce
I've followed the steps described here (https://github.com/nginxinc/nginx-kubernetes-gateway/blob/main/docs/installation.md) to install Nginx Gateway Api and instead of creating a Loadbalancer service I created a ClusterIP service for the gateway and then added a path in Nginx Ingress to route traffic to the service. I was expecting to have the traffic go to that service and from there to the gateway. Not only it didn't work but it also broke the other paths defined in the ingress that were working before installing the gateway Api.
Error messages or relevant logs:
I only see this error in gateway pod logs
{"level":"error","ts":"2023-04-27T19:45:07Z","logger":"eventHandler","msg":"Failed to update NGINX configuration","error":"failed to find NGINX main process: open /etc/nginx/nginx.pid: no such file or directory","stacktrace":"github.com/nginxinc/nginx-kubernetes-gateway/internal/events.(*EventHandlerImpl).HandleEventBatch\n\t/home/runner/work/nginx-kubernetes-gateway/nginx-kubernetes-gateway/internal/events/handler.go:85\ngithub.com/nginxinc/nginx-kubernetes-gateway/internal/events.(*EventLoop).Start.func1.1\n\t/home/runner/work/nginx-kubernetes-gateway/nginx-kubernetes-gateway/internal/events/loop.go:67"}

v4.6 seems to be the version of the Helm chart for the K8s community ingress controller which is currently at version 1.7
Does that seem correct?

correct @brianehlert . My bad

I see a couple issues in here..

I created a ClusterIP service for the gateway and then added a path in Nginx Ingress to route traffic to the service. I was expecting to have the traffic go to that service and from there to the gateway.

The gateway implementation should not be considered to be 'behind' an ingress but rather 'beside' an ingress.  It functions similar to an ingress controller.
Your issues appear to be with the configuration of the ingress in this case.
commenting on this statement Not only it didn't work but it also broke the other paths defined in the ingress that were working before installing the gateway Api.
You also describe the gateway pod log error, that is something we can look into

@brianehlert, thanks for replying. Does the Nginx gateway API have all the capabilities of Nginx Ingress controller? Does it support all the configurations, etc?

I filed an issue for the error log in the gateway Pod: https://github.com/nginxinc/nginx-kubernetes-gateway/issues/600
This is a known issue that can occur on startup. Once the NGINX container is running, the reloads will succeed.

What can I do to fix that? Even on a cluster without Nginx Ingress Controller I wasn't able to get the gateway api running with Loadbalancer service. I'm using AKS and k8s version 1.25.6

What can I do to fix that? Even on a cluster without Nginx Ingress Controller I wasn't able to get the gateway api running with Loadbalancer service. I'm using AKS and k8s version 1.25.6

You shouldn't have to do anything. Have you tried running through this example?
Are you seeing other errors besides the one you posted above?

@kate-osborn  I was able to run the cafe example successfully, but the traffic splitting one -which is the one I'm interested in- doesn't work for me. The same error happens every time for me. I tried delete and recreate the nginx-gateway deployment but even that didn't help

@pfekrati can you share the full logs of the nginx-gateway container and the statuses of the gateway and httproute resources?

authors: brianehlert kate-osborn pfekrati

repo: nginxinc/nginx-kubernetes-gateway


text:
Allow extra parameters based to be added based on redirected domain

This PR allows you to add extra parameters to the redirected URL based on the redirected domain. Consider the following use-case;

A server setup to listen to two domains [A] and [B]. The domain [B] is some site that has content migrated to domain [A].
Whenever a user requests a page from domain [B], the request is redirected to site [A] using misdirection.
Using the new "Domain Parameters" option in misdirection, you can add parameters to the request based on the origin (domain [B]). The page the user is redirected to, can use these parameters to display - for instance - a banner to inform the user that content from domain [B] has been migrated to domain [A]

[original request] domain.b/  -> [rewritten to] domain.a/?parameter1

Thanks for the contribution here, but can't you already redirect to another domain with custom parameters? This is even easier when using a regular expression to capture the request. It's been a while since I've done so, but I recall this being something I've had to solve in the past.

If this still applies, can you please resolve the conflicts and open another PR on https://github.com/symbiote/silverstripe-misdirection, as this will be the source of truth moving forwards.

If this still applies, can you please resolve the conflicts and open another PR on https://github.com/symbiote/silverstripe-misdirection, as this will be the source of truth moving forwards.

authors: jvdanker nglasl

repo: nglasl/silverstripe-misdirection


text:
fix encoding for dyndns2 basic auth header

Hey,
didn't catched up my fork by now.
Could you please apply the following fix?
change line 121 to be string for split lateron:
auth_header = str(base64.b64decode(auth_header), 'utf-8')
Thanks

Updated the code. Thank you!

authors: ngoduykhanh thomasDOTde

repo: ngoduykhanh/PowerDNS-Admin


text:
add app protocol to http builder

Resolves https://github.com/ngrok/ngrok-javascript/issues/107
Why
To allow specifying http1 or http2 for listeners
How
Add app_protocol to both HTTPListenerBuilder and config.
Add an example with node's http2 example.
Validation
Run example with local build and can view Hello World in browser on listener url.

Note: I didn't add an Express example because Express 5 and http2 support is still in beta. 😂
https://expressjs.com/en/5x/api.html

repo: ngrok/ngrok-javascript


text:
For the UK

Hey
Any chance this could be updated so it can be used in the UK.
I.e. also adding in .amazon.co.uk/ as well as .com so primenow.amazon.co.uk will work with this tool currently it doesn't and im sure the two websites are built the same.
Thanks

@greeenbe , I just updated, see if that works for you. Thanks


text:
async loading model

[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ x] support request

Current behavior
 ngOnInit() {
    this.route.params.subscribe(params => {
        this.service.getConvenzioneById(params['id']).subscribe((data) => {
          this.model = data;
        });
    });
}

When the model is populated for the first time all the field.formControl.valueChanges emit the event.
How to avoid to listen the first event?

the easy way is to avoid init the form until model is ready, using
<formly-form *ngIf`

I used:
 <formly-form *ngIf="model"
and in my template
       lifecycle: {                     onInit: (form, field) => {                                               field.formControl.valueChanges.pipe(               distinctUntilChanged(),                            takeUntil(this.onDestroy$),                             tap(selectedField => {

onInit is called only with field has value
valueChanges is called only after a user modification
Seems working well.

Drawback is a completely empty page until it arrives the model
I'm not satisfy 100% seems the form + model need to be provided at the same time.
Other stuff related to the init of the form:  I need in the template access to other part of the model is possible?
onInit: (form, field) => {
//HERE ... other part of the model ...
           field.formControl.valueChanges.pipe(

@aitboudad I have problem in reloading form.

I'm going to close since it's not clear, please elaborate to get an answer.

@aitboudad you are right. Thank you for your response.
I have this button with this disabled condition: !form.valid || !form.dirty
<button class="btn btn-outline-primary border-0 rounded-0" [disabled]="!form.valid || !form.dirty" (click)="onSubmit()" >
<form *ngIf='model' [formGroup]="form" >
onReload (){
      this.model = null;
      let id = this.model.id;
      this.service.getConvenzioneById(id).subscribe((data) => {
        this.model = data;
      });
}

I call the onReload function.
The form remain in dirty state.
If I did this.form = new FormGroup({});  the form became not modified but it is disconnected from the button.

try with this.form.reset()

Hi @aitboudad the result of the form now is coerent. Thank you.
But the side effect is the form disappear during the update process. Any suggestion?
   this.model = null;
   this.service.updateConvenzione(tosubmit, tosubmit.id).subscribe(
        result => {
          this.form.reset();
          this.isLoading = false;
          this.id = result.id;
          this.model = result;
        },

ok instead of setting model to null rely on a locale variable (for example isLoading) to ignore emitted value during updateConvenzione and use options.reset() to update model value:
// field config
field.formControl.valueChanges.pipe(
  distinctUntilChanged(),
  takeUntil(this.onDestroy$),
  filter(() => !this.isLoading),
  ...
)

// updateConvenzione
   this.service.updateConvenzione(tosubmit, tosubmit.id).subscribe(
        result => {
          this.id = result.id;
          this.options.resetModel(result);
          this.isLoading = false;
        },
  )

Ok great!
I have a valueChanges inside a formly template, how to pass the local variable isLoading inside the template or set it externally?

pass it through formState
https://formly-js.github.io/ngx-formly/examples/form-options/form-state

I like the solution with form-state.
I lost only one behavior in select
          {
            key: 'tipoemittenti_codice',
            type: 'select',
            className: "col-md-6",
            templateOptions: {
              options: [],
              valueProp: 'codice',
              labelProp: 'descrizione',
              label: 'Autorizzato da',
              required: true
            },
            lifecycle: {
              onInit: (form, field) => {
                field.templateOptions.options = [this.model.tipoemittente];
                this.service.getEmittenti().subscribe((data)=> {
                  field.templateOptions.options = data
                });
              }
            }

Using the *ngIf on model when start the onInit the model is populated and
the set  field.templateOptions.options = [this.model.tipoemittente]; in the onInit
work well.
@aitboudad is better uselifecycle onChangeor register form.get('dipartimemto_cd_dip').valueChanges.pipe( and control isLoading

Any update ?
When i try to load model with
<ion-content *ngIf="loadedGame | async as game" padding>
Validation does not work

@rdev-software not sure what you mean, please elaborate in a new issue instead.

@aitboudad That if I have observable loadedGame and I want to async load it in parent in this case ion-content and place dynamic form inside and use loaded object game inside, validation does not work (and maybe other things as well as per this thread)

@rdev-software please replicate what you've done using our starter example https://stackblitz.com/edit/ngx-formly-ui-ionic in order to see what going wrong + fill a new issue.

authors: aitboudad enricooliva rdev-software

repo: ngx-formly/ngx-formly


text:
fix: make FormlyModule ModuleWithProviders generic for ng9 support

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes ability to use ng9 due to new requirement that ModuleWithProviders is generic: https://github.com/angular/angular/pull/33187
Please check if the PR fulfills these requirements

[x] The commit messages follow our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[x] A unit test has been written for this change.
[x] Running npm run build produced a successful build. (Unit testing can be done by running npm test;)
[x] My code has been linted. (npm run lint to do this. npm run build will fail if there are files not linted.)

Thank you @intellix!

v5.5.2 is released which includes the fix!

authors: aitboudad intellix


text:
Document POSIX data type usage.

This change is

You are welcome, thank you for all the clear instructions :-)


text:
getHTML() Method in Toast-UI/Editor Returns Unexpected Value When Editor is Empty

Describe the bug
When using the getHTML() method in toast-ui/editor, it consistently returns  instead of null when the editor is empty.
To Reproduce
Steps to reproduce the behavior:

Initialize toast-ui/editor.
Use the getHTML() method when the editor is empty.
See that "" is returned instead of null.

Expected behavior
The getHTML() method should return null when the editor is empty just like getMarkdown().
Screenshots

Additional context
This bug affects the behavior of the getHTML() method specifically when the editor is empty. It may impact applications relying on the returned value to determine whether the editor content is empty or not.

I agree. Same for getMarkdown It grabs things from the HTML of the editor itself. Here in French language you see some values:
For me empty or not !


text:
fix: cannot toggle custom toolbar state(active, disabled)

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has a description of the breaking change

Description

fixed that which cannot toggle custom toolbar state(active, disabled)
related issue

#1592
#1596
#1619

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@adhrinae

리뷰 완료합니다.
사소한거지만 classname 이어붙이기 할 때 상태가 엮여들어가면 역시 코드가 지저분해지게 되는데 classnames 같은 방식으로 정의하는 것은 어떻게 보시나요? 캘린더에도 이런 형태의 헬퍼 함수를 하나 만들어서 적용하려 합니다.

넵 좋네요~ㅎㅎ 아래처럼 변경해서 조건에 따라 사용할게요~
export function cls(...names: (string | [boolean, string])[]) {
  const result = [];

  for (const name of names) {
    let className: string | null;

    if (Array.isArray(name)) {
      className = name[0] ? name[1] : null;
    } else {
      className = name;
    }

    if (className) {
      result.push(`${CLS_PREFIX}${className}`);
    }
  }

  return result.join(' ');
}


text:
How do you get the text added on the toast ui image editor?

Summary
I would like to save on the database the text I added on the toast ui image editor, and I am lost on how to get the text added on the canvas. I can't seem to find the text I added on the toast ui image editor on the console.log using
const canvas = useRef(null);
canvas.current.getInstance();
console.log('canvas: ', canvas.current.getInstance());
I even tried the onAddText and it only showed the originPosition and clientPosition.
Screenshots

Version
"^3.15.2"

@djparulan
Thank you for the question.
There is no API that reads text directly. But try the following method.

Save the id of the added text object.
imageEditor._graphics.getObject(id);


text:
크롬 브라우저에서 example 실행해보고 있는데요.

Run it! 버튼을 눌러도 반응이 없는데요. 확인 부탁해요.

어떤 페이지에서 어떤 코드를 실행했는지 알려주시겠어요?
현재 예제페이지에서 작성된 코드에 예외가 발생한경우 이전상태의 코드를 실행시켜주고 있습니다.

제 경우에는 codemirror 관련 라이브러리가 없어서 발생하여 별도로 다운로드 받은 후, 옮겨두었더니 실행이 되었습니다. (tui.chart/lib/codemirror 아래에 codemirror 다운로드한 파일 위치)

authors: gem874 junghwan-park nunghyun


text:
feat(nhost_flutter_auth): Allow displayName and locale to be passed on signInAnonymous

Added this PR with regards to https://github.com/nhost/hasura-auth/blob/main/src/routes/signin/anonymous.ts that allows user to send displayName and locale to override the default "Anonymous User"

How to make the checks pass?
Run PERCENTAGE=$((( $TOTAL ) * 100 / ( $TOTAL_MAX - ${MAX_PANA_MISSING_POINTS:-10} )))
  PERCENTAGE=$((( $TOTAL ) * 100 / ( $TOTAL_MAX - ${MAX_PANA_MISSING_POINTS:-10} )))
  if (( $PERCENTAGE < 100 ))
  then
    echo "Score to low ($TOTAL/$TOTAL_MAX)."
    exit 1
  fi
  shell: /usr/bin/bash -e {0}
  env:
    TOTAL: 1[2](https://github.com/nhost/nhost-dart/actions/runs/7671882054/job/21057417995?pr=131#step:6:2)0
    TOTAL_MAX: 1[4](https://github.com/nhost/nhost-dart/actions/runs/7671882054/job/21057417995?pr=131#step:6:4)0
Score to low ([12](https://github.com/nhost/nhost-dart/actions/runs/7671882054/job/21057417995?pr=131#step:6:13)0/140).
Error: Process completed with exit code 1.

Added metadata and also modified the function to set the session after api call. @dbarrosop

repo: nhost/nhost-dart


text:
Seed health questions if consent given

470bf3098 added a callback to the ConsentForm model that removes health questions when a user changes their response from given to refused.
However, because health answers were being initialised on the create action in the controller, this meant that if a user then subsequently changes their response from refused to given, the health_answers array would be empty. So users would be unable to answer health questions.
This moves the initialisation logic from the controller to the model and makes it consistent with the removal logic.

Tested locally and works, no useful comments to make in the code, however. :shipit:

authors: paulrobertlloyd tvararu

repo: nhsuk/manage-childrens-vaccinations


text:
Gather examples of error message guidance

What
https://confluence.service.nhs.uk/display/SMDS/Error+messages
Capturing examples of error messages being used in various services at the moment.
For review and to include good, tested examples in the service manual.
Why
Tasks

Great idea.

Thanks Caroline. Once I've got a full, organised spreadsheet, I'll see if I can publish it in the open for people outside NHSD to see and comment on.

authors: cjforms lucyha sarawilcox

repo: nhsuk/nhsuk-service-manual


text:
Global Prefix

I like that the tests for HW3 have a prefix variable for where the files are located.
Could we agree to extend this system with a global prefix?
My goal is to have this repository outside of my personal hw directories but be able to access the tests from there.
If we have one global ml file with a prefix variable, which everybody can configure to a value to their liking, the repo could be cloned anywhere, and from then on, updating would be as easy as a pull. Also, for future homeworks, the only setup one needed to do would be to configure the main and makefile.

I'll happily implement the changes if you don't have the time to do it yourself ;-)

I like the idea, please go ahead and thanks! :)

see #36

Solved with https://github.com/nicdard/compiler-design-eth-tests/pull/36

authors: Multimodcrafter nicdard

repo: nicdard/compiler-design-eth-tests