Update bitlocker-csp.md

Looks like all the issue tickets (https://github.com/MicrosoftDocs/windows-itpro-docs/issues?utf8=✓&q="1fd9efc2-8946-75e4-27f3-95ac43a9d987"&in=body) connected to this page (https://docs.microsoft.com/en-us/windows/client-management/mdm/bitlocker-csp) are closed already, so I presume this PR is not based on any known existing ticket.

+@Stan <EMAIL_ADDRESS>@Rahul<EMAIL_ADDRESS>I submitted the change for the CSP documentation, but I don’t have an appropriate bug handy.  Can you guys confirm/approve?
Thanks,
Chris
From: Trond B. Krokli<EMAIL_ADDRESS>Sent: Wednesday, December 9, 2020 07:20
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Chris Black (INTUNE<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
Looks like all the issue tickets (https://github.com/MicrosoftDocs/windows-itpro-docs/issues?utf8=✓&q="1fd9efc2-8946-75e4-27f3-95ac43a9d987"&in=bodyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%3Futf8%3D%25E2%259C%2593%26q%3D%25221fd9efc2-8946-75e4-27f3-95ac43a9d987%2522%26in%3Dbody&data=04|01|Christopher.Black%40microsoft.com|db84d18a293e49a5d59608d89c55e3c4|72f988bf86f141af91ab2d7cd011db47|0|0|637431239996497454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=BJgCg5y0Rle3Tho0bTckJFk1mjHq12Qkk5srVhIZy3U%3D&reserved=0) connected to this page (https://docs.microsoft.com/en-us/windows/client-management/mdm/bitlocker-csphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fclient-management%2Fmdm%2Fbitlocker-csp&data=04|01|Christopher.Black%40microsoft.com|db84d18a293e49a5d59608d89c55e3c4|72f988bf86f141af91ab2d7cd011db47|0|0|637431239996507448|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lOO0AsGZpIReTe4%2Fx5Rvkxsi56ZJw%2BhfTz0yEUlXZek%3D&reserved=0) are closed already, so I presume this PR is not based on any known existing ticket.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fpull%2F8775%23issuecomment-741839444&data=04|01|Christopher.Black%40microsoft.com|db84d18a293e49a5d59608d89c55e3c4|72f988bf86f141af91ab2d7cd011db47|0|0|637431239996517441|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=pn13kitHalErVIX%2BLGgzy8SAUa8NOuG%2BimrlyMk%2BoCM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACCSFYHSNVV5554DD23OPTLST6IR3ANCNFSM4UQ5Q6VA&data=04|01|Christopher.Black%40microsoft.com|db84d18a293e49a5d59608d89c55e3c4|72f988bf86f141af91ab2d7cd011db47|0|0|637431239996517441|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=G%2FYKutv4uii8QHxIu%2FOFeaLj34vXgQkXZaJBn%2BqvG3E%3D&reserved=0.

<EMAIL_ADDRESS>I am not sure how can I help here. Do you want us to create a documentation bug?
From: Chris Black (INTUNE<EMAIL_ADDRESS>Sent: Friday, December 11, 2020 11:29 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Stan White<EMAIL_ADDRESS>Rahul Muthoo<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
+@Stan <EMAIL_ADDRESS>@Rahul<EMAIL_ADDRESS>I submitted the change for the CSP documentation, but I don't have an appropriate bug handy.  Can you guys confirm/approve?
Thanks,
Chris
From: Trond B. Krokli <notifications<EMAIL_ADDRESS>Sent: Wednesday, December 9, 2020 07:20
To: MicrosoftDocs/windows-itpro-docs <windows-itpro-docs<EMAIL_ADDRESS>Cc: Chris Black (INTUNE) <Christopher.Black<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
Looks like all the issue tickets (https://github.com/MicrosoftDocs/windows-itpro-docs/issues?utf8=✓&q="1fd9efc2-8946-75e4-27f3-95ac43a9d987"&in=bodyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%3Futf8%3D%25E2%259C%2593%26q%3D%25221fd9efc2-8946-75e4-27f3-95ac43a9d987%2522%26in%3Dbody&data=04|01|ramuthoo%40microsoft.com|884d5173896049d4a4f708d89e0af248|72f988bf86f141af91ab2d7cd011db47|1|0|637433117146523526|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YFS7IFSUp8soAAx5jrrGJoCTGiVZrg97HASZ9OQA2Mk%3D&reserved=0) connected to this page (https://docs.microsoft.com/en-us/windows/client-management/mdm/bitlocker-csphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fclient-management%2Fmdm%2Fbitlocker-csp&data=04|01|ramuthoo%40microsoft.com|884d5173896049d4a4f708d89e0af248|72f988bf86f141af91ab2d7cd011db47|1|0|637433117146523526|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=83k%2F8IttbgoeVj8xfTrttO0B7YZTuuCQ3ORVnK1wALI%3D&reserved=0) are closed already, so I presume this PR is not based on any known existing ticket.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fpull%2F8775%23issuecomment-741839444&data=04|01|ramuthoo%40microsoft.com|884d5173896049d4a4f708d89e0af248|72f988bf86f141af91ab2d7cd011db47|1|0|637433117146533480|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Q5iUrhimoGGcfT4qZaDLcMnCkeKUGe5ga8JVPUX12Mk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACCSFYHSNVV5554DD23OPTLST6IR3ANCNFSM4UQ5Q6VA&data=04|01|ramuthoo%40microsoft.com|884d5173896049d4a4f708d89e0af248|72f988bf86f141af91ab2d7cd011db47|1|0|637433117146533480|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=sYzWNoR%2Bd0CjoYtAlYey66gFUmqrkO0ncdhQV7kEBmw%3D&reserved=0.

I’m not sure either 😊  I hoped if you say “Yes, I own that and the change is correct”, that would either be sufficient or the “windows-itpro-docs” alias could tell us what else they need.
Chris
From: Rahul Muthoo<EMAIL_ADDRESS>Sent: Friday, December 11, 2020 11:45
To: Chris Black (INTUNE<EMAIL_ADDRESS>MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Stan White<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
<EMAIL_ADDRESS>I am not sure how can I help here. Do you want us to create a documentation bug?
From: Chris Black (INTUNE) <Christopher.Black<EMAIL_ADDRESS>Sent: Friday, December 11, 2020 11:29 AM
To: MicrosoftDocs/windows-itpro-docs <reply<EMAIL_ADDRESS>MicrosoftDocs/windows-itpro-docs <windows-itpro-docs<EMAIL_ADDRESS>Stan White <Stan.White<EMAIL_ADDRESS>Rahul Muthoo <ramuthoo<EMAIL_ADDRESS>Cc: Author <author<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
+@Stan <EMAIL_ADDRESS>@Rahul<EMAIL_ADDRESS>I submitted the change for the CSP documentation, but I don’t have an appropriate bug handy.  Can you guys confirm/approve?
Thanks,
Chris
From: Trond B. Krokli <notifications<EMAIL_ADDRESS>Sent: Wednesday, December 9, 2020 07:20
To: MicrosoftDocs/windows-itpro-docs <windows-itpro-docs<EMAIL_ADDRESS>Cc: Chris Black (INTUNE) <Christopher.Black<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Update bitlocker-csp.md (#8775)
Looks like all the issue tickets (https://github.com/MicrosoftDocs/windows-itpro-docs/issues?utf8=✓&q="1fd9efc2-8946-75e4-27f3-95ac43a9d987"&in=bodyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%3Futf8%3D%25E2%259C%2593%26q%3D%25221fd9efc2-8946-75e4-27f3-95ac43a9d987%2522%26in%3Dbody&data=04|01|Christopher.Black%40microsoft.com|22670e5c66e64be982c008d89e0d2fff|72f988bf86f141af91ab2d7cd011db47|0|0|637433126836839634|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Pk7yruMqzu8GCHl0ThI9VAh%2FLK9lcq30Yps7DCK6yUM%3D&reserved=0) connected to this page (https://docs.microsoft.com/en-us/windows/client-management/mdm/bitlocker-csphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fclient-management%2Fmdm%2Fbitlocker-csp&data=04|01|Christopher.Black%40microsoft.com|22670e5c66e64be982c008d89e0d2fff|72f988bf86f141af91ab2d7cd011db47|0|0|637433126836849627|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=atDKtnoBB5IeF25ws5xWnyL5U%2BZHVOY2rdI6PaPniGc%3D&reserved=0) are closed already, so I presume this PR is not based on any known existing ticket.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fpull%2F8775%23issuecomment-741839444&data=04|01|Christopher.Black%40microsoft.com|22670e5c66e64be982c008d89e0d2fff|72f988bf86f141af91ab2d7cd011db47|0|0|637433126836859621|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=3Pg1R9tJKKPedT%2B3daMoLGMxKV1jARxIOXb%2FqBooUKM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACCSFYHSNVV5554DD23OPTLST6IR3ANCNFSM4UQ5Q6VA&data=04|01|Christopher.Black%40microsoft.com|22670e5c66e64be982c008d89e0d2fff|72f988bf86f141af91ab2d7cd011db47|0|0|637433126836859621|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XKCh9MEZxbU5WVs8U4c6OtB0UBOHG%2F1MpguDhaYZTU0%3D&reserved=0.

No worries. This PR will be reviewed, approved, and merged, either by the current document author or manager, depending on MS Docs team decisions.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Python Type Hints

Hi,
Now I'm using Synthizer in my own projects, it would be extremely useful to have Python type hinting.
Is this something you're interested in?
I'm more than happy to port the existing code myself, or even be responsible for type hints in the code base, if that would be helpful.
I'm going to have a poke around in the build/python directory to see what's going on there. If you're interested in this feature and / or help with it, could you let me know if there's anywhere else I should be looking?
Not really sure what other questions to ask at this stage.

I'm not adverse to this, but it changes frequently enough that I'd probably need to be responsible for maintaining them, I've not done it before, and I don't know how well it works with Cython (though I think they have annotation support now).  If you or someone else gets it working and wants to send a PR I'll merge and keep it up to date, but building Synthizer from source is required to build the bindings.
It's a good idea, but we have bigger fish to fry like not being able to loop streams, so unless someone else does the work it won't happen for a while.

OK,
As I said, I'm happy to be responsible for it, if you just let me know when
it's stable enough that it won't change on an hourly basis.
I've only just started using type hints in Python myself, and it's easier
than I gave it credit for. Especially with Synthizer, where many of the
functions are primitive types anyways (int, float, bool).
Basic idea is:
def f(a: int, b: int) -> int:
"""Add two numbers together."""
return a + b
Take care,
Chris Norman
On Sat, 25 Jul 2020 at 18:53, Austin Hicks<EMAIL_ADDRESS>wrote:

I'm not adverse to this, but it changes frequently enough that I'd
probably need to be responsible for maintaining them, I've not done it
before, and I don't know how well it works with Cython (though I think they
have annotation support now). If you or someone else gets it working and
wants to send a PR I'll merge and keep it up to date, but building
Synthizer from source is required to build the bindings.
It's a good idea, but we have bigger fish to fry like not being able to
loop streams, so unless someone else does the work it won't happen for a
while.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/synthizer/synthizer/issues/9#issuecomment-663882876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUZ2MYSD4O56ATSG4EOPDR5MLZHANCNFSM4PGC5PWQ
.

It's stable enough that if you give me an initial version I can keep it updated, which is what I was trying and failing to say.
I'm familiar with the Python-side annotations but I don't know how this interacts with Cython and I'm not set up to easily find out if they're working, either.

OK, do I just put changes in the bindings/python/synthizer/synthizer.pyx
file? or are there other places?
Also, are there build instructions, so I can test the version I build?
Take care,
Chris Norman
On Sun, 26 Jul 2020 at 00:38, Austin Hicks<EMAIL_ADDRESS>wrote:

It's stable enough that if you give me an initial version I can keep it
updated, which is what I was trying and failing to say.
I'm familiar with the Python-side annotations but I don't know how this
interacts with Cython and I'm not set up to easily find out if they're
working, either.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/synthizer/synthizer/issues/9#issuecomment-663917095,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUZ2M2R47HHPQYY6TJVBDR5NUINANCNFSM4PGC5PWQ
.

Yeah. The annotations would go in synthizer.pyx.  Build instructions are on the to-do list but it's the standard cmake out-of-tree process.  Get VS 2019 with all the C++ tools, get Clang from releases.llvm.org, then:
mkdir build
cd build
cmake -G Ninja -DCMAKE_C_COMPILER="c:\program files\llvm\bin\clang-cl.exe" -DCMAKE_CXX_COMPILER="c:\program files\llvm\bin\clang-cl.exe" ..
ninja

Only need ninja after the first time, but since you're doing Python you'll be good and don't need to rebuild this. Then, for Python, using powershell:
cd bindings\python
$Env:BUILDING_SYNTHIZER=1
python setup.py develop

That's all typed from memory, so there might be errors or omissions.  Also if you don't have the tools already it'll take quite a while to set up unless you've got good internet.

@chrisnorman7
I'm going to look at this as part of 1.0. Is there a convenient way to test to make sure they're working?  Which VSCode extensions do you use?  Any help is appreciated.

@camlorn I'm using Pylance, which pretty well does it all. Just make sure you've got Mypy installed.
When you start typing member names, autocomplete should come up and show you what the type of a value should be.
Let me know if there's anything else you need to know, and sorry I never got around to looking at the type hinting myself. I did look into cython briefly, but when I failed to build Synthizer I sort of forgot about it.

Unfortunately, I just don't have the bandwidth and there's too many other fish to fry.  I need someone to start by creating typing information for the current bindings, whether that be through figuring out if/how Cython can annotate or if we can maintain a separate index, and to get them relatively in sync with what exists. If someone gets it that far I'll revisit and put this in a milestone, though possibly not 1.0.

That's fine. I'll look into making stubs at some point.
Cheers,
Take care,
Chris Norman
On Sat, 6 Feb 2021 at 20:44, Austin Hicks<EMAIL_ADDRESS>wrote:

Unfortunately, I just don't have the bandwidth and there's too many other
fish to fry. I need someone to start by creating typing information for the
current bindings, whether that be through figuring out if/how Cython can
annotate or if we can maintain a separate index, and to get them relatively
in sync with what exists. If someone gets it that far I'll revisit and put
this in a milestone, though possibly not 1.0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/synthizer/synthizer/issues/9#issuecomment-774540858,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUZ2OJMQXOCFRLZFNIIALS5WS4FANCNFSM4PGC5PWQ
.

I'm currently experimenting with Automatic stub generation (stubgen), to generate type stubs for Synthizer, for use in Earwax.
If I'm successful, do you want a PR with the stubs I've made?
The original stub file was created with stubgen -m synthizer, and I'm just modifying the types by hand.

I would want a PR if you can get it actually working with Synthizer itself on install.  Otherwise, I don't much see the value.

OK, so I've made a stub file, and it works fine. I'm not sure where to place it in the Synthizer source tree though.
It works from the route directory of the Earwax source tree, so MyPy is recognising it, and it giving me the expected things.
I was going to put it into site-packages inside the synthizer folder, but there isn't one.
Where would I put it for a PR to ensure it's shipped with the Synthizer module?

I don't know.  I think maybe you modify Manifest.in.  You'll have to build the package, install it, and check if files go where they're supposed to.
Advanced Python packaging is where good devs go to die of frustration. It's not even fully documented in one place.

OK, I can do that. Are there build instructions somewhere? I've managed to
build with cmake. Do I then just read the ps1 files in the binding/python
directory to see what happens next?
Take care,
Chris Norman
On Mon, 15 Feb 2021 at 17:26, Austin Hicks<EMAIL_ADDRESS>wrote:

I don't know. I think maybe you modify Manifest.in. You'll have to build
the package, install it, and check if files go where they're supposed to.
Advanced Python packaging is where good devs go to die of frustration.
It's not even fully documented in one place.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/synthizer/synthizer/issues/9#issuecomment-779361841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUZ2LDEDYOWGGLIKMG5XTS7FKL7ANCNFSM4PGC5PWQ
.

The readme tells you how to build Python too if you read the section on Python.  Do you need more clarification than than what I've got there?
Also left you a review on your PR.


Update Q-and-A.md valence answer with definitions of terms from the whitepaper

It seemed appropriate to add the definitions of the various valence terms from the keri whitepaper to clarify what the answer and diagram were attempting to convey.

The Q-and-A.md in the KERI repo is outdated. The new one is here: https://github.com/WebOfTrust/WOT-terms/blob/main/docs/07_education/q-and-a.md Thanks for your PR, I integrated it by hand in the right repo.
Feel free to join the discussion here: https://github.com/WebOfTrust/WOT-terms/discussions/30

@henkvancann roger, I'll close this one then.


merge dev-2022

issues fixed:

[X] compatibility issues with nbconvert v6.x
[X] jinja template searching path (mathjax URL)
[X] deprecated function calls
[X] inconsistent behaviors of Selenium send_keys() when dealing with datetime-local field among various system settings

Great work!  I resolved a conflict to see about the tests.
But I wouldn't merge this as it is: we shouldn't add any commits to our 'master' branch (it should directly follow the upstream, nothing new in our 'master'.  We shouldn't really use it for anything, for that matter).  And this series of commits has some Aalto-specific version number increases.
I can take and clean this up onto a single branch which is ready to merge upstream.  Maybe it would be good for me to do this and others to watch, since all this cherry picking can be complex and hard to start with?

This is done, it seems!  Different PR, but by working together someone was able to do it.  I'll close this to keep things clean.
Thanks for your work on this!


RequireHttpsAttribute doesn't change the port

RequireHttpsAttribute changes the scheme but not the port, so http://localhost:44380/ gets redirected to https://localhost:44380/, which is invalid because you can't serve both http and https on the same port.
The only way this ever worked was if the original request's host header did not include the port (e.g. localhost rather than localhost:80)
There's even a (wrong) test:
https://github.com/aspnet/Mvc/commit/25838cee55216bd02432390996b95c5480b23e94#diff-ba666c0ff94ddf047b6f11caa0ac94c9R42
@sebastienros @blowdart @javiercn

Kind of related to https://github.com/aspnet/Mvc/issues/4093
In the default case we should indeed drop the port number when redirecting to HTTPS, and fix the offending test.

3-Done?


Delegate pushing the file to S3

Motivation
Some files may be large and can not be uploaded directly to S3 through the upload endpoint and has to be delegated to the client for performance and reliability reasons.
One example are jobs in the parallel cluster which will rely on S3-DRA to push the file to S3
Acceptance criteria

The user can provide optional metadata
If the user has no write access to this storage (permission/deprecated project) if the metadata/id is invalid, an error is returned
If the provided input is valid, a temporary he necessary information to push the file (bucket / object key / id) is returned to the user
This payload is signed to make sure it is not modified by the client
Once the file is pushed to the bucket, the client calls Delta again the signed payload, if the payload can be verified and has not reached an expiration date, the file is created in Nexus
The default storage must be a S3 storage

Implementation notes
https://connect2id.com/products/nimbus-jose-jwt/examples/jws-json-multiple-signatures
Not in scope

The client having write access to the location in the given bucket
Using presigned urls for other cases
Providing a storage id which is not the default one
Providing a file id
Updating the file

@imsdu "If the provided input is valid" what is the provided input? Is it only the metadata? What about filename, media type, etc? Can they also provide a tag?
I'm also not clear on where the client includes the Id for the second call?

@dantb This operation is meant to replace the legacy link operation so it would be the metadata + the content type.
And other controls must be performed related to access, project / storage deprecation, ...
Don't hesitate to do 2 PRs if you feel like it helps, one to generate the signed payload, the second to submit it again


Arrow z-index

Fix z-index of popover arrow.
Reproduce problem:
Add box-shadow to popover inner and arrow goes under shadow if you don't add z-index for it.
Fix:
Place arrow after inner block -> you dont need use z-index.

Thanks! :+1:


Alignment meeting with teams before the release

Task: Coordinate a meeting with the different teams to show the last updates before the release and thank everyone for their participation.

Meeting will be on Monday 17, July.


My erizo library is behind NAT, getCandidate() always fails to collect srflx and the full host address

Hi， My erizo library is behind NAT, and when I use libnice+stun server, getCandidate() always fails to collect srflx and the full host address! May I ask how I should modify the code?
thanks

Now I can get the srflx address, but I can only get one local host address, not the other host addresses.


fix(workflows): use unions instead of OR

Changes

#1170  👈
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


Running paket install fails?

Description
Clone my repository https://github.com/halcwb/GenForm.git. Run mono .paket/paket.exe install And you get:

Error: SecureChannelFailure (The authentication or decryption has failed.) (Github - cached (temporarily ignore updates))

Repro steps
Please provide the steps required to reproduce the problem

See above

If possible then please create a git repository with a repro sample or attach a zip to the issue.
Install of packages
Please provide a description of the behavior you expect.
See above
Please provide a description of the actual behavior you observe.
Problem need to commit paket genererated files as well! But why?
Please provide a description of any known workarounds.

Have you tried updating your bootstrapper since

https://github.com/fsprojects/Paket/pull/3066
https://github.com/fsprojects/Paket/issues/3068

?

also mono updated?

@forki  Indeed it seemed that particular machine had an outdated mono version. I already had tried using the lates bootstrapper in 'magic mode'. But it was the mono framework.
Thanks!


[Esp32] mdns filter to handle MDNS traffic for esp32

This PR adds a filter to handle MDNS packets in queue and if queue gets full it drops the packets
The filter will also drop all IPV4 mdns packets
It helps to save heap during commissioning in crowded MDNS environment

Testing

Tested commissioning and control 30-40 times and verified heap changes

@PSONALl Please ping me on Slack when you make the updates per above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.PSONALl seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

CI not followed up with in 1 year. Closing as stale.


docs: make the required backticks in email password more explicit

updated the example config to make the needed backticks around the password more obvious

🤖 🎺


Saving non-arrayifiable data at each likelihood evaluation (either to disk or in memory)

General information:

emcee version: 3.1.4
platform: OS X, Linux
installation method (pip/conda/source/other?): conda or pip

Problem description:
We have an epidemic modeling pipeline (flepimop.org) with a custom MCMC inference engine, from our trials it seems that EMCEE outperforms it. So we are really excited to include it in our package.
However, our pipeline saves some outputs (in 6 parquet files) that are needed for us at each evaluation. The filenames follow a convention. For example, the spar metadata for the 1000 iteration of worker 17 is stored model_output/rsv_CA_SIR_v6_none_IHRadj/CA_v6/spar/global/intermediate/000000017.000001000.CA_v6.spar.parquet.
My question is: Is it possible for workers to know their number, and to know which iteration is currently being done? So that I can write this file (and if not, can use blob to save non-arrayifiable data?)
It is probably a very common problem: as our simulations are expensive, we cannot just rerun them at the end for the values stored in the chain.
I have tried various hacks (such as running sample in a loop that updates the log prob function with a lambda that includes the iteration number) but I got nothing that worked correctly, so I would appreciate any help on that :). It seems like it may not be possible without modifying emcee/ensemble.py ?
Best regards, and thanks for EMCEE, happily using it since 2018 !! and always so performant !

Good question! I'm not sure that there's anything that will work out of the box.
Just so that I'm sure I understand: The idea here is that within your evaluation of the log prob function, you need to be able to pass the iteration number and the walker id to your computational model? Something like:
def log_prob_func(params):
    # Note: these functions don't exist
    walker_id = get_walker_id()
    mcmc_iteration_number = get_mcmc_iteration_number()

    log_prob = expensive_pipeline_that_writes_files(params, walker_id, mcmc_iteration_number)

    return log_prob

right?
To get the walker id number you could set the vectorize=True in your EnsembleSampler, although even in this case it would be a little bit awkward because only half of the walkers are passed for each call and we don't have access to all the appropriate information.
But, if you have some control over how the files are saved, here's an idea for a workaround. Instead of tracking the walker and step number, and handling the bookkeeping yourself, you could instead get a unique id for each step and use that in the filenames. Here's a simple example:
import numpy as np
import emcee

import hashlib
import uuid
import pathlib

def log_prob(x):
    # Hash the input parameters
    id = hashlib.sha256(x.data).hexdigest()

    # OR

    # A random unique id
    id = uuid.uuid4().hex

    # Build a filename based on the unique id
    filename = f"samples/{id[:6]}/{id[6:]}"

    # A placeholder for writing the appropriate file
    pathlib.Path(filename).parent.mkdir(parents=True, exist_ok=True)
    with open(filename, "w") as f:
        f.write(str(x))

    # Return the filename as a "blob"
    return -0.5 * np.sum(x ** 2), filename

ndim, nwalkers = 2, 10
p0 = np.random.randn(nwalkers, ndim)

sampler = emcee.EnsembleSampler(nwalkers, ndim, log_prob)
sampler.run_mcmc(p0, 12)
sampler.get_blobs()

Here we're indexing the filename for each step by either:

Hashing the buffer behind the parameters, or
Generating a random unique id using the uuid module.

Then, we track this unique id using blobs. At the end of the run, the blobs array contains the appropriate filename that corresponds to each sample, taking into account whether or not it was accepted.
Hope this helps!

This is a truly great idea and thanks for providing an example implementation. It works like a charm (I came up with a much dirtier solution, based on running in a loop with different function arguments at each iteration, but it misses the worker identification which is important to me). We are rolling emcee on flepiMoP! Let me know if you'd like me to add your answer to the emcee's documentation.
Thank you so much. I have a small clarification question that I did not get from the doc. I believe the blobs are updated on "accept" (not for every iteration). e.g

If iteration 4 proposes parameter p_4, and it is accepted, then my blob/files are the one obtained by running with p_4. All good.
if p_5 is proposed but gets rejected, does my blob correspond to the run from p_5 or p_4?

I believe it is the blob corresponding with p_4 (from reading the code, I'm not on my laptop to test right now), which is great for my use case. This is correct?

if p_5 is proposed but gets rejected, does my blob correspond to the run from p_5 or p_4?

You're right, the blob will correspond to the run for p_4 (the previously accepted sample) rather than p_5 (the rejected sample). In other words, the blobs array will generally have duplicates in it. I think this is typically what you would want, and it might be a bit tricky to get the other behavior.

Great, that's indeed my expected behaviour. Thank you Dan for everything! Implementing this neat solution :)


chore(docs):  Added gemini, together, open router and local models pages support docs via our OpenAI sdk compatibility

Given our native support of OpenAI SDK and many companies supporting that format by default, as per slack discussion, added a few pages that outline how Weave can be utilized with those companies even before we have a native integration with their pytho clients.
Also added a local models category, with links to the most popular local model runners with support for OpenAI SDK

closed in favor of #1910


fix(Mixed Chart Filter Control): Allow delete condition for adhoc_filters_b

SUMMARY

This is meant to fix a bug that would prevent the deletion of filters from Query B in the control panel of a mixed chart.
The condition to run the canDelete function for a filter previously only allowed for adhoc_filters as a valid value but the mixed chart uses adhoc_filters_b for the Query B group. This change should allow any string starting with adhoc_filters to be checked for deletion conditions.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

B:

A:

TESTING INSTRUCTIONS

Create a Mixed Chart
Add a filter to Query B
Delete filter
Delete last remaining temporal filter to test warning if is displayed

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rtexelm can you add a small test for this, too? Thanks!


Bug: listbox::unsort causes potential mismatch between clicked item and selected item

When the listbox items are sorted by a column, and then the listbox::unsort method is called, clicking an item selects the item that is at the absolute position associated with the display position that was clicked. For example, if you click the item at display position 2, the item that is at absolute position 2 is selected. In other words, the item that gets selected is the item that was at display position 2 before the items were sorted.
To see what I'm talking about, run the following test code, click the "unsort" button, and then click any item other than the first:
#include <nana/gui.hpp>
#include <nana/gui/widgets/button.hpp>
#include <nana/gui/widgets/listbox.hpp>

int main()
{
	nana::form fm{nana::API::make_center(234, 234)};
	fm.div("vert margin=15 <listbox><weight=15><button weight=30>");

	nana::listbox lb{fm};
	fm["listbox"] << lb;
	lb.enable_single(true, false);
	lb.append_header("Col 0");
	lb.at(0).append("first absolute");
	lb.at(0).append("second absolute");
	lb.at(0).append("third absolute");
	lb.at(0).append("fourth absolute");
	lb.column_at(0).fit_content();
	lb.sort_col(0);

	nana::button btn{fm,"Unsort"};
	fm["button"] << btn;
	btn.events().click([&]
	{
		lb.unsort();
	});

	fm.collocate();
	fm.show();
	nana::exec();
}

I found this to be caused by the function nana::drawerbase::listbox::es_lister::at, which takes a display position as an argument, and only translates it to an absolute position if there is a sort column:
category_t::container::value_type& at(const index_pair& pos)
{
	auto acc_pos = pos.item;
	if (npos != sort_attrs_.column)
		acc_pos = index_cast(pos, true).item;	//convert display position to absolute position

	return get(pos.cat)->items.at(acc_pos);
}

The conditional conversion to an absolute position doesn't make sense, because since listbox::unsort exists, having no sort column doesn't necessarily mean that each display position coincides with the corresponding absolute position.
As a consideration for the future, maybe unsort should actually unsort the items, instead of only cancelling automatic sorting. sort_col(npos) could still be used to only cancel automatic sorting.

Merged!


Fix WebSocket notifications

When any errors occur, the websocket thread is automatically terminated. And the websocket notification process stops.

It is necessary to log errors, but the websocket thread should be continue
Remove flag "fRunning = false" in src/eventloop.h in section catch in method "void Start"

Fixed in commit cd559aa10b0ab89479be9d37593e87a560e927d2


Any solution to upload the scritp (host cpanel)?

When trying to upload the script to hosting, cpanel detects it as a virus, by ftp it is uploaded and after a few seconds it is removed by the cpanel antivirus.
Attached screenshot

Your Cpanel does not support to use Vinaget. You should change other hosting provider.


Compiler crashes: Unexpected null character in regular expression

https://github.com/xp10rd/coolc/blob/main/coolc/src/lexer/Lexer.cpp#L5
I tried llvm11 and llvm 15. The same crash
@xp10rd

@Kakadu Can you please provide OS version, build and run commands?

@Kakadu Can you please provide OS version, build and run commands?

https://gist.github.com/Kakadu/8c1d62b08adfe61dd1dd492bd6fddb76

@Kakadu Sorry, at this moment I don't have a x86 machine with Linux. I checked this issue in Docker with Ubuntu 22.04 where default llvm and clang/clang++ versions are 14.0.0 and all seems ok. You can use Dockerinfo to build and run project. From the coolc dir run:
docker build -t ubuntu-22-04-env .
docker run --platform linux/amd64 --rm -it -v $(pwd):/home/xp10rd/coolc ubuntu-22-04-env bash

It compiles in docker, but I still don't understand what goes wrong on newest Ubuntu's. The only thing that I discovered that fixing the same version of clang++-14 is not enough.


"Gorgota" is listed in the drop-down menu.

Description:
"Gorgota" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Gorgota" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Gorgota, PRAHOVA".


Copying a Renderable without Collision Shape fails

Hello,
To avoid picking a renderable, it's possible to set the collision shape to null (See the setter in Renderable.class).
Calling the function makeCopy() on such a renderable fails because the property collisionShape is copied without checking the value (See the copy constructor)
Thanks for ur help,
Best regards.

Thanks for the report.  This issue is fixed internally and will be available in an upcoming release.


How to install  MetaMorpheus on Linux?

The command-line version is supported on Linux. However, I cannot find an Installation Manual. Can anyone help me？
Thanks！

Welcome to MetaMorpheus!
I've just added a new Wiki page describing how to use MetaMorpheus via command-line: https://github.com/smith-chem-wisc/MetaMorpheus/wiki/Using-the-Command-Line
Note that you'll need to install .NET Core version 2.1. I found it a bit difficult to get that particular version installed, as the current version is 3.0 or 3.1. MetaMorpheus will soon be updated to that version of .NET Core. Feel free to post here for more help or email us at mm_support at chem dot wisc dot edu :)

I should probably mention - to download MetaMorpheus for Linux, go to the latest release and download the "MetaMorpheusDotNetCoreAppveyor.zip" package. This contains the .NET Core command-line version of MetaMorpheus.


how to handle KeyboardInterrupt at 2.0

Consider this
        prompt = Prompt()

        try:
            text = prompt.prompt("> ", enable_suspend=True)
        except:
            pass

        text = prompt.prompt("> ")
        print(text)

At the first prompt, I sent a ^C. And at the second prompt, I typed foo. However, at the end of the application, it shows Future exception was never retrieved. It may be because the exception was never probably handled. How I should handle it correctly?
>
> foo
foo
Future exception was never retrieved
future: <prompt_toolkit.eventloop.future.Future object at 0x110e3c6a0>
Traceback (most recent call last):
  File "/Users/Randy/Dropbox/R/Role/role/application.py", line 15, in run
    text = prompt.prompt("> ", enable_suspend=True)
  File "/Users/Randy/Dropbox/R/Role/role/deps/prompt_toolkit/shortcuts/prompt.py", line 657, in prompt
    return run_sync()
  File "/Users/Randy/Dropbox/R/Role/role/deps/prompt_toolkit/shortcuts/prompt.py", line 641, in run_sync
    return self.app.run()

May need sth like prompt.reset!?

This looks like a bug. It shouldn't be like that. Thanks for reporting.

Should be fixed in: https://github.com/jonathanslenders/python-prompt-toolkit/commit/b57f354933ede8027ec5f1f019f4f9571c7b5514

confirm fixed.


No code in this repo

Was there not supposed to some Python sample code in this repo?
The tutorial seems to indicate so.

+1

+1

+1

Sorry for the delay on this sample. We will be updating the article in the coming days as well. This article was shipped slightly ahead of planned.


blackbox - apiserver health check fails

Describe the bug
The blackbox check https://kube-apiserver.example.com:/healthz fails because it returns 200 and not 401.
To Reproduce
Steps to reproduce the behaviour:

Go to any cluster Promethues alerts and check the EndpointDown alert

Expected behaviour
This should not fail if the endpoint is up and running.
Version (add all relevant versions):

v0.17.0

In my own environment this happens were an OK status is returned. For some reason the basic auth seems to be out of play.
In other environments v0.18.2 it works as expected and returns unatorized.

I had had just messed up DNS.
@crssnd could it be the same for you?


RealSolarSystem conflicts with PlanetPack

All the other Planet pack mods (Outer Planets, New Horizons,
Forgotten Worlds, etc) conflict/provide PlanetPack. RSS
should do the same (unless it really can work alongside the
other mods).

I'm working on this on KSP-CKAN/CKAN-meta/issues/588

Per https://github.com/KSP-CKAN/CKAN-meta/issues/588 we're trying to phase out provides and conflicts for PlanetPack and instead conflict with the appropriate planet packs themselves. In case issues arise in the future or we create a metapackage for all planets, having a comment of PlanetPack will assist in tracking them down.

Closing this as planetpacks are being addressed by @Olympic1 in https://github.com/KSP-CKAN/NetKAN/issues/2292 and KSP-CKAN/CKAN-meta#588


Curated contigs number can be zero ,even less than 1

The final GCI-score result shows me that the curated contigs number can be zero, which will make the GCI-score infinity according to the formula.

Curated contigs number is computed by removing curated gaps on the contig so in few cases, the quality of the assembly is so poor, for example, that the contig might absolutely be "removed" and get the Curated contigs number zero. Other users had reported this issue and I had edited the code to make GCI score zero instead of infinity (which will get an error in python in reality).

I will close this issue due to long time of inactivity.


s/RELATED_IMAGES_/RELATED_IMAGE_/

Use singular so that OLM tooling recognizes image names in the CSV and puts them in the relatedImages list.

/cherry-pick release-2.0

/lgtm


clicking out of project navigation dropdown should close navigation

Steps

Visit an OSF project
Reduce the width of your window until the navigation collapses
Open the project navigation dropdown
Click elsewhere on the screen (not on the dropdown items,  e.g. bottom half of this image)

Behavior

Nothing happens

Expected Behavior

Clicking elsewhere on the screen should close the dropdown

@jlcohoon I don't think this is the expected behavior in general. In a lot of mobile devices the space remaining of the page is very small anyway so it's not a very beneficial feature.
Example of iPhone 5 for instance:

Also Bootstrap is not doing it in their website.

Originally, I also thought the dropdown menu should be closed after user clicked on other area. After I test some companies' website, I found none of them did that. It makes sense for me now. In mobile setting, it is easy to click something by mistake.  It may be annoying for user when they click other places and the dropdown disappears unexpectedly.

If click away isn't the solution, then the menu should push the content down, not cover it up. Note that in that image the project title is blocked.

I tend to agree; an element as important as the menu should not cover in XS, it should displace.

That's a good point, in that case though the navigation can't be fixed so that it can scroll with the page which is not a trivial change but that's what bootstrap is doing so there is good example there.

@jlcohoon @lbanner
So the finalized task I think is this:

In small windows (XS view) make the project menu expand and push the content lower instead of being fixed. (Which means users will need to scroll to the top)

This is what it would look like

@erinmayhood is this something we should do?  Making this change would be minor and it would not change user expectation.
On a side note: You'll see that the comment icon is causing issues there but that placement will be fixed. I'm inclided to also change comment pane to open to full width at that point for small devices.

That's good!

I think this is wonderful!
On Thu, Aug 13, 2015 at 1:38 PM, LarryB<EMAIL_ADDRESS>wrote:

That's good!
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3677#issuecomment-130774102
.


Please use .ruby_version when available

I know there are other closed issues with regards to this subject, but I have a situation where I think it justifies changing this plugin to look first for a .ruby_version file.
In our Capistrano recipes we first stop the sidekiq worker before proceeding with the deploy. Now, consider when we are upgrading Ruby from 2.2.4 to 2.3.0 with rbenv. The capistrano-sidekiq gem will run "bundle exec sidekiqctl stop..." before the deploy starts. By that time, .ruby_version points to 2.2.4 and bundle install was run previously for the current path and it should just work. But since capistrano-rbenv is overriding the Ruby version to 2.3.0 this command will fail because bundle install wasn't executed yet before trying to stop sidekiq and it will yield to an error which I had to fix manually by running bundle in that path first for the newer Ruby.
If the project .ruby_version was used when present it should just work transparently...

This may not work in your case, but have you considered enabling rbenv for non-interactive shells on your server and dropping capistrano-rbenv altogether?
For example, in my deployments where I am using Ubuntu 14.04 and have rbenv installed in $HOME/.rbenv, I just add this to the very top (i.e. preceding the interactive shell check) of my deployer user's .bashrc:
if [ -d $HOME/.rbenv ]; then
  export PATH="$HOME/.rbenv/bin:$PATH"
  eval "$(rbenv init -)"
fi

Then all my Capistrano deployments seamlessly use rbenv, and I get the automatic .ruby_version behavior like you described. And I haven't needed the capistrano-rbenv gem at all.

@mattbrictson indeed, this is something I may consider doing. I measure the performance for setting up rbenv and it only takes about 2ms, which is reasonable enough I guess as a payload for the other commands not relying on rbenv...

@rosenfeld What about this?
set :rbenv_ruby, File.read('.ruby-version')

This only works when Capistrano is part of the application's repository. In our case, we use Capistrano in a separate project, integrated with Chef to handle both server configuration and deploy with a single command. This project handles different applications possibly running on different Ruby versions. That's why this solution doesn't work for us.

Hi, we came up against this issue even with Capistrano being part of the application's repository.
To reproduce:

Create a task which uses bundle, and trigger it after "deploy:starting".
Run a deploy.

Expected behaviour

Sorry for the pre-completion comment, updated the above to make more sense!

Closed via #67

Released in v2.1.0 🚢


Azure App Service Deploy Degrated task

I've gone through all log files and don't see a single error in them .
the deploy start copying files to web app  and somtimes is failing with:
2021-07-07T09:09:04.0100663Z Copying file: 'Infrastructure/Prod/test/Translations/de.json'
2021-07-07T09:09:04.0101255Z Copying file: 'Infrastructure/Prod/test/Translations/en.json'
2021-07-07T09:09:04.0101622Z Omitting next output lines...
2021-07-07T09:09:04.0101972Z An unknown error has occurred. Check the diagnostic log for details.
2021-07-07T09:09:04.0138369Z ##[error]Failed to deploy web package to App Service.
2021-07-07T09:09:04.0160092Z ##[error]Error: Package deployment using ZIP Deploy failed. Refer logs for more details.
2021-07-07T09:09:06.3384542Z Successfully updated deployment History at https://XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX/api/deployments/9301625648944216
2021-07-07T09:09:06.5481355Z App Service Application URL: https://XXXXXXXXXXXXX.azurewebsites.net
2021-07-07T09:09:06.5581253Z ##[section]Finishing: Azure App Service Deploy: XXXXXXXXXXXXXXXXXX

Environment
I use Azure DevOps (dev.azure.com)
i use hosted agent pool

please share the debug logs. One can enable debug logs by setting system.debug variable wit value 'true'.

@chshrikh , when enabling debug log : this logs appears many time :
2021-07-13T15:47:05.3219322Z ##[debug]setting affinity cookie ["ARRAffinity=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;Secure;Domain=XXXXXXXX.test.scm.azurewebsites.net","ARRAffinitySameSite=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;SameSite=None;Secure;Domain=XXXXXXXXXXXXXX.scm.azurewebsites.net"]
2021-07-13T15:47:05.3222529Z ##[debug][GET]https://XXXXXXXXXXXX.scm.azurewebsites.net:443/api/deployments/latest?deployer=VSTS_ZIP_DEPLOY&time=2021-07-13_15-31-50Z
2021-07-13T15:47:08.1744437Z ##[debug]POLL URL RESULT: {"statusCode":202,"statusMessage":"Accepted","headers":{"transfer-encoding":"chunked","content-type":"application/json; charset=utf-8","location":"http://XXXXXXXXXXXX-test.scm.azurewebsites.net:80/api/deployments/latest?deployer=VSTS_ZIP_DEPLOY&time=2021-07-13_15-31-50Z","server":"Kestrel","date":"Tue, 13 Jul 2021 15:47:07 GMT","connection":"close"},"body":{"id":"f757a63e40404f0ba61f9f52d7991bf0","status":0,"status_text":"Building and Deploying 'f757a63e40404f0ba61f9f52d7991bf0'.","author_email":"N/A","author":"N/A","deployer":"VSTS_ZIP_DEPLOY","message":"Created via a push deployment","progress":"Running deployment command...","received_time":"2021-07-13T15:32:00.0279854Z","start_time":"2021-07-13T15:32:01.4480598Z","end_time":null,"last_success_end_time":null,"complete":false,"active":false,"is_temp":false,"is_readonly":true,"url":null,"log_url":null,"site_name":"XXXXXXXXXXXXXXX"}}
2021-07-13T15:47:08.1748414Z ##[debug]Deployment status: 0 'Building and Deploying 'f757a63e40404f0ba61f9f52d7991bf0'.'. retry after 5 seconds

after that , il's seems that he start  coping file , but , he failed
2021-07-13T15:56:58.1459713Z ##[debug]getDeploymentLogs. Data: {"statusCode":200,"statusMessage":"OK","headers":{"transfer-encoding":"chunked","content-type":"application/json; charset=utf-8","server":"Kestrel","date":"Tue, 13 Jul 2021 15:56:57 GMT","connection":"close"},"body":[{"log_time":"2021-07-13T15:32:00.2404084Z","id":"905e676f-0d1f-456e-a36a-344e3f2e484f","message":"Updating submodules.","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:01.3822298Z","id":"c926c2de-6bbb-4131-83b9-80b2e4c4c824","message":"Preparing deployment for commit id 'f757a63e40'.","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:01.7318831Z","id":"f6940954-0395-47d8-984f-4be57eb48260","message":"Generating deployment script.","type":0,"details_url":"https://XXXXXXXXXXXX-test.scm.azurewebsites.net/api/deployments/f757a63e40404f0ba61f9f52d7991bf0/log/f6940954-0395-47d8-984f-4be57eb48260"},{"log_time":"2021-07-13T15:32:01.9173996Z","id":"e83f53f0-bcd4-4ec7-ad07-ab88986287f9","message":"Running deployment command...","type":0,"details_url":"https://XXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net/api/deployments/f757a63e40404f0ba61f9f52d7991bf0/log/e83f53f0-bcd4-4ec7-ad07-ab88986287f9"},{"log_time":"2021-07-13T15:56:54.0777945Z","id":"d01c9ad7-6cd2-4478-a191-fce26784744f","message":"An unknown error has occurred. Check the diagnostic log for details.","type":2,"details_url":null}]}
2021-07-13T15:56:58.1467328Z Updating submodules.
2021-07-13T15:56:58.1468028Z Preparing deployment for commit id 'f757a63e40'.
2021-07-13T15:56:58.1468388Z Generating deployment script.
2021-07-13T15:56:58.1469825Z ##[debug]setting affinity cookie ["ARRAffinity=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;Secure;Domain=XXXXXXXXXXXXXXX-test.scm.azurewebsites.net","ARRAffinitySameSite=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;SameSite=None;Secure;Domain=XXXXXXXXXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net"]
2021-07-13T15:56:58.1471575Z ##[debug][GET]https://XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net/api/deployments/f757a63e40404f0ba61f9f52d7991bf0/log/f6940954-0395-47d8-984f-4be57eb48260
2021-07-13T15:56:58.4504025Z ##[debug]getDeploymentLogs. Data: {"statusCode":200,"statusMessage":"OK","headers":{"transfer-encoding":"chunked","content-type":"application/json; charset=utf-8","server":"Kestrel","date":"Tue, 13 Jul 2021 15:56:57 GMT","connection":"close"},"body":[{"log_time":"2021-07-13T15:32:01.7894805Z","id":"","message":"Using cached version of deployment script (command: 'azure -y --no-dot-deployment -r \"/tmp/zipdeploy/extracted\" -o \"/home/site/deployments/tools\" --basic --sitePath \"/tmp/zipdeploy/extracted\"').","type":0,"details_url":null}]}
2021-07-13T15:56:58.4506362Z Using cached version of deployment script (command: 'azure -y --no-dot-deployment -r "/tmp/zipdeploy/extracted" -o "/home/site/deployments/tools" --basic --sitePath "/tmp/zipdeploy/extracted"').
2021-07-13T15:56:58.4506983Z Running deployment command...
2021-07-13T15:56:58.4508074Z ##[debug]setting affinity cookie ["ARRAffinity=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;Secure;Domain=XXXXXXXXXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net","ARRAffinitySameSite=c06e9bb74f52245b3695b3079a52f6acbc70c3ee812f67e4fa3f5f65088ff4f7;Path=/;HttpOnly;SameSite=None;Secure;Domain=XXXXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net"]
2021-07-13T15:56:58.4509860Z ##[debug][GET]https://XXXXXXXXXXXXXXXXXXXXXX-test.scm.azurewebsites.net/api/deployments/f757a63e40404f0ba61f9f52d7991bf0/log/e83f53f0-bcd4-4ec7-ad07-ab88986287f9
2021-07-13T15:56:58.7262364Z ##[debug]getDeploymentLogs. Data: {"statusCode":200,"statusMessage":"OK","headers":{"transfer-encoding":"chunked","content-type":"application/json; charset=utf-8","server":"Kestrel","date":"Tue, 13 Jul 2021 15:56:57 GMT","connection":"close"},"body":[{"log_time":"2021-07-13T15:32:01.9305874Z","id":"","message":"Command: \"/home/site/deployments/tools/deploy.sh\"","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.0934687Z","id":"","message":"Handling Basic Web Site deployment.","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.1612206Z","id":"","message":"Kudu sync from: '/tmp/zipdeploy/extracted' to: '/home/site/wwwroot'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.2253098Z","id":"","message":"Copying file: '.editorconfig'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.2553Z","id":"","message":"Copying file: '.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.2842859Z","id":"","message":"Copying file: '.env.example'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.3233467Z","id":"","message":"Copying file: '.gitattributes'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.3552332Z","id":"","message":"Copying file: '.gitignore'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.3788146Z","id":"","message":"Copying file: '.htaccess'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.419646Z","id":"","message":"Copying file: '.rnd'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.4570761Z","id":"","message":"Copying file: '.styleci.yml'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.4806388Z","id":"","message":"Copying file: 'README.md'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.5105596Z","id":"","message":"Copying file: 'artisan'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.5299435Z","id":"","message":"Copying file: 'azure-pipeline-ci.yml'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.5517645Z","id":"","message":"Copying file: 'azure-pipeline.yml'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.5797277Z","id":"","message":"Copying file: 'body.xml'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.6096593Z","id":"","message":"Copying file: 'composer.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.6339983Z","id":"","message":"Copying file: 'composer.lock'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.6615555Z","id":"","message":"Copying file: 'php'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.6828092Z","id":"","message":"Copying file: 'phpunit.xml'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.7284729Z","id":"","message":"Copying file: 'server.php'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.7638779Z","id":"","message":"Copying file: 'web.config'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.7891125Z","id":"","message":"Copying file: 'webpack.mix.js'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.8153747Z","id":"","message":"Copying file: 'Infrastructure/.htaccess'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.8472043Z","id":"","message":"Copying file: 'Infrastructure/.htaccess_public'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.8743119Z","id":"","message":"Copying file: 'Infrastructure/composer.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.8986604Z","id":"","message":"Copying file: 'Infrastructure/database.php'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.9239322Z","id":"","message":"Copying file: 'Infrastructure/web.config'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.951395Z","id":"","message":"Copying file: 'Infrastructure/Dev/prod/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:02.9865508Z","id":"","message":"Copying file: 'Infrastructure/Dev/prod/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.0234788Z","id":"","message":"Copying file: 'Infrastructure/Dev/prod/Translations/fr.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.0553384Z","id":"","message":"Copying file: 'Infrastructure/Dev/prod/Translations/pl.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.092061Z","id":"","message":"Copying file: 'Infrastructure/Dev/staging/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.1215056Z","id":"","message":"Copying file: 'Infrastructure/Dev/staging/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.1543451Z","id":"","message":"Copying file: 'Infrastructure/Dev/staging/Translations/fr.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.1858096Z","id":"","message":"Copying file: 'Infrastructure/Dev/staging/Translations/pl.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.2147781Z","id":"","message":"Copying file: 'Infrastructure/Dev/test/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.2377502Z","id":"","message":"Copying file: 'Infrastructure/Dev/test/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.2622026Z","id":"","message":"Copying file: 'Infrastructure/Dev/test/Translations/fr.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.2987033Z","id":"","message":"Copying file: 'Infrastructure/Dev/test/Translations/pl.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.3199744Z","id":"","message":"Copying file: 'Infrastructure/Prod/prod/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.3510841Z","id":"","message":"Copying file: 'Infrastructure/Prod/prod/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.3722155Z","id":"","message":"Copying file: 'Infrastructure/Prod/prod/Translations/fr.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.3977295Z","id":"","message":"Copying file: 'Infrastructure/Prod/prod/Translations/pl.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.4304565Z","id":"","message":"Copying file: 'Infrastructure/Prod/staging/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.4555785Z","id":"","message":"Copying file: 'Infrastructure/Prod/staging/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.5057538Z","id":"","message":"Copying file: 'Infrastructure/Prod/staging/Translations/fr.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.538899Z","id":"","message":"Copying file: 'Infrastructure/Prod/staging/Translations/pl.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.5597901Z","id":"","message":"Copying file: 'Infrastructure/Prod/test/.env'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.5922133Z","id":"","message":"Copying file: 'Infrastructure/Prod/test/Translations/be.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.6186015Z","id":"","message":"Copying file: 'Infrastructure/Prod/test/Translations/de.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.6614758Z","id":"","message":"Copying file: 'Infrastructure/Prod/test/Translations/en.json'","type":0,"details_url":null},{"log_time":"2021-07-13T15:32:03.6859619Z","id":"","message":"Omitting next output lines...","type":0,"details_url":null}]}
2021-07-13T15:56:58.7282064Z Command: "/home/site/deployments/tools/deploy.sh"
2021-07-13T15:56:58.7282466Z Handling Basic Web Site deployment.
2021-07-13T15:56:58.7283224Z Kudu sync from: '/tmp/zipdeploy/extracted' to: '/home/site/wwwroot'
2021-07-13T15:56:58.7318931Z Copying file: 'Infrastructure/Prod/test/Translations/be.json'
2021-07-13T15:56:58.7319571Z Copying file: 'Infrastructure/Prod/test/Translations/de.json'
2021-07-13T15:56:58.7320185Z Copying file: 'Infrastructure/Prod/test/Translations/en.json'
2021-07-13T15:56:58.7320583Z Omitting next output lines...
2021-07-13T15:56:58.7320986Z An unknown error has occurred. Check the diagnostic log for details.
2021-07-13T15:56:58.7358182Z ##[error]Failed to deploy web package to App Service.
2021-07-13T15:56:58.7370374Z ##[debug]Processed: ##vso[task.issue type=error;]Failed to deploy web package to App Service.
2021-07-13T15:56:58.7371241Z ##[debug]task result: Failed
2021-07-13T15:56:58.7372683Z ##[error]Error: Package deployment using ZIP Deploy failed. Refer logs for more details.

@chtourou-youssef the logs starting with "2021-07-13T15:56:58.1459713Z ##[debug]getDeploymentLogs. Data:" has the KUDU log url, can you check on KUDU what error happened


怎么删除干净

怎么删除干净，包括用户npm包之类
因为我一开始做个测试
但是在正式使用后
npm uninstall verdaccio
数据还都在

Please post issues in English language. Thanks 👍

How to delete it, including user NPM package and so on
Because I started with a test
But after the official use
npm uninstall verdaccio

The data is still there

Indeed, the data will still be there if you just uninstall the npm package.
All data is stored in ~/.config/verdaccio. It's prefectly fine to delete that folder if you don't need that data anymore.

thanks :smile:


Re-add port-based rules and gate them behind kernel version

Fixing firezone/product#435
Still a draft since I have to test this in an older kernel and decide how to handle tests in older kernels.

Pull Request Test Coverage Report for Build 4bf748d1ca636dd86eb1941e2f69cd81d2426c36-PR-890

39 of 158 (24.68%)  changed or added relevant lines in 11 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-2.2%) to 67.199%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

apps/fz_http/lib/fz_http/rules.ex
11
12
91.67%

apps/fz_wall/lib/fz_wall/cli/sandbox.ex
0
1
0.0%

apps/fz_wall/lib/fz_wall/server.ex
0
1
0.0%

apps/fz_http/lib/fz_http_web/live/rule_live/rule_list_component.ex
11
16
68.75%

apps/fz_wall/lib/fz_wall/cli/helpers/has_feature.ex
0
6
0.0%

apps/fz_http/lib/fz_http/int4range.ex
13
28
46.43%

apps/fz_wall/lib/fz_wall/cli/helpers/sets.ex
0
16
0.0%

apps/fz_wall/lib/fz_wall/cli/helpers/nft.ex
0
34
0.0%

apps/fz_wall/lib/fz_wall/cli/live.ex
0
40
0.0%

Files with Coverage Reduction
New Missed Lines
%

apps/fz_wall/lib/fz_wall/cli.ex
1
0%

apps/fz_wall/lib/fz_wall/cli/live.ex
1
0%

Totals

Change from base Build 58e48457ad1cd75bd39e7e8d3cfe730f6dc931d7:
-2.2%

Covered Lines:
1051

Relevant Lines:
1564

💛  - Coveralls

Fixing firezone/product#435
Still a draft since I have to test this in an older kernel and decide how to handle tests in older kernels.

Was thinking we would just stub the kernel version returned from the uname call?
I think we may just be able to get away with letting the CI build stage test the whole enchilada on ubuntu 18.04

Was thinking we would just stub the kernel version returned from the uname call for the unit tests?

Sounds good

I think we may just be able to get away with letting the CI build stage test the whole enchilada on ubuntu 18.04

So maybe add a couple of rules-related functional tests?

Sorry, CI still failing github UI was outdated. Will fix it and re-open.


Implement session revocation

Signout function here only deletes the session locally on the server, but does not actually revoke the authentication with the provider: https://github.com/denoland/deno_kv_oauth/blob/main/lib/sign_out.ts
This causes browser to remember earlier credentials even after signout, so that when any user tries to sign in again using the same computer, the earlier authentication is just renewed without any prompt.
You can work around this issue, at least when using Auth0 provider, with the prompt parameter:  const response = signIn(request, oauthConfig, { urlParams: {prompt: "login"}});
This forces the provider to present the login screen again every time signin is called, even if there is a previous session left dangling. This is not secure however, as really the session should be revoked with the provider upon signout..

I recall that token revocation still needs to be supported in the underlying OAuth module, which is a blocker. I raised an issue requesting the functionality to be added. There are two solutions:

Add support for token revocation in the OAuth module.
Switch the OAuth module to something that supports token revocation, like https://github.com/panva/oauth4webapi. This might be the best path forward for token revocation and other features like OpenID Connect.

We might discuss this and come back to you. We may have to live without token revocation for the time being.

Thank you. Good to know about oauth4webapi, that seems like it might be a nice lower level alternative to fall back to if it comes to that.
For anyone struggling with the same issue, here's how to actually sign out (when using Auth0 provider, but other providers probably work in similar manner), while still working with deno_kv_auth:
// these are probably already defined where you have set up your oauthConfig object for deno_kv_auth, included here for clarity
const domain = "MY_AUTH0_DOMAIN";
const returnTo = encodeURIComponent("URL_TO_RETURN_AFTER_SUCCESS");
const clientId = encodeURIComponent("MY_OAUTHCONFIG_CLIENT_ID");

const url = `https://${this.authconfig.domain}/v2/logout?client_id=${clientId}&returnTo=${returnTo}`;

// here we get the would-be-signout-response from deno_kv_auth, which would delete the cookie from user and redirect to the success url....
const response = signOut(request, this.authconfig.oauth);

// ...but we don't return it as is, but instead change the success url redirect to point to the Auth0 logout endpoint instead, which then redirects user back to our actual success url
response.headers.set("Location", url);

// ok
return response;

In addition, you may want to use the forced login prompt mentioned earlier upon signIn, however this may hide any issues you may have with the signOut procedure, so use with caution:
const response = signIn(request, oauthConfig, { urlParams: {prompt: "login"}});

Hi @jtoppine @iuioiua,
I would like to work on this task If you are aligned on the approach.
Would be glad if you give me some more context on Slack.


Implicit generics inference with nested types

Bug Report
🔎 Search Terms
Generics, Implicit Generics, Generics Inference
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about Generics

⏯ Playground Link
Playground link with relevant code
💻 Code
type Command = (...args: Array<any>) => any;

type Client<C extends Command> = {
  command: C;
};

type ClientOptions<C extends Command> = {
  command: C;
}

type Setup<C extends Command> = (client: Client<C>) => unknown;

function test1<C extends Command>(
  setup: Setup<C>,
  options: ClientOptions<C>
): void {}

test1(async client => {
  // `client.command` type is `Command`, but should be inferred from `options.command`
  client.command('message');
  //     ^?
}, {
  command(message: string): void {
    console.log(message);
  }
});

function test2<C extends Command>(
  setup: Setup<C>,
  command: C
): void {}

test2(async client => {
  // works as expected, "client.command" type is inferred from `command` (the next argument)
  client.command('message');
  //     ^?
}, (message: string): void => {
  console.log(message);
});

🙁 Actual behavior
test1 - client.command type is Command
test2 - works as expected
🙂 Expected behavior
test1 - client.command type should be (message: string) => void

another demo

Duplicate #38623


TypeError: Cannot read property 'resolve' of undefined

Node-CommonJS Plugin Version: 10.1.0
Rollup Version: 1.27.2
Operating System (or Browser): Windows 10
Node Version: 12.13.0

How Do We Reproduce?

Reproduction steps are outlined in this reduced test case repo:
https://github.com/Dan503/BUG-rollup-plugin-node-resolve/tree/CommonJS-bug
Note this part of gulpfile.js in particular:
const rollupJS = (inputFile, options) => {
	return () => {
		return (
			rollup({
				input: options.basePath + inputFile,
				format: options.format,
				sourcemap: options.sourcemap,
				plugins: [
					CommonJS(), // <<-- This is the line that calls rollup-plugin-commonjs
				],
			})
				.pipe(source(inputFile, options.basePath))
				.pipe(buffer())
				.pipe(sourcemaps.init({ loadMaps: true }))
				.pipe(sourcemaps.write('.'))
				.pipe(gulp.dest(options.distPath))
		)
	}
}

Expected Behavior
It should not error
Actual Behavior
It throws this error when trying to run the code:
TypeError: Cannot read property 'resolve' of undefined
    at resolveId (C:\xxx\node_modules\rollup-plugin-commonjs\dist\rollup-plugin-commonjs.cjs.js:157:17)
    at C:\xxx\node_modules\rollup-stream\node_modules\rollup\dist\rollup.js:1750:32
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

I can confirm I have the same issue. Here is my gulp file:
import { dest, parallel } from "gulp";
import rollup from "rollup-stream";
import nodeResolve from "rollup-plugin-node-resolve";
import commonjs from "rollup-plugin-commonjs";
import vue from "rollup-plugin-vue";

const js = () =>
  rollup({
    input: "src/js/index.js",
    output: {
      format: "iife",
      file: "index.js"
    },
    plugins: [commonjs(), nodeResolve(), vue()]
  }).pipe(dest("dist/js"));

const build = parallel(js);

export { build };

Here is the error my console prints:
$ yarn build
yarn run v1.19.1
$ gulp build
[22:48:37] Requiring external module @babel/register
[22:48:39] Using gulpfile C:\xampp\htdocs\expense-manager\gulpfile.babel.js
[22:48:39] Starting 'build'...
[22:48:39] Starting 'js'...
[22:48:39] 'js' errored after 38 ms
[22:48:39] TypeError: Cannot read property 'resolve' of undefined
    at resolveId (C:\xampp\htdocs\expense-manager\node_modules\rollup-plugin-commonjs\src\resolve-id.js:54:15)
    at promise.then.result (C:\xampp\htdocs\expense-manager\node_modules\rollup-stream\node_modules\rollup\dist\rollup.js:1750:32)
[22:48:39] 'build' errored after 80 ms
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

However, with the same config but running native rollup, no errors:
// rollup.config.js
import nodeResolve from "rollup-plugin-node-resolve";
import commonjs from "rollup-plugin-commonjs";
import vue from "rollup-plugin-vue";

export default {
  input: "src/js/index.js",
  output: {
    format: "iife",
    file: "index.js"
  },
  plugins: [nodeResolve(), commonjs(), vue()]
};

$ yarn rollup --config
yarn run v1.19.1
$ rollup --config

src/js/index.js → index.js...
created index.js in 621ms
Done in 1.66s.

Do you know if there is something linked with rollup-stream or the plugins?

Hey folks, we've moved this repo to a new home in our plugins monorepo at https://github.com/rollup/plugins. If this issue is still pertinent, please reopen, by filling out the appropriate issue template, in the new repo. Thanks for originally opening this issue, and we look forward to seeing you in the new repo. This issue is being closed and this repo is being permanently archived.


Implement Transactor Permissioning

This includes transactor permissioning checks when receiving a batch from a client and on block validation.  The permission verifier uses identity view to check the corresponding roles stored in state. If there is not role stored, the default policy is used. If the default policy is not stored, permit all transactors.

rebase before merging


fix(side-panel): refactor to match latest ibm-products

Related Ticket(s)
https://github.com/carbon-design-system/ibm-products/pull/4141
Description
Updates structure of SidePanel following restructure of @carbon/ibm-products SidePanel. This simplifies the HTML structure and title scroll animation.
Changelog
Changed

packages/carbon-web-components/src/components/side-panel/side-panel.scss
packages/carbon-web-components/src/components/side-panel/side-panel.ts

@ariellalgilmore corrections made, there were a few 'cds-' now replaced.


Documentation for parameter usage in HuggingFace runtime missing

I am interested in using MLServer with HuggingFace models. To get the full potential of these models, it is essential to modify the model parameters (https://huggingface.co/docs/transformers/main_classes/text_generation). I have tried to change these parameters, but it is not straightforward to me. Therefore, it might be a good idea to extend the documentation with a section on how to modify parameters like the temperature or the num_return_sequences.

Hi! Are there any news or plans on adding opportunity to pass such transformers config parameters as 'max_length' using Seldon HuggingFace server?

There're plans but we haven't got around it yet. We'll provide an update on this issue once it's done.
Alternatively, contributions are always welcome :wink:

Hey @coctel99, I'm no longer working for Seldon and no longer contributing to MLServer so I don't have any extra context on this one.
@sakoush has taken the reins of the project, so it'd be best to check with him.

@ramonpzg for reference

Hi @coctel99 -- I am taking care of this one for the next release :)
If you are in the community, I'll send you a message once it is merged.


[AzureMonitorDistro] remove vendored instrumentation libs (AspNetCore & Http)

OpenTelemetry.Instrumentation.AspNetCore and OpenTelemetry.Instrumentation.Http have both released stable versions. We can now remove the internal vendored copies of the beta librares and ship using package references instead.

Did you test these changes locally? We have a reflection logic in OpenTelemetryBuilderExtensions.cs that also needs to be removed.

Did you test these changes locally? We have a reflection logic in OpenTelemetryBuilderExtensions.cs that also needs to be removed.

not yet. investigating why the build is failing, it's compiling locally.


Formatting issue in output-clause-transact-sql.md at Arguments/column_list/column_name

See https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/t-sql/queries/output-clause-transact-sql.md
Under column_name, it says:

column_name
Is an explicit column reference. Any reference to the table being modified must be correctly qualified by either the INSERTED or the DELETED prefix as appropriate, for example: INSERTED**.**column_name.

There should not be “**.**”. It should just be “.”.

@binki -- Nathan, thank you again for your feedback.  I am updating this article.


Responsive contact page

This is the responsive contact page. Again, this has a background image which didn't look good on smaller devices so I replaced it with a gradient.
Here's the responsive design documentation I wrote for this page:
https://orble.atlassian.net/wiki/x/hADyBQ

After sending the message it sends me to a blank page - it should redirect me to the contact page or back to home

@JanaviChadha yes, this page is not published to the website yet, there will be a PR in the future adding functionality to the contact page, since it requires some server-side code that I'm working on rn


Add counterpartyChannelId param to IBCModule.OnChanOpenAck

Description
ref: #1075
I add counterpartyChannelID as a param to IBCModule.OnChanOpenAck to support ibc module that use counterpartyChannelID in their OnChanOpenAck logic. #1075
closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@joe-bowman -- is this a good solution for supporting cosmwasm with ibc-go v3 and cosmos-sdk 46?

Codecov Report

Merging #1076 (6df89ed) into main (fc452ac) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main    #1076   +/-   ##
=======================================
  Coverage   79.81%   79.81%
=======================================
  Files         151      151
  Lines       10882    10882
=======================================
  Hits         8685     8685
  Misses       1774     1774
  Partials      423      423

Impacted Files
Coverage Δ

...les/apps/27-interchain-accounts/host/ibc_module.go
100.00% <ø> (ø)

modules/apps/transfer/ibc_module.go
63.12% <ø> (ø)

modules/core/keeper/msg_server.go
57.90% <0.00%> (ø)

...ps/27-interchain-accounts/controller/ibc_module.go
93.47% <100.00%> (ø)

issue closed by #1086


Getting WSL process's version info causes PH to go into a never ending loop

Starting a wsl bash shell causes PH to go into spasms... I see many wsl processes spawned by PH in quick succession in a never ending loop. This causes my CPU usage to spike to 100%.
Procmon recording:

Disabling WSL support mitigates this behavior.
Related issue: #393
If I understand correctly, the issue seems to be that dpkg -S /init  fails (probably because /init doesn't belong to any package). This failing result is not cached. For each wsl process launched by PH, a parallel init process is launched by Windows. This in turn causes PH to launch more wsl processes in order to retrieve the (non-existent) version info...

the issue seems to be that dpkg -S /init fails (probably because /init doesn't belong to any package).

the /init binary gets injected/copied into the disto folder by wsl.exe during initialization of the Linux distro hence there isn't any 'owner' package we can query for version information.
The first public version of the code included a check for /init replacing it with /sbin/init:
https://github.com/processhacker/processhacker/blob/830487082dc3b09cb2cf67caaa732b9279da2dfc/phlib/wslsup.c#L161-L166
The problem is the /sbin/init package is owned by systemd (which doesn't even work with WSL) and so it was removed because its not the correct version information (which should be Microsoft).

This failing result is not cached.

Erroneous results should never be cached?
There is a hard limit of 1 query for each WSL process. If the WSL query succeeds or fails there isn't ever a second query.

and so it was removed because its not the correct version information (which should be Microsoft).

agreed...

Erroneous results should never be cached?

This is not an erroneous result. This is a valid result that indicates that no version information is available. The fact that there is no version info available should be cached.

There is a hard limit of 1 query for each WSL process. If the WSL query succeeds or fails there isn't ever a second query.

That may be true, but each attempt at querying dpkg/rpm for info will spawn more init processes which will in turn spawn more dpkg-quey etc. processes leading to high CPU usage.

@yossizahn
I've revered the check on /init with commit https://github.com/processhacker/processhacker/commit/ffea1dc8e2582e2cd76811bf061df4b1754858f5

Does it fix the issue?

Yes. Thanks.

Not sure what happened to my original comment back in October :/

Yes. Thanks.

Thanks!

Hi,
Issue has returned :(

I have done some digging in WSL area. If you need any information I can provide some secrets. 🤫

Quick fix! Thanks!


Convert yield tests to pytest parametrized tests

(On top of gh-7572)
Convert all nose yield test generators to either:

plain for loops
pytest.mark.parametrize

The reason for using nose yield tests previously was partly because nose test failures do not give much information on e.g. which test case the failure occurred at in for loops. Pytest can however print local variables (enabled by default in scipy.test), so this reason for parametrization becomes less relevant. So in many tests where there is no strong reason for parametrization, such as if there's just a  for loop over test cases, the parametrization can be just removed.
In other cases, the yield tests are converted to pytest.mark.parametrize, for example in stats tests that loop over different distributions. The advantage of parametrization is that you can run each test separately if needed (this works in pytest, wasn't possible in nose iirc). The natural code structure here seems to be that the yield for loop goes to a separate cases_* method that generates the parameter values, and the check method becomes the test body.
(As a nice bonus, after this, parallelized test runs work with pytes-xdist; cf. python3 runtests.py -- -n 4)

As a nice bonus, after this, parallelized test runs work

That is awesome! Should really take care of our test timeout issues - 'full' run on TravisCI now took 18 min rather than the normal >=35 min.

LGTM and all green, so merging. Thanks @pv!


Possible duplicate sessions

Opening an issue I actually won't be able to work on because I don't know most the underlying tech.
I've installed shynet but I see some "weird" hit counts and session counts.
In short inspecting the activity for the sites shows mostly three recordings for each person, with the same IP, most of the same hits but wiht minor differences.
I'm unsure if this is actually people loading the site three times, which I find doubtful in the amount of times I see it, or if t's some conflict in the way that sessions are grouped together.
Example.

Can you show the session details view for each of those sessions? I’m curious if this is someone’s browser calling the <noscript> image fallback.

nope, all of the sessions use the JS tracker.

Alright, I'll take a look at this. I haven't been able to replicate this issue, but will see if anything turns up.
(By the way—@Windyo I edited your comment to remove the screenshots, as I wanted to avoid the potentially private IP information from floating around. Just an FYI in case you were wondering what happened.)

I can reproduce it on my sites - if needed I can create you a shynet user on my instance. I can't give you ssh access though.

@Windyo by the way, do you have multiple frontend servers running?
I could see this being an issue if there are multiple frontend servers that aren't connected to the same cache backend.

Hmmm.
I do have multiple instances of servers running as everything is dockerized, but only one server per website.
Meaning the wiki's on server, with its own database, and the other websites as well.

Oh, I know what’s happening. Gunicorn is launching 3 instances of Shynet, which aren’t connected via the same cache. That’s causing the duplication. The fix is easy, fortunately: have Gunicorn launch only one instance. I’ll push an update to fix this shortly.

Alright, this is fixed in v0.3.2. Please reopen if you find the issue continues.
Thanks for the report! This is a pretty shocking issue to be honest; it's likely that it affects pretty much all default Shynet instances except for my own (because I use an external Redis server). So I would not have seen this if it weren't for your report!


[fix #7814] fix java function logging appender not added to java function logger

Fixes #7814
Motivation
JavaInstanceRunnable create an instance of logger named "function-" + instanceConfig.getFunctionDetails().getName() and pass it to Function Context, and the logger can be used to send user defined content to function's log-topic if --log-topic defined.
as issue #7814 mentioned, the logger is not working as expected since user cannot consume any self defined log content from log-topic.
this happens in process runtime with created functions, but not noticed with other situation such as localrun function.
Through debug to the created function, the logger in Function Context is different from the logger in JavaInstanceRunnable, such as the contextName as images shown below. In addition, the LogAppender set in JavaInstanceRunnable is not shown in Function Context's logger as well.

^^^^ from JavaInstanceRunnable

^^^^ from Function Context
After some tests, I find out that when get LoggerContext by LoggerContext.getContext(), the context's logAppender can be take effect to Function Context, and the Function Context's logger works great.
Modifications
Add LogAppender to the single context from LoggerContext.getContext().
Verifying this change

[ ] Make sure that the change passes the CI checks.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

Can we add a test?

@zymap thanks for the review, an integration test was added, please take another look when you have time ;)

Can we add a test?

@zymap thanks for the review, an integration test was added, please take another look when you have time ;)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

oops. The test seems failed. Please take a look.

oops. The test seems failed.


Installation of Java fails on Cygwin

Hi,
I tried to install Java (through bash from CygWin), but I get the following error message
-bash: /cygdrive/c/Users/thisuser/.sdkman/tmp/g22MTLT714qgj87zDVSwR8qkGIy7Znvg.bin: Permission denied
mv: cannot stat '/cygdrive/c/temp/jdk': No such file or directory

zip error: Nothing to do! (try: zip -qr /cygdrive/c/Users/thisuser/.sdkman/tmp/g22MTLT714qgj87zDVSwR8qkGIy7Znvg.zip . -i .)

Done repackaging...
mv: cannot stat '/cygdrive/c/Users/thisuser/.sdkman/tmp/g22MTLT714qgj87zDVSwR8qkGIy7Znvg.zip': No such file or directory
unzip:  cannot find or open /cygdrive/c/Users/thisuser/.sdkman/archives/java-8u131.zip, /cygdrive/c/Users/thisuser/.sdkman/archives/java-8u131.zip.zip or /cygdrive/c/Users/thisuser/.sdkman/archives/java-8u131.zip.ZIP.
rm: cannot remove '/cygdrive/c/Users/thisuser/.sdkman/archives/java-8u131.zip': No such file or directory

Stop! The archive was corrupt and has been removed! Please try installing again.

Any idea what might be going on? (Environment variable, permissions?)
Thanks!

run your Cygwin in Administrator Mode and give a try.
Note: still it may not work because above stated Path'/cygdrive/c/temp/jdk' is not correct
looking for the way to change this path to .sdkman/tmp instead

Please follow the other thread.
https://github.com/sdkman/sdkman-cli/issues/520
it is better to close this one

Running the CygWin bash shell in Administrator mode installed the package.

Found one alternative for Time being,
zip JDK7 home folder as java-7u80.zip copy inside "SDKMAN_HOME/achives"
then run sdk install java 7u80
this time the installation will be success, since the binary is already downloaded in right place.
Similarly you can do the same for 8 and 9
hope this helps for now.

Mark as duplicate #520


[community] update collaborators

Proposed changes
Issue Number: close #xxx
Problem summary
Add @Yukang-Lian as collaborator, and he helped a lot in good first issue.
Remove @luzhijing from collaborator because he has been Doris' committer.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

Thanks for the trust, it's my pleasure!


Add trailing newline on the last line.

Description

Add trailing newline on the last line. It is easier to work with files where all lines are properly terminated.
How Has This Been Tested?

Not tested.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@adelton This just needs a rebase for a conflict in the Makefile.

Rebased on main.


has any way to exclude ionic components and theme?

i want to use ionic2 with a clean style environment for use customize or 3rd part components, the ionic2 themes and styles has many destructive.

Ionic without Ionic components or styles is Cordova, so you could just use that.

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


Parsing BBEdit Sparkle feed causes error

Describe the bug
Trying to create a recipe for BBEdit by dropping the app on RR fails with the following
 Processing: /Volumes/Macintosh HD/Applications/BBEdit.app
Input path looks like an app.
Validating app...
    This app seems valid
Getting bundle name...
    Bundle name is: BBEdit
Getting bundle identifier...
    Bundle identifier is: com.barebones.bbedit
Checking for a Sparkle feed...
    Sparkle feed is: https://versioncheck.barebones.com/BBEdit.xml
Getting information from Sparkle feed...
[ERROR] Recipe Robot exploded with unexpected error: max() arg is an empty sequence
Traceback (most recent call last):
  File "/Applications/Recipe Robot.app/Contents/Resources/scripts/recipe-robot", line 92, in main
    process_input_path(facts)
  File "/Applications/Recipe Robot.app/Contents/Resources/scripts/recipe_robot_lib/inspect.py", line 177, in process_input_path
    facts = inspect_func(input_path, args, facts)
  File "/Applications/Recipe Robot.app/Contents/Resources/scripts/recipe_robot_lib/inspect.py", line 320, in inspect_app
    facts = inspect_sparkle_feed_url(sparkle_feed, args, facts)
  File "/Applications/Recipe Robot.app/Contents/Resources/scripts/recipe_robot_lib/inspect.py", line 2035, in inspect_sparkle_feed_url
    latest_info = max(sparkle_info, key=lambda x: APLooseVersion(x[key]))
ValueError: max() arg is an empty sequence

To Reproduce
Steps to reproduce the behavior:

Dropping BBEdit.app on RR's Gui area
See error above

Expected behavior
RR should create the individual recipe parts
Desktop (please complete the following information):

macOS: 10.13.6
Recipe Robot Version: 2.2.0
autopkg version: 2.4.1
/usr/local/autopkg/python --version: 3.7.5

A good catch — BareBones doesn't actually use Sparkle for their update check XML. I might adjust Recipe Robot to accommodate their format.
However, I'd suggest that most BareBones apps already have recipes in the AutoPkg org on GitHub. BBEdit, for example. Still, good for RR to know what to do in case a new app is released.

Thank you.
I am aware that a recipe for BBedit already exists. However, it fails to download anything.
(I had reported an issue about this and just checked, only to see that you had replied. This somehow escaped my notice; I am sorry about that.)
This is the reason I was trying to create a separate recipe for it with RR.

RR 2.3.0 now contains special handling for Bare Bones appcast feeds. However I'm getting a ditto: Couldn't read PKZip signature error when running the generated download recipes in AutoPkg 3.0rc2. I'll look more into this.


typechecker: Check whether each match arm returns

This fixes #394.

Can we have a test for the error condition too?

Done.


Not working for me on mac. Even not open

Can you describe in more detail what happens? Does it crash? Or do you get an error when trying to open the application?


Remove gist sidebar and docs about playground plugins

Probably also worth editing the Type or Treat blog posts e.g. https://devblogs.microsoft.com/typescript/type-treat-2021-day-1/ if you can , as these now won't work

Blog links (all the ones I could find anyway) are updated to the underlying gist


Supported frameworks since Cake 3.0 are net7.0 and net6.0

Guidance needs to be updated
https://github.com/cake-contrib/CakeContrib.Guidelines/blob/10515df04f9d3ff65dd6dcbfc042200a09ee3cba/src/Tasks/TargetFrameworkVersions.cs#L40-L42
Need to add 3.0 similar too
https://github.com/cake-contrib/CakeContrib.Guidelines/blob/10515df04f9d3ff65dd6dcbfc042200a09ee3cba/src/Tasks/TargetFrameworkVersions.cs#L89-L105

Fixed in #218

@all-contributors please add @devlead for bug.


Major Update SR6thEdition

Changes / Comments
RENAMED from SR6D to SR6 in general
Changed native language to English
Fixed a lot of existing bugs
Started to implement Matrix functionality
Cared for DE, EN, ES (thanks for providing Andres Martinez!) translation files

@yagrima you PR has some conflics, also it includes changes to translation files. We would not deny your Pull Request, but we would prefer sheet contributors use Crowdin for translations. Please email Cassie at<EMAIL_ADDRESS>and she will get you an invite to our Crowdin. Be sure to include the languages you would like to translate, and please use Crowdin for future translation updates! Thanks for all you do!

What do you suppose me to do now? I already have an CrowdIn invite and want to see the sheet updated.

Best wishes
Gesendet: Montag, 02. November 2020 um 21:19 Uhr
Von: "Miguel Peres"<EMAIL_ADDRESS>An: "Roll20/roll20-character-sheets"<EMAIL_ADDRESS>Cc: "yagrima"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Betreff: Re: [Roll20/roll20-character-sheets] Major Update SR6thEdition (#7588)

@yagrima you PR has some conflics, also it includes changes to translation files. We would not deny your Pull Request, but we would prefer sheet contributors use Crowdin for translations. Please email Cassie at<EMAIL_ADDRESS>and she will get you an invite to our Crowdin. Be sure to include the languages you would like to translate, and please use Crowdin for future translation updates! Thanks for all you do!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Dear @yagrima your PR has a conflict within "Shadowrun6th-German/translations/es.json" which needs to be resolved before I can merge it. Please rebase you PR against last Master and fix this issue. If you are unsure how to do this, take a look at the Git documentation.

As I am not that used to all the Git stuff. Can I just make another PR without the es.json?

Regards.

Gesendet: Montag, 09. November 2020 um 12:45 Uhr
Von: "Miguel Peres"<EMAIL_ADDRESS>An: "Roll20/roll20-character-sheets"<EMAIL_ADDRESS>Cc: "yagrima"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Betreff: Re: [Roll20/roll20-character-sheets] Major Update SR6thEdition (#7588)

Dear @yagrima your PR has a conflict within "Shadowrun6th-German/translations/es.json" which needs to be resolved before I can merge it. Please rebase you PR against last Master and fix this issue. If you are unsure how to do this, take a look at the Git documentation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Dear @yagrima, yes. That should be ok, once that is done I can merge the new PR and delete this one.