Custom Learning Pathways not showing

Describe the bug
When we add custom content to the M365 pathways it is not showing up on both the training page and inside a learning route.
To Reproduce
We create custom content within a learning route, but it doesn't show up. Also doesn't show when we empty our cache.
Expected behavior
We expect the learning route to be visible when we create it.
Screenshots

Learning Pathways version number
<IP_ADDRESS>

Hi @rickvana, I am not able to reproduce this with the instructions that you provided but I do have a guess.
Can you please tell me the default language of your Learning Pathways site? To do this go to Settings > Site Information > View All Site Settings >  Site Administration
| Language Settings. My guess is that the default language for the site is English.
Then check the custom content (sub-categories, playlists, and assets) you are trying to create. Do you have title, description, etc for all the fields for the default language? If not that is the problem, add an English translation and they should show up.
Your Learning Pathways site was provisioned through the LookBook. As a result the default language for the site is set to English (see system requirements). You always need to have content in the default language for the site in order for it to be displayed.
Give that a shot and let us know if that fixed the issue.

@rickvana great to hear it. Closing this as resolved.


versioning of the spec itself

While the spirit of the spec feels like maintaining a forward-compatible spec, it seems there is a need for versioning of the spec itself.
Like actool to check correctness, there will be a need for loose coupling of a new change and the compliance of ACIs to this change. There is no field for this (yet), but could be introduced, and infer that manifests without an AC_VERSION (or similar) are less than the initial version that includes versioning.
With a version of the spec then implementors can more clearly pin and implement. and staged their release criteria (as opposed to a moving target)
/cc @jonboulle @philips

@vbatts what is wrong with

acVersion (string, required) represents the version of the schema specification that the manifest implements (string, must be in semver format)

in both image and pod manifests?

ohman. I read that as an arbitrary version of the aci. Don't know what the annotation of "version" didn't flag in my brain for that.

@vbatts can we clarify?
On Thu, Apr 23, 2015 at 2:07 PM, Vincent Batts<EMAIL_ADDRESS>wrote:

ohman. I read that as an arbitrary version of the aci. Don't know what the
annotation of "version" didn't flag in my brain for that.
—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95719314.

absolutely. I can start a section for it, in addition to the loose ref link
PR I just opened
On Thu, Apr 23, 2015 at 5:08 PM, Jonathan Boulle<EMAIL_ADDRESS>wrote:

@vbatts can we clarify?
On Thu, Apr 23, 2015 at 2:07 PM, Vincent Batts<EMAIL_ADDRESS>wrote:

ohman. I read that as an arbitrary version of the aci. Don't know what
the
annotation of "version" didn't flag in my brain for that.
—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95719314.

—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95719531.

thinking further on this, it needs its own section, because there should be
a stance on version compatibility. e.g. "I'm building pod manifest from
version 0.x.y and it includes an app that is from from 0.x.y-1, is that
okay?", etc.
On Thu, Apr 23, 2015 at 5:13 PM, Vincent Batts<EMAIL_ADDRESS>wrote:

absolutely. I can start a section for it, in addition to the loose ref
link PR I just opened
On Thu, Apr 23, 2015 at 5:08 PM, Jonathan Boulle<EMAIL_ADDRESS>
wrote:

@vbatts can we clarify?
On Thu, Apr 23, 2015 at 2:07 PM, Vincent Batts<EMAIL_ADDRESS>wrote:

ohman. I read that as an arbitrary version of the aci. Don't know what
the
annotation of "version" didn't flag in my brain for that.
—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95719314.

—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95719531.

Yes, fair point..

I updated my PR, but will have to address compatibility in a bit
On Apr 23, 2015 5:22 PM, "Jonathan Boulle"<EMAIL_ADDRESS>wrote:

Yes, fair point..
—
Reply to this email directly or view it on GitHub
https://github.com/appc/spec/issues/313#issuecomment-95722196.


HTML rapports "Cannot read property 'riskcode' of undefined"

I know this issue is been asked a couple of times, but I tried to output a HTML rapport a couple of weeks ago (after the last time this issue is being asked) and it worked perfectly with a HTML rapport.
But now it gives the same error as the persons asked before  Error: Task Failed. Error: "Cannot read property 'riskcode' of undefined"; Markdown is fine but HTML is not working anymore

I do not get your question.
The first time ( a couple of weeks ago) HTML option is working perfectly
The second time right now : i got this error and I don't get any error.
Markdown doesn't give any errors and gives a normal repport

Could you check the HTML report you got when it worked and check the count for each type of security issues. and then take the markdown report (since HTML report is not working) and count each type of security issues reported and see if the count is different.

With the Html report i got 1 medium and 4 low risks
With the markdown report i got 1 low risk

Also can you confirm that, in the markdown file that you got only 1 risk, the riskcode value is not available?

I can confirm that in the markdown i got only 1 risk.
What do you mean by the riskcode value ?

I too am receiving the same error when trying to produce the HTML or XML report via the VSTS extension.

I got the same issue with TFS on Prem and the latest version of the extension. Any way to work around the issue?

I too got the same issue. I a getting this error whenever there are 0 alerts after the scan. I think its something related to report generation.


Added 1Inch-v4 connector.

[x] Added 1inch-v4
[x]  Deploy and comment addresses
[x] Add to dsa-connect here
[ ] Add to docs

Deployed on Polygon: 0x63995c71e2f57d0aF786485E255A805603e54e76

Deployed on Polygon: 0x63995c71e2f57d0aF786485E255A805603e54e76

Deployed on Arbitrum:
Name: 1INCH-V4
Address: 0x929376C77A2fB8152375a089a4Fccf84Ff481479
tx:

Deployed on Mainnet:
Name: 1INCH-V4-A
Address: 0x16ac1e894abb854519243e9ff982673ab5497549
tx:


Wallaby.js not working with the Babel 6 ASP.NET 5 skeleton

Wallaby.js has stopped working ever since the ASP.NET 5 ES2016 skeleton was updated to use Babel 6.
The configration, as is, throws the following error:
wallaby.js started
core v1.0.223
Failed to load configuration file:  Error: Cannot find module 'babel'
  at Module.s.require (C:\Users\Sergi\.wallaby\core\server.js:18:22437)
  at Object.<anonymous> (C:\Software Development\Projects\PPOZO\src\Web\wallaby.js:3:13)
wallaby.js stopped

Which of course makes sence since babel is not installed as a single package anymore.
@ArtemGovorov, I've tried using a wallaby configuration based on this es7 sample of yours, (I removed de require("babel") and copied the compilers section; the rest of the config is left untouched) but all I'm getting, even with debug: true is:
wallaby.js started
core v1.0.223
Failed to load configuration file:  Unexpected identifier
wallaby.js stopped

Any ideas? Has anybody gotten this to work?

Never mind, just changed require("babel") to
require("babel-core")

Sorry for wasting your time.

