Plaster Use Cases

I wanted to start a thread here where folks can see our initial list of use cases.  Propose others for debate and eventually wind up with a set of supported use cases for the initial release.  Here is the set of use cases I have been working off of for the preview.

Create files for the user.
This may be an empty PSM1 file or an about_${ModuleName}.help.txt file with some boiler plate text in it or a module manifest file filled in with information gathered from the user.
Create folders where necessary to provide required (or desired) structure.
For instance, projects targeting VSCode that supply tasks.json file need to put that file in a .vscode directory and if that directory doesn't exist, it needs to be created.  Similarly a module manifest Pester test might go in a Tests folder.
Modify existing files.
Templates are just for initial project scaffolding.  You can use them throughout project (module/dsc resource) development to: Add a new DSC resource or add PSake build support to a pre-existing module. Some additions might require modifications to existing files, like adding a new task to the VSCode tasks.json file.

I know some folks have wanted arbitrary code execution.  This is something we're currently going out of our way to prevent.  We want users who are expecting "scaffolding" i.e. the creation of files & folders, to be able to trust that the template isn't do something else.  In fact, I haven't implemented it yet but I'd like for a user to be able to trust that the template will not modify anything outside of the DestinationPath they've specified.  That said, if there are important use cases we're giving up, I'd like to know.

Hi!
Discussed this with @gaelcolas the other day.  One might imagine a situation where they could read a documented API (or even a config file they create) and dynamically generate a set of functions from those at module creation time.
Not a hugely common use case, but just one example among others of where code execution would be beneficial.
On the risks:

It would seem there are fewer vectors for injection attacks for this tool. Hypothetically, sure, someone might send you an evil template of sorts. That seems like a corner case.
It might be reasonable to offer something to enable or disable execution of scripts, similar to how you can call PowerShell with -NoProfile.  Maybe off by default with -EnableExecution, or on by default with -DisableExecution, or something along those lines

Gotta get to work, but again, love this project, looking forward to seeing it grow!

I think being able to create a folder/file structure based on a template and giving the user access to special variables that auto fill into the files would go a very long way. I even think you should give the user the ability to create their own variables for their own custom style projects.
I have been sick the last few days, so I have not gotten to really play with Plaster, and I'm still reading over everything, so I am a bit behind.

@gerane You've pretty much nailed the current scenario we have implemented in this early preview.  Take a gander at this example Plaster manifest - https://github.com/PowerShell/Plaster/blob/master/Examples/NewModuleTemplate/plasterManifest.xml  It should give you an idea of the possibilities.  And for file/template expansion, have a look at: https://github.com/PowerShell/Plaster/blob/master/Examples/NewModuleTemplate/MITLicense.txt

@RamblingCookieMonster do you really want to mix code generation into your scaffolding?

I have been thinking a little bit about use cases, and trying to get into the mode of thinking outside of the box. A few examples that I see being interesting:

Demo\Presentation Scaffolding
Reports
Forms
SCCM/Package Deployment Scaffolding for things like PSAppDeployToolkit or similar. (Already use something like this in production)
VSCode Editor Commands
Teaching\Helping users learn how to use new concepts. Instead of your typical parameters, you name them as steps and have the user select additional parameters as they learn more about the topic, each step has new guidance on new aspects of the topic. It lets the user gradually layer on new things in a manageable way while getting guidance each step of the way.
chocolatey package scaffolding

I think there were some others that I am forgetting, but I'll try to add those as I remember them.


atom-ide-ui does not reflect error count in status bar

When I've got errors in my Elm file, the errors show up in the diagnostics panel, but the little error count in the bottom left corner always shows 0.
@halohalospecial I saw that you merged support yesterday. Was it supposed to just work with no changes?

@splodingsocks, it should work with no changes :grin:  Do you have both linter and diagnostics (from atom-ide-ui) enabled?  I think you should only have 1 of those enabled.

Huh! It's working now. I probably did have both of those enabled. Thank you!


chore: update document and workfolws

Thank you for contributing to action!
Please add a summary of your change
chore: update document and workfolws
Does your change fix a particular issue?
Fixes #(issue)
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

Codecov Report

Merging #4 (f4c4c99) into main (e1bf386) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   91.50%   91.50%
=======================================
  Files           1        1
  Lines         106      106
=======================================
  Hits           97       97
  Misses          6        6
  Partials        3        3

Flag
Coverage Δ

unittests
91.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


update project sync notifications

added info window popup to project view change event
added logic to use 'Standard' launch mode if not set in project

@guw could you look at the CI config for this; seems like it's stuck waiting on macos-latest build and test results (which have been disabled)


Code contains invalid reference

Following the instructions in Visual Studio 2017. The code on line 2 says "using Microsoft.Data.SqlClient" but the assembly downloaded by NuGet is "System.Data.SqlClient" Changing the assembly path/name caused all the errors and unresolved dependencies to be resolved. Dont know if this assembly got moved between versions.

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

ID: 6f9d2f1e-c439-d629-2d03-9da4eb542cef
Version Independent ID: d6590169-e5b0-4c75-4465-3a72ee199ce5
Content: Use Visual Studio with .NET and C# to query - Azure SQL Database & SQL Managed Instance
Content Source: articles/azure-sql/database/connect-query-dotnet-visual-studio.md
Service: sql-database
Sub-service: development
GitHub Login: @stevestein
Microsoft Alias: sstein

@TexasNeo Thank you for the feedback. We are actively investigating and will get back to you soon.

@TexasNeo From the description of issue, it looks like incorrect package has been downloaded and installed. While searching for the 'Microsoft.Data.SqlClient', the other one also shows up at the top. When we tried to install the correct package, we were able to locate it under references.
Only pre-requisite mentioned is 'When using NuGet 3.x this package requires at least version 3.4.' which should definitely be above 3.4 version even with VS 2017.
Could you please try to remove this assembly and try installing the 'Microsoft.Data.SqlClient' once again and let us know if you still face the similar issue?

Thank you, that fixed it.
On Mon, Aug 17, 2020 at 3:11 AM AnuragSharma-MSFT<EMAIL_ADDRESS>wrote:

@TexasNeo https://github.com/TexasNeo From the description of issue, it
looks like incorrect package has been downloaded and installed. While
searching for the 'Microsoft.Data.SqlClient', the other one also shows up
at the top. When we tried to install the correct package, we were able to
locate it under references.
Only pre-requisite mentioned is 'When using NuGet 3.x this package
requires at least version 3.4.' which should definitely be above 3.4
version even with VS 2017.
Could you please try to remove this assembly and try installing the
'Microsoft.Data.SqlClient' once again and let us know if you still face the
similar issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/60940#issuecomment-674731873,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHQH4OMMFTD4YABD5RLQP3SBDQ43ANCNFSM4QATMXIQ
.

@TexasNeo Glad it worked out for you.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


[Core] Errors code, remarks and extentions

In relazione agli errori aggiungerei inoltre:
unauthorized_client
The authenticated client is not authorized to use this
authorization grant type.
unsupported_response_type
The authorization server does not support obtaining an
authorization code using this method._
login_required
The Authorization Server requires End-User authentication. This error MAY be returned when the prompt parameter value in the Authentication Request is none, but the Authentication Request cannot be completed without displaying a user interface for End-User authentication._
consent_required
The Authorization Server requires End-User consent. This error MAY be returned when the prompt parameter value in the Authentication Request is none, but the Authentication Request cannot be completed without displaying a user interface for End-User consent._
request_uri_not_supported
The OP does not support use of the request_uri parameter defined in Section 6._
registration_not_supported
The OP does not support use of the registration parameter defined in Section 7.2.1._
invalid_request_uri
The request_uri in the Authorization Request returns an error or contains invalid data._
invalid_request_object
The request parameter contains an invalid Request Object._
Originally posted by @fmarino-ipzs in https://github.com/italia/spid-cie-oidc-docs/pull/77#discussion_r993598280

Closed by https://github.com/italia/spid-cie-oidc-docs/pull/84


Math randering

Cannot align equations

when I typed:
$$
\begin{align}
\sqrt{37} & = \sqrt{\frac{73^2-1}{12^2}} \\
 &= \sqrt{\frac{73^2}{12^2}\cdot\frac{73^2-1}{73^2}} \\
 &= \sqrt{\frac{73^2}{12^2}}\sqrt{\frac{73^2-1}{73^2}} \\
 &= \frac{73}{12}\sqrt{1 - \frac{1}{73^2}} \\
 & \approx \frac{73}{12}\left(1 - \frac{1}{2\cdot73^2}\right)
\end{align}
$$

You can find all supported functions here.
For your case, you need to use aligned.

also about \tag{1}

tag seems to be a new feature. I will update the KaTeX version.

Sure
Thx

Updated. Please install the latest CI build. (the download link can be found at the end of README)

Shipped with v1.5.0

Thank for the nice extension. Very useful!
Just to check if it supports any katex symbol for
N   - Natural Number ($\N$)
Z   - Integer Number ($\Z$)
R   - Real Number ($\R$)
as currently it will show as \N, \R, \Z

😄
just found a shortcut to workaround for it
$\mathbb{N}$ will give the right symbol for Natural Number, and etc


Publish Templates to NuGet?

It seems like there is a template package Avalonia.FuncUI.Template, but I can't find it on NuGet...Is it not there because it does not work yet or?

refs #335 perhaps

Got that, I will try to pack and install them locally before they fully fixed them.

