Make IsNetworkFilterConfig exported

Before this change, it is hard to import the v1 API structures and define
a new custom NetworkFilterConfig in an external module.  A NetworkFilterConfig
needs to implement isNetworkFilterConfig, but that method is private to the v1
module.  This change makes it exported.
This is useful for people who want to write Pilot webhooks that mutate
the xDS data---the relevant data structures are already defined.
Signed-off-by: Spike Curtis<EMAIL_ADDRESS>

Due to the poor linter subtyping inference, you need to add a comment on every IstNetworkFilterConfig implementation.

@kyessenov fixed!

/lgtm
On Thu, Feb 15, 2018 at 2:30 PM Spike Curtis<EMAIL_ADDRESS>wrote:

@kyessenov https://github.com/kyessenov fixed!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/3252#issuecomment-366083614, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxuBE698MndeT7aoQl1lDKt_hghu0ks5tVLAbgaJpZM4R7_Dd
.


Possible to create a catch-all route that will reverse-proxy requests?

Hello,
I'm considering re-writing a Python Tornado web application in Go (Utilizing Revel). I'd like to have both services running concurrently and  basically like to have a catch-all route at the end of my revel routes file that will reverse proxy requests as-is to the Python service. That way I can write new routes / migrate existing routes on my own timeline, but keep both services operational with only one routes config.
I could also just have nginx/apache do the routing for me, but it would be nice to just have to maintain one simple routes file that would do it for me. I've read the routing section in the doc and this doesn't seem possible. Perhaps I could just have a catch all route that runs a Go function that will do it for me?
Has anyone done anything like this before?

Hi,
It certainly can be done. You need to write a filter that runs before the [RouterFilter] that checks if a route path could be found (similar to the route filter code) If the MainRouter returns nil then do a proxy call to the python instance, and write the results into controller.Response.GetWriter()
In the next release (this week hopefully) I have added a catch all path function it operates first though (so it isnt a catchall). The man page is here
If you have more questions feel free to chat on https://gitter.im/revel/community,


Research Mobx "observer batching" option

I am updating our mobxx-react dependency to 6.2.x and the addition of "observer batching" was noted in the changelog - see this doc which recommends the inclusion of a new import statement to enable.

Mobx is printing a warning to the console telling us we should include the documented import to enable observer batching. I confess I don't understand the issue, but I will push an update to develop now to include the import as per the docs / warning.
My reasoning is that it's better to test this 6.2 mobx generally in the recommended configuration. If we see any issues, we could look into them or roll back to 6.1, although note that I ran into issues with a bad 6.1.x release they published that failed to compile,

Thanks for this. Totally agree.


Code of conduct

We've been fairly lucky in this project especially given its broad reach to have maintained a high degree of cordiality and civility. This is much thanks to @espadrine who has a great deal of equanimity, and we continue to have a team of very friendly maintainers and mostly civil contributors.
That said, it's a best practice to provide a code of conduct. Projects without a code of conduct may be perceived as less safe / friendly to new contributors.
While I know that's not actually the case for this project, I want to make sure no one gets that impression from this omission. I also think it's a good idea to have something written down that we're ready to enforce when the time comes.
Contributor Covenant is a well respected template.

I'd like for us to adopt the Contributor Convenant as soon as possible and iterate from there if we need to refine anything. The one problem with it currently is that there's usually only one point of reporting/contact which could be a person someone has a CoC claim against.
Examples or bad behavior are not a concern for me, quite the opposite. Generalities are room for ambiguity that can make things much more difficult when someone is misbehaving in a borderline fashion. Having examples helps far more than it hurts. And it also sends a strong signal to the people who might be worried about the kinds of behaviors we often see in OSS projects that those specific kinds of behaviors they might have heard about or been subjected to will explicitly not be tolerated. Essentially it's a guarantee we're not making an empty promise to bring the hammer down if we have to.
Coraline who started Contributor Convenant recently started working on Beacon, a tool to manage CoC reporting so that it's not left to a single email (or email group). I'm asking her if we could part of her first users because to me this was historically the weakest point of CoCs: that the person or people receiving the complaint may be the exact same people contributors are complaining about.

Want to bump this
What steps do we need to take to roll out a CoC? Do we just need to pick one, do we need to agree to a process for selecting one, etc.
Given my own inexperience in this space, I can get on board with using a widely-used template, Contributor Covenant or otherwise. I'd just prefer that we find one we can adopt sooner than later

Good shout on bumping this. I'm on-board that adopting an off-the-shelf document like CC is the best way forward.

Personally, I'm trying to make 2020 a year of pushing forward on people-related parts of Shields. Getting the ops team going, and the hosting under the control of the maintainers has been a huge leap forward.
Adopting the Code of Conduct is a close second on that priority list. The recent killing of George Floyd by police in Minneapolis, and the actions which have followed, are a reminder of the importance of creating programming communities which are safe for everyone, and representative of the population as a whole.
This is the opening of the Contributor Covenant:

We as members, contributors, and leaders pledge to make participation in our community a harassment-free experience for everyone, regardless of age, body size, visible or invisible disability, ethnicity, sex characteristics, gender identity and expression, level of experience, education, socio-economic status, nationality, personal appearance, race, religion, or sexual identity and orientation.
We pledge to act and interact in ways that contribute to an open, welcoming, diverse, inclusive, and healthy community.

Shall we use this moment to carry this forward? ❤

Open a PR adding CC. I will :heavy_check_mark: it

#5236 opened to adopt Contributor Covenant .
One minor item regarding the contact email address to use, but hopefully it will be straightforward given the above discussion.
I'd suggest we keep that PR open for some period of time (I'd suggest we plan on merging by the end of the week at the latest) for maintainers to approve, or offer a specific alternative CoC if they have any objections to CC


Make windows version work

Use APIs from golang.org/x/sys/windows instead of sc cli command.
Add Executable interface to implement service control.
Pls review, thx

Thanks for your work and sorry me for delay in my feedback


Update urls.py

this line is not required.

Thanks for finding this piece of dead code!

Hopefully, edge 2.0 should address this PR as well :)


nixosTests.systemd-initrd-luks-fido2: fix test on aarch64-linux (maybe)

Description of changes
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

@ofborg build nixosTests.systemd-initrd-luks-fido2

My Understanding of the Issue is: The test failed because qemu-system-aarch64 apparently has different USB controllers enabled by default, resulting in a "speed mismatch" between the USB controller and CanoKey that only occurred on aarch64.
I could reproduce the issue on x86_64 by enabling the EHCI controller and then fix the issue by specifying which USB bus to use for the CanoKey.
This didn't fully fix the issue on my first attempt, because the UCHI controller enabled by -usb doesn't have the same bus name on aarch64 and x86_64. While bus=usb-bus.0 worked on x86_64, aarch64 didn't know that bus.
The final solution now manually enables the OHCI controller (which is similar UHCI, idk) and assigns it the id aka bus name usb-bus, so it works the same under both architectures.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3659

@ofborg build nixosTests.systemd-initrd-luks-fido2

This is the error message on fail:

qemu-system-aarch64: -device canokey,file=/tmp/canokey-file: Warning: speed mismatch trying to attach usb device "CanoKey QEMU" (full speed) to bus "usb0.0", port "3" (high speed)

https://logs.ofborg.org/?key=nixos/nixpkgs.97238&attempt_id=1159be80-378f-4fd0-8831-3bb50ab4680b
My Understanding of the Issue is: The test failed because qemu-system-aarch64 apparently has different USB controllers enabled by default, resulting in a "speed mismatch" between the USB controller and CanoKey that only occurred on aarch64. I could reproduce the issue on x86_64 by enabling the EHCI controller and then fix the issue by specifying which USB bus to use for the CanoKey. This didn't fully fix the issue on my first attempt, because the UCHI controller enabled by -usb doesn't have the same bus name on aarch64 and x86_64. While bus=usb-bus.0 worked on x86_64,

qemu-system-aarch64: -device canokey,bus=usb-bus.0,file=/tmp/canokey-file: Bus 'usb-bus.0' not found

https://logs.ofborg.org/?attempt_id=cdeb56ee-920d-4755-b173-9ae78d61ce68&key=nixos%2Fnixpkgs.297238
The final solution now manually enables the OHCI controller (which is similar UHCI, idk) and assigns it the id aka bus name usb-bus, so it works the same under both architectures.

Please add this to the commit message,  I'll do it later and then merge if you dont.


[zuul-core] Replace http only channel initializer with ALPN based

This is a breaking change and could impact users that don't want to run SSL-based servers.


how do i make a custom freeplay and main menu without haxe

Describe your problem here.
I use psych engine(NOT SORCE CODE) and I wanted to have a custom main menu and freeplay menu for my mod
if there's any way to do so please let me know also I wanted to know id there's a way to add a cutscene before the title screen with haxe
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

I dont think thats possible, ive been trying to figure that out myself, and all ive found is source code, but if you were restyling the menu, you could just use your art program, just not rearrange it

You can't (unless you just want to restyle while maintaing the positions the same)

well "technically" you could turn a song into a menu by preventing the song from actually starting in the first place, hiding the arrows, and coding a menu from scratch.

Yeah but they want to remak FREEPLAY and STORY MODE. Which aren't affected by lua

scripts only affect songs so


Difference in behaviour between Linux and Windows.

Godot version:
3.0 build 548bd4e
OS/device including version:
Windows 10 64-bit
Ubuntu Linux 17.10 64-bit.
Issue description:
Under Linux, time over (either through the debugging key or letting the timer run normally) will kill the player character, play the relevant scene, then resume play.
Windows, will go through the time over scene, but after that it does not resume control. Player character is not visible and/or reset to the correct position.
There may well be bugs in my code or scenes setup, but in this instance the behaviour should be consistent between platforms.
https://www.dropbox.com/s/qe83mmc81u75pmv/15218_demonstrator_windows.webm?dl=0 shows what happens under Windows.
https://www.dropbox.com/s/eu0ilcht03fsyym/15218_demonstrator_Linux.webm?dl=0 is what happens with Linux.
Steps to reproduce:
Run the relevant project under Windows and Linux to compare.
Minimal reproduction project:
https://github.com/Sslaxx/GodotSonicEngine

[x] I searched the existing GitHub issues for potential duplicates.

Closing as this seems to have been fixed. If it crops up with 3.1 will open a new issue.


[feat] Support multiple PRs in IntegrationJob

Changes

