Tae test

Agility ticket
Summary
Is there anything you would like reviewers to give additional scrutiny?
this article explains more about the approach used.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against f9f73f7751e519324066fa549ce2c67a3fc4896c

Test coverage on server failed.
Last run: Fri 13 Dec 2024 02:53:03 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


Fix indent at src/components/NavBar/style.sass

Related to Issue #210
Description
Fix indent at styles.sass
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)

Checklist:

[X] My code follows the code style of this project.
[X] All new and existing tests have passed and it does not give any unexpected error for the same.

@Abhijay007 @charithccmc
Good Morning sir is this issue fixed correctly if not can i work on this.


feat(operator): allow ranges to be specified for package filtering

Description
This PR adds the ability to specify a MaxVersion when package filtering on operators.
It also changes the MinVersion and MaxVersion from semver.Version types to strings to allow for DeepCopy methods to be used on API types later.
(Pointing to an operator-registry fork)
Closes #302
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[X] Test cases added to E2E
[X] Test cases added to unit test for conversion to DiffIncludeConfig

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
[ ] Any dependent changes have been merged and published in downstream modules

Putting this back to WIP pending the merge in operator-registry.
https://github.com/operator-framework/operator-registry/pull/943

I'm gonna lgtm on this PR and put a hold on it while you two work out the discussion regarding the range option. When you reach a decision, remove the hold for the PR to merge.

/hold
/lgtm

/lgtm

@dinhxuanvu Alex and I discussed and would prefer to keep the maxVersion and minVersion to limit the range flexibility. If you want to discuss more, we can submit a follow on PR to tweak.
/hold cancel


Use datatype: BirthDate and broader gender picker for "what is your name" screen

It's hard to enter a birthdate with the stock date picker.
Gender "other" is also not very inclusive--suggest using standard AL gender picker. https://suffolklitlab.org/docassemble-AssemblyLine-documentation/docs/question_library/gender

Might be silly but can we edit BirthDate datatype? The UK writes dates as DD/MM/YYY

Interesting point! I think the BirthDate picker has the right order of fields though--the idea is to have the alphabetic month drop down menu as the first input to help avoid mistakes. It isn't meant to be a locale-specific question.
This was part of the research we did when designing it that way, to match how Google asks it: https://ux.stackexchange.com/questions/91019/best-display-for-date-of-birth-inputs

Great! So interesting how much consideration often goes into seemingly simple fields. I've changed made the suggested changes.


Unused dyndns service keeps restarting the VPN connection

This is using Suricata in IPS mode and squid as a transparent proxy with ACLs.
Here is what I see in the logs
Jan 25 13:00:11	inetd[68786]: 19000/tcp: unknown service
Jan 25 13:00:10	configd.py: [u-u-i-d] Reloading filter
Jan 25 13:00:10	opnsense: /usr/local/etc/rc.openvpn: OpenVPN: One or more OpenVPN tunnel endpoints may have changed its IP. Reloading endpoints that may use VPN_TCP_VPNV4.
Jan 25 13:00:09	configd.py: [u-u-i-d] Restarting OpenVPN tunnels/interfaces VPN_TCP_VPNV4
Jan 25 13:00:09	configd.py: [u-u-i-d] Restarting ipsec tunnels
Jan 25 13:00:09	configd.py: [u-u-i-d] updating dyndns VPN_TCP_VPNV4

I'm not using dyndns, nor am I using IPsec.

did you only enable ips (netmap) on the physical device?

I didn't do anything else than ticking the box in Intrusion Detection and rebooting (since it was unstable)

If I don't monitor the VPN gateway, then everything is fine, but then I don't know when it goes down.

@oparoz You're not providing your configuration, which defeats the point of getting help. Is DynDNS disabled (or what do you mean by "unused") and which devices do you use for IPS mode...

Unused as in "I did not add any DynDNS service"
There is no DynDNS service listed under System: Diagnostics: Services
Suricata is set to filter both WAN and VPN_WAN
The Suricata command line  /usr/local/bin/suricata -D --netmap --pidfile /var/run/suricata.pid -c /usr/local/etc/suricata/suricata.yaml

netmap is enabled on WAN
re1: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
        options=182098<VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM,WOL_MAGIC,LINKSTATE,NETMAP>

And monitoring the WAN_VPN changed nothing in the end, I get the same messages in the logs.

@oparoz again, you can not enable IPS on a non physical interface "VPN_WAN"

@AdSchellevis OK, I've opened this then, so that users are not allowed to add any other interface.


Questions for citation format and data access

Pitch
I am currently a graduate student specializing in Communication and am in the process of writing my thesis. My research has led me to have a keen interest in the content displayed on the Reddit map page.
Would it be possible for me to obtain the data that underpins this map? Additionally, could you guide me on how to appropriately cite this data in my academic work?
Your support would be invaluable to my research. I genuinely appreciate your consideration and assistance in this matter.
P.S. I have reached out through this channel as I found it challenging to locate an official point of contact.
Motivation
Contribute to further research using this page.

Thanks for your message! I'm Kevin, researcher at UMass Amherst and the community manager for the Reddit Map project.
When it comes to citing the map, we'd ask that you cite this preprint that our research team has shared on arXiv: https://arxiv.org/abs/2309.14259
The preprint explains our approach to the data and preprocessing that underlies the clustering that is seen on RedditMap.social. The Python module for subreddit data processing and clustering is available here: https://github.com/UMassCDS/IHOP-Reddit/. If you have questions, please file another issue on GitHub and we'll get back to you when we can (we are an academic lab with many projects and limited resources, so our response times may be slow).
The Reddit comments dataset that our team used comes directly from Pushshift, and ongoing changes with Reddit's API and Pushshift's agreements with Reddit are impacting our ability to process data on a regular basis as we had intended for this project. We cannot provide you with Pushshift's comments data, but you may be able to obtain them on your own at https://pushshift.io/signup.


Very helpful thanks

These are very helpful, but I think there is a typo in the nginx-http-no-method jail - works for me if I use nginx-http-no-verb
Name of conf should match jail filter.

Thanks! That was a copy-paste issue from my internal system where the name for historical reasons is nginx-http-no-method. I have fixed it now.


Bmi calc/validation

Description
Added Validation to BMI Calculator for standard and metric
Removed calc function so BMI total would update automatically
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Change Status

[X] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[X] Tested out validation and calculation total on local server
[ ] Test B

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] 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] There are no merge conflicts

 BMI-Calc / Validation

Everything is looking good. It looks like the Labels and Inputs aren't associated together which will cause some accessibility issues. The labels are missing htmlFor attribute and inputs are missing the id attribute to associate the label with.

Thanks for catching that, I'll go ahead and look into it and make those changes.


Could not find 'svn2git' (>= 0) among 58 total gem(s) (Gem::LoadError)

Running svn2git gives me this error:
/usr/lib/ruby/2.1.0/rubygems/dependency.rb:298:in `to_specs': Could not find 'svn2git' (>= 0) among 58 total gem(s) (Gem::LoadError)
	from /usr/lib/ruby/2.1.0/rubygems/dependency.rb:309:in `to_spec'
	from /usr/lib/ruby/2.1.0/rubygems/core_ext/kernel_gem.rb:53:in `gem'
	from /usr/local/bin/svn2git:22:in `<main>'

It has worked in the past, about 6 months ago, so I have a hunch that it's related to an update of Ruby.
I am currently using Ubuntu 15.04, before is was using Ubuntu 14.10.
Ruby version: ruby 2.2.1p85 (2015-02-26 revision 49769) [x86_64-linux]
I have /home/amedee/.rvm/gems/ruby-2.2.1/bin:/home/amedee/.rvm/gems/ruby-2.2.1@global/bin:/home/amedee/.rvm/rubies/ruby-2.2.1/bin in my $PATH, before /bin

I’m seeing the same error under Mac OS X 10.6.8 Snow Leopard.
Is there a specific location that the svn2git gem should be installed?

If you're using the built-in Ruby on MacOS X, I think you need to install with "sudo gem install svn2git". Did you do that? Or did you just run "gem install svn2git"?

It was the sudo gem install svn2git that caused the issue. The gem was installed, but in the wrong location. When I reinstalled using gem install svn2git, svn2git was installed alongside my other gems and worked perfectly.

Good to know. It's been a while since i've set up with the system Ruby on MacOS X. I couldn't recall which was the correct form to use. Glad to hear it's working for you! I'll look at adding a caveat to the README.

It could have something to do with my particular install of Ruby on Snow Leopard, and not necessarily Mac OS X in general.
For example, sudo gem install svn2git installed the gem into:
~/.rvm/rubies/ruby-1.9.3-p429/bin/svn2git
Whereas gem install svn2git installed the gem into:
~/.rvm/gems/ruby-1.9.3-p429/bin/svn2git
… which is where the other active gems resided.

Ahh, yeah. If you're using RVM you shouldn't ever need to "sudo gem install" anything. Doing so almost always leads to problems like this :-/

The more you know.
(I’m a bit of a Ruby Nuby, as you can tell.)


Terminal range not infinite

The GUI says its got lvl4 Beacon (which it does). However, going more than 64 blocks away (or another dimension), it gives the "terminal out of range" error.
It worked perfectly fine, until I moved the inventory system to another beacon.  (completely destroyed the other one)
It is exactly 7 blocks away from the beacon item.
I relocated the terminal to be right next to the beacon, and its still not working as it should be. no idea what would cause it to not carry the beacon effect as it should.

sorry for not including: running MC 1.19.2, fabric, mod version v1.5.6.  (included as a part of Better MC)

Make sure the terminal is chunk loaded

Make sure the terminal is chunk loaded

That means it's not infinite, please update the description to include "as long as the chunk is loaded".
In order for it to be infinite, users need an additional chunk loader mod.

That is just how minecraft works. You can enter a forceload command to keep a chunk loaded without an additional mod.

except when you are on a public server that you do not have that command ability.
If the chunk requires force loading, then one would think it would be in the description, or do the forceloading inside the mod that requires it.

except when you are on a public server that you do not have that command ability. If the chunk requires force loading, then one would think it would be in the description, or do the forceloading inside the mod that requires it.

