Adding support to provide DNS server port

Provide support to provide a server and port when querying DNS Servers.
Usecase:
Ensuring consul is performing DNS queries as expected, you may have configuration like:
dns:
  consul.service.consul:
    resolvable: true
    server: <IP_ADDRESS>:8125

I like it, LGTM once the build is succeeding

I wasn't able to get the tests passing, as you can see from efc748ccd5df4f1c74e7bddcdc6faaffa99ccd21 and afe3cd856c009d2707e5be14a45c6a90b4bbfae8.
It looks as though the wheezy repository has moved on since the last docker build.
I also noticed a number of other open PR's with similar issues, leading me to believe this is not related to my change.
I'm happy to have another look into it, but building the images locally and running the tests pass.

@aelsabbahy - Any chance we can get this merged also? now that #382 has been merged all tests are now passing!

@davidcollom Tested this locally, it doesn't seem to work when defining the port.
Can you provide a working example of a test explicitly setting :53?
This works:
dns:
  dnstest.io:
    resolvable: true
    addrs:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    timeout: 500
    server: <IP_ADDRESS>

This does not:
dns:
  dnstest.io:
    resolvable: true
    addrs:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    timeout: 500
    server: <IP_ADDRESS>

$ goss v
FS

Failures/Skipped:

DNS: dnstest.io: resolvable:
Expected
    <bool>: false
to equal
    <bool>: true
DNS: dnstest.io: addrs: skipped

Total Duration: 0.005s
Count: 2, Failed: 1, Skipped: 1

Hi, I've not had things checked out for a while, it could take me a few
days to get round to this, I do have example services already deployed
which should help with reproducing.
On Mon, 25 Nov 2019, 3:26 pm Ahmed Elsabbahy<EMAIL_ADDRESS>wrote:

@davidcollom https://github.com/davidcollom Tested this locally, it
doesn't seem to work when defining the port.
Can you provide a working example of a test explicitly setting :53?
This works:
dns:
dnstest.io:
resolvable: true
addrs:
- <IP_ADDRESS>
- <IP_ADDRESS>
timeout: 500
server: <IP_ADDRESS>
This does not:
dns:
dnstest.io:
resolvable: true
addrs:
- <IP_ADDRESS>
- <IP_ADDRESS>
timeout: 500
server: <IP_ADDRESS>
$ goss v
FS
Failures/Skipped:
DNS: dnstest.io: resolvable:
Expected
: false
to equal
: true
DNS: dnstest.io: addrs: skipped
Total Duration: 0.005s
Count: 2, Failed: 1, Skipped: 1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aelsabbahy/goss/pull/378?email_source=notifications&email_token=AALPJQDFGIMZAI4EFC7AGRDQVPVCNA5CNFSM4FRFH6NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFCYRLI#issuecomment-558205101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALPJQBIZMGM64RDAHZFBU3QVPVCNANCNFSM4FRFH6NA
.

No biggie, few things things:

I would drop the dockerfile/md5 changes from this PR.

I'm 90% sure the issue is that it used <IP_ADDRESS>:53 as the server, and then 53 as the port. Server needs to be parsed correctly.

Can you update one of the goss-shared examples with the new syntax so CI can validate that both are working?

Hey @aelsabbahy, I've taken a quick look at this today, and believe I've resolved the outstanding issues.
Here's my test goss file:
dns:
  google.com:
    resolvable: true
    server: <IP_ADDRESS>:53

  www.bbc.co.uk:
    resolvable: true
    server: <IP_ADDRESS>

  plex.home.collom.co.uk:
    resolvable: true
    server: <IP_ADDRESS>

  consul.service.consul:
    resolvable: true
    server: <IP_ADDRESS>:8600

For the consul check, I was able to start a dev server via: docker run -d -p 8600:8600/udp -p 8600:8600/tcp consul
I also added some unit tests (first time in Go) to ensure that this was working as we expected.
It looks like the CI tests are failing again though 😭 Seemingly about Wheezy and package versions again.

Hey @aelsabbahy, I've taken a quick look at this today, and believe I've resolved the outstanding issues.
Here's my test goss file:
dns:
  google.com:
    resolvable: true
    server: <IP_ADDRESS>:53

  www.bbc.co.uk:
    resolvable: true
    server: <IP_ADDRESS>

  edition.cnn.com:
    resolvable: true
    server: <IP_ADDRESS>

  consul.service.consul:
    resolvable: true
    server: <IP_ADDRESS>:8600

For the consul check, I was able to start a dev server via: docker run -d -p 8600:8600/udp -p 8600:8600/tcp consul
I also added some unit tests (first time in Go) to ensure that this was working as we expected.
~It looks like the CI tests are failing again though 😭 Seemingly about Wheezy and package versions again.~ Looks like the latest build passed! 🎉

Eyeball/CI looks great. Appreciate you taking the time to implement this!
Thanks for adding unit tests!
I'll review this more in depth this weekend.

Thank you for your contribution!


net: Dial() connection refused error is not syscall.ECONNREFUSED value on windows

What version of Go are you using (go version)?

$ go version
go version go1.16 windows/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?

https://play.golang.org/p/9jvY6yme07T
What did you expect to see?
Expected error value returned from net.Dial() to be a syscall.Errno value equal to syscall.ECONNREFUSED
What did you see instead?
error is a syscall.Errno value: 0x274d
dial tcp [::1]:8080: connectex: No connection could be made because the target machine actively refused it.

The exact use of sysca.ECONNREFUSED is inherently system dependent.  It doesn't work on all systems.  In particular, as you've discovered, it doesn't work on Windows.
What are you really trying to do?

Ah very well then.
I'm just trying to suppress otherwise noisy error logging of attempts to connect to a service that is not guaranteed to be up. I worked around the problem on windows by comparing to the exact syscall.Errno value I discovered.


Implement deadlines within a project

Taken from the project notes.
Deadline of a deliverable on a project:
Add, extend AddCommand and support a different prefix
- add student: add -s id/A0123456X proj/.. e/… n/… t/…
- add deadline: add -d id/A0123456X dn/… dd/…
Delete, extend DeleteCommand, support a different prefix
- delete student: delete -s id/A0123456X
- delete deadline: delete -d id/A0123456X num/1
Edit, extend EditCommand (in parallel with modifying the parsing of EditCommand itself)
Sorting criteria for each student: Sort by status(IP, Done) first, then deadline (ascending order)

This issue is related to:

#6
#12
#41

Closed by #111.


Audit loading and error state display

Acceptance criteria

[ ] Consistent loading styles on detail view pages:

[ ] On page load, submission doesn't flash up in middle column before the data loads
[ ] On page load, when grouped by listing is still loading, other columns should show only one loading panel as opposed to one in each column.

[ ] Ensure message displayed on detail view page if no submission in round

Ready for review

Actioned CR feedback so ready for re-review

Re-reviewed maybe one small change as a result? Back to you Chris.

Thanks Todd. Have made the error states consistent now, can you take over my last commit? https://github.com/OpenTechFund/opentech.fund/pull/1012/commits/0a8912ec552711dc0a9ab852fb7d8f92419d29f2

Looks good to me @chris-lawton.
Only thing I noticed is that I am getting the following state intermittently for this round: https://dev-apply.opentech.fund/apply/rounds/732/

It seem to be struggling to load the applications, if I refresh again it does, but not sure whats going on here and the error state does not look right - the whole page should just be loading, right?

Yeah, i noticed this to and it's something that Tomasz has picked up on in the channel. Unsure if it's related to the work done here or another ticket. Best to keep this open and i'll check with Todd to see what's happened.

Hi @gibbsmaya,
The work in #991 should fix this which is now on dev.
Thanks,
Chris

I stand corrected, can still replicate this on dev.

Now fixed on dev

Looks good to me. Over to @todd-dembrey for deployment to test.


Dev

Coverage remained the same at 98.122% when pulling 086c8c03f1ed4e12bc7693b917ca048d5e7a2bdc on libotony:dev into b8dcdfacfa4d48eb45f6ae04b6c93e6e0a06b32d on vechain:master.


{KeyVault} Add deprecation messages for soft-delete related commands and parameters

Add deprecation messages to meet the service team's requirements. Texts were copied from the documentation.
az keyvault create/update -h

az keyvault delete -h

az keyvault key delete -h

az keyvault secret delete -h

az keyvault certificate delete -h

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

the change itself LGTM. we need sync with Jack on when to removing the message. it's little annoying.

Yes, those messages are really lengthy and they should be removed as soon as possible when the service side finish the change.


Ability to add custom RHCOS kernel option

Sometimes there is a need to add specific kernel option based on requirement. For example adding slub_max_order=0 to evaluate the memory footprint impact for containers.
This issue is to discuss and come up with such a mechanism

There are two phases when such settings will be needed:

Before start of OCP install
Post OCP install

For post OCP install, the settings can be performed via machineconfig operator and we can create an ansible playbook for the same. This is tracked here - https://github.com/ocp-power-automation/ocp4-playbooks/issues/2
For pre OCP install this either needs to be handled as part of this project or the helper node playbook