Alter 'Pull' to 'Pulls' in IntegrationJobRefs.
Before this commit, we handle only one Pull Request (PR) in
IntegrationJob. But we need to test multiple PRs at once because of
reducing time for re-testing.
As a result, this feature is to support the batch test(not implement
yet) in blocker.
Submitter Checklist

[ ] Docs included if needed
[x] Tests included if needed
[x] Follows the good commit messages standard

/approve


'Think Ahead' - CloudRip Mountain unit pathing issue [Python]

The units on wave 4's pathing puts three (3) units within range of the explosion instead of two (2), leading to the third unit always dying.
The code does not let me add a third target to the shielding mechanic, therefore always losing the battle.

@Kevitto, thank you for the report. Could you give your session id for this level? To get it - open the dev console in your browser and find a string sessionID. It can really help to reproduce the bug and fix it.

Sorry, I have no idea how to do any of that.  I play the free version on a browser.  I don't have a dev environment setup nor do I know how to set that up.

Ok. No problem, I think I'll find the problem. It's ok to post bugs here. Also you can write about problems with levels on the forum.

Fixed.


Azure hubs doc reference updated (Rich App)

Purpose of this pull request
This pull request...
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

@himanshu219, I made your requested changes. Could you review it again?


Karate not finding karate-config.js

In a maven script is written where -Dkarate.config.dir is set to a location where no config file is contained, the user will not be notified. It will also not use the file contained in the class path.  It will throw errors once it tries using a config variable in a feature file. Very minor issue but would have be nice to receive an error message for it.
JDK 17
Maven 3.8.5
karate-junit 1.2.0

tagging as help wanted

i personally think it should NOT use what is contained in the classpath, because we should honor user intent.

i agree with you but because it seems repositories running older versions and jdk8 don't have this issue (it just runs whatever config is in class path) since i got the script from one of those. An appropriate  error message would be enough in my opinion to avoid confusion.

chances of this being picked up improve if you follow this process: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


Setup sonar

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

Please upload report for BASE (main@26e01a2). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage        ?   91.74%
=======================================
  Files           ?        4
  Lines           ?      109
  Branches        ?       14
=======================================
  Hits            ?      100
  Misses          ?        6
  Partials        ?        3

Flag
Coverage Δ

unittests
91.74% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Date Format Does not apply

regardless of the settings entered the moment.js format used is always YYYY-MM-DD.

sort of. the {{date}} referenced in the filename is the filenameDateFormat which I've added a separate setting for. the existing code was using something called prependDateFormat which was fixed at YYYY-MM-DD and now its separate.
Possible that we might have a new variable called {{filename_date}} to distinguish between both.


Topological Sorting C#

💥 Proposal
I'd like to add code for topological sorting in C# with proper comments and explanation as part of GSSOC'21.
Have you read the Contributing Guidelines on Pull Requests?
yes

/assign


CI: added golangci-lint to the go.yml workflow

What does this PR change?

Adds golang-ci lint action to go.yml workflow
added .golangci.yml file to the root of the project

Does the PR depend on any other PRs or Issues? If yes, please list them.

#184

Checklist
I confirm, that I have...

[x] Read and followed the contributing guide in CONTRIBUTING.md
[x] Added tests for this PR(not applicable)
[x] Formatted the code using go fmt (if applicable)
[x] Updated documentation on the Paralus docs site (if applicable)
[ ] Updated CHANGELOG.md

@akshay196 please review and suggest changes (if any), i'll  raise another pr for fixing linting errors.

Hey @akshay196 @niravparikh05 are we proceeding with this? I have various other changes ready but its better to get this merged first.

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

Project coverage is 36.01%. Comparing base (9f8ce0c) to head (f6923b9).
Report is 10 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #311      +/-   ##
==========================================
+ Coverage   34.41%   36.01%   +1.60%
==========================================
  Files          74       74
  Lines       11783     9723    -2060
==========================================
- Hits         4055     3502     -553
+ Misses       7175     5668    -1507
  Partials      553      553

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

@akshay196 i have fixed the timeout error , on the ci github action, it should work fine now.

@akshay196  Does this require any further improvements?

@akshay196 if we enable the linters , it will at least prevent any new linting errors, by the time the older ones are fixed.

@jaydee029  The main branch CI will start failing because workflow start scanning main branch as per its trigger rule. I think only-new-issues flag only work for PR but for branch it scans everything.
Anyways we can start with fixing issues in main branch and eventually PRs also gets scan as we enable linters.

I think the CI scans only the contents of the PR that is raised, in my case it didn't fail on the last commit I made


drop redundant CSS

As hinted in #86 there's CSS code that is redundant (stale code from recent refactoring). A PR should be made to drop such styling.

Addressed in #99.


Multiple sources and multiple destinations

Is it possible instead of selecting *.scss to define individual files (multiple) which each have different destinations?
So lets say I want to have foundation.css and app.css as two separate files and two different sources.  I guess I could create multiple tasks.. but not sure if this can support it by default.
Thank you!

im also having issues with this. I have multiple includepaths and im trying to use sourcemaps.write() which is through an error due to the included files.

By multiple destinations, you mean different output folders or just file names?

If you mean different folder destination then no it's not possible as gulp-sass doesn't do the file saving bit.
If you mean different files in the same folder, it should already do it

You could probably put something together with gulp-if if you'd like, but writing files (and for that matter, reading files) is outside the scope of gulp-sass. You may need to write a custom file writer. I'd recommend searching Stack Overflow for a general Gulp answer, or posting a general Gulp question there.


[FEA] Allow all inactive in DEFAULT MODE.

SHM file
# system modes example
---

actuation:
  ros__parameters:
    type: system
    parts:
      drive_base
      left/manipulator
      right/manipulator
    modes:
      __DEFAULT__:
        drive_base: inactive.__DEFAULT__
        left/manipulator: inactive.__DEFAULT__
        right/manipulator: inactive.__DEFAULT__
      LEFT:
        # drive_base: active.__DEFAULT__
        left/manipulator: active.__DEFAULT__
        # right/manipulsator: inactive.__DEFAULT__
      RIGHT:
        # drive_base: active.__DEFAULT__
        # left/manipulator: inactive.__DEFAULT__
        right/manipulator: active.__DEFAULT__

left/manipulator:
  ros__parameters:
    type: node
    modes:
      __DEFAULT__:
        ros__parameters:
          max_torque: 0.1

drive_base:
  ros__parameters:
    type: node
    modes:
      __DEFAULT__:
        ros__parameters:
          max_speed: 0.1
          controller: PID

right/manipulator:
  ros__parameters:
    type: node
    modes:
      __DEFAULT__:
        ros__parameters:
          max_torque: 0.11

step to reproduce
zs@zs-vm-2204:~/zs_ws/ga_ros$ time ros2 service call /actuation/change_state lifecycle_msgs/ChangeState "{transition: {id: 1, label: configure}}"
waiting for service to become available...
requester: making request: lifecycle_msgs.srv.ChangeState_Request(transition=lifecycle_msgs.msg.Transition(id=1, label='configure'))

response:
lifecycle_msgs.srv.ChangeState_Response(success=True)

real    0m2.185s
user    0m0.459s
sys     0m0.086s

expected behavior
all nodes would be in DEFAULT mode, which is all inactivated
actual behavior

@ZhenshengLee, sorry for answering so late/slow on your issues. I'll analyze them in next week. Looks as if they are related, i.e., have the same root-cause.

@ZhenshengLee, sorry for answering so late/slow on your issues. I'll analyze them in next week. Looks as if they are related, i.e., have the same root cause.

Friendly ping...


Build argo-actions for local version

Description
Changes proposed in this pull request:

Build argo-actions for local version

Related issue(s)

maybe we will remove also the cloudsql-runner component as it is not maintained anymore?
Should we also add those images:

Published on Jun 14 by Capact.io in infra/json-go-gen
Published on Jun 14 by Capact.io in infra/jinja2
Published on Jun 14 by Capact.io in infra/graphql-schema-linter

Published on Jun 14 by Capact.io in infra/json-go-gen
Published on Jun 14 by Capact.io in infra/jinja2
Published on Jun 14 by Capact.io in infra/graphql-schema-linter

This three are not build at the time
make build-all-apps-images build-all-tests-images

I think we don't need jinja, as we always have fixed version in the manifests. Same with infra/json-go-gen and infra/graphql-schema-linter - we always use fixed image for all make targets and they are not Capact cluster components.

@pkosiec  @lukaszo so maybe we also shouldn't build helm-runner and terraform-runner images? as it's the same story


Implement SetLastError ourselves when allowMarshaling is false

It's worth noting that CsWin32's support for disabling marshalling won't actually be usable with DisableRuntimeMarshallingAttribute yet because of SetLastError: https://docs.microsoft.com/en-us/dotnet/standard/native-interop/disabled-marshalling#disabled-features
Originally posted by @alexrp in https://github.com/microsoft/CsWin32/issues/593#issuecomment-1179162557

https://www.pinvoke.net/default.aspx/kernel32.getlasterror
https://docs.microsoft.com/en-us/previous-versions/dotnet/netframework-3.0/ms182199(v=vs.80)?redirectedfrom=MSDN

@lonitra is this important for the winforms cswin32 project?

@lonitra is this important for the winforms cswin32 project?

I had not seen this attribute before, and it is not present in our codebase, so to my knowledge, I don't think so.

@lonitra is this important for the winforms cswin32 project?

I had not seen this attribute before, and it is not present in our codebase, so to my knowledge, I don't think so.

Does winforms want to use generated code from cswin32 without marshalling?

@lonitra WinForms uses SetLastError. CsWin32 does too. But as WinForms was moving from [DllImport] to [LibraryImport] which implements SetLastError in a trim-friendly way, CsWin32 will need to do the same.

@elachlan Would you be interested in implementing this? We'd need to do something fancy like LibraryImport does to ensure that we get to call GetLastError before the runtime has a chance on that same thread to call anything else that might call SetLastError.

I can give it a go. Do you know where I can find the .net implementation of it when using library import?
I figure I'd just check the setting and after the pinvoke call, add a call to GetLastError, then throw a win32exception if there is an error code.
https://docs.microsoft.com/en-us/dotnet/api/system.componentmodel.win32exception?view=net-6.0
Am I missing something?

