[bug]: fix httpParams.value placeholder

Purpose of the pull request
fix(ui): fix httpParams.value placeholder, it should be  "value(required)", but "value(optional)" is displayed

fix #13851
Brief change log
see dolphinscheduler-ui/src/views/projects/task/components/node/fields/use-http.ts
Verify this pull request
Manually verified the change by testing locally.

Codecov Report

Merging #13712 (2a029bc) into dev (394805b) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head 2a029bc differs from pull request most recent head 8d5a8c4. Consider uploading reports for the commit 8d5a8c4 to get more accurate results

@@             Coverage Diff              @@
##                dev   #13712      +/-   ##
============================================
- Coverage     39.06%   39.05%   -0.01%
+ Complexity     4420     4418       -2
============================================
  Files          1130     1130
  Lines         42099    42099
  Branches       4775     4775
============================================
- Hits          16445    16443       -2
  Misses        23839    23839
- Partials       1815     1817       +2

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hi, @SbloodyS @jieguangzhou ,it seems weird that I could not rerun the failed CI. Could you rerun it?

Hi, @SbloodyS @jieguangzhou ,it seems weird that I could not rerun the failed CI. Could you rerun it?

Due to the fact that the code for this PR lags far behind dev, I merged the latest dev to trigger CI again.

@want2012 Hi, could you make sure whether your change already fix in dev branch? it seems no commit after I rebase then latest dev branch

@want2012 Hi, could you make sure whether your change already fix in dev branch? it seems no commit after I rebase then latest dev branch

Yes, I already fixed it, It's wired why it shows "no commit"，but the sonar checks out the file changes https://sonarcloud.io/summary/new_code?id=apache-dolphinscheduler&pullRequest=13712


Return proper NWC errors on skipped invoices

Now we will return a proper error response to the relay when we get an invoice we won't pay over nwc. This way the relay can safely delete them and we don't have to replay them everytime.
Closes #863 also fixes for expired invoices and invoices with no amount.

damn need to update tests


Slightly more robust GPX parsing

Thanks for the handy tool!
The app fails to parse some GPX files with <wpt> elements. The parser would reach a nested <ele> tag while the parseState was .none, leading to an .error state.
This PR fixes that issue (and others of the same breed). When an unknown tag is encountered, the parseState is set to .ignored(<elementName>). When parser(... didEndElement elementName: String ...) encounters a closing tag in this state, it just returns early (or sets the state back to .none if closing out the current ignored tag). This prevents later code in the function (which expects a certain hardcoded parseState to go with certain elements, otherwise it errors out) from being reached unless intended.
In the future, a full parser implementation might use a stack to push and pop the current element.

Merged.  Thank you very much.  A stack to push/pop elements would be better.


Kanban board setup

Project link: https://github.com/orgs/portfolio-roy/projects/3
Team
@utsargo
@juanmanuelbyc

Hi @utsargo and @juanmanuelbyc
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
📌 Great working on the setup of the project board ✅
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update using-setupapi-to-uninstall-devices-and-driver-packages.md

fix formatting - add a space between two words

Hi @jasonknichel, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@tedhudek I think you overlooked this simple one the other day :)

@jasonknichel thanks for the reminder.  I accepted but it looks like MSDN still renders without the space.  I'll rephrase to fix, likely tomorrow.  See https://msdn.microsoft.com/windows/hardware/drivers/install/using-setupapi-to-uninstall-devices-and-driver-packages.


Fix Area2D instancing causes unexpected effects on RigidBody2D Issue#81550

The variable has_space_override is incorrectly modified causing the destructor unable to remove areas from the list properly.
While result and colliding are both true, has_space_override should not change.
After the pre_solve, process_collision should be set to false. I think it makes more sense than changing it in the setup method.

