[BUG] payment_method not found error when trying to make a payment using a payment token

Bug Description
The application throws a payment_method not found error when trying to make a payment using a payment token. The payment token is updated from the "list payment methods for customer" API.
Expected Behavior
The application should use the payment token to fetched the saved payment details from the vault.
Actual Behavior
The application throws an error saying the payment_method was not found.
Steps To Reproduce

Create a merchant account
Create a connector account
Make a card payment with setup_future_usage=on_session and a customer ID so the card gets saved.
List the payment methods for the customer ID and get the payment token for the previously used payment method.
Use this token for making another payment (with no explicit payment details passed, just the token)

Context For The Bug
  2023-06-06T11:27:13.456264Z ERROR router::services::api: error: {"error":{"type":"invalid_request","message":"Missing required param: payment_method","code":"IR_04"}}
├╴at crates/router/src/services/api.rs:556:55
│
╰─▶ {"error":{"type":"invalid_request_error","code":"IR_04","message":"Missing required param: payment_method"}}
    ├╴at crates/router/src/core/payments.rs:871:18
    ╰╴Missing required field payment_method in None
    at crates/router/src/services/api.rs:663

Environment
Are you using hyperswitch hosted version? No
If not (or if building/running locally), please provide the following details:

Operating System or Linux distribution: MacOS
Rust version (output of rustc --version): 1.69.0

Have you spent some time to check if this bug has been raised before?

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

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
No, but I'm happy to collaborate on a PR with someone else

@vspecky I am interested in looking at this issue. Please can you assign this to me?
@SanchithHegde just tagging if you can assign this to me.

@vspecky I am interested in looking at this issue. Please can you assign this to me? @SanchithHegde just tagging if you can assign this to me.

@jeevaprakashdr Hey, unfortunately, this is a high priority task which would require a lot of domain knowledge as well as internal knowledge so we want someone from the internal team to pick it up ideally. Thanks for expressing interest tho!


Sort the resources in the resource list

Version: main
UI: dev-6168160
Commit ID: 2123c63e14cb8461eb00fc5a594999b5bc82483d
Env: MacOS 13.1, Apple M2
Currently, the resource list is displayed as an unordered.  This clutter is especially evident when deploying multiple instances of the same module. Please sort the resources before displaying them.

Environment

Walrus version: v0.5.1

Test Result:
The frontend displays components based on the order returned by the API.
For resources of the same type, the order of components returned may vary inconsistently.  As shown in the two screenshots below:


Issue #14: Removes info fields

This PR removes info fields, moves allele-specific metadata to alleleInformation field, and adds a note about extending the records as per Marc's proposal.
Built on top of #33.

Closing as #14 was rejected.


Test rig for identifying leaks

We should set up a test rig for Tor sessions that detects any network activity that does not go through the Tor proxy.  E.g., we could mimic the iptables configuration in Tails, and automate doing tests of various kinds in this environment.
(This is a separate task from https://github.com/brave/muon/issues/465, which is to do code analysis to find such leaks by studying code and thinking hard.  Leaks found by such a test rig would demonstrate that we didn't study hard enough, and possibly inform where else we need to audit.)

WebRTC is always a good leaker, might want to disable that when used with Tor.


fix: examples, mac wasm

Fixes the examples, allows building of the 3d example on OS X by setting build flags on internal/gl files such that darwin builds resolve from the _wasm files/funcs.

Recommend using the "Hide whitespace" option to reduce the noise from go fmt


How to customize stroke width ?

Hi,
is there a way to customize stroke width of icons ?
regards,

Try to specify stroke-width on the component, i.e. <x-tabler::2fa stroke-width="5" />.


Add a radius option

Could you please consider adding a radius option so that the users could modify those corners?

@StevenLi-DS,  if you're interested, there is a current way to change the border radius manually with a snippet.
.page-gallery__tile {
border-radius: /* type custom value here */;
}


Fix generated code to follow the Go standard

Fix the generated code:

apply the Go standard for generated code defined at https://golang.org/s/generatedcode
add Makefile rules to run the code generators: make generate

Thank you. I (obviously) wasn't aware of those standards and conventions.


docs: try autobuild for local dev

What does this PR do?
This could be an exciting feature for faster interaction during local development, ref: https://github.com/sphinx-doc/sphinx-autobuild

Before submitting

[ ] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?
Make sure you had fun coding 🙃

📚 Documentation preview 📚: https://torchmetrics--2518.org.readthedocs.build/en/2518/

@baskrahmer, have you tried this one, and do you see the difference? :)

hited this issue https://github.com/sphinx-doc/sphinx-autobuild/issues/155

let's wait until https://github.com/sphinx-doc/sphinx-autobuild/pull/157 is released


[FR] Add TTS Api endpoint

This project is awesome, the only thing that i miss is an endpoint i can call programmatly to make TTS calls.
tried adding them to the server.py part but im no programmer sadly :/

you have a cli available at core.py with that you can make an api


[Violation] 'keyup' handler took long time : Browser Unresponsive

Browser becoming unresponsive when doing search. Also getting a violation like:

10:48:03.198 zone.js:1438 [Violation] 'keyup' handler took 532ms
10:48:03.711 zone.js:1438 [Violation] 'keyup' handler took 506ms
10:48:05.138 zone.js:1438 [Violation] 'keyup' handler took 520ms
10:48:05.640 zone.js:1438 [Violation] 'keyup' handler took 497ms

My code for search:
<input class="form-control form-control-sm mr-sm-2 mb-3 mb-sm-0" #pendingSearch (keyup)="pendingLeaveTable.gtSearch(pendingSearch.value)"
                placeholder="Search">

Is there any way to do the search async?

Hmm, haven't seen or had any issues with this myself, are you doing something very demanding in your rendering or value functions perhaps? You could also try adding debounce to the key up event so it's not fired immediately to prevent it from executing the search to fast.


Introduce Auto-Format to Sprig Editor #1399 - Edited

after the feedback from @leomcelroy on #1475 , i have changed the option as leo tested one which seems to be the way we want on editor.

This also opens all of the collapsed assets. You should probably collapse the assets by default when this is pressed.

Collapse the assets means to close the text of sprites, maps, and tunes.

Ah okie, I will commit the fix soon :)

lgtm, agree w/ feedback provided already

Hey @leomcelroy i have added the collapse after format lmk if this is done now :)

Okay looks good to me.


Matterport3D semantic annotation loading fails