Thanks!
SetLastError=true does not lead to an exception being thrown, and in this code change we do not propose to add that behavior.
DllImportAttribute.SetLastError indicates that the native method may call SetLastError. Some of these methods only call this in error conditions, and calling this method overwrites any previous value. But this value is in TLS (thread local storage) so each thread has its own copy. If you want to read the value set by a prior Win32 call, you must read it immediately after the call to ensure it wasn't overwritten by later code.
When the CLR emits the interop code for a DllImport method, if the SetLastError property is set to true, the CLR calls SetLastError(0) first, then invokes the method, and then calls GetLastError and stores the result in a CLR-specific TLS memory location so that even if the CLR internally makes other native calls, the user call to Marshal.GetLastWin32Error will always return the value from the last user-p/invoke on that thread.
I can't find it in the documentation now, but I believe when the CLR does do marshaling, it isn't exactly guaranteed that after an interop call completes that the CLR hasn't injected other interop calls for its own purposes, in which case the 'last error' may be overwritten by the time the first interop call returns. This means that if CsWin32 generates a method that tries to imitate the CLR's marshaling behavior, there isn't a guarantee that our own call to GetLastError will actually return the value from our own native call -- it might be from a CLR engine's call into Win32. So we have to be careful.
Now, by targeting net7.0 and using the [LibraryImport(SetLastError=true)] attribute, we can see what kind of code it generates:
internal static partial int GetTickCount()
{
    int __lastError;
    int __retVal;
    {
        System.Runtime.InteropServices.Marshal.SetLastSystemError(0);
        __retVal = __PInvoke();
        __lastError = System.Runtime.InteropServices.Marshal.GetLastSystemError();
    }

    System.Runtime.InteropServices.Marshal.SetLastPInvokeError(__lastError);
    return __retVal;
    // Local P/Invoke
    [System.Runtime.InteropServices.DllImportAttribute("Kernel32", EntryPoint = "GetTickCount", ExactSpelling = true)]
    static extern unsafe int __PInvoke();
}

Notice how the [DllImport] attribute in its generated code is on a local function, and the attribute lacks the SetLastError=true argument. So the extern API is no longer a method that others can call. The method is now a wrapper of the extern method. And that wrapper is equivalent to what the CLR would have produced if it were marshaling the call. This is what CsWin32 must do.
But here's the rub: while net7.0 evidently guarantees the correctness of this code, I don't think the CLR (e.g. net472) makes the same guarantee. Remember how I said the call into the extern method may inadvertently trigger the CLR to make other native calls before it returns? But I guess with Core CLR it's safe. Well, that and perhaps because LibraryImport always generates enough code to ensure that marshaling never occurs.
So... I think CsWin32 should emit this same kind of wrapper, but only when these conditions are true:

allowMarshaling has been set to false
we're targeting net7.0 or later.

Still want to tackle it, @elachlan?

Cool! Thanks for the write up, that is super helpful. Don't assign it to me just yet. I might focus on a couple of other issues first. But It seems mostly straight forward. I will see if I get time this week.

Would it be possible to at least not generate the SetLastError when allowMarshalling is false? I can write the surrounding code myself but SetLastError is making CsWIn32 unusable in project with marshalling disabled.

@AArnott any chance a beta build could be pushed sometime soon? Would love to test this particular feature out. 👀

I'd love for you to try it out. Would it work for you to consume the daily builds, per these instructions?
https://github.com/microsoft/CsWin32#consuming-daily-builds

Hmm, is there a way for me to query the versions of CsWin32 on that feed so I can update my Directory.Packages.props? 🤔

Good news: Updating CsWin32 to 0.3.31-beta (daily) and disabling runtime marshalling in all projects Just Works. 🙂


WFS support in iOS

See https://github.com/lutraconsulting/input/issues/880

Addressed in https://github.com/qgis/QGIS/pull/45737


nameko backport probably not working on MS Windows

As seen here: https://github.com/nameko/nameko/blob/v3.0.0-rc/nameko/cli/backdoor.py#L11
nameko is using which command to detect existence of particular executable.
This is very likely to fail on MS Windows as the command with this purpose is named where there.
Expected behaviour: Make sure, (possibly fixed) implementation runs on MS Windows too.

This is superseded by the decision in #625 to remove this command.
Worth noting though that Nameko does not officially support Windows. Anecdotally I gather that quite a few folks run it on Windows, but our CI currently tests on Linux only.

Closing as it will be resolved by #625 (removal of nameko backdoor subcommand removal).


Infused Reagents adding partial boosts to daily prep

Its adding 0.5 to infused reagents. from the intelligence mod when its a partial boost.

That wouldn't be from a partial boost. Does the character have rule elements adding fractional values to modifiers?

Does that include feats or general effects? if general effects there are none.

That wouldn't be from a partial boost. Does the character have rule elements adding fractional values to modifiers?

When my player used the partial boost to test the reagents it went from 14.5 to 14 then added it again. it went to 14.5
https://cdn.discordapp.com/attachments/828003871435391016/1165346429716406272/fvtt-Actor-arkhan-burnt-nose-kmXWi38XvM7CYF3u.json?ex=6546846f&is=65340f6f&hm=2bbfd3f0e7e911ca34d55d5ee979b44b7d46c08466721caddfe5671a790b457a&
here is a link to the json download to test.


[WIP]Whale in a Box

It amuses me to call this "Whale in a Box" because this is essentially howtowhale deconstructed for anyone to replicate and customize. When I tried making a single Docker image for people to run, I didn't like how it came out because it left important things like SSL certificates as an exercise to the reader.  It would be neat to update howtowhale to use this as a base, so that howtowhale runs on whaleinabox, proving that it works.
You can see the result of running the scripts at whaleinabox.carolynvs.com.
This is the initial prototype (or brain dump). Some instructions are in the README but comments/documentation needs to be fleshed out more, scripts tested/hardened, and possibly explain more potentially useful configuration settings.
I still need to work out what should be distributed in the repo for people to build locally and customize, and what should be published to Docker Hub. Right now I'm thinking that demo/jupyterhub, demo/proxy, demo/letsencrypt and demo/data could be published, leaving demo/web and demo/jupyter to be copied locally by the user and custom built.
I have placeholders for PowerShell scripts in the doc but won't add them until the bash scripts are how we'd like them. Then I'll make corresponding PowerShell scripts to match.
Closes #3.
/cc @willingc @rgbkrk

Whoa sweet

This is awesome

Cool. Was playing around with it a bit yesterday afternoon :smile:

🎉  http://whaleinabox.com/install.html 🎉
The base docker images are in https://github.com/carolynvs/whaleinabox and the user editable files are in https://github.com/carolynvs/whaleinabox-kit. Splitting them out made installation easier, as the user can just download or clone the kit repo.
I wasn't quite certain if there was value in having the jupyterhub-carina docker image available as a standalone docker image defined in this repository? It makes a few assumptions about being par of a larger environment, e.g. you have a domain name, are terminating SSL somewhere else, have mounted a volume to persist the db and cookie secret, etc. If you see value in the standalone docker image, let me know and we can hash out what needs to change so it can be moved here.


org.w3c.dom.DOMException: NAMESPACE_ERR on SIgn

Dear Team
I was getting the "org.w3c.dom.DOMException: HIERARCHY_REQUEST_ERR" after upgrading to Java 11 while signing a Document.
After going through the solution you have given, I changed the implementation by giving a New Document instead of the Original Document.
But, Now I am getting the "org.w3c.dom.DOMException: NAMESPACE_ERR" at "signature.sign(dsc)" Line in the below code:
public static String signCbCRDataXML(String filePath) {
    	OutputStream os = null;
    	try{
    		//Fetching the CbCR XML File and storing it in String
	    	DocumentBuilderFactory dbFactory = DocumentBuilderFactory.newInstance();
	    	dbFactory.setNamespaceAware ( false );
	    	Document doc = dbFactory.newDocumentBuilder().parse(new FileInputStream(filePath));
	    	String providerName = System.getProperty("jsr105Provider", "org.jcp.xml.dsig.internal.dom.XMLDSigRI");

	        XMLSignatureFactory fac = XMLSignatureFactory.getInstance("DOM", (Provider) Class.forName(providerName).newInstance());

	        // Next, create a Reference to a same-document URI that is an Object element and specify the SHA256 digest algorithm
	        DigestMethod digestMethod = fac.newDigestMethod(DigestMethod.SHA256, null);
	        Reference reference = fac.newReference("#CBC",digestMethod);
	        SignatureMethod signatureMethod = fac.newSignatureMethod("http://www.w3.org/2001/04/xmldsig-more#rsa-sha256", null);
	        CanonicalizationMethod canonicalizationMethod = fac.newCanonicalizationMethod(CanonicalizationMethod.INCLUSIVE, (C14NMethodParameterSpec) null);

	        // Create the SignedInfo
	        SignedInfo si = fac.newSignedInfo(canonicalizationMethod, signatureMethod, Collections.singletonList(reference));

	        // Create a KeyValue containing the RSA PublicKey that was generated
	        KeyInfoFactory kif = fac.getKeyInfoFactory();

	        // Set the x509Content from the Petronas Certificate to generate signature
	        FileInputStream fin = new FileInputStream(PETRONAS_CERTIFICATE_PATH);
	    	CertificateFactory cf = CertificateFactory.getInstance("X.509");
	    	List x509Content = new ArrayList();
	    	X509Certificate cert = (X509Certificate)cf.generateCertificate(fin);
	    	X509IssuerSerial issuer = kif.newX509IssuerSerial(cert.getIssuerDN().toString(), cert.getSerialNumber());
	    	x509Content.add(cert.getSubjectX500Principal().getName());
	        x509Content.add(issuer);
	        x509Content.add(cert);
	        X509Data xd = kif.newX509Data(x509Content);

	        // Create a RSA 2048 KeyPair
	        KeyPairGenerator kpg = KeyPairGenerator.getInstance("RSA");
	        kpg.initialize(2048);
	        KeyPair kp = kpg.generateKeyPair();
	        KeyValue kv = kif.newKeyValue(kp.getPublic());

	        XMLStructure content = new DOMStructure(doc.getDocumentElement());
	    	XMLObject obj = fac.newXMLObject(Collections.singletonList(content), "CBC", null, null);

	        // Create a KeyInfo and add the KeyValues to it
	    	List keyInfoItems = new ArrayList();
	    	keyInfoItems.add(xd);
	        keyInfoItems.add(kv);
	        KeyInfo ki = kif.newKeyInfo(keyInfoItems);
	        Document signedDocument = dbFactory.newDocumentBuilder().newDocument();
	        DOMSignContext dsc = new DOMSignContext(kp.getPrivate(), signedDocument);

	        // Create the XMLSignature and sign it
	        XMLSignature signature = fac.newXMLSignature(si, ki,Collections.singletonList(obj), null, null);
	        signature.sign(dsc);

	        TransformerFactory tf = TransformerFactory.newInstance();
	        Transformer trans = tf.newTransformer();

	        //Storing the Signed XML File in the File system
	        os = new FileOutputStream(filePath);
	        trans.transform(new DOMSource(signedDocument), new StreamResult(os));
    	}
}

