Tf 10645/create contact form handler

Ticket(s)
https://airtable.com/appfJZShN8K4tcWHU/pagXdnDYi0tVl4u8R?HjEAY=recDYN1fAgGDmJB3z
Type of change

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

Description
Context:
https://docs.google.com/document/d/1I3e1gh2HEkHALvTbVPWSjWb840FfuYyAmnL2jwnSuqY/edit#heading=h.npashoj11pcrhttps://docs.google.com/document/d/1P9_lk50jeYj8seUsxci9lhxhCKJ5iNlFMgt7bAhvF3s/edit#heading=h.nk3qsjen004f
Review your work

[x ] Everything works as expected
[ x] Check for any uncaught ESLint problems

Code Clarity and Readability

[ x] Are variable and function names descriptive, meaningful, in the appropriate case (ex: snake, camel)?
[ x] Is the code self-explanatory and easy to understand?
[x ] Are comments used where necessary to explain complex logic or provide additional context?

Code Efficiency

[x ] Is the code DRY?
[x ] Are there any areas of unnecessary complexity or duplication that might benefit from refactoring?

Code Modularity and Reusability

[ x] Is the code and any new components organized in a logical location in the directory?
[x ] Can any code snippets be refactored into reusable components?

Best Practices and Coding Standards

[x ] Does the code follow the established coding standards and style guide of the project?
[x ] Are there any violations of best practices that should be addressed?

Overall PR Checklist:

[ x] I have performed a self-review of my own code
[x ] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes to the documentation (if applicable)
[x ] There are no new linting errors/problems in my code
[x ] I have added tests that prove my fix is effective or that my feature works (if applicable)
[x ] I have filled this PR out fully, and cleaned up any extraneous items so that it is clear and easy to understand

@emilyemilyemilyemilyemilyemily this looks great, should we hold on the addition of the in-review submission architecture doc?

@e-papanicolas sure!

testing

Build failed. Logs:
2024-02-09T22:28:34.957Z [INFO]: # Cloning repository: git@github.com:clearviction-devs/clearviction-wa.git
2024-02-09T22:28:36.973Z [INFO]:
2024-02-09T22:28:36.974Z [INFO]: Cloning into 'clearviction-wa'...
2024-02-09T22:28:37.303Z [INFO]: From https://github.com/clearviction-devs/clearviction-wa
                                 * [new ref]         refs/pull/171/head -> pr-171
                                 Switched to branch 'pr-171'
2024-02-09T22:28:37.523Z [INFO]: Successfully cleaned up Git credentials
2024-02-09T22:28:37.524Z [INFO]: # Checking for Git submodules at: /codebuild/output/src487362801/src/clearviction-wa/.gitmodules
2024-02-09T22:28:37.532Z [INFO]: # Retrieving environment cache...
2024-02-09T22:28:37.572Z [WARNING]: ! Unable to write cache: {"code":"ERR_BAD_REQUEST","message":"Request failed with status code 404"})}
2024-02-09T22:28:37.572Z [INFO]: ---- Setting Up SSM Secrets ----
2024-02-09T22:28:37.572Z [INFO]: SSM params {"Path":"/amplify/dqqxalj4ndxdp/prbhb/","WithDecryption":true}
2024-02-09T22:28:37.611Z [WARNING]: !Failed to set up process.env.secrets
2024-02-09T22:28:41.538Z [INFO]: # Node version 18 is available for installation
2024-02-09T22:28:41.652Z [INFO]: # Installing Node version 18
2024-02-09T22:29:43.721Z [INFO]: # Now using Node version 18
2024-02-09T22:29:43.841Z [INFO]: # Retrieving cache...
2024-02-09T22:29:43.883Z [INFO]: # Retrieved cache
2024-02-09T22:29:48.915Z [INFO]: ## Starting Backend Build
                                 ## Checking for associated backend environment...
                                 ## No backend environment association found, continuing...
                                 ## Completed Backend Build
2024-02-09T22:29:48.923Z [INFO]: {"backendDuration": 0}
                                 ## Starting Frontend Build
                                 # Starting phase: preBuild
                                 # Executing command: npm ci
2024-02-09T22:30:26.316Z [INFO]: ><EMAIL_ADDRESS>prepare
                                 > husky install
2024-02-09T22:30:26.368Z [INFO]: husky - Git hooks installed
2024-02-09T22:30:26.386Z [INFO]: added 976 packages, and audited 977 packages in 37s
2024-02-09T22:30:26.387Z [INFO]: 176 packages are looking for funding
                                 run `npm fund` for details
2024-02-09T22:30:26.388Z [INFO]: found 0 vulnerabilities
2024-02-09T22:30:26.422Z [INFO]: # Completed phase: preBuild
                                 # Starting phase: build
                                 # Executing command: npm run build
2024-02-09T22:30:26.703Z [INFO]: ><EMAIL_ADDRESS>build
                                 > export build=true && next build
2024-02-09T22:30:27.253Z [INFO]: ⚠ No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
2024-02-09T22:30:27.258Z [INFO]: Attention: Next.js now collects completely anonymous telemetry regarding usage.
2024-02-09T22:30:27.259Z [INFO]: This information is used to shape Next.js' roadmap and prioritize features.
                                 You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
                                 https://nextjs.org/telemetry
2024-02-09T22:30:27.342Z [INFO]: Linting and checking validity of types ...
2024-02-09T22:30:37.331Z [INFO]: Creating an optimized production build ...
2024-02-09T22:30:50.643Z [INFO]: ✓ Compiled successfully
2024-02-09T22:30:50.652Z [INFO]: Collecting page data ...
2024-02-09T22:30:53.449Z [INFO]: Generating static pages (0/65) ...
2024-02-09T22:30:55.507Z [INFO]: Generating static pages (16/65)
2024-02-09T22:30:57.538Z [INFO]: Generating static pages (32/65)
2024-02-09T22:30:59.786Z [INFO]: Generating static pages (48/65)
2024-02-09T22:31:01.255Z [INFO]: ✓ Generating static pages (65/65)
2024-02-09T22:31:17.361Z [INFO]: Finalizing page optimization ...
2024-02-09T22:31:17.366Z [INFO]: Collecting build traces ...
2024-02-09T22:31:20.562Z [INFO]:
2024-02-09T22:31:20.568Z [INFO]: Route (pages)                                        Size     First Load JS
                                 ┌ ○ /                                                9.56 kB         208 kB
                                 ├   /_app                                            0 B             143 kB
                                 ├ ○ /404                                             182 B           143 kB
                                 ├ ○ /about                                           3.08 kB         186 kB
                                 ├ λ /api/revalidate                                  0 B             143 kB
                                 ├ ● /calculator/[slug] (13552 ms)                    186 kB          366 kB
                                 ├   ├ /calculator/m-circ-main-4-ineligible (598 ms)
                                 ├   ├ /calculator/m-offense-dv-1-cont (553 ms)
                                 ├   ├ /calculator/m-offense-pro-5-ineligible
                                 ├   ├ /calculator/m-offense-pro-6-eligible
                                 ├   ├ /calculator/m-terms-main-3-ineligible
                                 ├   ├ /calculator/m-offense-pro-7-cont
                                 ├   ├ /calculator/m-offense-pro-1-cont
                                 ├   └ [+49 more paths]
                                 ├ ○ /contact                                         4.22 kB         163 kB
                                 ├ ○ /donate                                          4.39 kB         186 kB
                                 ├ ○ /get-involved (343 ms)                           25.5 kB         228 kB
                                 ├   └ css/4df78f2cd73d6b26.css                       1.13 kB
                                 ├ ○ /get-started                                     5.34 kB         192 kB
                                 └ ○ /why-vacate                                      6.3 kB          188 kB
                                 + First Load JS shared by all                        143 kB
                                 ├ chunks/framework-467b11a89995b152.js             45.4 kB
                                 ├ chunks/main-75bdb96a41ba80f4.js                  33.1 kB
                                 ├ chunks/pages/_app-1a36736ad88442b0.js            62.1 kB
                                 ├ chunks/webpack-6cc2ac675212e550.js               2.09 kB
                                 └ css/c6b4809f3cddaab8.css                         831 B
                                 λ  (Server)  server-side renders at runtime (uses getInitialProps or getServerSideProps)
                                 ○  (Static)  automatically rendered as static HTML (uses no initial props)
                                 ●  (SSG)     automatically generated as static HTML + JSON (uses getStaticProps)
2024-02-09T22:31:20.617Z [INFO]: # Completed phase: build
                                 ## Completed Frontend Build