❓ Questions and Help
The Matterport3D region bboxes do not make a lot of sense, especially the Y coordinate for aabb.center and the sizes aabb.sizes. I ran the test script from issue 830.
To reproduce
from typing import Callable, List, Union
import numpy as np
import habitat_sim
from habitat_sim import bindings as hsim
from habitat_sim.agent.agent import AgentConfiguration

def _config_sim(scene_filepath):
    settings = {
        "scene": scene_filepath,  # Scene path
        "default_agent": 0,
        "sensor_height": 1.5,  # Height of sensors in meters
        "silent": True,
    }
    sim_cfg = hsim.SimulatorConfiguration()
    sim_cfg.gpu_device_id = 0
    sim_cfg.scene_id = settings["scene"]

    # create agent specifications
    agent_cfg = AgentConfiguration()
    return habitat_sim.Configuration(sim_cfg, [agent_cfg])

cfg = _config_sim('../data/scene_datasets/mp3d/17DRP5sb8fy/17DRP5sb8fy.glb')
sim = habitat_sim.Simulator(cfg)
print('Region 0 Center: {}'.format(sim.semantic_scene.regions[0].aabb.center))
print('Region 0 Sizes: {}'.format(sim.semantic_scene.regions[0].aabb.sizes))
print('Region 0 Category: {}'.format(sim.semantic_scene.regions[0].category.name()))

Expected output
Region 0 Center: [-8.821845   1.259409  -2.6915383]
Region 0 Sizes: [ 4.1633096  2.5356617 -4.207343 ]

Current output
I ran these trials on Ubuntu 20.04 with habitat-sim installation from source at this commit: b0a8b5999fd8421c1b93cff5fa3becc10e67a443
Flags used for installation: python setup.py install --bullet --headless --with-cuda
Trial 1
Region 0 Center: [-8.821845  0.       -2.691538]
Region 0 Sizes: [0.000000e+00 2.370487e+11 0.000000e+00]
Region 0 Category: bedroom

Trial 2
Region 0 Center: [-8.821845  1.25     -2.691538]
Region 0 Sizes: [      0. 4815464.       0.]
Region 0 Category: bedroom

Trial 3
Region 0 Center: [-8.821845  0.       -2.691538]
Region 0 Sizes: [0.0000000e+00 1.1089253e+26 0.0000000e+00]
Region 0 Category: bedroom

Note: I get the expected output on my macbook when I run the same code with the same environment. I used the same commit for installation, but used the following flags: python setup.py install --bullet

Could this line from Simulator.cpp be a bug?
          scene::SemanticScene::loadMp3dHouse(houseFilename, *semanticScene_);