Thanks for contribution! Commits need to be squashed according to workflow (see https://docs.godotengine.org/en/latest/contributing/workflow/pr_workflow.html)

Thanks for contribution! Commits need to be squashed according to workflow (see https://docs.godotengine.org/en/latest/contributing/workflow/pr_workflow.html)

OK I think I fixed it. How is it now?

Sry for the trouble, it is my first time

I've got pull request up for this here: https://github.com/godotengine/godot/pull/81434 although it's still awaiting review.

I've got pull request up for this here: https://github.com/godotengine/godot/pull/81434 although it's still awaiting review.

oh i didnt see yours. I will close this one then.

Closing per comment above (and as being superseded by https://github.com/godotengine/godot/pull/82961).


Use job restrictions plugin option on Linux build dockerhosts

The work done under https://github.com/adoptium/infrastructure/issues/3379 only changed the paramters on static build hosts, not the dockerhost systems used on Linux for running build containers. This issue should cover applying similar restrictions to those so that we can have the same isolation there. This will likely require further jobs to be on the allow list, such as the centos7_docker_image_updater job.
For reference, the original list that we added to most UNIX-based systems was build-scripts/jobs/.*|build-scripts-pr-test/build-test/jobs/.*|SXA-processCheck.*
Proposed new list: build-scripts/jobs/.*|build-scripts-pr-test/build-test/jobs/.*|centos7_docker_image_updater.*|openjdk_build_docker_multiarch.*|DockerFileCheck.*
Excluding azure dynamic ones currently provisioned, and the PLCTLab RISC-V ones (which aren't used and I'll remove the labels from them) the current list of systems to be covered by this are at https://ci.adoptium.net/label/build&&dockerBuild/ and are as follows:

Host
Changed?

build-marist-rhel8-s390x-1
No

docker-osuosl-ubuntu2004-ppc64le-1
Yes

dockerhost-azure-ubuntu2204-x64-1
Yes

dockerhost-equinix-ubuntu2204-armv8-1
No

dockerhost-equinix-ubuntu2404-armv8-1
No

dockerhost-skytap-ubuntu2004-ppc64le-1
Yes

dockerhost-skytap-ubuntu2204-x64-1
No

@steelhead31 Will we need additional jobs whitelisted here for any of the the installer jobs? (Or anything else I've missed)

@sxa, yes we'll need the adoptium-packages-linux-pipeline_new job adding, alongside whatever its replacement is called :), I'll add them, when I get a bit closer to production. I've temporarily added the string for my development process to a couple of hosts, but I'll formalise it a little more, once the process is a bit closer to ready.

I've added that one to the thee machines (x64 and ppc64le)

@adamfarley @andrew-m-leonard I don't think we've seen any issues with this change, so if you agree and we don't see any issues with these two platforms I'll implement it on the other linux dockerhosts next week (so happy to wait until any specific triage activities that happen on Monday).

Done all all systems now.


Fix bug #763 dom validity sync error

@logaretm I was playing around with fixing bug #763 and this seems like a simple approach that works and all tests pass. Can you see any downsides? Now it just runs the validation first, sets the flags etc, then checks if the field is disabled and if so, removes the validation errors and resolves the promise.

I think this is a pretty nice way to handle it.
This is unrelated but your solution highlighted a small issue in the pipeline, adding to the errors then removing them again is not very efficient, maybe error adding could be shifted up to the _validate method instead, or even the validate method. to reduce the amount of updates done to the component data. well that is more work for another day.

Thanks for merging @logaretm!
I did notice the inefficiency with adding and removing the errors. Gathering the validation errors in one place and adding them to the errorbag as a batch operation might have some other nice applications as well. Another possibility is to check isDisabled in _test().
However, I also noticed you added a "silent run" functionality that seems to run the validation without adding errors. Could you perhaps, from the validate() function, just start a silent run when isDisabled is true? It seems you would just need to pass in the silent parameter as true to _validate() there.

I guess that is a good way to solve it, if its disabled run _validate with silent mode on, and we wouldn't have to remove the messages.

@logaretm I changed it to the silent run version in PR #807

@ThomHurks Merged it, much thanks for your follow up.


Boss hangs creating boss group

# bs
warn: Boss has been configured to start with the group 'boss' but that group does not exist
prompt: create group 'boss' [Y/n]:  (Y) Y

Closing due to long.  If you see this again, please reopen the issue.


nightly update keep main world readable

When performing the nightly git pull update of the main branch, run chmod -R a+rX or equivalent to ensure world readability (git might reset it back to user+group only based upon the umask)

@marcelo-alvarez I'm reopening this ticket because PR #48 didn't fully fix it.  There are several actions that happen after the each git pull that can generate world non-readable files, e.g. the python -m compileall ... steps and the python setup.py build_ext --inplace steps of fiberassign and specex.
It looks like the fix needs to be to run chmod -R a+rX . per repo after all other steps are done instead of bundling it with the git pull step.
From @weaverba137, a command for identifying non-worldreadable files:
cd /global/common/software/desi/perlmutter/desiconda
find . -wholename \*/main/\* -not -perm /o+r -ls

@marcelo-alvarez the latest PR #49 still didn't fix it.  There are pycache files dated June 12 3:12am that aren't world readable from last night's update.  I will manually re-grant world read to those.  When you are back, please investigate why desitest isn't automatically making these world readable, including explicitly running the desitest scronjob and re-investigating the directory tree with
cd /global/common/software/desi/perlmutter/desiconda
find . -wholename \*/main/\* -not -perm /o+r -ls

@sbailey sorry, I missed that the commands to check it worked were inteded for me to use and not Ben. I am implementing a more robust solution that sets world read access in the script instead of desitest. I will post here once I've explicitly run the desitest scronjob and re-investigated the directory tree with the commands above.

@sbailey I have verified that the addition of the lines
droot=$DESICONDA/../../
for main in $(find $droot -name main -type d)
do
    chmod -R a+rX $main
done

near the end of
/global/common/software/desi/users/desi/perlmutter/code/desitest/main/etc/cron_dailyupdate.sh

to ensure all "main" source files world readable has worked, by setting the daily update scron job to run early and verifying after its completion that
cd /global/common/software/desi/perlmutter/desiconda
find . -wholename \*/main/\* -not -perm /o+r -ls

does not return any files or directories.
Closing now.


Add support to manage dependencies

The current tool cannot reference other OpenWhisk packages as dependencies.  At a minimum we should be able to pull in and reference entities in other packages w/o requiring the developer to copy the source code.  This requires:

create a module to parse dependency graph and validate it
create feature to download/obtain dependencies into the current project path and be able to reference these from inside the manifest and deployment yaml files.

This is a large task and likely should be broken up into multiple issues.
As a starting step let's propose:

support for getting a Github-based  OpenWhisk package
mange a semantic version applied to an entire deployment (as opposed to individual entities)

@paulcastro ,very interesting features, please split it into smaller issues, so we could help if possible.

When someone pushes a package to a repo, there should be a default package name for it that is unique.  We currently don't have a scheme for this though we could borrow from other dependency management frameworks.  For instance, we could use a URL as the canonical name for the package.
When someone integrates a package from a repo, they should have the option to override the default name.
An example of support for this kind of model is how Go treats imports.

After weighting several options, I would use a shorten (no protocol) URL as the canonical name for the package as you suggested. This has several advantages:

it is unique
no need to override the default name to avoid conflict
no need to go through a global naming service to guarantee the package name uniqueness
easier server-side deployment
the name can be used to directly locate the source code

The only minor problem is the package name cannot contains / characters, which has a simple solution.

Thinking about package bindings, perhaps something like this:
package:
   name: myPackage
   dependencies:
           Cloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  binding-name: myCloudant
                  inputs:
                         param1: value1

The above assumes there is an OpenWhisk package called Cloudant, it's location is "whisk.system", and I want to refer to it as myCloudant inside this manifest file.
I'm not sure how to handle name uniqueness for package names.  One option is to follow go's model, and use a internet style name based on whisk.system or GitHub repos:
package:
   name: myPackage
   dependencies:
           /whisk.system/cloudant:
                  version 1.2
                  binding-name: myCloudant
                  inputs:
                         param1: value
          github.com/openwhisk/openwhisk-client-swift
                  version .02.2
                  binding-name: mySwift

If you don't specify a binding-name, then you refer to the package using whatever the package itself specifies, e.g. /whisk.system/cloudant refers to itself as "cloudant" in its package manifest.

Another option is to specify the binding name upfront, and not have a binding-name key.
package:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1

If I want to use the default name (i.e. not specify a binding) then I just use the default name:
package:
   name: myPackage
   dependencies:
           cloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1

For uniqueness, see https://github.com/openwhisk/openwhisk-wskdeploy/issues/41#issuecomment-285750992
The first option does not work, as property keys in YAML must be unique and the same package can be bound multiple times. Option 2 might be clearer.

so we are all agree with this solution https://github.com/openwhisk/openwhisk-wskdeploy/issues/41#issuecomment-285799805, Ok, i'll investigate it and put it a high prio one, as our 101 use cases need to use dependencies.

@lzbj I can reassign to you - thanks.  We agree on option 2 (see below).
For iteration 1: we will only support binding so location should be the package we are binding to.
In the next iteration, we will can add support for remote GitHub URLs.
package:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1
          mySwift:
                  location:  github.com/openwhisk/openwhisk-client-swift
package:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1
          mySwift:
                  location:  github.com/openwhisk/openwhisk-client-swiftpackage:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1
          mySwift:
                  location:  github.com/openwhisk/openwhisk-client-swift

Ok, sure, got it. thanks.

I tried using url and source as keys to distinguish between GitHub and system bindings.  I'm not 100% this is correct and may refactor to overload location with two meanings.
package:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  source: /whisk.system/cloudant
                  inputs:
                         param1: value1
          mySwift:
                  url:  https://github.com/openwhisk/openwhisk-client-swift

I'm updating over the code to support the original format using location without prefixes:
package:
   name: myPackage
   dependencies:
           myCloudant:
                  version 1.2
                  location: /whisk.system/cloudant
                  inputs:
                         param1: value1
          mySwift:
                  location:  github.com/openwhisk/openwhisk-client-swift```

@paulcastro opened https://github.com/openwhisk/openwhisk-wskdeploy/issues/259 to track spec. updates for dependencies.

merged


Made callback params nullable.  Github #507

I made the callback params nullable for Swift compatibility.

Wrong branch.


(build) Enabled running tests on pull requests

With this PR we can finally enable tests for updating, installing, uninstalling package(s) on appveyor when a PR is submitted.
The benefit of this is to notice early on when a package update/install/uninstall is failing (although, some positive failures may still occur).
One drawback though, is that the install/uninstall is only subjective and should still also be tested in a choco test environment.
Feel free to test these changes out by opening a PR at https://github.com/AdmiringWorm/chocolatey-coreteampackages/pulls that is based of the test-pkgs-on-pr branch.
@gep13 this is what I suggested to you on sunday, it also requires the Pull Request web hook to be enabled here on github (as I mentioned on gitter).

@chocolatey/community-maintainers anyone up for review this PR?
any feedback would be greatly appreciated, as well as testing towards the previously mentioned fork for scenarios I forgot/didn't think about.
Of course, non-team members are also welcome to review and test this.

Its a good job nevertheless, the effort is evident.  🥇 🍨

I think refactoring should be done - its not that easy to follow things in a single file of 1K Posh lines.

Then be glad I continued working on it after I had something that already worked. Back then it had around double the amount of lines.

Would definitely make this available outside the AV - on my local branch I should be able to run this test.

You can run this outside of AV.
The code related to AV would basically not do anything if the APPVEYOR environment variable isn't set.

since no changes was requested (only suggestions) I'm merging this now,
it won't be available for PR's until either @gep13 or @ferventcoder enable Pull Request on the appveyor webhook though.

@AdmiringWorm I have enabled the webhook as requested.
Great work on this! 😄


AWS task to :stop_instances_by_stage["stage"]

Coverage remained the same at 38.38% when pulling 25c79223c888d9c31cff00cbc7a539c1b2fea270 on stop-instances-by-stage into 7106fa3bad87ec43324293e795e8692348085cd6 on master.


Showcase application

I was thinking to convert showcase components into container and the basic one into a presenter to shared a maximum of code.
What do you think about it ?

I was thinking to convert showcase components into container and the basic one into a presenter to shared a maximum of code. What do you think about it ?

Configuration and image path could be provided by input but that could be more complex or less relevant for the localization


Allow non-square arrays for tril and triu

It seems that the Dask Array implementations of tril and triu require a square array as input. However there is no similar constraint for NumPy's implementations of these functions. Would be nice if this constraint could be relaxed for Dask Arrays.

Similarly the square chunking constraint seems a little restrictive. Though I may simply be unaware of the issues being worked around with these constraints.

I believe that this is now resolved.


feat: split API URLs into two env vars

Splits the API URLs into two env vars, one for the Stacks Blockchain API, and another for the Stacks Core RPC API. This will allow anyone running the registrar to have greater control over the domains they use for each downstream service.

 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.


provide a way to give the inner input field an id

https://rally1.rallydev.com/#/29180338367ud/iterationstatus?detail=%2Fuserstory%2F376732857096
https://github.com/BrightspaceHypermediaComponents/activities/blob/master/components/d2l-activity-editor/mixins/d2l-activity-editor-container-mixin.js#L60
The story is making use of the isFocusable api, finding all the invalid inputs, and then put a focus on them. I need a way to ignore certain input fields. Using their id's provides just what I need and it is less intrusive.

I'm not sure this is what you want... the id on the internal input should definitely not be something we expose, since it should be completely internal. We may choose one day to render something completely different. And regardless, the id on the d2l-text-input needs to be for it, not for something else inside of it, otherwise you wouldn't be able to document.getElementById() for d2l-text-input... or if you did in a browser that doesn't support shadow DOM (like Edge), you'd get back both elements.

@AllanKerr FYI as it appears as the new form validation stuff may be what they actually want here...

I had a look at the Validation stuff from the demo. It doesn't look like it is what I am tackling here. It is more of a problem - after the getFocusableDescendant is returning a element that I should have no knowledge of (an element inside a shadow dom).  d2l-input-text actually provides a focus function to abstract away the need to set focus on the inner element. However, the foucs helper function is aware of the fact that the component can set focus.
I am going to close this PR and send a new one.


Live grep not working

Description
When using Live grep there are no results in telescope

Need to install ripgrep
brew install ripgrep

The pr just re-added git grep search. the fix for the problem was the missing rip grep dependency


fix #100

//a.js
import 'a.css'

//b.js
// no css

vitebuild a.js
vitebuild b.js
result:
a and b injected a.css

@shiyiya you should try relativeCSSInjection option or the other I said. Let me know if it works

relativeCSSInjection make all non-css

@shiyiya I think until we find another solution you can use "jsAssetsFilterFunction" option to solve your problem, maybe we can check if the user configured the lib mode so we can clear the global variable. Unfortunately I don't have the pc with me these days so I can't do all the necessary tests

Is this generateBundle only executed once? Should we just include globalcss in it?

Is this generateBundle only executed once? Should we just define globalcss in it?

There is a good solution for that problem. Instead of using the global css code for every user of that plugin, we should add a simple check for those who configured the lib mode, if the simple mode is enabled we can use the css code directly and skip the usage of the global css code. In that way we support the legacy plugin and prevent incorrect behavior when used in the lib mode. Can you change the code to do this? If not I can do the work and create a new version for to fix this.

Thank you for the contribution, since I don't have much time this month I will publish a version with the fix in minutes. Hope in your contribution in the future and thank you!


🤖 Update one or more manifest files in the .ci directory

(Apologies for the noise. I manually triggers this one more time for testing purposes.)


Add cast wallet ls

          I mostly use cast with hardware wallets and something like `cast wallet ls` for them would be extremely useful.

Originally posted by @lightclient in https://github.com/foundry-rs/foundry/issues/656#issuecomment-1916962482

Hey @mattsse looking at this issue, what is the difference with the current implementation of cast wallet ls as described here?
https://book.getfoundry.sh/reference/cast/cast-wallet-list

I believe the limitation @lightclient ran into was that wallet ls only targets the local keystores.
maybe we add an --all option to also check connected wallets, if any?

I believe the limitation @lightclient ran into was that wallet ls only targets the local keystores.
maybe we add an --all option to also check connected wallets, if any?

can take this one, I propose following options for cast wallet list
  --dir (alias to default)
      List all the accounts in the keystore default directory
  -l, --ledger
      List accounts from Ledger hardware wallet
  -t, --trezor
      List accounts from Trezor hardware wallet
  --aws
      List configured AWS accounts
  --all
      List accounts from keystore default directory, hardware wallets and AWS

with an all option output like
Keystore default directory
 - test1
 - test2
Ledger
 LedgerLive HD path
  - 0x001...
  - 0x002...
  ...
 Legacy HD path
  - 0x003...
  - 0x004...
  ...
Trezor
 - 0x005...
 - 0x006...
 ...
AWS
 - 0x007...

(could also add option for the number of addresses and specific HD path to show)

This makes sense to me.

made a draft PR https://github.com/foundry-rs/foundry/pull/7123 looking for comments

fixed in #7123 @mattsse this can be closed


run ember binary directly

This PR changes the behavior to never invoke the ember binary with node but use the generated .bin/ember directly.
On windows it's a .cmd that automatically finds node.
On linux/darwin a js file with a shebang line to run it with node
should fix #207 and maybe #191.
Please don't merge this yet, as I haven't had time to check if the tests are ok (or go ahead if CI is ok).
It works on windows (tried in a modern.ie vm) and linux.

@Hucster can you maybe check this branch out and build the plugin locally to confirm that it's fixed?

You fixed it, @makepanic !

I successfully started ember serve and created new route 'about'.
Thanks !

@Turbo87 I'm unsure if this is a right fix for the issue though.
It removes the intellij node interpreter found nodejs to rely on operating system level nodejs.
val interpreter = NodeJsInterpreterManager.getInstance(project).default as? NodeJsLocalInterpreter
val node = interpreter?.interpreterSystemDependentPath ?: "node"

The only alternative i can think of (that still uses intellij node interpreter):

find the node_modules ember-cli cli script and execute it directly with node

this has the disadvantage that if ember-cli changes it's cli path, then that will break

One could also try to find the cli bin script by parsing the package.json for its bin values.
"bin": {
    "ember": "./bin/ember"
  },

@makepanic I guess it's a tradeoff. Let's see if people complain about this change and then we can still look for a better way if necessary. I suspect this should work fine.


Updating - cashebusting

Hi! I am creating smaller website with aprox 500 smaller textonly articles. I thought i could make the site work offline - because the whole database is actualy quite small.
So i use dixie simply as readonly - clients cant change the database.
It works suprisingly well the question i have though is what would be the simplest solution to updating articles that clients already have in db? I dont see this happening very often but it might - grammar corrections etc.. Should i provide version nubmer on the backend and then check compare every article on first load of the client or is there some more clever solution?
Thanks!

Glad to hear that Dexie works well for you.
You use case is covered by addon dexie-syncable + your own sync-provider and backend. That said, let me first suggest a simpler solution for your case since you only have one-way sync, just a single table with just around 500 articles.
Put a digest on each article in a dedicated column (server-side and client-side). A sync request could let client query the ids and corresponding digests as an array. This would be a quick response with ~500 articles. Put the response in a javascript map of {articleID: digest}. Client compares:
var map; // Retrieve from server {articleId: articleDigest}
var modifiedArticleIds = [];
db.articles.each(article => {
    if (article.digest !== map[article.id])
        modifiedArticleIds.push (article.id);
    delete map[article.id];
}).then(()=> {
        var newArticleIds = Object.keys(map); // Those not deleted.
        return jQuery.ajax({
            url: "yourBackendUrl",
            data: {
                articles: modifiedArticleIds.concat(newArticleIds)
            }
        });
}).then(data => {
        return db.transaction('rw', db.articles, ()=> {
            data.forEach(article => db.articles.put(article));
        });
});

A full-blown solution for this is possible through dexie-syncable addon, which I personally use (even though still in alpha development!). It's a framework for two-way sync but you need to provide your own sync-provider plugin as well as a backend, and the backend need to be capable of tracking all inserts, updates and deletes revisioned by a revision number. I personally have a dedicated "changes" table at the server where I track all changes. When a client connects, it sends its revision and server queries all changes since that revision and responds back to the client.
There's a sample on how to implement a sync protocol and the server side part at https://github.com/dfahlander/Dexie.js/tree/master/samples/remote-sync/websocket. The server side is implemented as in-memory database. You must convert the code to talk with your real DB.

Thank you very much for your help!
You are very kind.
I knew about the syncable - i also knew about pouchdb and its way of syncing but i thought for this purpose it will be better someting simpler someting going one way. I never realy realized that the sync is useful not only for comunicaiton client->server but also for updating.
I will try to implement your proposed solution tomorrow.
Again thanks for your time.


letting the snapshot Id null will throw up error

as per stated in code, you can leave snapshot id null so that it will use any latest snapshot to load the brain. Im getting error when put null for the snapshot.

I see that you are using Unity SDK, However, I am using the C++ SDK and I am not having any issues in leaving the Snapshot ID blank while using the method m_service->GetBrain


[Fix] SerivceId should be limited to 80 Characters for ASG and LC names

Reviews
0/1

More info

Reviews
1/1

lian
1

Pull request is reviewed and can be merged to master!
More info

Reviews
1/1

lian
1

Pull request is reviewed and can be merged to master!
More info


feat: Automatic token provisioning on deploy

Notes on the current implementation

authorization endpoints are pending to be pushed to production thus this won't work yet against https://dash.deno.com
Uses Keychain for storing the token in Darwin. Other OSes store the token in memory (thus have to provision a new token on every deployment)

Need to decide how far we want to get supporting the other OSes, and decide if we are ok with defaulting to memory storage or prefer the less secure file storage.

 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.


Make it possible to add custom tags to CF (nested) stacks and underlying resources.

Description of changes:
We have to put certain tags (like project, stage, repository, ...) on every CF stack we deploy (and it propagates to (most) resources).
Example of parameters.sh where we define the TAGS.
...
TEAM_ACCOUNT_PROFILE=test-team
TEAM_ADMIN_GROUP="xxx"
TEAM_AUDITOR_GROUP="yyy"
TAGS="project=iam-identity-center-team stage=prod repository=https://bitbucket.org/stuff other=test"

The updated bash scripts are maybe not the cleanest way to achieve this, but it helps me to clarify my use case.
It makes it possible to tag the CF stack for Amplify + all underlying nested stacks.
Tags can also be updated.
We can't hardcode the values in the tags.json because we have a stage tag: prod vs non-prod (in a test organization). We try to use a separate params.sh file for each environment.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lvthillo this is a good addition. Can the tags be inferred from the parameters.sh file instead of a tags.json in the amplify backend folder ?

@tawoyinfa the proposed solution will get the tags from the parameters.sh. The deploy.sh script will generate the content of tag.json based on that. Something like this:
[
  {
    "Key": "Application",
    "Value": "iam-identity-center-team"
  },
  {
    "Key": "Stage",
    "Value": "prod"
  },
  {
    "Key": "Responsible",
    "Value": "xxx"
  },
  {
    "Key": "Repository",
    "Value": "https://bitbucket.org/yyy"
  }
]

but I'm not an Amplify expert so I don't know if there is another/better way to pass tags to it. I hope it's clear.

@lvthillo Makes complete sense. Can we add the logic to this script instead parameters.js. For consistency, it is where I have put any updates to parameters and json files.

@tawoyinfa
I have updated the parameters.js with a function which reads the TAGS (from parameters.sh and writes them to amplify/backend/tags.json during the build.

TAGS (in parameters.sh) are passed as --tags to the main TEAM-IDC-APP stack but to make them accessible during the build I also need to pass them as parameter.
TAGS can be empty (TAGS="")

Open "issue":
TAGS won't be updated if there is no change in the upstream repo because the amplify pipline won't start after running ./update.sh. You have to click "redeploy this version" in the Amplify console.


undefined symbol

According to your environment configuration, the following error will occur. The reason found online is that the versions of Torch CUDA and Mamba do not match.  ImportError: /home/lkp/anaconda3/envs/caduceus_env/lib/python3.8/site-packages/selective_scan_cuda.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops10zeros_like4callERKNS_6TensorEN3c108optionalINS5_10ScalarTypeEEENS6_INS5_6LayoutEEENS6_INS5_6DeviceEEENS6_IbEENS6_INS5_12MemoryFormatEEE

How to resolve this issue

What versions of mamba-ssm and causal-conv1d do you have installed?

m

mamba-ssm==1.2.0.post1  causal-conv1d==1.2.0.post2

Ya that should be good. What machine/GPU are you using?

A40

Ya that should be good. What machine/GPU are you using?

A40

I resolve it.
https://github.com/state-spaces/mamba/releases


Create protected subnets using this module

Hi,
I know protected subnets are barely used, but is there a chance to add the option to create protected subnets to this module like it does for private and public? Kudos on the module

NVM, saw that they are defined as intra_subnets


getting-started.md - Apache howto error

Hello,
Following the installation howto, I noticed an error in the Apache configuration section.
The ProxPass and ProxyPassReverse lines have an ending semi-column, wich leads to an error when restarting apache.
Additionnaly, it might be worth mentioning that the Apache proxy module should be enabled as well, throug
a2enmod proxy

Fixed the Apache config in b37503ed8fec7ee3d88e6cca7924fd10c08e5705

Running Apache 2.4, and I had to make the following changes to what was suggested in getting-started.md
(add "/" to the end of the guincorn URL)
ProxyPass / http://<IP_ADDRESS>:8001/
ProxyPassReverse / http://<IP_ADDRESS>:8001/
(remove "/" from the alias for static)
Alias /static /opt/netbox/netbox/static

Could someone please confirm that this change works fully with Apache? This will be the third time I've had to modify it.

Doing a clean install now. I will let you know.

I'm on a clean install of Ubuntu 14.04.4 LTS and can confirm that this is my Apache 2.4.7 config file:
<VirtualHost *:80>
    ProxyPreserveHost On

    ServerName ipam.ddaftech.com

    Alias /static /opt/netbox/netbox/static

    <Directory /opt/netbox/netbox/static>
        Options Indexes FollowSymLinks MultiViews
        AllowOverride None
        Require all granted
    </Directory>

    <Location /static>
        ProxyPass !
    </Location>

    ProxyPass / http://<IP_ADDRESS>:8001/
    ProxyPassReverse / http://<IP_ADDRESS>:8001/
</VirtualHost>

I also had to enable both of the following modules
a2enmod proxy
a2enmod proxy_http

I can confirm bzoellers apache configuration. Running the same configuration on a clean install of Ubuntu <IP_ADDRESS> LTS.

Here is my apache config...it differs a little bit than bzoellers but is close:
`<VirtualHost *:80>
ProxyPreserveHost On
ServerName netbox.lan.esu10.org

Alias /static /opt/netbox/netbox/static

<Directory /opt/netbox/netbox/static>
    Options Indexes FollowSymLinks MultiViews
    AllowOverride None
    Order allow,deny
    Allow from all
    #Uncomment below if running Apache 2.4
    Require all granted
</Directory>

<Location /static>
    ProxyPass !
</Location>

ProxyPass / http://<IP_ADDRESS>:8001/
ProxyPassReverse / http://<IP_ADDRESS>:8001/

`

Also received a report that the Alias line should read:
Alias /static/ /opt/netbox/netbox/static/

(with the trailing slash).

Hi,
I had to remove the trailing slash in the Alias directive, and add trailing slashes to the ProxyPass statements to have it work.
My config looks exactly like Billizoellers'.


Fixing integration with ActiveModel::Serialization

The integration with ActiveModel::Serialization module was failing because it expects Model#attributes keys to be strings and Cequel::Record::Properties.attributes method return a hash with symbols as keys.
To fix that reducing the incompatibility between versions I changed the #attributes method to return a hash with indifferent access (as was advised by @outoftime in #195).
Because of this bug, the #as_json with :only option was returning {} always.
Related issues : #194 #195.

@HugoLnx general approach looks good, left one comment for how we could do it a little more efficiently

My bad, I already made the refactoring. :+1:

@hugolnx LGTM! Will merge/release next time I'm at a computer : )


Error: rpc error: code = NotFound desc = rpc error: code = NotFound

$ fxcored keys show fxmyacc --keyring-backend test
- name: fxmyacc
  type: local
  eip55_address: 0xBd2628812a12d274DFd3a3bb7Afd1dA0e33DF3E9
  address: fx1h5nz3qf2ztf8fh7n5wah4lga5r3nmulftz0gfh
  pubkey: '{"@type":"/ethermint.crypto.v1.ethsecp256k1.PubKey","key":"AnEDcfCOwC5dlvEDqkmCy28r0hZLjjSLWLO5rxRCDQMd"}'
  algo: eth_secp256k1

$ fxcored tx slashing unjail --from=fxmyacc --gas="auto" --gas-adjustment=1.2 --gas-prices="6000000000000FX" --keyring-backend test
Error: rpc error: code = NotFound desc = rpc error: code = NotFound desc = account fx1h5nz3qf2ztf8fh7n5wah4lga5r3nmulftz0gfh not found: key not found

$ fxcored tx slashing unjail --from=fxmyacc --gas="auto" --gas-adjustment=1.2 --gas-prices="6000000000000FX" --keyring-backend test --dry-run
Error: fxmyacc.info: key not found

I'm trying to unjail the testnet validator, but it cannot find the key in the test backend, despite it's showing fine.

It seems to me like 'fxcored keys add' generated and added the wrong fx addresses based on the same mnemonic and index.
$ echo my previous mnemonic words to recover | fxcored keys add fxmyacc --index 123 --keyring-backend test --recover
{"name":"fxmyacc","type":"local","eip55_address":"0xBd2628812a12d274DFd3a3bb7Afd1dA0e33DF3E9","address":"fx1h5nz3qf2ztf8fh7n5wah4lga5r3nmulftz0gfh","pubkey":"{\"@type\":\"/ethermint.crypto.v1.ethsecp256k1.PubKey\",\"key\":\"AnEDcfCOwC5dlvEDqkmCy28r0hZLjjSLWLO5rxRCDQMd\"}","algo":"eth_secp256k1"}

This generated completely different FX address that I had before the EVM upgrade.

Ok, as for the workaround, I've copied the old binary into new container (as I'm using dockerized version):
$ docker cp OLDID:/usr/bin/fxcored .
$ docker cp fxcored NEWID:/usr/bin/fxcored-old
$ echo my previous mnemonic words to recover | docker exec -i fxcore-testnet fxcored-old keys add fxmyacc --index 123 --keyring-backend test --output json --recover
{"name":"fxmyacc","type":"local","address":"fx1337...now-it-is-correct-one","pubkey":"fxpub1now-it-is-correct-one"}

Now I was able to unjail my validator using the old binary without any issues:
$ docker exec -it fxcore-testnet /usr/bin/fxcored-old tx slashing unjail --from=fxmyacc --gas="auto" --gas-adjustment=1.2 --gas-prices="6000000000000FX" --keyring-backend test -b block --yes

As suggested by Rick, we should add --algo secp256k1 to use the previous algorithm for the address generation.


App Suggestion: Music Player GO

App Name
Music Player GO
Description
A simple yet fully-featured local music player aiming at simplicity and performance.
Category
Media Players
Links
Source: https://github.com/enricocid/Music-Player-GO
F-Droid: https://f-droid.org/packages/com.iven.musicplayergo
Play Store: https://play.google.com/store/apps/details?id=com.iven.musicplayergo

Added, thanks for the submission.


guest.blade install stage

Error:
Parse error: syntax error, unexpected ')', expecting ']' (View:voten/resources/views/layouts/guest.blade.php)

the code
    <script>
        window.Laravel = <?php echo json_encode([
            'csrfToken' => csrf_token(), 'env' => config('app.env'), 'pusherKey' => config('broadcasting.connections.pusher.key')) ?>
    </script>

I tried to revert to json_encode but then the page was locked up.
note: a goat told me about your site and then I found out it was laravel so had to try it out.
EDIT:
The token and env variables are causing the issue. If I take them out of the array I can see the page but then can't login with the default admin/password.

update: I upgrade brew and started a fresh install on a different machine. I got to the site. I had to refresh after entering the admin/password but I got in.
I went to change some settings and now I see this:
chmod(/Users/asus/Sites/voten/storage/oauth-public.key): Operation failed: Operation not permitted

storage is already 777.
Also, what services do I need to sign up for to make all this work properly?
I got pusher, sentry, redis.
Do I need echo and oauth?

remove that script it works

I haven't tried doing that yet. I will give that a try.

Have you tried running npm run dev?

still it is not working for me!

Please help to fix the issue! Support Open Source!

Your array doesn't have a closing tag

Any fix for this, please I need to fix this.

fixed


InvalidVDDKImage Error - how to troubleshoot / fix

          This is much needed!! I am struggling to understand what caused this sudden invalidvddkimage error on my vcenter running 7.x and I am using vddk 8.0.2 from broadcom - I queried the controller inventory / main logs - it just tells me InvalidVDDKImage - Steps tried

Deleted and recreated vsphere provider with diff. images of vddk - from quay.io/kubevirt/vddk , from broadcom site - vddk <IP_ADDRESS>.2,8.0.3///
Deleted plans and recreated them
Deleted controller pods and updated deployment to have more cpu & mem. resources... below error from provider with - vddk:8.0.2-22388865

Conditions
Type | Status | Updated | Reason | Message
--
{"level":"info","ts":"2024-07-24 07:37:30.842","logger":"plan|vvqrj","msg":"Condition added.","plan":{"name":"testmigration1","namespace":"konveyor-forklift"},"condition":{"type":"VDDKInvalid","status":"True","reason":"NotSet","category":"Critical","message":"VDDK init image is invalid","lastTransitionTime":null}}
{"level":"info","ts":"2024-07-24 07:37:30.842","logger":"plan|vvqrj","msg":"Condition deleted.","plan":{"name":"testmigration1","namespace":"konveyor-forklift"},"condition":{"type":"ValidatingVDDK","status":"True","reason":"Started","category":"Advisory","message":"Validating VDDK init image","lastTransitionTime":"2024-07-24T07:32:30Z"}}
{"level":"debug","ts":"2024-07-24 07:37:30.842","logger":"events","msg":"VDDK init image is invalid","type":"Warning","object":{"kind":"Plan","namespace":"konveyor-forklift","name":"testmigration1","uid":"a92ed2c6-1e2f-4725-8b34-306a6ad3e87d","apiVersion":"forklift.konveyor.io/v1beta1","resourceVersion":"136041944"},"reason":"VDDKInvalid"}
{"level":"debug","ts":"2024-07-24 07:37:30.842","logger":"events","msg":"Validating VDDK init image","type":"Normal","object":{"kind":"Plan","namespace":"konveyor-forklift","name":"testmigration1","uid":"a92ed2c6-1e2f-4725-8b34-306a6ad3e87d","apiVersion":"forklift.konveyor.io/v1beta1","resourceVersion":"136041944"},"reason":"ValidatingVDDK"}
{"level":"info","ts":"2024-07-24 07:37:30.851","logger":"plan|vvqrj","msg":"Reconcile ended.","plan":{"name":"testmigration1","namespace":"konveyor-forklift"},"reQ":0}

-- | -- | -- | -- | --
TargetNameNotValid | True | Jul 24, 2024, 1:02 PM | NotValid | Target VM name does not comply with DNS1123 RFC, will be automatically changed.
VDDKInvalid | True | Jul 24, 2024, 1:07 PM | NotSet | VDDK init image is invalid
Originally posted by @jsakil14 in https://github.com/kubev2v/forklift/pull/969#discussion_r1689298428

How did you create the vddk image? Did you follow the directions here? https://docs.redhat.com/en/documentation/migration_toolkit_for_virtualization/2.6/html-single/installing_and_using_the_migration_toolkit_for_virtualization/ ?

Yes exactly the same way step by step
I simply can't get through beyond invalid vddk error, clueless ...

You can take a look at the vddk-validator-$(PLANNAME)-$(RANDOM) pod and events associated with it to see if there is any additional information available.

@jonner  There is no pod getting created as soon as I create a plan - pod gets created after starting the migration I guess.
sharing plan details - Also , for note - am creating the image from MAC - vs code - is it supposed to have issues with symbolic links? Image am using broadcom 8.0.3 ...
kd plans.forklift.konveyor.io kubevirt
Name:         kubevirt
Namespace:    konveyor-forklift
Labels:       <none>
Annotations:  populatorLabels: True
API Version:  forklift.konveyor.io/v1beta1
Kind:         Plan
Metadata:
  Creation Timestamp:  2024-07-26T13:20:26Z
  Generation:          1
  Managed Fields:
    API Version:  forklift.konveyor.io/v1beta1
    Fields Type:  FieldsV1
    fieldsV1:
      f:spec:
        .:
        f:map:
          .:
          f:network:
          f:storage:
        f:provider:
          .:
          f:destination:
          f:source:
        f:targetNamespace:
        f:vms:
    Manager:      Mozilla
    Operation:    Update
    Time:         2024-07-26T13:20:26Z
    API Version:  forklift.konveyor.io/v1beta1
    Fields Type:  FieldsV1
    fieldsV1:
      f:status:
        .:
        f:conditions:
        f:migration:
        f:observedGeneration:
    Manager:         forklift-controller
    Operation:       Update
    Subresource:     status
    Time:            2024-07-26T13:20:49Z
  Resource Version:  138418008
  UID:               0824ee17-96c6-44d0-a5dc-3088c20e98f8
Spec:
  Map:
    Network:
      API Version:  forklift.konveyor.io/v1beta1
      Kind:         NetworkMap
      Name:         localhost-rspxr
      Namespace:    konveyor-forklift
      UID:          567e69c2-e970-4c41-a0bc-36d9ac8eac73
    Storage:
      API Version:  forklift.konveyor.io/v1beta1
      Kind:         StorageMap
      Name:         localhost-gr9fx
      Namespace:    konveyor-forklift
      UID:          724e735a-c1e2-4f1d-a784-c1c937c0a3ac
  Provider:
    Destination:
      API Version:  forklift.konveyor.io/v1beta1
      Kind:         Provider
      Name:         host
      Namespace:    konveyor-forklift
      UID:          60624b33-fe96-423e-b205-750d3b486332
    Source:
      API Version:   forklift.konveyor.io/v1beta1
      Kind:          Provider
      Name:          localhost
      Namespace:     konveyor-forklift
      UID:           2a02d0c6-9df3-419b-b8fb-4d798ae7b4cc
  Target Namespace:  konveyor-forklift
  Vms:
    Id:    vm-7836654
    Name:  test-01-kubevirt
Status:
  Conditions:
    Category:              Critical
    Last Transition Time:  2024-07-26T13:20:49Z
    Message:               VDDK init image is invalid
    Reason:                NotSet
    Status:                True
    Type:                  VDDKInvalid
  Migration:
  Observed Generation:  1
Events:
  Type     Reason          Age                 From  Message
  ----     ------          ----                ----  -------
  Normal   ValidatingVDDK  90s (x2 over 113s)  plan  Validating VDDK init image
  Warning  VDDKInvalid     90s                 plan  VDDK init image is invalid

I am also facing the same issue. I tried with 8.0.3 and 8.0.1 (since the instructions are saying to use 8.0.1) and both are failing with the same error and is very unclear what to do/where is the issue.
Also is not clearly documented what to put as vddkInitImage do I put the image stream ? The internal Registry Name?
I tried with:
image-registry.openshift-image-registry.svc:5000/openshift-mtv/vddk@sha256:3f536fb1815220447e25430492ea38c3709c2a9c75e597ec5045ce05050f93dc
image-registry.openshift-image-registry.svc:5000/openshift-mtv/vddk:8.0.1
And still same error.

Image name , is what you build from podman / docker - it can be
quay.io/kubevirt/vddk-init
For my case I redeployed entire operator and all of a sudden it worked.
I also think , we must emphasize the usage of any Linux OS to build the
docker image rather than windows or Mac , just my thoughts..
On Mon, 29 Jul, 2024, 12:51 pm camrossi, @.***> wrote:

I am also facing the same issue. I tried with 8.0.3 and 8.0.1 (since the
instructions are saying to use 8.0.1) and both are failing with the same
error and is very unclear what to do/where is the issue.
Also is not clearly documented what to put as vddkInitImage do I put the
image stream ? The internal Registry Name?
I tried with:
@.***
:3f536fb1815220447e25430492ea38c3709c2a9c75e597ec5045ce05050f93dc
image-registry.openshift-image-registry.svc:5000/openshift-mtv/vddk:8.0.1
And still same error.
—
Reply to this email directly, view it on GitHub
https://github.com/kubev2v/forklift/issues/971#issuecomment-2255143182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APS2FWDRAK7HH6MQGLJU2L3ZOXUQTAVCNFSM6AAAAABLMF5SLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJVGE2DGMJYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks @jsakil14 ! Apparently for me access to pull the images is failing and I get a msg="error authorizing context: access denied" not sure why.
Will troubleshoot it later on!

@jsakil14 so in the end my problem is with the internal registry.
Nothing I can do makes the forklift-controller able to pull images from there.
I tried adding an extra  rolebinding of type system:image-puller as well as enabling the imageStream lookupPolicy but nothing I can do solves the fact that the internal registry will just access deny the forklift-controller access to the image I have built.
So I just put the image on quay.io on a public repo and now all works.
The Validation seceded with both 8.0.1 and 8.0.3 (didn't try 8.0.2)
I think we should have some instruction on how to build this in-cluster and how to give access to forklift to it but is probably not for this git issue.

When you create a new plan for a provider that specifies a vddk init image, it should spawn a job to validate that vddk image immediately after creation with a name like vddk-validator-$PLANNAME. This job will create an associated pod named $JOBNAME-$RANDOM which tries to pull the vddk image and verify that it contains the expected files. If the image fails to be pulled, it will retry a couple times until the job fails (typically after 300 seconds) at which point the pod will apparently be deleted. So you may only be able to see the pod and view the events related to that pod while the validation is running.

https://github.com/kubev2v/forklift/issues/971#issuecomment-2258490414


CUPS reads PPD boolean values case-sensitively

The CUPS ppd generator can add boolean options to the ppd, with the boolean all in lower case. E.g.
cups/ppd_cache.c line 3182:
cupsFilePuts(fp, "*cupsManualCopies: true\n");
However, the CUPS ppd reader code checks options using a case-sensitive string comparison. E.g. cups/ppd.c line 882:
ppd->manual_copies = !strcmp(string, "True");
This means that some options in generated ppds appear to be correct, but in fact are then not being honoured when cups reads & uses the ppd. I can confirm this by seeing that filter pdftopdf does NOT do software copies evening though it has
*cupsManualCopies: true
If I change the ppd myself to 'True' the pdftopdf starts doing software copies as expected.
The same problem will for sure apply to other boolean options as well ... but I wonder how many IPP printers with machine-generated ppds are out there that aren't using cupsManualCopies as expected?

Fixed in OpenPrinting CUPS.


Add support for native bigint parsing

Closes #768.

Can this be merged? Having a failure with building the bcrypto library due to the associated bug.

Thanks!

Please try again with<EMAIL_ADDRESS>


It would be nice if twoslash-cli parses defaultCompilerOptions

Took me some time to realize that I have to specify the actual enum value to make the defaultCompilerOptions work:
<!-- twoslash: {
  defaultCompilerOptions: {
    moduleResolution: 99 // NodeNext
  }
} -->

Search terms:

Cannot find module or its corresponding type declarations.
Unexpected moduleResolution: NodeNext

Yeah, that makes sense 👍🏻


feat: add fypp linting support for Intel and NAG

Is there an existing request for this?

[X] I have searched the existing issues

Feature Request
Currently, fypp is only capable of handling stdin -> gfortran without creating any temporary files. ifort and nagfor cannot receive input from stdin hence we will have to generate the files and store them, then pass the expanded files through the linter.
This issue relies on #614 being implemented.

You could try ifort -fpp-name=$(command -v fypp) to change the preprocessor used with the Intel compilers.

You could try ifort -fpp-name=$(command -v fypp) to change the preprocessor used with the Intel compilers.

Cheers @awvwgk, will give it a go

@awvwgk, have you tried that before? It seems like fypp will need response files before it can work: https://github.com/aradi/fypp/issues/24


Sitemap does not include slash between siteRoot and routes when disableRoutePrefixing is true

Are you reporting a bug?
Yes
Environment

react-static -v: 5.9.7
node -v: v8.9.4
yarn --version: 1.5.1
Operating system: Mac High Sierra

Steps to Reproduce the problem
Base your steps off of any freshly installed react-static template!
(Using the 'basic' example template)

Add a siteRoot to static.config.js with or without a trailing slash
Add disableRoutePrefixing: true, to static.config.js
Ensure there are one or more routes
Update package.json to have "react-static": "^5.9.7" and update dependencies
Run yarn build and then look at the resulting sitemap.xml

Expected Behavior
sitemap.xml should include <loc>{siteRoot}/404/</loc>
Actual Behavior
sitemap.xml includes <loc>{siteRoot}404/</loc> (Without slash between the root and route)

The latest version of react-static has some updates to how sitemaps are created. This issue may or may not have been fixed as a side effect. It would probably be a good idea to have some unit tests to validate

Would you be willing to help out on those?
On Fri, Jul 13, 2018 at 11:31 AM Rachel Church<EMAIL_ADDRESS>wrote:

The latest version of react-static has some updates to how sitemaps are
created. This issue may or may not have been fixed as a side effect. It
would probably be a good idea to have some unit tests to validate
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/626#issuecomment-404901005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCU440a8_vMYrqYjyTKPXRVAIMLswks5uGNmNgaJpZM4Ub4pr
.


Change keys combination to keycode

Hi, your project is awesome.
I have been reading your code, it is good code.
But I found a little problem, well I am trying to fixing (in a local branch) but I could not fix it.
The little problem is the differences between languages, for example the action "Zoom out in Timeline panel to show the entire composition duration" in the "Adobe After Effects" is key combination [ shift ] + [ ; ] = [ : ](in english layout), but in spanish layout is key combination [ shift ] + [ . ] = [ : ] .
I think maybe it is better [ keycode ] -> [ action ] instead [ keys combinations ] -> [ action ].
And in the previous example is: [ colon ] -> [ Zoom out in Timeline panel to show the entire composition duration ]
The problem is change the javascript code and the source layouts. It is a lot of work, sorry, but I can help you.
Regards.

Thanks, this is a great suggestion. I've actually thought about doing this, but found out that not all applications do key detection like that. While this is correct for Adobe applications, it doesn't work for Blender. I then decided to take the easy route because thinking about it makes my brain hurt. But it may be worth doing!
We would need to add additional data to the keyboards, maybe something like this:
<button data-key="ONE" data-key-shift="EXCLAMATION">1</button>
<button data-key="TWO data-key-shift="AT"">2</button>
<button data-key="THREE" data-key-shift="HASH">3</button>

And then depending on the application, we'd handle it one way or the other.
There's a similar problem where some apps treat NUMPAD_ZERO and ZERO the same, and others treat them as different keys. I try to solve for that with the --explicit-numpad-keys option in the /utils/export_intermediate_data.py script.
We need to do a bit of research into how different applications behave with various languages. If a pattern can be found, we should be able to solve this problem.

Yes.
I have been trying as this steps:
<button data-key="LESSTHAN" data-key_shift="MORETHAN" data-text_shift=">"><</button>

Regards.

I have a problem as well that seems it could be solved with what you're suggesting here.
I can't access the following symbols "!"#€%&/()=?`" for shortcuts, since they are layered on "1234567890+´" on my keyboard, but modified with the ⇧ key. When I press ⇧ it looks for ⇧ + 1, but this is incorrect for me, because as soon as I press ⇧ the key in 1 position is now representing something else. (in this case "EXCLAMATION") This means that a lot of shortcuts are hidden for my keyboard layout.
Adding additional data for keyboards like you suggested would in my case for Danish Mac keyboard layout be:
<button data-key="ONE" data-key-shift="EXCLAMATION">1</button>
<button data-key="TWO" data-key-shift="AT">DOUBLE_QUOTE</button>
<button data-key="THREE" data-key-shift="HASH">3</button>

However when going down this road you might as well add data-key elements for data-key-option and data-key-shift+option since those modifiers will result in other keys.
Assuming a key can only represent one character goes against the whole idea of mapping shortcuts and unfortunately makes this project quite limiting.
I'm assuming there isn't currently a way to have one data-key result in multiple options?
If so that could solve the problem if shortcuts were just stored with a shift before. The software I'm working with exports everthing like this, so when I set a shortcut for "!" is exported as Shift+!. It would help my case, but there's probably many times where that is not the case.
Another way to solve it is to dictate that "!" is always the same as "Shift + 1", """ the same as "Shift + 2", "#" the same as "Shift + 3" and so on. But I think your approach is probably the best.


feat: add data pre-processing diagnostics

Fellows brainstormed a few quick tools we can provide to backend creators to confirm that the datasets they are loading follow the prescribed format and will not cause problems when calculating index. A few ideas:

Generate maps (quick RStudio output, does not have to be pretty) of where a dataset is missing values
Generate visualizations of NANs using naniar
Warn users that a high percentage of their overall data is NAN
Warn users that NAN values have to be imputed to calculate index; may have to be assigned 0 or median

None of these are critical path but may be useful backend tools for creators

this is pretty much done but I can continue improving it with more printed feedback about data


Duplo 11472

Change description
Add examples for Infrastructure settings and lambda permissions
For some reason
make doc

isn't generating the documentation for lambda permissions.
Related issues
https://app.clickup.com/t/8655600/DUPLO-11171
https://app.clickup.com/t/8655600/DUPLO-11472
Checklists
Development

[ ] Pull requests may not be submitted to the master branch (use develop instead) - or they will be closed.
[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

Task linked: DUPLO-11472 TF: Doc:Example for infrastructure setting

Task linked: DUPLO-11171 TF: Add example usage for duplocloud_aws_lambda_permission resource


Fix broken loading of client CAs

The SSL_load_client_CA_file() failed to load any CAs due to an
inccorrect assumption about the return value of lh_*_insert(). The
return value when inserting into a hash is the old value of the key.
The bug was introduced in 3c82e437bb3af822ea13cd5a24bab0745c556246.

I just see this :
=head1 BUGS

lh_TYPE_insert() returns B<NULL> both for success and error.

+1

commit 9d6daf9 pushed to master.


[ocis+web] activity: design API

Description
User Stories

As an ocis client application developer, I want an efficient search API for activities, so that I can query activities for certain contexts

As an ocis client application developer, I want an efficient search API for activities, so that I can build a personalized kind of "what happened in my ownCloud" page for users

As an ocis client application developer, I expect the activity search API to respect the KQL format like for file search so that I can reuse my already existing search ui implementation

Value
Make activity available for client applications.
Acceptance Criteria

activity search query syntax is KQL (like for file search)

resourceID:12345 depth:-1 limit:1000

it should be possible to set a limit (e.g. 1000 items)
pagination would be nice, but is optional
filters for the first iteration are:

resourceID: to limit an activity search to a file, folder, share or space root (ALL my accessible files / folders / shares / spaces if no resourceID is set, but we can make this resourceID required in the first iteration if that makes things easier)
depth (default -1): include all subfolders of the given fileId if set to -1.
activity types (optional): limit the search to certain activity types (see Event types in https://github.com/owncloud/web/issues/10800) (to be discussed... not needed for MVP but one of the next requirements after MVP)

if the user who makes the search request doesn't have the permission to see activities, early return with an error response (403?)
graph https://learn.microsoft.com/en-us/graph/microsoft-graph-activity-logs-overview

This issue should be solved in two steps:

1: design API, at least include client devs from the web ui, to be decided by @tbsbdr if other client teams should join as well. As an outcome, client teams can work with mock data to start the client implementation. The API design is this expected outcome of this issue.
2: implement the API, tracked via https://github.com/owncloud/ocis/issues/9196

Definition of ready

[ ] Everybody needs to understand the value written in the user story
[ ] Acceptance criteria have to be defined
[ ] All dependencies of the user story need to be identified
[ ] Feature should be seen from an end user perspective
[ ] Story has to be estimated
[ ] Story points need to be less than 20

Definition of done

Functional requirements

[ ] Functionality described in the user story works
[ ] Acceptance criteria are fulfilled

Quality

[ ] Code review happened
[ ] CI is green (that includes new and existing automated tests)
[ ] Critical code received unit tests by the developer

Non-functional requirements

[ ] No sonar cloud issues

Configuration changes

[ ] The next branch of the ocis charts is compatible

Visual

From what I currently have in my mind, this is a search api in every situation. Please involve me in concepting if possible. 🙈

MS-Graph docs

https://learn.microsoft.com/en-us/graph/api/activities-list?view=graph-rest-beta
https://learn.microsoft.com/en-us/graph/api/resources/itemactivity?view=graph-rest-beta

Activity type mapping oCIS <-> MS-Graph

added (ms action name: create)

postprocessing-finished (ms action name: does not exist)
folder-created (ms action name: does not exist)
file-touched (ms action name: does not exist)

item-trashed (ms action name: delete)
item-moved (ms action name: move)
item-renamed (ms action name: rename)
share

share-created (ms action name: does not exist, only general share exists)
share-removed (ms action name: does not exist, only general share exists)
link-created (ms action name: does not exist, only general share exists)
link-removed (ms action name: does not exist, only general share exists)
space-member-added (ms action name: does not exist, only general share exists)
space-member-removed (ms action name: does not exist, only general share exists)

(TBD) Unauthenticated download (ms action name: does not exist)

oCIS graph-api endpoints

GET /drives/{drive-id}/activities (redirect to /drives/{drive-id}/items/{drive-id}/activities)
GET /drives/{drive-id}/items/{item-id}/activities

Request

headers

Accept-Language: e,g, en-US

urls params

drive-id
item-id

query params

limit: e.g. limit=999
query (kql): e.g. depth:1

depth: default -1

Response

{
	"value": [
		{
			"id": "5380e156-d65e-4024-9691-0f0c1f2796e4",
			"action": {
				"delete": {
					"name": "some resource name",
					"objectType": "folder"
				}
			},
			"driveItem": {
				"id": "7f92b0a9-06ad-49dc-890f-0e0a6eb4dea6$e9f01ca3-577f-4d1d-acd4-1cc44149ac25!5fb9f87c-a317-467b-9882-eb9f171564ac"
			},
			"actor": {
				"user": {
					"id": "31c63047-b271-45bf-82fd-7aa597fb08a8",
					"displayName": "Albert Einstein"
				}
			},
			"times": {
				"recordedTime": "2017-07-29T18:34:40Z"
			},
			"@template": {
				"message": "{user} deleted {resource}.",
				"variables": {
					"user": {
						"id": "71f9de60-8b24-4cfe-9378-87d47aef0d04",
						"displayName": "Marie Curie"
					},
					"resource": {
						"id": "7f92b0a9-06ad-49dc-890f-0e0a6eb4dea6$e9f01ca3-577f-4d1d-acd4-1cc44149ac25!5fb9f87c-a317-467b-9882-eb9f171564ac",
						"name": "some folder name"
					}
				}
			}
		}
	]
}

Topics

We can only create multiple events for a resource that is shared with multiple individuals.
Grouping as shown in the mock is not easily possible.
Item restore is not mentioned in the ticket, microsoft has a dedicated event for that.
Should we consider that too, and if yes, should we add it to the added events or to the restore events?
In oCIS we do not differentiate between file-edit and file-added, therefore, we are not able to render the
message for that differently, e.g. Albert Einstein edited some file.
Unauthenticated download activity is not part of the MVP.
Getting activities for all spaces is not part of the MVP.
Should we create a new activities service or make it part of the existing graph service?

hm ... the ms graph api also has an endpoint for a user activity stream:

https://learn.microsoft.com/en-us/graph/api/resources/activity-feed-api-overview?view=graph-rest-beta
https://learn.microsoft.com/en-us/graph/api/resources/projectrome-activity?view=graph-rest-beta

The activity feed is a dedicated stream that can be fed by any application. It is part of Project Rome in case that rings a bell.

adding @kulmann here because he has more requirements for the KQL part, especially time ranges.

The kql part for the MVP is kind of minimal or even not existent.
Anyways the KQL query is just a string we send as query parameter and should not have a big impact on the API specification.

Done in ADR and in https://github.com/owncloud/libre-graph-api/pull/184


[CopyFilesOverSSH]: Forced task process completion after success

Task name: CopyFilesOverSSHV0
Description: Added force exit from task process.
For fixing the possible problem with stalling of the task, which described here: #15651, I suggest just to force stop the task process after the completion.
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) #15651
Checklist:

[x] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

Closing this due unsafe task execution method


Add documentation for 5A-75B V8.0 board

Hi,
I recently ordered a 5A-75B and got a new revision: V8.0.
This model seems to have only one 2MB SDRAM chip, but is otherwise very similar to the others 5A-75B boards.
I also runned pin-scan to map some of the output pins, but the mapping of SPI, SDRAM or PHY's pinouts is still in progress and are untested.
Thanks for all the previous work which allowed me to test this very quickly :)

Thanks for starting with the documentation of the v8 revision of the board. When examining it, I found the following:

The button is not located at M13 (as with v7), at least I can not get it to work.
J1 pin 8 (hub75 E) is assigned to E3, but on all other connectors to N4.
J8 pin 7 is assigned to N5, but that port is already assigned to hub75 A. If I had to make a guess, I think B15 would be a likely candidate.

I've been looking at the v8 revision myself, since that is what I got when I ordered the board. And there are indeed some errors in your documentation.

The pinout on J1 is not completely correct, pin 6 should be F5, pin 7 should be E3 and pin 8 should be N4. This is the result with my board at least. With pin 8 as N4 it also correctly matches all the other connectors for pin 8-15.
The pinout on J8 is not completely correct, pin 7 should be B15, as @erwin3169 mentioned in their comment.
The button is located at pin R7. I found this by trial and error looking at where the connection should approximately be.
I do not agree with @erwin3169 on the polarity of the LED. As far as I can tell the LED is still active low.

Hopefully these comments are helpful for improving the documentation

I think setting M13 to input has caused the LED to become active high, because after changing the button port to R7 the LED is indeed active low again. Pretty weird. Thanks @msvisser.

Hi @erwin3169 and @msvisser , thank you for your feedback on this revision.
It indeed contained some errors =(

The button is not located at M13 (as with v7), at least I can not get it to work.

The button  seems to be located at R7 as indicated by @msvisser

J1 pin 8 (hub75 E) is assigned to E3, but on all other connectors to N4.

Corrected, J1 pin 8 is assigned to N4 similarly to all others connectors.

J8 pin 7 is assigned to N5, but that port is already assigned to hub75 A. If I had to make a guess, I think B15 would be a likely candidate.

B15 was indeed a good guess (checked with pin-scan)

The pinout on J1 is not completely correct, pin 6 should be F5, pin 7 should be E3 and pin 8 should be N4. This is the result with my board at least. With pin 8 as N4 it also correctly matches all the other connectors for pin 8-15.

It is now corrected

The pinout on J8 is not completely correct, pin 7 should be B15, as @erwin3169 mentioned in their comment.

Also corrected

The button is located at pin R7. I found this by trial and error looking at where the connection should approximately be.

Confirmed with pin-scan

Hopefully these comments are helpful for improving the documentation

They were very useful, thank you both!

I have found some more connections, this time for the ethernet PYHs.

MDC is on R5
MDIO is on T4
PHYRSTB is on R6
These three connections are shared for both of the PHYs, so lowering the PHYRSTB line will reset both PHYs. The management interface over MDC/MDIO can access the registers for both PHYs by addressing them with address 1 and 2.

Hi! Could you check/confirm that the JTAG pinout is correct? JTAG pins are backwards-numbered on this version of the board.


fix: slt fail due to bors problem

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

This PR resolve logical conflict (bors problem).
Quick merge.

Thank you -- sorry about that @jackwener


devito: unstable-2022-04-22 -> 4.7.1

Description of changes
Changes
compiler: Fix checkpoint size when save != None @speglich (https://github.com/devitocodes/devito/pull/1906)
compiler: Revamp generation of implicit equations @FabioLuporini (https://github.com/devitocodes/devito/pull/1908)
API
compiler: Support interpolation with user-provided implicit dims @FabioLuporini (https://github.com/devitocodes/devito/pull/1948)
api: Add op.cinterface for C-level interoperability @FabioLuporini (https://github.com/devitocodes/devito/pull/1843)
Examples
examples: Fix grammar and spelling mistakes in pml example @PershingSquare (https://github.com/devitocodes/devito/pull/1930)
examples: Update MPI tutorial notebook and scripts @georgebisbas (https://github.com/devitocodes/devito/pull/1923)
examples: Use solve instead of by hand derivation @mloubout (https://github.com/devitocodes/devito/pull/1879)
examples: Rename the viscoacoustic equations @nogueirapeterson (https://github.com/devitocodes/devito/pull/1869)
examples: Update attribute names and backend for synthetics notebook @EdCaunt (https://github.com/devitocodes/devito/pull/1895)
examples: Add colab/wsl dependencies to gempy notebook @EdCaunt (https://github.com/devitocodes/devito/pull/1866)
Fix 2D Gzz when sintheta==0 @mloubout (https://github.com/devitocodes/devito/pull/1855)

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.devito
python39Packages.devito

Thanks for the review as always!

Added the backport label since this is finally a stable release.


Blogs and Events  Section Added{conflicts resolved}

Pull Request Format
Solved Issue #377   : CONTENT AND BLOGS SECTION solved
Type of PR

Add X in the box to specify the improvement type.
[] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): ___________

Description
This pull request aims to add a new feature.

@swaraj-das I have resolved conflicts and created a new pull request. Please review it.
Checklist

Add X in the box to specify.
[X] I have performed a self-review of my code.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Thank you for reviewing my pull request!

@swaraj-das Please review this PR.


Final renaming of activities to events

Please merge after PR #722 (may require me to rebase this one before accepting it)
Includes:
File and folder renaming
Final source file updates
SQL file updates
CSS class updates
This final PR fixes issue #705

this one can be rebased now

is it me or are there several changes that aren't really changes... Did spacing change?

@tonysurma I've rebased off of the latest master. I'm not sure why those files are showing complete changes. I renamed some folders which perhaps caused git to see complete file changes? I'm not really sure how to fix it though. Any thoughts?

I unfortunately don't.  @BillWagner or @MisterJames any thoughts?  Maybe it is the folder name switch?

I don't know. I hope to learn something from @MisterJames :smile:

I'm wondering if perhaps the line endings were different in some files and when I did my renaming of folders and final manual check of the files they got updated and saved.


CodeMirror-5.58.3 demo syntax error

After fixing the codemirror.js src (#6486), I am getting an error in Firefox when opening the demo
Uncaught SyntaxError: import declarations may only appear at top level of a module    codemirror.js:1

I have no idea why that is.
Could there be an error in main.js? :thinking:
// EDITOR CONSTRUCTOR

import { CodeMirror } from "./CodeMirror.js"
export { CodeMirror } from "./CodeMirror.js"

I think this one can be closed, as its premise (#6486) isn't correct.


Fix a potential undefined constant reference

There's a chance of the VERSION constant being undefined if a file that uses it is loaded before the file with GetStream\Stream\Client. In PHP 7.4 results in an error exception being thrown. We actually didn't experience this error until yesterday, so by sheer luck the files have always been loaded in the right order.
This PR moves the constant into the Client class (and marks both constants explicitly as being public as it's good practice to do so) and references this class constant explicitly in the other files.

Thanks for this PR :heart:
It seems it was missed before but I think this is already addressed on master so I will close this. However, I can ensure next time any issues and/or PRs will be reviewed ASAP.
Thank you for your contribution :pray:


chore(deps): bump dependencies

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


Random DCC decoder failures

Sometimes DCC decoder is running out of memory buffer. It leads to blinking white frames.

It has 100% reproducibility with StoneLambda object (Cairn stones)


Registration logic/verification services

The user must now specify their status (Student, Alum, etc.) on the registration form. Additionally, the verification services now exist.
Yet to be implemented:

Verification service tests
Actual registration logic/email

LGTM!

LGTM!


Odd behavior of cached, USA-wide datasets for islands

It's somewhat strange to see islands with no providers that still have access to services under the driving time metric. We should investigate what values OSRM is returning in these cases.

this is also bad

we should put unknown or something, right?


Russian localisation for hoi4modutilities

Hi! I'm impressed with your mod and decide to help with russian localisation (I'm native russian speaker and with auto translate most of sentences don't have sense at all^)
i18next really nice choice for localisation and if you want, i can help with future localisation when more great functions will be ready coz i'm already starred repo :)

Thank you, @Ivan-Corporation!
There's another i18n related file: https://github.com/herbix/hoi4modutilities/blob/master/i18n/template.ts
Are you going to translate it too?

Oh, yeah of course, i'm just thought that it is some future logic. I'm translated sites only on json format)
I'm think i will translate it today, this repo is really helpful and how i know game really popular in Russia, so it's be really useful to have full translation

well it was not so easy but ru.ts is ready. I hope i can help in future too

@Ivan-Corporation It's late for me now. I'll create a new version containing your translation tomorrow.


Genai/optimum support streaming mode

Support chunk streaming mode, mainly to reduce the number of decode calls, thereby improving performance.

@zhaohb now for 24.6 =)

@zhaohb now for 24.6 =)

I had submit a new PR for 24.6.


[Question] 403 Forbidden when accessing Overview of all environments in 2.2.0

Seeing odd behavior (403 Forbidden) when trying to navigate to the overview page for all environments on our new 2.2.0 build of puppetboard.
puppetboard.example.com/*/ (overview of all environments) results in a 403 Forbidden page
puppetboard.example.com/*/nodes (nodes in all environments) works as expected
puppetboard.example.com/example_env/ (overview of example_env environment) works as expected
puppetboard.example.com/example_env/nodes (nodes in example_env environment) works as expected
Puppetboard logs the following from the request that results in 403 error:
INFO:puppetboard.utils:PuppetDB Version 7.1.0
ERROR:pypuppetdb.api:Forbidden request: /metrics/v2/read/puppetlabs.puppetdb.population%3Aname%3Dnum-nodes (method :get). Please see the server logs for details. pdb-svc:8080 over HTTP.<IP_ADDRESS> - - [18/Mar/2021:20:21:26 +0000] "GET /%2A/ HTTP/1.1" 403 234 "https://puppetboard.example.com/%2A/nodes" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.192 Safari/537.36"ERROR:puppetboard.utils:403 Client Error: Forbidden for url: http://pdb-svc:8080/metrics/v2/read/puppetlabs.puppetdb.population%3Aname%3Dnum-nodes

PuppetDB logs the following from the request that results in 403 error:
<IP_ADDRESS> - - [18/Mar/2021:20:21:26 +0000] "GET /pdb/query/v4/environments HTTP/1.1" 200 51 "-" "python-urllib3/1.26.4"
<IP_ADDRESS> - - [18/Mar/2021:20:21:26 +0000] "GET /pdb/meta/v1/version HTTP/1.1" 200 25 "-" "python-urllib3/1.26.4"
2021-03-18 20:21:26,735 ERROR [p.t.a.rules] Forbidden request: <IP_ADDRESS> access to /metrics/v2/read/puppetlabs.puppetdb.population%3Aname%3Dnum-nodes (method :get) (authenticated: false) denied by rule 'puppetlabs puppetdb metrics'.
<IP_ADDRESS> - - [18/Mar/2021:20:21:26 +0000] "GET /metrics/v2/read/puppetlabs.puppetdb.population%3Aname%3Dnum-nodes HTTP/1.1" 403 140 "-" "python-urllib3/1.26.4"

Puppet Server version: 7.0.3
PuppetDB version: 7.1.0
puppetboard version: 2.2.0
Postgres version: 11.11
bash-5.0# pip3 list
Package      Version
------------ ---------
certifi      2020.12.5
chardet      4.0.0
click        7.1.2
commonmark   0.9.1
Flask        1.1.2
Flask-WTF    0.14.3
gunicorn     20.0.4
idna         2.10
itsdangerous 1.1.0
Jinja2       2.11.3
MarkupSafe   1.1.1
pip          21.0.1
puppetboard  2.2.0
pypuppetdb   2.3.0
requests     2.25.1
setuptools   53.0.0
urllib3      1.26.4
Werkzeug     1.0.1
wheel        0.36.2
WTForms      2.3.3

I ran into the same problem a few days ago, and it appears that the /metrics/v2 PuppetDB endpoint does not allow for unauthenticated access. Maybe this was introduced in PuppetDB 7.1.0, at least there's mention of it in the release notes. This is configured by the 'puppetlabs puppetdb metrics' rule in the /etc/puppetlabs/puppetdb/conf.d/auth.conf file.
Because my PuppetDB is not externally accessible anyway, I fixed it by allowing unauthenticated access to the /metrics/v2 endpoint, by changing the rule to this:
        {
            "allow-unauthenticated": true,
            "match-request": {
                "method": [
                    "get",
                    "post"
                ],
                "path": "/metrics",
                "query-params": {},
                "type": "path"
            },
            "name": "puppetlabs puppetdb metrics",
            "sort-order": 500
        }

and now my overview works again.

Thanks @gerardkok for your help!
Allowing unauthenticated access in the 'puppetlabs puppetdb metrics' rule resolved our issue with the overview page too!

Is this the proper way to resolve this issue? Isn't there a way to authenticate instead of allow-unauthenticated?


Add pt and pt-BR translation

Add pt and pt-BR translation

@darioajr thank you for the contribution!


Routing table

How do I print a routing table to the log or stdout?

There's debug logging which is enabled via DEBUG=koa-router shell variable.
If you want the array of route layers they are kept in router.stack array.

Thanks!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 40 ms

Resolved: ClamAV performance has improved in 8cafeab.


Instead of implementing Term::bindAt primitively, we should implement bindTele primitively

https://github.com/aya-prover/aya-dev/blob/70893a5a0ec59c7e2e76c3d9c24db36bbd92b481/syntax/src/main/java/org/aya/syntax/core/term/Term.java#L61-L65
Right now bindTele is based on bindAt, I think it should be the other way around: bindAt should be based on bindTele.

Also AbstractTele.Locns::bindTele should use bindTele on terms, I think


Question: why not depot_tools?

A bit of yet another question about incremental builds, but i haven't quite seen the answers to the question i have in other issues, Has been looking into why and how. For some context i have only a single windows machine and it took me about a day to build chromium from scratch.
It seems as if depot_tools allow for shallow clone and shallow fetch. I have tried shallow clone and shallow fetch via git with base chromium repository / no submodules and shallow fetch from one tag to another is miles faster. I would assume depot tools has an option of switching by tags while remaining shallow or at the very least it shouldn't be too hard to edit it to that point. So i was wondering why so?
depot_tools seem to be glorified git wrapper anyway and we have to get source code from google this way or another. And using git seems to solve many problems: faster update to new versions while remaining shallow, incremental builds and with a bit of tinkering unchanged dependencies might be left patched with new versions of chromium.
My general idea of how updates would work is as follows:

something akin to git reset --hard to restore chromium and submodules state
shallow fetch to a new tag with chromium and submodules
apply patches and stuff again
build

Theoretically should be only a few hundred files to rebuild after.
So why not? is it a tinfoil hat issue to minimize talking to the google services or does it undermine the ungoogled-ness of the final chromium binary somehow?
To clarify, for me it is not as much of trust issue in the binaries provided by community, but more of a curiosity and fun adventure. I understand  that i might be blabbering nonsense here, so if i am wrong i'd gladly be educated on the matter.

So why not? is it a tinfoil hat issue to minimize talking to the google services or does it undermine the ungoogled-ness of the final chromium binary somehow?

Those are good questions! My guess would be that neither someone did actually a side-by-side comparision of the two ways of obtaining sources, nor did anyone investigate how does this approach affect maintenance of the patches and pruning list. There is already a pretty solid workflow in place, why fix what is not [yet] broken…

To clarify, for me it is not as much of trust issue in the binaries provided by community, but more of a curiosity and fun adventure.

It would be of immense value to the community if you could take the adventure and repot us back all your findings :)


[BUG] org.opensearch.cluster.coordination.AwarenessAttributeDecommissionIT/testInvariantsAndLogsOnDecommissionedNodes flaky

org.opensearch.cluster.coordination.AwarenessAttributeDecommissionIT/testInvariantsAndLogsOnDecommissionedNodes

https://build.ci.opensearch.org/job/gradle-check/6792/
https://build.ci.opensearch.org/job/gradle-check/6738/
https://build.ci.opensearch.org/job/gradle-check/6825/

Closing since #5252 has been merged


file missing

Seems the file mail_compose.html is missing. This is the link when in inbox.html and pressing reply.
does it exist or is it removed for some reason from the repo.
Alex

Remove it and add compose functionalities here https://github.com/puikinsh/gentelella/pull/133


pkexec with RPM

pkexec does not exist for RPMs, therefore it is not recognized by the fastboot-assistant.
Information:

OS: Fedora
OS version: 41
Program version: v.0.6.2
Comes with version: v.0.7.x
Operating mode: native / VM
Location of error: on run

There is no fix for this, as RPMs with version v.0.7.x are no longer supported.


Footer v2

These conflicts will be resolved when we merge the previous pull requests

Will merge FooterV2.1


Kale should support additional languages

Kale was written to support different languages, but currently the only language that we have output messages for is English.  We should look into generating messages for the other language files.

Could you display English as a fallback language. It's a bit ridiculous when you are typing
kale help
to see this output
Je ne parle pas franτais
If you could simply display the help in English that would be way better.
TY.

Ugh. Thanks for pointing with out @CDeschietere! Decidedly unfriendly behavior. We'll do something to improve this.

New release published Don't show non-English message stubs to users here: https://github.com/IBM-Watson/kale/releases/tag/v1.8.1


Wishlist feature

I would like to add the wishlist feature, and create a wishlist page.
@soham0005 , can you assign me this issue under gssoc-extend, I would like to work on it.

Please elaborate what things you will be adding in this issue

@soham0005 i would like to include add these features:
1)Wishlist Icon: Add a small heart beside each product. When clicked, this will add the item to the user's wishlist.
2)Wishlist Page: Create a wishlist page where users can view all the items they have added to their wishlist. Using Context API to track the wishlist across all pages. This will avoid duplicating logic for each page and ensure the wishlist is accessible from anywhere.
3)Header Wishlist Icon: Include a wishlist icon in the website's header, allowing users to quickly navigate to their wishlist page.

@Veda273 you can start working on it. Do reach out if you have any doubts

@Veda273 any updates?


[DotNet] Fixed StyleCop warnings in all projects

Proposed Changes

Fix syntax that's non-compliant with StyleCop guidelines

Set to ignore rule SA0001  by modifying the ruleset files

OK @johnataylor, we will close the PR.


Can this package create vCard QRcode ?

I saw some QRcode which scanned by normal phone camera will directly go to phone contacts and allow to save. Can I create such QRcode using this library ? If yes, can advise how ?

I figured it out already. Thanks


Browserstack screenshots hold connection until timeout

@wpietri observed some weird behavior with how we're taking screenshots with Browserstack:

william [2:21 PM]
I can't disable them. But they currently hold the connection open after the screenshot until the timeout.
I'd like them to close after they're done, freeing up resources for some automated testing I'm doing.

@Mr0grog can you look into this when you have time?

Ah, thanks, being able to see the code helps. It looks like @Mr0grog already tried to fix this:
https://github.com/codeforamerica/snap-it-up/commit/4e77ab6259bc744ba747ca7b42ed6a0a543642ab
But BrowserStack is still showing things as continuing to run, so either this code isn't deployed or it isn't having the intended effect.

Looks like this was coming from the rails branch and appears to be solved with 63cf08b7f7b97899a7a7e9b671d4e195eab27ac7
I’ve also added project names to both master and rails branches so we can see what’s going on in Browserstack.


Merge 6.14.x into master

I think we can close this PR. All changes are already on master except the changelog update, we can cherry-pick the changelog update individually.

Opening a new PR only with the changelog then, I thought we would merge everything, but it seems we have never done it this way. Note that with this approach, our release.json doesn't contain any patch version release, as well as all the releasenotes/notes/prelude-release-6.* for minor releases are missing on master.


[Feature] Add support at .net 3.5  Framework target

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

What is the point of FW 3.5 support in 2019? Making our code base and build processes more complicated for absolute edge cases is not helpful (we just removed 3.5 support from AvalonEdit for good).

Hi @christophwille
sorry if I didn't open a problem before creating a pr.

What is the point of FW 3.5 support in 2019? Making our code base and build processes more complicated is not helpful (we just removed 3.5 support from AvalonEdit for good).

FW 3.5 at now is more diffuse version of .NET Framework. It' is running from oldest win xp at new windows 10.
There is a lot of software compiled with FW 3.5 that cannot be updated since proprietary libraries are not available to the new version of framework. Especially in machine automation.

Yes, it is old. And that is precisely my point - it makes modernizing the library hard. No, we won't re-add support for this ancient framework.

I respect your decision and I will close the PR, but I am not of your opinion.
If an old man continues to do his job well, do I have to treat him if he has a problem?

Well, the problem here is that it doesn't do the job equally well. This PR is in itself testament to that. You need workarounds, polyfills and preprocessor directives to get it to work. This means that we have to support code that we normally never run, have to do additional polyfills / backports for new features and as a result, spend less time doing actual development.


Add support for browser zoom

Currently, the athena CLI does not support the BrowserWindow web preference for scale. This forced canvas elements to render fairly pixelated. This PR allows the passing in of a zoom option to set the zoom scale (e.g. 2 = 200%, 3 = 300%).
Here is a screenshot of a PDF rendered at zoom scale 1:

And here is a screenshot at zoom scale 3:

Hey @cavneb, thanks a bunch! A much needed feature.
I've came across the DPI problem several times, but I haven't quite found a proper solution (the Electron community seems to have the same issue). We did things like converting the canvas element to an image, and adjusting the zoom through CSS transformations. I believe this should do it for now.

Glad to help. Great library!!


Can't parse GNU tar file with long paths

See the attached file. It was created using GNU tar:
$ touch<PHONE_NUMBER>1234567801234567801234567801234567801234567801234567801234567801234567899999999901234567890123456780123456780123456780123456780123456780123456780123456780123456780123456789999999990123456789
$ tar cf test.tar<PHONE_NUMBER>1234567801234567801234567801234567801234567801234567801234567801234567899999999901234567890123456780123456780123456780123456780123456780123456780123456780123456780123456789999999990123456789

test.zip

Hi,
Thanks for the report, will look into it

Ok, so ftr, this tar archive has an entry of type 'L' (which is in the "vendor-specific" reserved range which are not implemented yet), followed by an entry of type '2' which is supposed to be a symlink ubut I guess that's just some metadata for the 'L' one, and then an entry of type '0' which is the actual file.
I'm gonna look at gnu tar sources to get how 'L' is working, implement that and probably error-out when we encounter an entry with a non-implemented type.

Thanks for looking into it. It's a GNU extension used to implement long file names. The extra entry contains the real name (followed by a NULL, I think).

Sorry for the delay, this should be fixed in 0.3.0