@JaggerJo Well I intend to use this in a xplat project :(


Compilation Errors

Compilation errors due to the absence of the referenced frameworks (Epoch,HTTP..etc)

Did you run pod install?

Follow the instructions at https://github.com/Zewo/Examples/tree/master/Todo


security-release-process: add two new associate members

I will be mentoring two new associate members with the goal of getting
them onboard by September/October.
Luke Hinds works in the Office of the CTO for Red Hat and has a long history helping open source projects handle security incidents. He has served as Project Team Lead of the OpenStack Security Project which included a number of experiences  performing secure code review,  managing embargoed security notes, and proactive scanning in CI gate with bandit.  Further he was Security Manager for the OpenDayLight software defined networking system and managed all embargoed security vulnerabilities and helped to implement various proactive check systems (e.g. Java CI checks). He also set up the OPNFV security group and created their Vulnerability reporting process.
Swamy Shivaganga Nagaraju, works for Microsoft Security Response Centre (MSRC) and since 2012 he has worked on hundreds of security issues and discovered many  issues through variant investigation and bug hunting in components of Windows. Through his vulnerability research work he has also developed fuzzing tools and frameworks. Further, he has worked on various mitigations that have made it into products, such as CFG Export Suppression, Attack Surface Reductions & Dll Planting mitigations.

I am going to lazy consensus merge this on Friday June 28th

/lgtm

/lgtm
/approve


add to a binary search tree

Hi @Faraz1243, please assign this to me

hey! @Faraz1243 could you please describe and assign me this task?

ok

hey! @Faraz1243 could you please describe and assign me this task?

This task is already completed. If you want to contribute, then create issue and mention what needs to be added. I'll add hacktoberfest label and assign it to you


[ES 2] [Pi] Bug: freeze while running at 1080p

after setting my Pi model B to run at 1080p resolution (hdmi_mode=16 in /boot/config.txt) everything goes well until the carousel appears. In fact at the very moment the carousel appears on the screen (right after ES boot up logo) it get stuck with a white bar on the top (1/3rd of the screen). I can do nothing with my controller the UI is completely frozen. I could log in remotely to the Pi but that's all.
Worth mentioning that it works perfect at 720p.
My GPU has 384MB allocated. Nothing specific in the es_log file.

I've been trawling the internet for a fix for this issue, has there been any progress made on this? it's definitely to do with adding more systems or changing them. Only causes white bar after system reboot.

I get the same issue after adding sg-1000 to my list of systems and fds.

ColorPi theme works with as many systems as you need (I have 32 total right now including retropie setup icon). Set to es-theme-clean-look, es-theme-paradox, es-theme-tronkyfran, or zoid = works fine AFTER booting up with color-pi theme (may freeze after exiting some emulators too still...). However, these themes work great unless I reboot or exit some emulators. Seems to affect only multiple background themes (including Simple which has glitched icons and blank backgrounds even when switching from ColorPi unlike the other themes).
This has to be a bug :( I reproduced on multiple RasPi B+'s with RetroPie 3.0 (betas and final) plus Smithers dual boot openelec/retropie dual boot image.
I've tested GPU splits from 256 to 896 with no success.


Refactor -> Convert parameters to destructured object, error "Could not apply refactoring"

TypeScript Version:  3.8.3 (vscode 1.44.2)
Search Terms:
Could not apply refactoring, Convert parameters to destructured object
Code
function test(x: number, y: number) { }
type T = ReturnType<typeof test>;
test(5, 1);

Try to use Refactor -> convert parameters to destructured object on test.
Expected behavior:
Refactors test to use a destructured object as its argument.
Actual behavior:
Error, Could not apply refactoring
NOTE:
The error is triggered by typeof test. I'm not sure if this problem is even tractable (considering the type operations on typeof test might pull apart the arguments, and so could be impossible to update), however in this specific case I only use the ReturnType, so the conversion is simple.
Perhaps there should just be a warning if typeof is used on the function, or a more specific error with a message describing why the function can't be refactored.

We're separately working on having refactors show why they failed in a user-facing way.
I don't think we'll special case ReturnType<typeof f> for refactoring purposes; we have a very large amount of other higher priority work that needs taking care of first

Hey man it still doesn't work, it just says "Could not apply refactoring"

still doesnt work

Hi from 2024, still doesnt work

Still not working


feat:[CDS-100437]: Doc : Support customer attribute for OIDC with GCP  connector

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67210a160606981716d75eb7--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67210a296f1f190f1d4e3716--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6721f81306069856eed75f22--harness-developer.netlify.app


Revert to original position on release

Hi!
I have a handle centered with x and y 0.5 i would like them to be centered again on mouseup. Is there a way to do this?
regards Joel

You can use the dragStopCallback callback to capture the mouse up event and then use setValue(0.5, 0.5)


build scss failed

I got this error when i run npm start. I still cannot resolve it after searching on goolge.
ERROR in ./app.scss
Module build failed: C:\Users\Jack\myworkspace\github\material-components-web-codelabs\mdc-101\starter\app.scss:1
exports = module.exports = require("./node_modules/_css-loader@1.0.0@css-loader/lib/css-base.js")(false);
^
TypeError: require(...) is not a function
    at C:\Users\Jack\myworkspace\github\material-components-web-codelabs\mdc-101\starter\app.scss:1:98
    at ContextifyScript.Script.runInContext (vm.js:59:29)
    at ContextifyScript.Script.runInNewContext (vm.js:65:15)
    at Object.extractLoader (C:\Users\Jack\myworkspace\github\material-components-web-codelabs\mdc-101\starter\node_modules\_extract-loader@2.0.1@extract-loader\lib\extractLoader.js:88:12)
 @ multi ./node_modules/_webpack-dev-server@2.11.2@webpack-dev-server/client?http://localhost:8080 ./app.scss

Can please let us know what steps did you follow? What node & npm versions are you using?

The node which i am using is v8.11.1, and the npm is 5.6.0
I followed the steps in the below tutorials.
https://material.io/develop/web/docs/getting-started/
https://codelabs.developers.google.com/codelabs/mdc-101-web/
ps：
I resolved  the problem  by using extract-text-webpack-plugin at last.

Hm. I tried reproducing this issue within the codelabs with the same versions of node and npm, on both Windows and OS X, but still was unable to see this error. The path node_modules/_css-loader<EMAIL_ADDRESS>also seems suspicious to me; the version of css-loader that ends up installed when following the package-lock.json in mdc-101/starter is 0.28.11 (and npm 5+ should honor package-lock).
Anyway, since you were able to resolve the issue, I'm going to close this. Thanks!

Glad to hear you solved the problem!
Closing this issue for now. If you think there's a bug in MDC, feel free to reopen 😀


Use remoteService's hostname instead of referrer IP

In an environment with an mDNS "reflector" or repeater service (such as a Ubiquiti Unifi network), the referrer is not the actual target device but a network device. Therefore, its IP would be e.g. that of a router in the network.
Instead, it should be much safer to simply use the remoteService's hostname and let regular DNS resolution figure out its IP address.
Alternatively, remoteService also has a addresses property which contains the device's IP address(es) but imo using regular DNS to resolve the hostname is the preferred solution, unless it causes issues I have not experienced in my local testing.

You're of course absolutely right, thank you for your fix.
I was initially using the hostname property, but it caused some issues for one of my testers, so I switched to using IP addresses. However, I do thing using the hostname is the right way, so I'll make using the IP address an option while defaulting to using the hostname.


Level name populated in Slotcar RobotState msg

This PR addresses issue 6 for the romih dashboard.
To correctly initialize this level_name when the plugin is loaded, the plugin parses the BuildingMap msg as published by the building_map_server and identifies the level based on proximity of the spawned model to a waypoint on a level. Subsequently, the level_name is inferred from incoming PathRequest msgs which should take care of situations where the robot takes a lift to another level.

Thanks! Setting the level_name in the world file would be ideal. Will work towards that goal!


Update de-DE.coffee

Small changes at Diplomats section.
"Ausrüsten" did not fit ->Wähle (select)
"Schwierigkeitsgrad" -> Schwierigkeit

Coverage decreased (-0.007%) to 56.345% when pulling 8a37159554430b185f9c23ec8814974eb9cdb7c9 on joca16:patch-6 into e05de4fcca6aff346048c17526a524b3009023a1 on codecombat:master.

Thanks. Merged


No more auto starting

See this issue.
This PR changes the default behaviour. Riot will not start the router automatically. So users will have to start it manually but they'll be able to integrate their favourite router easily.

It looks ok to me you can merge it!

Now<EMAIL_ADDRESS>is published :-)


K8SPS-91: Check if backup is enabled to proceed with backup reconciliation.

https://jira.percona.com/browse/K8SPS-91
Backup reconciler now, besides checking if spec.Backup is set at all, checks if it is enabled.

Test name
Status

auto-config
passed

config
passed

init-deploy
passed

monitoring
passed

semi-sync
passed

service-per-pod
passed

scaling
passed

sidecars
passed

users
failed

limits
passed

demand-backup
passed

gr-init-deploy
passed

We run 12 out of 12

commit: https://github.com/percona/percona-server-mysql-operator/pull/132/commits/0e9e16bdc042e6b97b7d02f3751bd06615abc2f4
image: perconalab/percona-server-mysql-operator:PR-132-0e9e16b

@inelpandzic could you please check users test and fix it. I think root of the issue is that the mysql version was updated one week ago and now we can have a diff.

Hey @hors , since this failure is not caused by these changes, I suggest to have a different PR for this fix and I think we can merge this one. wdyt?

Test name
Status

auto-config
passed

init-deploy
passed

demand-backup
passed

gr-init-deploy
passed

monitoring
passed

semi-sync
passed

config
passed

service-per-pod
passed

scaling
passed

sidecars
passed

limits
passed

users
passed

We run 12 out of 12

commit: https://github.com/percona/percona-server-mysql-operator/pull/132/commits/75fa7db84b4367e6e42f0ea05c83fcbaded2ac80
image: perconalab/percona-server-mysql-operator:PR-132-75fa7db


comment how to avoid release bug in pypyi workflow

Issue #, if available:
Added comment about this issue: https://github.community/t/workflow-set-for-on-release-not-triggering-not-showing-up/16286/5
for anybody that releases future autogluon versions to be less confused if the Pypi release action failed to trigger.

Job PR-554-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-554/1/index.html


Feature/apmw 120 permission with ccc

closes #120

We need to rebase this branch to get rid of the conflicts. However, I expect #161 to introduce even more conflicts, so we shoud probably wait for this to be merged.

And by the time I submitted the review, it was even already merged :)

The problem in app/models/user.rb#L4 wouldn't occur if we used has_and_belongs_to_many as (kind of) suggested above. What do you think about this @Tratori @YoungBinKo @piabeeck ?

What I found out about the failing tests: spec/requests/rooms_spec.rb and spec/requests/openingtimes_spec.rb are the two problematic files. Interstingly enough, these are also the only two request tests that weren't touched in this PR. At the same time, these are (in contrast to buildings, locations and people) not included in the permission system.
I have no clue what exactly happens here, and especially not why this seamingly completely unrelated error occurs. However, one very simple fix could possibly be to simply add permissions for rooms and openingtimes as well. What do you think? @YoungBinKo @piabeeck @Tratori