I don't know. I don't recognise the error. You should probably ask either the developers of Apache Santuario, or the JDK itself.
There are issues related to the JDK's upgrade of Santuario https://bugs.openjdk.java.net/browse/JDK-8217878. The comments on the issue mention JDK-8218629 XML Digital Signature throws NAMESPACE_ERR exception on OpenJDK 11, works 8/9/10 which seems like it may be what you are experiencing.
Both Santuario and JDK seem to have downgraded the XML marshalling used by Santuario. The last comment requests the fix also to be applied to Java 11, but I am not sure if it has been done.
Which Java 11 version are you using? The last version of the OpenJDK seems to be 11.0.6. If the problem is still present in the lastest OpenJDK 11, maybe try Java 14.


Larger calls fail even with time-between-requests

I get the error below anytime I try to pull a larger dataset.   I tried to use the time-between-requests to address, but it still fails at random points.   I tried to pull the data in chucks but then run into issues with duplicate keys.
Traceback (most recent call last):---------------------| 13.0%
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/requests/models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/init.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/steve/Projects/nba-sql/stats/nba_sql.py", line 398, in
main(args, False)
File "/Users/steve/Projects/nba-sql/stats/nba_sql.py", line 333, in main
default_mode(settings, create_schema, request_gap, seasons, skip_tables, quiet or from_gui)
File "/Users/steve/Projects/nba-sql/stats/nba_sql.py", line 164, in default_mode
new_rows = play_by_play_requester.fetch_game(game_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/steve/Projects/nba-sql/stats/play_by_play.py", line 54, in fetch_game
response = requests.get(url=self.url, headers=headers, params=params_str).json()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/requests/models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Do you still get this error? Sometimes the NBA's API fails and only returns partial data, I'm not sure if this is a form of rate limiting or not.

Closing as this doesn't seem reproducible.


Adding non-dimension string coordinate gives different interpolation results

What happened?
When I interpolate on data by taking a mean along the y dimension the presence of a non-dimension string coordinate makes a difference to the result. I would expect this non-dimension coordinate to be passive.
What did you expect to happen?
I would expect the results to be independent of the non-dimension coordinate string_coord being present or not. But when string_coord is present I get:
<xarray.DataArray (x: 2, y: 2)>
array([[0. , 0.1],
       [0. , 0.1]])
Coordinates:
  * x             (x) int64 0 1
  * y             (x) float64 0.05 0.05
    string_coord  (x) <U21 '0' '1'

whereas after I uncomment the line where I add the non-dimension string coordinate I get the expected result:
<xarray.DataArray (x: 2)>
array([0.005, 0.005])
Coordinates:
    y        (x) float64 0.05 0.05
  * x        (x) int64 0 1

Minimal Complete Verifiable Example
import xarray as xr
import numpy as np
data = np.array([[0.0 , 0.1 ],
    [0.0, 0.1]])
data = xr.DataArray(
    data,
    coords=dict(
        x=[0,1],
        y=[0,1]
        )
)
data["string_coord"] = data.x.astype("str")
data_mean = data.mean(dim="y")
interp_data = data.interp(y=data_mean, method="linear")
print(interp_data)

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.
[X] Recent environment — the issue occurs with the latest version of xarray and its dependencies.

Relevant log output
No response
Anything else we need to know?


MM-19525 Trim contents of code spans

https://github.com/mattermost/marked/pull/14 changed the regex for code spans so that it captures more whitespace around the contents of the code span. Trimming it matches what the upstream library does now as well
Unit tests have been added as part of the marked PR.
marked PR: https://github.com/mattermost/marked/pull/15
Ticket Link
https://mattermost.atlassian.net/browse/MM-19525

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4057.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed

@hmhealey
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#4057-upstream-release-5.17-1572461957
Branch 'automated-cherry-pick-of-#4057-upstream-release-5.17-1572461957' set up to track remote branch 'release-5.17' from 'upstream'.
+++ Downloading patch to /tmp/4057.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/4057.patch

Applying: MM-19525 Trim contents of code spans
Using index info to reconstruct a base tree...
M	package-lock.json
M	package.json
Falling back to patching base and 3-way merge...
Auto-merging package.json
CONFLICT (content): Merge conflict in package.json
Auto-merging package-lock.json
CONFLICT (content): Merge conflict in package-lock.json
Patch failed at 0001 MM-19525 Trim contents of code spans
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU package-lock.json
UU package.json

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


libbsd: Darwin patch fails to apply after upgrade to 0.8.4 (probably 0.8.5 too?

Issue description
Not sure how difficult it'd be to fix this, but thought I'd note it somewhere.
Hydra build log
We recently bumped to 0.8.5 but I don't see any reason that would make the patch apply again (although I can't confirm this myself :)).
If it can't be resolved reasonably perhaps the patch should be dropped and "darwin" removed from meta.platforms?
EDIT: Looks like libbsd has a separate "port" for OSX: https://github.com/JackieXie168/libbsd which I've filed an issue on (like was done previously, apparently): https://github.com/JackieXie168/libbsd/issues/2
Steps to reproduce
Try to build libbsd on Darwin; look at Hydra/Travis build failures.
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Nix version: (run nix-env --version)
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)

Yeah, I have just been manually updating the patch since v0.8.3. The "correct" thing to do would be to get the patch applied upstream in libbsd. Perhaps freedesktop doesn't want it though.
/cc @JackieXie168


Open Secret edit form when tapping on a secret entry

In the navigation view, when tapping on a secret entry, load the edit form for that secret.

Implemented here: https://github.com/npalumbo/keepassui/pull/40


Updated wording for setup.sh file

Updated wording for setup.sh file
Additions

Added a better description to what setup.sh is doing

Removals
None
Changes
None
Testing
Checkout the readme
Review

@KimberlyMunoz
@sebworks
@anselmbradford

Preview
None
Preview this PR without the whitespace changes
Notes
None

:+1:


regression-test add debug log

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
When running stream load in regression tests and loading same file into different tables, "org.apache.http.ConnectionClosedException: Premature end of Content-Length delimited message body (expected: 129,961,387; received: 125,351,592)" maybe met.
This pr add log to try to debug this problem.
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


Herbert: Workflows: 2 weeks

Week 1: Check-In
I Need Help With:

Screen sharing zoom.

What went well?

Amazing, very patient Coaches
After initial struggles and some problems that still occasionally occur, I found git to be simple and intuitive.

What went less well?

I have to work on my schedule.

What lessons were learned:

How to use Git:

how to create a new branch and how to move from one to another.
how to add and pull a branch.
how to run CI checks and find errors with them

3.How to use GitHub better
4. How to write markdown
Sunday Prep Work
I did some reading from the suggested study. There are still some videos I am yet to watch

@HerbertHomolka1  You have a problem using Zoom?

@HerbertHomolka1  Please be carful when you add the Milestone it was agile development and not workflows


Do not change emoji colors in dark mode

Changing the color font dynamically depending on the selected theme makes sense, but not changing the emoji figures. The association I have emojis gets completely disrupted by modifying them. It makes it look very odd and some of them lose their meaning

@ctwhome  can I work on this issue


fix(userspace/libsinsp): fixed podman as user detection

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-ebpf

/area libscap

/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
The PR fixes a problem in podman as user detection; basically, in podman we have 2 runc matching cgroups layouts:
<EMAIL_ADDRESS>/machine.slice/libpod-$ID.scope/container

While we were able to parse the container ID just fine, we did not checked if the cgroup belonged to an user or root.
This behavior broke podman as user detection.
Moreover, fixed some inconsistencies in docker/base.cpp that was still using CT_DOCKER in various calls.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

fix(userspace/libsinsp): fixed podman as user detection

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @gnosek

LGTM label has been added.  Git tree hash: 8c8a14c0d31139138118ea16aa3704efe3c7c6ba

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Update sv_se.json

Medvärkande -> Medverkande

Thank you, I'll add your username to the CREDITS, since it wasn't included in this PR.


Stop using ovn-scale-test (rally-ovs) and browbeat.

Discussed with @LorenzoBianconi offline, will merge this.


Add patch to build gcc-4.6.4 with gcc-5 and newer

Building gcc-4.6.4 (the bundled gcc in the toolchain) with gcc-5 or newer will cause the build process to fail:
In file included from /home/tuankiet65/sources/toaruos/toolchain/tarballs/gcc-4.6.4/gcc/cp/except.c:912:0:
cfns.gperf: At top level:
cfns.gperf:101:1: error: ‘gnu_inline’ attribute present on ‘libc_name_p’
cfns.gperf:26:14: error: but not here

Relevant issue: DragonFlyBSD/DPorts#136
More infomation and the patch to apply can be found here: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00375.html

The linked patch modifies both the header file where this problem exists as well as the gperf source, which can cause Make to try to rebuild the header with gperf. To avoid this (as gperf is not in our dependency list and I do not wish to add it) I am only patching the header.

... which did not work as intended. Okay, we'll add gperf...


Codecraft Requests - MQTT over WIFI

Set Wifi
Image:

Code
const char ssid[] = SECRET_SSID;
const char pass[] = SECRET_PASS;

This one is new, from scratch, because if you really drag the corresponding command from tinkergen you start don't understanding anything lol
Image

(ping me if you want the svg, I suspect you use the scratch engine to produce them though)
Code
#include <MQTT.h>

#define BROKER_IP    "broker.hivemq.com"
#define DEV_NAME     "mqttdevice"
#define MQTT_USER    "mqtt_user"
#define MQTT_PW      "mqtt_password"

We can subscribe to a new channel by creating it, a "payload" appears in the corresponding palette.
The payload will later allow us to read and analyse the value arriving.

Publishing on a channel affects the declaration of the publishing data
#include <MQTT.h>

#define MQTT_PUB     "/pubChannel"

voide setup(){

}

void loop(){

 client.publish(MQTT_PUB, data);
}