Post OCP install config is addressed by the following PR - https://github.com/ocp-power-automation/ocp4-upi-powervm/pull/36

For pre-install (day-1) also machine config can be used as described here - https://docs.openshift.com/container-platform/4.3/installing/install_config/installing-customizing.html#installation-special-config-kargs_installing-customizing
@yussufsh @Prajyot-Parab

@yussufsh @Prajyot-Parab @bpradipt
If I see it correct, then this issue can be closed?! I have used the option in the some weeks ago and it was working like I expected it.

Yeah.. Thanks @torwen1


Notifying change on URI results in a call to io.pivotal.arca.dispatcher.QueryListener, passing a cursor with the right count, but no data.

We have an ArcaQuerySupportFragment
who's data is populated by calling:
final Query query = new Query(uri, QUERY_ID);
query.setForceUpdate(true);
query.setSortOrder(sort);
execute(query);
Another activity in our app updates the data and notifies the change:
ContentResolver contentResolver = context.getContentResolver();
contentResolver.update(uri, data, null, null);
contentResolver.notifyChange(uri, null);
our io.pivotal.arca.dispatcher.QueryListener is being called, and is passed a:
QueryResult result
whose:
result.getResult()
is a cursor with the right getCount(), and the right getColumnNames()
however after we call cursor.moveToPosition(), any calls to cursor.getString return null,
and any call to getInt returns 0.
So basically, we get a cursor back that looks right, except it has no data.
We have reproduced this issue against Arca 1.0-beta.3, as well as the latest dev branch code as of 7/28/2015.

Writing this out, I noticed our update call was not passing a where clause, so I think that was clearing our data.  I hit close on this issue.


OpenAPI oneOf not under V3 namespace

The readme said that you can find it under OpenAPI.V3. The lie detector determined that was a lie.

hhm I think that's because decided to not have a .V3 namespace and probably forgot to update the docs

Updated.


LICENSE.md?

Would be great to have a license, perhaps MIT, so others can freely use/derive from this lib.

Generally speaking, the absence of a license means that the default copyright laws apply. This means that you retain all rights to your source code and that nobody else may reproduce, distribute, or create derivative works from your work.

https://help.github.com/articles/open-source-licensing/

thanks @developerdizzle for pointing this out. Wasn't thinking of the licensing issues at all when I put this out.
I'll add an update as soon as I can with a LICENSE.md, for now you can assume MIT for this project.

@developerdizzle added a license, hope that helps!

You rock, thanks!


Upcoming update to WG notes patent policy text

@plehegar and @wseltzer agreed to simplify the patent policy text for WG Notes (other types of documents remain unchanged).
After https://github.com/w3c/specberus/pull/707 is merged, the markup should simply be:

  This document was produced by a group operating under the  W3C Patent Policy.

Note: we are adding a requirement on the data-deliverer attribute which should list the WG IDs which was already in one of the previous patent policy link.
Once we are certain respec and bikeshed have been updated, we will update pubrules.

It seems that bikeshed/boilerplate/dap/status-WG-NOTE.include is the only boilerplate for WG Notes, and it's updated in #1157, so this issue can be closed?


Provide faster memchr for windows

Follow up PR #3 with the same algorithm in forward search, manually implemented
memchr searching two usize at a time. This should be better than some libc implementations
on windows, so we enable this by default there.
I'm not very happy with the tangle of configuration conditions.

I suspect based on googling that OS X's impl is similarly unimpressive. We just need to find somone to test it.

Benchmark results on Mac OS X 10.11 Beta (15A262e), Rust 1.4.0-nightly (8f1b0aa32 2015-08-21):
test iterator          ... bench:       4,123 ns/iter (+/- 1,108) = 2425 MB/s
test iterator_reversed ... bench:       3,772 ns/iter (+/- 1,058) = 2651 MB/s
test libc_memchr       ... bench:         154 ns/iter (+/- 30) = 64935 MB/s
test libc_memrchr      ... bench:         754 ns/iter (+/- 224) = 13262 MB/s

Cool. Looks like OS X's got a solid vectorized memchr implementation already.

Thanks for running this!

@programble To explain, the libc_memchr scans the data so fast it is probably using something like SSE4.2 extensions. The memrchr result is for an implementation in pure rust, which is still a pretty good improvement compared to a naive loop at least. With this result we'll continue to use libc memchr for OS X.

Ok inspired by something I saw from @kimundi, I guess a build script is the only way to set our own config flags. It would move the configuration choices out and keep it a bit cleaner.

Awesome! Thanks! :-)


Update to binaryen 102

Just because v102 (latest) should in principle have better / more optimizations.
As part of this, let's compile binaryen from sources for both archs (Intel and Arm). And, let's run binaryen tests.

Because v101 was segfaulting when optimizing some contracts. Should be fixed in the latest version.


fix: stop and drain timers

Stop the timers and drain them.
I got a situation in which this was causing a bubbletea app to use A LOT of ram.
Per my understanding, some timed commands were being fired twice sometimes... on most cases this might not be perceptible, but in this particular case, each of these commands generated another message, and the program went on double firing those too.

after investigating a bit further, found what was causing the double fires: https://github.com/charmbracelet/mods/pull/252
this is still a good PR though, IMHO :)


Merge pull request #2 from securechain/0.9

Merge for 0.9.2 version with updated README.md

Leave a comment


Pass the UndoToken to onHide as well

pass the token to the listener onHide as well (not just in onUndo). this is useful for doing stuff that is not actually undoable and should only be executed after the user had his chance to undo.
Also included a little fix to avoid a (very very unlikely) race condition.

love


Add default search field for otel logs index.

I did not find a good default search field for the otel traces index.
I can change that if @guilload you have a good idea.
Tested locally.

Good question! I made this example exactly for that: https://quickwit.io/docs/log-management/send-logs/using-otel-collector
You will note that we need to add into the json  observed_time_unix_nano  so that the ingest does not fail.
If not, the OTEL collector will send the 0 value to quickwit... and it will fail which is expected as now observed_timestamp_nanos is a datetime.
I checked the OTEL log model and I think it's fine to have this additional constraint.


Openstack infra connect undercloud and overcloud

Connecting Undercloud and Overcloud, meaning we will be able to see Hosts for Openstack provider Vms and Vms for OpenstackInfra provider Hosts. That means we are connecting 2 providers. As a benefit we can see the same Host/Vm relations as for e.g. rhevm, VmVare, ... at least for NovaCompute hosts

@blomquisg Please review

@chessbyte: @Fryguy and I are gonna chat about this type of architectural change when linking provider managed objects together.

@blomquisg @Fryguy ok, changed to new provider<>manager structure.
I am creating provider as before_create hook for EmsOpenstackInfra. This would be removed when there is some management of providers.

@jrafanie any idea about the CI failure? never seen something like that

Spec failure in CI.  Seems unrelated, going to restart