What I found out about the failing tests: spec/requests/rooms_spec.rb and spec/requests/openingtimes_spec.rb are the two problematic files. Interstingly enough, these are also the only two request tests that weren't touched in this PR. At the same time, these are (in contrast to buildings, locations and people) not included in the permission system. I have no clue what exactly happens here, and especially not why this seamingly completely unrelated error occurs. However, one very simple fix could possibly be to simply add permissions for rooms and openingtimes as well. What do you think? @YoungBinKo @piabeeck @Tratori

I think this would be worth a try, will ad this to the ticket

Bug: You can no longer edit your own linked person
Reason: You implemented admin-only people editing permission here: db84f2f928e13a43e3048148bfc2359c48f17d51
Possible solution: Add a habtm relationship between people and users (just as we did between buildings/locations and users), and initialize the auto-generated person with the current user in app/models/user.rb:20

Bug: You can no longer edit your own linked person Reason: You implemented admin-only people editing permission here: db84f2f Possible solution: Add a habtm relationship between people and users (just as we did between buildings/locations and users), and initialize the auto-generated person with the current user in app/models/user.rb:20
What do you think about this @piabeeck @YoungBinKo @Tratori

This 5ff7133 looks pretty good! Should we add now then tests for people? @bewee

Currently, the permission system allows everyone to create/edit/show any openingtime.
We propose to leave this as it is and remove the routes for this (openingtimes, openingtimes/new, etc.) and thus make this impossible for the user.
Introducing a permission system for openingtimes would be too complicated to add in this PR since some kind of synchronization system would be required to sync owners with the corresponding person/room.

@Tratori as soon as you approve, this is ready to be merged


eth --rpccorsdomain "*" -j -v 0 fails

When building remix.  I am using a freshly built geth binary as theeth command.

if you use
geth --rpc --rpcapi 'web3,eth,debug' --rpcport 8545 --rpccorsdomain '*'
together with
http://ethereum.github.io/remix ?

I think the person with the cute dog on the back is referring to the start_node command in the package.json. We might change that to auto-detect if geth or eth is in the path.


[Question] Is it possible to execute an AppImage?

Now that RPG Maker XP games are supported on Ubuntu 16+ Linux by using MKXP-Z library, I wonder if it's possible to execute AppImage files on ArkOS. Should I decrypt the AppImage?
Thanks in advance

I doubt it since ArkOS is based on Ubuntu 19.10 and the base packages haven't been updated since late 2020.


Build out sample waiting room page to end-to-end site

Include landing page, link to shop, redirect to waiting room, direct back to shop for checkout against API.

Done and launch URL provided at end of Samples stack and Getting Started stack.


Fix firstName and lastName render & typing for AD-logged user

Changes that we forgot in pair review: Active Directory log-in didn't render anything for given_name or family_name.

Codecov Report

Merging #1851 (dcdfd59) into develop (6b52b17) will decrease coverage by 6.22%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1851      +/-   ##
===========================================
- Coverage    65.97%   59.75%   -6.22%
===========================================
  Files         1074      114     -960
  Lines        29684     1814   -27870
  Branches      3007      371    -2636
===========================================
- Hits         19583     1084   -18499
+ Misses        9636      663    -8973
+ Partials       465       67     -398

Impacted Files
Coverage Δ

...d/benefit/handler/src/components/header/Header.tsx

frontend/shared/browser-tests/shared.components.ts

...ponents/forms/inputs/SocialSecurityNumberInput.tsx

...nd/kesaseteli/youth/src/pages/already_assigned.tsx

frontend/shared/src/contexts/DialogContext.tsx

...plications/migrations/0009_add_application_user.py

...applications/applicationList/ApplicationList.sc.ts

.../applications/migrations/0014_application_batch.py

...nents/application/steps/step2/Step2Employments.tsx

...ed/audit_log/jobs/quarter_hourly/send_audit_log.py

... and 950 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Master node type is missing, 'master' node type exists in more than one topology element.

Resource definition
{
    version: '8.6.1',
    deploymentTemplateId: 'aws-storage-optimized-v3',
    elasticsearch: {
      autoscale: 'true',
      topologies: [
        {
          id: 'hot_content',
          size: '8g',
          zoneCount: 2,
          autoscaling: {
            maxSize: '15g',
            maxSizeResource: 'memory'
          },
        },
        {
          id: 'warm',
          size: '0g',
          zoneCount: 1
        }
      ],
    }
}

One tie breaker master was automatically provisioned.
Expected Behavior
Changes will still be applied even without a master node definition.
Current Behavior
On apply the error below is thrown.
* api error: clusters.cluster_invalid_plan: Cluster must contain at least a master topology element and a data topology element. 'master' node type is missing,'master' node type exists in more than one topology element (resources.elasticsearch[0].cluster_topology)
* api error: clusters.cluster_invalid_plan: Instance configuration [aws.es.master.c5d] does not allow usage of node types [data,ingest]. You must either change instance configuration or use only allowed node types [master]. (resources.elasticsearch[0].cluster_topology[4].instance_configuration_id)

We are also not able to define/add a master node since we have less than 6 data nodes.
Context
After we have provisioned our deployment, we are no longer able to apply updates to our deployment.
Your Environment
We provision via Pulumi.

Version used: v0.5.1

Might be related
https://github.com/elastic/terraform-provider-ec/issues/343

I tested around with v 0.7.0 and different node counts and found out the following:

Starting from 6 nodes, you need to specify at least one master via terraform.
Below 6 nodes, you must not specify any masters via terraform.
When switching from 5 nodes to 6 nodes with master, terraform apply will fail (a subsequent plan will show no drift related to this change though):