We add to the connect() function this mqtt part to make things simpler.
Another way could be using two different functions.
What do you think?
void connect() {
  Serial.print("checking wifi...");

  while (WiFi.status() != WL_CONNECTED) {
    Serial.print(".");
    delay(1000);
  }
  Serial.print("\nconnecting...");

  while (!client.connect(DEV_NAME, MQTT_USER, MQTT_PW)) {

    Serial.print(".");
    delay(1000);

  }

  Serial.println("\nconnected!");

}

void setup() {
  Serial.begin(115200);
  WiFi.begin(ssid, pass);

  client.begin(BROKER_IP, 1883, net);
  client.onMessage(messageReceived);

  connect();

}

The messageReceived function is called whenever a message is received. We can use the payload to compare with other elements of the code.
void messageReceived(String &topic, String &payload) {

}

client.subscribe(MQTT_SUB);  has to be addedd to the connect() function, the #define MQTT_SUB  as an absolute declaration.

void connect() {
  Serial.print("checking wifi...");

  while (WiFi.status() != WL_CONNECTED) {
    Serial.print(".");
    delay(1000);
  }
  Serial.print("\nconnecting...");

  while (!client.connect(DEV_NAME, MQTT_USER, MQTT_PW)) {

    Serial.print(".");
    delay(1000);

  }

  Serial.println("\nconnected!");
  client.subscribe(MQTT_SUB); //SUBSCRIBE TO TOPIC /hello

}

I  would need a prototype to make a better example of messageReveived()

Done


Can’t use IP DNS server as a fallbak

on my configuration, IP DNS and DNS over TLS doesn’t work together.
Created a new entry for my customized DNS. Using a self-built AdGuard for DNS over TLS, and try to setup <IP_ADDRESS> as as failover.
DoT works along, but whenever I added <IP_ADDRESS> under server section, the network stop working. No page loaded.
Tried DoH as well, same issue
either DoH or DoT, my AdGuard works along without any IP entered but can’t work together.
I also tried self-signed profile which seem having same issue. Not sure if it’s Apple side issue or what. Those build in configurations work though.
Since this app doesn’t have logs, so I can’t really debug it myself. Anyone have similar issue and know the solution?

IP addresses on the servers section are tied to the DNS-over-TLS/HTTPS settings section's server name or URL. For example, to use Cloudflare's <IP_ADDRESS> as DoT, you must specify cloudflare-dns.com as the server name on the DNS-over-TLS settings section. So, you cannot use your customized DNS server and <IP_ADDRESS> at the same time.

Is that mean the DoT/DoH hostname need to match the IP address? What if I use my own DoT? I need to also use my own DNS's IP for it?
I believe the failback should be another DNS, so for example if I use my own DoT, dns.example.com, I would like to have a failover DNS such as <IP_ADDRESS> so in case my DoT fail or for whatever reason not working, I still have fallback <IP_ADDRESS> to resolve domain without any issue. (Maybe a warning but not completely lost internet)

Is that mean the DoT/DoH hostname need to match the IP address?

No. In case of DoT, it's a domain name of the certificate, and in case of DoH, it's an endpoint of DoH resolver.
For example, Cloudflare is using cloudflare-dns.com for their DoT's certificate as described here, and https://cloudflare-dns.com/dns-query as an endpoint of DoH as described here.

I believe the failback should be another DNS, so for example if I use my own DoT, dns.example.com, I would like to have a failover DNS such as <IP_ADDRESS> so in case my DoT fail or for whatever reason not working, I still have fallback <IP_ADDRESS> to resolve domain without any issue. (Maybe a warning but not completely lost internet)

I believe that iOS's NEDNSSettingsManager API doesn't support a fallback like that. If so, we can't do it. I'll read the documentation again.


ServerHardware #GetServerHardwareList Issues

In ServerHardware #GetServerHardwareList, I think there are a few issues:

The REST here call doesn't handle errors
This section doesn't appear to do anything useful (does it?). Maybe it was left in after a refactor?
Unmarshalling the data doesn't set the Client property, so many subsequent calls on the objects returned (e.g. PowerOff()) will fail unless you set the Client property manually.

The fix for this issue is provided in https://github.com/HewlettPackard/oneview-golang/pull/137.
Closing this issue now. Feel free to reopen if the issue still exists.


fix: resolve incompatibilities with DHIS 2.38

test: resolve incompatibilities with DHIS 2.38
ci: run tests again DHIS 2.36, 2.37, and 2.38

Indeed. I'll leave a comment about this in the commit message.


[FLINK-36452][mysql] Replace ArrayList with TreeSet to speed up searching of FinishedSnapshotSplitInfo for specific chunkKey.

Avoid EOFException for table with a large amount of data.

method getExecutorService()  is not found in binlogReader, method BinlogSplitReader#getExecutorService should commit

method getExecutorService() is not found in binlogReader, method BinlogSplitReader#getExecutorService should commit

I test this PR, it can indeed solve the problem of cdc EOFException in our production Mysql.


Add support for BigQuery customer-managed encryption keys

Copied from original issue: https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2801
@choendenFebruary 6, 2018 4:33 PM
BigQuery customer-managed encryption keys allow users to specify a Cloud KMS key to protect their BigQuery table.
API:
Jobs: https://cloud.google.com/bigquery/docs/reference/rest/v2/jobs
In a job, there is a destinationEncryptionConfiguration field, which indicates which Cloud KMS key should be used for the destination.
Tables: https://cloud.google.com/bigquery/docs/reference/rest/v2/tables
In a table, there is a encryptionConfiguration field, which indicates which Cloud KMS key protects (or should protect in case of CreateTable) a BigQuery table.
These are the main APIs that are required for day-to-day interaction.
With lower priority, support for getServiceAccount would also be nice: https://cloud.google.com/bigquery/docs/reference/rest/v2/projects/getServiceAccount
Note that unlike the other methods mentioned above, this would generally only be called once and the resulting value (the email address) does not change - so it can easily also be called from UI/API/CLI without much hindrance (hence lower priority).

Actually, upon trying the client libraries, it looks like the encryption configuration can be set directly as part of the json object and changes for this are not required.

Oh, great!


JS libraries import with an incorrect character set

From messages.js in dashboard.nsf:

This looks to be Domino's fault originally: the files are exported with mangled characters. I'm not sure what a good approach is here - they're already imported back just as byte streams directly, so the code shouldn't care what the character set is to begin with. Maybe ICU4J would have some mechanism to properly re-encode them. UTF? LMBCS?


Get Event for incoming client request

Is their any event in YARP which gets fired when incoming client request comes ? Event should get fired before forwarding the request to the destination If configuration exists else add dynamic configuration and then forward the request.

What are you trying to achieve?
If you want to add your own logic before the the forwarding happens, you can insert a middleware before/in MapReverseProxy. See an example from the samples.

I want to start my application (ASP.NET Web application) programmatically If not started before the forwarding happens.

You want to lazily spin up the destination server/app as the request for it comes into YARP?
You could do so from a middleware I mentioned above. In the sample, the custom step is filtering destinations, but you could use it to start a service instead.

I will also get the client request information in the middleware. As I also want to validate the client request is of specific pattern or not.

The HttpContext contains the request information.

Ok, Thanks. Will check and see If I can adapt as per my needs.


Additional custom property types

Is your feature request related to a problem? Please describe.
We have a number of custom properties we would like to tag tables with.
Open Metadata only supports string, markdown and integer types for custom properties.
Describe the solution you'd like
We would like to add a boolean (true / false) as well as a list type (where a list of possible options is selected from a dropdown). The list type should be single or multi select capable, similar to selecting tags.
Whilst it is possible to use tags for multi-select options, we see the types of data we would like to add into custom properties are more technical in nature and not the kind of thing normal users should see.
Describe alternatives you've considered
As an interim measure we are storing Y/N and entering strings into string fields. This however is prone to user error.
Additional context
None

I ran into a similar Issue. For one use case, we'd like to store a list of urls for each table, and so a custom property seems like a good fit. The API allows me to add this by specifying an appropriate JSON schema, and it works as long as OM is up, but then on restart OM has boot errors and poor behavior.
INFO [2023-08-07 16:30:26,542] [main] o.o.s.TypeRegistry - Adding custom property table.customProperties.customTableProperty with JSON schema "#" : {"$comment":"@om-field-type","description":"A String type.","type":"string"}
INFO [2023-08-07 16:30:26,542] [main] o.o.s.TypeRegistry - Adding custom property table.customProperties.foo with JSON schema "#" : {"$comment":"@om-field-type","description":"A String type.","type":"string"}
INFO [2023-08-07 16:30:26,542] [main] o.o.s.TypeRegistry - Adding custom property table.customProperties.markfoo with JSON schema "#" : {"$comment":"@om-field-type","description":"Text in Markdown format.","type":"string"}
ERROR [2023-08-07 16:30:26,558] [main] o.o.s.r.t.TypeResource - Error loading type table
java.lang.NullPointerException: null
        at org.openmetadata.service.TypeRegistry.addCustomProperty(TypeRegistry.java:70)
        at org.openmetadata.service.TypeRegistry.addType(TypeRegistry.java:56)
        at org.openmetadata.service.jdbi3.TypeRepository.updateTypeMap(TypeRepository.java:86)
        at org.openmetadata.service.jdbi3.TypeRepository.addToRegistry(TypeRepository.java:76)
        at org.openmetadata.service.resources.types.TypeResource.lambda$initialize$1(TypeResource.java:118)
        at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
        at org.openmetadata.service.resources.types.TypeResource.initialize(TypeResource.java:101)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at org.openmetadata.service.resources.CollectionRegistry.createResource(CollectionRegistry.java:260)
        at org.openmetadata.service.resources.CollectionRegistry.registerResources(CollectionRegistry.java:172)
        at org.openmetadata.service.OpenMetadataApplication.registerResources(OpenMetadataApplication.java:438)
        at org.openmetadata.service.OpenMetadataApplication.run(OpenMetadataApplication.java:190)
        at org.openmetadata.service.OpenMetadataApplication.run(OpenMetadataApplication.java:124)
        at io.dropwizard.cli.EnvironmentCommand.run(EnvironmentCommand.java:67)
        at io.dropwizard.cli.ConfiguredCommand.run(ConfiguredCommand.java:98)
        at io.dropwizard.cli.Cli.run(Cli.java:78)
        at io.dropwizard.Application.run(Application.java:94)
        at org.openmetadata.service.OpenMetadataApplication.main(OpenMetadataApplication.java:482)