That is your personal problem, not the mods. If the server does not allow chunk forceloading, its not mods fault you cant access your terminal

The mod could be a bit more precise on the requirements, not everyone knows that you must forceload the chunk of your storage system.
I have a server with casual players and they can't figure this out by themselves.
Please add the forceload notion to the description.


Develop

Codecov Report

Merging #32 (485dfcf) into release (a31a93a) will increase coverage by 0.92%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           release      #32      +/-   ##
===========================================
+ Coverage    69.12%   70.04%   +0.92%
===========================================
  Files           28       29       +1
  Lines          583      601      +18
  Branches        68       70       +2
===========================================
+ Hits           403      421      +18
  Misses         180      180

Impacted Files
Coverage Δ

src/auth/auth.controller.ts
100.00% <100.00%> (ø)

src/auth/auth.service.ts
100.00% <100.00%> (ø)

src/auth/dto/request/refresh.dto.ts
100.00% <100.00%> (ø)

src/auth/dto/response/accessToken.dto.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a31a93a...485dfcf. Read the comment docs.


break up index.js into files per class

Proposed changes
Break up index.js into separate files

Add AudioEngine.js
Add AudioPlayer.js
Add Loudness.js

Reason for changes
Some code organization before adding tests and overhauling scratch-audio with a different architecture.

(but @ericrosenbaum should look before merging)


Is there any way I can obtain routes with specified origin

I wonder is there any function I can use to obtain routes with specified origin.
It should has the similar results as the !g option, for exmaple
whois -h whois.radb.net '!gAS19679'.
A1761
<IP_ADDRESS>/21 <IP_ADDRESS>/20 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/21 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/21 <IP_ADDRESS>/22 <IP_ADDRESS>/22 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/23 <IP_ADDRESS>/23 <IP_ADDRESS>/23 <IP_ADDRESS>/23 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/20 <IP_ADDRESS>/24 <IP_ADDRESS>/16 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/18 <IP_ADDRESS>/20 <IP_ADDRESS>/24 <IP_ADDRESS>/20 <IP_ADDRESS>/20 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/20 <IP_ADDRESS>/22 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/21 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/21 <IP_ADDRESS>/21 <IP_ADDRESS>/21 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/26 <IP_ADDRESS>/20 <IP_ADDRESS>/21 <IP_ADDRESS>/22 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/23 <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24
C

It is preferred to return a list of ipaddress network objects or at least a list of CIDRs instead of just plain text.

ipwhois.asn.ASNOrigin pulls routes and other info already:
https://ipwhois.readthedocs.io/en/latest/ASN.html#asn-origin-usage-examples
Correct me if I'm wrong, but !g just returns the same routes, but without all the other information.

I found several issues in current project.

the ASNOrigin initiation should not require Net object

The ASN origin lookup does not require any IP or network parameter. Thus the ASNOrigin should not require this. However, in current code, we have to initiate an ASNOrigin object by passing a Net object. It can be any Net object which is completely irrelevant to target ASN. So it is very confusing. The only reason, that the current code has it, is because it requires several member methods from Net class.

The ASN lookup related functions should not be in net.py at all.

As we said before the ASN lookup has nothing to do with IP/network, thus the get_asn_origin_whois and get_htp_raw should not located in net.py as member methods. The only resources shared in Net class is timeout and the proxy_opener, which can easily be passed as arguments.

The ASNOrigin may not be class at all.

Since the only shared member in ASNOrigin class is the useless/unwanted Net object. After we move the needed functions out, we can avoid define the ASNOrigin as class. The entire thing will be much more clear.

successful query does not stop looping through all asn_methods

One would expect avoiding http method if the whois method works. However, the current code repeats the lookup query with get_http_raw(). Thus it slows down the process dramatically (normally 2-4 times slower than os whois command)

Here is my test results before and after my modification.

Thanks for the feedback, see my responses below.

 I did this as a quick new feature prior to 1.0.0; this was to avoid delaying the release, which I expedited due to several bugs. The plan is to create a base class for proxy support, but I didn't implement this for that release due to pending changes in #153.

I decided a while ago to group all functions that perform network queries in a single location. I don't plan to change the file for these. I do, however, plan to break out any non-object dependant functions from the Net class.

I made this a class with future growth under consideration. I agree that its current state does not benefit from being a class.

Let me look into this one as a potential bug.

I can't accept your pull request for the following reasons:

See my above responses regarding the code changes
I will not accept a request that modifies the license. I give recognition to contributors via the changelog, and substantial contributors in the readme.
Your Travis/tests failed
This project is now following GitFlow: https://ipwhois.readthedocs.io/en/latest/CONTRIBUTING.html#gitflow-model

On a similar note, I want to be able to take any ASN and find its country without having to specify an IP address or network. Is there a way to do this in the current version of ipwhois?

@ZaneHintzman Not currently. I know you can get this data via Cymru ASN lookups. Marking this as an enhancement.


Bug in JSON parser example: strings are assumed to be alphanumeric