2024-02-09T22:31:20.688Z [INFO]: ## Build completed successfully
2024-02-09T22:31:20.689Z [INFO]: # Starting caching...
2024-02-09T22:31:20.698Z [INFO]: # Creating cache artifact...
2024-02-09T22:31:41.130Z [INFO]: # Created cache artifact
2024-02-09T22:31:41.311Z [INFO]: # Uploading cache artifact...
2024-02-09T22:31:48.131Z [INFO]: # Uploaded cache artifact
2024-02-09T22:31:48.215Z [INFO]: # Caching completed
2024-02-09T22:32:14.757Z [INFO]: # Starting environment caching...
2024-02-09T22:32:14.889Z [INFO]: # Environment caching completed
Terminating logging...
\```

Build failed. Logs:
2024-02-09T22:28:34.957Z [INFO]: # Cloning repository: git@github.com:clearviction-devs/clearviction-wa.git
2024-02-09T22:28:36.973Z [INFO]:
2024-02-09T22:28:36.974Z [INFO]: Cloning into 'clearviction-wa'...
2024-02-09T22:28:37.303Z [INFO]: From https://github.com/clearviction-devs/clearviction-wa
* [new ref]         refs/pull/171/head -> pr-171
Switched to branch 'pr-171'
2024-02-09T22:28:37.523Z [INFO]: Successfully cleaned up Git credentials
2024-02-09T22:28:37.524Z [INFO]: # Checking for Git submodules at: /codebuild/output/src487362801/src/clearviction-wa/.gitmodules
2024-02-09T22:28:37.532Z [INFO]: # Retrieving environment cache...
2024-02-09T22:28:37.572Z [WARNING]: ! Unable to write cache: {"code":"ERR_BAD_REQUEST","message":"Request failed with status code 404"})}
2024-02-09T22:28:37.572Z [INFO]: ---- Setting Up SSM Secrets ----
2024-02-09T22:28:37.572Z [INFO]: SSM params {"Path":"/amplify/dqqxalj4ndxdp/prbhb/","WithDecryption":true}
2024-02-09T22:28:37.611Z [WARNING]: !Failed to set up process.env.secrets
2024-02-09T22:28:41.538Z [INFO]: # Node version 18 is available for installation
2024-02-09T22:28:41.652Z [INFO]: # Installing Node version 18
2024-02-09T22:29:43.721Z [INFO]: # Now using Node version 18
2024-02-09T22:29:43.841Z [INFO]: # Retrieving cache...
2024-02-09T22:29:43.883Z [INFO]: # Retrieved cache
2024-02-09T22:29:48.915Z [INFO]: ## Starting Backend Build
## Checking for associated backend environment...
## No backend environment association found, continuing...
## Completed Backend Build
2024-02-09T22:29:48.923Z [INFO]: {"backendDuration": 0}
## Starting Frontend Build
# Starting phase: preBuild
# Executing command: npm ci
2024-02-09T22:30:26.316Z [INFO]: ><EMAIL_ADDRESS>prepare
> husky install
2024-02-09T22:30:26.368Z [INFO]: husky - Git hooks installed
2024-02-09T22:30:26.386Z [INFO]: added 976 packages, and audited 977 packages in 37s
2024-02-09T22:30:26.387Z [INFO]: 176 packages are looking for funding
run npm fund for details
2024-02-09T22:30:26.388Z [INFO]: found 0 vulnerabilities
2024-02-09T22:30:26.422Z [INFO]: # Completed phase: preBuild
# Starting phase: build
# Executing command: npm run build
2024-02-09T22:30:26.703Z [INFO]: ><EMAIL_ADDRESS>build
> export build=true && next build
2024-02-09T22:30:27.253Z [INFO]: ⚠ No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
2024-02-09T22:30:27.258Z [INFO]: Attention: Next.js now collects completely anonymous telemetry regarding usage.
2024-02-09T22:30:27.259Z [INFO]: This information is used to shape Next.js' roadmap and prioritize features.
You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
https://nextjs.org/telemetry
2024-02-09T22:30:27.342Z [INFO]: Linting and checking validity of types ...
2024-02-09T22:30:37.331Z [INFO]: Creating an optimized production build ...
2024-02-09T22:30:50.643Z [INFO]: ✓ Compiled successfully
2024-02-09T22:30:50.652Z [INFO]: Collecting page data ...
2024-02-09T22:30:53.449Z [INFO]: Generating static pages (0/65) ...
2024-02-09T22:30:55.507Z [INFO]: Generating static pages (16/65)
2024-02-09T22:30:57.538Z [INFO]: Generating static pages (32/65)
2024-02-09T22:30:59.786Z [INFO]: Generating static pages (48/65)
2024-02-09T22:31:01.255Z [INFO]: ✓ Generating static pages (65/65)
2024-02-09T22:31:17.361Z [INFO]: Finalizing page optimization ...
2024-02-09T22:31:17.366Z [INFO]: Collecting build traces ...
2024-02-09T22:31:20.562Z [INFO]:
2024-02-09T22:31:20.568Z [INFO]: Route (pages)                                        Size     First Load JS
┌ ○ /                                                9.56 kB         208 kB
├   /_app                                            0 B             143 kB
├ ○ /404                                             182 B           143 kB
├ ○ /about                                           3.08 kB         186 kB
├ λ /api/revalidate                                  0 B             143 kB
├ ● /calculator/[slug] (13552 ms)                    186 kB          366 kB
├   ├ /calculator/m-circ-main-4-ineligible (598 ms)
├   ├ /calculator/m-offense-dv-1-cont (553 ms)
├   ├ /calculator/m-offense-pro-5-ineligible
├   ├ /calculator/m-offense-pro-6-eligible
├   ├ /calculator/m-terms-main-3-ineligible
├   ├ /calculator/m-offense-pro-7-cont
├   ├ /calculator/m-offense-pro-1-cont
├   └ [+49 more paths]
├ ○ /contact                                         4.22 kB         163 kB
├ ○ /donate                                          4.39 kB         186 kB
├ ○ /get-involved (343 ms)                           25.5 kB         228 kB
├   └ css/4df78f2cd73d6b26.css                       1.13 kB
├ ○ /get-started                                     5.34 kB         192 kB
└ ○ /why-vacate                                      6.3 kB          188 kB
+ First Load JS shared by all                        143 kB
├ chunks/framework-467b11a89995b152.js             45.4 kB
├ chunks/main-75bdb96a41ba80f4.js                  33.1 kB
├ chunks/pages/_app-1a36736ad88442b0.js            62.1 kB
├ chunks/webpack-6cc2ac675212e550.js               2.09 kB
└ css/c6b4809f3cddaab8.css                         831 B
λ  (Server)  server-side renders at runtime (uses getInitialProps or getServerSideProps)
○  (Static)  automatically rendered as static HTML (uses no initial props)
●  (SSG)     automatically generated as static HTML + JSON (uses getStaticProps)
2024-02-09T22:31:20.617Z [INFO]: # Completed phase: build
## Completed Frontend Build
2024-02-09T22:31:20.688Z [INFO]: ## Build completed successfully
2024-02-09T22:31:20.689Z [INFO]: # Starting caching...
2024-02-09T22:31:20.698Z [INFO]: # Creating cache artifact...
2024-02-09T22:31:41.130Z [INFO]: # Created cache artifact
2024-02-09T22:31:41.311Z [INFO]: # Uploading cache artifact...
2024-02-09T22:31:48.131Z [INFO]: # Uploaded cache artifact
2024-02-09T22:31:48.215Z [INFO]: # Caching completed
2024-02-09T22:32:14.757Z [INFO]: # Starting environment caching...
2024-02-09T22:32:14.889Z [INFO]: # Environment caching completed
Terminating logging...


[BEAM-7770] Add ReadAll transform for SolrIO

This follows the ReadAll pattern style introduced on HBaseIO but for the case of non SDF based DoFn.
R: @aromanenko-dev

retest this please

Since readAll() is a new public API method, please ,add test(s) for it. Apart from that, it's fine for me.

readAll() is already tested. Notice that read() was refactored to rely on readAll() in line 392. I can write an extra test if you prefer but it is just duplication because it is already covered. Should I write the extra test?

@iemejia You are right - it's clear that readAll() is tested implicitly through calling read(). However, tests are not (and shouldn't be!) aware  about internal implementation of tested method, so we have to have tests for all public API methods. So, yes, please add an extra test for that.

testReadAll added can you PTAL again @aromanenko-dev. Thanks!


Issue with magic-ext/auth in react native?

Hello,
I'm developing an app with react native and wanted to integrate with Magic, but i'm hitting an issue with magic-ext/auth where i get an error that the imported file can't be found. Most specifically: ...dist/react-native/index.native.js.
I'm fairly new to developing in react native, so there might be something that i missed here, but it seems that this line is pointing react native in the wrong direction, as there isn't any dist build directory with the name react-native. I have looked at the expo packages and they don't have this react-native prop.
The issue seems to be with the following line in package.json:
https://github.com/magiclabs/magic-js/blob/276bad5d201c4c37a86e7d914f8373ff84edb81f/packages/%40magic-ext/auth/package.json#L20
At the moment, the fix i have for it is pointing the react-native prop to the index.js file, or simply deleting the line completely.
Am i doing something wrong when installing this module? AFAIU, I don't think so.
Also, it seems that there is something similar on the webauthn extension too. There is a index.native.ts file, but the reference doesn't seem to be pointing to the right file, and it seems that there isn't a dist/react-native folder too
https://github.com/magiclabs/magic-js/blob/276bad5d201c4c37a86e7d914f8373ff84edb81f/packages/%40magic-ext/webauthn/package.json#L20
Please let me know if these lines are something leftover from a refactor or something of the like, and if you'd like me to submit a PR pointing those lines to the right files, or changing them in some other way.
Thank you

I'm having the same issue, is there a workaround or a fix coming up for this issue?

Did you guys react to the deprecation notice when you call any of the auth methods, and use the auth extension?

Yes, this is using the auth extension, but has nothing to do with the deprecation notice. It's a different error.

Is this reproducible when using Magic SDK without the auth extension?


[Feature Request] Slack mentions

Would really like to notify specific users of build failures. Any chance we could integrate that kind of functionality?
Could use a github user -> slack userr mapping like in this action: https://github.com/abeyuya/actions-mention-to-slack

If necessary I could implement and submit a PR

Hey it looks cool but I think it might be out of scope for this project. Passing custom messages is not something I wanted for this project


1.41

You need to configure an Azure solution that meets the following requirements:

Secures websites from attacks
Generates reports that contain details of attempted attacks
What should you include in the solution?

A. Azure Firewall
B. a network security group (NSG)
C. Azure Information Protection
D. DDoS protection

C

D*


Fixing a shadow/shallow typo :)

Hi Ulises!
I've just found this little typo. Congrats to the MVP!!

Thanks @oneeyedman! 🥹🤗


Is this dead project

Is there any chance this project will be updated, I saw on commits and some already 2years the latest was last month

I'm currently hesitating between BackPack and laravel admin. So I'd be interested too on the status of the project

I'm currently hesitating between BackPack and laravel admin. So I'd be interested too on the status of the project

me too, but I am still interested on this project. I hope this will be updated as I prefer this Laravel Admin over BackPack or Voyager

I don't think this is already in a road map, I was eager to update and maintain it.


Update apache-client, kinesis, sqs to 2.25.27

About this PR
📦 Updates

software.amazon.awssdk:apache-client
software.amazon.awssdk:kinesis
software.amazon.awssdk:sqs

from 2.25.21 to 2.25.27
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

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

Superseded by #852.


Create Neo-Forge Build

Create path for Neo-Forge Build

Neo-forge isn't far from forge at this time. According to their website, all that we need to do is update the repositories.

il read it ty


Will gRPC be supported in future?

Thank you for your work and sharing.I want to ask a question, when will this project support grpc?@tobegit3hub

Actually we has the Python grpc server  in this project  https://github.com/tobegit3hub/tensorflow_template_application .
But I do not think the Python grpc server is better than the one in C++ which is how TensorFlow Serving works. So we may not add Python grpc server in this project.

Actually we has the Python grpc server in this project https://github.com/tobegit3hub/tensorflow_template_application .
But I do not think the Python grpc server is better than the one in C++ which is how TensorFlow Serving works. So we may not add Python grpc server in this project.
But you can implement the simple one like this project does.

Thanks for your reply, I have implemented a grpc version using simple_tensorflow_serving. This is mainly because I want to use the features of custom op, however, tensorflow serving is not very well implemented.

After using custom ops, the speed of inference is 1.5 times faster than tensorflow serving. This op is  fastertransformer,which implemented by  Nvidia. I tried many ways to use it as a custom op in the tensorflow serving but failed. If I can add this op to tensorflow serving, the inference speed should be faster. Have a good weekend.


For #5852: Clicking an Extension Manager tab while still loading will show it

For #5852 and #9466.

@prksingh Changes pushed.

@MarcelGerber Merged. Thanks for the fix!


Please add a level  reward system

Hey. Sorry, I don't update this anymore. I put it on here for others to explore it and use it how they want. I might end up doing it if it gets requested enough but for now, maybe not. I do appreciate your interest in my repo though.

Made it public again: https://github.com/hyphonz-bots/discord-leveling-bot


inconsistency in e2e tests

I have noticed the same issue in 3 DIFFERENT PRs about e2e tests (all 3 are failing in the same place, while they are completely different PRs so it shouldn't happen [one is passing e2e tho {the 4th one}])

This mostely has to do with #1749, which we've mitigated for now (swept under the rug, see issue for more details).
However there is still a problem with the grunt integration test:
16:29:04 (5175) INFO InitialTestExecutor Starting initial test run. This may take a while.
16:29:04 (5175) WARN ChildProcessProxy Child process [pid 5221] exited unexpectedly with exit code 1 (without signal). Last part of stdout and stderr was:
	internal/process/main_thread_only.js:29
	    binding.chdir(directory);
	            ^

	Error: ENOENT: no such file or directory, chdir '/home/runner/work/stryker/stryker/e2e/test/grunt-stryker-test' -> '/home/runner/work/stryker/stryker/e2e/test/grunt-stryker-test/.stryker-tmp/sandbox3039564'
	    at process.chdir (internal/process/main_thread_only.js:29:13)
	    at process.chdir (/home/runner/work/stryker/stryker/e2e/node_modules/@stryker-mutator/core/node_modules/graceful-fs/polyfills.js:20:9)
	    at ChildProcessProxyWorker.handleMessage (/home/runner/work/stryker/stryker/e2e/node_modules/@stryker-mutator/core/src/child-proxy/ChildProcessProxyWorker.js:39:29)
	    at process.emit (events.js:210:5)
	    at emit (internal/child_process.js:876:12)
	    at processTicksAndRejections (internal/process/task_queues.js:81:21) {
	  errno: -2,

It runs stryker twice with different configs. I think the first run is conflicting with the second one. For some reason the .stryker-tmp directory is deleted by the first run while the second one started. We can mitigate this by configuring a different temp dir.


luci-app-shadowsocks-libev: add server links import feature

@yousong
Related docs:
https://github.com/Jigsaw-Code/outline-shadowsocksconfig
https://shadowsocks.org/en/config/quick-guide.html
https://shadowsocks.org/en/spec/SIP002-URI-Scheme.html

Ping @yousong

Please either write a small summary about what has changed between each of your revisions, or add commits and squash them only when they are ready to be merged.  That should be easier to review what has happened.

Thank you!


Add golang-lint to travis-ci

Proposed changes
Add support for golang lint
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (improve a current implementation without adding a new feature or fixing a bug)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Build (changes that affect the build system)
[ ] Docs (documentation only changes)
[x] Test (adding missing tests or fixing existing tests)
[ ] Other... Please describe:

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING documentation
[] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
With this change, the CI build fails due to errors in the linter.
Fixes #17

I can see a few ways to go:

Modify travis to not fail the build due to lint errors
Leave linter errors for this PR and fix in subsequent PRs
Fix linter errors in this PR

Do you have a preference?

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is 77.61%.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   73.75%   73.75%
=======================================
  Files          12       12
  Lines         621      621
=======================================
  Hits          458      458
  Misses        125      125
  Partials       38       38

Impacted Files
Coverage Δ

crypto/public_key.go
81.57% <100%> (ø)
:arrow_up:

crypto/fixtures.go
100% <100%> (ø)
:arrow_up:

crypto/transaction.go
80% <100%> (ø)
:arrow_up:

crypto/serializer.go
62.2% <63.41%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Allow setting AUTOUPDATE_MAX_DURATION

Currently, if you want to run the autoupdate for more then an hour, say during the weekend, you need to work around this cli. While the amount of shell script needed isn't that much, it's bound to be error prone if you just want to raise the cut off time.
Instead of having a hardcoded constant, AUTOUPDATE_MAX_DURATION, use that as the default and allow it to be specified as an configuration/environmental variable. Seems reasonable?

Hi,
Actually, we are removing this limitation, as everyone can handle timeouts on their side.
You can grab the 0.2.7 version in the releases section.
I will publish deb files on apt.gemnasium.com tomorrow.
Thanks!

(apt.gemnasium.com has been updated)


About Data File

Hi, Thanks for your excellent work!
How were the four files, betas_stats_eft_fits.npz,betas_stats_spin_fits.npz, pare_feat_stats.npz, and prohmr_feat_stats.npz, obtained? @statho
Thanks and best wishes!

+1. Have you figured out this issue?

The values in those files are calculated over the training datasets.

Does that mean calculating the corresponding features from the training set and calculating the mean and standard deviation for those features?

Thanks for your reply!


[UI] Colored bars under the menu items are not consistently showing up or hiding

Version: v0.13.0
Under each menu item, there is a bar that sometimes has colors and sometimes not. I'm not sure what the design should be, but here are some screenshots.
Initial screen shot with no hosts.

After clicking around the menu items with a host.

After clicking on Containers/Volumes, then it removes the colored bar.

Using chrome Version 41.0.2272.89 (64-bit)

Fixed in rancherio/ui#108

Cleared out cache and recheck and the bottom colored lines are not longer there.  Verified Fixed.


Version problem

Hello @giotiskl ,
I am the member of cdnjs project.
We want to host this library.
But there is a question I want to ask.
I install the library on npm which is the 1.2.4 release version.
But the version in /src/jquery.filterizr.js shows 1.2.3.
Please help me confirm that.
Thanks for your help!
https://github.com/cdnjs/cdnjs/issues/7820

It is a typo mistake, I forgot to update the file

Thanks!


walk until you hit config

Here's a problem I'm trying to solve:
In yargs I'd like to introduce the concept of a "yargs" stanza, which allows you to turn off some parsing features, etc. I'd like to have this configuration be part of the parser which I'm splitting into a separate module. I'd like the configuration to be read in from the parent library using yargs:
---
    package.json <-- read me.
           node_modules
               ------  yargs
                                 package.json <-- has no yargs stanza, don't read.
                                 node_modules
                                           ---- yargs-parser
                                                       package.json <-- has no yargs stanza, don't read.

Basically I'd like to keep reading up directories until I hit a package.json with the appropriate stanza, similar to the behavior of .npmrc. Would it make sense to add this behavior to this library, as a configuration option? is this a different module?

That won't work with npm@3 though, as yargs-parser and the users package might be on the same level. Instead I recommend you instead use a less fragile method of finding the parent config. Incidentally, I worked on a module for this a couple of weeks ago and finally took some time to polish it up and publish it: https://github.com/sindresorhus/parent-module See the Tip section. Just replace read-pkg-up with pkg-conf. Does that sound like a workable solution to you? Happy to discuss :)


Automate "hmda" realm setup in Keycloak

Keycloak provides realm and user provisioning automation with its Export/Import functionality.  On service startup, the export can dump the full state of a given realm and/or its users.   Conversely, on service startup, in import mode, it can import a previously-exported file.
The export does include sensitive data such as private keys and client secrets.  I have scrubbed those values out of the export file, and all appears to work OK.  I have confirmed that the private/public keys and cert is regenerated when not present.  I believe the same is true for the client secrets, but I have not confirmed it yet.
This system is honestly pretty awkward, especially in a Docker setup, but it does work.  We'll need to work on this more to make it a more reliable workflow, but I don't think we have to figure all that out as part of this PR.

I should probably update the README.md with all this before we merge.  I'll get to work on that.

I've updated the README with details about the automated build.  Should be good-to-go for review now.

I agree that this feels somewhat awkward and error-prone (import -> fiddle in admin conosle -> have to re-export -> re-scrub -> examine diff, etc.) Definitely would be worth having at least a script that does as much of this as possible through Keycloak's API.
Also, it could be worth putting in a default User into the file to be imported (so the UsersExportStrategy would be changed from SKIP to probably REALM_FILE) Perhaps this is worth another ticket.
That said, this is definitely an improvement over the current manual process, merging.


Special logic for scripts

It is very popular pattern to have && list of commands in package.json scripts:
  "test": "jest --coverage && eslint . && size-limit && yarn spell"

We can tread these scripts as lists: if locally one element changed we can ignore this element, but update others.
Of course, we should not hurry with this issue and do https://github.com/lamartire/sharec/issues/47 first

Got it. Yeah, this is a good idea.

@ai released at 2.5.0 :tada:


Fix an issue with conflicting directory names leading to box created in wrong folders.

The case is this. In my application, from the root, I have a directory config/..., as well as build/another-project/config/.... It so happens that both config directories contain boxes.
Without this bugfix, packr would put both packages under the config key in the builder. It would just append the box of the second directory (where order is somewhat arbitrary) to the first package. However, the full directory of the second package is lost. In the end, we end up with a single package under the config key, with two boxes, both generated in one of the two directories. Obviously, the Go code in the other directory would not be able to load the contents from its box properly.
I'm changing this by using the full path. There will be separate packages, each containing a single box, assigned to those keys. Therefore, the code will be generated at the right place.
Please take a look. Thanks!

@markbates Could you please take a look? Thanks!

This is a known issue, with a simple fix, use unique folder names. I usually just go up one more level, so instead of ./templates I do ../foo/templates.
There is a fix in the next release for this.

Got it. Any idea when the next release is coming out? We are blocked by this. Thanks!

@markbates The bug still exists in 1.13.7. Did you mean version 2 will contain a fix?
Using ../foo/bar is just a hacky solution, and it doesn't even solve every single case (because there may be multiple different valid ../foo/bar in the folder structure).

V2 contains a fix. Yes.
Thanks.

Mark Bates
On Sep 27, 2018, 11:06 PM -0400, Thomas Feng<EMAIL_ADDRESS>wrote:

@markbates The bug still exists in 1.13.7. Did you mean version 2 will contain a fix?
Using ../foo/bar is just a hacky solution, and it doesn't even solve every single case (because there may be multiple different valid ../foo/bar in the folder structure).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Implement composition API

Is your feature request related to a problem? Please describe.

Add a native composition API for use with Vue 3.
Some more info at https://dev.to/alvarosaburido/how-to-migrate-your-library-from-vue2-to-vue3-1h81

Hola guys, any news on this? Or any work around for the short term?
What's the best way to access methods such as localePath inside the setup method currently?

See https://github.com/nuxt-community/composition-api/issues/425#issuecomment-813102698

@rchl any ideas on how to get a translated string inside setup() (when using an <i18n> custom block)? Everything comes back as a raw translation path... totally lost on this. Anything I am doing incorrectly in the below example? I am using Nuxt 2 with Composition API.
<i18n>
{
  "en": {
    "exampleString": "Test"
  }
}
</i18n>

...

setup() {
  const { i18n } = useContext();

  console.log(i18n.t("exampleString"));
}

On nuxt 2 bridge:
<script setup lang="ts">

const { $apollo, $store, $i18n } = useNuxtApp()
$i18n.t('key')

</script>

@thisismydesign  is this suppose to work ?


Add usersList and chat.postEphemeral methods

Codecov Report

Merging #2 (4c3f3ae) into main (348aa90) will decrease coverage by 1.63%.
The diff coverage is 9.09%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   60.76%   59.14%   -1.63%
==========================================
  Files          19       19
  Lines         678      700      +22
==========================================
+ Hits          412      414       +2
- Misses        266      286      +20

Files
Coverage Δ

lib/slack_bot/api_client.rb
62.06% <9.09%> (-32.38%)
:arrow_down:


Profile card component

This pull request contains the solution to the Profile card component challenge on Frontend Mentor,
as a part of my Optimum BH Internship.
Profile card component solution provided here represents the user profile card. Card is composed of three sections, upper one with pattern image, middle one with user profile picture and lower one with user detail informations. Implemented mostly by using Flexbox CSS model and background-image CSS property(for background pattern around profile card) .
The solution on this branch contains the following:
Html page containing the corresponding DOM structure, including the Optimum header with anchor tags with urls(whose presense was requested by script tests).
Tailwind configuration file containing the extended theme atributes, like colors, sizes, breakpoints, etc.
Challenge starter files, incl. images and style guide
Solution design images(mobile & desktop layout)
CSS style file with tailwind directives and custom classes

You can put some background on credits if you want to make them more visible.


[Fix #677] Atomic produce and consume actions

Many thanks for submitting your Pull Request :heart:!
Please specify parts of this PR update:

[X] Specification
[X] Schema
[X] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

Discussion or Issue link:

Fixes https://github.com/serverlessworkflow/specification/issues/677
What this PR does / why we need it:

Separated produceEventRef from consumeEventRef, removing eventRef, and promoting both to first class actions.
Remove kind property from event definition. Therefore the same event definition can be both used for consumption and publishing. if needed.
Special notes for reviewers:
I have couple of doubts about naming notation that I reflected as comments.
Additional information:

Great work, thank you!
Could I please ask you to update the roadmap.md to include the changes brought by this PR?

This is already there in the roadmap for 0.9

This is already there in the roadmap for 0.9

@fjtirado You are right! Sorry about that!

Merging this PR as per contribution rules: it has been out for more than 3 weeks already.


Can't show FluentDialog unless initially shown

🐛 Bug Report
I cannot seem to add a FluentDialog that is not initially visible but can be shown later on.
💻 Repro or Code Sample
<FluentDialog @ref="MyDialog" Hidden="true">

Dialog referenced from object

</FluentDialog>

<Menu>
    <FluentMenu>
        <FluentMenuItem @onclick=ShowDialog>Show dialog…</FluentMenuItem>
    </FluentMenu>
</Menu>

@code {
    FluentDialog MyDialog;

    private void ShowDialog()
    {
        MyDialog.Show();
    }

    private void HideDialog()
    {
        MyDialog.Hide();
    }
}

🤔 Expected Behavior

The dialog should not initially show
When clicking the menu item, it should show

😯 Current Behavior

The dialog does not initially show
However, when clicking the menu item, it doesn't show either

If I remove Hidden="true", the menu item does work, but (obviously), the dialog also immediately shows.
🔦 Context
I need a dialog that shows up once I click a menu item. It should not show up any sooner than that.
🌍 Your Environment

OS & Device: macOS 12.6 on MacBook Pro (14-inch, 2021) (ARM64)
Browser: Apple Safari Version 16.0 (176<IP_ADDRESS>.10, 17614)
.NET: 6.0.10
FAST: 1.5.3

OK, if I instead bind the Hidden property myself, it works:
<FluentDialog @ref="MyDialog" Hidden="@MyDialogIsHidden">

Dialog referenced from object

    <button @onclick=HideDialog>Close Me</button>
</FluentDialog>

<FluentMenu>
    <FluentMenuItem @onclick=ShowDialog>Show dialog…</FluentMenuItem>
</FluentMenu>

@code {
    FluentDialog MyDialog;
    bool MyDialogIsHidden = true;

    private void ShowDialog()
    {
        MyDialogIsHidden = false;
    }

    private void HideDialog()
    {
        MyDialogIsHidden = true;
    }
}

But I don't think the Show()/Hide() methods quite work as intended.

This is fixed in V2 (R1 is out now)


[incubator/fluentd-cloudwatch] Update Daemonset apiVersion to apps/v1

Signed-off-by: Richard Laub<EMAIL_ADDRESS>
What this PR does / why we need it:
While deploying the current chart using pulumi I got the following warning:
warning: extensions/v1beta1/DaemonSet is not supported by Kubernetes 1.16+ clusters. Use apps/v1/DaemonSet instead.

Since 1.16 is around the corner I figured I would update the apiVersion of our daemonsets.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@jmcarp @icereval @kenden this is my first PR to this repo. Does this look ok to you?

/ok-to-test
/lgtm


[stable/weave-scope] Add readOnly flag

What this PR does / why we need it:
This new flag allows to disable the controls completely. Once readOnly is set to true, it will add --probe.no-controls to the list of args for weave-scope-agent.
Which issue this PR fixes
No Issue.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @sameersbn

/ok-to-test
/assign

/lgtm


feat: improve configuration builder + return end user data instead of…

… low level type

Looks good! Thank you!


Method not found: '!!0 Microsoft.Extensions.AI.IChatClient.GetService(System.Object)'

Hi,
I'm using the Microsoft.Extensions.AI.IChatClient interface in my code (to be able to call ChatGPT, Anthropic, Mistral etc.).
It worked with Mistral.SDK v1.3.1, but after upgrading to v2.0.0 I get this error at runtime :
System.MissingMethodException: Method not found: '!!0 Microsoft.Extensions.AI.IChatClient.GetService(System.Object)'.
Any idea why?
.NET 9.0

Hi, I just added function calling, and was seeing that with Semantic Kernel in testing but thought I had baked in a workaround specifically for this. Can you provide a stripped down example that'll help guide me a bit and narrow it down? That would help a ton, thank you.

Hi,
Here is a sample project that reproduces the exception.
Bug.zip
Actually, the error only occurs when I reference the other packages (Anthropic.SDK and Betalgo.Ranul.OpenAI), so it might be a conflict between the three packages. I have also contacted the owners of the other packages, as the issue may not be with your package.

Thanks for the sample.  I actually wrote Anthropic.SDK as well.  I noted when I removed Betalgo.Ranul.OpenAI, the app seems to run fine.  This does appear to be some sort of version conflict and I note that there are some dependency mismatches between my libraries and theirs, but not related to IChatClient...it just has a LOT of dependencies so it's a little tough to tell where exactly the conflict is coming from.  I'll dig a bit further, but you may want to reach out to the author, this isn't related to function calling I don't think, especially since you're not using it in this sample and the issue I saw was only cropping up around that, and fairly certain I resolved it.

ok, thanks for your time! ;)

If you're in a bind: Microsoft.Extensions.AI.OpenAI seems to provide a drop-in replacement if you're using the IChatClient interface and doesn't produce the error, I just gave it a shot.  Hope it helps and you're more than welcome!


Moves login, lang, profile buttons into left menu for mobile

Description
Fixes #1413
Notes (out of scope, known isues, hints for reviewing code, ...)  (optional)
...
Test cases

[ ] Case 1:

Given I am the temp user
And I turn on mobile mode or narrow width
When I go to this page
Then I see no login and lang buttons
And I click on main menu button
Then I see login and lang buttons

[ ] Case 2:

Given I am the usual user
And I turn on mobile mode or narrow width
When I go to this page
Then I see no profile button
And I click on main menu button
Then I see profile button

[ ] Case 2:

Given I am the usual user
And I turn on mobile mode or narrow width
When I go to this page
Then I see no profile button
And I click on main menu button
Then I see profile button without login
Then I click on profile button
Then I see login in header of the menu

Could you fit closer to the design? So:

"profile" and "log out" rows with the same height as the login row.
left align "profile" and "logout", right align the icons


dynamically items?

how can add dynamically items with json records retrive from database?

Hi,
For the moment we can't add or refresh items, a workaround could be building the timeline only once json records are retrieved, but I think you won't be able to update it later... (could it work for you?)
It's a very important point so I'll work on that after horizontal mode. Thx!

Hi, any feedback on this?

Hi @jcr4e, thanks for the follow up, I've been quite absent last weeks...
Just added this on the TODO list, will try to work on it this WE!

@jcr4e @asadghazanfary just added a first try of dynamic load, if you have some time to check it and give me some feedback (some limitations you faced, issue, ...) don't hesitate.
Cheers!

This is how I do it:
public function displayTree() {
... your SQL query here.. table must have id and parent_id fields
$this->makeTree($nodes, 0, $tree);
echo json_encode($tree);
}
private function makeTree( $rst, $level, &$tree ) {
for ( $i=0, $n=count($rst); $i < $n; $i++ ) {
if ( $rst[$i]['parent_id'] == $level ) {
$branch = array(
'name' => $rst[$i]['step'],
'id' => $rst[$i]['id'],
'step_id' => $rst[$i]['step_id'],
'parent_id' => $rst[$i]['parent_id'],
'children' => array()
);
$this->makeTree( $rst, $rst[$i]['id'], $branch['children'] );
$tree[] = $branch;
}
}
}
you can then run displayTree using AJAX and load the result on the tree. check sample code below:
                    $('#treeview').tree({
                        data: result,
                        autoOpen: true,
                        dragAndDrop: true,
                        closedIcon: '+',
                        openedIcon: '–'
                    });

hope this helps.

Ok so normally you should be able to use it that way. Tell me if I missed something ;)


Stroke rounded rect visual artifacts

Vesion: 3.12 beta 2, also 3.11

screenshot taken from example - https://labs.phaser.io/view.html?src=src/game objects/graphics/stroke rounded rectangle.js
These artifacts appear only on Android devices.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Does this happen in Canvas and WebGL, or just one of them? My guess would be it only happens in Canvas.

@photonstorm Nope, it happen on Webgl

Can you test this with the current master version please? I pushed a fix for the arc function today which I'm hoping will solve this too.

@photonstorm CANVAS looks good, but WEBGL still has artifacts

Can you make this happen in any desktop browsers? (as I can't) which makes it close to impossible to fix.

@photonstorm it happens only on Android devices, and actually not on every,

What about the filled rounded rectangles?
And is it just rounded rects? Do other Graphics shapes appear ok?

only stroked rounded rect has artifacts, default rect (filled, stroked) looks fine, the same as filled rounded rect

This is what the function does:
    strokeRoundedRect: function (x, y, width, height, radius)
    {
        if (radius === undefined) { radius = 20; }

        var tl = radius;
        var tr = radius;
        var bl = radius;
        var br = radius;

        if (typeof radius !== 'number')
        {
            tl = GetFastValue(radius, 'tl', 20);
            tr = GetFastValue(radius, 'tr', 20);
            bl = GetFastValue(radius, 'bl', 20);
            br = GetFastValue(radius, 'br', 20);
        }

        this.beginPath();
        this.moveTo(x + tl, y);
        this.lineTo(x + width - tr, y);
        this.arc(x + width - tr, y + tr, tr, -MATH_CONST.TAU, 0);
        this.lineTo(x + width, y + height - br);
        this.arc(x + width - br, y + height - br, br, 0, MATH_CONST.TAU);
        this.lineTo(x + bl, y + height);
        this.arc(x + bl, y + height - bl, bl, MATH_CONST.TAU, Math.PI);
        this.lineTo(x, y + tl);
        this.arc(x + tl, y + tl, tl, -Math.PI, -MATH_CONST.TAU);
        this.strokePath();

        return this;
    },

Which is basically just a bunch of Graphics commands in a sequence. Could you try re-creating the above (so just calling the Graphics commands directly) but add in a closePath before the stroke? I'm thinking it's something like that.

i recreated, but it doesn't help at all.
this.graphics = this.add.graphics();
    const width = 100;
    const height = 50;
    const tl = 5;
    const tr = 5;
    const bl = 5;
    const br = 5;
    const x = window.innerWidth / 2;
    const y = 20;
    this.graphics.beginPath();
    this.graphics.lineStyle(2, 0x00ff00);
    this.graphics.moveTo(x + tl, y);
    this.graphics.lineTo(x + width - tr, y);
    this.graphics.arc(x + width - tr, y + tr, tr, -Math.PI * 0.5, 0);
    this.graphics.lineTo(x + width, y + height - br);
    this.graphics.arc(x + width - br, y + height - br, br, 0, Math.PI * 0.5);
    this.graphics.lineTo(x + bl, y + height);
    this.graphics.arc(x + bl, y + height - bl, bl, Math.PI * 0.5, Math.PI);
    this.graphics.lineTo(x, y + tl);
    this.graphics.arc(x + tl, y + tl, tl, -Math.PI, -Math.PI * 0.5);
    this.graphics.closePath();
    this.graphics.strokePath();

i think we can mark this bug as known issue

How about breaking it down a bit - so rather than the whole rectangle, how about just the first moveTo, lineTo and arc? Do you still see it then? (comment out the rest of the commands) - if not, add the next line in and try again. I'm curious to know at which point you get one of those extra lines to appear. I'd rather solve this than leave it as-is.

moveTo -> lineTo -> arc

moveTo -> lineTo

Hi, i added moveTo after lineTo and then call arc - suddenly that moveTo fixed artifacts  -__-

Does it work if you repeat for all 4 corners? If so we can merge this back in with master.

Could there be a problem with the negative angles? I guess there should not be as angles can be negative, but you never know. Below the angles are changed to be positive.
        this.beginPath();
        this.moveTo(x + tl, y);
        this.lineTo(x + width - tr, y);
        this.arc(x + width - tr, y + tr, tr, 3*MATH_CONST.TAU, 2*Math.PI);
        this.lineTo(x + width, y + height - br);
        this.arc(x + width - br, y + height - br, br, 0, MATH_CONST.TAU);
        this.lineTo(x + bl, y + height);
        this.arc(x + bl, y + height - bl, bl, MATH_CONST.TAU, Math.PI);
        this.lineTo(x, y + tl);
        this.arc(x + tl, y + tl, tl, Math.PI, 3*MATH_CONST.TAU);
        this.strokePath();

@photonstorm yeah it does

@alexeymolchan you could try the non-negative version above, but otherwise please paste the full working combination here so I can merge it into master, as I've never been able to reproduce this, so I need to see what combination gets it working for you.

@photonstorm i already tried with non-negative angles, but it still produce artifacts.
Working combination:
strokeRoundedRect: function (x, y, width, height, radius)
    {
        if (radius === undefined) { radius = 20; }

        var tl = radius;
        var tr = radius;
        var bl = radius;
        var br = radius;

        if (typeof radius !== 'number')
        {
            tl = GetFastValue(radius, 'tl', 20);
            tr = GetFastValue(radius, 'tr', 20);
            bl = GetFastValue(radius, 'bl', 20);
            br = GetFastValue(radius, 'br', 20);
        }

        this.beginPath();
        this.moveTo(x + tl, y);
        this.lineTo(x + width - tr, y);
        this.moveTo(x + width - tr, y);
        this.arc(x + width - tr, y + tr, tr, -MATH_CONST.TAU, 0);
        this.lineTo(x + width, y + height - br);
        this.moveTo(x + width, y + height - br)
        this.arc(x + width - br, y + height - br, br, 0, MATH_CONST.TAU);
        this.lineTo(x + bl, y + height);
        this.moveTo(x + bl, y + height);
        this.arc(x + bl, y + height - bl, bl, MATH_CONST.TAU, Math.PI);
        this.lineTo(x, y + tl);
        this.moveTo(x, y + tl);
        this.arc(x + tl, y + tl, tl, -Math.PI, -MATH_CONST.TAU);
        this.strokePath();

        return this;
    },

Just to say that I can re-create this locally now, so I'll look into it further. While the above did work for me I need to understand why it's happening at all, so will carry on debugging it.

Thank you for submitting this issue (yes, 2 years ago!). We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that, although of course given the age of this issue, isn't expected at all.

I'm still seeing this issue in Phaser 3.55.2 on older Android devices (Android 5 or so) as well as Kindle devices - even new Kindles, such as the Kindle 7 running FireOS 7, however FireOS is based on an older version of Android, so I think that is why. The issue doesn't appear on Windows, Mac, or iOS, seems to be just older Android devices. The Browserstack device I test with is a Samsung Galaxy S6 running Android 5, and we have a Kindle 7 running FireOS 7 that the issues appears in as well.

I have the same issue with strokeRoundedRect() too.
Phaser 3.55.2 on Android 10 (Honor 10i).

I wrote my own func with arc. Now it works without extra lines.
This code works good:
function drawRoundedRect(x, y, width, height, angle, object){ 	let graph = object.add.graphics(); 	graph.beginPath(); 	graph.lineStyle(4, 0x00ff00); 	graph.moveTo(x, y); 	graph.arc(x + angle,y + height,angle,Phaser.Math.DegToRad(180),Phaser.Math.DegToRad(90),true); 	graph.arc(x+angle+width,y+height,angle,Phaser.Math.DegToRad(90),Phaser.Math.DegToRad(0),true); 	graph.arc(x+angle+width,y,angle,Phaser.Math.DegToRad(0),Phaser.Math.DegToRad(270),true); 	graph.arc(x+angle,y,angle,Phaser.Math.DegToRad(270),Phaser.Math.DegToRad(180),true); 	graph.strokePath(); 	return graph; }
Hope that strokeRoundedRect() will be fixed finally :)

@photonstorm I have the same issue with strokeRoundedRect() too. Phaser 3.55.2 on Android 10 (Honor 10i). Any browsers on this device. Phaser was installed as npm package.

I wrote my own func with arc. Now it works without extra lines.
This code works good:
function drawRoundedRect(x, y, width, height, angle, object){
	let graph = object.add.graphics();
	graph.beginPath();
	graph.lineStyle(4, 0x00ff00);
	graph.moveTo(x, y);
	graph.arc(x + angle,y + height,angle,Phaser.Math.DegToRad(180),Phaser.Math.DegToRad(90),true);
	graph.arc(x+angle+width,y+height,angle,Phaser.Math.DegToRad(90),Phaser.Math.DegToRad(0),true);
	graph.arc(x+angle+width,y,angle,Phaser.Math.DegToRad(0),Phaser.Math.DegToRad(270),true);
	graph.arc(x+angle,y,angle,Phaser.Math.DegToRad(270),Phaser.Math.DegToRad(180),true);
	graph.strokePath();
	return graph;
}

Hope that strokeRoundedRect() will be fixed finally :)

I changed your code a little, and I managed to fix the box better in my case, maybe it's a different version, and it worked fine for me:
  function drawRoundedRect(x, y, width, height, object) {
    let graph = object.add.graphics();
    graph.beginPath();
    graph.lineStyle(4, 0x00ff00);
    graph.moveTo(x, y+angle);
    graph.arc(x + angle, y + height-angle,angle,Phaser.Math.DegToRad(180),Phaser.Math.DegToRad(90),true);
    graph.arc(x-angle+width,y+height-angle,angle,Phaser.Math.DegToRad(90),Phaser.Math.DegToRad(0),true);
    graph.arc(x-angle+width,y+angle,angle,Phaser.Math.DegToRad(0),Phaser.Math.DegToRad(270),true);
    graph.arc(x+angle,y+angle,angle,Phaser.Math.DegToRad(270),Phaser.Math.DegToRad(180),true);
    graph.strokePath();

  }


File Change Detection Should be less sensitive; match to saved file

VS Code thinks the file has changed when anything has been typed, regardless of whether the change has been reversed. So it thinks it needs to be saved (and pops up the "Do you want to save changes" message), even if the file is technically the same as the last-saved version. In contrast, the Atom editor (which I'm trying to migrate to VS Code from) actively diffs file so that no matter what you've typed, if you undo back to the state of the last save, it detects that no material changes were made, and you can just close the file without any further interruption.

VSCode Version: 1.16.1
OS Version: OS X El Capitan (10.11.6)

Steps to Reproduce:

In any file, type a character.
Delete the character you just typed, returning the file to its previous state.
In the explorer and the file's tab, the changed dot is still present, and trying to close the file gives the "Do you want to save changes" box, even though the file hasn't really changed.

Reproduces without extensions: Yes

https://github.com/Microsoft/vscode/issues/919


TST: _lib: disable refcounting tests on PyPy

Because PyPy uses a different GC approach, such tests are not meaningful
when running on it.

@pv since you seem to be looking into this, how does the test suite fare on PyPy?

@person142: it's almost 100% everything working with pypy3-5.10.1 + current numpy master. The only bigger thing is that scipy.signal has some refcounting bugs with object arrays that cause crashes (PR incoming).


Keyboard Interrupt not working as intended.

Currently, a keyboard interrupt is only detected at the start of a new trial. It would be nice to have the interrupt occur at ANY point throughout the program

It may be better to leave the handling of this to the profiler then? At least for now


Add ACT

College's Official Website: https://www.act.ac.th/
Course Details (1+ years): https://www.act.ac.th/program/ML_DS.html
College's Physical Building Location: https://goo.gl/maps/863fDAd6GtN9tcfQ9
College's Official Facebook Page: https://www.facebook.com/ACT2504
WHOIS Domain Details:

@Achnan First stage of verification passed, official domain is act.ac.th as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Achnan Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add missing check before calling toString

This PR fixes #25409
The PR makes sure the resource property on the activeEditor contains a truthy value before toString is invoked.

Does this change what a user will see (aside from the console error) when they expand OE node before extension activation?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 41.741%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
30722

Relevant Lines:
68884

💛  - Coveralls

@caohai, it doesn't change what the user will see.


formValueSelector values not updated as fields change

bug report
What is the current behavior?
In both 6.8.0 and 6.7.0, the values returned by a formValueSelector do not change  as field values are changed.
What is the expected behavior?
Following the formValueSelector example usage should result in selector values that get updated as their source field values change.
For example, this code:
GameSettingsForm = reduxForm({
form: 'gamesettingsForm'
})(GameSettingsForm);
const selector = formValueSelector('gamesettingsForm');
GameSettingsForm = connect(
    state => {
        let openDateValue = selector(state, 'openDate');
        let dueDateValue = selector(state, 'dueDate');
        return {
            openDateValue,
            dueDateValue
        }
    }
)(GameSettingsForm);

export default GameSettingsForm;

results in openDateValue and dueDateValue props that always match the initial values for openDate and dueDate fields.
I need access to the current values of the openDate and dueDate fields and expected the selector would provide the in the openDateValue and dueDateValue props.

You are probably doing something wrong. Can you show us a working example?
See, this works: https://codesandbox.io/s/r2yR3O0E

Your example works nicely. However, my form component is a React.Component and my code does not have a similar structure.
What did finally work for me is to invoke getFormValues in the form container's mapStateToProps function and return the resulting formValues as a property. I got the idea from the solution to this question: https://stackoverflow.com/questions/39878035/using-redux-formv-6-0-5-getformvalues-does-not-add-values-to-props
// component
class GameSettingsForm extends React.Component {
    render() {
        const openDate = this.props.formValues.openDate;
		...
    }
}

GameSettingsForm.propTypes = {
    initialValues: PropTypes.object.isRequired,
    formValues: PropTypes.object.isRequired,
};

export default reduxForm({
    form: 'gamesettingsForm'
})(GameSettingsForm);

// container
function mapStateToProps(state, ownProps) {
    const initialValues = {
        openDate: new Date()
    };
    const formValues = getFormValues("gamesettingsForm")(state) || {};
    return {
        initialValues,
        formValues,
    };
}

const GameSettingsFormContainer = connect(
    mapStateToProps,
    mapDispatchToProps
)(GameSettingsForm);

export default withRouter(GameSettingsFormContainer);

I tried doing something similar using formValueSelector, but it's value becomes undefined after the field value is changed. So, I cannot show you a working example of that.

I would suggest providing a fully runnable not working example.

Hmm, my fully runnable example works: https://codesandbox.io/s/DkWGnrG4y
so the fault was in my code rather than redux-form.


Possible to accidentally erase paper contents when saving

Description
When saving your edits on a piece of paper, if you hit ctrl+enter or hit the UI button multiple times within the same second or so, it can cause your paper to be saved blank, thus discarding not only your recent edits but everything that was on it. Obviously this can be really problematic for a player who accidentally does this (as I have).
Reproduction

write on paper
very quickly double-click or double-press save/ctrl+enter
the paper will save as "This page intentionally left blank"
cry about the loss of your magnum opus

This just happened to me again while working on something and it was very fustrating...

I'm pretty sure this has happened to me too

Paper had some changes recently (see #32418) so I'm going to check and make sure this bug still happens

Should be fixed in the PR above.


Cleaning test

Trying to use pytest paramterize when needed, adding comment and trying to create better tests (not just basic integration)

Failing because Hdf5_to_bids does not put image fil in sub-xx/anat but just in sub_xxx. Will fix in another pr and retest

Pull Request Test Coverage Report for Build 225971543

0 of 22 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 225971370:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


feat: adapt to did core spec

closes #155
added support for blockchainAccountId
This PR introduces verification capabilities for DID documents that use blockchainAccountId (with eip155)
Support for ethereumAddress is maintained for backward compatibility.
added support for proofPurpose
Verification can now be done using a proofPurpose, which filters the verificationMethod based on the entries in the respective section of the DIDDocument.
Examples:
verifyJWT( token, { proofPurpose: 'authentication', ... })
verifyJWT( token, { proofPurpose: 'assertionMethod', ... })
verifyJWT( token, { proofPurpose: 'capabilityDelegation', ... })
// ...

// legacy API for 'authentication' purpose, maintained for backward compatibility.
verifyJWT( token, { auth: true, ... })`