curl 'http://<IP_ADDRESS>:8585/api/v1/tables/e9c7a59a-3222-4a50-942a-06c4199003b1' \                                                                                main ⬇ ✚ ✱
  -X 'PATCH' \
  -H 'Accept: application/json, text/plain, */*' \
  -H 'Accept-Language: en-US,en;q=0.9' \
  -H 'Authorization: Bearer ....
  -H 'Content-Type: application/json-patch+json' \
  --data-raw '[{"op":"add","path":"/extension/urls","value":["fools", "tool"]}]' \
  --compressed \
  --insecure

{"code":400,"message":"Unknown custom field urls"}%


Filter Idea

So I was thinking that maybe adding a check box for extended or basic parts would be very useful. since that way we can narrow down the search when we have some wiggle room, like E24 resistors seem all to be basic but I don't remember all the values that well but if I filter for basic parts I could narrow it down a lot, also JLC has some odd categories , there are basic leds in like 3-4 places and finding those gems would be that much easier with a basic check box.

The first parametry in property filter isn't enough?

i was blind and i did not see it, well in a way i was strugling a bit but i found it now THX


Add keyboard modifier example (#1656)

This PR adds a small example that shows how to use Keyboard modifiers, as shown in this snippet.


Cloudflare Tunnel client outdated

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
After activating the Cloudflare tunnels, it show in the Cloudflare interface that the client is outdated
Can we update it mannually (how ?) or this will be fixed in next release ?

🐻 Uptime-Kuma Version
1.14.1
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Vivaldi last version
🐋 Docker Version
Docker Version 20.10.14
🟩 NodeJS Version
No response

@computman007 I think this is a CI issue, I just submitted a PR to take care of this in the future. Ideally every time a new release of Uptime-Kuma is created the latest version of cloudflared and other dependencies are installed.

TBH, unless uptime-kume gets released by building on a local dev machine, the docker cache can mostly be ignored :thinking:.
In the case of GHA the environment should be clean by default for each run, hence why there are cache actions :wink:
Anyway, this still is indeed a CI issue in a way, that it depends on when the last image was built which version will get downloaded (given how this script works).
I don't really have a solid solution for this beyond something similar to what you can do with Renovate-Bot and the RegexMatcher directive, and thereby pinning which version gets installed as well as updating it once there is an update:
https://github.com/k3rnelpan1c-dev/uptime-kuma-helm/blob/3f1ff2a97889003e00ae6a0df2bdd944c472831c/container/Containerfile#L8-L9
However, that won't save you if there is a release in between releases.
Allow me question though: given that cloudflared seems to warn you that a version is outdated just cause you are not on the latest (2022.4.0, in the screenshot, is 18 days old and 2022.4.1 is the latest with an age of 13 days), yet it still working fine, is it really that important?

It's not a big issue as it's working. 18 days are not important if you always keeiping up to date the client .


Add JSON to demo radio and checkbox description text

What is the context of this PR?
Adds a new survey JSON to demo the feature implemented in https://github.com/ONSdigital/eq-survey-runner/pull/471
How to review

Go to the dev page
Select the test_radio_checkbox_descriptions.json schema
Complete survey and verify the appropriate checkbox and radio option labels appear on the summary page (not the description text)

We're going to have a discussion today about tests and test data, so we'll review this after that.

I'm happy with this to go in as is. I think we need a strategy for test schemas going forward. We have a backlog item for this - https://trello.com/c/z9iikwAZ/271-create-a-separation-for-business-survey-configuration-and-test-survey-configuration


[FEATURE] Implement Github Pages

It goes without saying a webfacing, clear and precise webpage containing information about both the YSCEDC and YSCE is of our best interest, and I was reminded of this thanks to U2Fly bringing up a Pull Request - PR #73.
At this time, utilizing a service like Github Pages, which allows us one page per organization, and one page per project, will allow us, for the time being, to have a public facing webpage for YSCE, until such time we can establish a proper webhost and domain.
I'll begin work on a webpage over the coming days, but I am leaving this FR here as a proper notice of intent for a project.
Links:
https://pages.github.com

Update on this issue - I have started work on a static page using TeleportHQ for YSCE, it is more or less ready to go live, but will need a few things tended to before going live.


Adjust Queue Message Invisibility Timeout

Is there a way to set the invisibility timeout on the queue messages?  The default of 30 seconds is too short for what we are doing.

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

ID: 141a682b-afaa-a36c-f3cc-baab732b38a9
Version Independent ID: 6202c305-b765-6237-fb49-18ad0fa7374c
Content: Load files from Azure Blob storage, Azure Data Lake Storage Gen1 (limited), or Azure Data Lake Storage Gen2 using Auto Loader - Azure Databricks - Workspace
Content Source: databricks/spark/latest/structured-streaming/auto-loader.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@dereklokey Thanks for the question. We're investigating this and will get back to you shortly.

Fantastic, thank you!
Get Outlook for iOShttps://aka.ms/o0ukef

From: BhargaviAnnadevara-MSFT<EMAIL_ADDRESS>Sent: Saturday, November 21, 2020 12:49:50 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Derek<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Adjust Queue Message Invisibility Timeout (#66431)
@dereklokeyhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdereklokey&data=04|01||597fef2e67bc44a56a3708d88de9a516|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637415381913215399|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=wkn%2BNemI%2FmMC5FpgCU8sfsucAdhWxQHfmM9D3U99wwo%3D&reserved=0 Thanks for the question. We're investigating this and will get back to you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F66431%23issuecomment-731519019&data=04|01||597fef2e67bc44a56a3708d88de9a516|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637415381913225396|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PJ0I8XU%2BjYdxnisvQOqAhreYCMJ1fRn%2FbDKHSv3yV40%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABPE4WGDYDSGXNK6SRGALL3SQ5PI5ANCNFSM4T5AYVWA&data=04|01||597fef2e67bc44a56a3708d88de9a516|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637415381913235386|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=b9wWGYZWskMeQ%2BdFKLV%2BvCVhISYGuG6GEuPmZhE4Wk8%3D&reserved=0.

@mssaperla - Can you please take a look and share your thoughts.

Hi @dereklokey – Apologies for the delay. Can you let us know whether this is still an open question?

Thanks for your dedication to our documentation. Unfortunately, we have been unable to address your issue and apologize for the delayed response. We are closing this issue, but if you feel that it's still a concern, please let us know directly at<EMAIL_ADDRESS>#please-close

#please-close


SMTP Auth

added option to handle smtp authorization. There exists stuff that requires SMTP authorization, and cannot use no auth at all. So I added an option to enable this using options:
--smtp-auth - enable SMTP authorization
--smtp-username, --smtp-password

Thaks for the PR! I'm curious, what client are you using maildump with that requires smtp auth?

Hey, I know it's been a long time, but if you rebase this (and adapt to maildump now being Python 3.6+ only) I'll promise to merge it quickly :)

Hi,
I've refreshed this pull request. Please review :)


Running behind basic HTTP auth proxy

Hello and thanks for your great work. I am trying to run this on my node, but I want some sort of authorization. So I put it behind an Apache proxy where I set the auth. It works on the standard requests, but apparently the AJAX requests to /api/... do not have the auth headers and thus are receiving 401. If I disable the auth, it works fine.
I am not sure if it's even solvable with the current setup so I'd like to get some ideas how to handle the authorization. Anyone tried something like that?
Here is the relevant part from my Apache conf:
ProxyPreserveHost On
ProxyPass "/" "http://localhost:8001/"
ProxyPassReverse "/" "http://localhost:8001/"

<Proxy *>
    AuthType Basic
    AuthName "LND explorer"
    AuthBasicProvider file
    AuthUserFile "/opt/lnd-explorer/passwords"
    Require valid-user
</Proxy>
...

Network connections are being made with the fetch API.  This functionality should be able to be supported by adding credentials: 'same-origin' option to the fetch calls.
On a side node, a bit of cleanup should be done to make that standardized across the app.  Right now, fetches are being directly invoked by container components, so the change would need to be made in a bunch of files.

Thanks for the pointer. I submitted PR for your consideration.


is mobile version available?

thanks in advance

Thank you for the idea! We're still working on adding more features to the PC version (chat, private rooms, etc). Making it work on mobile browsers could be the next step after those. That said, feel free to contribute or share more thoughts!

Hey Logan, thanks for the interest in contributing, any help is extremely appreciated!
I think we can start with something basic and add things gradually along the way. Having a virtual joy-stick to move around sounds like a reasonable first step. Please don't hesitate to share your thoughts and let us know if you need any help!

Thanks to @jga-dev, we now have a joystick for mobile usage :)


Error - unable to load ltui.lcurses - undefined symbol COLORS

I'm having trouble testing and using ltui.  I hope the information below is enough to be helpful.
System:
Chromebook with Linux - Acer 514
uname -a output:
Linux local.______ 4.19.16-02893-g2cf2c17c8a43 #1 SMP PREEMPT Sat Jan 19 09:52:07 PST 2019 x86_64 GNU/Linux
I have compiled and installed:
lua 5.3.5
luarocks 3.0.4
Rocks installed:
lcurses
9.0.0-1 (installed) - /usr/local/lib/luarocks/rocks-5.3
ltui
1.3-1 (installed) - /usr/local/lib/luarocks/rocks-5.3
Error output:
Lua 5.3.5  Copyright (C) 1994-2018 Lua.org, PUC-Rio

local foo = require("ltui")
error loading module 'ltui.lcurses' from file '/usr/local/lib/lua/5.3/ltui.so':
/usr/local/lib/lua/5.3/ltui.so: undefined symbol: COLORS
stack traceback:
[C]: in ?
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/curses.lua:31: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/application.lua:30: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui.lua:31: in main chunk
[C]: in function 'require'
stdin:1: in main chunk
[C]: in ?

ncurses version:
$ dpkg -l | grep curses
libncurses5:amd64         6.0+20161126-1+deb9u2  amd64    shared libraries for terminal handling
libncurses5-dev:amd64  6.0+20161126-1+deb9u2  amd64    developer's libraries for ncurses
libncursesw5:amd64      6.0+20161126-1+deb9u2  amd64    shared libraries for terminal handling (wide character support)
ncurses-base                 6.0+20161126-1+deb9u2    all         basic terminal type definitions
ncurses-bin                    6.0+20161126-1+deb9u2  amd64    terminal-related programs and man pages
ncurses-term                  6.0+20161126-1+deb9u2    all         additional terminal type definitions

/usr/local/lib/lua/5.3/ltui.so: undefined symbol: COLORS