VmdbDatabaseConnection returns nil for wait_resource where there are no locks
Failure/Error: expect(no_locks).to be
expected nil to evaluate to true
./spec/models/vmdb_database_connection_spec.rb:28:in `block (2 levels) in <top (required)>'

@tenderlove See sporadic failure above (though I think you already know about this one)

I am creating provider as before_create hook for EmsOpenstackInfra. This would be removed when there is some management of providers.

Cool...I'll take a look.

@blomquisg @Ladas We've had recent PRs that introduced sporadic test failures.  We really need to keep an eye on it and open issues when they're found.  Please do this with any "unrelated" issues you see on travis, capture the exact test failure, what git sha1 it happened on and anything else that's useful.  I'm more than willing to offer ways to isolate these problems but we can't ignore them.  :disappointed:

@Fryguy changed to ProviderOpenstack
I kept it before_save, the autosave nor dependent_destroy works for me for belongs_to relation. There is something weird.
It should be acceptable this way, it will be better when we manage it in provider, then I will delete this before save hook.

I haven't really re-reviewed this yet (since the change to Providers was introduced), but I'm so excited to see this PR moving in that direction!
:smile_cat:

@dclarizio can you have someone review this change?  It has UI impacts.  Namely, the OpenStack infrastructure provider can be "added" to an OpenStack cloud provider in the edit provider page.
Whoever verifies this from the UI side can work with @ladas to get an OpenStack infrastructure provider for testing.

@fryguy, after @dclarizio has someone from UI verify this, I'm good with it.  Anything else you see?

@blomquisg: I can work with @Ladas on verifying this. Just I am pretty busy atm. Is Monday morning ok for the final review? @Ladas, can we meet Monday morning for a walk-through to get this merged?

@martinpovolny ok
@blomquisg btw. i somehow pushed here code from https://github.com/ManageIQ/manageiq/pull/2374
damn you github, let me fix that

now that is better, the wrong commits are removed


Outlook WSA Doesn't Save Attachments

Steps to reproduce
Click on a file "More Button". Provides opportunities to Open the file by clicking on it, Save to a Cloud or Save to Device. Save to the Cloud Opens Cloud folders to choose saving destination.
Choosing "Save to Device" does nothing.
✔ Expected Behavior
Choosing "Save to Device" Opens list of device folders like it used to.
❌ Actual Behavior
Nothing
Other Software
No response
Please specify the version of Windows Subsystem for Android
2311.40000.5.0

lol, the project has been abandoned, and you are opening new issue? rofl.

Hope this project will continue in the future or at least Microsoft can make it open-source for the community to help or develop them


[Refactor] Reanme PStatus to StatusPB

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

[FE PR Coverage check]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/qe/Coordinator.java
0
1
00.00%


Close host tester in tests

By implementing the Close method on the hostTester type. consensusSetTester.Close is an existing example.

transactionPoolTester needs a Close method too.
Why does consensusSetTester only call Close on the gateway and consensus set?  Do the miner and transaction pool close automatically?  And there is no Close method at all for the wallet -- should there be?


Update to engine 1.14.14

For now this PR updates the engine to 1.14.14-SNAPSHOT but when it is tested I will release 1.14.14.

Fixes https://github.com/daisy/pipeline-ui/issues/148

I confirmed it fixes #148 in my test

@NPavie updated to 1.14.14


Build pipeline

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


Profiled windows builds

nix build -f release.nix cardano-node-win64-profiled currently fails with
<no location info>: error:
    /nix/store/ghcll0zjzfn1ss2x03msjclfv1vb8zm5-iserv-wrapper/bin/iserv-wrapper-prof: createProcess: runInteractiveProcess: exec: does not exist (No such file or directory)

I could not find from where this non-existing iserv-wrapper-profis called from.
Only iserv-wrapper is present in that directory.
@angerman would you have some more insight into this error?

closing for now. More investigation on issues of the compilation pipeline is required.

Further investigation shows that we are failing to malloc in the linker, looking at the wine process, I don't see it running out of memory (and the machine has 64G). Not clear yet what's really the issue with profiling here, however profiling libraries are usually larger (+50%) than regular libraries


Adds support for Fail2ban and integrates with teler

Metadata

Fixed Issue: #35
Contributors: @iosifache

Proposed Changes

Adds support for Fail2ban.
Creates an integration with the already-supported teler.
Changes the README.md generation workflow to include solutions under refactoring.

New Functioning
MutableSecurity now have Fail2ban as a supported solution. In addition, the integration from teler can be enabled by changing rhe fail2ban_integration information.

Settings from Coveralls were changed to allow a decreased coverage. The new settings are:

Minimum coverage: 90%
Maximum coverage decrease: 1%.


Change master branch to main

Could we change the master branch to main? The original master name is offensive to some people.

done


pivpn: command not found after uninstallation and re-installation

PiVPN Issue Template

Issue
i have having touble setting up pivpn (i am new to VPN completely) to i thought i would uninstall it and re-install it givent that the setup was simple and uninstallation can be achieved with pivpn -u.
after having re installed and rebooted, i tried to add a client (pivpn add), but it gives me the error  pivpn: command not found. so i tried to install again using the suggested command on the pivpn homepage curl -L https://install.pivpn.io | bash. after the second time it still did not work. i dont know how to proceed from this point.
Have you searched for similar issues and solutions?
(yes)  i have tried to search the exact error i recive in the command prompt ` pivpn: command not found`. i cannot find any abody else with the same issue. i dont know if this is a new issue.

Console output of curl -L install.pivpn.io | bash
  sudo curl -L https://install.pivpn.io | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:39 --:--:--     0curl: (6) Could not resolve host: install.pivpn.io
➜  ~ sudo curl -L https://install.pivpn.io | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   178  100   178    0     0    221      0 --:--:-- --:--:-- --:--:--   221
100 54741  100 54741    0     0  49494      0  0:00:01  0:00:01 --:--:-- 49494
:::
::: sudo will be used for the install.
::: Verifying free disk space...
:::
::: Checking apt-get for upgraded packages.... done!
:::
::: Your system is up to date! Continuing with PiVPN installation...
::: Using interface: eth0
::: Static IP already configured.
::: Using User: pi
:::
::: Checking for existing base files...
:::    Checking /etc/.pivpn is a repo...:::    Cloning https://github.com/pivpn/pivpn.git into /etc/.pivpn... [/]  fatal: unable to access 'https://github.com/pivpn/pivpn.git/': Could not resolve host: github.com
 done!
:::
::: Stopping OpenVPN service... done.
:::
::: Installing scripts to /opt/pivpn...cp: cannot stat '/etc/.pivpn/scripts/makeOVPN.sh': No such file or directory
::: Install Complete...
::: Restarting services...
Synchronizing state of openvpn.service with SysV service script with /lib/systemd/systemd-sysv-install.
Executing: /lib/systemd/systemd-sysv-install enable openvpn
::: done.
:::
::: Installation Complete!
::: Now run 'pivpn add' to create an ovpn profile for each of your devices.
::: Run 'pivpn help' to see what else you can do!
::: It is strongly recommended you reboot after installation.
:::
::: The install log is located at: /etc/pivpn/install.log

Can you fill the issue template with the other commands?

Are you sure you can access github.com from your Pi? According to your log, it doesn't appear that that hostname could be resolved. Could be a problem with your DNS or your Internet connections.

no activity. closing.


conkeyscan: init at 1.0.0

Description of changes
Tool to scan Confluence for keywords
https://github.com/CompassSecurity/conkeyscan
Related to https://github.com/NixOS/nixpkgs/issues/81418

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)

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 320784 run on x86_64-linux 1

6 packages built:

conkeyscan
conkeyscan.dist
python311Packages.random-user-agent
python311Packages.random-user-agent.dist
python312Packages.random-user-agent
python312Packages.random-user-agent.dist

Result of nixpkgs-review pr 320784 run on x86_64-linux 1

6 packages built:

conkeyscan
conkeyscan.dist
python311Packages.random-user-agent
python311Packages.random-user-agent.dist
python312Packages.random-user-agent
python312Packages.random-user-agent.dist


How to Add Custom Sound Into Client and Play it on game by using directory on roblox studio

so i was thinking i want upload this sound
https://github.com/pizzaboxer/bloxstrap/assets/98821815/04b1481d-910d-4c56-8a24-d57f09ada3d2
(Yeah its dioxide.exe first payload sound)
but idk how to direct it to work it

can someone help by doing that?

This place is strictly for help/support with Bloxstrap only, sorry.


Create gecdahod.txt

This a college affiliated to GTU in Gujarat.

@satishp962 Pull request merged. Thank you!


fix: use 409 http code for workspace already exist

fix: use 409 http code for workspace already exist
Description
Instead of using 500 http code which is for internal server code , we should now use 409 http code , which is for conflict

[ ] This change requires a documentation update
[ ] I have made corresponding changes to the documentation

Related Issue(s)
closes #1391
Screenshots

Notes
Please add any relevant notes if necessary.

@quest-bot loot #1391

The error itself was descriptive enough , we might need to not to expand it more

@divanshu-go we assigned @RS-labhub to this issue. Please respect that.
You can find other issues you want to work on. Thanks.

@Tpuljak , the person you assigned has no progress till now . I think I have more progress than him

I see what died here today , opensource

he did the same changes as me ,  and and I dont think this require talking on slack ,


Add GitHub CI/CD Workflows

Tasks:

CI:

add a Chord.Lib build pipeline
run all Chord.Lib unit tests
deploy the Chord.Lib to a dockerized Chord node image
run all component and system tests

CD:

release the Chord.Lib as DLL
release the Chord.Lib as Docker image (optional)

not so urgent. postpone


Update owners and docker images list

@maoo I just realized we don't have a legend-doc-maintainers group that corresponds to this repo finos/legend, can we create that and add Pierre like what we do for the other Legend repos? Thanks!

@maoo I just realized we don't have a legend-doc-maintainers group that corresponds to this repo finos/legend, can we create that and add @pierredebelen like what we do for the other Legend repos? Thanks!

Done! See https://github.com/orgs/finos/teams/legend-doc-maintainers (they have maintainer role on this repo)

@maoo Thank you so much!!


FSCK Repair Table command for Delta Lake

Thank you for contributing to Microsoft Fabric documentation
Fill out these items before submitting your pull request:
If you are working internally at Microsoft:
Provide a link to an Azure DevOps Boards work item that tracks this feature/update.
https://msdata.visualstudio.com/A365/_workitems/edit/3275566/
Who is your primary Skilling team contact? @mention them individually tag them and let them review the PR before signing off.
<EMAIL_ADDRESS>For internal Microsoft contributors, check off these quality control items as you go

[ ] 1. Check the Acrolinx report: Make sure your Acrolinx Total score is above 80 minimum (higher is better) and with no spelling issues. Acrolinx ensures we are providing consistent terminology and using an appropriate voice and tone, and helps with localization.

[ ] 2. Successful build with no warnings or suggestions: Review the build status to make sure all files are green (Succeeded).

[ ] 3. Preview the pages:: Click each Preview URL link to view the rendered HTML pages on the review.learn.microsoft.com site to check the formatting and alignment of the page. Scan the page for overall formatting, and look at the parts you edited in detail.

[ ] 4. Check the Table of Contents: If you are adding a new markdown file, make sure it is linked from the table of contents.

[ ] 5. #sign-off to request PR review and merge: Once the pull request is finalized and ready to be merged, indicate so by typing #sign-off in a new comment in the Pull Request. If you need to cancel that sign-off, type #hold-off instead. Signing off means the document can be published at any time. Note, this is a formatting and standards review, not a technical review.

Merge and publish

After you #sign-off, there is a separate PR Review team that will review the PR and describe any necessary feedback before merging.
The review team will use the comments section in the PR to provide feedback if changes are needed. Address any blocking issues and sign off again to request another review.
Once all feedback is resolved, you can #sign-off again. The PR Review team reviews and merges the pull request into the specified branch (usually the main branch or a release- branch).
From the main branch, the change is merged into the live branch several times a day to publish it to the public learn.microsoft.com site.

@Sovima
Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Get started contributing to the Microsoft Learn platform.
We'll close this PR. Thank you.


Update README.md

Restore the old read me with appropriate renaming to BLT - also add links to new repos

The screenshots of the app aren't showing up as there's no proper one in assets/github. Add these to close the issue

I think it was a GitHub problem, happened with other repos too, closing.


Pagination for apps and db operations pages

@mixonic for your review

Override findQuery for operation adapter to allow pagination
add pagination in {{aptible-pagination}} component
Paginated route and controller mixins for shared behaviors
update 'page' query param in url
dry tests with shared test helper functions

fixes #49 refs #51
cc @fancyremarker
This uses the current_page, total_count and per_page meta data
provided by the API as of https://github.com/aptible/api.aptible.com/pull/146

closed in favor of #60


Added design

Description
Added the Appbar design as you have shown screenshot of if you want any changes in it then please let me know
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Hey @Jhalakupadhyay first list your UI component in main ui_collections.dart to render it
and can you please let me know why you have added .jar files?

Hey @Jhalakupadhyay first list your UI component in main ui_collections.dart to render it and can you please let me know why you have added .jar files?

ohhk I will add it to the main.dart and about that .jar file I think because I use to do the backend for android in flutter in java so when I ran the code the android studio has added some file which I don't have any Idea why.

Added the design in ui collection and the app worked fine without crashes but while committing it showed conflict in the ui_collection because some other design was there at 18 tag so i changed mine to tag:20 and i hope it will work.


$user->roles()->attach($admin->id) returns NULL

I tried to attach roles to my admins. I followed all the steps carefully and did able to create rows  in Role table in db with data. but when i tried to assign role using, $user->roles()->attach($admin->id) it returns Null. How can i fix it ?

try this $user->attachRole($admin);

tried with $user->attachRole($admin); also .. but it didn't help. now I fix it by -
$user->attachRole($admin->id);

now working great

if $admin = new Role(); it works...
I am now using this package...

I also used    $admin = new Role(); but in my case not happened, it returns null in both case mentioned here. So i tried my stuff then it works. BDW thanks for reply !!


automation: bump golang version

What
Enrol the golang autobump
Test
Given .ci/bump-go-release-version.sh 1.17.10 it produced
Update go version 1.17.10
[feature/autobump-golang 01d69b0be] [Automation] Update go release version to 1.17.10
 2 files changed, 5 insertions(+), 5 deletions(-)
commit 01d69b0be95a1acd6f62a2984a4777b6056cde1f (HEAD -> feature/autobump-golang)
Author: Victor Martinez<EMAIL_ADDRESS>Date:   Tue May 24 18:30:08 2022 +0100

    [Automation] Update go release version to 1.17.10
You can now push and create a Pull Request

with the below changes
diff --git a/.go-version b/.go-version
index 622f042fd..ada2e4fce 100644
--- a/.go-version
+++ b/.go-version
@@ -1 +1 @@
-1.17.6
+1.17.10
diff --git a/.golangci.yml b/.golangci.yml
index 776454f56..124173022 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -104,7 +104,7 @@ linters-settings:

   gosimple:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.17.6"
+    go: "1.17.10"

   misspell:
     # Correct spellings using locale preferences for US or UK.
@@ -148,11 +148,11 @@ linters-settings:

   staticcheck:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.17.6"
+    go: "1.17.10"

   stylecheck:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.17.6"
+    go: "1.17.10"

   unparam:
     # Inspect exported functions, default is false. Set to true if no external program/library imports your code.
@@ -163,4 +163,4 @@ linters-settings:

   unused:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.17.6"
+    go: "1.17.10"

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-24T17:31:50.970+0000

Duration: 9 min 57 sec

Pipeline error

This error is likely related to the pipeline itself. Click here
and then you will see the error (either incorrect syntax or an invalid configuration).

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

Unrelated failures to these changes, as far as I see, I could not reproduce the issue in https://github.com/elastic/elastic-agent-autodiscover/pull/13


Adding a primary email address in bC creates a new Mailchimp contact

I have a different solution for adding Mailchimp contacts. Will be on the UM DEV soon.


feat(#144): GET user

In this pull, I've introduced user handler for /users/:login, as mock version of respective endpoint in GitHub REST API.
closes #144
History:

feat(#144): /user/:login
feat(#144): user()
feat(#144): no jeff

PR-Codex overview
This PR adds a new user module with a user handler function that returns user data or a 404 response based on login. It includes unit tests for the handler function.
Detailed summary

Added user module with user handler function
Handler returns user data or 404 response
Included unit tests for the handler function

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 52min)

@h1alexbel Hey there! 👋 Great job on your contribution! You've earned +4 points, which is our base reward. Your 103 hits-of-code added +5 points, but we had to deduct 16 for no code review and 4 for only 3 comments. We bumped it up by 15 to keep you motivated. Your balance is now +16. Remember, code reviews and comments are crucial for quality. Keep the contributions coming, and aim for that sweet spot of 200-800 hits-of-code for max points next time!


Adjust reference to modsecurity::utils::string::VALID_HEX

what
Fix build after integrating PR #3231 & #3233 because of reference to modsecurity::utils::string::VALID_HEX in pm.cc's parse_pm_content.
why
These two PRs were independent of each other, but when we added a reference to VALID_HEX to simplify the code, we introduced an incompatibility between both PRs because PR #3231 moved VALID_HEX into the namespace modsecurity::utils::string (alongside all other helper functions in src/utils/string.h) and adjusted all call sites to this function (previously a macro). However, in that branch the call added to parse_pm_content that we introduced later in PR #3233 was not present and thus was not adjusted.

Ahm, thanks - meanwhile I also started to work on this fix :smile:
Btw I don't understand the situation, because the PR's check were successful, but later I get reports about failed checks.
Seems like now everything is fine.
I also added two new tests (a positive and a negative) - do you think that would be useful to check the @pm's correct behavior? If yes, could you pick up those modifications?

Btw I don't understand the situation, because the PR's check were successful, but later I get reports about failed checks.

VALID_HEX used to live in the global namespace because it was a macro (#define), and in PR #3231 I converted it to a function (in order to continue removing preprocessor dependencies), and the function was included in the namespace modsecurity::utils::string (alongside the rest of the functions in src/utils/string.h, which is where VALID_HEX was defined).
You can see in commit a6d64bf that I adjusted the source files that were using this macro because the macro/function was no longer in the global namespace.
When we introduced the use of VALID_HEX in PR #3233 to simplify the code in parse_pm_content, a new reference to this macro/function was introduced, which needed to be adjusted to the namespace change.

I also added two new tests (a positive and a negative) - do you think that would be useful to check the @pm's correct behavior? If yes, could you pick up those modifications?

ok, I'll do that and push an update to this PR.

ok, I'll do that and push an update to this PR.

done. I added your repository as a remote in my local repository in order to cherry-pick your commit and preserve your authorship. :-)

in PR #3231 I converted it to a function

Oh, thanks - I didn't noticed that. I looked up for that and found as macro so that was confused.
Never mind, thanks for all again. If the tests will passed I'm going to merge this PR.

Hmm... after this PR was merged, I receive notifications about failed tests, especially on macOS, with SecRemoteRules - like this one.


Show total mitigation

Show in portal info total mitigation.
For example, a portas with 3 Very_rare new sheilds and 1 Very_Rare old shield -> 30+30+30+10 -> Total Mitigation = 100

I think with the current state of many disabled plugins, this issue could probably be marked closed to reduce clutter.


Invalid DNS due to spaces in logged in username

Extension and version:
serviceconnector-passwordless v2.0.3
What is the command being used:
az webapp connection create postgres-flexible
What is the error:
cli.azext_serviceconnector_passwordless._credential_free: Fail to connect to postgresql. invalid dsn: missing "=" after "Team" in connection info string
Why is this happening:
The logged in service principal is named "Some Team Pipeline  SPN", because the user name is not properly quoted when passed as part of the connection string in _credential_free.py at line 830, when it tries to connect using conn = psycopg2.connect(conn_string) at line 798, it does not know how to parse the connection string.
Reproduction Steps:

Run: az login --service-principal --username "$azlogin" --password "$clientDeploySecret" --tenant "$tenantId" where azlogin is the client id of a service principal with the username "Some Team Pipeline" or equivalent. The important part is the spaces
Run:

az webapp connection create postgres-flexible \
    --connection $dbConnectionName \
    --resource-group $resourceGroupName \
    --subscription $deploySubscription \
    --name $apiName \
    --target-resource-group $resourceGroupName \
    --server $dbServer \
    --database $dbName \
    --user-identity client-id=$identityId subs-id=$deploySubscription \
    --client-type java \
    --yes

Observe the error: cli.azure.cli.core.azclierror: Fail to connect to postgresql. invalid dsn: missing "=" after "Team" in connection info string
Go to the _credential_free.py located at /root/.azure/cliextensions/serviceconnector-passwordless/azext_serviceconnector_passwordless/_credential_free.py and modify the get_connection_string  function to wrap the username in singlequotes. IE: conn_string = "host={} user='{}' dbname=postgres password={} sslmode=require".format(self.host, self.admin_username, password)
Repeat steps 1 and 2
Observe that the connection was created without errors

Thanks @alexparadigm5416 for feedback. We are looking into the issue.


Allow processing of "fork"

Also slight change to logging.

Pull Request Test Coverage Report for Build 473

0 of 23 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 50.477%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

preprocessors/preprocessors.py
0
1
0.0%

webhook.py
0
22
0.0%

Totals

Change from base Build 471:
-0.2%

Covered Lines:
3177

Relevant Lines:
6294

💛  - Coveralls


Feedback on task and command documentation.

I've just spent several hours trying to figure out whether the following is possible using only public documentation and it has been...challenging. Part of this is asking about what is possible whereas the other part is identifying shortcomings in existing docs.
Here is what I am trying to do
I want to build an extension that lets the user type bb //foo:bar in the Command Palette and have it open a new terminal that runs buck build //foo:bar. Note this should be able to take a variable number of arguments, so bb @mode/opt //foo:bar and bb //foo:bar //foo:baz should also be possible.
Taking a step back, this is intended to be a workaround for the fact that it is not possible for an extension to draw its own toolbar GUI into the VS Code window. Ideally, we would also have autocomplete support to help the user fill in the argument after bb , but that is lower priority. (Some sort of fixed-size quick pick is not an appropriate option here because it is not practical to enumerate all of the Buck build targets up front, particularly when operating in a large repository.)
My Research
At first glance, assuming this was possible, I suspected the building blocks would be one or both of commands or tasks.
Commands Documentation
I started looking at commands first, partially because I thought this would be more likely to do what I wanted and partially because the docs for commands were far simpler than those for tasks. This certainly seemed promising, as the command docs mention things like:

Some commands take arguments that control their behavior.

and the primary code sample showed a command handler taking an argument:
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
  const command = 'myExtension.sayHello';

  const commandHandler = (name?: string = 'world') => {
    console.log(`Hello ${name}!!!`);
  };

  context.subscriptions.push(vscode.commands.registerCommand(command, commandHandler));
}

I tried running that example. (By the way, that code sample does not appear to be valid. TypeScript tells me "Parameter cannot have question mark and initializer.", so it has to be name rather than name?.)
After adding the appropriate snippet to my package.json:
  "contributes": {
    "commands": [
      {
        "command": "myExtension.sayHello",
        "title": "bb"
      }
    ]
  },

If I enter >bb in the command palette and run it, it works fine, but if I enter >bb //foo:bar, the palette tells me No commands matching and doesn't let me run anything. At this point, commands no longer seemed like a fruitful path to pursue.
Searching GitHub Issues
Before diving into the tasks documentation, I tried searching a combination of existing GitHub issues and StackOverflow posts to see if I could find an answer that way first. For example, these two links seemed promising:
https://github.com/Microsoft/vscode/issues/1574
https://stackoverflow.com/questions/45582698/is-it-possible-to-pass-arguments-to-a-task-in-visual-studio-code
And this ultimately took me to:
https://github.com/Microsoft/vscode/issues/64214
With some difficulty, I got the Echo input example to run. Once again, the code sample did not run verbatim: label had to be changed to id and each type also had to be updated: pickString instead of pick and promptString instead of prompt. (Note that #64214 is locked, so I could not suggest changes there.)
The flow to test this out feels a bit convoluted (though I get it):

Choose Tasks: Run Task from the Command Palette.
Choose Echo input from the quick pick.
Type in the free-form input box for echoPrompt.
Pick a value from the dropdown for echoPick.
Pick a value from the dropdown for the node process. (Incidentally, I ran rg pickNodeProcess over the vscode repo and did not get any matches, so I have no idea where support for this comes from.)

Note that if you want to change a previous value, you have to fill out the form all over again and remember what you already entered. I used a combination of this example and the docs to make progress.
Tasks Documentation
Before I get into this section, I feel that it is important to note that the Tasks documentation never explains what a "task" is! Even more confusing for me, at least given my primary objective, was figuring out that the tasks I define in the .vscode/tasks.json in my extension project serve the purpose of developing my extension. They are not, as I was trying to use them, there for the purpose of making tasks available to those who installed my extension.
In trying to streamline the experience of launching the "Echo input" task, I discovered the Binding keyboard shortcuts to tasks docs and added the following to my keybindings.json:
  {
    "key": "ctrl+h",
    "command": "workbench.action.tasks.runTask",
    "args": "Echo input"
  },

Now if I hit ctrl+h, I would go straight to the third bullet in the list of five bullets above. Definitely an improvement!
I did not have a chance to play with all of the other options for a task yet (like presentation, args, or problemMatcher). Again, I was still trying to have a cleaner association with the Command Palette. For example, it is fairly common to alternate between running buck build and buck test, so assuming the last two commands I ran were bb //foo/... and bt //foo/..., I could hit cmd+shift+p to quickly pop open the Command Palette and choose the appropriate one. In practice, this would give me quick keyboard access to command I am running frequently without formally configuring something via keybindings.json.
Playing around some more, I discovered that I could enter task echo input (or task bb, in my case) to trigger my task from the Command Palette, but I still could not do something like task echo input //foo:bar to specify the value of echoPrompt in the same line.
If I had things working the way I wanted, the one keyboard shortcut I would want to setup would be to map ctrl+b to something that opens the Command Palette with bb  pre-populated so that you could type your Buck build target straight away (ideally defaulting to the last thing you built).
Task Provider Documentation
Again, I was pretty confused that when I ran my extension, the tasks defined in the .vscode/tasks.json file were not recognized in the window that opened when I hit F5. Somewhere around this time, I suspected that a task provider might be the thing that I needed.
Similar to my frustration with the Tasks documentation, the Tasks Provider docs don't really explain what one is or why you would use it. It jumps into information about auto-detection, but I'm not sure what that is or why I should care about it. All I want to do is define a task in my extension for others to use.
This is another instance where the code samples appear to be out of date (unless this is because I'm using Insiders?). For example, there's this:
let task = new vscode.Task(
  { type: 'rake', task: 'compile' },
  'compile',
  'rake',
  new vscode.ShellExecution('rake compile')
);

I'm not sure where that comes from, but the thing I got to compile is this (as you can see, the types are quite different):
  let task = new vscode.Task(
    {
      type: 'buck',
      command: 'build',
      arg: '//xplat/nuclide/fb-JavaLib/...',
    },
    vscode.TaskScope.Global,
    /* name */ 'buck build',
    /* source */ 'buck',
  );

Also, in the code sample from the docs, it feels like the arguments to new vscode.Task duplicate information, so I really have no idea what is going on.
Unable to compose any sort of mental model for what a task provider is, this is about where I gave up.
Next Steps
Looking at the source for VS Code itself, it seems that the internal registerQuickOpenHandler() API is basically what I want. Is this something the team would be willing to explore as a proposed API?

@bolinfest thanks for the feedback on the tasks documentation in particular. I will add some more intro stuff to the beginning of the documentation. It certainly does assume a base level of knowledge that is not necessarily safe to assume in documentation like that.
Your goal is to make running various buck actions easy, and to maintain some kind of history of recently run action + parameter sets? Unless you add a command for every set of parameters, there is no way supported way for you to just type everything into the command palette in one go.
To get the history of recently run with Tasks, you'd need to make a task provider that provides a task for every single combination of parameter. Quite reasonably, that doesn't seem like what you're looking for.
You could use commands to do this in an ok way:

Have a bb command
When the command is executed, show a quick pick with the recent parameter sets the user has used (your extension would need to maintain the list of recent parameter sets). One of the options could be to "start over" and have no parameters.
When the user selects one of the recent parameter sets, you then show an input box prefilled with the recent parameter set the picked.

@alexr00 Thanks for the detailed response and suggested workaround! I think I'll focus on #70889 since if we can make that work, that seems like a more elegant solution (and hopefully useful to others, as well).
You can feel free to close this unless you want to leave it open as an action item for yourself to address the various documentation issues?

I'm going to leave it open as an action item to improve the tasks documentation 😊. We don't have a timeline for when (or if) we will do #70889. Because of that, you might want to consider looking into other ways to accomplish what you want!

Closing since this is a March issue. I'll be adding additional tasks documentation this week.


feat: Add semantic-release/github plugin

This reverts commit f5544523fc4a9c985fd4222e4c941ad7cbac089a.

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


fix: change from uppercase to lowercase (fix #2932)

Please check if the PR fulfills these requirements

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

Description
react + tui.editor 를 사용한 프로젝트 코드 리뷰를 하고 있는데, npm run build 를 하니 이런 메세지가 출력 됬습니다.

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

ㅠㅠ

@emravoan
How can a pull request be merged?
I am very curious.

@emravoan
How can a pull request be merged? I am very curious.

@Kyungseo-Park I think a pull request be able to merge after getting approval from a maintainer.


Connect to Chromecast doesn't mute desktop speaker

Hi !
I received my Chromecast today and when i try de cast from my laptop (with Chromecast in chrome, ChromeCast Desktop Audio Streamer, or other), the sound is controlled by my laptop mixer, so i have sound on my TV and my laptop, this is very annoying because when i mute my laptop, i mute Chromecast too..
I've tried a bit of things but no one solved my problem. Did someone fix that ?
(Running in win10, have the same Wi-Fi connection for both laptop and chromecast)

I'm not sure why your Chromecast (Audio?) would mute as well. However, the acidhax/chromecast-audio-stream is not updated and, at least in my case, was not able to pick up the names I've given to speaker sets (and groups).
Using instead SamDel/ChromeCast-Desktop-Audio-Streamer you get a maintained app that does recognize names that you've defined for speakers. For me, muting the PC speaker doesn't affect my Chromecasting - hopefully it'll be the same for you.
Cheers!


before function doesn't complete before scroll animation

Is there a way to add in a delay or a way to make sure the before function (in my case an animation) completes before the actual scrolling is triggered?

There isn't a way to achieve this with Scrollify. Scrollify will always scroll when the user scrolls, to prevent or delay this action would create a user experience that feels broken.

Ah okay. Thanks!


mvcc: record start_ts in write conflict error

Signed-off-by: disksing<EMAIL_ADDRESS>
What have you changed? (mandatory)
tidb uses the start_ts to track down a specific transaction. It will be more useful to use start_ts in the WriteConflict error messages.
Update WriteConflict error to include both start_ts and commit_ts. When convert the error to kvproto::WriteConflict, use start_ts instead of commit_ts.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)

make dev

Does this PR affect documentation (docs/docs-cn) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

PTAL @Connor1996


Noti-Free-Game

:pencil: Checklist
Make sure that your PR fulfills these requirements:

[x] Tests have been added for this feature.
[ ] All tests pass on your local machine.
[ ] Code has been linted with the proper rules.

:page_facing_up: Description

Add a brief description of your PR.

:pushpin: Does this PR address any issue?

If so, add the # of the issue this is addressing.

Hi

Hey there, thanks for your PR. However, I'm not seeing anything of value being added to the project.
I'm gonna close this, unless you feel there may be a mistake, feel free to re-open it.


Changing base layout

Hi @trenpixster
thanks for this great library! It's been super helpful for me so far.
I'm trying to configure the styling of the addict templates. I can modify the form elements, but how can I change the base layout (addict.html) to use my app's existing base layout (app.html)?
As far as I can tell, using addict.html is hard-coded here... https://github.com/trenpixster/addict/blob/master/lib/addict/controller.ex#L151-L154
Would appreciate any pointers. Thanks.

That's a good question.. I didn't input any extensibility point there unfortunately. But it makes sense to have it. In the mean time I'd say you could take a peek at the controller implementations and override those routes on your router. Does this make sense for your use case?

seems like this is an issue where the readme should be updated?


[1731] Recruitment cycles for other provider commands

Context
Changes proposed in this pull request
Guidance to review
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Add option to specify recruitment cycle to mcb commands.


Added Commands, Fixed Trunk/Frunk

New Commands:

Window Control
Software Update (if available)
Frunk Actuation

Fixed Commands:

Trunk Actuation

Thank you for your contribution, @boxystudio! I left some comments regarding some changes that would likely be beneficial to ensure full usage of the commands added.
Let me know your thoughts!

@boxystudio just checking in! Awaiting the windowControl function change to allow for this to behave properly, otherwise the usage of "close" would not work without the vehicle's current latitude/longitude.
More context:
https://github.com/timdorr/tesla-api/commit/e8ea847d73592c7aedcea83a11a41e9486600141

Pushed an update to resolve this issue.


dynamically adding ng-template

First of all I want to congratulate you on building this awesome timesaver module... Thank you.
This is not an issue but a request to see if you can allow for adding ng-template dynamically.  I can easily add an element after dynamic-form-bootstrap-control but I think it will be great if we can have ng-template defined as dynamic component to be inserted as part of the dynamicformcontrolmodel.
Or may be you can help me with your suggestion as to how I can implement it as I am quite new to this.

@hpn1975 Hi! Thanks for your kind words!
There have been a couple discussions on this. In my opinion the current solution already pushes the envelope. Besides I would not like to mix up pure model and additional UI information.
What exactly would you like to achieve with such a dynamic component template?

The use case is simple:
Based on the user inputs on the form (for example: user selecting multiple checkboxes in questionnaire), I want to present them with different action buttons to perform different functions.  I understand that I can do this by just coding in all buttons and using the userselections to show and hide the buttons.
To your point, it does pushes it a bit but it will also allow me to put custom templating in an array and genereate formgroups on the fly.
It is already a great module... and works great.  I just thought my request to be a next logical extension.... however, if you have already discussed it then please feel free to close this.

@hpn1975 Alright, could you please give me an idea of how you'd like to declare such a use case directly on a DynamicFormControlModel?

Based on documentation, the idea of a custom template is providing additional mark up to the form control.  However, in Angular, it can be just another component.  So, for me it will be an Optional Array of Components.
Now, it will be developers responsibility to ensure he is importing/registering these components as part of the App Module.
This will open up a world of possibility in terms of adding interesting components to a form (like an stop clock timing your delay in entering data etc).
I have not studied your module in detail so I am not sure how the events will be emitted and control in such scenario.
As I said, I am new to Angular so pardon my newbie enthusiasm and mistakes.

@hpn1975
There are some interest thoughts in your message. But can't you already achieve this with current state of impementation?
<ng-template modelId="myInputModel">
    <my-stop-clock></my-stop-clock>
</ng-template>

I assume you're looking for a way to define that on the model itself, right?

You are right... Looking to drive it via Model.  Like putting in  some custom highlight... timers etc.

@hpn1975 Thank you for being creative and contributing your idea!
I think such a feature could be implemented by using a dynamic component loader.
For the moment I'd like to close that issue as this is a very advanced feature.
At first I'd like to deliver full and stable support for all UI libraries, especially Kendo and Material.


POST /licences fails

Hi @jbonnet
$http.post(ENV.apiEndpoint+"/licences",data)
with var data={"service_uuid":id, "user_role":$localStorage.currentUser.user_role};
returns {"error":{"code":400,"message":"{}"}}
Ther "user_role" field is on the request data since the initial develop version and it is not needed but I think that it shouldn't fail because of it, right?

Hi, @srodriguezOPT
I think PR #877 (and others before) fixes this... can you please confirm?
p.s.: thanks, @felipevicens !

Hi @jbonnet,
The GK is sending the private service with no user_licence field so BSS thinks that is public.

Ok, @srodriguezOPT
I'll close this issue and open a new one to support fixing that problem.


ardana: patch the ardana code before beginning to install

this is something I was using for sometime and found it useful, so sharing.

rebased

The reason for simple curl is this
pushd /tmp/patches/ && python -m SimpleHTTPServer -p 1234

then
cp my.patch /tmp/patches
and add url as
http://myhost:1234/my.patch
http://myhost:1234/your.patch
http://myhost:1234/that.patch

It might be nice if this patched /usr/share/ardana/ansible right before ardana-init is run, instead of /var/lib/ardana/openstack/ardana/ansible. ardana-init runs some playbook code, like the osconfig-ansible tasks.

Good idea @cmurphy

@aspiers I do want to increase the scope of this patch to such an extent.
@here
Thank you for the reviews so far but the PR is taking up far more of my resources than I had initially thought of dedicating, so closing.


Correct PAGE09 damage component parameters

The old PAGE09 damage component is currently using zero-value parameters which would result in zero damages if this component was used for market damages.
According to the PAGE-ICE excel file, W_MarketImpactsatCalibrationTemp is Triangular(0.2, 0.6, 1), resulting in a mean of 0.6.
ipow_MarketIncomeFxnExponent is Triangular(-0.3, -0.1, 0), resulting in a mean of -0.1333
iben_MarketInitialBenefit is Triangular(0, 0.1, 0.3), resulting in a mean of 0.1333
I checked these values against anthofflab/MimiPAGE2009.jl but they have W_MarketImpactsatCalibration at 0.5 with a Triangular(.2, .5, .8). So apparently, this was updated as part of PAGE-ICE.

Probably my fault: the Excel file is set to replace those values with zeros when the Burke economic impact is chosen, so I just pulled those values. This looks fine to me. Feel free to merge it.


Document how to add a new Alignment/Guidetree/Eval module

Description of feature
As we talked about, there should be something like a Checklist or Step-by-Step Tutorial on this in the documentation.

Set up a branch for this and some skeleton initial documentation: https://github.com/lrauschning/multiplesequencealign/tree/doc

Closed by #125


enforce damage issue

The output of command /version on my server is: git-Spigot-2cf50f0-8166d17 (MC: 1.12.2)
The output of command /version citizens on my server is: Citizens version 2.0.25-SNAPSHOT (build 1751)
The output of command /version sentinel on my server is: Sentinel version 2.0.1-SNAPSHOT (build 327)
A pastebin.com link to my (full!) Sentinel/config.yml file is: https://pastebin.com/nHCfrpPd

Hey there, I got a problem/bug with Sentinel.
If I set enforce damage to true in the config, sentinel npcs can also do damage to players in worldguard regions with the flag pvp = deny players. The problem is that the npcs can then no longer do any damage to mobs. They do attack the mobs, but no damage happens.
Unfortunately there are no error logs in the console.

Not able to replicate this problem.
Please test for a plugin conflict: https://wiki.citizensnpcs.co/Plugin_Conflict


Add Aster Linux Logo

Description
Please add my Aster Linux Logo to Neofetch repository
Features
Issues
TODO

Looks good!

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!

Actually, please put "Aster"* after "AsteroidOS"* because the wildcard matching in Aster* will always match anything that matches "AsteroidOS"*, but not the other way around.

Looks good!

Thanks! I updated my pull request to  @dylanaraps and installed hyfetch instead of neofetch, didn't knew about your repo before. It's amazing to keep projects like this updating with time


opening imv crashes hyprland

System Info and Version
 System/Version info
Hyprland, built from branch  at commit cba1ade848feac44b2eda677503900639581c3f4  (props: bump version to 0.40.0).
Date: Sat May 4 15:42:32 2024
Tag: v0.40.0, commits: 4606

flags: (if any)

System Information:
System name: Linux
Node name: envy
Release: 6.9.3-hardened1-1-hardened
Version: #1 SMP PREEMPT_DYNAMIC Sat, 01 Jun 2024 00:25:24 +0000

GPU information:
00:02.0 VGA compatible controller [0300]: Intel Corporation Raptor Lake-P [Iris Xe Graphics] [8086:a7a1] (rev 04) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

hyprland.conf
Bug or Regression?
Bug
Description
Somehow opening imv sometimes crashes hyprland, I've found it to work more consistently if you reduce the amount of env variables?
How to reproduce
env -i WAYLAND_DISPLAY=wayland-2 XDG_RUNTIME_DIR=/run/user/1000 imv-wayland ~/pic/lock.png
Crash reports, logs, images, videos
hyprlandCrashReport346825.txt

is it fixed on git?

It seems so. After I bit of testing I couldn't get it to crash again there.


Option 'Static Map': Seperate Zoom and Movement

Is is possible to make 2 options out of one?
-Static Zoom
-Static MapMovement
Because on Touchscreen (f.e. smartphone), if you scroll down in the HA-App and hover over the radar, you move the map with your finger and dont scroll down the Dashboard. Therefore would be a move-static-option nice.
And on the other hand, the zoom could be still active to use zomm in /out with 2-finger-gesture.

I'll second this,  trying to use the swipe card with this inside and cannot swipe past the radar card nor back,


Use chef-style comment header

Auto-corrected with Cookstyle
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger


Clarify docs and help messages on cli commands

Excited pipenv user here. I don't understand what several of the commands do. Here is the output of pipenv:
Commands:
  check      Checks for security vulnerabilities and against PEP 508 markers
             provided in Pipfile.
  clean      Uninstalls all packages not specified in Pipfile.lock.
  graph      Displays currently–installed dependency graph information.
  install    Installs provided packages and adds them to Pipfile, or (if none
             is given), installs all packages.
  lock       Generates Pipfile.lock.
  open       View a given module in your editor.
  run        Spawns a command installed into the virtualenv.
  shell      Spawns a shell within the virtualenv.
  sync       Installs all packages specified in Pipfile.lock.
  uninstall  Un-installs a provided package and removes it from Pipfile.
  update     Runs lock, then sync.

lock "Generates Pipfile.lock." Is it doing that based on the current environment, or based on what's specified in the Pipfile, or some other thing? Sometimes it seems to be accessing the internet, which is altogether unexpected.

Does open open the editor with the virtualenv bin directory prepended to the PATH? Or something else?

install "Installs provided packages". What are "provided packages"? Do they include the ones in my Pipfile? If no arguments are given, it installs "all" packages; which "all" are you talking about: in the Pipfile, Pipfile.lock, current active environment?

Which of these commands modify my environment? Which modify my Pipfile? Which modify my Pipfile.lock?

Could the cli help text and documentation clear this up, please?
Thank you for all your work on pipenv.

How Pipenv manage things can be generally thought in a linear manner:
Your mind
  ↓ [manual Pipfile edit]
Pipfile
  ↓ [lock]
lockfile
  ↓ [sync]
environment

There are three kinds of commands: essential (lock and sync, noted in the above graph), shorthands (combination of essential commands), and utilities (not related to essential commands, but make things easier to work with a project).
Shorthands:

install/uninstall [package] = edit the Pipfile (to add/remove the package), lock, and sync
update = lock + sync

Utilities:

clean = A companion of sync. Due to how Python packaging works, sync only makes sure everything you want is installed, but does not check if there are stray packages maybe you don’t need. This command plugs (sort of) this hole.
shell and run = Run things inside the virtual environment easily.
graph = Shows a visual representation of your environment.
open = Opens the editor. What do you mean with prepending PATH? An editor does not know about your PATH, this is not how editors work???

Pull requests on the help messages are welcomed.

@uranusjr Regarding open, I'm wondering whether pipenv open requests opens requests inside the virtualenv or not? This is relevant if the virtualenv contains configuration that the editor can access, like environment variables, linter executables, etc.

yes of course, pipenv manages virtualenvs.  Everything involved is about virtualenvs. It finds the module in your virtualenv and opens it in an editor. That's all there is to it.

@uranusjr Thanks for the detailed overview. That's quite helpful.

@apnewberry No problem at all. I’m quite bad at literature and writing help messages however, so it would be awesome if you could help incorporating them into the actual documentation and help output. I’d say you’re not the only one having this question.

Yes, they are correct in general. Two minor technical details:

Lock does not always download the most recent. Say if I specify Django (latest is 2.0.x), but one of my dependencies require 1.11.x, it will download the latest 1.11.11 (currently the latest in 1.11 series). Might want to clear this up.
Sync does not really download anything on its own. The cache is managed by pip, sync only tells it to install things. You can actually set PIP_NO_CACHE_DIR to disable it, and sync won’t care. Overall I think it would suffice to simply say sync hands the requirements in Pipfile.lock for pip to install them.

Really those are technical details, but better to do them absolutely right than risk giving people wrong impressions.

You can actually set PIP_NO_CACHE_DIR to disable it, and sync won’t care.

What do you mean by "sync won't care": in PIP_NO_CACHE_DIR=1 pipenv lock , Is the PIP_NO_CACHE_DIR value passed to pip?
I tried testing it but couldn't really understand what was supposed to happen in the pip usage of this option either (see https://github.com/pypa/pip/issues/2897).

Is the PIP_NO_CACHE_DIR value passed to pip?

Yes. Well, not passed, it’s an environment variable that pip picks up automatically. sync does not do anything with that flag.


⚠ Samsung has degraded performance

In 7e5fb58, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2654 ms

Resolved: Samsung performance has improved in 832f876.


Secure Cookie Enforcement Breaks Logins Behind Proxy

Expected Behavior
Upgrading to @backstage/plugin-auth-backend >= 0.6.0 and resolving all of the noted breaking changes should allow the app to continue to function.
Current Behavior
After upgrading (to 0.6.1, though the change was made in 0.6.0), we are unable to log into the application when deployed, as the connect.sid cookie is no longer being set, and the openid-client Passport Strategy requires sessions.
Possible Solution
The changes made in https://github.com/backstage/backstage/pull/8592, while good from a CodeQL standpoint, were breaking changes to anybody running behind a proxy that relies on express-session for their Passport Strategy.
The breaking change is caused due to express-session requiring the use app.set('trust proxy', 1); in the express app in order to run behind a proxy if enforcing secure cookies. (https://github.com/expressjs/session#cookiesecure)
There is not currently a way (that I am aware of) to set the trust proxy policy when using the ServiceBuilder, meaning that short of re-implementing the logic in the ServiceBuilder, there is nothing that can be done in the current state.
My recommendation is to revert the secure cookie change (or make it configurable) until such time as the hooks are in place to set the trust proxy policy in the underlying express server.
Steps to Reproduce

Rely on a Passport Strategy that requires sessions (such as OIDC)
Have the Backstage app running behind a proxy (AWS ALB in our case).
Update @backstage/plugin-auth-backend to >= 0.6.0.

Context
We are trying to keep our implementation up to date with the Backstage project, and this problem appeared after running backstage-cli versions:bump.
Your Environment
OS:   Linux 5.4.0-1063-azure - linux/x64
node: v14.17.6
yarn: 1.22.17
cli:  0.10.5 (installed)

Dependencies:
  @backstage/app-defaults                                  0.1.3
  @backstage/backend-common                                0.10.2
  @backstage/catalog-client                                0.5.3
  @backstage/catalog-model                                 0.9.8
  @backstage/cli-common                                    0.1.6
  @backstage/cli                                           0.10.5
  @backstage/config-loader                                 0.9.1
  @backstage/config                                        0.1.11
  @backstage/core-app-api                                  0.2.1, 0.3.1
  @backstage/core-components                               0.8.3
  @backstage/core-plugin-api                               0.3.1, 0.4.1
  @backstage/dev-utils                                     0.2.16
  @backstage/errors                                        0.1.5
  @backstage/integration-react                             0.1.17
  @backstage/integration                                   0.7.0
  @backstage/plugin-api-docs                               0.6.21
  @backstage/plugin-app-backend                            0.3.21
  @backstage/plugin-auth-backend                           0.6.1
  @backstage/plugin-catalog-backend                        0.19.4
  @backstage/plugin-catalog-import                         0.7.8
  @backstage/plugin-catalog-react                          0.6.10
  @backstage/plugin-catalog                                0.7.7
  @backstage/plugin-cost-insights                          0.11.16
  @backstage/plugin-explore-react                          0.0.9
  @backstage/plugin-explore                                0.3.24
  @backstage/plugin-github-actions                         0.4.30
  @backstage/plugin-graphiql                               0.2.26
  @backstage/plugin-org                                    0.3.33
  @backstage/plugin-permission-common                      0.3.0
  @backstage/plugin-permission-react                       0.2.1
  @backstage/plugin-proxy-backend                          0.2.15
  @backstage/plugin-scaffolder-backend-module-cookiecutter 0.1.7
  @backstage/plugin-scaffolder-backend                     0.15.19
  @backstage/plugin-scaffolder-common                      0.1.2
  @backstage/plugin-scaffolder                             0.11.17
  @backstage/plugin-search                                 0.5.4
  @backstage/plugin-shortcuts                              0.1.18
  @backstage/plugin-sonarqube                              0.2.11
  @backstage/plugin-tech-radar                             0.5.1
  @backstage/plugin-techdocs-backend                       0.12.2
  @backstage/plugin-techdocs                               0.12.13
  @backstage/plugin-user-settings                          0.3.15
  @backstage/search-common                                 0.2.1
  @backstage/techdocs-common                               0.11.2
  @backstage/test-utils                                    0.2.1
  @backstage/theme                                         0.2.14
  @backstage/types                                         0.1.1
  @backstage/version-bridge                                0.1.1

Thank you for reporting, and sorry for the breakage!
So if I understand the feature from express-session correctly, it's avoiding sending cookies back in HTTP requests in case someone is snooping in on those? It seems like a bit of an overly defensive check since if you're worried about that you'd usually enforce HTTPS ahead of the session handling.
I think it makes sense to revert the secure cookie change for now tbh, as we're not storing anything particularly sensitive just yet so not too critical that it is available. Another option could be to use the 'auto' option for now, as that will still let people use secure cookies in some cases, but it won't work in the proxy case. Yet another option is to introduce something like a backend.trustProxy config, it won't fix the breakage but is probably closer to the state that we want to be at.
Regarding access to the express app, we've pretty much settled that the ServiceBuilder is operating at the wrong abstraction level and needs to be broken down into pieces that compose better. Not a short-term fix ofc, but something we're likely to look at this spring.
As a workaround for now you could perhaps add a middleware before the auth backend that sets req.secure?
const authRouter = Router()
authRouter.use((req, _res, next) => {
  req.secure = true
  next()
})
authRouter.use(await auth(authEnv))

apiRouter.use('/auth', authRouter);

Didn't test the above so I'm not sure if req.secure is writable, other options are req.connection.encrypted and req.headers['x-forwarded-proto'] = 'https', basically something that makes the check at https://github.com/expressjs/session/blob/66634e9f77132a444dfda7e279a399054e3b28df/index.js#L623 pass

I think you are correct in your understanding of the express-session feature, but to re-word, it avoids returning the set-cookie header for the connect.sid if:

The cookie was set to be secure AND
It cannot determine that the connection is secure for a number of reasons

I will attempt the workaround, but probably won't know for certain until tomorrow or Friday.  I will reply to this issue whether or not I was able to workaround.
With regards to the options to resolve:

auto should work on the proxy, as it sets the secure boolean to the result of the issecure call, which in the current case of the proxy returns false, essentially making the behavior as it was before.  The check on authUrl.startsWith('https') can also be removed at this point, as that is only one part of the ability to set a secure session.
In my opinion, adding a backend.trustProxy would probably be too narrow of a solution, as it just solved this one specific problem.  I think it would be more beneficial to have a builder.configureExpress(callback); method on the Builder (even after it is broken down) to allow for app owners to configure the Express app directly.  This would allow not only solve the trust proxy problem, but any future express modifications that may be desired.

I was able to sneak a code change in-between meetings and run locally (not behind a proxy, yet).  Unfortunately, req.secure is a get-only.  I'll look through the source for req.secure and see if there is anything else I can modify.  Below is the error I see in the login popup.
{
        "error": {
                "name": "TypeError",
                "message": "Cannot set property secure of #<IncomingMessage> which has only a getter",
                "level": "error",
                "appName": "backstage",
                "environment": "local",
                "version": "local"
        },
        "request": {
                "method": "GET",
                "url": "/api/auth/oidc/start?scope=&origin=http%3A%2F%2Flocalhost%3A3000&env=local"
        },
        "response": {
                "statusCode": 500
        }
}

Ah shiet, was worried about readonly req.secure 😅, you could try Object.defineProperty, or the other options

I was able to do a workaround by registering the middleware in the authRouter, virtually identical to the setup in the plugin-auth-backend router config, except changing the secure cookie to auto.  I wasn't sure what (if any) impact having the express-session middleware registered twice would have, but it would appear (at least initially) there is none.
I'll put this in place until there is a more permanent resolution to this issue.  Thanks for the assistance!
  const authRouter = Router();
  const secret = config.getOptionalString('auth.session.secret');
  if (secret) {
    authRouter.use(cookieParser(secret));
    authRouter.use(
      session({
        secret,
        saveUninitialized: false,
        resave: false,
        cookie: { secure: 'auto' },
      }),
    );
  }
  authRouter.use(await auth(authEnv));

  apiRouter.use('/auth', authRouter);

I encountered the similar issue. Is that the recommended approach?


Add delay option to page tracking call

Hi there,
Following this PR, I wanna have the ability to configure the delay applied to the page tracking call. Seems that sometimes, 50ms is not enough on my side so the tracking vs page update is not synced properly. I think it'd be better to let the user be able to set the delay that fits him the most than bumping it up.

did you see this btw?  https://github.com/benjaminhoffman/gatsby-plugin-segment-js/pull/34

I did, but seems to me that it describes a different use case, where the delay applies to the initial load of Segment instead of the page tracking event applied at every route change like I need.
I will update the comment per you reply.

@TomPridham thoughts on this?  i understand the use case but goes to what i was saying few days ago about setting bloat.  maybe it's inevitable tho.

i think this could be useful, and it's a fairly minimal change

@TomPridham I've made the updates and resolved the conflicts, all clear.

just published @sreucherand


Use ContentReveal for delegates' large description

It's maybe me, but I think having a modal for the large description for delegates is not intuitive.
I'd suggest to have the same design as the locks with a ContentReveal component.
WDYT?

Completely agree. That was during the Hackathon that we were hasty.
Will definetely check the ContentReveal and fix it.

In addition a small redesign on the Delegate Cards will take place:

Remove the 3 dots button and replace it with a delegate button.
add the Content Reveal button
add the address (+copy) in the main card (instead of the popup)


NgsF

script to run through ngsF from ngstools.  TVK working on initial version. Should ideally output file of values for use in thetas etc. scripts.

whoops added a new issue for this.  Twice the importance?


[build] only install ARM-specific dependencies on an ARM architecture

In the interest of not installing things where they are not required, this PR adds a check for the local architecture and only installs some dependencies when indeed needed.
Notes:

explicitly add make to the first package install: it is required by jsonnet and probably included implicity via the ARM dependencies

Note: the last failure affected two seemingly random configurations with 502's from official docker registries
 rid_bootstrapper Error Head "https://registry-1.docker.io/v2/interuss/dss/manifests/v0.17.0": received unexpected HTTP status: 502 Bad Gateway
 crdb Error context canceled
 scd_bootstrapper Error context canceled
Error response from daemon: Head "https://registry-1.docker.io/v2/interuss/dss/manifests/v0.17.0": received unexpected HTTP status: 502 Bad Gateway