Hello!
I've got a minor nitpick:
If I read it correctly, in the example on JSON parsing (https://github.com/Geal/nom/blob/master/examples/json.rs#L58), it is assumed that strings are alphanumeric, which breaks down on strings like Hello world!.
Thanks for a great library, otherwise!

If I remember correctly what @geal told me when I asked him about that parser, this is just an example of how to write parsers, and it is not designed to be a full "correct" JSON parser, for the simple reason that JSON is loosely defined and there are as many "specifications" as there are parsers. So I don't want to speak for him but I don't think this is really a bug; the JSON example is supposed to be the base to write your own parser I believe.


Improve README.md

We can improve "How to use" in README.md  to include new feature

Yes, I'll do that soon.

I have added the options tab in the readme

good work


Add Vagrant File for both Centos and Ubuntu

We need to create a Vagrantfile for both Ubuntu and Centos that will serve as our base OS for Docker.

Removed centos for now due to performance concerns...closing issue for now. commit d8281e0e6e379cb0c11f2af16ff07ee5364e55de


Move attachments with note

Sometimes one moves a note. It's attachments are then left dangling in the previous location.
Would it be possible to have them moved with the note? Might be out of scope for this plugin, I realize.
For my personal use case it would be sufficient to look for a filename match. So if the attachment name contains the note name it is considered as a child of that note, and moved with it.
Different naming schemes might complicate this logic / not work at all.

https://github.com/dy-sh/obsidian-consistent-attachments-and-links
is doing the wanted


The tenant list should have a Reset action per tenant

The Reset action would perform the following tasks with a tenant:

Display the following confirmation prompt:

"Warning: All tables [whose names start with DataTablePrefix] will be deleted from database 'DatabaseName'. Are you sure you want to proceed?"

Set the tenant state to Uninitialized in ShellSettings (Settings.txt)
Delete all database tables prefixed with `DataTablePrefix´ from the tenant's configured database
Delete all contents from the tenant's Media folder (there could be a checkbox for this)
Restart the tenant (distributed signals would propagate to other farm nodes)

After the operation is complete, the tenant is in a clean uninitialized state and can go through setup.

Fixed in b6d0fa3dd216a12051fdd10b9cc69742ad69437c.


Continue error on "url_encoded_fmt_stream_map"

Still receiving this in pytube '9.1.0'
File "...", line 26, in download_local
yt = YouTube("https://www.youtube.com/watch?v=" + id )
File "/usr/local/lib/python3.6/site-packages/pytube/main.py", line 84, in init
self.prefetch_init()
File "/usr/local/lib/python3.6/site-packages/pytube/main.py", line 93, in prefetch_init
self.init()
File "/usr/local/lib/python3.6/site-packages/pytube/main.py", line 125, in init
mixins.apply_descrambler(self.player_config_args, fmt)
File "/usr/local/lib/python3.6/site-packages/pytube/mixins.py", line 81, in apply_descrambler
for i in stream_data[key].split(',')
KeyError: 'url_encoded_fmt_stream_map'

can you please provide a video id that causes this?

many, for examples:
nBLA0H2gTTc
SSEq6lkRARY
49_Eoa2iOgE
n3O1wXVKiFw
xs_Shf_VMiI

and more, like this
rEErXHmm-cQ
but seems like version 8.0.2 still does the job


BINDINGS: Add Umka bindings to BINDINGS.md

raylib-umka provides Umka scripting language bindings to raylib.

@RobLoach Wow! Great addition! Thanks! 👍😄


[SPARK-28256][SS] Failed to initialize FileContextBasedCheckpointFileManager with uri without authority

What changes were proposed in this pull request?
reproduce code:
CREATE TABLE `user_click_count` (`userId` STRING, `click` BIGINT)
USING org.apache.spark.sql.json
OPTIONS (path 'hdfs:///tmp/test');

INSERT INTO user_click_count SELECT xxx from ${streaming_source_table};

error:
java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
	at org.apache.hadoop.fs.AbstractFileSystem.newInstance(AbstractFileSystem.java:136)
	at org.apache.hadoop.fs.AbstractFileSystem.createFileSystem(AbstractFileSystem.java:165)
	at org.apache.hadoop.fs.AbstractFileSystem.get(AbstractFileSystem.java:250)
	at org.apache.hadoop.fs.FileContext$2.run(FileContext.java:342)
	at org.apache.hadoop.fs.FileContext$2.run(FileContext.java:339)
	at java.security.AccessController.doPrivileged(Native Method)
	at javax.security.auth.Subject.doAs(Subject.java:422)
	at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1911)
	at org.apache.hadoop.fs.FileContext.getAbstractFileSystem(FileContext.java:339)
	at org.apache.hadoop.fs.FileContext.getFileContext(FileContext.java:456)
	at org.apache.spark.sql.execution.streaming.FileContextBasedCheckpointFileManager.<init>(CheckpointFileManager.scala:297)
	at org.apache.spark.sql.execution.streaming.CheckpointFileManager$.create(CheckpointFileManager.scala:189)
	at org.apache.spark.sql.execution.streaming.HDFSMetadataLog.<init>(HDFSMetadataLog.scala:63)
	at org.apache.spark.sql.execution.streaming.CompactibleFileStreamLog.<init>(CompactibleFileStreamLog.scala:46)
	at org.apache.spark.sql.execution.streaming.FileStreamSinkLog.<init>(FileStreamSinkLog.scala:85)
	at org.apache.spark.sql.execution.streaming.FileStreamSink.<init>(FileStreamSink.scala:98)
	at org.apache.spark.sql.execution.datasources.DataSource.createSink(DataSource.scala:297)
	at org.apache.spark.sql.execution.datasources.FindDataSourceTable$$anonfun$apply$2.applyOrElse(DataSourceStrategy.scala:379)

...

Caused by: java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.apache.hadoop.fs.AbstractFileSystem.newInstance(AbstractFileSystem.java:134)
	... 67 more
Caused by: org.apache.hadoop.HadoopIllegalArgumentException: Uri without authority: hdfs:/tmp/test/_spark_metadata
	at org.apache.hadoop.fs.AbstractFileSystem.getUri(AbstractFileSystem.java:313)
	at org.apache.hadoop.fs.AbstractFileSystem.<init>(AbstractFileSystem.java:266)
	at org.apache.hadoop.fs.Hdfs.<init>(Hdfs.java:80)
	... 72 more

How was this patch tested?
N/A

Test build #107262 has started for PR 25056 at commit 89216cb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12412/
Test PASSed.

Test build #107262 has finished for PR 25056 at commit 89216cb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107262/
Test FAILed.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12415/
Test PASSed.

Test build #107265 has started for PR 25056 at commit 89216cb.

Test build #107265 has finished for PR 25056 at commit 89216cb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107265/
Test FAILed.

I've had a look and this can be unit tested so +1 on having an initially failing test.
I't easier to see what's the problem + the proposal.

Just for the sake of my understanding what's happening when path changed to hdfs://tmp/test? I remember in a cloudy way I've faced similar problem where such step overcome the situation (this is definitely not the solution but to understand things better).

@gaborgsomogyi @HeartSaVioR Thanks for review

this can be unit tested so +1 on having an initially failing test.

I will add a unit test later.

Just for the sake of my understanding what's happening when path changed to hdfs://tmp/test? I remember in a cloudy way I've faced similar problem where such step overcome the situation

The tmp here in PATH is just the authority field, right? It will fail directly with java.net.UnknownHostException but not overcome the situation.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15668/
Test PASSed.


radard: calculate lead acceleration on cluster of tracks

or initialize from the model (it's a prediction on two frames rather than three, so it cannot ever be too accurate currently).
Right now we track lead accel per reported radar track. Lead accel is very important for the planner during high lead accel/decel situations.
We are fortunate that Toyota's reported radar tracks are mostly self consistent, in that a position/lead in space doesn't switch very often between the messages. However, it does happen. Here we could have reached 1 m/s^2 20% faster if the lead accel did not oscillate/start from 0:
https://connect.comma.ai/a2bddce0b6747e10/0000030f--995be3b11f/153/182

Another case a2bddce0b6747e10/000003eb--0f891e5a10/9:

Another case
https://connect.comma.ai/57048cfce01d9625/0000034a--9f0d6fc36b/103/130

Another example on Toyota: https://connect.comma.ai/6ac3968bd1f66985/0000003b--e11775e688/430/482


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2020.6.91350
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.46.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.


Full console: suppress new line when echo is disabled

When echo is switched off via console_echo(0), also new line should be suppressed.

Hi @dwld, I tell you why this takes so much time with such simple reasonable change.
For a long time console code that was modified by a few people did not really handled few control characters resulting in some echo while console was configured to be silent except for NLIP protocol.
In late 2019 I changed it so not echo for CR LF (and other control characters showed) but one company that uses mynewt was affected since they relayed on this not so obvious behavior.
So 153dfb8b7d13f0cf16298fd6e3b5235909fe2f30 restored echo for CR/LF just as not to break backward compatibility.
I probably should have add conditional code for allowing this echo if someone really needs it, instead of always, which is not something that would be expected from the code.
So that is why no one is rushing to accept this.

Hi @kasjer, thanks for the explanation. I understand. For our use cases, this behavior breaks the asynchronous communication. Would the introduction of another switch to suppress the echo for CR/LF be an option?

Hi @dwld quick question, are you using nlip protocol or some other custom protocol and you are not really interested in nlip handling?
If you are not going to use nlip (that seem like reasonable option) that I will add switch to just disable nlip handling and this will reduce code size and runtime cpu usage.
But what I've noticed when looking at you proposal is that console_echo() function that is public api for disabling console echo and it should not be used inside console code to turn on/off echo for other reason as it is done currently. And this also cries for fix.
btw you proposal for call console_echo(echo); would not make any difference and could be just totally dropped to achieve same goal.
Fixes are on the way...


Tree-widget: fixes for itwinUI-react 3.13

Fixes #1021

There is a new iTwinUI version 3.14.0. I think it should be updated in this PR. https://github.com/iTwin/iTwinUI/releases/tag/%40itwin%2Fitwinui-react%403.14.0


JFXDatePicker inside JFXDialog bug

The label indicating the current month/year is not correctly positioned, overlapping the navigation arrows.

JFXTimePicker inside JFXDialog is even worse :/

Hello,
Please note that JFXDialogLayout applies style classes to heading, body and actions container.
in the main demo, we are applying the following css:
 .jfx-layout-body .label {
    -fx-font-size: 14.0px;
    -fx-pref-width: 400.0px;
    -fx-wrap-text: true;
}

this will update any label in JFXDialogLayout including date/time pickers. you can fix this by changing in jfoenix-main-demo.css and put .jfx-layout-body > .label instead of  .jfx-layout-body .label.
in case you were using jfoenix-main-demo.css in your application, it's advised to review the css file first as it's not written to be used as a template.
Regards,


Adding support for building an aarch64 image on dockerhub (x86)

Patch for #287
Don't merge - if accepted it should stay on an aarch64 branch, unless we can parameterize the build using https://github.com/fabric8io-images/fish-pepper.

What is the intent of this pull request?  All the links are changed and the tests are broken.  Looks like your personal fork.

Closing this PR in favor of https://github.com/kylemanna/docker-openvpn/pull/292, which simply adds an Dockerfile.aarch64 file for an aarch64 container.

Hi, I'm confused. Just tried, from docker hub, to run openvpn in my raspberry pi, and it didn't work. (gave an error about not being able to run in my arm architecture). What is going on? Shouldn't it be working by now?

@lucaszanella this wasn't merged for the reasons exposed in #292
You can however build a working version on your pi:
git clone https://github.com/kylemanna/docker-openvpn.git
cd docker-openvpn
docker build -t kylemanna/openvpn .

A alternative way, to build the openvpn image from github directly
docker build -t kylemanna/openvpn https://github.com/kylemanna/docker-openvpn.git


TypeError when asking y-xml.getDom() object

Hi,
I get a TypeError: this._model is undefined when I try to cal Y-XML.getDom.
The code I'm using is:
 var connector = new Y.WebRTC("test", {url: config.server});

 var y = new Y(connector);
 var yxml = new Y.Xml(document.getElementById("comm"));

 connector.whenSynced(function() {
   y.val("xml", yxml);
 });
 yxml.getDom(); // ← error here

You have to put yxml on the y object first, before you can use it. Otherwise the types model cannot be defined. You can work on your shared structure even when you are not synced yet: so do the following
 var connector = new Y.WebRTC("test", {url: config.server});

 var y = new Y(connector);
 var yxml = new Y.Xml(document.getElementById("comm"));

 // Set it here
 y.val("myxml",yxml)

 /* leave this
 connector.whenSynced(function() {
   y.val("xml", yxml);
 });
 */
 yxml.getDom(); // ← error here

I'm going to implement a meaningful errormessage for this.

Wouldn't it leave me with two objects ("xml" and "myxml" in my y object) ? If so, what's the purpose of doing so?

I commented out the following portion of code:
 /* leave this
 connector.whenSynced(function() {
   y.val("xml", yxml);  // you have to execute this _before_ you call yxml.getDom
 });
 */

So you would end up with only "myxml" in your y object. The reason why I commented it is that the callback function is executed sometime after the yxml.getDom() is executed. The following will also work:
var connector = new Y.WebRTC("test", {url: config.server});

var y = new Y(connector);
var yxml = new Y.Xml(document.getElementById("comm"));

// Set it here
y.val("myxml",yxml)

connector.whenSynced(function() {
   y.val("xml", yxml);
   yxml.getDom(); // ← no error here
});

I close this because I implemented a warning


[SPARK-9582] [ml] Clarified log likelihood for LDA models

Small cleanups to recent LDA additions and docs.
CC: @feynmanliang

Merged build triggered.

Merged build started.

Test build #39618 has started for   PR 7916 at commit c566915.

Test build #39618 has finished for   PR 7916 at commit c566915.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

@feynmanliang Thanks for the comments.  I think that should fix things.

Test build #39627 has started for   PR 7916 at commit 97947aa.

LGTM pending tests

Test build #39627 has finished for   PR 7916 at commit 97947aa.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

OK, I updated it so (I think) it broadcasts all of the large stuff.

Merged build triggered.

Merged build started.

Test build #39656 has started for   PR 7916 at commit f7021d9.

Test build #39656 has finished for   PR 7916 at commit f7021d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

@feynmanliang @hhbyyh  I'll wait for a final OK from you.

Looks fine

OK thanks, merging with master and branch-1.5


[WIP] (a) Diagnose overflows during assignment of floating point literals

(b) Diagnose overflows during conversion of floating point values to integers.
(c) Test cases for checking correctness of diagnostics for floating point
calculations.

@swift-ci Please smoke test

@swift-ci please smoke test

@swift-ci  please smoke test

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 7afb298ef93356074bd934f9f91e309e4f398705

Build failed
Swift Test Linux Platform
Git Sha - 7afb298ef93356074bd934f9f91e309e4f398705


Multiple aws profiles

Card
Implemented support for multiple AWS profiles via a text file of multiple profiles separated by new lines or simply asking cloudfox to use all profiles in the credentials file.
Details

The init function of cli/aws.go has a call to a new function called initAWSProfiles which sets values to a slice of strings which is used as the AWS profiles to be used for the remaining commands.
The initAWSProfiles function at cli/aws.go:855 ensures that only one option of list of profiles, one profile, or all profiles is selected. It then uses the new util functions to get the AWS profiles from a given file or the default AWS credentials file. As noted in the readme of cloudfox, this functionality also assumes the user will have an AWS credentials file in place. When neither of -p, --profiles-list (new), or --all-profiles (new) is selected and empty string is passed and existing functionality takes care of using the AWS environment variables or the default profile.
The two helper functions that are used in initAWSProfiles are utils.GetSelectedAWSProfiles and utils.GetAllAWSProfiles. Both do what their names suggest and I have included tests in utils/aws_test.go.
To accommodate multiple profiles I simply wrapped a for loop iterating over the list of profiles around every function call. Thats it :)
The --all-profiles / -a and --profiles-list / -l flags were added.
Some modules were updated in the process and I also updated a small logging statement in AWSConfigFileLoader in utils/aws.go so that it would specify the given profile its having trouble reading.

For --all-profiles, added check for profiles in config (in addition to credentials)
For --all-profiles, added a default interactive sanity check to display all profiles and confirm you wan't to really do it
For --all-profiles, added a -y flag for non-interactive mode so you can skip the interactive prompt if running in a script
Updated error messages to include which profile failed
Added a way for cloudfox to continue in multi-profile mode if one of the many profiles fails


fix: added missing ssl configuration missing from helm chart

Issue
No Issue open yet, will be updated with one by<EMAIL_ADDRESS>in the near future.
Description
The helm chart currently does not support propagating required SSL configurations for Redis to the gateway configmap and ultimately underlying deployed pods. This causes pods to enter a crash loop, as Gravitee is unable to locate required SSL configurations for Redis in the gravitee.yml config file and establishes invalid connections with the Redis Instance/Cluster.
Evidence
log

gravitee.yml

gateway-configmap.yaml

Additional context
This fix was tested and<EMAIL_ADDRESS>was informed of the tested fix in a Graviteeio client environment.
Steps to Reproduce
Simply enable SSL in the helm chart and point to the correct port, Gravitee pods will immediately enter a crashloop once the configmap is reloaded.

Thanks a lot @gh0stsrc.
As we have a little issue with our CI for external contributors, I've reopened a PR with your commi: https://github.com/gravitee-io/gravitee-api-management/pull/7426
I've also added a unit test and did minor fixes.

I will close this PR in favor of this one: https://github.com/gravitee-io/gravitee-api-management/pull/7426


add caching

as part of your development app.coffee, it would be great to be able to define a temporary cache for local development. It might just be a cache: true option that writes the json to a temp file so a local developer can avoid the network overhead (and paging) of doing the API requests for each compile.

Agreed, good call. I'll add this today.


Add missing AWS/SES metric, dimensions

Source: https://docs.aws.amazon.com/ses/latest/dg/receiving-email-metrics.html
Closes  #164

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

Hi @jangaraj, just one suggested change, otherwise looks good. Thanks for the comtribution!


Adding more sound clip goodness.

Purpose
Adding more sound clip goodness.
Again all PG.
Several TV & movie clips and 3 short song clips.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test
I think we all know how to test this.


Additional Option: Get Turing result not in the same TX (async)

Having a specific "use-case" in mind, which might be interesting marketing wise and from a scientific standpoint. But this would require Turing to be able to make calls which may take e.g. 600 seconds.
Nevertheless, as discussed with @InoMurko having that option could also solve future scaling problems. Some snippets for more context from our conversation:

Ino: so now we do it inside estimateGas  RPC call . it could be longer, but the implications to geth are unknown. so if you’re not in california (your API) having just 1200 latency is going to be problematic
Kevin: true that, nevertheless it would just cover one more spectrum of the universe + the scaling issue. Just as an option, the call per tx is really useful, but having the option for greater calculations might have interesting implications.
Ino: yeah sure it makes sense. having an RPC endpoint to request data + proof of payment and async data push

@jliphard @souradeep-das @InoMurko Opinions?

interesting! just a thought - for implementing (or adding the option of) an async push oracle, should we be relying on Turing- (in turn l2geth changes) or should we have architecture along the lines of common l1 oracles like chainlink, band?
it of course might be sensible though to add this as an option - since we already have turing

Best practice/low hanging fruit probably would be to just try to onboard ChainLink on Boba, but this wouldn't drive the Boba ecosystem by definition and would weaken the Turing use-case.
Nevertheless, before we go for an own architecture like ChainLink, I think it would be from a cost-to-revenue ratio more profitable to just onboard the real ChainLink on Boba.
Doing l2geth changes for sure has the highest technical risk, but probably is the best middle way in terms of growing the Boba ecosystem, technical use case and actual amount of hours needed to add that option.
Just my 2 cents :)


added regular expressions to the registration page

seems good, need to change the redirect from "add" to "addtask" in Signup.jsx, line 81


⚠ ClamAV has degraded performance

In 070cbdf, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 75 ms

Resolved: ClamAV performance has improved in 057db56 after 540 days.


Error in userbuild with a source file name containing special characters

Development environment used

Z Open Editor version: 2.0.3
Editor Platform

[x] Visual Studio Code
[ ] Red Hat CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version: 1.65.2
Operating System (on which VS Code runs such as Windows 10 2004, otherwise name and version of platform such as OpenShift v4.3): Windows 10
Java Version (when using VS Code or Theia, execute java -version and paste the details here): 1.8.0_341
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version: 7.5.1
Node.js version: 16.18.0

Logs attached (see here how to get them): no

Problem Description
Hello,
we try to build a source file with $... characters in the filename with the userbuild feature of Z Open Editor.
While analyzing the issue, we tried to start the userbuild directly from the z/OS shell and surrounded the source file name in the command with quotes (' ') to escape special characters in the script parameters. This works as expected.
So our conclusion is, that Z Open Editor has to be changed to surround the source file name in the generated user build command with quotes to avoid handling of special characters in the command args by the shell.
Regards
Friedhelm
Detailed steps for reproducing the problem:
You can reproduce this error by running a userbuild with an arbitrary source file containing a $ in the filename.
Observed behavior
The build fails with a FileNotFoundException, because the file SS.asm is not found. We found out, that the substring $EMAIL in the generated command is replaced by the shell on z/OS with the content of the (empty) environment-variable EMAIL.
Expected behavior
Even source files with special characters in their name can be built.

Fixed in 3.0.1

Fixed in 3.0.1
Works perfectly, thank you!


Bitstamp missing altcoin withdrawal methods

Using ccxt-1.6.103.
Bitstamp's params in init (the ones defining requests).
In private API there should be methods for fetching adress and doing withdrawal.
There seems to be Ripley(XRP) but not the other currencies.
private
         post
               'xrp_address/',
               'xrp_withdrawal/',

Requests missing:
Private post
https://www.bitstamp.net/api/v2/ltc_address/
https://www.bitstamp.net/api/v2/ltc_withdrawal/
https://www.bitstamp.net/api/v2/eth_withdrawal/
https://www.bitstamp.net/api/v2/eth_address/
https://www.bitstamp.net/api/bitcoin_deposit_address/
https://www.bitstamp.net/api/unconfirmed_btc/
https://www.bitstamp.net/api/bitcoin_withdrawal/

I know that bitcoin is v1 version of API but do you think it could be squized in?
Maybe they have something like btc_address and btc_withdrawal but they forgotted to put it.
:)

I'll take a look. We will either implement those methods in bitstamp v1, or we will add support for v2. Thx for this proposal, we hope to deliver very soon!

Ok, we have added missing endpoints to Bitstamp v2, and we have also added an implementation of Bitstamp v1, just in case it's needed... We don't have all withdrawal methods implemented and unified (yet, hehe ;) but you have the full set of endpoints at least.
This issue should be fixed in v1.6.109+, which will be available for downloading in a few minutes.
We will be happy if you update and help us test it. Please, report back if you find any errors, bugs or something...
As always, thanks for your participation! We will update you on the progress of withdrawal unification.

Hi, has this issue been resolved? I don't see it in ccxt.bitstamp or ccxt.bitstamp1. Good work on this whole library, 10/10 effort

Hi, has this issue been resolved?

Yes.

I don't see it in ccxt.bitstamp or ccxt.bitstamp1. Good work on this whole library, 10/10 effort

You don't see them, because those methods get generated upon instantiation. To see them, do this:
Object.keys (new ccxt.bitstamp ())
      .filter (key => typeof exchange[key] == 'function')
      .map (key => console.log (key))

print(dir(ccxt.bitstamp()))

That makes sense. Thank you very much, and again, thanks for the library!


feat: add fantom network

Description
Adding Fantom network support. Perhaps more is needed, but this is what I could find.
If it helps at all, otherwise feel free to close it.
Type of change

[ ] Chore (non-breaking change that addresses non-functional tasks, maintenance, or code quality improvements)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] Code in this PR is covered by automated tests (Unit tests, E2E tests)
[x] My changes generate no new warnings
[x] I have reviewed my own code
[x] I have filled out all required sections
[ ] I have tested my changes on the preview link
[ ] Approver of this PR confirms that the changes are tested on the preview link

@paintoshi thank you for your contribution! Our API currently doesn't support Fantom yet. So this line won't work.  And we are currently working on adding more exported networks including the ones Viem provides.
So Fantom will be enabled as well. Appreciate the effort, let's close it for now until we come up with a wider solution for other networks as well. Best.


During install, a failure part way through save for a dependency will cause that dependency to be skipped on retry

I ran into this issue when I had access denied due to #347, but was surprised to see cargo winrt install appear to succeed on a second attempt, but skip over trying to make the symlink.
I believe the cause of this is that (when not forcing) we filter by d.already_saved, which only checks Ok(self.directory_path()?.exists()) and not anything downstream of that.

This is no longer an issue as the dedicated tool is no longer used.

This is no longer an issue as the dedicated tool is no longer used.


Qingwei/fix react bug

This PR depends on https://github.com/binary-com/binary-components/pull/16

Coverage remained the same at 31.721% when pulling fcf618efb2fe90171277f451c8f5abfbf2bd3de9 on qingweibinary:qingwei/fix_react_bug into 5b520529a744c968a7e9d030a4066fff241041fe on binary-com:master.


Revert kubelet server tls bootstrap

Why is this PR needed?
The kucero will not officially release at CaaSv4, so remove it from Kubernetes 1.17.
The kucero must existed before kubelet serverTLSBootstrap: true, therefore, we revert the code
to configures serverTLSBootstrap: true.
We'll enable the kubelet serverTLSBootstrap: true in the next coming release.
What does this PR do?
Revert the code for enabling kubelet serverTLSBootstrap: true, related to #1152.
Anything else a reviewer needs to know?
It needs to backport to v5 branch release-caasp-5.0.0.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
The kubelet configuration /var/lib/kubelet/config.yaml would have serverTLSBootstrap: true in fresh install or upgraded cluster.
Status AFTER applying the patch
The kubelet configuration /var/lib/kubelet/config.yaml would not have serverTLSBootstrap: true in fresh install or upgraded cluster.
Docs
https://github.com/SUSE/doc-caasp/pull/912
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

This PR needs to backport to branch release-caasp-5.0.0, adding backport flag back.

@evrardjp Could u please take a look, it need to backport to v5 otherwise the upgrade scenario would have metrics-server temporarily in crash loopback state unless the kucero addon installed later in "skuba addon upgrade apply", thx.

If we backport this, serverTLSBootstrap will not be set for 5.0.0, which is fine for upgrades I guess, but I am not sure if it's a good idea for greenfields. Do we intend to have this feature as opt-in ?

What does "We'll enable the kubelet serverTLSBootstrap: true in the next coming release." mean here? Do we have an update/upgrade path?

For greenfield, it's fine to enable the kubelet "serverTLSBootstrap: true" since kucero addon will be deployed when node bootstrap so the kubelet server CSR signer kucero is in cluster.
However, to make it in general behavior for greenfield and brownfield cluster, I'd suggest enable this in the skuba next release (along with kubernetes 1.19).

I haven't tested, but it all makes sense. I would love seeing some extra eyes on this.

For greenfield, it's fine to enable the kubelet "serverTLSBootstrap: true" since kucero addon will be deployed when node bootstrap so the kubelet server CSR signer kucero is in cluster.

I'm not sure if I understand this.  It looks like the other parts necessary are in place already; why would we not want to enable TLS boostrapping independent of having kucero manage cert renewal?  Seems like that's generally a good idea, and a bootstrap failure due to an expired cluster CA cert would be acceptable (since that cert will need updated anyway due to all the other stuff that will also be failing).  Am I missing something important?

For greenfield, it's fine to enable the kubelet "serverTLSBootstrap: true" since kucero addon will be deployed when node bootstrap so the kubelet server CSR signer kucero is in cluster.

I'm not sure if I understand this. It looks like the other parts necessary are in place already; why would we not want to enable TLS boostrapping independent of having kucero manage cert renewal? Seems like that's generally a good idea, and a bootstrap failure due to an expired cluster CA cert would be acceptable (since that cert will need updated anyway due to all the other stuff that will also be failing). Am I missing something important? :)

The problem is when the admin upgrades from 1.17 -> 1.18, the admin would do

skuba node upgrade apply (loop all control plane nodes and worker nodes)
skuba addon upgrade apply

then at the first control plane node upgrade, usually the admin checks all the pods in the stable state, however, since the first control node sets serverTLSBootstrap: true which means it sends out kubelet server CSR in the cluster but there is no CSR signer right now (kucero installed later in skuba addon upgrade apply).
therefore, the metrics-server would be crash loopback state because the kubelet server does not have it's server certificate right now (it seems like the kubelet does not honor in-disk kubelet.crt which is generated by skuba if serverTLSBootstrap: true.
if the admin is willing to accept the temporarily metrics-server unstable state during cluster upgrade, I'm fine to close this PR; otherwise, this PR should proceed because it solves the kubelet server certificate will expires after 1 year.


FIX memory exhaustion in MySQLStatement->bind()

This is an attempt to fix #4322. It definitely fixed my problem but I'm not sure what side-effects there may be.
I basically done what was advised here: http://php.net/manual/en/mysqli-stmt.bind-result.php#57564
cc: @tractorcow

I tested this on my machine that was running out of memory and it's fixed the memory issue :)

Wow nice find, @micmania1.


/error500 requires authentication - Story Id: 119378205

Link: https://www.pivotaltracker.com/story/show/119378205
Labels:
Requester: Filip Hanik
Estimate: 0.0

not sure this error page requires authentication

Link: https://www.pivotaltracker.com/n/projects/997278/stories/119378205/comments133737665
User: Filip Hanik

@stummidi
Comment Id: 133737665


Update launcher.user.js

Previous pull requests to change the name have been closed, since they're mainly personal in nature. See #136 and #150.

Most people don't know how to use Git and don't even know that they've requested a pull request.


Update cats-effect to 3.5.4

About this PR
📦 Updates org.typelevel:cats-effect from 3.5.2 to 3.5.4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel", artifactId = "cats-effect" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #307.


Information lost while parsing Word document

Hello. Thank you for your project! It has become quite instrumental in helping me to automate a tedious task on my computer. There seems to be a bug in the parsing that I'm not sure how to correct. In parsing a Word document and println debugging its output, I've noticed that certain text, and their style information, etc. are missing. I employed a hack of pasting the code that's generated via the strong-xml-derive crate into a local copy of this crate, and was able to verify that the text in question does show up, but when I println debug the Document structure, the text isn't there. The test file being used is attached. If you want or need anything else to debug the issue, let me know. Thanks!

Thanks for reporting this! This be fixed by
2ce1cca which allows table cell to contains multiple paragraphs. It's released as 1.1.0.

Thanks!


How to install this without Docker

Thanx for this script. Could you help me to get this to work in a non docker LXC? I am using tteck proxmox scripts (https://tteck.github.io/Proxmox/). I copied all files from your github including pmail_linux to the script folder on the LXC. I then did chmod +X pmail_linux. When I tried to execute the script with ./pmail_linux config.yaml I get an error 'could not load configuration:yaml: line 2: did not find expected key'

Sure! Everything you’ve done seems correct. Can you double check that the yaml file you’ve created matches the outline of “sample.yaml” found in the repo? Possibly use a text editor like Sublime or command line tool to ensure your yaml syntax like indents and such are correct?

Absolutely! It seems like you have a formatting error in the yaml configuration file you created.
Can you make sure that it matches the format here:
https://github.com/VMannello/paperless-email/blob/main/sample.yaml

Absolutely! It seems like you have a formatting error in the yaml configuration file you created.
Can you make sure that it matches the format here: https://github.com/VMannello/paperless-email/blob/main/sample.yaml

It is now processing a bit further but stuck on [WARN]  tag '' not found in configuration, skipping

Oh great. So the program is telling you that no tags were found on the input. Are you running this as a “post-consume” script as outlined by paperless?
https://docs.paperless-ngx.com/advanced_usage/
The consumption process adds some environment variables that the script uses to determine what tags you’ve applied during consumption.
Today it won’t work on documents that have already been processed by paperless but that might be the next thing I build into it.

Oh great. So the program is telling you that no tags were found on the input. Are you running this as a “post-consume” script as outlined by paperless? https://docs.paperless-ngx.com/advanced_usage/
The consumption process adds some environment variables that the script uses to determine what tags you’ve applied during consumption.
Today it won’t work on documents that have already been processed by paperless but that might be the next thing I build into it.

Thanx, I will try for future consumptions!

If it works could you come back and comment or close the issue? And of course happy to help further if it’s still buggy.

Next hurdle:
In my configuration file I have this
:
PAPERLESS_POST_CONSUME_SCRIPT=/opt/paperless/scripts/pmail/pmail_linux config.yaml
pmail_linux and config file are at that location but I got the following error in paperless:
[2024-02-08 17:16:31,263] [ERROR] [paperless.consumer] Configured post-consume script /opt/paperless/scripts/pmail/pmail_linux config.yaml does not exist.

Can you try putting it in quotes? If it can’t find the config.yaml you might need to include the entire path to it:
PAPERLESS_POST_CONSUME_SCRIPT=“/opt/paperless/scripts/pmail/pmail_linux config.yaml”
The other option is possibly the path to the script relative to where the paperless binary is running?

Can you try putting it in quotes? If it can’t find the config.yaml you might need to include the entire path to it (i.e. include /opt/paperless/… in front of the config.yaml)
PAPERLESS_POST_CONSUME_SCRIPT=“/opt/paperless/scripts/pmail/pmail_linux config.yaml”
The other option is possibly the path to the script relative to where the paperless binary is running?

No unfortunately not working. I tried with and without quotes, and using the entire path before config.yaml. If I try another script like "/opt/paperless/scripts/test.sh" it is working. I think it has something to do with the combination of loading pmail_linux with config file.

Thanks for this software, it looks like the perfect addition to my scanning workflow!
Unfortunately it doesn't work, I run into this same issue.
I generated some PDFs here, and put them in my consume folder. In a subfolder that should be processed as tag.
This subfolder-tag should result in an e-mail from pmail.
I have the paperless-ngx stack in Docker / Portainer, configured as per the instructions.
PAPERLESS_POST_CONSUME_SCRIPT: "/usr/src/paperless/scripts/pmail_linux pmail.yaml"
The volume is mapped and this folder contains the executable (+x) and the config.
File-PcjyN.pdf: Configured post-consume script /usr/src/paperless/scripts/pmail_linux pmail.yaml does not exist.
I think @tomdh76 is on to something, when I can all simple .sh-script without additional arguments, it results in different output (command line arguments?), see here:
`File-H6k0o.pdf: Error while executing post-consume script: Command '['/usr/src/paperless/scripts/pmail.sh', '941', '2024-04-03  File-H6k0o.txt', '/usr/src/paperless/media/documents/originals//File-H6k0o.txt', '/usr/src/paperless/media/documents/thumbnails/0000941.webp', '/api/documents/941/download/', '/api/documents/941/thumb/', '', 'Tag1,Inbox,mail-tag']' returned non-zero exit status 127.

I seem to have some success with a small bash-wrapper.
I used full paths for both, because you can't assume the working directory to be the scripts-folder.
#!/usr/bin/env bash

/usr/src/paperless/scripts/pmail_linux /usr/src/paperless/scripts/pmail.yaml

First time it sent me a .txt-file, second time it was a 0 byte .pdf.

I seem to have some success with a small bash-wrapper.
I used full paths for both, because you can't assume the working directory to be the scripts-folder.
#!/usr/bin/env bash

/usr/src/paperless/scripts/pmail_linux /usr/src/paperless/scripts/pmail.yaml

First time it sent me a .txt-file, second time it was a 0 byte .pdf. The body of the mail is also empty.
Hope this helps!

Yes this seems to be the way, thx!


Read only folder (but prevent writing locally)

I know it's possible to make a shared folder master on  a device, this makes it read only and prevents any changes from remote devices.
My suggestion is to share this information to all other devices and locally prevent those devices from editing the folder. (e.g. making it read only on each local devices and only allow changes to come in from the master synchting server or other devices which are in synch with the master)

This sounds like a suggestion for Syncthing, not SyncTrayzor. Please start a topic on the syncthing forum, or maybe open an issue on Syncthing's issue tracker. Thanks!

Will do :+1:


Bishop implementation

Criteria:

Rook can move up/down/left/right anywhere
1.5) Rook cant move if enemy/friendly is blocking the path
Rook can capture enemy pieces