The definition of the COLORS macro was not found in your curses.h
I tried to modify the ltui source code, and I have done compatibility with this problem. You can try to install ltui again through luarocks.

Thank you for your help. Unfortunately I'm getting a COLOR_PAIRS error now.
Lua 5.3.5  Copyright (C) 1994-2018 Lua.org, PUC-Rio

local foo = require("ltui")
error loading module 'ltui.lcurses' from file
'/usr/local/lib/lua/5.3/ltui.so':
/usr/local/lib/lua/5.3/ltui.so: undefined symbol: COLOR_PAIRS
stack traceback:
[C]: in ?
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/curses.lua:31: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/application.lua:30: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui.lua:31: in main chunk
[C]: in function 'require'
stdin:1: in main chunk
[C]: in ?

On Sat, Mar 16, 2019 at 9:32 PM ruki<EMAIL_ADDRESS>wrote:

/usr/local/lib/lua/5.3/ltui.so: undefined symbol: COLORS
The definition of the COLORS macro was not found in your curses.h
I tried to modify the ltui source code, and I have done compatibility with
this problem. You can try to install ltui again through luarocks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tboox/ltui/issues/5#issuecomment-473610247, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYziqtfCzaU-7x1am_FTHjltHAKqV3Kks5vXalBgaJpZM4b35WK
.

--

mark

    /usr/local/lib/lua/5.3/ltui.so: undefined symbol: COLOR_PAIRS

I have fixed it. You can reinstall ltui and try it again.

If there is anything I can do on my end to help troubleshoot, please let me know.  If I can compile ncurses with certain flags set, or compile lcurses, etc.. I'm happy to help.
New error: undefined symbol: winnstr
Lua 5.3.5  Copyright (C) 1994-2018 Lua.org, PUC-Rio

local foo  = require ("ltui")
error loading module 'ltui.lcurses' from file '/usr/local/lib/lua/5.3/ltui.so':
/usr/local/lib/lua/5.3/ltui.so: undefined symbol: winnstr
stack traceback:
[C]: in ?
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/curses.lua:31: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui/application.lua:30: in main chunk
[C]: in function 'require'
/usr/local/share/lua/5.3/ltui.lua:31: in main chunk
[C]: in function 'require'
stdin:1: in main chunk
[C]: in ?

You can attempt to use xmake to build ltui and run tests.
$ cd ltui
$ xmake
$ xmake run test dialog

I git cloned your code again and followed the instructions.  Here is the output:
Script started on Sun 17 Mar 2019 03:13:34 PM UTC
markvoltz@local: /ltui$ xmake
checking for the architecture ... x86_64
note: try installing these packages (pass -y to skip confirm)?
in xmake-repo:
-> ncurses 6.1
please input: y (y/n)
y
=> installing ncurses-6.1 .. ok
[  0%]: compiling.release src/core/lcurses/lcurses.c
[100%]: linking.release libltui.so
markvoltz@local: /ltui$ xmake run test dialog
error: execv(/usr/local/bin/lua tests/dialog.lua) failed(-1)!
markvoltz@local: /ltui$ exit
-- I also ran the script using 'lua tests/dialog.lua' --
Script done on Sun 17 Mar 2019 03:14:01 PM UTC
markvoltz@local: /ltui$ lua tests/dialog.lua
Segmentation fault
I will also try this from another machine that is not a Chromebook.  Maybe the results will be different.

you can use luajit to run tests.
xmake f --luajit=y -c
xmake
luajit tests/dialog.lua

That's better.  See below. The odd characters may have something to do with
the ChromeOS terminal program.  I can look into that.
Thank you for your help.
lqmain dialogqk
xThe project focuses on making development and building easier and
provides many features (.e.g package, install, plugin, macro, action,
option, task ...), so that ax
xny developer can quickly pick it up and enjoy the productivity boost when
developing and building project.
x
xlqkx
xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx                                                         lqk
          xx