For backward compatibility the verifier supports legacy DID resolvers that do not return DIDResolutionResult and also DID Documents that do not list assertionMethod at all.
This support will be removed once the ecosystem is more uniform with compliance.

@oed now I think it's ready. please take another look

LGTM from a spec perspective although I think verificationRelationship is more appropriate than proofPurpose.

that's true, but it is so very verbose 😅


feat(serviceinfo,tes): add models

This PR contains all the autogenerated models created by running the script in the #27
This PR doesn't contain any manually written code, and everything is autogenerated

Summary by Sourcery
Add autogenerated models for the Task Execution Service (TES) API and GA4GH service-info API. Introduce scripts to automate the generation of these models from OpenAPI specifications.
New Features:

Add autogenerated models for the Task Execution Service (TES) API, including TesServiceInfo, TesTask, TesExecutor, TesResources, TesInput, TesOutput, TesState, TesTaskLog, TesExecutorLog, TesOutputFileLog, and TesCreateTaskResponse.
Add autogenerated models for the GA4GH service-info API, including Service, ServiceOrganization, and ServiceType.

Enhancements:

Add a script (utils/script.sh) to automate the generation of models from OpenAPI specifications using the OpenAPI Generator CLI.
Add a build script (utils/build-models.sh) to generate models for multiple APIs by calling the main script with appropriate parameters.