Basic Bishop Implementation Criteria:

Diagonal Movement
Captures Enemy Piece
Should consider that the bishop can be blocked by pieces

accidently deleted lol


ASSY-77>AdptA64-Om32x2-sm>RHD2164 connector

We are using a connection from an acute Cambridge probe ASSY-77-H3 via a Neuronexus adaptor A64  -> 2x Omnetics 32 and finally connect to the intan headstage RHD2164. It is quite a common connection for so would be great to make it as an available automatic connection.

Hi @Enakcihc
If you have the mapping, it would be great if you made a pull request to add it. The pathways should go here: https://github.com/SpikeInterface/probeinterface/blob/main/probeinterface/wiring.py
We'd be happy to review it and double check the mapping with you :)

I have the file.
I will push the mapping.


[stable/docker-registry] Improve full name definition

What this PR does / why we need it:
It makes possible to create deployments named like the chart. Currently, if the same name is set, all resources are named like this: docker-registry-docker-registry. I have copied the definition that is used in the Jenkins chart (among others).

/check-cla

/check-cla

/ok-to-test

/lgtm

@victordzmr can you bump the Chart.yaml version?

@davidkarlsen Done! :)

@victordzmr still a conflict

@davidkarlsen Sorry, I didn't realize. Now it is ready to be merged.