xx                                                         xhello ltui! (
https://tboox.org)               x
xx
xx                                                         xA
cross-platform terminal ui library based on x
xx
xx                                                         xLua
x
xx
xx                                                         x
x
xx
xx                                                         x
< Yes >     < No >              x
xx
xx                                                         mqj
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xx
          xx

xmqjx
x
          x

x                                        < Tips >                < Input

          < Help >                < Quit >

            x

mqj
On Sun, Mar 17, 2019 at 5:30 PM ruki<EMAIL_ADDRESS>wrote:

you can use luajit to run tests.
xmake f --luajit=y -cxmakeluajit tests/dialog.lua
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tboox/ltui/issues/5#issuecomment-473721639, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYzipFvzWgqAmQq5ZG8Ksui4noZ06S9ks5vXsH-gaJpZM4b35WK
.

--

mark

? Can you add -v -D arguments to xmake and give me the full error log?
xmake f --luajit=y -c -v -D
xmake -v -D
luajit tests/dialog.lua

here you go. attached.
On Sun, Mar 17, 2019 at 9:31 PM ruki<EMAIL_ADDRESS>wrote:

? Can you add -v -D arguments to xmake and give me the full error log?
xmake f --luajit=y -c -v -Dxmake -v -Dluajit tests/dialog.lua
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tboox/ltui/issues/5#issuecomment-473747821, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYzikkIKAEqSVbtJ76jSTMpkVijFnayks5vXvqYgaJpZM4b35WK
.

--

mark

？ I didn't see anything.

here it is in text. I don't know how to get rid of the escape codes.
---luajit output
checking for the architecture ... x86_64
[0mchecking for the unzip ... [38;5;2m/usr/bin/unzip [0m
[0mchecking for the git ... [38;5;2m/usr/bin/git [0m
[0mchecking for the gzip ... [38;5;2m/bin/gzip [0m
[0mchecking for the tar ... [38;5;2m/bin/tar [0m
[0mchecking for the ping ... [38;5;2m/bin/ping [0m
pinging for the host(gitlab.com) ... 189 ms
pinging for the host(github.com) ... 65 ms
pinging for the host(gitee.com) ... 304 ms
finding luajit from xmake ..
[0mchecking for the xmake::luajit ... [38;5;2m2.0.5 [0m
{
links =
{
luajit
}
,   linkdirs =
{
/home/markvoltz/.config/.xmake/prefix/linux/x86_64/release/lib
}
,   version = 2.0.5
,   includedirs =
{
/home/markvoltz/.config/.xmake/prefix/linux/x86_64/release/include/luajit
}
}
finding ncurses from xmake ..
[0mchecking for the xmake::ncurses ... [38;5;2m6.1 [0m
{
links =
{
ncurses
,   form
,   panel
,   menu
}
,   linkdirs =
{
/home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/lib
}
,   version = 6.1
,   includedirs =
{
/home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/include/ncurses
,
/home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/include
}
}
configure
{
luajit = true
ccache = true
clean = true
plat = linux
kind = static
buildir = build
mode = release
arch = x86_64
host = linux
}
---xmake output
configure
{
luajit = true
ccache = true
mode = release
plat = linux
kind = static
buildir = build
arch = x86_64
clean = true
host = linux
}
[0mchecking for the gcc ... [38;5;2m/usr/bin/gcc [0m
[0m [2mchecking for the c compiler (cc) ... [38;5;2mgcc [0m
[0mchecking for the ccache ... [38;5;1mno [0m
[0mchecking for the /usr/bin/gcc ... [38;5;2mok [0m
[0m [2mchecking for the flags (-Wno-error=deprecated-declarations) ...
[38;5;2mok [0m
[0m [2m> gcc -Wno-error=deprecated-declarations -m64 -I/usr/local/include
-I/usr/include [0m
[0m [2mchecking for the flags (-fno-strict-aliasing) ... [38;5;2mok [0m
[0m [2m> gcc -fno-strict-aliasing -m64 -I/usr/local/include -I/usr/include
[0m
[0m [2mchecking for the flags (-Wno-error=nullability-completeness) ...
[38;5;1mno [0m
[0m [2m> gcc -Wno-error=nullability-completeness -m64 -I/usr/local/include
-I/usr/include [0m
[0m [38;5;3mcheckinfo: [0;2m
/home/markvoltz/.local/share/xmake/core/base/os.lua:705: cc1: error:
-Werror=nullability-completeness: no option -Wnullability-completeness
stack traceback:
[C]: in function 'error'
[/home/markvoltz/.local/share/xmake/core/base/os.lua:705]: in function
'raise'
[...markvoltz/.local/share/xmake/core/sandbox/modules/os.lua:311]: in
function 'runv'
[...local/share/xmake/modules/detect/tools/gcc/has_flags.lua:48]: [0m
[0m [2mchecking for the flags (-fomit-frame-pointer) ... [38;5;2mok [0m
[0m [2m> gcc -fomit-frame-pointer -m64 -I/usr/local/include -I/usr/include
[0m
[0m [2mchecking for the flags (-Wno-expansion-to-defined) ... [38;5;2mok [0m
[0m [2m> gcc -Wno-expansion-to-defined [0m
[0mchecking for the g++ ... [38;5;2m/usr/bin/g++ [0m
[0m [2mchecking for the shared library linker (sh) ... [38;5;2mg++ [0m
On Sun, Mar 17, 2019 at 10:10 PM ruki<EMAIL_ADDRESS>wrote:

？ I didn't see anything.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tboox/ltui/issues/5#issuecomment-473754179, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYzioRP_5Ni4ihhvdDz92cnrnwy_aXlks5vXwOTgaJpZM4b35WK
.

--

mark

I still don't see the complete log error message. I don't know what problem you are currently experiencing. Is there strange text in running dialog.lua?

Replying via email is not working out.  I have pasted directly into github.  Sorry for the time wasted.
markvoltz@local:~/ltui$ xmake f --luajit=y -c -v -D
checking for the architecture ... x86_64
checking for the unzip ... /usr/bin/unzip
checking for the git ... /usr/bin/git
checking for the gzip ... /bin/gzip
checking for the tar ... /bin/tar
checking for the ping ... /bin/ping
pinging for the host(github.com) ... 37 ms
pinging for the host(gitlab.com) ... 59 ms
pinging for the host(gitee.com) ... 311 ms
finding luajit from xmake ..
checking for the xmake::luajit ... 2.0.5
{
links =
{
luajit
}
,   linkdirs =
{
/home/markvoltz/.config/.xmake/prefix/linux/x86_64/release/lib
}
,   version = 2.0.5
,   includedirs =
{
/home/markvoltz/.config/.xmake/prefix/linux/x86_64/release/include/luajit
}
}
finding ncurses from xmake ..
checking for the xmake::ncurses ... 6.1
{
links =
{
ncurses
,   form
,   panel
,   menu
}
,   linkdirs =
{
/home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/lib
}
,   version = 6.1
,   includedirs =
{
/home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/include/ncurses
,   /home/markvoltz/ltui/.xmake/prefix/linux/x86_64/release_543d989b/include
}
}
configure
{
luajit = true
ccache = true
clean = true
plat = linux
kind = static
buildir = build
mode = release
arch = x86_64
host = linux
}

markvoltz@local:~/ltui$ xmake -v -D
configure
{
luajit = true
ccache = true
mode = release
plat = linux
kind = static
buildir = build
arch = x86_64
clean = true
host = linux
}
checking for the gcc ... /usr/bin/gcc
checking for the c compiler (cc) ... gcc
checking for the ccache ... no
checking for the /usr/bin/gcc ... ok
checking for the flags (-Wno-error=deprecated-declarations) ... ok

gcc -Wno-error=deprecated-declarations -m64 -I/usr/local/include -I/usr/include
checking for the flags (-fno-strict-aliasing) ... ok
gcc -fno-strict-aliasing -m64 -I/usr/local/include -I/usr/include
checking for the flags (-Wno-error=nullability-completeness) ... no
gcc -Wno-error=nullability-completeness -m64 -I/usr/local/include -I/usr/include
checkinfo: /home/markvoltz/.local/share/xmake/core/base/os.lua:705: cc1: error: -Werror=nullability-completeness: no option -Wnullability-completeness

stack traceback:
[C]: in function 'error'
[/home/markvoltz/.local/share/xmake/core/base/os.lua:705]: in function 'raise'
[...markvoltz/.local/share/xmake/core/sandbox/modules/os.lua:311]: in function 'runv'
[...local/share/xmake/modules/detect/tools/gcc/has_flags.lua:48]:
checking for the flags (-fomit-frame-pointer) ... ok

gcc -fomit-frame-pointer -m64 -I/usr/local/include -I/usr/include
checking for the flags (-Wno-expansion-to-defined) ... ok
gcc -Wno-expansion-to-defined
checking for the g++ ... /usr/bin/g++
checking for the shared library linker (sh) ... g++

There are some incompatibilities in drawing boundary characters on your system, and you may need to debug the following lua code.
https://github.com/tboox/ltui/blob/e84c9b2972f0b36c5e9ef03132e5d3b157aa77bb/src/ltui/border.lua#L66-L74


Memory leaks in the use of SSR

Hi, we have a system in which we set up our websites using SSR, with a flow over 2000 rpm in production environment. Actually, we are facing memory leaks issues by rendering react component throught SSR. We are looking for alternatives for handling SSR efficiently by avoiding possible memory leaks generated by the react compoenents to render. We are evaluating the use of react-render-server as framework for SSR. Have you face similar memory leaks issues before? Does react-render-server have mechanism to avoid those memory leaks? Thank you so much for your support.

I'm sorry, we haven't found memory leaks like that (that I know of!)  We do LRU caching, which may help with memory leaks, but you'd really need to try the product to see if it helped in your situation.


FAI-983 bug and additional test (delta from RobGeada/trustyai-explainability)

Issue: https://issues.redhat.com/browse/FAI-983
Fixed bug where standard deviation ~= 0.  Added epsilon parameter to precompute() and FourierMMD ctor.
Added random data test.

resubmitting on main branch


Add endpoints.onErrorAdd

Summary
Add endpoints[].onErrorAdd to add response body or parts of it as error, when the conditions fail for easier debugging.
Implements https://github.com/TwiN/gatus/issues/742
Checklist

[X] Tested and/or added tests to validate that the changes work as intended, if applicable.
[X] Updated documentation in README.md, if applicable.

I would be more okay with having the debug parameter print the body in the logs honestly.
The big problem is that it's pretty common for firewalls or outages to cause normally JSON or plain text endpoints to return HTML pages, and those HTML pages can be pretty lengthy, something that may cause issues both at the storage level and at the display level on the UI

My intention was, that I see immediately what my micro services are lacking because they write it in the body.
But I see your point.

Would you prefer to have a limit of text that could be added, and if the body is over the limit, then to truncate or not print it?


Support for the experimental syntax 'jsx' isn't currently enabled when using toMatchInlineSnapshot

🐛 Bug Report

i'm using babel-jest with babel-preset-react-app in a Vite project, everything works fine except toMatchInlineSnapshot, whenerver i use it, Jest throws
  SyntaxError: .../name.test.js: Support for the experimental syntax 'jsx' isn't currently enabled

Add @babel/preset-react (https://git.io/JfeDR) to the 'presets' section of your Babel config to enable transformation.
    If you want to leave it as-is, add @babel/plugin-syntax-jsx (https://git.io/vb4yA) to the 'plugins' section to enable parsing.

jest.config.js
{
  "moduleNameMapper": {
    "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/jest/mocks/fileMock.js",
    "\\.(css)$": "identity-obj-proxy",
  },
  "setupFiles": ["<rootDir>/jest/setup-test-env.js"],
  "globalSetup": "<rootDir>/jest/globalSetup.js",
  "testMatch": ["<rootDir>/src/**/*.test.js"],
  "verbose": false,
  "testEnvironment": "jsdom",
  "setupFilesAfterEnv": ["<rootDir>/jest/setup-test-after-env.js"],
  "transformIgnorePatterns": ["node_modules", "^.+\\.module\\.(css)$"],
  "modulePathIgnorePatterns": ["<rootDir>[/\\\\](build|docs|node_modules|scripts|dist)[/\\\\]"],
  "transform": {
    "^.+\\.(js|jsx|mjs|cjs|ts|tsx)$": "babel-jest",
    "^.+\\.css$": "<rootDir>/jest/mocks/cssMock.js"
  },
}

babel.config.json
{
  "presets": ["react-app"]
}

To Reproduce
Steps to reproduce the behavior:
create a Vite project with Jest.
Expected behavior

Expect to parse JSX for toMatchInlineSnapshot too
Link to repl or repo (highly encouraged)

Repo with codesandbox: https://github.com/SarKurd/jest-bug-11730
envinfo

  System:
    OS: macOS Mojave 10.14.6
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
  Binaries:
    Node: 14.17.3 - ~/.nvm/versions/node/v14.17.3/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.13 - ~/.nvm/versions/node/v14.17.3/bin/npm
  npmPackages:
    jest: ^27.0.6 => 27.0.6

Hi @SarKurd,
Perhaps the Jest help page on testing React could offer some hints on how to get this working, if you haven't seen it already.
For getting further help, please check out the following resources:

The Jest help page
Our #testing channel in Reactiflux
The jestjs tag on StackOverflow

If you believe there is a bug in Jest, please provide a minimal reproduction (in a repl.it demo or a repository on GitHub) that demonstrates the issue. See https://stackoverflow.com/help/minimal-reproducible-example for information on how to create good reproductions.

Hi @SarKurd,
Perhaps the guide on testing React could offer some hints on how to get this working, if you haven't seen it already.
For getting further help, please check out the following resources:

The Jest help page
Our #testing channel in Reactiflux
The jestjs tag on StackOverflow

If you believe there is a bug in Jest, please provide a minimal reproduction (in a repl.it demo or a repository on GitHub) that demonstrates the issue. See stackoverflow.com/help/minimal-reproducible-example for information on how to create good reproductions.

Thanks @sigveio, i added a minimal repo with Codesandbox link in the readme

Thanks for adding a repo, @SarKurd. =) Would you be able to explain why you think this is a bug in Jest and not a problem with your code or configuration?

It only fails when using toMatchInlineSnapshot so i'm not sure if there is some i missed but followed the docs as described

I understand, but this issue tracker is not meant to be a help forum - so if you are not sure what the problem is, it would be best if you use one of the resources I linked earlier to ask for help.
The process outlined in the page I linked to about creating minimal reproductions can also help you troubleshoot.
For example; If you do as the suggestion at the beginning of the Testing React Apps guide and create a clean preconfigured react act with create-react-app and then use the react-test-renderer as shown in the guide, I think you would find that toMatchInlineSnapshot() works. Perhaps you can compare that to your own setup and try to figure out what's different. Or alternatively, start from scratch using the setup that works.

@sigveio sorry but i was confident it was a bug that's why i posted it here.
I rolled back to jest 26.6.3 and it works fine

@SarKurd No problem, but I think you need to troubleshoot further. And there should be some valid tips for you in what I have said so far. Both in terms of what you can do, and where you can get more help.
Before I suggested it to you, I tested with the preconfigured setupt from create-react-app - and toMatchInlineSnapshot() worked fine. This is still using Jest 26.x, but I also tested again with upgrading all the jest and babel packages to their latest version on that same configuration, and with Jest 27 it still worked. The only thing I had to manually fix was to add a require-workaround for createTransformer in babelTransform.js as suggested here.
In case you are not aware of it; after installing with the create-react-app util, you can use yarn eject (inside the directory it creates) to copy build dependencies, configuration files, and scripts into the app directory. Please note that this is a one-way operation, so it'd be best to do it on a clean install just to be able to look at the configuration and compare to your own setup.

Thanks @sigveio it indeed fixed the issue but still looks like a bug to me since i had to use a workaround for tha.
Will close this.
for future readers
jest.config.json
  "^.+\\.(js|jsx)$": "<rootDir>/jest/babelTransform.js",

babelTransform.js
const babelJest = require("babel-jest").default;

module.exports = babelJest.createTransformer();

I just experienced this. Spent all week off and on trying to figure out why my babel config wasn't applying... after removing the 'inline' portion of my new assertion, my test started working.
I used this to see the applied config...  nothing popped out at me as wrong (unless presets do not merge across configs, in which case jests' preset is stomping mine)
BABEL_SHOW_CONFIG_FOR=<path to spec> jest <path to spec>

in jest config i have a mapping like the above too:
'\\.[jt]sx?$': resolve('config/jest/babelTransform.js'),

and it's this:
const { default: babelJest } = require('babel-jest');

module.exports = babelJest.createTransformer({
	configFile: false,
	presets: [ '<absolute path to project babel config>' ],
});

I'm fine with not using inline snapshots, but man... it doesn't make sense to me yet. It should be using my babel config... but jsx in the spec file mixed with toMatchInlineSnapshot fail.

Hey @jsg2021 - take a look at the last few posts in #11741. Perhaps it could help make a bit more sense out of it?