@spapaseit You may actually completely remove the babel parameter, should still work for you (wallaby will automatically use babel-core from node modules). However, currently require("babel-core") would still use babel 5 (installed in your node modules).
I have sent the PR to update to babel 6.
BTW, in the sample you have referenced, it works without any config, because wallaby in this case is using babel config from .babelrc file (and there's no .babelrc in the skeletons).

@ArtemGovorov, I've changed my config and package.json to match that of your PR. Everything works fine now. Thanks!

@ArtemGovorov, I thought everything was working fine unit I started writing tests.
RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>
RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>
RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>
RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>
RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating http://localhost:49980/src/parent/edit-parent.js

RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating http://localhost:49980/src/parent/edit-parent.js
	Evaluating http://localhost:49980/test/unit/edit-parent.spec.js
	Error loading http://localhost:49980/test/unit/edit-parent.spec.js

RangeError: Maximum call stack size exceeded.
	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating<EMAIL_ADDRESS>	Evaluating http://localhost:49980/src/parent/edit-parent.js
	Evaluating http://localhost:49980/test/unit/edit-parent.spec.js
	Error loading http://localhost:49980/test/unit/edit-parent.spec.js
at undefined:

Of course, I assumed it's my fault and that I probably have a circular reference somewhere in edit-parent.js. The fact that wallaby starts whenever I remove the line import {EditParent} from "../../src/parent/edit-parent"; from my tests seemed to confirm my suspicions... I cannot, however, find any circular reference anywhere.
Is there anything els I could be missing? Could it be that this is not my fault after all? I know it seems unlikely but...

@spapaseit It doesn't seem like an issue with wallaby.js.
Also, I can't reproduce it. Since there's no code you've mentioned (edit-parent) in the skeleton (must be your app specific code), but for example the file skeleton-es2016-asp.net5/src/skeleton-navigation-es2016-vs/src/users.js has
import {inject} from 'aurelia-framework';
import {HttpClient} from 'aurelia-fetch-client';
...

and commenting/uncommenting it doesn't cause any issues.

@ArtemGovorov it has to be something in my code, I know; it's the most logical explanation but... Thanks for you help!


Automate the creation of GitHub Enterprise eval server

Document the process to spin up a new GitHub Enterprise eval service, license it (45-day eval license), and provision it with:

webhook configration
user account for webhook server callback to deployment repo
creation of service template examples

@npearce
👋
Some thoughts:

Document the process to spin up a new GitHub Enterprise eval service, license it (45-day eval license), and provision it with:

The licensing is approved manually and issued, so this could be a chokepoint in full automation. However, once you have the license, the upload is available via the API:
https://developer.github.com/enterprise/v3/enterprise-admin/management_console/#upload-a-license-for-the-first-time

webhook configration

Are there specific events you're looking to configure for webhook events?

user account for webhook server callback to deployment repo

We don't have a POST endpoint to the api for user creation, however here's a dirty workaround using curl to send a post to the first user creation screen:
$EC2_IP correlates to the hostname of GitHub Enterprise
$1 = github username you wish to define
$2 = email address for user
$3 = password
  echo "Creating first GitHub Enterprise administrator account"
  TEMPDIR=`mktemp -d /tmp/XXXXXXXXXXXXX`
  echo $TEMPDIR
  # Check if the join endpoint is up and running
  curl -iskL https://$EC2_IP/join | grep 'Status: 200 OK'
  curl -k -v -L -c $TEMPDIR/cookies https://$EC2_IP/login > $TEMPDIR/github-curl.out
    authenticity_token=$(grep 'name="authenticity_token"' $TEMPDIR/github-curl.out | head -1 | sed -e 's/.*value="\([^"]*\)".*/\1/')
    curl -X POST -k -v -b $TEMPDIR/cookies -c -L $TEMPDIR/cookies \
    -F "authenticity_token=$authenticity_token" \
    -F "user[login]=$1" \
    -F "user[email]=$2" \
    -F "user[password]=$3" \
    -F "user[password_confirmation]=$3" \
    -F "source_label=Detail Form" \
    https://$EC2_IP/join >$TEMPDIR/github-curl.out 2>&1
    cat $TEMPDIR/github-curl.out
    grep "< Set-Cookie: logged_in=yes;" $TEMPDIR/github-curl.out
    rm -rf $TEMPDIR

creation of service template examples

Assuming this is a big-ip thing?

@webdog
That cover bullet 2 nicely! Thanks.
#3 (BIG-IP Service Definition templates) I can cover by sharing a repo on github.com, which an administrator can download and then change the remote to their GitHub Enterprise, like with these instructions: https://help.github.com/articles/changing-a-remote-s-url/
Hmmm, feels like we're getting there...

@npearce Sounds great!
What are your thoughts around a Service Template being around only as long as a Pull Request is opened?
User opens PR -> Webhook to BIG IP, create service template for architecture and rules
User updates PR with new commits -> BIG IP examines the commit object for any necessary changes to the template (Maybe a port being updated in an application?
User closes/merges PR -> The environment is shut down

Ok!!! This is exactly where my head is at! I’m looking at ephemeral entry points, or eventemplate representations of such, so long as I’m solid with the source of truth being in the GitHub repo.
The first rule of IaC club is, never look at the “I” to understand the truth.


Reduce server log retention time

logs are too big right now, reduce to around 3 hours

Originally written by chomp

Hopefully this is what y'all are looking for given the context https://discord.com/channels/875684761291599922/875707258074447904/1192939409088188417
Please review at:
https://dev.sp-tarkov.com/SPT-AKI/Server/pulls/186

Originally written by r2go


Missing Theater screen size mode option when script is enabled

First of all thanks a lot i haven't seen any issue with the script so far, i just randomnly saw that when it is enabled you don't have the option anymore to go in/out of theater screen size.
Not really an issue but i just wanted to point it out

Thank you for pointing this out! I also tried to fix it! but because youtube is trying to fix it! Therefore, I can only use iframes to completely remove ads in videos, so some features such as screen size mode will be limited.


Enchantments work in Peaceful land

Im using FactionsUUID and the enchantments such as sandstorm and pull work in peaceful land to peaceful players.. How would I prevent that?

This should be fixed in the next update.


This has got to be user error somewhere.  - Error: The requested accessory 'sonymutetv' was not registered by any plugin.

Hi Guys,
There are plenty of posts with this exact same error.
So far all of those issues are misspellings or capitalization.
In my case I've got to be missing something obvious.
Working on writing my own plugin.  All was working fine and then all of a sudden I started seeing the below.
It's not a case thing, it's not spelling
Has to be misunderstanding on my part.
Thanks for any help you can provide.
Russells-MBP-2:.homebridge russellhogg$ DEBUG=* homebridge -D -P /../HBDEV/
[2017-11-9 01:10:32] Plugin /usr/local/lib/node_modules/homebridge-sonymutetv does not have a package name that begins with 'homebridge-'.
[2017-11-9 01:10:32] Loaded plugin: homebridge-sonybraviatv
[2017-11-9 01:10:32] Registering accessory 'homebridge-sonybraviatv.SonyBraviaTV'
[2017-11-9 01:10:32] ---
[2017-11-9 01:10:32] Loaded config.json with 1 accessories and 0 platforms.
[2017-11-9 01:10:32] ---
[2017-11-9 01:10:32] Loading 1 accessories...
/usr/local/lib/node_modules/homebridge/lib/api.js:64
throw new Error("The requested accessory '" + name + "' was not registered by any plugin.");
^
Error: The requested accessory 'sonymutetv' was not registered by any plugin.
at API.accessory (/usr/local/lib/node_modules/homebridge/lib/api.js:64:13)
at Server._loadAccessories (/usr/local/lib/node_modules/homebridge/lib/server.js:263:42)
at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:86:38)
at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:40:10)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (module.js:635:30)
at Object.Module._extensions..js (module.js:646:10)
at Module.load (module.js:554:32)
at tryModuleLoad (module.js:497:12)
at Function.Module._load (module.js:489:3)
Russells-MBP-2:.homebridge russellhogg$ npm list -g --depth=0
/usr/local/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>-> /Users/russellhogg/HBDEV
Russells-MBP-2:.homebridge russellhogg$
{
"bridge": {
"name": "HBDev0003",
"username": "CA:22:3D:E3:CE:30",
"port": 51826,
"pin": "111-11-111"
},
"description": "config.json for DEV purposes",
"accessories": [
{
"accessory": "sonymutetv",
"name": "TV Mute",
"macaddress": "44:1c:a8:91:b3:89",
"ipaddress": "<IP_ADDRESS>",
"presharedkey": "0000"
}
]
}
Pulling my hair out, has to be something stupid.
Russell

Try ”accessory”: “homebridge-sonymutetv” in config.json.

Thanks for the speedy reply,
That's where I started.  I shortened up while grasping at straws.
It's back at homebridge-sonymutetv now and still showing the same issue.
I'm new to this, but I believe that if I had an issue in index.js I would see it when I tried to npm install it.  I gather that this error is happening before that code would ever execute anyway.
I've actually completely removed and reinstalled nodeJS over the last several hours as well.
Wiped the persist and accessories directories.
Scratching my head.
-R

[2017-11-9 01:10:32] Plugin /usr/local/lib/node_modules/homebridge-sonymutetv does not have a package name that begins with 'homebridge-'.

This one is issued by homebridge when loading the plugin.  Make sure the name in the plugin’s package.json matches the directory name.

The name of the directory has to match?
the directory now is HBDEV.  I'd need to make it homebridge-sonymutetv?

I see the warning you mentioned, but I can't find any information on how to resolve that.
I did change my folder name to match the plugin name.  That doesn't seem to have had any affect.

Typically, the name of the package (as specified in its package.json) equals the name of the diectory.  This name must match the package name passed by index.js in registerAccessory() or registerPlatform().  The plugin name in that call should match the value of accessory or platform in config.json.

It started working after getting the keywords in there.  I guess it's possible I'd tweaked something else.  I'm a little bleary-eyed.  Thank you much for the help.  I probably won't know I have it for sure until tomorrow.

I'm in good shape here.  Thanks to @ebaauw
Thank you Erik!


Should --mode be in webpack-serve?

I was disappointed to see that --mode doesn't work.
For example -
webpack-serve ./webpack.config.js --mode development
This forces me to add additional logic to my webpack.config.js file.
Should --mode be in webpack-serve given that webpack-serve should be "lean".

😞 your issue is invalid because you removed the issue template.
I'll comment on the question, however. webpack-dev-server allows pass-through CLI options from WDS through webpack-clis shared modules. This makes the module overly and needlessly complex. webpack-serve does not follow that pattern and does not think it's proper. mode can easily be set in a config and should be for use with webpack-serve.

Sweet two for two :)

+1 to the initial request.
In my use-case, I have a single webpack.config.js file that is used for both build and dev-server. Currently, it is required to do some tricks like webpack-dev.config.js to override this property. Not zero-config anymore.

@just-boris use environment variables. if cross-platform is an issue, use cross-env. webpack-serve is not webpack, nor is it webpack-cli. It's a development server and there are a host of reasonable alternatives to a command line flag.

Also of note, this was added a while back https://github.com/webpack-contrib/webpack-serve/blob/a4368071f9aaa1605e6d5e3690933ebba8db6a5b/index.js#L16 so you can use that to detect you're in the webpack-serve environment.

@shellscape I understand your motivation and the desire to keep this module simple. That said, it's unfortunate that omitting these CLI options usurps the no-config-file-required-ness of webpack 4. It's easy to avoid using a config file while getting started with webpack 4, and likewise it would be nice if a new user of webpack could start using webpack-serve without immediately needing to write a config file.
Is it unduly complicated to simply pass unknown arguments to the webpack config and hope for the best? A limitation could be that only simple string arguments are "supported", to avoid requiring config validation. This would cover the basic cases of single entry, single output, and mode, which users would typically need to get started.

Thanks, @shellscape!
I would go for the environment variable solution. In my pacakge.json
"scripts": {
   "build": "MODE=production webpack",
   "start": "MODE=development webpack-serve"
}

@just-boris I like what you're thinking.  Maybe WEBPACK_MODE ?

I like the automatic inclusion of an environment variable. I stuck this in my config:
    mode: process.env.WEBPACK_SERVE ? 'development' : 'production',