| When applying changes to module.[deployment_1.ec](http://deployment_1.ec/)_deployment.this, provider "provider[\"[registry.terraform.io/elastic/ec](http://registry.terraform.io/elastic/ec)\"]" produced an unexpected new value: .elasticsearch.hot.node_roles: planned set element cty.StringVal("master") does not correlate with any element in actual.

Switching back from 6 nodes and master to only 5 nodes is not possible at all via terraform (it does work via UI though):

* clusters.cluster_invalid_plan: Cluster must contain at least a master topology element and a data topology element. 'master' node type is missing,'master' node type exists in more than one topology element (resources.elasticsearch[0].cluster_topology)

Just from a quick look at the issue - I think we need to change node_roles plan modifier to fix item 3 from that list.


change method of hiding nodes marked for deletion

For link_to_remove_association method when wrapper_class attribute set to any bootstrap 4+ display class jQuery hide() function does not work.
That happens because all bootstrap 4 classes has !important flag for display properties. You can check source bootstrap 4 code here
Against display: flex !important property display: none; becomes useless.

Owww. That feels incredibly ugly, having to do that. It starts of course with the d-* implemented so "harshly"...  I still try to avoid !important as much as possible, so we can still use it for those exceptions where we need it.  But when the framework you use starts injecting it everywhere .... Mmmmmmmm. Let me think this through.

Well, I was thinking and I thought to introduce a callback instead, so any user could do whatever they want.
Because imho your code is pretty specific: only adding css? why not do addClass('...') and removeClass('...'). So I would prefer callbacks.
Well: those callbacks already exist: cocoon:before-remove and cocoon:after-remove.
So you could actually, without having to change any code  write something like:
$(document).ready(function() {
    $('#owner')
      .on("cocoon:after-remove", function(e, removed_item) {
        removed_item.attr('style', 'display: none !important')
      });

or remove the class d-* you are using, before remove. Does that help?


[Bug]: Add sequence transform's dialog's messed up

Apache Hop version?
2.5.0
Java version?
openjdk-11
Operating system
Windows
What happened?

Issue Priority
Priority: 2
Issue Component
Component: Transforms

this is a duplicate for #3029 which has already been fixed

Oh my god I'm sorry @hansva. My bad :-(


Bulk Import of List doesn't work

I was having issues with bulk import, and I finally gave up on my code and used the snippet from the docs:
    collection.import(
        [// array stream with header
            [ 'username', 'password' ], // keys
            [ 'admin', 'hunter2' ], // row 1
            [ 'jcd', 'bionicman' ], // row 2
            [ 'jreyes', 'amigo' ],
            [ 'ghermann', 'zeitgeist' ]
        ]
    )

I get:
ArangoError: expecting a valid JSON array in the request. got: Expecting EOF

I can do the equivalent query using curl without issue, and I've looked at the generated query and I don't see what's different compared to the raw query that curl generates, so I'm going to throw up my hands for now and just pass in an array of objects (which does work).

Same problem for me.

same here
this syntax works fine:
[
{key1: value1, key2: value2}, // document 1
{key1: value1, key2: value2}, // document 2
...
]
this syntax doesn't work at all:
[
['key1', 'key2'], // key names
[value1, value2], // document 1
[value1, value2], // document 2
...
]
i get the error response:
(node:11676) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): ArangoError: expecting a valid JSON array in the request. got: Expecting EOF

I'm currently waiting for clarification of the correct behaviour of the ArangoDB import API. See #496 for details.


Export Tables as CSV

When a markdown table is rendered, offer a button to export to CSV/XLS, etc.
Requirements:

inject a button right after (or top-right corner) to download the table
may intercept the Remark/GFM plugins to enable this

Requested by Vaibhav

Going to look into implementing this today

Closed by @aj47 with #392.


Cannot start example locally on Windows 10x64

Env: windows 10 x64 + DevKit-mingw64-64-4.7.2-20130224-1432-sfx
ruby -v
ruby 2.2.6p396 (2016-11-15 revision 56800) [x64-mingw32]
gem install bundler
Successfully installed bundler-1.14.6
cd pact_broker\example
bundle

Output is
................................................
Bundler could not find compatible versions for gem "activesupport":
  In Gemfile:
    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      trailblazer (~> 0.1.0) x64-mingw32 was resolved to 0.1.0, which depends on
actionpack (>= 3.0.0) x64-mingw32 was resolved to 3.0.0, which depends
on
          activesupport (= 3.0.0) x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      trailblazer (~> 0.1.0) x64-mingw32 was resolved to 0.1.0, which depends on
reform (= 1.2.0.beta1) x64-mingw32 was resolved to 1.2.0.beta1, which
depends on
          activemodel x64-mingw32 was resolved to 5.0.2, which depends on
            activesupport (= 5.0.2) x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
padrino-core (~> 0.12.4) x64-mingw32 was resolved to 0.12.4, which depends
on
        activesupport (>= 3.1) x64-mingw32
Bundler could not find compatible versions for gem "rack":
  In Gemfile:
    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      trailblazer (~> 0.1.0) x64-mingw32 was resolved to 0.1.0, which depends on
actionpack (>= 3.0.0) x64-mingw32 was resolved to 5.0.2, which depends
on
          rack (~> 2.0) x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
padrino-core (~> 0.12.4) x64-mingw32 was resolved to 0.12.4, which depends
on
http_router (~> 0.11.0) x64-mingw32 was resolved to 0.11.2, which
depends on
          rack (>= 1.0.0) x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      rack x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      rack x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      rack x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
      trailblazer (~> 0.1.0) x64-mingw32 was resolved to 0.1.0, which depends on
actionpack (>= 3.0.0) x64-mingw32 was resolved to 5.0.2, which depends
on
rack-test (~> 0.6.3) x64-mingw32 was resolved to 0.6.3, which depends
on
            rack (>= 1.0) x64-mingw32

    pact_broker (~> 1.9.3) x64-mingw32 was resolved to 1.9.3, which depends on
padrino-core (~> 0.12.4) x64-mingw32 was resolved to 0.12.4, which depends
on
        sinatra (~> 1.4.2) x64-mingw32 was resolved to 1.4.8, which depends on
          rack (~> 1.5) x64-mingw32

    thin x64-mingw32 was resolved to 1.7.0, which depends on
      rack (< 3, >= 1) x64-mingw32

Update:
The same happens on Ubuntu 16.04.2 LTS

Reckon this is related to https://github.com/bethesque/pact_broker/issues/78 @tancnle? Did we get rid of Trailblazer?

I can't replicate this on my mac, I'm not sure why.
$ rbenv local 2.2.6
$ gem install bundler
Fetching: bundler-1.14.6.gem (100%)
Successfully installed bundler-1.14.6
Parsing documentation for bundler-1.14.6
Installing ri documentation for bundler-1.14.6
Done installing documentation for bundler after 10 seconds
1 gem installed
$ cd example
$ bundle install
Fetching gem metadata from https://rubygems.org/...........
Fetching version metadata from https://rubygems.org/.
Using minitest 5.10.1
Installing i18n 0.7.0
Installing thread_safe 0.3.5
Installing json 1.8.6 with native extensions
Installing builder 3.2.3
Installing erubis 2.7.0
Installing mini_portile2 2.1.0
Installing rack 1.6.4
Using as-notifications 1.0.1
Using awesome_print 1.7.0
Using blockenspiel 0.5.0
Using diff-lcs 1.3
Using multi_json 1.12.1
Installing daemons 1.2.4
Using find_a_port 1.0.1
Installing tilt 2.0.6
Installing uber 0.0.15
Using multi_xml 0.6.0
Using randexp 0.1.7
Using rspec-support 3.5.0
Using thor 0.19.4
Using webrick 1.3.1
Using redcarpet 3.4.0
Installing tins 1.13.0
Installing eventmachine <IP_ADDRESS> with native extensions
Installing sequel 4.42.1
Using bundler 1.14.6
Installing sqlite3 1.3.12 with native extensions
Installing tzinfo 1.2.2
Using rack-test 0.6.3
Using url_mount 0.2.1
Using rack-protection 1.5.3
Using versionomy 0.5.0
Using webmachine 1.4.0
Using httparty 0.14.0
Using rspec-core 3.5.4
Using rspec-expectations 3.5.0
Using rspec-mocks 3.5.0
Using haml 4.0.7
Installing thin 1.7.0 with native extensions
Installing term-ansicolor 1.4.0
Installing activesupport <IP_ADDRESS>
Using http_router 0.11.2
Installing nokogiri <IP_ADDRESS> with native extensions
Using rspec 3.5.0
Installing sinatra 1.4.7
Installing rails-deprecated_sanitizer 1.0.3
Installing activemodel <IP_ADDRESS>
Installing padrino-support 0.12.4
Installing pact-support 0.6.0
Installing padrino-core 0.12.4
Using pact-mock_service 0.12.1
Using pact 1.10.0
Installing rails-dom-testing 1.0.8
Installing loofah 2.0.3
Installing representable 2.1.8
Installing rails-html-sanitizer 1.0.3
Installing actionview <IP_ADDRESS>
Installing disposable 0.0.9
Installing roar 1.0.4
Installing reform 1.2.6
Installing actionpack <IP_ADDRESS>
Installing trailblazer 0.1.2
Installing pact_broker 1.9.3
Bundle complete! 3 Gemfile dependencies, 64 gems now installed.
Use `bundle show [gemname]` to see where a bundled gem is installed.

What happens if you set the specific versions of the conflicted gems to the ones that got installed on my machine?
gem "activesupport", "<IP_ADDRESS>"
gem "rack", "1.6.4"

@bethesque Thanks! I added gem to the pact_broker/example/Gemfile
source 'https://rubygems.org'

gem "activesupport", "<IP_ADDRESS>"
gem "rack", "1.6.4"
gem "pact_broker", "~>1.9.3"
gem "sqlite3" # Replace with your choice of database driver eg. gem "pg"
gem "thin" # Keep, or replace with your choice of web server

it failed with
checking for sqlite3.h... no
sqlite3.h is missing.

So I had to installed it
sudo apt-get install libsqlite3-dev
sudo gem install sqlite3 -v '1.3.13'
bundle install
....
...
Bundle complete! 5 Gemfile dependencies, 64 gems now installed.

bundle exec rackup -p 8080
Thin web server (v1.7.0 codename Dunder Mifflin)
Maximum connections set to 1024
Listening on localhost:8080, CTRL+C to stop

Now example works fine.
Does it make sense to add these steps to the local example procedure?

I think @tancnle actually fixed it by upgrading the pact_broker version in the gemfile. I've merged in his change, so try doing a git pull on your repo and bundle install and run again.

git pull
git log -1
commit 381b8e501a8d9e6ada13045b7a68f58b126d8c7a
Merge: f35eb3e 6d7653b
Author: Beth Skurrie<EMAIL_ADDRESS>Date:   Tue Mar 28 20:16:40 2017 +1100

    Merge pull request #84 from tancnle/kaizen/example-pact-broker-version-bump

    Bump pact_broker version to 1.14.0 to resolve twisted dependencies.

it works out of the box.
thanks a lot!

Thanks to @tancnle!


http://v4-alpha.getbootstrap.com/about/accessibility returns 404

Hi Guys,
Not sure if this is correct , but the following link returns a 404:
http://v4-alpha.getbootstrap.com/about/accessibility
Thank you, Sacha

Duplicate of #19540; Please close this.

indeed. and fixed already in https://github.com/twbs/bootstrap/pull/18819


Variables with default = null are marked as required

Prerequisites
Put an x into the box that applies:

[x] This issue describes a bug.

Description
In terraform 0.12.x it is possible to use the null type for variables in cases where you want a conditionally omitted variable:

variable "test" {
  description = "example of an optional variable that should be ignored if not assigned"
  default        = null
}

This makes for a variable that by default is NOT required for terraform, and thus should be marked as Required: no in the output from terraform-docs.
Actual Behavior
Variables with default = null are marked as required.

For json output, variables with default = null are treated same as variables without any default value, which are marked as "default": null.
For markdown table output, all variables without default = null are marked with Required: yes
For markdown document output, variables with default = null are treated same as variables without any default value, which are not appended any default value information.

Expected Behavior
Variables with default = null should not be marked as required.

For json output, variables with default = null should be distingushed from variables without any default value.
For markdown table output, all variables without default = null should be marked with Required: no.
For markdown document output, variables with default = null should be marked with Default: null.

Replication Case
Tell us which steps to take to replicate your problem.

Add default = null to a variable
Run terraform-docs md table . or terraform-docs md document .
Examine output

Version
terraform-docs version v0.8.0 linux/amd64 BuildDate: 2020-01-21T16:47:59+01:00

It seems they are not going to proceed with https://github.com/hashicorp/terraform-config-inspect/pull/24, and as such I'm gonna attempt to work around this within this repo. I have a base working fix which I'm gonna open a PR soon (it's kinda hacky though! :man_facepalming:)


Add components and deprecate deployments

As per title.
Fixes #1155
Proposed Changes

Add spec.components
Deprecated spec.deployments

Release Note

knativeeventing.spec.deployments and knativeserving.spec.deployments is deprecated, use knativeeventing.spec.components and knativeserving.spec.components instead.

TODO: change documentation in knative/docs

/cc @houshengbo @matzew

Per my comment here: https://github.com/knative/operator/issues/1155#issuecomment-1271740272
I will say do not deprecate the spec.deployment, at least in the current v1beta1 CRDs.
Once we have a new version of CRDs, we can deprecate the spec.deployment.

cc @houshengbo

I'd say we add the deprecated warning, as a vehicle to communicate that things are changing, via the code

@matzew done!

LGTM

This PR needs to add more test coverage.

/unhold
/lgtm
/approve

/cherry-pick release-1.7

/cherry-pick release-1.6

/cherry-pick release-1.5


register extension

Codecov Report

Merging #228 (0729b15) into master (4a35313) will decrease coverage by 0.06%.
The diff coverage is 91.93%.

@@             Coverage Diff             @@
##            master     #228      +/-   ##
===========================================
- Coverage   100.00%   99.93%   -0.07%
===========================================
  Files           81       81
  Lines         8059     8095      +36
===========================================
+ Hits          8059     8090      +31
- Misses           0        5       +5

Impacted Files
Coverage Δ

fugue/extensions/transformer/__init__.py
100.00% <ø> (ø)

fugue/workflow/workflow.py
98.72% <64.28%> (-1.28%)
:arrow_down:

fugue/__init__.py
100.00% <100.00%> (ø)

fugue/extensions/__init__.py
100.00% <100.00%> (ø)

fugue/extensions/_utils.py
100.00% <100.00%> (ø)

fugue/extensions/creator/__init__.py
100.00% <100.00%> (ø)

fugue/extensions/creator/convert.py
100.00% <100.00%> (ø)

fugue/extensions/outputter/__init__.py
100.00% <100.00%> (ø)

fugue/extensions/outputter/convert.py
100.00% <100.00%> (ø)

fugue/extensions/processor/__init__.py
100.00% <100.00%> (ø)

... and 2 more

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 4a35313...0729b15. Read the comment docs.


Country field is filled by first counry from the list

Current behavior
Country field is filled by first counry from the list and cover placeholder - on Payment section, when user uncheck checkbox to copy data

Expected behavior
Country field should be empty
Steps to reproduce the issue

Add some product to the Cart
Go to checkout
Fill fields in Personal details and Shipping sections
Go to Payment section:
when fields in this section are empty then check Copy address data from shipping or Use my billing data (it is for looged user) and then uncheck it

Environment details
https://test.storefrontcloud.io/
Additional information
This issue is for unlogged and also for logged users

@pkarw Im working on this

Closed by #2430

I verfied this issue and it is still. I cleaned storage data, but problem still occurs

that's because this has not been deployed yet
so I think also about #2439


[Cherry-pick to master] remove "dom" from bf-streaming tsconfig and refactor code

Fixes #1565 in master by cherry-picking #1575

Copy-paste from #1575:
Description
For the 4.7.0 release, botbuilder and bf-streaming shipped with an inadvertent requirement for TypeScript developers to include the "dom" lib in the developer's own tsconfig.json.
This problem arises for users even when they don't take a direct dependency on botframework-streaming. Building a library that uses only botbuilder fails without including "dom". (https://github.com/howdyai/botkit/issues/1894)
Additionally, another side effect of relying on WebSocket and FileReader types and definitions via TypeScript leads us to potential breaking changes in the future. The "dom" types included are also tightly coupled with the version of TypeScript, so relying on these types could allow the code to successfully transpile, but not actually work at runtime.
Specific Changes

Create IBrowserWebSocket and IBrowserFileReader interfaces for WebSocket and FileReader
Add helper methods to create WebSocket and FileReader without relying on "dom" being in the tsconfig.json.

Testing
Unit tests run locally pass.

Second comment:
https://github.com/microsoft/botbuilder-js/pull/1575/commits/f6434298f12d6a67a58575698ceb3740b02b8c92 removes the isBrowser() utility and instead adds helpers that check for the presence of global WebSocket and FileReader to better align with the shipped implementation.
With @ckkashyap's help I tested the code against his branch (ckk/protocoljs) in microsoft/BotFramework-DirectLineJS to verify no breaking changes downstream. Live and recorded tests were passing against this branch.

Pull Request Test Coverage Report for Build 99988

9 of 20 (45.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 84.331%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botframework-streaming/src/webSocket/browserWebSocket.ts
3
14
21.43%

Totals

Change from base Build 99861:
-0.07%

Covered Lines:
8439

Relevant Lines:
9682

💛  - Coveralls


Works on one phone but not on another phone

When I try using in on my main phone Redmi Note 2 with LineageOS, it shows this error

But when I tried with my secondary phone Xperia ZR also with LineageOS, scrcpy is working just fine.

What if you use a lower definition?
scrcpy -m800

The similar problem happens again and again. I recommend to do something for better user experience and reduce the number of the same issues as well (like the case of missing adb in the past).
For example, maybe we can catch exception and try lower definition automatically. After it, give a warning.

Using

scrcpy -m800

still results in the same error.
Just making a wild guess here but is it because some phone do not have/support H264 encoders?

because some phone do not have/support H264 encoders?

Maybe.


SSMS 19.1 is stuck on loading packages

I removed my old instance of SSMS and all associated files. When I try to install the new version it gets stuck on the loading packages page. So I deleted that install package from downloads, restarted my laptop and attempted to retry the install. Now the install goes straight to the loading packages section and stays there for hours.
How do I fix this? I need SSMS for school.

Had to do a system restore to resolve the issue.

Closing issue


Problem in test Make a Person

Challenge Make a Person has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
please look at the following img "https://www.dropbox.com/s/xstx0qqy01smvxd/Capture.JPG?dl=0"
the test says "Object.keys(bob).length should return 6."
but I think it's supposed to be 8, not 6.
because we already have 6 properties and we added 2 which is firstname and lastname in the line :
var bob = new Person('Bob Ross');
My code:

var Person = function(firstAndLast) {
  this.getFirstName = function() {
    return this.firstName;
  };
  this.getLastName = function() {
    return this.lastName;
  };
  this.getFullName = function() {
    return this.firstName + " " + this.lastName;
  };
  this.setFirstName = function(name) {
    this.firstName = name;
  };
  this.setLastName = function(last) {
    this.lastName = last;
  };
  this.setFullName = function(firstLast) {
    this.setFirstName(firstLast.split(" ")[0]);
    this.setLastName(firstLast.split(" ")[1]);
  };
  if (firstAndLast !== undefined)
    this.setFullName(firstAndLast);
};

var bob = new Person('Bob Ross');
bob.getFullName();

This is, infact the catch to the problem. The firstname and lastname variables do not need to be keys, but be what is called in languages like C++ and Java, a private variable.
Therefore, Object.keys(bob).length should return 6 as the keys attached remain 6 only, but firstname and lastname are stored in the object as private variables.
This is evident from the two test cases (cases 3 and 4) that expect bob.firstName and bob.lastName to return undefined

I know what you are saying
But can't understand why my solution is wrong
On Sep 18, 2016 10:01 AM, "Ujjwal Sharma"<EMAIL_ADDRESS>wrote:

This is, infact the catch to the problem. The firstname and lastname
variables do not need to be keys, but be what is called in languages like
C++ and Java, a private variable.
Therefore, Object.keys(bob).length should return 6 as the keys attached
remain 6 only, but firstname and lastname are stored in the object as private
variables.
This is evident from the two test cases (cases 3 and 4) that expect
bob.firstName and bob.lastName to return undefined
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10784#issuecomment-247830068,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMBnvA2hWb7iCrkZbDJZOY0oqg0LgL-pks5qrOHJgaJpZM4J_ufL
.

@Mograbi because you've defined firstName and lastName using this, those variables are accessible outside of the object and become properties of the object. As @ryzokuken has pointed out, the challenge tests imply that firstName and lastName should be private variables. Your solution isn't "wrong" per se but it is incorrect given the requirements for the challenge. This challenge is testing your knowledge of scope and object creation. You can read more about scope here. I hope that makes sense. Happy coding!

var Person = function(firstAndLast) {
this.getFirstName = function() {
return this.firstName;
};
this.getLastName = function() {
return this.lastName;
};
this.getFullName = function() {
return this.firstName + " " + this.lastName;
};
this.setFirstName = function(name) {
this.firstName = name;
};
this.setLastName = function(last) {
this.lastName = last;
};
this.setFullName = function(firstLast) {
this.setFirstName(firstLast.split(" ")[0]);
this.setLastName(firstLast.split(" ")[1]);
};
if (firstAndLast !== undefined)
this.setFullName(firstAndLast);
};
var bob = new Person('Bob Ross');
bob.getFullName();


Is there non-cutlass implementation?

I found is some issues there are discussions to use cutlss. So I am wondering was flashattention implemented in non-cutlass way before? If so, where or which tag I could find it. I am curious to learn that code.

v1 was implemented (mostly) without cutlass. You can see e.g. tag v1.0.9


Timesheet - 21/04/2021 - Andrey Bortnikov

Task 1
Implemented task https://github.com/thenewboston-developers/thenewboston-node/issues/98
Proof of Work

https://github.com/thenewboston-developers/thenewboston-node/issues/98
https://github.com/thenewboston-developers/thenewboston-node/pull/118

Time Spent
6 hours

Task 2
Implemented task https://github.com/thenewboston-developers/thenewboston-node/issues/101
Proof of Work

https://github.com/thenewboston-developers/thenewboston-node/issues/101
https://github.com/thenewboston-developers/thenewboston-node/pull/117

Time Spent
1.5 hours

Task 3
Implemented task https://github.com/thenewboston-developers/thenewboston-node/issues/85
Proof of Work

https://github.com/thenewboston-developers/thenewboston-node/issues/85
https://github.com/thenewboston-developers/thenewboston-node/pull/119

Time Spent
0.5 hours

Total Time Spent
8 hours
Account Number
5781e239ee805141ae7d3d22ca9cd8eaa4c8ecb704dd1d0943226d276e26c886

Approved


Iceberg tables on S3 have an extra directory level that was not configured

Hi, I set up this sink connector and it's working but I have an issue.
I'm getting an extra level in the data directory, but before the partition folders.  Almost like some kind of sharding.  It looks something like this:
s3://my_bucket/topics/some.topic/data/00u8bQ/partition_field=xyz/part_file_000.parquet
Where the "00u8bQ" folder is one of many 6-character folders under the data directory.

Yes, that's the object store file layout feature of Iceberg's S3FileIO. See https://iceberg.apache.org/docs/latest/aws/#object-store-file-layout for more info. You can disable it but I'd leave it on for the reasons given in the docs.

Thanks @bryanck !
Yes, it makes sense to keep it, but I cannot for the life of me figure out how to change that property on my table in glue/athena?  Any tips?  I tried all sorts of combinations of ALTER TABLE, but none of my queries are accepted.

actually, when I look at the iceberg metadata file, the write.object-storage.enabled is already true. hmm.  I'm not sure why I get zero rows then.  I inspected the data files, and they look good.

@bryanck - I think I found something.
So, in the object store file layout documentation they show a path like this:
s3://my-table-data-bucket/2d3905f8/my_ns.db/my_table/category=orders/00000-0-5affc076-96a4-48f2-9cd2-d5efbc9f0c94-00001.parquet
Where as my data path looks something like this
s3://my-bucket/datalake/topics/my_topic/data/2d3905f8/category=orders/00000-0-5affc076-96a4-48f2-9cd2-d5efbc9f0c94-00001.parquet
where my config would have the equivalent of:
iceberg.catalog.warehouse = s3a://my-bucket/datalake
topics.dir = topics
I think with my deterministic hash being inserted after the table name, might be a problem?
Also, the apache example is a bit weird too, as the data is not under /data (counterpart to /metadata) in standard iceberg data structure.

Another thing.  I'm not getting any manifest lists, or updates to the metadata.  Which is probably the true root cause of all this!

It looks like you may be running an older version of the sink, can you try with the latest version? (Note, some property names were changed if going from 0.4 -> 0.5). The task should fail if the coordinator throws an exception so I'll make sure that's the case.

Hi @bryanck upgrading the version now results in a different error:

"org.apache.kafka.connect.errors.ConnectException: No partitions assigned, cannot determine leader\n\tat io.tabular.iceberg.connect.IcebergSinkTask.lambda$isLeader$1(IcebergSinkTask.java:109)\n\tat java.base/java.util.Optional.orElseThrow(Optional.java:408)\n\tat io.tabular.iceberg.connect.IcebergSinkTask.isLeader(IcebergSinkTask.java:108)\n\tat io.tabular.iceberg.connect.IcebergSinkTask.open(IcebergSinkTask.java:86)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.openPartitions(WorkerSinkTask.java:637)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.access$1200(WorkerSinkTask.java:72)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask$HandleRebalance.onPartitionsAssigned(WorkerSinkTask.java:733)\n\tat org.apache.kafka.clients.consumer.internals.ConsumerCoordinator.invokePartitionsAssigned(ConsumerCoordinator.java:317)\n\tat org.apache.kafka.clients.consumer.internals.ConsumerCoordinator.onJoinComplete(ConsumerCoordinator.java:466)\n\tat org.apache.kafka.clients.consumer.internals.AbstractCoordinator.joinGroupIfNeeded(AbstractCoordinator.java:474)\n\tat org.apache.kafka.clients.consumer.internals.AbstractCoordinator.ensureActiveGroup(AbstractCoordinator.java:385)\n\tat org.apache.kafka.clients.consumer.internals.ConsumerCoordinator.poll(ConsumerCoordinator.java:552)\n\tat org.apache.kafka.clients.consumer.KafkaConsumer.updateAssignmentMetadataIfNeeded(KafkaConsumer.java:1276)\n\tat org.apache.kafka.clients.consumer.KafkaConsumer.poll(KafkaConsumer.java:1240)\n\tat org.apache.kafka.clients.consumer.KafkaConsumer.poll(KafkaConsumer.java:1220)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.pollConsumer(WorkerSinkTask.java:472)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.poll(WorkerSinkTask.java:328)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.iteration(WorkerSinkTask.java:234)\n\tat org.apache.kafka.connect.runtime.WorkerSinkTask.execute(WorkerSinkTask.java:203)\n\tat org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:189)\n\tat org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:244)\n\tat java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)\n\tat java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)\n\tat java.base/java.lang.Thread.run(Thread.java:829)\n"

Any ideas?  I tried bumping the connector version already.

It looks like, as with the previous error, that the source topic cannot be found. Can you post your full config and how you are running Kafka Connect?

Using terraform, the same repo/deployments for all of our different sinks (mostly just confluent s3 sinks, some jdbc).
resource "kafka-connect_connector" "sink-iceberg-block-ingestion" {
  name = "sink-s3-block-ingestion-block-all-iceberg-v02"

  config = merge({
    "name"            = "sink-s3-block-ingestion-block-all-iceberg-v02"
    "connector.class" = "io.tabular.iceberg.connect.IcebergSinkConnector"
    "timezone"        = "UTC"
    "locale"          = "en-GB"

    "topics"         = "prod.block-ingestion.block_data.block.all"
    "topics.dir"     = "iceberg-ingested/kafka-uat/topics"
    "iceberg.tables" = "uat_ingested_block.block_all"
    "tasks.max"      = "2"

    "iceberg.catalog.catalog-impl"                            = "org.apache.iceberg.aws.glue.GlueCatalog"
    "iceberg.catalog"                                         = "uat_ingested_block"
    "iceberg.catalog.warehouse"                               = "s3a://data-team-uat-datalake/iceberg-ingested"
    "iceberg.catalog.io-impl"                                 = "org.apache.iceberg.aws.s3.S3FileIO"
    "iceberg.catalog.client.region"                           = local.aws_region
    "iceberg.tables.evolve-schema-enabled"                    = "true"
    "iceberg.tables.upsert-mode-enabled"                      = "true"
    "iceberg.tables.auto-create-enabled"                      = "true"
    "iceberg.tables.default-id-columns"                       = "id"
    "iceberg.tables.default-partition-by"                     = "base_chain"
    # "iceberg.table.uat_ingested_block.block_all.id-columns"   = "id"
    # "iceberg.table.uat_ingested_block.block_all.partition-by" = "base_chain"

    "value.converter"                             = "io.confluent.connect.avro.AvroConverter"
    "value.converter.schema.registry.url"         = var.schema_registry_url
    "value.converter.value.subject.name.strategy" = "io.confluent.kafka.serializers.subject.RecordNameStrategy"
    "key.converter.schema.registry.url"           = var.schema_registry_url

    "schema.compatibility"                = "NONE"
    "rotate.schedule.interval.ms"         = "60000"
    "errors.retry.delay.max.ms"           = "5000"
    "errors.retry.timeout"                = "60000"
    "consumer.override.auto.offset.reset" = "earliest"
    "connect.meta.data"                   = "false"
    "enhanced.avro.schema.support"        = "true"
    "behavior.on.null.values"             = "ignore"

    "consumer.override.max.partition.fetch.bytes" = "120000000"
    "consumer.override.max.request.size"          = "120000000"
    "consumer.override.max.poll.records"          = "20"
    "consumer.override.fetch.min.bytes"           = "500000"
    "consumer.override.fetch.max.bytes"           = "120000000"
    "consumer.override.session.timeout.ms"        = "200000"
    "consumer.override.request.interval.ms"       = "180000"
    "consumer.override.max.poll.interval.ms"      = "180000"
    "consumer.override.max.message.bytes"         = "100000000"
    "consumer.override.fetch.max.message.bytes"   = "100000000"
    "consumer.override.receive.buffer.bytes"      = "100000000"
    "consumer.override.request.timeout.ms"        = "90000"

    "transforms"                           = "tombstoneHandler"
    "transforms.tombstoneHandler.type"     = "io.confluent.connect.transforms.TombstoneHandler"
    "transforms.tombstoneHandler.behavior" = "ignore"
  }, local.funding_conn_override) #These are the connection/ssl details.
}

It could be that the sink is not picking up the correct Kafka settings. It will attempt to read them from the worker properties file. You can try setting Kafka properties for the sink's internal Kafka client explicitly by setting iceberg.kafka.* properties. The prefix for these properties will be stripped off and then they will be passed to the Kafka client during initialization.

Another user reported a similar error where the leader can't be determined, so I'm taking a further look, I'll report back.

I have a new build 0.5.8 that hopefully addresses the leader election failure you're seeing.

@bryanck We have no errors now, the sink is writing data to S3, but still no metadata updates, no manifest list, no snapshots.
Is this because we manually created the table?
The auto-create did not work for us.

Unfortunately I’m not able to reproduce your issue, I’m able to auto create and insert a using Glue catalog. If you can give me a way to reproduce your issue with a simplified example I’d be happy to investigate.

In case it helps, here's the config I used:
    topics: test-topic
    header.converter: org.apache.kafka.connect.storage.StringConverter
    key.converter: org.apache.kafka.connect.storage.StringConverter
    value.converter: org.apache.kafka.connect.json.JsonConverter
    value.converter.schemas.enable: false
    iceberg.tables: kc_glue_test.foobar
    iceberg.table.kc_glue_test.foobar.partition-by: id
    iceberg.tables.auto-create-enabled: true
    iceberg.control.commit.interval-ms: 30000
    iceberg.catalog.catalog-impl: org.apache.iceberg.aws.glue.GlueCatalog
    iceberg.catalog.io-impl: org.apache.iceberg.aws.s3.S3FileIO
    iceberg.catalog.warehouse: s3://kc-glue-test/iceberg
    iceberg.catalog.s3.staging-dir: /home/kafka/iceberg-staging

I'm going to close this, feel free to open a new ticket if you have further issues and a method to reproduce.


Gather.lua - new route creation broken after last update

it is not registering the coords correctly when creating a new gather route.
the coords seem to be missing a digit.

Hey @Gluegungeek we released some significant changes to OCR (how Automato reads text) today to get it working for main chat and the new windows. Unfortunately I also had to re-train it how to read coordinates with these changes, and there are a few more edge cases that are giving it trouble. You can see the coords it is reading in real time with test_find_coords: If you could provide screenshots that include the full clock where it is reading the coordinates incorrectly I can train it to handle those. Thanks!

2nd bug i've found. When I manually update the coords, the macro bounces back and forth between the first two coords.

@KavadATITD from my above screenshot. the coords should read:
648, -170
647, -168
648, -168
647, -164
etc...
in the first two entries, it did not input the 2nd and 3rd digit on the X coord
in the rest of the entries, it missed the middle digit on the X coords

@Gluegungeek if you can provide a screenshot of the clock at those coordinates I can train it to read them properly. I'm guessing you are in a Hyksos controlled region with the yellow background. Otherwise you'll have to wait until I make a trip down to Hyksos or someone else submits a screenshot I can train on.

Yep it is Hyksos. I'm on the border of Swenett and Thebes. Here's some screens of test where a digit disappears when i move between regions.

I'm no macro expert. Mostly a noob users. Not exactly sure what screenshots you need. I can wait too tho. Thank you for all your work tho. this program is amazing!

Ahhh, sorry about that, I see where I was unclear in hindsight. I'm looking for something like this

We also have a discord if you find that more convenient than github: https://discord.gg/eKXHJaAZ
I'm also in discord

Ah, here u go:

Err, not sure why that first screen was grayish text: here's a new one:

Awesome that second one is exactly what I needed. Got that 5 fixed. I think there was also some trouble reading 4 and 8 at 648, XXX? If you send shots of those I can fix them uip too.

You ROCK! ty!! here's a 4 and 8. let me know if you need more:

@Gluegungeek I couldn't find anything wrong with the 4 or 8 in the 648, -157 or 648, -148 shots but the 5 in the 648, -157 was a new variant I added. You may have to go back to exactly 648, -170 to get the bad 4/8

648, -170

maybe region change? here's a Thebes 4 and 8

Hmmm Those ones are all working fine for me. I just pushed an update with the new 5s at least. Also make sure your mouse isn't hovering over the clock making it gray when using the gather macro. If you have more trouble, try pulling up test_find_coords and if you find coordinates where it isn't showing the correct values, send clock screenshots for those please.

I just updated and created a new route. It is working perfectly now. Thank you sooo much my man!

 I'm glad it's working again, thanks for your patience and help, this will benefit everyone using it!


fix: Use correct Klaviyo notification key

It was checking on the incorrect key to determine a Klaviyo push.
It was checking for the Intent instead of the push

A new version has been published with this fix.

Much appreciated 🙏

PR with new native SDKs integration https://github.com/drybnikov/klaviyo_flutter/pull/15 is ready for review


Added Optimist to command-line-utilities

Optimist is a light-weight command-line option parser.

It's deprecated.


[PORT] Updating mention stripping scenario for Teams with special character support

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3107

Fixing bug: #3106

The JavaScript code can use this function:
function escapeRegExp(string) {
  return string.replace(/[.*+\-?^${}()|[\]\\]/g, '\\$&'); // $& means the whole matched string
}

(refer to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Escaping)

Looking at this more closely it seems odd that the C# code uses regex at all. The JavaScript code looks correct, and doesn't bother with a regex. Test coverage looks good in both repos.
The C# code not only shouldn't use regex (perhaps) but also its not clear how the upper part of the if (when mention text == null) ever happens.
Closing this for JavaScript as there doesn't appear to be any issue here.


Ask for required user input before installation

When running brew upgrade, Homebrew will tick away doing its job, but suddenly it will stop (at Vagrant for example), and ask for my root password.
First of all, I thought that Homebrew was based on not requiring the root password, but this is a Cask so maybe that's why.
Anyway, it's kind of annoying if you start a large upgrade, go to the zoo, only to come home hours later and find Homebrew's done almost nothing because it needed the root password after 5 minutes.
I don't know if the update/upgrade process requires any other user interaction except entering the root password, but here are my proposals for resolving the issue:
A) When running Homebrew, do a two pass operation, where the first checks if any packages will require the root password. If so, ask for it up-front.
B) Forbid any packages to require the password, so they'll have to rewrite their app to work at a user level. (They may alternatively require the root password the first time the new version is run.)

B) Forbid any packages to require the password, so they'll have to rewrite their app to work at a user level. (They may alternatively require the root password the first time the new version is run.)