/lgtm

/approve


Can't make taxon inactive if it has active children

There are currently about 2,500 inactive taxa in the db with active children.
Select COUNT(*)
from taxa a
  join taxa b on a.id = any(string_to_array(b.ancestry, '/')::int[])
  WHERE a.is_active = FALSE AND b.is_active = TRUE;

These create issues with models that depend on ancestries free of inactive taxa. While they can be fixed, new ones are continually introduced.
We need validations to prevent:

an active taxon from being inactivated if it has active children
an inactive taxon from being activated if it has inactive parent
an active taxon from being created under/moved to an inactive parent

Since the only other pathway that can result in taxon activation inactivation other than taxa/new and taxa/edit are taxon_change#commit
we need to make sure taxon_change#commit checks:

that the input taxa (to be inactivated) don't have any active children
that the output taxa (to be activated) don't have inactive parents

@kueda are there any other edge cases I didn't think of here?

Not sure about edge cases, but is the implication here that if you want to make a taxon change with a taxon, you must first deal with all its descendants? Otherwise you'll be inactivating a taxon with active children. That means we should remove the functionality we have that automatically creates swaps for children when they can be made unambiguously. Not opposed to that, but it will make more work for curators.

I think we should either remove the move_input_children_to_output functionality and only allow a swap/merge/split to be committed if the input taxa have no descendants. I agree with will create more work for curators but will avoid three common 'bad' side-effects:

gender agreement mess
e.g. Mimulus guttatus  moved to Erythranthe is Erythranthe guttata not Erythranthe guttatus

unneccessary swaps
e.g. if a Genus named GenusA is merged into a Genus with the same name, it creates unneccessary swaps

when instead the children should be moved

Or we should allow move_input_children_to_output functionality as an option which would mean a swap/merge could be committed if the taxa have descendants, but we'd have to improve it as follows:

make move_input_children_to_output work on all children regardless of whether the input RANK_LEVEL <= Taxon::GENUS_LEVEL
and figure out whether it should move the children, e.g.

Or swap descendants who names might be impacted (e.g. species and ssp). And in this case it should act on both species and ssp descendants not just children, e.g.

Of these two options, I think improving move_input_children_to_output is a better solution to removing it. But requires more work. We need do one or the other though rather than the status quo. Thoughts?

do you think you can take a crack at this on @kueda? do you prefer remove or improve?

Make move_input_children_to_output work on all children regardless of whether the input RANK_LEVEL <= Taxon::GENUS_LEVEL

As far as I can tell this is how it works now. See https://github.com/inaturalist/inaturalist/blob/master/spec/models/taxon_swap_spec.rb#L203

Or swap descendants who names might be impacted (e.g. species and ssp). And in this case it should act on both species and ssp descendants not just children

Again, this is how it works now. The only situation in which names would be impacted is when you swap or merge a genus, and in that situation, move_input_children_to_output will get called for the new swaps / merges created. I just added a spec to verify this: 78bc76cb2923a5c4fe799b0893fc98127129ad30
So it sounds like the changes you want is:

[ ] Moving descendants should be an opt-in option chosen by the committer (or the creator of the change?)

Is that correct?

Sorry, I see now that I was testing with locked branches which is why I didn't realize that it was moving descendants. Here's a new failing spec for that locked use case in a new taxon_swap_improvements branch:
https://github.com/inaturalist/inaturalist/blob/taxon_swap_improvements/spec/models/taxon_swap_spec.rb#L213
Also I didn't realize that it autogenerated swaps not just for child sp but also descendant ssp. However, it does look like the existence of subgenera screw this up. Here's a failing spec
https://github.com/inaturalist/inaturalist/blob/taxon_swap_improvements/spec/models/taxon_swap_spec.rb#L633
Would you prefer that we:

protect around these edge cases by making it so that you can't commit swaps when the input taxon descends from a locked taxon or the input taxon is a genus and has subgeneric descendants

or

make it so that taxon swaps skip locks and properly handle the subgenus edge case?

But yes aside from the above, the only change I want is:

[ ] Moving descendants should be an opt-in option chosen by the committer (or the creator of the change?)

Thoughts?

Whether or not to respect locks is up to you. I can work on making moving descendants opt-in this week.

My preference is 'make it so that taxon swaps skip locks and also properly handle the subgenus edge case' if its not too much work. Thanks

@kueda - this hypothetical example is a common structure for a taxon merge where Acris (sensu stricto) is on the left and Acris (sensu lato) is on the right:

I just realized that 'move children to output' is creating taxon swaps for all the children going from Acris (sensu stricto) to Acris (senus lato):