I would contend creating this config file is an unfriendly onboarding experience for someone getting started with webpack (feels very permanent when you have no clue what you're doing), but it's friendly enough for me! 😄

@benwiley4000 though I don't agree with the concern about the zero-config folks, I would say that there's an opportunity there for someone to submit a PR to webpack proper for reading environment variables to determine options like that. lib/WebpackOptionsDefaulter would be the place to look.

Fwiw, running NODE_ENV=production webpack-serve appears to set the mode.

@dylanized from webpack documentation:

Please remember that setting NODE_ENV doesn't automatically set mode.

put the production env var set only in the build script.
and in the config do like:
mode = process.env.NODE_ENV || 'development'

this worked for me for a very long time.

What if we don't have a config file?

@eirikb That's what this is for: #231

@cilliemalan Thanks. I didn't see the PR. I think it was very related to this issue, and now it has a link, which is nice 😄


[Fluid tank scenario] Remove mesh utilities

This PR removes the mesh utilities associated with the fluid tank scenario.
Meshing is now done as a pre-processing step on the API side. To allow for this, the properties of the fluid tank are written to a JSON file that is passed to the API.

Note that this does not fully remove the dependency from gmsh, since it is also used by the structures submodule under development.
To circumvent this, I have removed structures from the main __init__.py file, which prevents the cascade of imports that causes an error when running import inductiva on a machine without the libraries required to run gmsh.

Awesome!!! Can we already remove the magic line from the requirements.txt file? Or is it pending on things related to the structures sub package (I saw that you commented it out here, so it should be fine?)

See my comment above, https://github.com/inductiva/inductiva/pull/538#issuecomment-1710361373.

@hpenedones, the import inductiva statement doesn't fail because gmsh is in the requirements. It fails because we import gmsh in some file under inductiva.submodule.subsubmodule..... Commenting the structures import statement from the main __init__.py plus the changes proposed here should be enough to make the package run without trouble.

But we can also remove the gmsh from the requirements so that it doesn't trigger the installation of gmsh and its dependencies, no?

But we can also remove the gmsh from the requirements so that it doesn't trigger the installation of gmsh and its dependencies, no?

Technically we still have code that depends on gmsh, associated with the structural mechanics simulator. However, since that submodule is still under development, it should be fine to remove gmsh from the requirements. The people developing that submodule (@Joao97ribeiro @baribeiro) should be aware that they need to install gmsh anyway to keep developing.

But we can also remove the gmsh from the requirements so that it doesn't trigger the installation of gmsh and its dependencies, no?

Technically we still have code that depends on gmsh, associated with the structural mechanics simulator. However, since that submodule is still under development, it should be fine to remove gmsh from the requirements. The people developing that submodule (@Joao97ribeiro @baribeiro) should be aware that they need to install gmsh anyway to keep developing.

OK! Once we get the new version out in pypi I will test it.
Also: we need to start having a plan for 3D object manipulation.

I removed gmsh and meshio from the requirements.

I removed gmsh and meshio from the requirements.

To remove them from the dependencies that pip installs, you should remove them from setup.cfg

I removed gmsh and meshio from the requirements.

To remove them from the dependencies that pip installs, you should remove them from setup.cfg, otherwise they will still be installed. To avoid confusion later, we can change the requirements.txt to just install "-e ."

Oh, wasn't aware of this... this is not respecting the DRY principle (in a quite important place), because we have two files saying the "same thing" and it's quite error prone... what do we need to do to fix this?

I removed gmsh and meshio from the requirements.

To remove them from the dependencies that pip installs, you should remove them from setup.cfg, otherwise they will still be installed. To avoid confusion later, we can change the requirements.txt to just install "-e ."

Oh, wasn't aware of this... this is not respecting the DRY principle (in a quite important place), because we have two files saying the "same thing" and it's quite error prone... what do we need to do to fix this?

Change the requirements.txt to install "-e .", and leave the setup.cfg with the dependencies of the package.

Ok, great, do you mind doing a small Pull Request with that change? It's very important to remove barriers of entry in the earliest possible interactions with the package, e.g. installation time, or import time.

Ok, great, do you mind doing a small Pull Request with that change? It's very important to remove barriers of entry in the earliest possible interactions with the package, e.g. installation time, or import time. And if developers believe that removing the entry from requirements.txt is enough, we will be under the illusion of fixing something, but the problem is still there... Thanks!!

#538, but this one should be merged first because the branch I'm using in that PR was created from this one, so it includes all of these changes.


Make Table Sortable

Notifications Table should be sortable.

Hey @asseti6 , thanks for the PR.
Have you tested it on your site? Did it work?
Just tried it out on my side and the table didn't load the results anymore. Didn't had time yet to debug why.

@chrisophrumpel I did, however our instance is highly modified and this was a quick fix.  I'll pull down a clean branch this evening and debug.

Thanks 👍

Any news here?


Build Window / Build APPX Fails cannot find Microsoft.Cpp.props

MRTK 2.3.0
Unity 2019.3.3f1
Visual Studio 2019 v16.4.5 (with MSBuild installed v<IP_ADDRESS>4)

====
Unity builds the Unity Project fine, but when it tries to compile the VS Project to build the appx it fails to find the below file:
3>C:\Users\napla\Desktop\MRTK2Play\Builds\WSAPlayer\Il2CppOutputProject\Il2CppOutputProject.vcxproj(3955,3): error MSB4019: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\VC\VCTargets\Microsoft.Cpp.props" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk.
UnityEngine.Debug:Log(Object)
Microsoft.MixedReality.Toolkit.<>c__DisplayClass1_0:<StartProcessAsync>g__OnOutputDataReceived|2(Object, DataReceivedEventArgs) (at Assets/MixedRealityToolkit/Extensions/ProcessExtensions.cs:104)
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback()

File does not exist in that directory.
I have created that directory and added the file and then a new error:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\VC\VCTargets\Microsoft.Cpp.props(29,5): error MSB4186: Invalid static method invocation syntax: "[Microsoft.Build.Utilities.ToolLocationHelper]::FindRootFolderWhereAllFilesExist($(_VCTargetsPathFolders), $(_RelativeToolsetFiles))". Method 'Microsoft.Build.Utilities.ToolLocationHelper.FindRootFolderWhereAllFilesExist' not found. Static method invocation should be of the form: $([FullTypeName]::Method()), e.g. $([System.IO.Path]::Combine(`a`, `b`)).  [C:\Users\napla\Desktop\MRTK2Play\Builds\WSAPlayer\Il2CppOutputProject\Il2CppOutputProject.vcxproj]
UnityEngine.Debug:Log(Object)
Microsoft.MixedReality.Toolkit.<>c__DisplayClass1_0:<StartProcessAsync>g__OnOutputDataReceived|2(Object, DataReceivedEventArgs) (at Assets/MixedRealityToolkit/Extensions/ProcessExtensions.cs:104)
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback()

It appears that $(VCTargetsPath) is not resolving correctly when run from Unity.
If I open the generated VS Project I am able to build fine.
I checked and $(VCTargetsPath) resolves to C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Microsoft\VC\v160 on my machine. And indeed the file in question exists there.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Exec tachikoma update<PHONE_NUMBER>4708

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 67.647% when pulling 994d28f57b86ea6929f4095fadd8bac66eea28e8 on pazjacket:tachikoma/update-20171219214708 into ec251e61adc687ae0589e5283f1773ece07521c0 on sanemat:master.


Preformatted Markdown is split up (even when it doesn't contain spaces

Description
When using preformatted text, such as
`UIViewRepresentable`

the Markdown parser that DocC uses can split it up:

Checklist

[ ] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue.

Expected Behavior
Do not split up preformatted text.
Actual behavior
Here is the code that is generated:

To use MapLibre ... (using UIView<wbr data-v-05f4a5b7="">Representable. The...

Steps To Reproduce

Include pre-formatted text near the end of a line
Watch it get split up!

Swift-DocC Version Information
No response
Swift Compiler Version Information
swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.56 clang-15<IP_ADDRESS>.5)
Target: arm64-apple-macosx14.0

DocC itself generates a single inline code element for this. I suspect that it's DocC-Render who divides the word on capitalized letters to allow it to span multiple lines.

I see! Thanks for transferring it.


how to return different values from different tables?

based on the readme example, it always resolves the same values regardless of what table name is passed in the command, how do I resolve values based on table names?

From README:
Specify mock behavior on receiving given command with given payload only:
snsMock
    .on(PublishCommand, {
        TopicArn: 'arn:aws:sns:us-east-1:111111111111:MyTopic',
        Message: 'My message',
    })
    .resolves({
        MessageId: '12345678-4444-5555-6666-111122223333',
    });

So for DynamoDB, it will be something like:
import { DynamoDBClient, GetItemCommand } from "@aws-sdk/client-dynamodb";

const dynamodbMock = mockClient(DynamoDBClient);

dynamodbMock
    .on(GetItemCommand, {
        TableName: 'myTable1'
    })
    .resolves({
        ... // data from table 1
    })
    .on(GetItemCommand, {
        TableName: 'myTable2'
    })
    .resolves({
        ... // data from table 2
    });


Mention default type of Service in “Defining a Service” section

Fixes #41839

Preview page: https://deploy-preview-41855--kubernetes-io-main-staging.netlify.app/docs/concepts/services-networking/service/#defining-a-service

@dipesh-rawat ,
Thanks for your review.
Can anyone else please approve this PR!!

Assigning to this week's PR wrangler Karen to move this forward.
/assign @kbhawkey

Updated preview: https://deploy-preview-41855--kubernetes-io-main-staging.netlify.app/docs/concepts/services-networking/service/#defining-a-service

/lgtm

/label tide/merge-method-squash

Thanks!
/approve


Cells with HTML  tags do not appear in filter choices

(Original issue 28 created by {u'display_name': u'altmany', u'account_id': None} on 2012-09-30T10:31:53+00:00)
What steps will reproduce the problem?
1. create a JTable with a cell that includes "<html><img src='image.gif'>"
(i.e., just an icon image, no text)
2. filter the table; try to select the image rows from the filter

What is the expected output? What do you see instead?
I expect to see the cell image in the list of filter options. Instead, I see nothing
- I cannot filter for rows that contain images.

What version of the product are you using? On what operating system?
4.4

Please provide any additional information below.
This is a direct consequence of the fix for issue #23 - If the cell contents are entirely
within HTML tags, then the filter disregards the cell and the user cannot filter accordingly.
HTML <Img> tags are a special case that should get special treatment.

Original issue reported on code.google.com by altmany on 2012-09-30 10:31:53

(Original issue 28 last updated on 2012-09-30T10:31:53+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2012-10-03T01:51:02+00:00)
Hi,

There is basically to behaviors associated to each filter editor in the filter header:
text-oriented editors perform text searches, implementing logic such as wildcards and
comparison operators. The implementation done for issue 23 implied extracting the valid
HTML information from the content, which by default is to exclude those HTML tags -this
behavior can be modified by providing a different Parser implementation, by the way-.

The other type of filter editors is choice-oriented, where the user can only choose
one option from the associated combobox. In this case, what is already possible is
to define a net.coderazzi.filters.gui.ChoiceRenderer for that column. Note that a basic
implementation for this Renderer, using a JLabel would already show perfectly your
image -or any HTML content- (well in so far as the JLabel does it correctly).

Two caveats:
1- The height of the editor is fixed (larger ones can be created by modifying the TableFilter'
settings). Care must be taken to have proper renderers.
2- The default implementation will still be accessing the content as text: that is,
not because the library detects HTML in the column, it should be expected to use HTML
renderers. That is up to the user.

So, in summary, I believe your issue can be solved by setting a ChoiceRenderer that
delegates into a JLabel to render the HTML content -and such renderer could be, indeed,
part of the FilterHeader library-.

What do you think?

Cheers,

 Lu.

Original issue reported on code.google.com by coderazzi on 2012-10-03 01:51:02

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2012-10-03T01:51:36+00:00)
Hi,

There are basically two behaviors associated to each filter editor in the filter header:
text-oriented editors perform text searches, implementing logic such as wildcards and
comparison operators. The implementation done for  issue 23  implied extracting the
valid HTML information from the content, which by default is to exclude those HTML
tags -this behavior can be modified by providing a different Parser implementation,
by the way-.

The other type of filter editors is choice-oriented, where the user can only choose
one option from the associated combobox. In this case, what is already possible is
to define a net.coderazzi.filters.gui.ChoiceRenderer for that column. Note that a basic
implementation for this Renderer, using a JLabel would already show perfectly your
image -or any HTML content- (well in so far as the JLabel does it correctly).

Two caveats:
1- The height of the editor is fixed (larger ones can be created by modifying the TableFilter'
settings). Care must be taken to have proper renderers.
2- The default implementation will still be accessing the content as text: that is,
not because the library detects HTML in the column, it should be expected to use HTML
renderers. That is up to the user.

So, in summary, I believe your issue can be solved by setting a ChoiceRenderer that
delegates into a JLabel to render the HTML content -and such renderer could be, indeed,
part of the FilterHeader library-.

What do you think?

Cheers,

 Lu.

Original issue reported on code.google.com by coderazzi on 2012-10-03 01:51:36

(Original comment by {u'display_name': u'altmany', u'account_id': None} on 2012-10-03T23:04:52+00:00)
Sounds like a good plan. But why not automatically use the HTML renderer if <html> is
detected in 1+ cell(s) of the column? I think this would be more intuitive. Of course
the user can always switch between text/HTML/custom renderers, I'm just saying regarding
the default behavior.

Original issue reported on code.google.com by altmany on 2012-10-03 23:04:52

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2012-10-04T00:49:24+00:00)
I think that letting the developer choosing how to handle the column is better, also
from a performance point of view.
Furthermore, the header handles automatically changes in the table, so how to react
if one cell in the column has HTML info, and after an update, it doesn't?
It is not just a matter of handling the content differently, but the user interaction
would change as well (with a renderer, the user cannot enter free text on the editor).