Ok, I will move transport and configuration to utils folder, and will move tes and serviceinfo to a clients folder in further PRs.
@uniqueg, I have merged #30, and have moved to a utils folder in this PR, and moved the models to clients, since the comment regarding changing folders is mentioned here. I think this PR is ready to be merged.


rollup: add JSDoc, spacesToTabs fix, improved debugging builds

I would like to suggest some changes I figured over time:

spacesToTabs(enable) requires true as argument, or it simply does nothing
debugger statements were stripped by the strip plugin, even in debug mode, which annoys and surprises me every time again. Since no function is stripped in Debug mode, I just enable the strip plugin for the other modes.
The regex in spacesToTabs never possibly could work, hence no output file actually contained tabs. Even if regex would work, it would still be a dangerous regex, that doesn't respect "line start spaces" only, so it would e.g. even replace spaces inside a string.
I added JSDoc with IntelliSense (the jsconfig.json changes are required for this - as far I tested it doesn't affect anything else  so far)
Plugins needs to specify a name, it is a mandatory property, otherwise Babel error messages cannot point out the name of the plugin that encountered an exception.
I would like to keep comments in my debug builds, they always help me to understand issues while debugging

I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

This looks good. But can you please ensure there's a newline before every JSDoc block please?
For comments in the debug build, didn't we make it so source maps are now available with engine builds now?

For comments in the debug build, didn't we make it so source maps are now available with engine builds now?

I just checked if sourcemaps still work and they cause no problems with comments:

Basically just npm run build:sourcemaps and then clicking on pc.Entity.prototype.translate in devtools.
Were you wondering about this, if it still works?

No, what I meant was, why keep the comments in the debug build when you can just use source maps and see the original comments?

No, what I meant was, why keep the comments in the debug build when you can just use source maps and see the original comments?

Oh, that's what you mean, I don't really like sourcemaps for debugging. If you change/fix some code directly in the build then you won't even see your changes through the "sourcemap lense".
So what I can do right now is to just make a small fix to rollup.config.mjs to ignore inline debug sourcemaps and then I am free to edit the debug output files again, without sourcemaps messing around with my changes.
I realize my work flow is maybe a bit strange, but I just try to not rebuild everything all the time, since it takes quite some time.
Then the argument is, that I could also use the watch-mode, but even though it's quicker, it still takes like three seconds - for updating one file change. And then always these background tasks you need to deal with (which block my default VSCode shell aswell... ending up creating multiple ones that I need to cognize), I feel way too impatient for all that.

@mvaligursky LGTM.

Thanks for an awesome contribution @kungfooman !


Revert "Promote SPI/RemoteSecret from staging to prod (#2192)"

This reverts commit f04d15aa091939bac7f97c667bdafc8198c37f21.

/lgtm


Use properly aligned array during xtl testing

Should fix #812

I wrote 'should fix' because I wasn't able to reproduce. @emmenlau  can you test that patch?

A first test shows that this fixes the problem on Ubuntu Linux 20.04 with clang 14.0.6. I'll test also in our CI on Windows and macOS next...

I can confirm the fix works on all platforms! Nice!

Thanks for checking!


Error in cluster

I see this in my logs regularly:
api_1             | [2020-05-31T22:02:33.704741-04:00] worker-17.DEBUG: Accept <IP_ADDRESS>:41080 on <IP_ADDRESS>:1337 #413 [] []
api_1             | [2020-05-31T22:02:33.708023-04:00] worker-17.DEBUG: GET http://local.pl.com:1337/secret/ HTTP/1.1 @ <IP_ADDRESS>:41080 [] []
api_1             | [2020-05-31T22:02:34.612730-04:00] cluster-6.ERROR: STDERR from PID 17: src/unix/core.c:877: uv__io_stop: Assertion `loop->watchers[w->fd] == w' failed.

I'm using lib-uv in docker from php:7.4-cli image. When I disable the uv-extension the problem is gone.
For reference, here is my Dockerfile:
FROM php:7.4-cli

RUN apt-get update \
    && apt-get install libuv1-dev libpq-dev -y \
    && pecl install uv-beta raphf \
    && docker-php-ext-enable uv raphf \
    # we need to install pq in extra step, because raphf needs to be there already
    && pecl install pq \
    && docker-php-ext-enable pq

RUN echo date.timezone = "America/New_York" >  /usr/local/etc/php/conf.d/timezone.ini \
    && echo expose_php = Off > /usr/local/etc/php/conf.d/expose_php.ini

ADD . /var/www

WORKDIR /var/www

ENTRYPOINT ./vendor/bin/cluster ./bin/server.php -- --port 1337

Does it only happen with cluster or also without it?

I encountered this issue yesterday, didn't change anything in the code and cannot reproduce it anymore right now. So I can't tell, sorry.

I can tell now, this problem also happens when using the server without the cluster. But it only happens on first start of the docker container. When I restart the container, everything works fine. It seems to be this way pretty consistent.

@kelunik any idea why I need to restart the docker containers before it works?

@prolic This seems to be a duplicate of https://github.com/amphp/amp/issues/292 actually, so I'm closing this.


chore: reduce duplicated code

Removed code duplicate from oracle
Update version of price-feeder to #732

Codecov Report

Merging #258 (1aca628) into main (5287352) will decrease coverage by 1.59%.
The diff coverage is 44.08%.

@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
- Coverage   60.68%   59.09%   -1.60%
==========================================
  Files          17       17
  Lines        1989     1985       -4
==========================================
- Hits         1207     1173      -34
- Misses        685      712      +27
- Partials       97      100       +3

Impacted Files
Coverage Δ

orchestrator/cosmos/broadcast.go
93.20% <ø> (ø)

orchestrator/eth_event_watcher.go
78.40% <ø> (ø)

...rchestrator/ethereum/gravity/message_signatures.go
80.95% <ø> (ø)

orchestrator/ethereum/gravity/peggy_contract.go
69.23% <ø> (ø)

orchestrator/ethereum/gravity/submit_batch.go
85.43% <ø> (ø)

orchestrator/ethereum/gravity/valset_update.go
77.92% <ø> (ø)

orchestrator/main_loops.go
7.88% <0.00%> (-0.28%)
:arrow_down:

orchestrator/oracle_resync.go
39.72% <ø> (ø)

orchestrator/relayer/find_latest_valset.go
72.59% <ø> (ø)

orchestrator/relayer/main_loop.go
0.00% <ø> (ø)

... and 9 more


Include database name in "schemas" jinja on-run-end variable

Describe the feature
I would like the "schemas" jinja variable, available in the on-run-end configuration, to include the database name (e.g. [my_database.schema_1, my_database2.schema2])
Additional context
Our project creates schemas in multiple databases
Who will this benefit?
Folks that want to automate granting permissions to schemas created by DBT, where their project uses multiple databases

Hey @dmateusp, check out database_schemas!

Thanks! I missed it in the docs :(


[Refactor] License header need to be modified

Currently, the license header is Copyright 2019 Xlab, when the year changes, the header need to be changed.
We can change it into Copyright 2019 - present Xlab to avoid the problem.

/self-assign


Cranelift: Misoptimization of imul + ireduce

.clif Test Case
set opt_level=speed
target x86_64

function u0:11(i8) -> i8 system_v {
block0(v0: i8):
    v1 = uextend.i64 v0
    v2 = imul_imm v1, 256
    v3 = ireduce.i8 v2
    return v3
}

Steps to Reproduce

Compile reproducing test case with opt_level=speed.

Expected Results
The function unconditionally returns 0.
Actual Results
The function gets optimized to returning the argument:
function u0:11(i8) -> i8 system_v {
block0(v0: i8):
    return v0
}

Versions and Environment
Cranelift version or commit: 0.105 and 36fb62ca3bea7dd1456f12ead03084e2ceb48cda
Operating system: N/A
Architecture: x86_64
Extra Info
Found by @cbeuw in https://github.com/rust-lang/rustc_codegen_cranelift/issues/1460. Replacing i8 with i32 returns the correct result.

Thanks for the bug report!
Might be related to https://github.com/bytecodealliance/wasmtime/pull/7882 -- cc @elliott

Is this perhaps the wrong reduction? With rustc this code also optimizes to return 0:
#[no_mangle]
pub extern "C" fn a(a: u8) -> u8 {
    let a = u64::from(a);
    let a = a * 256;
    a as u8
}

although I fear I may be missing something subtle by accident

Doing a little bit of debugging, reverting #7882 doesn't fix the problem, but reverting #7719 as well does. Here's the assembly before reverting #7719:
   0:   55                      pushq   %rbp
   1:   48 89 e5                movq    %rsp, %rbp
   4:   48 89 f8                movq    %rdi, %rax
   7:   48 89 ec                movq    %rbp, %rsp
   a:   5d                      popq    %rbp
   b:   c3                      retq

And here's the output after reverting both #7882 and #7719:
   0:   55                      pushq   %rbp
   1:   48 89 e5                movq    %rsp, %rbp
   4:   48 0f b6 c7             movzbq  %dil, %rax
   8:   48 c1 e0 08             shlq    $8, %rax
   c:   48 89 ec                movq    %rbp, %rsp
   f:   5d                      popq    %rbp
  10:   c3                      retq

I think there's an unexpected interaction of rules happening here, because in isolation the rules we have for imul x, 0 and the rules introduced in #7719 for reordering ireduce and extend should be sufficient to rewrite this to moving 0 into %rax.

Actual Results
The function gets optimized to returning the argument:

Aha that's what I was missing, actually reading! Apologies!


Add an index on TxId/TxIn to the utxo indexer

It fixes the perf regression introduced by the future spent PR.

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[x] Commit sequence broadly makes sense and have useful messages
[ ] Important changes are reflected in changelog.d of the affected packages
[ ] Relevant tickets are mentioned in commit messages

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[ ] Targeting main unless this is a cherry-pick backport
[x] Self-reviewed the diff
[x] Useful pull request description
[ ] If relevant, reference the ADR in the PR and reference the PR in the ADR
[x] Reviewer requested

Closed it as we finally decided to go back to two tables.


feat(ui): only display plugins in StepUIPlugin and GlobalUIPlugin that contain step and global schema properties, respectively

As of 85ef214fdeeba0ac655d3b8a88b5bc9f3f53542d:

the plugins displayed in the StepUIPlugin plugin are hardcoded
the plugins displayed in the GlobalUIPlugin plugin are hardcoded

Proposed feature
Only display those plugins in the StepUIPlugin and the GlobalUIPlugin which contain step and global schema properties, respectively.  For example, the routes-controller has no global properties and the device-ui-plugin has neither step nor global properties (no schema at all).

4534fb9830c2a682135bf82327ad619f28bdba94


Haavardm/joyride

Tuning process:

Reset PI so all states have identical transition probabilities (i.e prefer to stay in the same mode)
Increase r from 5 to 6
Decrease lambda to 1e-4 from 1e-3
Increase qCV, qCT (v) and qCV_high to 3, 5 and 50.

We should prefer the CV model over the CV_high model. The QV_high model should only explain certian weird behaviours such as sudden acceleration in the current direction.


Az Cli does not support using AES for encrypt/decrypt when using Managed HSM

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
AKV and MHSM both support the ability to ecrypt/decrypt using asymmetric keys (RSA), but MHSM also supports using AES keys. Using the Az CLI with MHSM I can encrypt/decrypt using RSA, but if I use AES, this raises an error even though MHSM supports AES. The same AES encrypt process works fine when using the Azure SDK and C#.
To Reproduce
TYPE:
az login
az keyvault key encrypt --algorithm oct-hsm --value 1234 --data-type plaintext --name testingtesting123 --hsm-name MHSMDemoInstance
ERROR:
az keyvault key encrypt: 'oct-hsm' is not a valid value for '--algorithm'. Allowed values: RSA-OAEP, RSA-OAEP-256, RSA1_5
Note the algs are ONLY RSA, there's no AES or OCT or OCT-HSM (OCT is AES in this instance)
Expected behavior
The cli should return a JSON file with the ciphertext and no error msg.
Environment summary
Using Azure Portal Shell, PS.
azure-cli is 2.28.0
core is 2.28.0
Additional context
Please note that this process, encrypting with AES, works correctly from C#
using Azure.Security.KeyVault.Keys.Cryptography;
using Azure.Identity;
using Azure.Security.KeyVault.Keys;
using System.Text;
string managedHsmUrl = @"https://xxxxxxxxx.managedhsm.azure.net";
var managedHsmClient = new KeyClient(new Uri(managedHsmUrl), new DefaultAzureCredential());
var octKeyOptions = new CreateOctKeyOptions($"OctKey-{Guid.NewGuid()}") {
KeySize = 256,
};
KeyVaultKey cloudOctKey = managedHsmClient.CreateOctKey(octKeyOptions);
var cryptoClient = new CryptographyClient(cloudOctKey.Id, new DefaultAzureCredential());
byte[] plaintext = Encoding.UTF8.GetBytes("Some data to encrypt");
byte[] aad = Encoding.UTF8.GetBytes("additional authenticated data");
EncryptParameters encryptParams = EncryptParameters.A256GcmParameters(plaintext, aad);
EncryptResult encryptResult = cryptoClient.Encrypt(encryptParams);
DecryptParameters decryptParams = DecryptParameters.A256GcmParameters(
encryptResult.Ciphertext,
encryptResult.Iv,
encryptResult.AuthenticationTag,
encryptResult.AdditionalAuthenticatedData);
DecryptResult decryptResult = cryptoClient.Decrypt(decryptParams);

keyvault

@yonzhan technically, the issue lies with Managed HSM, not Key Vault... same team, different product..

@x509cert for adding label purpose, we only mark the bigger module/family:)


It is unclear how to enable the Gopaddle addon and which flavor it is

Summary
The following command doesn't work:
microk8s enable gopaddle-lite

To enable Gopaddle one needs to use this command instead:
microk8s enable gopaddle

This will enable the lite repository though, see:

https://github.com/canonical/microk8s-community-addons/blob/main/addons/gopaddle/enable#L39

The MicroK8s documentation uses gopaddle-lite as a name.
The Gopaddle documentation for MicroK8s uses gopaddle-lite as a name as well.
What Should Happen Instead?
The experience for this addon should be consistent.
I believe changing the addon name to gopaddle-lite would be a better solution here.
Alternatively the gopaddle could be used but then the documentation and our blog post would need to be updated.
Reproduction Steps
Try to enable the gopaddle-lite addon:
$ microk8s enable gopaddle-lite
Addon gopaddle-lite was not found in any repository

Introspection Report

Can you suggest a fix?

Are you interested in contributing with a fix?

Hi @Gayathri-Bluemeric, do you think you can help us in addressing this? Thank you.


field max_alerts not found in type config.plain

What did you do?
when reload alertmanager.yml with webhook config which has max_alerts set
What did you expect to see?
alcf.Load success, and reload should succeed
What did you see instead? Under which circumstances?
fail to reload and get the error:  field max_alerts not found in type config.plain"

Alertmanager version:
v0.21.0

Alertmanager configuration file:

    global:
      smtp_smarthost: ***
      smtp_from: ***
      smtp_require_tls: true
      smtp_auth_username: ***
      smtp_auth_password: ***
      resolve_timeout: 1h

    receivers:
    - name: alertstorage-receiver
      webhook_configs:
      - url: 'http://alertstorage:2113/webhook'
        send_resolved: false
        max_alerts : 1

    route:
      group_wait: 10s
      group_interval: 5m
      receiver: alertstorage-receiver
      repeat_interval: 3h

It works file in v0.23.0, you also have extra space in your config max_alerts : 1, should be max_alerts: 1


How to recover securitygrouppolicies.vpcresources.k8s.aws CRD in EKS cluster

Hello,
I accidentally deleted the default CRDs securitygrouppolicies.vpcresources.k8s.aws, cninodes.vpcresources.k8s.aws and eniconfigs.crd.k8s.amazonaws.com in an EKS cluster. How can i recover them ?
Thanks!

@bettaps you can re-install securitygrouppolicies and cninodes from https://github.com/aws/amazon-vpc-resource-controller-k8s/tree/master/config/crd/bases. You can reinstall eniconfigs from https://github.com/aws/amazon-vpc-cni-k8s/blob/master/config/master/aws-k8s-cni.yaml#L2-L22
Also EKS platform version updates also check securitygrouppolicies and cninodes installation. If they are missing, they will be installed.


empty users in the list

User Story
'U' users showing in the list

T-shirt size:

User Story
"As a PO of Next, I want the list of users to show correctly so I'm not embarrassed by my app during demos."

Description
LDAP query is catching empty users and ADM users...  why?

Acceptance Criteria/Tests
show the list of people in the no-org view, make sure there are no empty entries or ADM OU showing in the list.

related to #1100 maybe?

No code is needed for this ticket. Our prod parallel test AD instance has been cleaned up to remove any users with no display name field. Also, the scope of the USER_BASE_DN has been limited to not show our service accounts anymore (the ADM_ accounts). Previously on our demo and our staging environment I passed in the USER_BASE_DN to get all users within our prod parallel test AD.
Also this does not relate to #1100 because that was when NEXT was trying to look up users that have not been imported. Names were not showing up here because the AD user did not have a value set in their display name.


Improve tab order in form, especially source tab

Well, ok—it's not good on the accessibility front, but it does solve the standard-browser user problem of being unable to tab through the fields while entering data. I suggest we test & move this PR forward, and open another branch to address it from an accessibility perspective.


Support ENI Idle connection tracking timeout settings

Description
What problem are you trying to solve?
From reading the documentation it seems there is not currently a way to manage the idle connection tracking timeouts on nitro-based instance ENI. https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/security-group-connection-tracking.html#connection-tracking-timeouts
I would like to be able to manage these settings via Karpenter so my nodes can have this configured on launch.
How important is this feature to you?
These settings are important for managing connection tracking limits on high throughput nodes.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Confirming, you would like Karpenter to surface a way to configure the ConnectionTrackingSpecification through the EC2NodeClass.
// From github.com/aws/aws-sdk-go@v1.53.6/service/ec2/api.go:69445

// A security group connection tracking specification request that enables you
// to set the idle timeout for connection tracking on an Elastic network interface.
// For more information, see Connection tracking timeouts (https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/security-group-connection-tracking.html#connection-tracking-timeouts)
// in the Amazon Elastic Compute Cloud User Guide.
type ConnectionTrackingSpecificationRequest struct {
	_ struct{} `type:"structure"`

	// Timeout (in seconds) for idle TCP connections in an established state. Min:
	// 60 seconds. Max: 432000 seconds (5 days). Default: 432000 seconds. Recommended:
	// Less than 432000 seconds.
	TcpEstablishedTimeout *int64 `type:"integer"`

	// Timeout (in seconds) for idle UDP flows classified as streams which have
	// seen more than one request-response transaction. Min: 60 seconds. Max: 180
	// seconds (3 minutes). Default: 180 seconds.
	UdpStreamTimeout *int64 `type:"integer"`

	// Timeout (in seconds) for idle UDP flows that have seen traffic only in a
	// single direction or a single request-response transaction. Min: 30 seconds.
	// Max: 60 seconds. Default: 30 seconds.
	UdpTimeout *int64 `type:"integer"`
}

Are you able to elaborate any more on the use case? IIUC theres a limit to the number of connections a Security Group can track, and once that limit has been exceeded packets from connections that should be tracked are dropped.
I'm not sure support in Karpenter is going to be sufficient for your use case though. Karpenter is only responsible for creating the primary ENI, additional ENIs are going to be created by the VPC CNI. It doesn't look like this is supported there quite yet, though there is an open issue (https://github.com/aws/amazon-vpc-cni-k8s/issues/2677).


Add headless CMS tool

Once we have the tags pretty well defined, we don't have to force folks to write yaml + md
Perhaps something like https://forestry.io/ to lessen friction to authroting content

Forestry Web App available at:
https://app.forestry.io/sites/crpeotnao2c-9g/


poorly aligned agenda in conky

I'm using gcalcli for displaying my agenda in conky. Great fan here!
However alignment in conky is strange (contrary to gcalclis output in terminal.
I'm just using:
gcalcli agenda --nocolor --military now
Comparison of alignment in conky and in terminal can be seen in attachments.
Any solution to this?

Use a fixed width font.  Alignment is done via spaces, so if your font has different sizes for different characters, it won't align correctly.

Oh thanks! That did the trick! (Although it's not so good looking with a Mono font :P )


feat(classnames): updated classnames import for a consistent import name

Closes #799
Summary
The classnames import used multiple names depending on the file (classnames, classNames, cx, cn), so the import names were updated so only one name is used (classnames).
Change List (commits, features, bugs, etc)

Updated classnames import for a consistent import name
Opened issue #1325 to track the progress for a potential ESLint rule that will maintain consistent naming for future imports.

Acceptance Test (how to verify the PR)

In VSCode, do a project wide search for "classnames"
This search should show that each time classnames is imported, it will be imported as "classnames"

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to disable downloading of videos? [QUESTION]

I wanted to disable download ability for users, I.e. Admin post videos, and users can only watch them they should not download that video in anyhow.

Click "Admin" on the top right next to the user button, scroll down to "Let users download Videos" and untick that.

Thanks for your response.


Examples failing for ESP32

https://github.com/espressif/esptool/wiki/ESP32-Boot-Mode-Selection#early-flash-read-error

that wasn't the problem - the examples work on ESP32


fix bug

Fixes a minor deployment bug

Codecov Report

Merging #59 (237ba7b) into main (a875c8c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #59   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          165       165
=========================================
  Hits           165       165

Continue to review full report at Codecov.

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

Codecov Report

Merging #59 (237ba7b) into main (a875c8c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #59   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          165       165
=========================================
  Hits           165       165

Continue to review full report at Codecov.

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


Kiosk: Making delete button disappear when locked

When locking the kiosk, the "add game" button would disappear, but the "delete game" button would still be under user-added games. We want the UI to be consistent, so the delete button should go away as well. Because it is locked the user won't be able to navigate down to the button if they try, but this will make the kiosk seem broken since the button is still visible.
Closes https://github.com/microsoft/pxt-arcade/issues/6015

Will pull these changes into the other PR I have open for the new kiosk build.


$newState and $oldState issues

Example code used:
{
    type: "voiceStateUpdate",
    $if : 'v4',
    channel: '998943588547305542',
    code: `
joined
$onlyIf[$newState[channelID]!=;2]
$onlyIf[$oldState[channelID]==;1]`
}

The unexpected error message or issue:
this code returns: 2, when the user joins the voice channel.

Information:

aoi.js version: 5.5.0
node.js version: 17.1.0
Operating system: linux

the option is channelId
and options are case sensitive

the option is channelId
and options are case sensitive

$channelID return channel where logs have been sent.

the option is channelId
and options are case sensitive

$channelID return channel where logs have been sent.

yes and that's intentional
so not a bug 👍

Resolved.


Does not work

Just tried from android tablet.
Finds devices but only get hourglass on ON/OFF commands.

please, send us a screenshot of:

Ewelink Official Mobile App
Our web app

that way may be we can check and make a research about those devices


Change Office Request > Some titles at task progress columns aren't suitable

Pre-condition:
User logins successfully and is on the W2 homepage
Steps

Click on the "Change Office Request" option at the sidebar
Observe the task progress columns

Expected Result

"Wait PM Approval" should be changed to "PM Approval"
"Destination Office Approval" should be changed to "New Office Approval"

Actual Result
Some titles at task progress columns aren't suitable

Done


Interlock -- Integrate Model Validation From Core-Contracts

This issue is to track any mods to the edgex-go services that may be necessary to integrate core-contract modifications for internal validation per https://github.com/edgexfoundry/go-mod-core-contracts/issues/66.
I expect these will be minor.

Welp, not gonna be minor. Some of these export tests...hooo, doggies....


Offline when wire:poll="#refresh"

Describe the bug
When you use wire:poll="$refresh" to refresh a component and the internet connection goes offline or you server crashes, the component does not update anymore. You still see the last status of the component. Would be nice you can catch this and trigger an 'Offline' error message. Also when you do wire:click when your internet is down, you also don't get a response.
To Reproduce
Steps to reproduce the behavior:
1.
<div wire:poll="$refresh">
    {{ $changing_string }}
</div>

Turn off your internet connection
The $changing_string does not update anymore and you don't see an error message

Expected behavior
Would be nice to be able to catch an incorrect response so you can show an error message.

This could be really useful (at least for my specific case). Maybe a generic network message - or even as  a standard component we could extend?

Issue has been fixed in version v0.3.9
Thanks, great solution!


Outbound connections - centralized articles - Load Balancers & NAT Gateway

This article is the primary results from search engines when searching for how Outbound Connections work in Azure. However this article doesn't link or refer to the 'NAT Gateway' service that can also control Outbound connections.
Suggest either:

Add a reference or link to NAT Gateway - https://docs.microsoft.com/en-us/azure/virtual-network/nat-overview
Consider scrapping this article and instead have an "all-up" outbound connections guide

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

ID: a96406ce-2ffc-cdb3-1234-5154ccd5cc76
Version Independent ID: f840a092-e1bf-16e2-1916-4270cdfbbcdd
Content: SNAT for outbound connections
Content Source: articles/load-balancer/load-balancer-outbound-connections.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

Thanks for the feedback! I have assigned the issue to the content author to investigate this and update the document as appropriate.


Rotation fails on Ubuntu 18.04.1 LTS (Bionic Beaver)

aws-rotate-keys fails on Ubuntu 18.04.1 LTS (Bionic Beaver)  with the following error message:
Using access key <AWS_OLD_ACCESS_KEY> from profile "<AWS_PROFILE>".
Your user arn is: <ARN>

You have 1 access key associated with your user:
- <AWS_OLD_ACCESS_KEY> (Active, created <CREATED_TIMESTAMP>, last used <LAST_USED_TIMESTAMP> for service <SERVICE> in <AWS_REGION>)

Do you want to create a new key and deactivate <AWS_OLD_ACCESS_KEY>? [yN] y
Created access key <AWS_NEW_ACCESS_KEY>.
Failed to replace old access key. Aborting.
Please verify that the file <HOME_DIR>/.aws/credentials is formatted correctly.
Deleted access key <AWS_NEW_ACCESS_KEY>.

According to colleagues, this issue also appeared on MacOS

Can you check your ~/.aws/credentials file and see whether or not the regexes here would match? https://github.com/Fullscreen/aws-rotate-key/blob/d7d551fe440160298880547d6cc6a673c600d01d/main.go#L137-L140
I think an improvement can be made to see if the program can find the current keys in the file first so that this error occurs before even attempting to create a new access key.


don't enclose base_image and parent_images when image is custom base …

…image
Signed-off-by: Robert Cerven<EMAIL_ADDRESS>

we can't enclose custom base image, since that is just placeholder for us, and builder.parent_images has unmodified koji/image-build, so if we enclose it in change_in_df plugin, it will fail with parentimagemissing, because koji/image-build != organization/koji-image-build

Would be nice to have a test case demonstrating it.

added test case for that

rn updated: - Custom base images work with organization defined