which seems unnecessarily disruptive compared to just moving them (e.g. moving Acris blanchardii from Acris (sensu stricto) to Acris (sensu lato) when lots of IDs are involved.
Would it be hard to change this so that if the names for the input and the output are the same, it moves rather than swaps even if the inputs are species?

Check out https://github.com/inaturalist/inaturalist/compare/1888-opt-in-to-move-input-children-to-output and see what you think.


xDnsServerZoneTransfer: Missing integration tests

We should add integration tests to verify the DSC resource.

To test this resource we need to have MS DNS Server installed in the build worker on udp/tcp port 53, and we need to have access to a primary DNS server (MS DNS Server or BIND) on udp/tcp port 53. Since both can't be running on port 53 on the build worker, and the resource does not support accessing a DNS Server on another port that 53, we need to have access to a primary DNS Server by some other means.


feat: reference architecture

Description

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Light Mode:

Dark Mode:

Light Mode:

Dark Mode:

Update:
Light Mode:

Dark Mode:

/run pipeline

/run pipeline

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
v1.0.0

Your semantic-release bot :package::rocket:


Opening /terms or /privacy gives me an error.

I think this is better now. Can you check it?

Works!


x/net/http2: Transport should do transparent gzip like HTTP/1

package main

import (
	"bytes"
	"compress/gzip"
	b64 "encoding/base64"
	"encoding/json"
	"fmt"
	"io/ioutil"
	"log"
	"net/http"
	"net/url"
	"strconv"
)

const (
	ConsumerKey    = "XXXXXXXXXXXXXXXXXXXXXX"
	ConsumerSecret = "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"
)

func main() {
	client := http.DefaultClient
	encodedKeySecret := b64.StdEncoding.EncodeToString([]byte(fmt.Sprintf("%s:%s",
		url.QueryEscape(ConsumerKey),
		url.QueryEscape(ConsumerSecret))))
	reqBody := bytes.NewBuffer([]byte(`grant_type=client_credentials`))
	req, err := http.NewRequest("POST", "https://api.twitter.com/oauth2/token", reqBody)
	if err != nil {
		log.Fatal(err, client, req)
	}
	req.Header.Add("Authorization", fmt.Sprintf("Basic %s", encodedKeySecret))
	req.Header.Add("Content-Type", "application/x-www-form-urlencoded;charset=UTF-8")
	req.Header.Add("Content-Length", strconv.Itoa(reqBody.Len()))
	resp, err := client.Do(req)
	if err != nil {
		log.Fatal(err, resp)
	}
	defer resp.Body.Close()

	var respBody []byte
	// WHY THIS IS REQUIRED?
	if resp.Header.Get("Content-Encoding") == "gzip" {
		gr, err := gzip.NewReader(resp.Body)
		if err != nil {
			log.Fatal(err)
		}
		respBody, err = ioutil.ReadAll(gr)
	} else {
		respBody, err = ioutil.ReadAll(resp.Body)
	}
	if err != nil {
		log.Fatal(err, respBody)
	}

	var b struct {
		AccessToken string `json:"access_token"`
	}
	json.Unmarshal(respBody, &b)
	twitterEndPoint := "https://api.twitter.com/1.1/application/rate_limit_status.json"
	req, err = http.NewRequest("GET", twitterEndPoint, nil)
	if err != nil {
		log.Fatal(err)
	}
	req.Header.Add("Authorization",
		fmt.Sprintf("Bearer %s", b.AccessToken))
	resp, err = client.Do(req)
	if err != nil {
		log.Fatal(err, resp)
	}
	defer resp.Body.Close()
	respBody, err = ioutil.ReadAll(resp.Body)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Println("response Status:", resp.Status)
	fmt.Println("response Headers:", resp.Header)
	fmt.Println(string(respBody))
}

try with your consumer keys. twitter send http2 response in recently.
the response Body should be decoded because content-encoding is gzip. but not decoded Body.
/cc @bradfitz

Twitter is replying with "Content-Encoding: gzip" even without "Accept-Encoding: gzip"?
The http2 transport is not sending "Accept-Encoding: gzip". Thus, it's not the Go Transport's job to decompress it.
The HTTP/1 Transport code only decompresses it when it asks for it itself, using the logic in transport.go:
        // Ask for a compressed version if the caller didn't set their
        // own value for Accept-Encoding. We only attempt to
        // uncompress the gzip stream if we were the layer that
        // requested it.
        requestedGzip := false
        if !pc.t.DisableCompression &&
                req.Header.Get("Accept-Encoding") == "" &&
                req.Header.Get("Range") == "" &&
                req.Method != "HEAD" {
                // Request gzip only, not deflate. Deflate is ambiguous and
                // not as universally supported anyway.
                // See: http://www.gzip.org/zlib/zlib_faq.html#faq38
                //
                // Note that we don't request this for HEAD requests,
                // due to a bug in nginx:
                //   http://trac.nginx.org/nginx/ticket/358
                //   https://golang.org/issue/5522
                //
                // We don't request gzip if the request is for a range, since
                // auto-decoding a portion of a gzipped document will just fail
                // anyway. See https://golang.org/issue/8923
                requestedGzip = true
                req.extraHeaders().Set("Accept-Encoding", "gzip")
        }

So, yes, HTTP/2 should do the same. But it's a little weird that Twitter is replying like that too.

The http2 transport is not sending "Accept-Encoding: gzip". Thus, it's not the Go Transport's job to decompress it.

Yes, I looked the code you pointed before. And I wonder why HTTP/1 works fine but HTTP/2 not.

It works fine because HTTP/1 is decompressing it because it asked for it.
HTTP/2 is not asking for it, so it should not have to decompress it. Twitter should not be sending uncompressed content without "Accept-Encoding: gzip" in the request.
Try HTTP/1 with Transport.DisableCompression set to true. I bet it would also fail, if it's a Twitter problem.

I bet it would also fail, if it's a Twitter problem.
Yeah, indeed.

http.DefaultTransport.(*http.Transport).DisableCompression = true

I put this on top of code but not. :(

But not what?

Response always compressed even though I set DisableCompression=true.

Additional Information.
req.Header.Add("Accept-Encoding", "identity")

Twitter still reply gzip response.

I just hit this problem when doing oauth2 authentication with facebook. It seems the facebook http2 implementation doesn't care for the Accept-Encoding header and sends gzip no matter what. Manually adding the Accept-Encoding: gzip to the request will correctly fix the problem.
I suppose this is because a previous draft of the spec required the support for gzip, they removed that since. Seems more of a bug on Twitter/Facebook side than on go. I'll let you judge.

I (or somebody) will implement this before Go 1.6.


LogEntry table causing error "Unable to serialize database: User matching query does not exist." when using datadump after if a user who edited is deleted

Found a bug? Please fill out the sections below. 👍
Issue Summary
I was dumping my database data and found that i was getting this error and debugged it to find that i have to exclude this table now because when a user is deleted after they change a page. I dont want to have to exclude more pages.
CommandError: Unable to serialize database: User matching query does not exist.
A summary of the issue.
Steps to Reproduce

(for example) Start a new project with wagtail start myproject
Create a couple of users
Add a new wagtail page
Edit this page with a user
Delete the user you edited with
call pyhton Datadump command.

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
I was expecting to dump my data with no errors since I want to save this data when uploading it to github.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes / no)
yes.

Technical details

Python version: Run python --version. 3.7.8
Django version: Look in your requirements.txt, or run pip show django | grep Version. 3.1.5
Wagtail version: Look at the bottom of the Settings menu in the Wagtail admin, or run pip show wagtail | grep Version:. 2.11.3
Browser version: You can use https://www.whatsmybrowser.org/ to find this out. Chrome 90

This is probably happening because we've added on_delete=DO_NOTHING, db_constraint=False to the user field on LogEntry: https://github.com/wagtail/wagtail/blob/cee245e7015b8462f2aa921e42803cd7d0962b8b/wagtail/core/models.py#L4769-L4770
It was implemented this way so that we at least preserve some info about what tasks were done by the same user, even if their details were deleted.
Not sure what the best way to solve this is. But hopefully we could configure this field to become null if the user doesn't exist?


ThreadId should be in APS dictionary

The thread-id should be part of the aps dictionary as stated here (see Table 9.1) and not in the request header (see Table 8.2).
In the current implementation thread-id is treated as request header:
`Notification.prototype.headers = function headers() {
let headers = {};
if (this.priority !== 10) {
	headers["apns-priority"] = this.priority;
}

if (this.id) {
	headers["apns-id"] = this.id;
}

if (this.expiry > 0) {
	headers["apns-expiration"] = this.expiry;
}

if (this.topic) {
	headers["apns-topic"] = this.topic;
}

if (this.collapseId) {
  headers["apns-collapse-id"] = this.collapseId;
}

if (this.threadId) {
	headers["thread-id"] = this.threadId;
}

return headers;

};`

Will be released as part of new version, soon?

Hello! I'm the main author of Pushy, an APNs library for Java. We're just getting around to adding support for thread-id (oops), but are having a hard time verifying that our changes are actually having an effect. @florianreinhart If you have a moment and wouldn't mind providing some advice, I was curious about how you've verified these changes.
Like you've just done in dea8951, we're adding thread-id to the aps section of the payload. What behavior change do you observe when sending notifications with a thread-id header? I haven't been able to spot any changes to message ordering or appearance in my phone's Notification Center in testing. In particular, I've been alternating sending messages with and without a thread ID, but they all seem to just appear serially in the order in which they were received. Have you seen something different?
Thanks very much for your time!

@florianreinhart will this fix pushed in a new version?? The latest available is v2.1.2 which released at 8 November and this is not included.


Improve string handling and add more widgets

Replaces im_str! in favour of using Into trait, and adds support for Ui::{button, invisible_button, progress_bar}.

Weird... for nightly and beta: https://travis-ci.org/Gekkio/imgui-rs/jobs/117204435#L189
no matching package named `imgui-sys` found (required by `imgui`)

Thanks!
Unfortunately, I'm not fond of changing everything to use Into in that way. Please check out #7 for thorough discussion about string handling. While Into still retains the possibility of using the im_str! macro and other zero-overhead methods, in practice everyone will just use normal strings if the into() call is moved from user code to the library code. This makes the macro pointless, and encourages bad practice. There's two solutions that are from my point of view acceptable:

adding the null terminator at compile-time using constexpr or similar feature once Rust has it in the future
changing dear imgui itself. A such fork exists, but I haven't made a decision yet to use it. The decision is complicated, because a fork without a proper test suite would significantly increase the risk of having subtle but deadly bugs, and would require active maintaining. So far it looks good, but I'm not ready to make the leap yet.

The new widgets look good! I noticed just one thing: progress bar should be usable without an overlay.
In the original API you can pass a null pointer, so we should either use Option or use the builder pattern as used in other places in this library.
The build failure in nightly and beta seems to be caused by the combination of using a version and a path in Cargo.toml. I'll see if I can figure out how to fix it properly, so don't worry about it.

Dang, that's a shame. I guess seeing as complex UIs could be using text heavily, performance is a concern. Hopefully string views become part of imgui in the future!

Perhaps it would be good to add the reasoning behind im_str! in the readme?

@bjz FYI my fork which removes the need for im_str! while avoiding string copies is here https://github.com/bitshifter/imgui-rs/tree/imstr

Nice! :)

Sorry, I should've linked the fork here since I mentioned it.

Perhaps it would be good to add the reasoning behind im_str! in the readme?

That's a good idea! It's a significant source for confusion, so I'll mention it in the readme.
In any case, since compile-time functions in Rust seem to be a bit far off, I think I'll probably switch to bitshifter's imgui/cimgui forks soon once I manage to create a basic test suite in this project.

That would be super nice from a UX point of view! <3


Add the ability to execute script with optional arguments to specifiy WLC IP addresses

The script should provide the ability to execute without having to run the interactive prompts, possibly by specifying the IP addresses in arguements when the command is run. i.e. python cisco43calc.py --i <IP_ADDRESS> <IP_ADDRESS>. This would simplify the interaction when called exterrnally.

Used sys.argv to allow arguments. Script can also be run at CLI by using cisco43calc.py <IP_ADDRESS> <IP_ADDRESS>


Update legacy lexers

Scintillua 6.0 considers lexers from Scintillua 5.x to be legacy lexers. While legacy lexers should still function properly, they ought to be migrated using this migration guide.
In addition to the steps outlined in the migration guide, programming language lexers should ideally distinguish between functions, builtin functions, and methods. They should also distinguish between constants and builtin constants, and variables and builtin variables. For example, the Lua lexer does so here and here. The Makefile lexer does so here.
Lexers that use custom tokens should try and pick from the updated list of tags. If that is not possible/reasonable, it may be worth considering adding to that list. Otherwise, it should remain a custom tag name and a style should be set for it for all themes in themes/. That will signal that applications will need to add styling for it.
The following unchecked Scintillua lexers still need to be migrated:

[ ] Actionscript
[ ] Ada
[ ] ANTLR
[ ] APDL
[ ] APL
[ ] Applescript
[x] ASM (NASM)
[x] ASP
[x] AutoIt
[x] AWK
[ ] Batch
[x] BibTeX
[ ] Boo
[x] C
[x] C++
[ ] C#
[x] ChucK
[ ] Clojure
[x] CMake
[ ] Coffeescript
[ ] ConTeXt
[x] CSS
[x] CUDA
[x] D
[ ] Dart
[x] Desktop Entry
[x] Diff
[x] Django
[x] Dockerfile
[ ] Dot
[ ] Eiffel
[ ] Elixir
[ ] Elm
[ ] Erlang
[ ] F#
[ ] Fantom
[ ] Faust
[ ] Fennel
[ ] Fish
[ ] Forth
[ ] Fortran
[x] fstab
[ ] GAP
[x] Gemini
[ ] gettext
[ ] Gherkin
[x] git-rebase
[ ] Gleam
[x] GLSL
[x] Gnuplot
[x] Go
[ ] Groovy
[x] Gtkrc
[x] Hare
[ ] Haskell
[x] HTML
[ ] Icon
[ ] IDL
[ ] Inform
[ ] ini
[ ] Io
[x] Java
[x] Javascript
[ ] jq
[x] JSON
[x] JSP
[ ] Julia
[x] LaTeX
[ ] Ledger
[ ] LESS
[ ] LilyPond
[ ] Lisp
[x] Literate Coffeescript
[ ] Logtalk
[x] Lua
[x] Makefile
[ ] Man Page
[x] Markdown
[x] MATLAB
[ ] MediaWiki
[x] Meson
[ ] MoonScript
[ ] Myrddin
[ ] Nemerle
[ ] Networkd
[ ] Nim
[ ] NSIS
[ ] Objective-C
[ ] OCaml
[ ] Pascal
[x] Perl
[x] PHP
[x] PICO-8
[ ] Pike
[ ] PKGBUILD
[ ] Pony
[ ] Postscript
[ ] PowerShell
[ ] Prolog
[x] Properties
[ ] Pure
[x] Python
[ ] R
[ ] rc
[ ] Reason
[ ] REBOL
[ ] ReStructuredText
[ ] Rexx
[x] RHTML
[ ] RouterOS
[ ] RPM Spec
[x] Ruby
[x] Ruby on Rails
[x] Rust
[x] Sass
[ ] Scala
[ ] Scheme
[x] Shell
[ ] Smalltalk
[x] strace
[ ] Standard ML
[ ] SNOBOL4
[ ] Spin
[ ] SQL
[ ] Systemd
[x] TaskPaper
[ ] Tcl
[x] TeX
[x] Texinfo
[x] TOML
[ ] txt2tags
[x] TypeScript
[ ] Vala
[ ] vCard
[ ] Verilog
[ ] VHDL
[x] Visual Basic
[x] Windows Script File
[x] XML
[ ] Xs
[ ] Xtend
[x] YAML
[ ] Zig

Is this check list kept up-to-date?

Yes, I try to keep it up to date.

Hit the Haskell lexer in #123


samples: usb: mass: Sample.usb.mass_flash_fatfs fails on non-secure nrf5340dk

Describe the bug
A test scenario sample.usb.mass_flash_fatfs https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/subsys/usb/mass/sample.yaml#L29 fails on nrf5340dk_nrf5340_cpuapp_ns. The test expects
The device is put in USB mass storage mode. in the output. However, the only output produced is:
DEBUG   - DEVICE: *** Booting Zephyr OS build zephyr-v3.0.0-3725-g806ee8601ca7  ***
DEBUG   - DEVICE: Area 6 at 0xfa000 on NRF_FLASH_DRV_NAME for 24576 bytes

and the test timeout
The test passes on not "non-secure" version, i.e. nrf5340dk_nrf5340_cpuapp.
To Reproduce
Steps to reproduce the behavior:

have nrf5340dk connected
run scripts/twister -v -v --device-testing --device-serial /dev/ttyACM2 --west-flash="--erase" -p nrf5340dk_nrf5340_cpuapp_ns -s samples/subsys/usb/mass/sample.usb.mass_flash_fatfs
See error

Expected behavior
Test passes
Impact
Not clear
Environment (please complete the following information):

OS: Ubuntu 18.04
Toolchain zephyr-sdk
Commit SHA or Version used: zephyr-v3.0.0-3725-g806ee8601c

@PerMac actually it is mass_flash_littlefs timeouts not mass_flash_fatfs, see first commit in #47031. The reason is that there is no devicetree overlay for nrf5340dk_nrf5340_cpuapp_ns, the fix is either to add integration platform or overlay.


Strong passwords

If I am not mistaken “strong” passwords in AAD for cloud users are enabled by default and cannot be turned off.
In this article do we need to mention “strong password only” if there are non-other passwords than strong?
Strong passwords only: Requires three out of four of the following:
o	Lowercase characters.
o	Uppercase characters.
o	Numbers (0-9).
o	Symbols (see the previous password restrictions).

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

ID: b1765376-03c9-829a-9e42-e72cc8a0daea
Version Independent ID: 684c7d7c-09f4-8170-6f7a-132b2d79e1df
Content: Azure AD Self-service password reset policies
Content Source: articles/active-directory/authentication/concept-sspr-policy.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@pawp81 Thank you for your feedback . We will investigate and update the thread further.

@pawp81 People ask what we consider strong passwords so therefore it is in the doc.

@MicrosoftGuyJFlo I understand but saying that something applies to "Strong Password only" (In Characters not allowed and Password restriction) implies that there is an option to configure a not strong password.

Making the following change in our private repo to address this based on your feedback.

#please-close

@pawp81 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Topic/optimizations

Current coverage is 75.34%

Merging #128 into master will decrease coverage by -0.02% as of c897584

Powered by Codecov. Updated on successful CI builds.


paperless-ngx Redis Connection Fails

Chart Name
paperless-ngx
Chart Version
0.15.1
Bug Description
My paperless-ngx cannot connect to Redis anymore, printing out an authentication error. Using the same Redis connection URL as in the paperless-ngx environment variable via redis-cli inside the Redis container gives me the same error. To connect successfully I have to either explicitly include the default username inside the connection string or alternatively remove the colon before the password.
values.yaml
No response
Relevant log output
No response

@RikuXan I suspect this is related to the recent Redis Helm chart update. I'm working on a fix!

@RikuXan It's interesting because their Redis env documentation still shows the syntax including the :, so I'm hesitant to update the default value. I'm able to replicate, though. Hmm

@RikuXan Oh whoops...I'm not actually able to replicate. It was a network policy issue in my setup. My pod seems to be healthy, and the env got set to PAPERLESS_REDIS=redis://:KZ8fXPJJSC@paperless-ngx-redis-master. Do you have any redis configuration set other than the default?

@gabe565 No, my Redis configuration is completely default. The only customizations I'm currently using in the overall chart are:

Ingress and persistence configurations
Using existingSecretss for paperless & PostgreSQL credentials
Pinning the PostgreSQL Image Version to a 15.x (haven't gotten around to do the manual major upgrade process)
Extending the containers and environment variables with Tika and Gotenberg

My Redis connection string looks the same as yours, but using it via redis-cli (redis-cli -u <connection-string>) inside the Redis container gives me the aforementioned error. The Redis container's image version is 7.2.4-debian-11-r0.
If it's not an issue for you, a solution might be to include a variable that is empty by default, but can be customized for the username. This way people who experience the issue could customize their Redis connection string, as well as creating compatibility with externally-hosted Redis instances with ACLs enabled (where the username can then actually be relevant as far as I understand it.


Is Nightly Rust Required?

When I tried running the minimal example, I received the error:
error[E0658]: use of unstable library feature 'stdsimd'

When I change to Nightly with rustup default nightly, the minimal example works.

This library does not use any unstable rust, no.


Add option to override MGS host

Added the config:
DAC_MGS_HOST
By default it is localhost, which is existing behaviour where
MGS always lives on the primary brick host.
We found operational issues of when to clean out the MGS after
creating lots of filesystems. Eventually the MGS will fill up.
As a work around, we allow operators to specify a custom location
(host and device) for the MGS. This means you could easily rotate
a single MGS (via change of config followed by reboot of dacd)
that ensures the correct thing happens.

Codecov Report

Merging #132 into master will increase coverage by 0.15%.
The diff coverage is 76.47%.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   50.44%   50.59%   +0.15%
==========================================
  Files          31       31
  Lines        1911     1925      +14
==========================================
+ Hits          964      974      +10
- Misses        875      879       +4
  Partials       72       72

Impacted Files
Coverage Δ

internal/pkg/config/filesystem.go
0.00% <0.00%> (ø)

internal/pkg/filesystem_impl/mount.go
42.22% <66.66%> (+0.55%)
:arrow_up:

internal/pkg/filesystem_impl/ansible.go
45.29% <84.61%> (+2.16%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ebf84b8...a772b49. Read the comment docs.


Add cross compile target for ARM

LGTM - thanks!

cool, thanks ! Fancy doing a release so that I could refer to the arm binary from a Dockerfile ?

Added you as collaborator so you can release. Will chat on IRC later so can tell you the magic incantations.


[GitHub CI/CD] ISPN-16812 Concurrency group for PR

Cancel concurrent jobs with same head_branch same repo
see ISPN-16812

merged! thanks @rigazilla !


how do you do the mouse hover effect?

similar to the green squares on the left side of the website https://simpleparallax.com/, how did you do that? I can't seem to find anything in this library to achieve that effect

This was on the old website and simpleParallax was not used, it was custom JS.


tabris run ios syntax error

tabris run syntax error

Your Node.js version does not support certain ES6 features utilized in Tabris.js CLI. Please update to the latest LTS or current version.

@gacal Can you pwd on that last screenshot?  It's probably this

tabris init okay
working
thanks

Do you want a fee?