Finally, although the solution of using a renderer would solve the problem of handling
images:
1- Perhaps it is still better searching for text.
2- I am not confident on how the rendering of the cell will look on the editor. That
is, the rendering will show the image, but obvious work like fitting the content to
the editor cell will be missing...

Cheers!

Original issue reported on code.google.com by coderazzi on 2012-10-04 00:49:24

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2012-11-18T00:54:22+00:00)
As proposed, a new renderer called HtmlChoiceRenderer can be setup on the columns with
html content.
Code included already on the repository, will be included on release 4.5.0

Original issue reported on code.google.com by coderazzi on 2012-11-18 00:54:22


Bug :: Problem will come multiple image uploads

@ayush013 thanks for this package. I implemented quill image upload by following this package, i found one issue with this library. you are storing the current handlerId as

this.handlerId = this.helpers.generateID();
https://github.com/ayush013/ngx-quill-upload/blob/master/src/handlers/BaseHandler.ts#L87>
if you try to upload multiple images this.handlerId will reference the latest generated id. You can see this by reducing the connection speed in dev tools to slow 3g so that it can be debugged.

Hi, thanks for reporting. i just checked and seems like the reference is getting updated. I will see what can be done


MemorySanitizer: use-of-uninitialized-value in deflate_medium.c:289

==4810==WARNING: MemorySanitizer: use-of-uninitialized-value
#0 0x4b09a7 in deflate_medium /home/spop/s/zlib-ng/deflate_medium.c:289:21
#1 0x4a181d in zng_deflate /home/spop/s/zlib-ng/deflate.c:951:18
#2 0x497da9 in zng_compress2 /home/spop/s/zlib-ng/compress.c:59:15
#3 0x4970cc in check_compress_level /home/spop/s/zlib-ng/test/fuzz/compress_fuzzer.c:18:3
#4 0x496cc2 in LLVMFuzzerTestOneInput /home/spop/s/zlib-ng/test/fuzz/compress_fuzzer.c:70:3
#5 0x496840 in main /home/spop/s/zlib-ng/test/fuzz/standalone_fuzz_target_runner.c:23:5

I think this is duplicate of previous bug... s->strstart is uninitialized...

You are right. It happens in the same function as bug #186 in a very similar piece of code as in deflate_medium.c:259.

s->strstart should be initialized inside lm_init() which gets called in deflateReset() and indirectly from deflateInit2_().


Add CI 1.47 RN.

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

Please describe your changes: CI 1.47 RN
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://66ec4e558ec8bf0ddbd186ea--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://66ec557e2bb4562106cd2258--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://66ed89e31e173291a6987cd0--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://66ee0a97ef2baf8cb423fa44--harness-developer.netlify.app


OIDC Integration (OpenID) example

Hello,
Because almost all pull requests to this repo stay ignored by maintainer for a long time, I'm creating this as issue, that may helpful for someone else. This is PoC integration oauth2-proxy, that allow to use OpenId providers (list can be found at https://oauth2-proxy.github.io/oauth2-proxy/docs/configuration/oauth_provider) with this app. PoC uses docker-compose, but can easy back ported to k8s or etc.
This is steps works on current master revision.

Application build should be fixed by merging this: https://github.com/timeoff-management/timeoff-management-application/pull/531/files
If you want to use mysql 8+, then next patch should be applied:

Index: lib/model/db/company.js
===================================================================
diff --git a/lib/model/db/company.js b/lib/model/db/company.js
--- a/lib/model/db/company.js	(revision e0ca678927862600b65d22e090be92f608373f7f)
+++ b/lib/model/db/company.js	(date<PHONE_NUMBER>019)
@@ -84,7 +84,7 @@
       comment      : "Indicate which mode the company account is in.",
     },
     timezone : {
-      type         : DataTypes.TEXT,
+      type         : DataTypes.STRING,
       allowNull    : true,
       defaultValue : 'Europe/London',
       comment      : 'Timezone current company is located in',

File lib/middleware/oidc_handler.js should be created:

"use strict";

module.exports = async function (req, res, next) {
    const oidcEmail = req?.headers?.["x-forwarded-email"].toLowerCase();
    const oidcName = req?.headers?.["x-forwarded-user"];
    const oidcGroups = req?.headers?.["x-forwarded-groups"]; //May be used to match with DepartmentId
    const models = req.app.get('db_model');

    if (!req.user) {
        let user = await models.User.find_by_email(oidcEmail);
        if (user === null) {
            const newUser = {
                email: oidcEmail,
                lastname: oidcName,
                name: oidcName,
                companyId: 1, //TODO: move to config
                DepartmentId: 1, //TODO: map to oidcGroups
                password: "null", // it should be string, not null
                admin: false,
                auto_approve: false,
                end_date: null,
                start_date: new Date(),
            }
            await models.User.create(newUser);
            user = await models.User.find_by_email(oidcEmail);
        }
        req.user = await user.reload_with_session_details();
    }

    next();
};

Apply patch:

Index: app.js
===================================================================
diff --git a/app.js b/app.js
--- a/app.js	(revision e0ca678927862600b65d22e090be92f608373f7f)
+++ b/app.js	(date<PHONE_NUMBER>183)
@@ -33,8 +33,6 @@
 app.use(cookieParser());
 app.use(express.static(path.join(__dirname, 'public')));

-
-
 // Setup authentication mechanism
 const passport = require('./lib/passport')();

@@ -44,6 +42,7 @@
 app.use(passport.initialize());
 app.use(passport.session());

+app.use( require('./lib/middleware/oidc_handler') );

 // Custom middlewares

docker-compose.yaml example for simples https://oauth2-proxy.github.io/ integration:

version: '3.8'
services:
  app:
    build: .

  oauth2-proxy:
    image: bitnami/oauth2-proxy
    depends_on:
      - app
    ports:
      - '3000:3000'
    command:
      - oauth2-proxy
      - --upstream=http://app:3000/
      - --http-address=:3000
      - --scope=email read_api read_user openid profile
      - --cookie-secure=true
      - --provider=gitlab
      - --email-domain=*
      - --oidc-issuer-url=https://gitlab.example.com
      - --redirect-url=http://localhost:3000/oauth2/callback
       # generate cookie-secret according to https://oauth2-proxy.github.io/oauth2-proxy/docs/configuration/overview#generating-a-cookie-secret
      - --cookie-secret=change_me
      - --client-id=change_me_to_gitlab_app_id
      - --client-secret=change_me_to_gitlab_app_secret

  db:
    image: mysql:8
    command:
      - mysqld
      - --default-authentication-plugin=mysql_native_password
    environment:
      MYSQL_ROOT_PASSWORD: 'password'
      MYSQL_PASSWORD: 'password'
      MYSQL_USER: 'user'
      MYSQL_DATABASE: 'timeoff'

replace DB config in config/db.json, and start app
login directly to timeoff-management app and create first campaign
on http://localhost:3000 you should see oidc proxy login page

Hope this project will be supported again. Big thanks to authors.

@vsychov Thanks for sharing this. I'm trying to get it working with azure ad but I'm running into a problem with oidc_handler.js.
I'd really appreciate it if you could look at the below and point to what I'm doing wrong.
I'm not sure if the dependencies have changed since you originally did the design but I had to edit the Dockerfile to make it build:
FROM node:18-alpine

EXPOSE 3000

LABEL org.label-schema.schema-version="1.3.0"
LABEL org.label-schema.docker.cmd="docker run -d -p 3000:3000 --name timeoff-management"

RUN apk update
RUN apk upgrade
#Install dependencies
RUN apk add \
    git \
    make \
    python3 \
    g++ \
    gcc \
    libc-dev \
    clang

#Add user so it doesn't run as root
RUN adduser --system app --home /app
USER app
WORKDIR /app

#clone app
RUN git clone https://github.com/timeoff-management/application.git timeoff-management

WORKDIR /app/timeoff-management

#Add in OIDC integration
COPY oidc_handler.js /app/timeoff-management/lib/middleware/oidc_handler.js
COPY app.js /app/timeoff-management/

RUN rm package-lock.json

#Update some dependencies
RUN sed -i 's/formidable"\: "~1.0.17/formidable"\: "1.1.1/' package.json
RUN sed -i 's/sqlite3"\: "^4.0.1/sqlite3"\: "^5.1.5/' package.json
RUN sed -i 's/node-sass"\: "^4.5.3/node-sass"\: "^7.0.3/' package.json
RUN sed -i 's/graceful-fs"\: "^4.4.2/graceful-fs"\: "4.4.2/' package.json

#install app
RUN npm install -y

CMD npm start

For simplicities sake I'm sticking with SQLite so haven't applied the MySQL patch.
The container runs but when you access the login page (original inbuilt one) it doesn't load and crashes the container. The logs show the following:
/app/timeoff-management/lib/middleware/oidc_handler.js:4
    const oidcEmail = req?.headers?.["x-forwarded-email"].toLowerCase();
                                                         ^

TypeError: Cannot read properties of undefined (reading 'toLowerCase')
    at module.exports (/app/timeoff-management/lib/middleware/oidc_handler.js:4:58)
    at Layer.handle [as handle_request] (/app/timeoff-management/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/app/timeoff-management/node_modules/express/lib/router/index.js:328:13)
    at /app/timeoff-management/node_modules/express/lib/router/index.js:286:9
    at Function.process_params (/app/timeoff-management/node_modules/express/lib/router/index.js:346:12)
    at next (/app/timeoff-management/node_modules/express/lib/router/index.js:280:10)
    at strategy.pass (/app/timeoff-management/node_modules/passport/lib/middleware/authenticate.js:325:9)
    at SessionStrategy.authenticate (/app/timeoff-management/node_modules/passport/lib/strategies/session.js:71:10)
    at attempt (/app/timeoff-management/node_modules/passport/lib/middleware/authenticate.js:348:16)
    at authenticate (/app/timeoff-management/node_modules/passport/lib/middleware/authenticate.js:349:7)
    at Layer.handle [as handle_request] (/app/timeoff-management/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/app/timeoff-management/node_modules/express/lib/router/index.js:328:13)
    at /app/timeoff-management/node_modules/express/lib/router/index.js:286:9
    at Function.process_params (/app/timeoff-management/node_modules/express/lib/router/index.js:346:12)
    at next (/app/timeoff-management/node_modules/express/lib/router/index.js:280:10)
    at initialize (/app/timeoff-management/node_modules/passport/lib/middleware/initialize.js:53:5)
    at Layer.handle [as handle_request] (/app/timeoff-management/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/app/timeoff-management/node_modules/express/lib/router/index.js:328:13)
    at /app/timeoff-management/node_modules/express/lib/router/index.js:286:9
    at Function.process_params (/app/timeoff-management/node_modules/express/lib/router/index.js:346:12)
    at next (/app/timeoff-management/node_modules/express/lib/router/index.js:280:10)
    at Model.<anonymous> (/app/timeoff-management/node_modules/express-session/index.js:506:7)
    at Model.tryCatcher (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/util.js:16:23)
    at Promise.successAdapter [as _fulfillmentHandler0] (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/nodeify.js:23:30)
    at Promise._settlePromise (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:601:21)
    at Promise._settlePromise0 (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:729:18)
    at _drainQueueStep (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/async.js:93:12)

Node.js v18.19.0

If I take out .toLowerCase(); like so: const oidcEmail = req?.headers?.["x-forwarded-email"]; //.toLowerCase(); I get email cannot be null error as follows:
><EMAIL_ADDRESS>start
> node bin/wwww

node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^
Error [SequelizeValidationError]: notNull Violation: email cannot be null,
notNull Violation: name cannot be null,
notNull Violation: lastname cannot be null
    at /app/timeoff-management/node_modules/sequelize/lib/instance-validator.js:74:14
    at tryCatcher (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:547:31)
    at Promise._settlePromise (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:604:18)
    at Promise._settlePromise0 (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:729:18)
    at Promise._fulfill (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:673:18)
    at PromiseArray._resolve (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise_array.js:127:19)
    at PromiseArray._promiseFulfilled (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise_array.js:145:14)
    at Promise._settlePromise (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:609:26)
    at Promise._settlePromise0 (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/promise.js:729:18)
    at _drainQueueStep (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/async.js:93:12)
    at _drainQueue (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/async.js:86:9)
    at Async._drainQueues (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/async.js:102:5)
    at Async.drainQueues [as _onImmediate] (/app/timeoff-management/node_modules/sequelize/node_modules/bluebird/js/release/async.js:15:14)
    at process.processImmediate (node:internal/timers:476:21) {
  errors: [
    {
      message: 'email cannot be null',
      type: 'notNull Violation',
      path: 'email',
      value: null
    },
    {
      message: 'name cannot be null',
      type: 'notNull Violation',
      path: 'name',
      value: null
    },
    {
      message: 'lastname cannot be null',
      type: 'notNull Violation',
      path: 'lastname',
      value: null
    }
  ]
}

Node.js v18.19.0

I'm not really sure where to go from here.

@pwpbarney, it's simple, it seems you don't have the x-forwarded-email header, which should always be included, https://github.com/oauth2-proxy/oauth2-proxy/blob/5e30a6fe948fc470108d9e522fe00ea656c94785/docs/docs/configuration/overview.md?plain=1#L145


global variable should be set with configurable: false

In javascript, when a variable is declared in global scope, it will be set to global object as member with configurable: false as well.  Sval doesn't follow this rule thoroughly.
Test case:
const Sval = require('sval')
const interpreter = new Sval({ ecmaVer: 8 })

interpreter.run(`
    var x = 4;

    console.log('configurable?: ', Object.getOwnPropertyDescriptor(window, 'x').configurable);
    console.log('in operator?:', 'x' in window); // correct
    console.log('before deleting:', x)
    console.log('deleted by window.x?:', delete window.x); // incorrect
    console.log('after deleted:', x)

    exports.end = true;
`)

module.exports = 'End: ' + interpreter.exports.end

Expected result:
configurable?:  false
in operator?: true
before deleting: 4
deleted by window.x?: false
after deleted: 4

Actual result:
configurable?:  true
in operator?: true
before deleting: 4
deleted by window.x?: true
after deleted: 4

Cause:

Maybe it should be changed to something like this:
Object.defineProperty(win, name, {
   value,
   writable: true,
   enumerable: true,
   configurable:false, // can omit
});

By the way, it seems sval works in strict mode,  so the expected behavior is throwing TypeError when deleting un-configurable property instead of returning false


Use of TestKit should not impose an slf4j logger binding

This is PITA for testkit users, if the testkit jar happens to be earlier on the classpath than the binding you actually want.
I've spent a bit of time looking into this and it seems that the best solution would be to change all references or slf4j in the generated JAR to use the Gradle logging API directly, and not emit Gradle's StaticLoggerBinder from the generated JAR.

It seems to be related to #2030 (and still nice to have).

This is still a problem. I just hit this again on another project.


Merging development back to master

Informational text for BHS

Coverage remained the same at 97.54% when pulling 27a0e7fb5e3ba21f82e9785a194b45be5c6f916b on development into f809315ec9c41826fbef3cd4dad24670d5fde3fd on master.

Coverage remained the same at 97.54% when pulling 27a0e7fb5e3ba21f82e9785a194b45be5c6f916b on development into f809315ec9c41826fbef3cd4dad24670d5fde3fd on master.


Support different domains config at runtime

Describe the feature
One build to rule them all!
A good practice is to be able to deliver a project on a staging or prod environment just but changing some environment variables. It would be nice to have the ability to set the related domain for a locale at runtime level.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide (The contribution guideline of nuxt-modules/i18n is compliant with Nuxt too).
[X] Check existing discussions and issues.

I also need this behavior and have a WIP patch. It allows overriding a locale's domain at runtime by setting the NUXT_PUBLIC_I18N_LOCALES_{locale}_DOMAIN environment variable.

Resolved by #2446!

I think I missed something in the setup or maybe it's not working at all.
When I build my project with an certain .env file, domain values are not updated when I run it with another .env file.
To reproduce:
1/ edit your /etc/hosts with at least the following content to be able to have different local domains
<IP_ADDRESS>	localhost local-dev-en local-dev-nl local-dev-ja local-prod-en local-prod-nl local-prod-ja

2/ clone the project https://stackblitz.com/edit/bobbiegoede-nuxt-i18n-starter-jvph93?file=.template.env and yarn dev with the following .env
NUXT_PUBLIC_I18N_LOCALES_EN_DOMAIN=local-dev-en:3000
NUXT_PUBLIC_I18N_LOCALES_NL_DOMAIN=local-dev-nl:3000
NUXT_PUBLIC_I18N_LOCALES_JA_DOMAIN=local-dev-ja:3000

Everything works fine, the domain is correctly mapped to the locale
3/ with the same .env file, yarn build and then run it with the .env
In Ubuntu I use this command set -a && source .env && set +a && node .output/server/index.mjs
Everything works fine as in dev mode.
4/ update the .env file with the following values
NUXT_PUBLIC_I18N_LOCALES_EN_DOMAIN=local-prod-en:3000
NUXT_PUBLIC_I18N_LOCALES_NL_DOMAIN=local-prod-nl:3000
NUXT_PUBLIC_I18N_LOCALES_JA_DOMAIN=local-prod-ja:3000

5/ run the previous build with the new values without rebuilding it
run set -a && source .env && set +a && node .output/server/index.mjs again.
The new domains are injected into the runtime config but not into the i18n module domain detection, it keeps the the env from the build time.
What am I missing? :thinking:

I think it needs to be reopened or to provide a fully working project

I suspect this issue is not fixed.
I've opened a new ticket https://github.com/nuxt-modules/i18n/issues/2602


No Tasks Found There are no tasks in the queue.

hello, the node blocks the display of images in confortui see crashes the browser, tested with chrome and firefox.
No Tasks Found
There are no tasks in the queue.

Hi the issue is there when opening the queue. Thanks for reporting.
As a temporary workaround, if you, can keep the queue/iage list closed to avoid the crash.
As I'm not getting any error / exceptions in the console it's quite hard to understand what's going on but I'm investigating.

yes it is a solution, on the other hand a lot of node style
pythongosssss also blocks the browser when using compositor, it's a shame because this node is really handy.

so, not entirely sure I can fix this.
it looks like the problem is in comfyui (or related to the node interaction with latest fromntend in comfy ui),
if you start with the legacy frontend there seems to be no issue:
https://github.com/comfyanonymous/ComfyUI#accessing-the-legacy-frontend
to start with legacy frontend use
as commandline argument --front-end-version Comfy-Org/ComfyUI_legacy_frontend@latest
I will keep on investigating. at the root I think the problem lies with stopping the current flow with /interrupt api endpoing, some thread might be stuck but there is no error in the logs. also it might be due to some frontend stuff but again, not sure yet

I'm still investigating what's causing the issue and it's cryptic.
for the time being I'm closing this.
if you don't want the issue just use the legacy toolbars until they are out of beta.


npm run build is running out of memory

Describe the bug
 Running 'npm run build' in /root/***/kafka-ui-react-app
[INFO]
[INFO] ><EMAIL_ADDRESS>build /root/***/kafka-ui-react-app
[INFO] > react-scripts build
[INFO]
[INFO] Creating an optimized production build...
[INFO] The build failed because the process exited too early. This probably means the system ran out of memory or someone called `kill -9` on the process.
[INFO] The build failed because the process exited too early. This probably means the system ran out of memory or someone called `kill -9` on the process.npm ERR! code ELIFECYCLE
[INFO] npm ERR! errno 1
[INFO] npm ERR<EMAIL_ADDRESS>build: `react-scripts build`
[INFO] npm ERR! Exit status 1
[INFO] npm ERR!
[INFO] npm ERR! Failed at the<EMAIL_ADDRESS>build script.
[INFO] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[INFO]
[INFO] npm ERR! A complete log of this run can be found in:
[INFO] npm ERR!     /root/.npm/_logs/2021-08-26T20_40_19_667Z-debug.log

How much memory does the build need? I am running with 4GB of RAM and it doesn't seem to be enough.

when trying to build locally on my personal win10 laptop:
800MB were used for npm i
npm run build failed with:
.\src\redux\reducers\clusters\selectors.ts
Cannot find module: 'generated-sources'. Make sure this package is installed.

You can install this package by running: npm install generated-sources.

which installing didnt work with cause of npm ERR! 404 Not Found - GET https://registry.npmjs.org/generated-sources - Not found

@GneyHabub could you take a look at this please?

@jhonDoe15 After you've done npm i, you should generate some files with npm run gen-sources command

Didn't the comment saying running: npm install generated-sources. specified it ran it by itself?

@jhonDoe15  generated-sources is a directory, not a package. It contains some API-related classes that are supposed to be generated with the OpenAPI generator. npm mistakenly took it for a package

tried running npm i again followed by the suggested command, then running npm run build and got similar (might be same same) error:
kafka-ui\kafka-ui-react-app>npm run build

><EMAIL_ADDRESS>build
> react-scripts build

Creating an optimized production build...
Failed to compile.

.\src\redux\reducers\clusters\selectors.ts
Cannot find module: 'generated-sources'. Make sure this package is installed.

You can install this package by running: npm install generated-sources.

Terminate batch job (Y/N)? y

on branch origin/master

@jhonDoe15 well, yes, openapi-generator uses Java


Lua mod support

While Java is our main coding language, support for Lua mods (or parts of a mod) would be good if it is possible to do in a nice way.

This is now given very low priority, giving priority to #180 instead.

Considering the power of the new JS-modding, I don't think this is going to happen.
If anyone wants to see this happen, add a comment.


Add a "." separator to datatype tags

This should protect against accidental collisions.
Without a separator, Foo.BarBaz and FooBar.Baz would end up with the
same tag (FooBarBaz).

Codecov Report

Merging #127 into lablua will increase coverage by <.01%.
The diff coverage is 94.57%.

@@            Coverage Diff             @@
##           lablua     #127      +/-   ##
==========================================
+ Coverage   93.37%   93.37%   +<.01%
==========================================
  Files          12       12
  Lines        2203     2204       +1
==========================================
+ Hits         2057     2058       +1
  Misses        146      146

Impacted Files
Coverage Δ

titan-compiler/parser.lua
95.49% <100%> (ø)
:arrow_up:

titan-compiler/checker.lua
93% <100%> (ø)
:arrow_up:

titan-compiler/typedecl.lua
93.33% <100%> (+0.47%)
:arrow_up:

titan-compiler/types.lua
83.57% <83.72%> (ø)
:arrow_up:

titan-compiler/coder.lua
95.13% <94.73%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix typo in JestPlatform.md

Summary
Fix incorrect "where" => "were". It is already in another pull request where are requested updates in versioned docs. But it was more than 2 weeks ago, so here are also those changes done.

Codecov Report

Merging #9212 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9212   +/-   ##
=======================================
  Coverage   65.07%   65.07%
=======================================
  Files         281      281
  Lines       11864    11864
  Branches     2921     2920    -1
=======================================
  Hits         7720     7720
  Misses       3515     3515
  Partials      629      629

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 7df4cda...fecce8c. Read the comment docs.


candid-extractor --version

Similar to ic-wasm --version, it would be handy to add support for a candid-extractor --version command line option.

It would be even useful for build scripts that want to pin a precise version of the tool.


Upgrade 0.19 to 0.20 error

After upgrading from 0.19 to 0.20, whenever I try to run the app (iOS) I get a red screen error saying: "SyntaxError /Users/MyName/MyProjectName/node_modules/react-native/Libraries/Components/ActivityIndicatorIOS/ActivityIndicatorIOS.ios.js: Unexpected token (35:4)"
Also in the packager window, I'm getting the errors:
transforming [============================            ] 69% 478/691[BABEL] Note: The code generator has deoptimised the styling of "/Users/MyName/MyProject/node_modules/immutable/dist/immutable.js" as it exceeds the max of "100KB".
transforming [==============================          ] 76% 526/691[BABEL] Note: The code generator has deoptimised the styling of "/Users/MyName/MyProject/node_modules/moment/moment.js" as it exceeds the max of "100KB".
transforming [===================================     ] 87% 603/691[BABEL] Note: The code generator has deoptimised the styling of "/Users/MyName/MyProject/node_modules/rx/dist/rx.all.js" as it exceeds the max of "100KB".
transforming [===================================     ] 87% 604/691[BABEL] Note: The code generator has deoptimised the styling of "/Users/MyName/MyProject/node_modules/react-native-htmlview/vendor/htmlparser2.js" as it exceeds the max of "100KB".
transforming [===================================     ] 88% 605/691Error while persisting cache: SyntaxError /Users/MyName/MyProject/node_modules/react-native/Libraries/JavaScriptAppEngine/Initialization/ExceptionsManager.js: Unexpected token (38:8)

I'm using:
npm - 3.7.3
node - 4.2.3
Any ideas?

@pangers While upgrading, did you run the react-native upgrade command to update some of the project files?

@arjunkomath Yes I have done that aswell when upgrading to 0.20.

Hi @pangers,
You actually have only 1 error:
transforming [=================================== ] 88% 605/691 Error while persisting cache: SyntaxError /Users/MyName/MyProject/node_modules/react-native/Libraries/JavaScriptAppEngine/Initialization/ExceptionsManager.js: Unexpected token (38:8)

I think if you run a packager with --reset-cache flag, that would solve your problem:
./node_modules/react-native/packager/packager.sh start --reset-cache

@Kureev,
Thanks for the suggestion, I gave it a try and I still seem to be having the exact same problem with the same error message..Still unable to upgrade as of yet.

Solved the issue. I stumbled across this thread http://stackoverflow.com/questions/34112110/upgrade-to-react-native-0-16-error?rq=1
It was because I had my own .babelrc file when in fact 0.20.0, if you specify your own .babelrc file, "React Native does not apply its default Babel configuration in this case". So when I deleted the .babelrc, the problems was solved.


Use snakeCase for parameter which compliant to official naming convention

Refer to https://developers.google.com/protocol-buffers/docs/style

Howdy @franky-li,
Can you please provide some more context to this change?
The general idea of this tool is to output a Protobuf spec that matches the inputted OpenAPI spec. If I'm reading this right, this is changing the field names to snake case, which would break that contract.

My use case to switch Restful api to grpc, and restful api will be throw away. so that I would like to generate a protobuf spec which compliant to grpc naming convention.
May I know what is the contract you mean?

The OpenAPI spec is the “contract”.
When people use this tool, the idea is to generate a proto spec that actually matches the OpenAPI spec. If the specs don’t match, this tool isn’t really useful.
Basically, the idea is not to “throw away” the OpenAPI spec.
If you would like your Proto spec to have snake cases fields, I’d suggest you make that change in the OpenAPI spec.
Closing this as we don’t have plans to make this change.


[Button] [ButtonLink] [Link] Support ref on Button / Link components

Description

Using React's ref attribute returns null when used on ButtonLink. Haven't tested on all other Button/Link type components, but I would assume it happens on them as well. It'd be great if these could support the ref attribute [ref={ (input) => { this.button = input } }] so that it's possibly to reference (and focus!) on the button for accessibility purposes.

Reproduction Steps

[ ] Add ButtonLink component to app
[ ] Add ref attribute ex ref={ (input) => { this.button = input } }
[ ] Call this.button
[ ] Observe that it returns null and no reference to DOM element

Version Number of TDS
0.29.0

This recent React RFC for "innerRef" should make this feature much much easier! https://github.com/reactjs/rfcs/pull/17

Hi @heathervv, I can't seem to reproduce this issue. Is this still a problem for you?

@marcod1419 I haven't tested for this bug since March - we either didn't use TDS components if we required this feature, or sacrificed using refs! Sorry but I'm not able to tell you whether or not it's an issue still.

@heathervv Thanks for the quick response! I'll close this issue for now then. If you or your team ever come across it again, feel free to comment on here again and we'll take another look.


ADFS DKM Master Key Export Update

Fixes #

Rule name ADFS-Certificate-Export does not represent the behavior that we are trying to detect with the rule logic
ADFS-Certificate-Export does not trigger (SecurityEvent datasource)

Proposed Changes
1) Rule Name should be changed to ADFS-DKM-Masterkey-Export or ADFS-DKM-GroupKey-Export or ADFS-DKM-EncryptionKey-Export