This is pretty much impossible, because in most, if not all, cases, it's not the application that needs the password, but the installer.

When running Homebrew, do a two pass operation, where the first checks if any packages will require the root password. If so, ask for it up-front.

As a side note, for now you can easily accomplish exactly that with sudo -v (docs):

If given the -v (validate) option, sudo will update the user's time stamp, prompting for the user's password if necessary.  This extends the sudo timeout for another 5 minutes (or whatever the timeout is set to in sudoers) but does not run a command.

This is pretty much impossible, because in most, if not all, cases, it's not the application that needs the password, but the installer.

Something that I think is good practice, though, is to ensure whenever the password is prompted that you tell the user why, even if it's just installing X needs sudo: or even installing a .pkg file, you may be prompted for your password:
@forthrin Do you have any logs of the operation so we might be able to see what caused it?

==> Downloading https://releases.hashicorp.com/vagrant/1.8.6/vagrant_1.8.6.dmg
Already downloaded: /Users/olav/Library/Caches/Homebrew/Cask/vagrant--1.8.6.dmg
==> Verifying checksum for Cask vagrant
==> Running installer for vagrant; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
Password:
==> installer: Package name is Vagrant
==> installer: Upgrading at base path /
==> installer: The upgrade was successful.
🍺  vagrant was successfully installed!