The definition of loadMp3dHouse requires a rotation to be provided as the 3rd argument. But there is not third argument above.
bool SemanticScene::loadMp3dHouse(
    const std::string& houseFilename,
    SemanticScene& scene,
    const quatf& rotation /* = quatf::FromTwoVectors(-vec3f::UnitZ(),
                                                       geo::ESP_GRAVITY) */ ) {

Could this line from Simulator.cpp be a bug?
          scene::SemanticScene::loadMp3dHouse(houseFilename, *semanticScene_);

The definition of loadMp3dHouse requires a rotation to be provided as the 3rd argument. But there is not third argument above.
bool SemanticScene::loadMp3dHouse(
    const std::string& houseFilename,
    SemanticScene& scene,
    const quatf& rotation /* = quatf::FromTwoVectors(-vec3f::UnitZ(),
                                                       geo::ESP_GRAVITY) */ ) {


No commits since 3 years?

Is this project not active any more? Was it superseded by another one? It looks very complete and powerful and I'm thinking to use it for myself. But no activity (and support) worries me a bit.

I still accept PRs but I no longer maintain it myself.


Update BASE_VERSION to 1.13-2022-02-23T19-01-17

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio_release-1.13
be575eccb11c5ee6cec58c7a9dac3d74a76edbd3
link
true
/test integ-telemetry-istiodremote_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio_release-1.13
be575eccb11c5ee6cec58c7a9dac3d74a76edbd3
link
true
/test integ-telemetry-istiodremote_istio_release-1.13

unit-tests_istio_release-1.13
be575eccb11c5ee6cec58c7a9dac3d74a76edbd3
link
true
/test unit-tests_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Later version merged with https://github.com/istio/istio/pull/37523


Use TorchVision's Multi-weight Support and Model Registration API on Lightning

Hey Lightning team,
Thanks a lot for all your work on this project.
I'm raising this PR to raise awareness of two important API changes that happened on TorchVision. In v0.13 TorchVision released its Multi-weight support API which allowed us to deploy updated weights for the majority of the models. This replaced the pretrained=True parameter with a weights=something (something can be a string or an enum). In addition to this, TorchVision has recently introduced a new Model Registration API which allows the easier listing of models and weights. Its aim is to eliminate the somehow hacky idiom of using torchvision.models.__dict__ to retrieve models and weights. This is expected to be released on v0.14.
I would love to hear your thoughts on the proposed changes. For full transparency, though the Multi-weight API is stable, the Model Registration is currently considered Beta. We've intentionally marked it as such to ensure that we can act upon the feedback of our partners and users. We are currently collecting feedback at https://github.com/pytorch/vision/issues/6365 and hoping to promote to stable before TorchVision v0.14. If you have any feedback or feature requests, we would be happy to discuss more.
Please note that it's likely that you might decide not to merge this PR and that's totally OK. Perhaps this can be used as an example for future PRs. Finally, apologies if I put things on the wrong place or use idioms you don't favour,  I'm very new to Lightning's code-base.
Looking forward to your feedback.

@datumbox If this is the recommended way by torchvision moving forward, I think we should go ahead with your changes :)
In the future, once our minimum torch version pushes also the minimum torchvision compatible version over the deprecation end, we can then remove again the backward compatibility.

@awaelchli @carmocca @Borda Thanks a lot for your help on this.


Comment out modes that do not work from the menu yet.

Once a menu item is selected and doesnt lead to anything it should atleast show a text on what the mode is supposed to be in the future and say it isnt finished yet.

The menu is going to get reorganized with the new design and for modes that don't work such as Challenges we can just comment them out for the time being.


Lucky env predicates

This PR addresses parts of #4 by moving the Lucky::Env code into lucky_env itself.

Thanks for adding this in! I've opened up https://github.com/luckyframework/lucky_cli/issues/654 to address what the generated apps should do now.


homebridge v2 issue collection

With newer versions of homebridge there is now this log message in the log view:
NOTICE TO USERS AND PLUGIN DEVELOPERS
> Homebridge 2.0 is on the way and brings some breaking changes to existing plugins.
> Please visit the following link to learn more about the changes and how to prepare:
> https://github.com/homebridge/homebridge/wiki/Updating-To-Homebridge-v2.0

It seems like the update doesn't really affect this plugin, and we don't have any major changes. But I still need some time upgrading to the v2 beta and testing the plugin.

After testing I have found no issues. There is also no deprecated code used.
If anything comes up, please add it to this issue!


Prevent change

Hi, how can I prevent change in toggle, example, I'm receiving data from database, and when I click in toggle-button I want to execute a method which changes value in database, but I can cancel that operation, problem is when I click on button it changes automatically even canceling operation, what I want is that the button wait if response is true or flase to change. Many thanks

toggle(event) {       if (!this.sync) {         this.toggled = !this.toggled;       }       this.$emit('input', !this.toggled);       this.$emit('change', {         value: !this.toggled,         srcEvent: event,       });     },
I forked and made a quick fix like this, yes you end up getting a delay in movement on the toggle - but I can add a tiny loader, which is enh.

You can do those with @click.native
<toggle-button @click.native="yourFunction"/>

and inside your function call preventDefault() like this
click(event){
    event.preventDefault();
    //TODO
}

Even simpler:
<toggle-button @click.native.prevent="toggle" />

That also prevents an issue I was seeing where the toggle function was being called twice for a single click.


Fix doc relative links

Signed-off-by: Tom Huang<EMAIL_ADDRESS>

Thank you so much @tomhuang12


Theme: Checkbox in the table row, when left-clicked is not toggled

Hello,
I'm on Chrome and search for a cron by description name
then it returns a list of related crons...
ok, but I can't select the cron row
when I click on the checkbox it loads the related cron page instead of just selecting...
thanks

Hi,
Could you please be kind making video-screencast because however I try I can't reproduce it under any browser/OS?
Example

You can always use right-click to select the row.

..but again, I can't reproduce it with the left click. When I click the checkbox with the left click it selects/deselects it and not enters the row..

Could you make the video screencast of how it doesn't work? I can't reproduce it!
I'm working now on another release and I made those rows more clear, when it's clicked and when it's not. I guess that you're just miss-clicking the checkbox.

Miss-clicking on the checkbox will open the link and make you feel that something is going on. With new release, it will be correspondent cursor on hovering the row, that will be more user-friendly.
Remember, you could always user right click, to select the row. There is no need for clicking checkbox.
I'm closing issue for now and in case it doesn't work for you on 17.80, please provide video screencast.

Actually I got it and think it's related to 39974.
It should be fixed by 17.80.
Please be kind posting here after upgrading.

Probably testing it now might already work. Please test and clear the cache after updating the file:
rostovtsev.ru/pub/src/authentic/17.80b/authentic.min.js

Please upgrade 17.80 is out!


[#4587] docs: Refactor the security document and add authorization push-down page

What changes were proposed in this pull request?

Refactor the security document
Add authorization push-down page

Why are the changes needed?
Fix: #4587
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
CI Passed

What's the difference between access-control.md and permission-system.md?

The terms of new document vary from origin document.

Use resources instead of securable object
Use permission system instead of access control system.
Use permission instead of privilege in many places.

The terms of new document vary from origin document.

Use resources instead of securable object
Use permission system instead of access control system.
Use permission instead of privilege in many places.

OK, I updated document.

@jerqi Please help me review this PR, Thank you!

@jerqi Please help me review this PR, Thank you!

I'm ok for current state. Let @yuqi1129 take another look.


CIPD package duplicate-tag causes builds to fail.

Impact: All builds were broken.
Cause: Accidentally uploading a CIPD package with the same version-tag as that was already uploaded. This causes gclient sync during builds to fail due to ambiguity in resolving.
Fix: CIPD does not support removing uploads, nor removing tags. So the only fix is a fix-forward fix in which we update the DEPS file to something that is not ambiguous. The fix landed in master in https://github.com/dart-lang/sdk/commit/8d4d3a33adcf579babfc1f2fd628768d481ac96c. stable/beta require cherry picking CP request, dev requires rolling.
(Making old builds reproducible can only be done by asking chrome infra to manually delete the upload or tag, and manually purging the CIPD package manager locally. See https://crbug.com/1094879.)
Background: We use the CIPD package manager to "check in" large files to the repository. gclient sync pulls in packages (with these larger files). The DEPS file specifies which version of a package should be pulled in. gclient caches packages locally, just deleting the bad upload will still keep everything in a broken state.
Having duplicate tags on CIPD package uploads is a CIPD-feature, some teams have workflows which use this. Dart does not.
Mitigation: Use hashes instead of version tags for this specific CIPD package, landed in d87046342fc3f78b86a1631579c9b793b6ea1280.
Further mitigation to consider, prefer using version hashes always over tags, but that also makes the DEPS file harder to read.
@franklinyow @athomas

Commit has landed with fix-forward, and the bad tag has been removed which fixes old build reproducibility (when locally purging the cipd cache).

The cherry-pick was not needed, because the removal of the duplicate tag was sufficient to be able to do a dev release (and there is no reason to believe it would fail for beta or stable).
I reached out to the Luci team, and the current recommendation is to use instance ids. There is, however, a plan to improve this in Q3.


github_linter - log the updater's output to a file

actually, don't need to do this... worked out async tasks!


Finish basic dao Motoko example

Finish this example with tests, work was started here: https://github.com/demergent-labs/azle/issues/157

[ ] Use the new ic.candid_encode and ic.candid_decode where applicable

Implementing all of the Motoko examples is no longer a priority


It will be thankful if installation guide added !!!

maybe installation docs
maybe docker image

maybe installation docs

maybe docker image

sure~ coming soon

how can i build myself,please help


Git for Windows 2.3.5 - SSH Fix

This fixes an issue with the openssh version of ssh-agent where you need to enter credentials every time you connect over ssh.
I'm not sure if this is something we really want to do or not.  If you want to go back to the openssh version, you can just scoop reset openssh

Will this still be the case with the new version (2.3.7)?


collapse_whitespace in

The documentation says:

..is normally ignored by the browser outside of tags like <pre> and , one can use CSS properties such as "white-space: pre" to make the browser preserve whitespace within a portion of the document..

It would be nice if collapse_whitespace would also ignore <code> tags, as these are quite a common case for "white-space: pre".
Additionally, MDN says:

The HTML <code> element displays its contents styled in a fashion intended to indicate that the text is a short fragment of computer code.

Based on that description, it's plausible that a browser may choose to pay attention to whitespace, where it normally wouldn't (given that, for example, Python code without the whitespace would be invalid code).
So, skipping <code> content would make this filter a fair bit safer to use.

Sounds reasonable to me,  and looks like it's easy to accomplish. It looks like it is just a matter of adding <code> to kSensitiveTags  here:
https://github.com/apache/incubator-pagespeed-mod/blob/master/pagespeed/kernel/html/collapse_whitespace_filter.cc#L39


Update program ID to v3

@nathanielparke Can you also update the devnet id to  DESVgJVGajEgKGXhb6XmqDHGz3VjdgP7rEVESBgxmroY which Mango and now us (PsyOptions) are using? This should prob be merged soon btw

Yep will do!


Not working in 4.1 beta 2

using Test sample included with Bundle getting this in 4.1
me.createFieldAccessExpression is not a function
[Break On This Error] code.push(' dest', ...on(field, varName, 'source'), ';\n');
ext-all-debug.js (line 19350)
4.0.7 works fine

Please check it now, it should be fixed.

Looks good. Thank again.
On Feb 1, 2012, at 10:08 PM, Maximiliano<EMAIL_ADDRESS>wrote:

Please check it now, it should be fixed.

Reply to this email directly or view it on GitHub:
https://github.com/elmasse/Ext.i18n.Bundle/issues/1#issuecomment-3771906


Introduce Yourself pop up is empty

The "Say Hello to NEO" card displays with all options to Introduce yourself, Take a Tour, and View weekly Plan.
The bot responds when clicked on Introduce Yourself, with a pop-up window titled New Employee Onboarding that is empty.
The SharePoint list with the intro questions has been populated.
I need help with troubleshooting why the Introduce Yourself pop-up is empty, nothing is displayed.

Hi @mustafafarooq
Thank you for raising this issue, this issue occurs if there is any configuration not updated correctly, please check the solution provided for this issue.
If you still face the issue, kindly share your email so that we can troubleshoot over the call.

Hi @mustafafarooq,
Please let us know if you've any updates, if not we'll proceed with closing the issue.

here's my email<EMAIL_ADDRESS>

Thank you for sharing your email, I've sent an email and deleted it from this issue.

Can we connect again on this as the issue is still not resolved

@mustafafarooq sure, I will send meeting invite for tomorrow.

Lets try again to connect on this...

I'm seeing the same issue.  @magurrapu , @mustafafarooq - did you find a resolution to this issue?

Hey guys, my Introduce Yourself pop up is empty.  I've looked at the solution in the other thread, reviewed configuration and it looks OK.  Here is the most common error in Application Insights:
"Signature validation failed. Unable to match key: kid: '[PII is hidden. For more details, see https://aka.ms/IdentityModel/PII.]'. Exceptions caught: '[PII is hidden. For more details, see https://aka.ms/Iden…[show more]"
It also appears I'm getting 404 errors on Azure Storage.  Any pointers on what configuration would need changes?

The issue for me was configuration.  The Site Name should be set to the url segment for the site not the site title (Step 4 > 7 in deployment guide).


Error: Unable to find a publisher for URL "github.com", Please make sure to register this host under an integration in app-config

Hi Team,
I am getting an error while creating templet as following the steps mentioned in url https://backstage.io/blog/2020/08/05/announcing-backstage-software-templates
I have submitted all the required fields as:

Fill in template parameters
Name,
Description
Port
Choose owner and repo
Owner
Store path
Access

Then in next step create the details as:

Http Port
8080

Component Id
testingreact

Description
testingreact

Owner
Ravi

Store Path
https://github.com/ravichadda/testingBackstage

Access
Backend

Expected Behavior
*** Prepare Successfully in 31 seconds
***Template Successfully in 10 seconds
***Publish failed
Current Behavior
2021-03-03T08:59:56.000Z Beginning step Publish
2021-03-03T08:59:56.000Z Error: Unable to find a publisher for URL: https://github.com/ravichadda/testingBackstage. Please make sure to register this host under an integration in app-config
at Publishers.get (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/publishers.ts:43:13)
at Object.handler (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/legacy.ts:103:36)
at TaskWorker.runOneTask (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:127:24)
at async eval (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:53:9)
Possible Solution
No Idea
Steps to Reproduce

Step 1: Create http://localhost:8080/create/tasks/a9ad7f25-2a68-4f70-bc28-f6b4b4f11731
Step 2: Fill in all the required information
in creating process getting error as explained above.

Context
Your Environment

NodeJS Version (v12):v14.15.5
Operating System and Version (e.g. Ubuntu 14.04): Mac OS 11.1
Browser Information: Chrome

Do you have a GITHUB_TOKEN set with permission to create a new repository?
Related https://github.com/backstage/backstage/issues/4773

Yes, I have set the GITHUB_TOKEN as well.
integrations:
github:
- host: github.com
token:
$env: 14453b0814e316f41877ec8c3333ac6c40dbc148
Getting the same error:
2021-03-03T17:23:54.000Z Beginning step Publish
2021-03-03T17:23:54.000Z Error: Unable to find a publisher for URL: https://github.com/ravichadda/backstage_microservice.git. Please make sure to register this host under an integration in app-config
at Publishers.get (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/publishers.ts:43:13)
at Object.handler (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/legacy.ts:103:36)
at TaskWorker.runOneTask (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:127:24)
at async eval (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:53:9)

We are not able to debug exactly where is the problem, in Backstage documentation not clearly explained what type of configuration(app-config) required to create the template.
If possible could you please schedule a call to discuss the issue.
Thanks

@ravichadda Oops, don't paste tokens publicly - you should probably expire/rotate that one ASAP.

Yes, I have revoked it, Please suggest how do I resolve the above-said issue.

When using env, that means you're pulling it FROM an environment variable - not putting it IN your actual YAML file.
So I would have expected you to have one of two things.  Either,
integrations:
  github:
    - host: github.com
      token:
        $env: GITHUB_TOKEN

Which comes straight out of the box, and then you set a local environment variable. So for example, for a local environment you could say,
$ GITHUB_TOKEN=1010101010101010 yarn dev

And the ALTERNATIVE, is to statically hardcode it just for testing purposes, ideally in an app-config.local.yaml (which will be ignored with .gitignore) and it would look like this:
integrations:
  github:
    - host: github.com
      token:<PHONE_NUMBER>10101

What you pasted looked like maybe you were putting the value as ENV in your YAML.  Does that help at all?

I have added the same as you suggested
integrations:
github:
- host: github.com
token:<PHONE_NUMBER>10101
found another error :
2021-03-04T04:48:30.000Z Beginning step Publish
2021-03-04T04:48:30.000Z info: Will now store the template {"timestamp":"2021-03-04T04:48:30.724Z"}
2021-03-04T04:48:31.000Z HttpError: Repository creation failed.: {"resource":"Repository","code":"custom","field":"name","message":"name already exists on this account"}
at /Users/vandanasharma/Documents/BackStage_Code/backstage/node_modules/@octokit/request/dist-node/index.js:66:23
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async GithubPublisher.createRemote (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/github.ts:139:22)
at async GithubPublisher.publish (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/github.ts:86:23)
at async Object.handler (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/legacy.ts:105:45)
at async TaskWorker.runOneTask (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:127:11)
at async eval (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:53:9)
Please suggest accordingly.

The error from GitHub
2021-03-04T04:48:31.000Z HttpError: Repository creation failed.: {"resource":"Repository","code":"custom","field":"name","message":"name already exists on this account"}

suggests that you are trying to create an already existing repository. :) Try testingBackstage1 or something new.

I have tried the same "testingBackstage1", found the same error as:
2021-03-04T09:27:52.000Z Beginning step Publish
2021-03-04T09:27:52.000Z info: Will now store the template {"timestamp":"2021-03-04T09:27:52.680Z"}
2021-03-04T09:27:53.000Z HttpError: Repository creation failed.: {"resource":"Repository","code":"custom","field":"name","message":"name already exists on this account"}
at /Users/vandanasharma/Documents/BackStage_Code/backstage/node_modules/@octokit/request/dist-node/index.js:66:23
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async GithubPublisher.createRemote (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/github.ts:139:22)
at async GithubPublisher.publish (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/github.ts:86:23)
at async Object.handler (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/legacy.ts:105:45)
at async TaskWorker.runOneTask (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:127:11)
at async eval (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:53:9)

I have commented on the code where the error is shown as "backstage/node_modules/@octokit/request/dist-node/index.js:66:23"
The commented code is:
if (status >= 400) {
// return response.text().then(message => {
//   const error = new requestError.RequestError(message, status, {
//     headers,
//     request: requestOptions
//   });
  //   try {
  //     let responseBody = JSON.parse(error.message);
  //     Object.assign(error, responseBody);
  //     let errors = responseBody.errors; // Assumption `errors` would always be in Array format

  //     error.message = error.message + ": " + errors.map(JSON.stringify).join(", ");
  //   } catch (e) {// ignore, see octokit/rest.js#684
  //   }

  //   throw error;
  // });
}

Than after tried again and found another error as:
2021-03-04T09:35:33.000Z info: Creating new remote {dir=/var/folders/1q/rxlpdn1d0x9d4x5w7kwb1fqw0000gn/T/8330c5f3-e385-4b1a-99c5-f46dfb3d97a3/result,remote=origin,url=undefined} {"timestamp":"2021-03-04T09:35:33.052Z"}
2021-03-04T09:35:33.000Z MissingParameterError: The function requires a "url" parameter but none was provided.
at assertParameter (/Users/vandanasharma/Documents/BackStage_Code/backstage/node_modules/isomorphic-git/index.cjs:4329:11)
at Object.addRemote (/Users/vandanasharma/Documents/BackStage_Code/backstage/node_modules/isomorphic-git/index.cjs:4996:5)
at Git.addRemote (webpack-internal:///../backend-common/src/scm/git.ts:71:66)
at initRepoAndPush (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/helpers.ts:64:13)
at async GithubPublisher.publish (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/publish/github.ts:94:5)
at async Object.handler (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/stages/legacy.ts:105:45)
at async TaskWorker.runOneTask (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:127:11)
at async eval (webpack-internal:///../../plugins/scaffolder-backend/src/scaffolder/tasks/TaskWorker.ts:53:9)

So, I can see that the https://github.com/ravichadda/testingBackstage1 repository already exists but it's empty. Hence the "name already exists on this account" once again. @ravichadda Hopefully you are not creating the repository yourself, since Scaffolder is supposed to create that repository for you. So, please provide a URL in the create workflow to a repository which does not exist.
If the error happens again, it could be that Scaffolder is trying to create the repository twice.

The issue has been resolved. Thanks for your quick reply and support.

@ravichadda Nice to know! Could you share what worked out finally? (In case some looks at the issue in future)


Use Kubernetes release-notes tool for releases

Part of https://github.com/ossf/scorecard/issues/1676
ref: https://github.com/kubernetes/release/blob/master/cmd/release-notes/README.md
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Redirect here from another issue, but felt more applicable to post this here.
I played briefly with release-notes to get an idea of what the CLI call might be if generating the patch notes for v4.10.3.
release-notes --org ossf --repo scorecard --branch main \
  --dependencies=false \
  --required-author "" \
  --start-rev v4.10.2 \
  --end-rev main

a portion of the output would look like.
Note: there are some issues with the autogenerated summaries for PRs with multiple code blocks:
- Accept pip installs with `-e` flag along with `--no-deps` flag in Pinned-Dependencies check. Editable pip installs will be accepted if coming from a local source or a remote Git source pinned by hash, as long as it's not installing dependencie
s. (#2731, @gabibguti)
- Add internal flags to support GitHub's internal scans (#2773, @laurentsimon)
- Add wasm files as binary artifacts (#2548, @gabibguti)
- Azure Devops is now a recognized CI system (#2662, @pettermk)
- Code-Review check uses leveled scoring
  Code-Review check returns inconclusive results when recent activity is solely made up of bot commits (#2542, @raghavkaul)
- Disable scorecard on PRs (#2571, @laurentsimon)
- GitLab support for Security-Policy check (#2754, @raghavkaul)
- Ko-build/setup-ko is detected as a packaging workflow (#2692, @imjasonh)
- Structured results for permissions (#2584, @laurentsimon)
- Suggested remediations for unpinned Docker images now maintain any tags that were present. (#2595, @spencerschrock)
- The CI-Tests check will show the correct check name when it encounters a runtime error. (#2756, @spencerschrock)
- The date format in the JSON results has been changed to RFC3339 (#2712, @naveensrinivasan)
- Update docs on how to run and debug Scorecard locally (#2587, @gabibguti)

There would need to be better review of the release-note field in the PR body.  Is this for a CLI user? a library user? etc
I'm not sure what the interaction would look like between release-notes and goreleaser would look like.

Closing as part of #3322


Allow quoting full messages when replying

This adds a configuration option quote_messages which allows quoting full messages when replying to a message in Matrix.
Disabled (default, and current behavior):
<alice> hello
<bob> alice: hi

Enabled:
<alice> hello
<bob> > alice: hello
<bob> hi

I find that when there are a lot of people talking on a channel, or when answering an old message, quoting it fully makes more sense than just mentioning the author.
Since I didn't want to change the behavior too much, I kept the current behavior as the default, but I suggest we change the default to the new behavior (or even remove the config option and make it the only behavior...?).

I would personally really like to see this merged.

Would be nice(r) to have it in the style of #214 so it's a general reply quote configuration that could have more states than two but I'd like to keep the original behavior at least for now as the default.
Otherwise I'm good with merging this.

So, would you prefer something like
REPLYBEHAVIOR ( --mention-author | --quote-message )

? Tell me if you prefer different names :)
I think I'll also change the quoting style to something more irc-y as explained in the issue, without option to choose the style
<bob> <alice> hello
<bob> hi

If that's fine with you

Sounds good!
Maybe REPLYMODE (--mention | --quote) so it's a bit more concise?
One thing I noticed just now was that this needs to be a network level setting (unfortunately) as setting and enforcing a global mode for shared Heisenbridges isn't ideal. It's also not ideal that you need to reconfigure it per each network but it's better than forcing everyone to use a single style.

All done. This is now a network setting only, with the naming you proposed. I kept the default as mention and used the IRC-style quoting.

Hello,
Do you think this could be merged? This is still a feature I miss regularly


More powerful derived kpis

Previously, a derived KPI specified as  X/Y would be treated by dividing each element of X by the average value of Y, and then analysis would proceed on these adjusted values:
 adjusted_i = x_i / mean(Y)

This, correctly, meant that the mean(Adjusted) was equal the weighted mean of all the x_i/y_i. However the variance was overestimated and therefore less powerful. This PR improves the power
The priority in this PR is make_delta. This PR doesn't not attempt to apply this fix to make_group_sequential, as that will soon be removed and replaced by other changes that are coming. Bootstrapping is also not affected by this PR (but perhaps it should be!)

Coverage increased (+0.7%) to 91.91% when pulling 5d0619bea8227bed8a803023ae2255afd8fda1c0 on aaron-mcdaid-zalando:more_powerful_derived_kpis into 62c042d1ab0bc0d8546ab8eb0364933a63bf05bd on zalando:master.

Don't merge yet, even though the tests pass! I found some bugs that weren't caught by any test. As well as fixing them, I think I'll add some tests. Basically, the closures/lambdas return by the four "worker factories" aren't really tested

OK, I'm happier now with this version of the code. Let's start reviewing!
I'm going to write another test though, to quickly run a large number of A/A tests (with small sample size) to confirm that the p-values are nicely distributed

@aaron-mcdaid-zalando thanks for this great fix. I'll take a closer look later today, but I have two general comments atm:

I merged your other PR, and now this PR has merge conflict with master branch you might want to resolve.
I've given admin to aaron-mcdaid-zalando. You should be able to merge  now.
Please also update corresponding documentation in the /docs folder when this PR is finished.

Thanks a lot again!

I updated documents in a two commits this afternoon

This morning, Monday, I pushed another few important commits to this. If a derived KPI is 5/0 then we should remove it. This might seem crazy, but we sometimes see that in real data. My first commits in this PR were a bit inconsistent about how they handled zeros and NaNs
Simply put: Any datapoint with NaN in the numerator should be discarded. Also, any datapoint with zero or NaN in the denominator should be discarded.

I merged with master (v1.2.5) this morning, and then discovered a bug. So I pushed a fix.
Earlier in this PR, I rewrote delta to handle the derived KPIs in a more correct way. But, while doing that, I broke support for pandas Series in the function. Now, I've changed this function so that all inputs are changed to np.array(dtype=float) as soon as possible
Tests passing again now

A few changes pushed since the last review:

A little extra commentary in the Glossary
delete some variables that are computed, but no longer used. And delete the _get_weights function as it's no longer used
Improve the testing a little, in particular adding a test where there is a non-zero effect. This is done to confirm that the pvalues are pulled down

I think we can merge this


Bicepparam inline overrides don't work when defaulting in bicep file instead of bicepparam file

Bicep version
23.1
Describe the bug
I can't override parameters that don't exist in the bicepparam (maybe this is intentional but i couldn't find any documentation specifically calling this out). I have a param someId string = '' in my bicep file and since it's defaulted i completely omitted it from my bicepparam file. If i do this i cannot override the default of '' inline via my AZ create command. It would be nice to allow the empty defaults in the bicep file and not require them in the bicep param file and still support inline overidding

We do not support bicepparam overrides currently. This is something we are hoping to eventually include, which would come with the feature of bicep deploy. Please comment there or upvote to gain traction! https://github.com/Azure/bicep/issues/11434

Doesn't the documentation say, if you are using a bicepparam or json parameter file in your command, then any and all inline parameters are ignored?


List VMs by Security Group & HA

Description

List VMs by "security group ID" and/or "ha enabled"
Improve List VMs command allowing to list VMs filtering by security group or HA enabled.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?

List VMs via API with different combinations. All worked fine.

filter vms with security group ID
list virtualmachines filter=id,name,haenable,securitygroup listall=true securitygroupid=<SG UUID>

list VMs according to the HA enabled
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=true
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=false

combione both added parameters
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=false securitygroupid=<SG UUID>
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=true securitygroupid=<SG UUID>
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=false securitygroupid=<SG UUID2>
list virtualmachines filter=id,name,haenable,securitygroup listall=true haenable=true securitygroupid=<SG UUID2>

@blueorangutan package

@GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2163

@blueorangutan package

@GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2166

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

LGTM based on code

Trillian test result (tid-2932)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37369 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4397-t2932-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 83 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Failure
295.16
test_vpc_redundant.py

test_hostha_kvm_host_fencing
Error
98.88
test_hostha_kvm.py

Very valid comment from @sureshanaparti ! @GabrielBrascher Can you elaborate on that?

Are you talking about @Pearl1594 's comment @wido ? (affinity group affinity ;)

Indeed @DaanHoogland, I quoted the wrong person. I meant @Pearl1594 his comment indeed.

@GabrielBrascher indeed please advice. As per enough 👍 and test effort I merged but overlooked @Pearl1594 's question. Should we revert or fix forward or is there a legitimate explanation to the answer the query? (cc @wido , @sureshanaparti )

@DaanHoogland seems affinityGroupId code is accidentally removed. As this PR changes impacting existing func, better revert and fix that, and merge it again.

My mistake @DaanHoogland @Pearl1594 @sureshanaparti. Should we revert this commit?

revert or fix forward @GabrielBrascher your choice for my part,, but with priority please.

@DaanHoogland @sureshanaparti @Pearl1594 @wido I have created PR #4405 addressing the removed affinity group ID.
I am sorry for all the trouble, guys. This was the first time that I messed this big with a PR.
I really appreciate all the reviews and heads up!


Fix: Check for layer extent in CanvasLayerRenderer.getDataAtPixel

Current behavior
If the CanvasLayerRenderer gets cross-origin image data, it returns an empty array so map.forEachLayerAtPixel will detect a layer  even if it is clicked outside of the layer extent.
Fixes #10099
New behavior
The CanvasLayerRenderer checks for each pixel if it is in the layer extent, when not it will return null in getDataAtPixel.

@ahocevar I will further add a test for this fix, but currently it looks like the async beforeEach in ol.renderer.canvas.ImageLayer#forEachLayerAtCoordinate  is not called so the test case fails. Maybe I have to check if my local development environment is set up correctly.

@boeckMt I re-ran the CI, and now all tests pass. The last failure you saw was unrelated to your pull request. You can mark the pull request "Ready for review" now, unless there is something else you want to address before.

@ahocevar, thanks also for merging this fix!


Fix when and unless return value

[x] Added or updated tests
[ ] Documented user facing changes
[ ] Updated CHANGELOG.md

Changes
This PR fixes the return value validation for the when and unless methods on the Models
I'm also not sure about this whole feature as it does what the when method should do but not what it actually can.
A more realistic (and invalid) version:
    /**
     * Apply the callback's query changes if the given "value" is true.
     *
     * @template TValue
     * @template TCallbackResult
     * @template TDefaultResult
     *
     * @param TValue  $value
     * @param callable($this, TValue): TCallbackResult $callback
     * @param callable($this, TValue): TDefaultResult|null  $default
     * @return TCallbackResult|TDefaultResult|$this
     */
    public function when($value, $callback, $default = null);

Or a valid version: A more realistic (and invalid) version:
    /**
     * Apply the callback's query changes if the given "value" is true.
     *
     * @param mixed  $value
     * @param callable($this, mixed): mixed $callback
     * @param callable($this, mixed): mixed $default
     * @return mixed|$this
     */
    public function when($value, $callback, $default = null);

Breaking changes
None

Could you add a test for each method?

@szepeviktor Is this test acceptable?

It seems so.

Thank you!


[ENH] Memory blockfile delete

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

#2162  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @HammadB and the rest of your teammates on  Graphite


dynamically generating tests doesn't work with tdd (works with bdd)

Prerequisites

[X] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[X] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[X] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[X] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
I have a suite that generates tests, mocha is configured with tdd. When I run the tests in vscode, they are properly discovered.
However when I run them on the command line, no tests are dynamically added.
However if I change mocha's config to bdd, change suite to describe and test to it, then everything starts working.
Steps to Reproduce
This works:

	const mocha = new Mocha({
		ui: 'bdd',
		color: true
	});
// ...

describe('foo', () => {
  [1,2,3,4].forEach(i => { it(i, () => {
     assert(i);
  } });
});

This doesn't:

	const mocha = new Mocha({
		ui: 'tdd',
		color: true
	});

// ...

suite('foo', () => {
  [1,2,3,4].forEach(i => { test(i, () => {
     assert(i);
  } });
});

Expected behavior: tests are added dynamically for tdd too
Actual behavior: "0 tests passed"
Reproduces how often: 100%
Versions

The output of mocha --version and node node_modules/.bin/mocha --version: 8.1.1
The output of node --version: v14.16.0
Your operating system

name and version: Windows
architecture (32 or 64-bit): 64

Your shell (e.g., bash, zsh, PowerShell, cmd): powershell
Your browser and version (if running browser tests): n/a
Any third-party Mocha-related modules (and their versions): n/a
Any code transpiler (e.g., TypeScript, CoffeeScript, Babel) being used (and its version): TS 4.2.2

Additional Information

const assert = require('assert');
suite('foo', () => {
  [1,2,3,4].forEach(i => { test(i + "", () => { assert(i); });});
});

@asklar node bin/mocha --ui tdd ../test.js is working perfectly.
I don't know why you are trying to squash "Mocha via Interface" and "Mocha via API" into one mess.
It's just one way or the other.


typo and paths fix

several typos
Fix an Issue with Paths, now same as:
https://github.com/FabianFG/CUE4Parse/blob/22ad6c42a27071cd91fdad71f2a02e8597031de9/CUE4Parse/FileProvider/AbstractFileProvider.cs#L63

I'm merging this without testing


feat: connect file uploads to the backend (make it full-stack)

Yay file uploads! 💪🏿 This was quite the implementation, but glad it's full stack now. During our meeting on Wednesday, I'll likely be going over this PR as a great introduction to full-stack feature development.

Added fileSize, pages, userId, createdAt, storageLimit, and storageUsed to course_materials and users.
Added foreign key constraint for userId in course_materials.
New files/stats API route for user file stats, requires authentication.
PDF upload route now authenticates users before upload.
file-list component: manages file display, search, sort, and delete.
file-stats component: shows total files, pages, storage usage with progress bars.
New server-side delete function for files, triggers file list revalidation.
Added pdf-lib for handling PDF operations.

Demo
https://github.com/user-attachments/assets/c5af6f16-5622-4d09-b3d8-3fabec04535d

Build failing, but I'll get to it in the morning. Been working on this for the past 45 minutes 🥲


JSON.SET NX|XX invalid options handling

Description : JSON.SET NX|XX invalid options handling added. In the future if we need to add more options after NX|XX then this logic needs to be updated, for now we are assuming that NX|XX is last option in JSON.SET command.
Reference Pr : https://github.com/DiceDB/dice/pull/228
Testing : Added tests for various invalid combinations of nx and xx json.set commands.

@vaibh1297 thanks for the quick turnaround. LGTM.


Client startup crash with 2.4.0

it's borked
https://gist.github.com/Vazkii/15286ab2f2ac3268be28b357fe1c8f30#file-gistfile1-txt-L334

I'm seriously frustrated by Forge and Gradle at this point.
Is ./gradlew --refresh-dependencies setupDecompWorkspace build still not enough?

Fixed with 2.4.1 release.

thank coppi


rmallox

Hello, I become a victim of the "corporate" version of Mallox, My files are encryped and added *.rmallox.
Can you help me with decryping of one file?

Hello! rmallox is a new version of Mallox. I can't decrypt rmallox files.
If you have a ransomware sample, I can tell you for sure about the possibility of decryption.

Thanks for your quick respons. The sample is attached. I had to change file type from *.i01.rmallox to txt.
Polned.txt

Hallo,
I just send you the encrypted file without changes of extension. See attachment.
From: Andrey Zhdanov @.>
Sent: Saturday, April 20, 2024 12:41 AM
To: rivitna/Malware @.>
Cc: ppvidi @.>; Author @.>
Subject: Re: [rivitna/Malware] rmallox (Issue #10)
Hello! rmallox is a new version of Mallox. I can't decrypt rmallox files.
If you have a ransomware sample, I can tell you for sure about the possibility of decryption.
—
Reply to this email directly, view it on GitHub https://github.com/rivitna/Malware/issues/10#issuecomment-2067362550 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AV4KOJ4PJWYNARPDY57SDKDY6GMP3AVCNFSM6AAAAABGPWZ7QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRXGM3DENJVGA .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AV4KOJ44AJ3WJX6O7UX333TY6GMP3A5CNFSM6AAAAABGPWZ7QWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT3HFZPM.gif Message ID: @.*** @.***> >

Unfortunately, I can't decrypt the rmallox files.


Account where a certain field (int) is empty

First of all, many thanks for this great PS Module!!
I have one issue though.
When I search for an account and one of the properties is empty, it does not return it.
For example:
Get-AtwsAccount -Filter {KeyAccountIcon -eq 1 -or KeyAccountIcon -eq ''}
I also tried (example would have 1,2,3 and empty as value):
Get-AtwsAccount -Filter {KeyAccountIcon -ne 2 -and KeyAccountIcon -ne 3}
If I remove all filters it returns the account just fine.
Something I'm doing wrong maybe?
Also tried $null IsEmpty etc.

We are looking into how to specify $null as a search filter using parameters. But until we figure that out you can try:
Get-AtwsAccount -Filter {KeyAccountIcon -isnull}
'-isnull' is a special QueryXML operator, so it not easy to guess that you may need it...
Hope that helps!

That works like a charm! Thanks for the quick answer

I think we have solved the issue now. I have updated the module on PSGallery, too. With the updated module you can use:
  Get-AtwsAccount -IsNull KeyAccountIcon # or any other parameter, of course
# and
  Get-AtwsAccount -IsNotNull KeyAccountIcon


Harvest metrics to monitor file-level performance

Is your feature request related to a problem? Please describe.
In a virtualized environment where a large number of VMs are running, I want to find and take action on VMs that are generating excessive storage I/O. Since there is no metric for file-based performance monitoring, it is very difficult to find the problematic VM among numerous VMs.
Describe the solution you'd like
I need a harvest metric that shows the top 25 files as shown in ONTAP's FSA (File System Analytics) feature.

@ceojinhak Could you confirm the version of ONTAP that you are currently using?

There are several ONTAP versions of NetApp gears in the data center. (ONTAP 9.13.x - 9.15.x)

hi @ceojinhak these changes are in the latest nightly if you want to try them out. To enable, edit your conf/restperf/9.12.0/volume.yaml template and uncomment the VolumeTopClients plugin. You will see a new section there named objects that includes client and file.
Import the nightly dashboards and on the ONTAP: Volume dashboard you should see two new rows


virt-top: init at 1.0.8

Motivation for this change
A top-like utility for showing stats of virtualized domains
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

It did not pass osx build
I have no osx locally, so travis-ci is the only way to try osx build.
It is slow and unconvinient to debug the build that way, so I am going to split this PR to few per-package PRs.

ocaml-gettext fails to build with OCaml ≥ 4.02.
File "pr_gettext.ml", line 68, characters 22-45:
Error: Unbound module Camlp4

@vbgl fixed for ocaml 4.02 and 4.03


Remove Hard-Coded Paths in Python

In the https://github.com/mikelor/TsaThroughput/blob/main/src/tsa_throughput/src/data/main.py we are hard coding the paths for reading/writing data. We should change these to relative paths(?) or something like that.
Right now, it's a "users" name is included in the path

completed


Any chance you could do a tweetdeck version?

I tried hacking the url, but no text displays in the tweets.  I prefer tweetdeck because you can filter out all the promotional tweets.
Cheers
Steve

Thanks for the suggestion, I'll see if I can add it to settings of this extension. If not, I'll create another version of the add-on for Tweetdeck.

This is live in the latest release v1.5.2 - thanks for your patience!

Thanks! All working well so far.


.travis.yml: Switch to latest xcode 10/11 point release

Currently, people must wait for a long time due to there are a lot of ci tasks.

Please add the images rather than replacing old ones. Please also update the README and add the respective macOS/Xcode versions.

If the newer version is compatible, i think it is better to hold one task for an env.


Does Dynaconf support multithreading?

Hello, does Dynaconf support multithreading?

Hi @mketiku
We assume Dynaconf can work a multithreaded program, each thread may or have its own dynaconf object or share the object.
we don't have tests for multithread cases yet, that would be nice to add some tests under example folder.
The only mutable attribute dynaconf has is current_env that can be changed at runtime, the test must assert that 2 threads can have different envs at a time.
For that I recommend instead of using_env method/context manager, to use the from_env because it will clone the settings object as a new object in memory.
I am interested in experimenting and testing it.

Thanks @rochacbruno, I appreciate the response.

Dynaconf is running in multi-threading environments


Wrong spelling of push request status

In cumulusci/tasks/push/tasks.py we look for a status of "Cancelled" but it should be "Canceled".
https://developer.salesforce.com/docs/atlas.en-us.object_reference.meta/object_reference/sforce_api_objects_packagepushrequest.htm

Also in cumulusci/salesforce_api/metadata.p


Farming Station tool slots are not honoring axe/hoe.

Either slot takes either tool. This is causing issues trying to have a farming station do normal farms and wood together.
Build 317.

I could not implement this strictly as asked because there are more than 2 types of tools that can go in the station. As it is implemented now the farm will take ONE of any tool type, regardless of the slot.

Nice, thanks!