The token signing and decryption certificates are stored as PFX blobs in the ADFS database
Certificates are encrypted using a technology called Distributed Key Manager (DKM)

ADFS DKM Master Keys are used during the following process:

ADFS Service starts
It reads service settings from the database (Windows Internal Database (WID))  in XML format
As part of the process, it loads ADFS certificate collection (Reads certificate encrypted PFX blob (base64 encoded) from the XML object)
Decodes Base64 string
DKM Key is not used to decrypt certificate. Key is derived  (standard NIST SP 800-108)
Derived key is used and certificate is decrypted

Therefore, the adversary needs the obtain the ADFS DKM Master Key to then use it in a similar way once it obtains a signing certificate. It can then decrypt it and use the signing certificate to sign SAML tokens.
The ADFS DKM master key(s) are stored in Active Directory.
One could read the DKM key as a byte array and convert it to a usable string from AD by running the following command (My lab environment):
$key=(Get-ADObject -filter 'ObjectClass -eq "Contact" -and name -ne "CryptoPolicy"' -SearchBase "CN=ADFS,CN=Microsoft,CN=Program Data,DC=azsentinel,DC=local" -Properties thumbnailPhoto).thumbnailPhoto

[System.BitConverter]::ToString($key)

As you can see, the DKM Container path would be something like this CN=ADFS,CN=Microsoft,CN=Program Data,DC=azsentinel,DC=local . It will have its own GUID unique to the domain environment.
That's it! That's all the adversary would need to do then move to the next steps to decrypt signing certificates to then, once again, sign SAML tokens and use them to authenticate.
As you can see, this rule ONLY focuses on identifying the action of reading/exporting the DKM key and no the ADFS signing certificate. Therefore, I propose the name to be changed.
2) Rule logic (SecurityEvent Table section) needs to be adjusted to identify the right field names where it would find part of the indicators needed to detect someone accessing the DKM key in the DC.
Pre-Requirements:

An Access Control Entry (ACE) in the System Access Control List (SACL) of the DKM container (Active Directory Object)
Rule documentation: https://github.com/OTRF/Set-AuditRule/blob/master/rules/activedirectory/adfs_cryptopolicy_thumbnailphoto.yml
Rule logic:

Set-AuditRule -AdObjectPath 'AD:\CN=CryptoPolicy,CN=ADFS,CN=Microsoft,CN=Program Data,DC=azsentinel,DC=local' -WellKnownSidType NetworkSid -Rights GenericRead -InheritanceFlags None -AuditFlags Success -AttributeGUID '8d3bca50-1d7e-11d0-a081-00aa006c33ed'

First Comments*

Event 4662 does not translate the thumbnailPhoto GUI to the "thumbnailPhoto" string. That needs to be an enrichment
thumbnailPhoto Attribute GUID: 8d3bca50-1d7e-11d0-a081-00aa006c33ed
The attribute is not part of the object name. It is an attribute/property. Therefore, the value would show up in the filed name Properties
We can filter events also by using the ObjectType GUID of class type contact and then look for the thumbnailPhoto GUID value.

Simulation:

** Event 4662 generated** (after setting the SACL)

The highlighted section is the thumbnailPhoto attribute.
XML Representation (This is where we do not get the objectName string and just the GUID
- <Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
- <System>
<Provider Name="Microsoft-Windows-Security-Auditing" Guid="{54849625-5478-4994-a5ba-3e3b0328c30d}" />
<EventID>4662</EventID>
<Version>0</Version>
<Level>0</Level>
<Task>14080</Task>
<Opcode>0</Opcode>
<Keywords>0x8020000000000000</Keywords>
<TimeCreated SystemTime="2020-12-20T07:53:41.092054600Z" />
<EventRecordID>330446</EventRecordID>
<Correlation />
<Execution ProcessID="708" ThreadID="836" />
<Channel>Security</Channel>
<Computer>DC01.azsentinel.local</Computer>
<Security />
</System>
- <EventData>
<Data Name="SubjectUserSid">S-1-5-21-1640822366-3528877384-3060188657-1103</Data>
<Data Name="SubjectUserName">adfsuser</Data>
<Data Name="SubjectDomainName">AZSENTINEL</Data>
<Data Name="SubjectLogonId">0x4235ba</Data>
<Data Name="ObjectServer">DS</Data>
<Data Name="ObjectType">%{5cb41ed0-0e4c-11d0-a286-00aa003049e2}</Data>
<Data Name="ObjectName">%{8cd0a7fa-b3c9-4572-85e5-9359c2783031}</Data>
<Data Name="OperationType">Object Access</Data>
<Data Name="HandleId">0x0</Data>
<Data Name="AccessList">%%7684</Data>
<Data Name="AccessMask">0x10</Data>
<Data Name="Properties">%%7684 {77b5b886-944a-11d1-aebd-0000f80367c1} {8d3bca50-1d7e-11d0-a081-00aa006c33ed} {5cb41ed0-0e4c-11d0-a286-00aa003049e2}</Data>
<Data Name="AdditionalInfo">-</Data>
<Data Name="AdditionalInfo2" />
</EventData>
</Event>

Running new query

Remember that the DKM container's GUID is unique to the domain. Therefore, the query will need to be adjusted when it is used to provider additional context.
Let me know if you have any questions! Happy to do more testing :)
References
https://www.powershellgallery.com/packages/AADInternals/0.2.3/Content/Export-ADFSSigninCertificate.ps1
https://www.youtube.com/watch?v=5dj4vOqqGZw
https://www.microsoft.com/en-us/research/wp-content/uploads/2014/04/DKM-Verification.pdf
https://msresource.wordpress.com/2016/05/04/the-use-of-distributed-key-manager-dkm-in-active-directory-federation-services-ad-fs/

Nice work @Cyb3rWard0g, love it.


The MixedRealityToolkit has no children and does not report an error

[在此处输入反馈]

Follow the steps of the document, no error is shown, cannot be resolved

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 24d36428-004c-e29f-ac5a-602bcbabdb25
Version Independent ID: 01015fb5-d01b-c738-6671-9a0ca9af000e
Content: 初始化项目并部署第一个应用程序 - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/unity/tutorials/mr-learning-base-02.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

@iluvian MixedRealityToolkit with no child Objects is correct.

@veeruby-developer Can you update the screenshots or send me correct ones?

@iluvian MixedRealityToolkit with no child Objects is correct.
the Screenshots with child object for MixedRealityToolkit is from the older MRTK versions.
The version numbers I used were MRTK2.5.3，Unity 2019.4.8，Is this an older one？

@iluvian  MRTK 2.5.3 is the right version for this tutorial you can proceed with it.


Upload to PyPI

I'm creating the Parquet rows plugin and need this library available on PyPI to do it properly.
As it was not available there and the last commit here is almost 1 year old, I've uploaded it: https://pypi.python.org/pypi/parquet
@jcrobak, if you want access on PyPI, please tell me your username and I can give you.
If the library is not maintained anymore, then probably I'm going to create a fork of it and maintain the version on PyPI with the fork.

Cool! I don't have much time to work on it right now, so I can't promise anything...but would be willing to try to help.

@turicas Did you ever make any progress with a fork?

@peterbe, no, I didn't fork since it worked as expected for me and I don't have too much demand on using Parket files. Do you have any additional needs?

Do you have any additional needs?

Being able to write/create parquet files.

Sorry, I don't have enough time to dig into this kind of development now, since I'm not needing this feature in my current projects.

@turicas could you add me on pypi? I'm jcrobak. Thanks!

@jcrobak, already added! Please go to https://pypi.python.org/pypi/parquet and see the package administration page. :)
Since the package is already on PyPI and you now have access to it, I'm closing this issue.


build and symlink each output of the packages

fixes #306
the output paths will be similar to the regular nix results, the following results are tested with https://github.com/NixOS/nixpkgs/pull/215728
4 packages built:
geogram geogram.bin geogram.dev geogram.lib

$ tree results
results
├── geogram -> /nix/store/jd1ys0jn7sp9bw647ch8ydp5ic3l09g6-geogram-1.8.2
├── geogram.bin -> /nix/store/j64i56kvgg5wp5m1snhxwmm9wcbg2zm1-geogram-1.8.2-bin
├── geogram.dev -> /nix/store/cr6wlvc22gj072r0fw0j52kbmqmpr6my-geogram-1.8.2-dev
└── geogram.lib -> /nix/store/299za097p633g4w1ig63g8k5zcf86nk6-geogram-1.8.2-lib

The result is different.
[davidak@gaming:~/code/nixpkgs]$ nix-build . -A openssl.all
these 3 paths will be fetched (10.30 MiB download, 17.28 MiB unpacked):
  /nix/store/mqrkghijrs8cl35z6pqgm79qljwj24pd-openssl-3.0.8-doc
  /nix/store/qpkc8189fwa37faanivhfz5sdjnsfi33-openssl-3.0.8-man
  /nix/store/v84frgj57lkz1svhlsnbdrd20lgak4ms-openssl-3.0.8-debug
copying path '/nix/store/qpkc8189fwa37faanivhfz5sdjnsfi33-openssl-3.0.8-man' from 'https://cache.nixos.org'...
copying path '/nix/store/v84frgj57lkz1svhlsnbdrd20lgak4ms-openssl-3.0.8-debug' from 'https://cache.nixos.org'...
copying path '/nix/store/mqrkghijrs8cl35z6pqgm79qljwj24pd-openssl-3.0.8-doc' from 'https://cache.nixos.org'...
/nix/store/69cw78hmw7lsgjicxyp7dcky372fb4dy-openssl-3.0.8-bin
/nix/store/81h1lh105i20xyks9qw1c58bim79jmvx-openssl-3.0.8-dev
/nix/store/dsf1m9azqqz6c3nqj9yk0nnardqmaia0-openssl-3.0.8
/nix/store/qpkc8189fwa37faanivhfz5sdjnsfi33-openssl-3.0.8-man
/nix/store/mqrkghijrs8cl35z6pqgm79qljwj24pd-openssl-3.0.8-doc
/nix/store/v84frgj57lkz1svhlsnbdrd20lgak4ms-openssl-3.0.8-debug

[davidak@gaming:~/code/nixpkgs]$ ll
total 84
-rw-r--r--  1 davidak users 8391 Feb 15 18:21 CONTRIBUTING.md
-rw-r--r--  1 davidak users 1097 Feb 15 18:21 COPYING
-rw-r--r--  1 davidak users  971 May 28  2022 default.nix
drwxr-xr-x 12 davidak users 4096 Feb 15 18:21 doc
-rw-r--r--  1 davidak users 2266 Feb 15 18:21 flake.nix
drwxr-xr-x  5 davidak users 4096 Feb 15 18:21 lib
drwxr-xr-x  3 davidak users 4096 Feb 15 18:21 maintainers
drwxr-xr-x  7 davidak users 4096 Feb 15 18:21 nixos
drwxr-xr-x 18 davidak users 4096 May 28  2022 pkgs
-rw-r--r--  1 davidak users 6189 Feb 15 18:21 README.md
lrwxrwxrwx  1 davidak users   57 Feb 15 23:19 result -> /nix/store/dsf1m9azqqz6c3nqj9yk0nnardqmaia0-openssl-3.0.8
lrwxrwxrwx  1 davidak users   61 Feb 15 23:19 result-bin -> /nix/store/69cw78hmw7lsgjicxyp7dcky372fb4dy-openssl-3.0.8-bin
lrwxrwxrwx  1 davidak users   63 Feb 15 23:19 result-debug -> /nix/store/v84frgj57lkz1svhlsnbdrd20lgak4ms-openssl-3.0.8-debug
lrwxrwxrwx  1 davidak users   61 Feb 15 23:19 result-dev -> /nix/store/81h1lh105i20xyks9qw1c58bim79jmvx-openssl-3.0.8-dev
lrwxrwxrwx  1 davidak users   61 Feb 15 23:19 result-doc -> /nix/store/mqrkghijrs8cl35z6pqgm79qljwj24pd-openssl-3.0.8-doc
lrwxrwxrwx  1 davidak users   61 Feb 15 23:19 result-man -> /nix/store/qpkc8189fwa37faanivhfz5sdjnsfi33-openssl-3.0.8-man