==> Running installer for vagrant; your password may be necessary.

@forthrin To be fair, it did/does say that. Do you think it would be better if that was after "Package installers may write to any location ...."?

@MikeMcQuaid If I understood @forthrin’s problem correctly, the issue isn’t with a particular install not saying it will need a password, it’s when you are installing a bunch of them and a single one stops the others from proceeding because it needs your password but you weren’t present to give it (because you set the upgrades going and went somewhere). So the issue is only relevant when more than one cask is being installed.
@forthrin On another note, I’m not sure what command you ran, but brew upgrade does not touch casks and we have no upgrade functionality for casks yet. Your comment can be taken into account when considering exactly how to implement brew cask upgrade, but at the moment the official stance is that we (the HBC part) officially do not support any upgrade functionality, and that any method you pick to fake it is unsupported and done at your our risk.

So the issue is only relevant when more than one cask is being installed.

I see. In that case I think it's somewhat unavoidable for security reasons as maintaining it so any brew subprocess can sudo with impunity after a password has been requested is a bad idea for security.

@vitorgalvao is right. The problem having to sit in front of the computer for hours in case an installer wants a password during a long series of installs (which can happen anywhere).
The command I'm using is:
alias bu='brew update; brew upgrade; brew cask list | xargs brew cask install --force'

sudo brew cask install handles that, FYI, but I have no idea whether the @Homebrew/cask folks recommend it.

Don't use sudo brew cask, it'll lead to permission issues down the line.
ie. https://github.com/caskroom/homebrew-cask/issues/25547

@adidalal In that case it's probably worth removing https://github.com/Homebrew/brew/blob/de880f1e870d91dd838228de2987f9bd6d194aef/Library/Homebrew/brew.sh#L244. I do still think that cask should prompt with for exactly the command it's going to run before running it and make the preceding message specifically state why root access is needed.

I agree with @adidalal that it should be an absolute requirement for packages to write specifically and exactly why they need root access. Once you give a package root access the privilege, how do you know it does not do illegitimate things with these privileges? Security must be a priority these days.
Fictional example of how this could be solved:
$ brew cask upgrade
Checking packages ...
The following packages require your root password to upgrade:

Vagrant:
cp /tmp/installer/default.cfg /etc
mkdir /usr/local/vagrant

MySQL:
chown $USER /usr/local/mysql

Enter password:

Refusing to enter the password would upgrade the rest of the packages, but not those listed.

@forthrin You seem to be going from the idea we control what the exact actions the package is taking and asking for sudo, but we don’t have such granularity. The only thing we’re doing is saying “Hey, installer, install this” and it’s going to say “Can do! But I need elevated privileges”.
That’s why I also think that printing the exact command we’re going to run is not that useful in this case. If we had
==> Running installer for vagrant; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
==> The command we’re running is `sudo installer -pkg /path/to/package.pkg -target /`
Password:

Would that be better? Perhaps, but marginally, as you still have no idea what’s happening anyway, and still have no control.

Would that be better? Perhaps, but marginally, as you still have no idea what’s happening anyway, and still have no control.

Personally I think something like that would be superior, yeh 👍

Some specifics would make it easier to understand the situation:

How many percent of packages require the root password?
What are the most common reasons packages require the root password?
Can all packages in theory be reprogrammed not to require the root password? If no, why not?
Could a requirement/mechanism be created that let Homebrew ask a package, "Will you require the root password"? (I would assume there is already a dogma that packages must adhere to.)

grep for the pkg stanza and you’ll get a starting idea (on a phone, now. Can’t do it).
Installing a pkg. Also, uninstalling things.
No. because we just install things, and try to be as close as possible to the manual installation, so we’re dependent on how the developers upstream decide to do it. Also, the system simply requires elevated privileges for some things. Unless you convinced every developer and Apple to work so these things wouldn’t require sudo, no chance.
In theory, evaluating casks with pkg would already do most of that.

I’ll add that some of this has already been discussed in the past, but there was lack of interest in actually implementing, so it was scrapped for more urgent things.

How about this:

Echo to the terminal the commands that requires the root password. A little or incomplete information is better than nothing.
grep for pkg, or whatever other easily detectable giveaway for a root password prompt. Then do either of the following:

A) Order these packages first in the installation queue, so the password prompt comes straight after starting the upgrade process. This also increases the chances of UNIX reusing the password.
B) Explicitly ask for the root password before starting the installation process, and then feed it to whatever password prompt occurs during an installation.

This also increases the chances of UNIX reusing the password.
Explicitly ask for the root password before starting the installation process, and then feed it to whatever password prompt occurs during an installation.

These two things make the process significantly less secure as now anything else can get unauthenticated root access for the duration of that process.

Good point. Homebrew maybe forces the prompt every time? Option A or C are still valid: Isolate the packages that require the root password so they don't abrupt the installation flow of those who don't.

Closing this out as there doesn't seem to be forward motion on it but reopen if you disagree Cask folks.


Allow horizontal tab as valid header field body

Need unit tests

Absolutely not.
This functionality exists to close ZF2015-04, which exists to prevent message splitting. RFC 2822 clearly outlines what characters are valid for header values, and the horizontal tab is not in that list.
As soon as we start selectively allowing characters disallowed by the spec, we begin opening ourselves to potential security vectors.

This is related with https://github.com/zendframework/zf2/issues/7591

Based on zendframework/zf2#7591, this is specific to the Received header. The general ABNF for headers does not allow whitespace except as part of a header folding sequence (which we already detect and handle). (See section 2.2 for details).
As such, the appropriate fix is to provide custom validations inside of Zend\Mail\Header\Received, and not create a general change that affects all headers.

@FraGoTe I'm willing to do entertain the change, but it has to be specifically in the Received header implementation; I don't want to introduce new potential vectors in other headers.

@weierophinney create a custom validation in Zend\Mail\Header\Received it is ok, I think.

Thanks for discussing this one. RFC2822 was obsoleted by RFC5322 in 2008. Section 2.2 was revised so HTABs in header field bodies seem to be allowed in general since then.


Error: HiveError: Box not found. Did you forget to call Hive.openBox()

I am getting the following error when I try to open the box tableDeviceShadow in my code:
Error: HiveError: Box not found. Did you forget to call Hive.openBox()

I have already added the await keyword before the Hive.openBox() function, but the error is still occurring.
I have tried the following things to fix the issue:

I have updated the Hive library to the latest version.
I have restarted my computer.
I have cleared the cache and data for the app.

None of these things have worked.
I am attaching a screenshot of the error message.
I would appreciate any help you can provide in resolving this issue.
Thank you.

Where are you calling the openBox in your app?

@milindgoel15 On Crud operation in DB
like: add, update, and get data from db.

You have to call openBox because the app runs, in the main() function before runApp. And after that you can do crud operation anywhere in the app by creating an instance of created box using Hive.box(boxname)

You mean I should open it only once when the app initializes?

You mean I should open it only once when the app initializes?

yes, it should only be initialised once in the main function.


setting failTaskOnFailedTests on PublishTestResults task does not fail task

Hello ,
I still find https://github.com/Microsoft/azure-pipelines-tasks/issues/9711 broken and we need this functionality working as a continuation of our workflow of next step to trigger bug creation for even single failed test inside.
Log file for TASK - Publish Test Results
7_Automated UI Test Results.txt
complete Pipeline log files
logs_1506.zip
Please , do needful earliest as it is bug and very important functionality for our workflow.

Hey,
Could you please cross verify if the task is failing or not. We tried reproducing this at our end and the logs seem to be matching but our task failed. If your task is not failing, then we will need more details about your scenario.