Can you make it consistent?
How  can i test this easily?

The result is different.

If you are referring to the change from - to ., this is intentional to make sure the attributes don't conflict with each other, since even though unlikely, it is still technically possible for e.g. openssl-lib to be a valid attribute that evaluates something other than openssl.lib

How can i test this easily?

You can test it on a PR with something like this
nix run github:figsoda/nixpkgs-review/outputs pr 215728

If you want to test it locally, just add a space in one of the phases and run it with wip or rev HEAD

I like having one results directory instead of many cluttering the current one. Maybe Nix should do it this way as well? Do you want to bring your argument also there?

Requesting another review just to triple check that I don't break anything

bors merge


Code

Hi, just read you paper, could you please upload the code accompanying it here?
Best, Lukas

The code has been uploaded, thank you for your patience.


tests: add unit tests for pkg/injector/patch

Description:
Relates to https://github.com/openservicemesh/osm/issues/3432
I've added missing unit tests for pkg/injector/patch. Thank you for your review!

Affected area:

Functional Area

New Functionality
[ ]

Documentation
[ ]

Install
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Ingress
[ ]

Egress
[ ]

Networking
[ ]

Observability
[ ]

SMI Policy
[ ]

Sidecar Injection
[ ]

Security
[ ]

Upgrade
[ ]

Tests
[X]

CI System
[ ]

Demo
[ ]

Performance
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

No

Is this a breaking change?

No

Codecov Report

Merging #3519 (5bcf6bd) into main (63c2fd2) will increase coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3519      +/-   ##
==========================================
+ Coverage   65.99%   66.19%   +0.20%
==========================================
  Files         169      170       +1
  Lines        7895     8085     +190
==========================================
+ Hits         5210     5352     +142
- Misses       2654     2703      +49
+ Partials       31       30       -1

Flag
Coverage Δ

unittests
66.19% <ø> (+0.20%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/envoy/eds/response.go
37.77% <0.00%> (-32.23%)
:arrow_down:

pkg/catalog/catalog.go
92.85% <0.00%> (-7.15%)
:arrow_down:

pkg/injector/envoy_config.go
90.43% <0.00%> (-4.91%)
:arrow_down:

pkg/injector/envoy_config_health_probes.go
92.62% <0.00%> (-3.09%)
:arrow_down:

pkg/envoy/registry/services.go
81.81% <0.00%> (-3.04%)
:arrow_down:

pkg/envoy/proxy.go
87.32% <0.00%> (-1.91%)
:arrow_down:

cmd/cli/trafficpolicy_check.go
43.95% <0.00%> (-1.88%)
:arrow_down:

pkg/envoy/sds/response.go
88.39% <0.00%> (-1.79%)
:arrow_down:

cmd/cli/install.go
70.79% <0.00%> (-0.76%)
:arrow_down:

pkg/envoy/ads/stream.go
3.72% <0.00%> (-0.71%)
:arrow_down:

... and 12 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 63c2fd2...5bcf6bd. Read the comment docs.

@shashankram That makes sense. Thank you for your clarification!

Thank you for your review. I'll squash the commits now 👍


Getting "The given range isn't in document" error when inserting ""

Steps to reproduce:
On http://neilj.github.io/Squire/, in Chrome:

replace the iframe and the Squire-UI div with <textarea id="foo"></textarea> (i.e., reset the editor to get a handle on it)
in the console, run:

var s = new SquireUI({replace: 'textarea#foo', height: 300});
s.insertHTML('

')

Expected:

no error

Actual:

The given range isn't in document. is logged in the console.

Using

 doesn't raise an error.
This error comes from DOMSelection.addRange and is triggered (AFAICT) when the range's start and end containers aren't "connected", i.e., "if its shadow-including root is [not] a document."
On Squire's side, addRange is called in setSelection.
(the actual error-case was naturally more complex than that but that's what I was able to reduce it to)

It sounds like you might be trying to use Squire with an iframe, and trying to write to the document before the iframe is loaded?
Not sure; SquireUI itself is an unsupported demo integration which I didn't write. If you can give a reproduction path using Demo.html in the main Squire repo I'm happy to take a look.

I am indeed using Squire inside an iframe. But the document is definitely loaded at that time. Hmm… 🤔
I can't reproduce in Demo.html right now 😞  I'll close if I find a way. (seems unlikely)


[language/tools] Add binary flag to disassembler

A simple PR that adds a flag to allow us to disassemble binary files.

Yep! We emit serde from the IR compiler.

Just FYI: Afer some discussion, going to change the default to be binary, and add a --json flag that will disassemble from JSON.

Default serialization format being binary was changed in  PR #2843. Closing for now. Will put up a separate PR for JSON decoding if we think that's useful.


Unsupported operation: x[:, :, 1:, 1:]

@xxradon It gives error in this operation: x = self.conv(x[:, :, 1:, 1:]). The operation x[:, :, 1:, 1:] divides the area of the feature map, which is somewhat similar to slice, but slice is for the channel dimension.
I don't know if there is a corresponding operation in caffe, can you give me some advice
thank you

x[:, :, 1:, 1:] are compound operations.and compound operations
with index or slice operation in caffe is not supportted.so you should split this operation to many small  operations,and make sure small operation is supported by caffe.

@是什么意思？？？没有听明白，你说caff ewith index or slice operation in caffe is not supportted，都不支持，那怎么解决？？？

就是说，这种Python式的index,slice，如x[:, ],x[1:, :]这几种操作，请换成非Python式写法。如torch.index torch.index_slice函数。

tmp = rpn_softmax_scores[:, :, :, :, 1]
x1 = x[:, :(x.shape[1]//2), :, :]
x2 = x[:, (x.shape[1]//2):, :, :]
这种取值都不支持？  这么替换呢

您好 ，，我按照你的方法，用torch.split 的方法替换了[ :, :, :, :, 1]  但是还是出现了问题
TypeError: None has type NoneType, but expected one of: bytes, unicod
可否为了解答一下。。。。在线等。。。


Unhandled exception when updating mod over a manually-updated mod

CKAN Version if known (or site URL)
v1.22.6 (Windows 10 64-bit)
Please describe the expected behaviour
CKAN aborts install with error message and returns to mod-selection window.
Please describe the actual behaviour
CKAN aborts install with error message and throws an Unhandled Exception "Object reference not set to an instance of an object."
Hitting "Continue" appears to resume normal operation of CKAN, but only parts of the GUI are responsive; most of it is unresponsive after this and the app has to be manually killed using Task Manager.
What steps can be taken to reproduce the issue?

Install a mod (in my case, KDEX v1.04)
Manually install an update of the mod
Attempt to update the mod in CKAN

Error message:
About to install...

Downloading "https://github.com/mwerle/kdex/releases/download/v1.10/kdex-1.10.zip"

Oh no!

It looks like you're trying to install a mod which is already installed,
or which conflicts with another mod which is already installed.

As a safety feature, the CKAN will *never* overwrite or alter a file
that it did not install itself.

If you wish to install KDEX v1.10 via the CKAN,
then please manually uninstall the mod which owns:

GameData/masTerTorch/kdex.version

and try again.

Your GameData has been returned to its original state.

Error!

Exception call stack:

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.NullReferenceException: Object reference not set to an instance of an object.
   at CKAN.Main._MarkModForUpdate(String identifier)
   at CKAN.Main.PostInstallMods(Object sender, RunWorkerCompletedEventArgs e)
   at System.ComponentModel.BackgroundWorker.OnRunWorkerCompleted(RunWorkerCompletedEventArgs e)

************** Loaded Assemblies **************
mscorlib
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2110.0 built by: NET47REL1LAST
    CodeBase: file:///C:/Windows/Microsoft.NET/Framework64/v4.0.30319/mscorlib.dll
----------------------------------------
ckan
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 1.22.6
    CodeBase: file:///C:/Games/ckan.exe
----------------------------------------
System.Core
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2102.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
----------------------------------------
System
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2110.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
----------------------------------------
System.Configuration
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
----------------------------------------
System.Xml
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2102.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
----------------------------------------
System.Windows.Forms
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2110.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
----------------------------------------
System.Drawing
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
----------------------------------------
System.Transactions
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_64/System.Transactions/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Transactions.dll
----------------------------------------
System.Numerics
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Numerics/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Numerics.dll
----------------------------------------
System.Runtime.Serialization
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2112.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Runtime.Serialization/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Runtime.Serialization.dll
----------------------------------------
System.Data
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2102.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_64/System.Data/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Data.dll
----------------------------------------
Anonymously Hosted DynamicMethods Assembly
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2110.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_64/mscorlib/v4.0_<IP_ADDRESS>__b77a5c561934e089/mscorlib.dll
----------------------------------------
Microsoft.GeneratedCode
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2102.0 built by: NET47REL1LAST
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
----------------------------------------
Microsoft.CSharp
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Microsoft.CSharp/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Microsoft.CSharp.dll
----------------------------------------
System.Dynamic
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Dynamic/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Dynamic.dll
----------------------------------------
System.EnterpriseServices
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_64/System.EnterpriseServices/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.EnterpriseServices.dll
----------------------------------------
Accessibility
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.7.2046.0 built by: NET47REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
----------------------------------------

************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.

For example:

<configuration>
    <system.windows.forms jitDebugging="true" />
</configuration>

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

I'm not sure why it's throwing an exception, but the behaviour is as expected. You'll need to delete the manually-installed mod before installing from CKAN. CKAN will never overwrite a file it did not install, nor will it overwrite a file it knows belongs to one mod while attempting to install a different mod.

Yes i know it's expected behaviour to not install over the top of a manually modified mod - i was just reporting the exception.

Cool. We seem to have a few unhandled exceptions and crash states that crept in over the last few changes.

Can you repost the details to an issue on the main CKAN repo, since that is a client code issue rather than a metadata issue?

Argh :) Second time I got the wrong repo for this bug report (first time was to a completely unrelated CKAN repo!)
I need more coffee.. thanks, and will do.