Hello, Task is not failing at our end. I have provided all logs in zip and .txt also. Let me know what exactly should I provide other that that? Thank you

Hey,
[https://dev.azure.com/sahy/myproject/_build?definitionId=4] this is the link to our project pipeline where the task failed. Please share your yaml file if possible.

Sorry for delay -> YAML
steps:

task: PublishTestResults@2
displayName: 'Automated UI Test Results'
inputs:
testResultsFiles: '*.xml'
searchFolder: '$(System.DefaultWorkingDirectory)\screenshots'
failTaskOnFailedTests: true
continueOnError: true

Hey,
In the case where both continueOnError and failTaskOnFailedTests are true, continueOnError takes the precedence. So, instead of failing the task, it proceeds to the next task. If you want to fail the task, then enable only failTaskOnFailedTests.

@jp108 : Please let me know if you need any help ? Based on the logs also we see that the task is failed. If you are still facing the problem please let us know. And if possible can you point to your pipeline for better understanding.

If you guys can update the agents, please update the agent to latest agent version to get unblocked.
For all others some might have received the latest fixes and not see this issue, for few the fix will rollout in few days.

Is there any estimate for the West EU rollout (unless agents are rolled out per account)? All the hosted agents we use are on 2.164.6 and there are quite a lot of pipelines to do a manual checks now...
We have tested a private agent with the version 2.164.7 and it fixes the issue unfortunately we are using the hosted agents which are not yet updated...

It was fixed, but now I'm having the issue again

Are using using agent with version 2.164.7?

Can you guys please confirm, if the issue is resolved now?

@navin22 Issue is fixed for me now. Thanks!

If anyone is still facing the issue let me know, or reopen the issue.

I have the same issue as well.

This is an issue with 2.167.0. Please reopen.

I'm still seeing this problem on 2.171.0.

If anyone is still facing the issue let me know, or reopen the issue.

@navin22 I don't have permission to reopen.

We're suddenly seeing this issue. Agents are on 2.174.1.

We are seeing this issue on 2.165.0. failTaskOnFailedTests: true does nothing: failed tests do not fail the task.

Same issue for agent 2.195.1 and JUnit test results on linux:
  - task: PublishTestResults@2
    inputs:
      testResultsFormat: "JUnit"
      testResultsFiles: "report.xml"
      failTaskOnFailedTests: true

This is still a bug. At least with NUnit results format.

It's there something preventing us from opening a PR to fix this ourselves? I'll dig in if there's a good chance of a PR being accepted.

This amuses me, 3 years for this bug and no one cares. Still reproducing for me (NUnit, fail = true)

We worked around it by letting the actual tests task fail (which goes red and breaks the build as you would want) but running the publish test results regardless (always) even if the previous task failed. That way the publish task is green, the tests are red and the pipeline fails.


Update build configuration

The build configuration of this project is outdated and may no longer work.
This pull request will be merged automatically if there are no conflicts.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Check if ActiveStorage used before fetching Run's csv_file

Right now, both:
https://github.com/Shopify/maintenance_tasks/blob/0c5813cd2d5acd8a4457d9ddfe6072dd9bd48ed6/app/views/maintenance_tasks/runs/_info.html.erb#L12
and
https://github.com/Shopify/maintenance_tasks/blob/0c5813cd2d5acd8a4457d9ddfe6072dd9bd48ed6/app/models/maintenance_tasks/run.rb#L191
will blow up when called in an app that does not have ActiveStorage set up.
Arguably
https://github.com/Shopify/maintenance_tasks/blob/0c5813cd2d5acd8a4457d9ddfe6072dd9bd48ed6/app/models/maintenance_tasks/run.rb#L189
and
https://github.com/Shopify/maintenance_tasks/blob/0c5813cd2d5acd8a4457d9ddfe6072dd9bd48ed6/app/models/maintenance_tasks/runner.rb#L48
also blow up, but these are scenarios where the app is attempting to use CSV functionality, so I think it's okay that these result in the missing table error.
IMHO the way to solve this is to just return early from Run#csv_file if the ActiveStorage::Attachment table doesn't exist, and to then use the safe access operator for the first two lines above.
I figured we wouldn't want to use the safe access operator for the second two scenarios (L189 in Run and L48 in Runner), since ActiveStorage should be configured here if the user wants to write CSV Tasks, so it makes sense for this to 💥 .
EDIT: The preloading query in TaskData also fails if the table doesn't exist, so I've tweaked that to not preload unless it's the table exists there too.

@etiennebarrie Yeah I'm still open to making ActiveStorage an explicit dependency of the gem! IMHO it's kind of an unrelated conversation to this patch. Maybe it's something we don't want to force on our users (also given it's a rare occurrence that this would be an issue, since most Rails apps include the library by default anyways), but I don't think it's a problem to add it either.
I just wanted to get your opinion on https://github.com/Shopify/maintenance_tasks/pull/359/commits/3c39814fb93d7910d34948eb3b281321d4a59926 before we ship - attempting to override the scope causes all sorts of oddities (I'm seeing the tests blow up with
RuntimeError: /Users/adriannachang/src/github.com/Shopify/maintenance_tasks/app/models/maintenance_tasks/run.rb:8: warning: already initialized constant MaintenanceTasks::Run::STATUSES

)
And also just seems to be discouraged by Rails:
https://github.com/rails/rails/blob/7b9cfde741662603f7fdd78698190b2fdefead17/activerecord/lib/active_record/scoping/named.rb#L197-L203
We can just create our own scope instead, which is what I've done, but there is a slight concern that the original scope would get used by mistake. WDYT?

No tests, not sure how to add some that won't be just stubbing table_exists?, but to tophat, I did:
$ rails db:migrate:down VERSION=20210112141723
$ rm test/dummy/db/migrate/20210112141723_create_active_storage_tables.active_storage.rb


Broken links

schema links are broken in the following sentences:

Use the ASC for IoT security agent configuration schema to make changes.
Make sure to validate your agent configuration against this schema. An agent will not launch if the configuration object does not match the schema.

Both links should be "https://aka.ms/iot-security-github-module-schema"

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

ID: 90f9c7bb-7e46-fb31-4095-7697b50e3f0f
Version Independent ID: 24ade002-b489-3e3b-6d0f-c1600c48bdd7
Content: Configure an ASC for IoT agent Preview
Content Source: articles/asc-for-iot/how-to-agent-configuration.md
Service: ascforiot
GitHub Login: @mlottner
Microsoft Alias: mlottner

@mibarg Thanks for the feedback and reporting this. We are currently investigating and will update you shortly.

PR has been created to update the document.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Read water sample(s) for specific date

I am not able to get a water sample for any date. is there any way to read the water sample?

I have implemented in following any one can use to modify as need,
add in package.json
"rn-apple-healthkit": "git+https://github.com/zainlinux1989/rn-apple-healthkit.git#9361139",
run npm install or yarn install


Add support for the dim command.

This returns the dimensions of an image when it can.

You can see the PHP implementations of the DIM command:
https://github.com/Studio-42/elFinder/blob/2.1/php/elFinder.class.php#L1890
https://github.com/Studio-42/elFinder/blob/2.1/php/elFinderVolumeLocalFileSystem.class.php#L449


8.2-3

Compared to CLRS2e, explicitly requires to prove that the modified versin of Counting-Sort is not stable. Adds the requirement to rewrite the algorithm to make it stable.


Can't log into server - don't know how this patch was installed in the first place

My server was working fine last night.  We tried to log in this morning and everyone is getting this message:
java.lang.NullPointerException: Cannot invoke "io.dogboy.serializationisbad.core.config.PatchModule.getClassAllowlist()" because "this.patchModule" is null
I am honestly not sure how serializationisbad.jar got put in my server's mod folder but I also can't remove it.  When I remove it and then restart my server it is re-installed.  Who is doing this?  Is this Bisect Hosting doing this or is it being auto-patched by Forge somehow?  The thing that is invading my server and messing it up without permission... is this patch... ironically.  We are a 5 person, private, whitelisted, server... do we even need this?
I understand the problem you are trying to solve... and I appreciate that... but what solves this problem?

We have pushed this to all packs on our systems that are potentially affected to ensure customers are safe from this exploit. If you are having issues logging in, please create a support ticket and we will get this looked into an disabled for you!
-BisectHosting

Uhm.. while we were intially in contact with some parties about automatic inclusion of this patch after approval of the user, nobody mentioned to us that that was actually implemented so I'm a little bit confused about that part as well. We're definitely not persisting that in some way.

We are a 5 person, private, whitelisted, server... do we even need this?

If anyone on the server can be trusted, then I would probably say no, but it also shouldn't hurt having it on the server just as a precautionary measure (at least if that bug is solved)
Regarding that exception, do you have a full stacktrace for me so we can look at that bug?

Checked on this customers server - Looks like there is no stacktrace unfortunately, just shows the attached error.

This is on Forge 1.19.3, I will leave it up to @Plisko1 to share their mods they are using as I don't want to share too much as it's their server.
@Plisko1 I have disabled the mod on your server for now! :)

@dogboy21 Same issue on the Techopolis 2 pack if you needed a pack for testing it - https://www.curseforge.com/minecraft/modpacks/techopolis-2

@BisectStudios i am having the same issue for no reason
again a small whitelisted server w 3 players. This is really annoying, especially for people who don't know how to find this thread.

@OtherMongrel To disable it, rename the file to "serializationisbad-1.3.jar.d", an announcement will be going up shortly in regards to this also within the control panels, apologies for the issues here. As this is a pretty major issue security wise, we wanted to roll out a fix as soon as possible to ensure customers using our services are protected.

@BisectStudios thank you for disabling it.  I have a copy saved and I will try to integrate it back in if I can... or if this issue is resolved.  In the future I wonder if it is possible to let me opt into something like this?   I understand the urgent need... but mods are so flakey together at times... it is nice to know when a hammer might drop... if you know what I mean?
@dogboy 21 I will help if I can.  Will the latest log help?  I am attaching it to this post.  This is a custom mod server we have been slowly adding mods to over time.  I just recently added a bunch more and updated everything to the latest 1.19.3 versions for Forge. 44.1.23.  We are now in the process of putting the new pack through it's paces and new versions and sorting through various issues.
latest (1).log

Thanks for the quick support guys!

@Plisko1 Yes, that helped, thank you!
After looking at the log, this seems to be related to #30 since it's the only mod that got patched in your pack.
We're now looking into that problem as well.

Should be fixed with version 1.4