Multiple login prompts when using an account with MFA enabled

Storage Explorer Version: 1.3.1
Platform: Windows
Architecture: ia32
Build Number: 20180711.1
Commit: 9a954726
Steps to Reproduce:
Using an MSA account (hotmail, live, etc).
Account is tenant admin in Azure and has MFA enabled in Azure.
Account has two subscriptions (one PAYG, another Visual Studio credits).
Signing into storage explorer, I have to login three times.
First time:  Username-Password.
Second and third times:  Username, Password, MFA text message code.
Expected Experience:
Login once.
Actual Experience:
Have to login three times, as described above.

I also have to login 3 times. When this was only 2 times with MFA I was promised it was fixed in a future release. In fact it is worse. One thing I would note is that once I finally login my pay as you go account is listed 3 times but does not expand to anything.

I also have to login 3 times!

This is by far the worst thing about storage explorer. I work as a consultant and have access to many azure subscriptions. Many of which require MFA (for good reasons). I get 10+ login prompts now. I try to only accept the first one, and then just close all the others, eventually it usually works, but it is so annoying that I'm starting to use access keys instead, which is very bad security practice.
Could you please add an option to just login to a single (user defined) tenant, I'm okay if I have to start storage explorer with a commandline argument that specify tenantId or something similar, but the current situation is so frustrating.

I hope the fact this has been added to a milestone means it will soon be addressed (after nearly two years!).  Really, it is quite a bad experience to get annoyed with an application even before you have finished opening it!

Could you please add an option to just login to a single (user defined) tenant

@sjkp , this is where we would like to go in the future. Unfortunately as of right now though, logging into every tenant is the only option the auth library we are using exposes.

I hope the fact this has been added to a milestone means it will soon be addressed

@cbailiss I hate to dash your hopes, but I've done some more digging, and this unfortunately is not going to be fixed for 1.14. I'm not even sure when we'll get the capability to only log into user specified tenants, so I'm going to be moving this to a later milestone to check in on it then. I definitely understand how the experience is currently cruddy

Same issue for my team. This is a real problem because it encourages bad practices like using connection string instead login with our user who have been authorized to access storage account. Multiple tenants and multiple subscriptions all with MFA require a very long time before accesssing just one azure blob storage. I hope it will get fix really soon.

Proof that we are working on something:

Plan is:

you'll only be signed into home tenant initially
if you want to work with other tenants you can filter those in/out as needed
when you filter tenants in, if they need MFA you'll know, and you can do MFA for that specific tenant
you'll be able to use those chevrons to collapse tenants/subscriptions lists
Disclaimer: design may change slightly between now and release, but if y'all have feedbacks for it let us know.

Do you have an ETA for this feature?  I have 25 Tenants with many subs underneath. This leads to many, many MFA logins and 100's of subscriptions to parse through.

The best way to track when a feature will be released is to:

check the issue
look at the milestone it is in
look at the due date for that milestone

The due dates for most of our milestones are at or around when we expect so ship it. Granted, dates can change and features/bugs can be cut for various reasons.
For this specific feature, I'm really hopeful/determined that it makes 1.19 (current due date of April 15th).  I'll do my best to keep y'all updated with my progress. For sure once I have the UI hooked up to real business logic, I'll try to to post a preview version here for y'all to demo. Thank you for all the patience!
 This build contains the new account panel as I previewed in my screenshot. Before links, let me go over everything there is in this build:

When you sign into Storage Explorer, you will only be signed into your home tenant. If you already signed in though, all of your tenants will still be active.
If you want to filter out/in tenants, you can now do so via the account panel. Under each account in the panel, there are two levels of checkboxes. The first level is for tenants. When you check those on and off they filter then tenants in and out. The exception to this is your home tenant, that cannot be filtered out at this time.
When you filter out a tenant, Storage Explorer will not load subscriptions for it and you will not be able to interact with resources in that tenant.
When you filter in a tenant, Storage Explorer will attempt to load subscriptions for it. If subscriptions can be loaded, they will load. If they cannot, you can view whatever error was encountered during loading and attempt to reauthenticate for that specific tenant.
One tenant failing to load does not block other tenants.
The checking of tenants/subscriptions is now immediate. You no longer have to click apply on the account panel.
The tree view no longer does a full refresh after changing what subscriptions you have filtered in/out. Instead, subscription nodes are just added/removed as needed.

I think that basically covers all there is for the account panel. There's other changes in the build that will be a part of 1.19 but I won't get into those.
So in general with this build:

Please try it out! Although I have some test accounts which are in 2 or 3 tenants with various MFA requirements, they are still just test accounts. Whether or not these changes work for y'all and your accounts is what matters. So if you find something wrong please comment here!
Since this is a preview build, we do not recommend working against production data with it. Be careful and try not to use it for more than is needed to evaluate these changes.

Ok, link time!

Windows installer: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel/StorageExplorer-ia32.exe
macOS .zip: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel/StorageExplorer-darwin-x64.zip
Linux .tar.gz: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel/StorageExplorer-linux-x64.tar.gz

@MRayermannMSFT - I installed it, and everything just worked fantastic.  This is a significant and needed improvement, and thank you for the changes here!
The only thing I'd please like to see yet is at least an option to include the tenant display name, tenant GUID, or even subscription GUID next to each loaded subscription in the Explorer.  The majority of the tenants that I work in are all "Azure Subscription 1", so I have what appear to be duplicate accounts in the Explorer:

Azure subscription 1<EMAIL_ADDRESS>Azure subscription 1<EMAIL_ADDRESS>...

... with no way to differentiate between them without recognizing the different storage account names, etc., contained within.  In the "Properties" panel, after clicking on each, it at least shows the "Subscription ID", along with "Account Email" and "Environment" (but here, the "Account Email" and "Environment" are also all identical).  Ideally, the tenant display name and tenant GUID would also be included as properties here - with at least one additional bit available for display inline in the tree.  I can open a new issue for this, if desired.

@ziesemer, I'm glad to hear the preview build seems great!
With regards to your feedback, right now we have #2621 to track adding a tenant node to the tree view. We'll be revisiting that issue once this one is wrapped. So please follow/comment on that issue (if you haven't already) so we can know to tag you if we have any questions regarding what we should do in that area.
For 1.19 though, it shouldn't be too hard for me to add tenant ID to the properties panel real quick today.

Thank you very much for working on this sir!
I was able to get the test version installed, and the new Account Management screen does look very useful.
I'm one of those PITA users with access to 7 tenants, 4 of which have their own layers of MFA (which makes the current stable release practically unusable).
In my testing of this new preview build, I was unable to access a non-home tenant that had its own MFA.  When I clicked "Reauthenticate now" under the non-home tenant a browser window would open, quickly pass through SSO (without any interaction on my part) and land on a local page that served the message "Authenticated. You can return to Storage Explorer. You might need to authenticate again if you close this browser tab.".  Back in storage explorer, it would try to then authenticate and list the subscriptions, but then return to the message of "Subscriptions could not be loaded for this tenant. You may need to reauthenticate. See the error details for more information.".  Clicking on the error details shows this message:
{
  "message": "\"{\\n  \\\"message\\\": \\\"\\\\\\\"AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access 'redacted'.\\\\\\\\r\\\\\\\\nTrace ID: redacted\\\\\\\\r\\\\\\\\nCorrelation ID: redacted\\\\\\\\r\\\\\\\\nTimestamp: 2021-03-24 19:23:28Z\\\\\\\"\\\"\\n}\""
}

(I redacted the GUIDs in the error details above.
So I like the UI and the workflow, but it doesn't seem to be triggering the MFA step for my non-home tentants as it should.  I'll try to switch my default browser to one that isn't enabled for SSO perhaps...
Happy to provide any additional information or even do a screenshare if it would help.  Thanks!

@shea-parkes are you on Windows? If not, can you get access to Windows machine to test on? I'd like you to grab a fiddler trace and some other logs so we can see what reauthentication is being done.

@MRayermannMSFT tested your build and if authentication works directly it looks ok in the list. But if there is any error and i am asked due to error to "Reauthenticate now.." if i press that link the browser opens up and ask me to login to another tenant then the directory i clicked the link for.

@MRayermannMSFT - Thanks for the swift reply sir.  It was a Windows machine I was having issues on.  That one was an ephemeral VM alas.  But I can setup the test on my laptop (that is running Windows).  This is pretty important to us, so if you can provide some instructions for what specific logging you'd like I'd be happy to run the tests for ya.

I confirmed I am able to reproduce the error of "Reauthenticate now..." not triggering an MFA check on my non-home tenants on my laptop running Windows.  Let me know what logging you'd like gathered and I'd be happy to gather it and help out.

For both @shea-parkes and @dlindblom, attached to this comment are some instructions for you to gather us logs. Additionally, I have a new version (FYI: it's not signed) for y'all to try if you have time to install that as well:
https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel-20210325.3/StorageExplorer-ia32.exe
IMPORTANT: the Fiddler trace generated as part of this is going to contain passwords you entered/sent in your browser during the gathering of the trace. Please read the instructions on how to remove passwords from the Fiddler trace. If you have any questions regarding the instructions, or you do not feel comfortable sending/cannot send the Fiddler trace, let me know. Ideally we get the Fiddler trace, but we can get a lot of good info even without it/we can be creative about extracting the information needed from it.
instructions.txt
@dlindblom So you're saying that:

You signed in, everything was good
At some point, the home tenant had an error
You tried to "reauthenticate now" on the home tenant, but the sign in flow took you to a different tenant
Yes?

If yes then:

Do you have any other tenants checked, or just the home tenant?
Do any of those other tenants also have errors?

Hey @shea-parkes and @dlindblom, I think we've identified what was going wrong. Feel free to hold off on getting logs until I get a new version to y'all to try. Thanks! :)

@MRayermannMSFT ok. To your question yes and only home tenant was selected.
After having these problem for very long time it's nice to see the issue addressed but i see same problem with other tools as well and not only Storage Explorer. This problem exist also with i.e Datastudio, even SQL Management Studio try to logon to wrong tenant with MFA (not the home tenant where SQL Server is located).  I happy for a solution coming in Storage Explorer but would be good if you are able to share findings with other developer teams so we do get solution for all tools/authentication.

@shea-parkes and @dlindblom, here's the new build:

macOS: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel-20210329.2/StorageExplorer-darwin-x64.zip
Windows: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/account-panel-20210329.2/StorageExplorer-ia32.exe

@dlindblom yes, it is definitely a tricky problem to solve. We will be certain to pass our learnings along to other teams. :)

We're locking down for the 1.19 release fairly soon. If anyone has any feedback to share, or is having an issues with, the new account panel now would be a great time to share!

@MRayermannMSFT latest build looks good for me.
Only feedback would be and i guess it's depends who the user is: Whenever you switch to Account Management it auto expand list of subscriptions for all tenants that you have at least one subscription selected. This list can be very large if you work i.e as consultant with multiple tenants, but this might not apply to the majority of the users which may prefer auto expanding list.

Hey @dlindblom yes, I feel exactly the same way! I have access to many many subscriptions in the home tenant of my work account. And all of my "testing" accounts are sorted after my work account in the list. I often get annoying at having to scroll/re-click to collapse the tenant after doing a restart. I'll keep trying to think of a better solution for deciding when to auto expand...
Thanks for the feedback on the latest build!

It works fine thanks :)

Installed it yesterday... THANK YOU soooo much... Storage Explorer for me (as a consultant) is usable again!

Nice! Glad to hear @wolfgangstrasser. What a great way to end my day. 🙂  We should be shipping tomorrow as planned, so I'm going to go ahead and close this issue now. Many thanks to y'all for the help this past month and half, and for the two years of patience.


Rename 'modshogun' to 'shogun'

https://travis-ci.org/shogun-toolbox/shogun/jobs/249789498#L3545

I don't get it yet

@karlnapf ready to go. It is a major change so please take a look.

shogun.Classifier etc stops working once merged.

@lisitsyn we kind of deprecated that already a longer time ago :D (submodules)

@lisitsyn although i dont get why, i mean both in case of python as well as in case of java it'd be great to have a module for classifier etc. no?

@vigsterkr yeah namespaces are good but IIRC in the end they were too intertwined.

@lisitsyn but that's rather because we are not clear on that internally... or you reckon it's unavoidable..?

Yeah too many dependencies to have separate modules so we gave up :)

YOLO!


Data Controller causes a crash when switching scenes.

Describe the bug
When an object in a scene is freed (due to scene change) the attached Data Controller causes a crash.
Error
Attempt to call function 'get' in base 'previously freed' on a null instance.
Stack Trace
0 res://addons/panku_console/modules/data_controller/exporter/exporter_2.gd :122
Code line
row.update_ui(obj.get(prop_name))
To Reproduce
Steps to reproduce the behavior:

Create a project with 2 scenes
Run one of the scenes
Open Panku console and open a data controller for a node in the scene current.get_node("MyNode")
Go to a different scene
See error (Godot Debugger)

Expected behavior
Data Controller changes output to a notification such as "Missing node" , without interrupting the flow.
Additional context
A simple check for the existence of obj in the update_rows() function will prevent the crash.
I am very new to PankuConsole, maybe I am using it incorrectly?

Add is_instance_valid() check, I hope it works now.

@thiscris is it fixed now?


Initial version of new, refactored metabolic process, metabolism_redux.py

Only uses data from sim_data.py, doesn't "jump" to reconstruction/dataclasses/metabolism.py
No modular_fba.py, about 75% fewer lines of code. Reduced complexity.
Runs somewhat faster. Could be improved with warm start.

Interesting, pytest is saying:
ERROR ecoli/analysis/antibiotics_colony/amp_plots.py - ModuleNotFoundError: N...
[434](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:435)
ERROR ecoli/analysis/antibiotics_colony/amp_plots.py
[435](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:436)
ERROR ecoli/analysis/antibiotics_colony/exploration.py - ModuleNotFoundError:...
[436](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:437)
ERROR ecoli/analysis/antibiotics_colony/exploration.py
[437](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:438)
ERROR ecoli/analysis/antibiotics_colony/plot.py - ModuleNotFoundError: No mod...
[438](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:439)
ERROR ecoli/analysis/antibiotics_colony/plot.py
[439](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:440)
ERROR ecoli/analysis/antibiotics_colony/spatial_correlation.py - ModuleNotFou...
[440](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:441)
ERROR ecoli/analysis/antibiotics_colony/spatial_correlation.py
[441](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:442)
ERROR ecoli/processes/antibiotics/antibiotic_hydrolysis.py - ModuleNotFoundEr...
[442](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:443)
ERROR ecoli/processes/antibiotics/antibiotic_hydrolysis.py
[443](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:444)
ERROR ecoli/processes/antibiotics/antibiotic_transport.py - ModuleNotFoundErr...
[444](https://github.com/CovertLab/vivarium-ecoli/actions/runs/4641768599/jobs/8215066404?pr=193#step:8:445)
ERROR ecoli/processes/antibiotics/antibiotic_transport.py

These are all related to the antibiotics project. But it doesn't look like this PR changed these modules... What is going wrong?

Some of the antibiotics analysis scripts are using packages that are not supported on M1 Macs. We have discussed having an "auxiliary" requirements.txt for scripts that require less stable packages.

@thalassemia For now I just left the problematic packages in requirements.txt, even though I can't actually install them on my M1. All tests are passing.


configFilePath is not defined

When I run parcel watch src / index.html --public-url ./ - out-dir dist / I have the following output:
⚠  configFilePath is not defined     at Object.<anonymous> (/home/man1gold/Projetos/www/cnt/www/modulos/administrativo/relatorio/node_modules/parcel-plugin-change-file/index.js:8:26)     at Module._compile (/home/man1gold/.npm-global/lib/node_modules/parcel/node_modules/v8-compile-cache/v8-compile-cache.js:178:30)     at Object.Module._extensions..js (module.js:664:10)     at Module.load (module.js:566:32)     at tryModuleLoad (module.js:506:12)     at Function.Module._load (module.js:498:3)     at Module.require (module.js:597:17)     at require (/home/man1gold/.npm-global/lib/node_modules/parcel/node_modules/v8-compile-cache/v8-compile-cache.js:159:20)     at localRequire (/home/man1gold/.npm-global/lib/node_modules/parcel/src/utils/localRequire.js:10:10)     at <anonymous>
And the script in my index.html still remains commented out in dist / index.html:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>CNT Sistemas - Gerenciador de Relatórios.</title>

@rogeriomq @ymzuiku  I do get the same error as well.

I had the quite same issue. One look into the sourcecode shows that the plugin is trying to either
A: get the config from the package.json
B: read the confg from the file parcel-plugin-change-file.js
In my case (and yours) the file can't be read but you can still place the configuration in the package.json like this:
{
  "name": "app",
  "version": "1.0.0",
  "private": true,
  "dependencies": {
    "parcel-bundler": "^1.10.3"
  },
  "devDependencies": {
    "parcel-plugin-change-file": "1.3.0"
  },
  "parcel-plugin-change-file": {
    "timeout": 30, // setTimeout replace Html file
    "replaceName": "parcel-plugin-change-file", // default html replaceName
    "html": ['hello'], // change string to html
    "copy": ["src/assets"], // copy files in outDir
  }
}

@SeriousM
The plan B you mentioned is quite easy to realize by adding the following code  to the index.js of this plugin (can be found in the node_modules or just simply click the error log 😝 )
const configFilePath = path.resolve(process.cwd(), 'parcel-plugin-change-file.js');
// Assuming your parcel-plugin-change-file.js is in the root directory

Though you need to do this every time you install the plugin 😕

You can use patch-package to save changes you make to node modules and apply them after each install.

Aha great idea, the problem is perfectly solved then


shadow property for both plantuml exporters

currently we have c4plantuml.shadow
Could you please generalize this property for both exporters?

@simonbrowndotje thank you so much!
Sorry if I did not explain myself.
By "generalize" I meant to have one sibgle property for both exporters, rather than two properties.
Maybe not a property but some keyword ("shadowed" or "shadow true").
I use the same .dsl to generate both C4 and plain plantUML and I have to conditionally generate one or the other at build time, unless I can set both properties at the same time and I won't get any error.

Yes, you can set both properties at the same time.


Variable overflow when using more than 21 assignments

Hello there,
First let me thank you for this amazing plugin. Being able to deconstruct values and weaves implicit in for-comprehension is a game changer.
I'm using this plugin to power up the main function of my application. As such, there are a lot of straight assignments (dependencies) in it. I found out that, if there are 22 or more assignments consecutively, the plugin start to reuse variable name in the rewritten code. For example:
def reproducer = for {
    v1 <- Some(1)

    x1 = 1
    x2 = 2
    x3 = 3
    x4 = 4
    x5 = 5
    x6 = 6
    x7 = 7
    x8 = 8
    x9 = 9
    x10 = 10
    x11 = 11
    x12 = 12
    x13 = 13
    x14 = 14
    x15 = 15
    x16 = 16
    x17 = 17
    x18 = 18
    x19 = 19
    x20 = 20
    x21 = 21
    x22 = 22

    v2 <- Some(2)

    useVs = s"$v1$v2"
    useXs = List(x1, x2, x3, x4, x5, x6, x7, x8, x9, x10,x11,x12,x13,x14,x15,x16,x17,x18,x19,x20,x21,x22).mkString
  } yield s"$useVs/$useXs"

will fail to compile with the error
[error] Hello.scala:6:5: not found: value x$1
[error]     v1 <- Some(1)
[error]     ^

For now we bypassed the issue by inserting an artificial <- in the middle of the assignment list, but having to explain why this no-op flatMap is present is a bit confusing :)
22 reminds me of the tuple limit, so maybe there isn't much that can be done to fix this issue. But I though reporting it for others to see would not be a bad idea.

Just ran into this issue myself. :( Is a fix in progress?

I wound one more "workaround" - add implilcit0(useless: Int) = 42 somewhere in middle of xXX vals - creates virtual "block", which allows to compile code, similar as extra flatMap created by <-
Generated code is using TupleX as "closure" - seems that unwrapping of values has some problem


Creating Fungible Asset Scarcity

I already opened this issue in Algorand's default repo
Please read it here.
Thanks.

hey @ObiajuluM -
thanks for your contribution and inputs. this is a great approach to how we can consider fungible asset scarcity. one major roadblock is that Algorand will not fork so afaik the only viable approach here would be to consider updating to add a new global address.


Refactor findspam.py

Since we have a lot of similar objects in findspam.py (the rules, which are currently individual dictionaries), they're best wrapped into a class.
Some time earlier I already wrapped chat commands into a class (4629191ad3ca9af6156e740f1ede3a484861138f), but this time it's going to provide more maintainability than the previous one.

Rules can have default values for attributes and a more unified handling
Any function can be wrapped into a rule with a decorator, so adding a function-based rule only requires adding one block of code, not two. Currently you need to define a function and then add a dict into FindSpam.rules
Default values for rule properties, like default "max rep", similar to default arguments for the chat command decorator. This keeps things clean and minimal
Extending rule creation is possible (not implemented yet), like creating a new rule based on an existing rule, or templating

Idea: Maybe we can now separate post-checking logic and the actual rules? Like moving all those create_rules to a separate file, rules.py

More waiting to be discovered

The new way to create a spam check is very simple and easy
For regex check, simply call create_rule at the end of findspam.py like this:
create_rule("reason", "regex")

All other settings like sites or max_rep can be supplied as keyword arguments to create_rule. For example, this will create a rule with max rep being 30 and max score being 1, and it won't check answers.
create_rule("reason", "regex", max_rep=30, max_score=1, answer=False)

Note that questions and answers have been renamed to their singular forms (question and answer).
To create a functional check, use create_rule as a decorator, similar to chat commands:
@create_rule("reason", question=False)
def new_check(s, site):
    return True, "all answers are spam"

The default settings for create_rule is:
all = True
sites = []
title = True
body = True
username = False
body_summary = False
stripcodeblocks = False
max_rep = 1
max_score = 0
question = True
answer = True
disabled = False  # Create rule but don't add to rule list

How do you feel about using a namedtuple instead?

@j-f1 Looks like a better alternative. I'll try it tomorrow.

@j-f1 Not quite suitable here. Although it's going to be useful to some extexts, it's not pure data collection (bundled with methods) so namedtuple doesn't provide extra benefits. I'll stick to classes.

Ready to merge?

@ArtOfCode- Yes, please.


Added PostgresAdmin.database_in_recovery? method to PostgresAdmin class

During failover execution there should be no attempt to switch to database which is in recovery mode.
In this PR: added method PostgresAdmin.database_in_recovery?(connection)
@carbonin
@miq-bot add-label core, wip

@miq-bot rm-label wip

Looks good :+1:

👍 LGTM


TCP connection for managing the broker

To communicate to the broker a TCP connection on some port would be best. For things like channel join commands and such.

the tcp server works good now commands need to be thought of.
there should be a CONNECT command that needs like multiple parameters like
CONNECT username oauth:123 authkey123
where authkey is for getting access to the broker itself.
Maybe good to do: check if the oauth key is correct otherwise throw error.
After that anything from the same connection will be sent via the username.
And you can send commands to the broker like JOIN channel1 channel2 channel3 channel4 ...

prototype of JOIN channel1 is there, but the actual join of the channels is troublesome. Seems to work sometimes but sometimes not.
CONNECT or LOGIN or whatever I will call it is the next step before working on the connectionmanager


segmentation fault

When I run the command：tempo2 -gr plk -f example1.par example1.tim，P0 and P1 equivalents can be calculated well, but nothing can be displayed when drawing is enabled, and an error is reported:

why is this？I am using the conda command to download directly.

Sorry, is this an issue with libstempo or tempo2?

Yes, I don't know if it's the Mac system or tempo2. Because my colleagues ran into the same problem when they ran tempo2 on Mac, it can work if they switched to Linux

You'll need to try asking on the tempo2 repo page - maybe it's the same issue as https://bitbucket.org/psrsoft/tempo2/issues/70/segmentation-fault-with-the-plk-graphics.


2.x minor updates

NetflixOSS » eureka » eureka-pull-requests #199 SUCCESS
This pull request looks good


Miscellaneous functionalities

Add some improvements to the bot like:

[x] Profile picture
[ ] Dice roll
[ ] Coin flip
[ ] poll
[ ] ping
[ ] Timeout
[x] Time connected to a voice channel

I can help take a look at some of these features you want added to the bot.

Added Dice roll and Coin flip in #11. I would be happy to check out the other ones if you could elaborate on them.

willing to work on poll/ping

willing to work on poll/ping

Go for it!

Added Dice roll and Coin flip in #11. I would be happy to check out the other ones if you could elaborate on them.

You can work on timeout. It should mute someone for a predefined amount of time

Added poll bot under Miscellaneous functionalities #15


update poster from link

My movies.yml file is like this :

Alien - Saga:
template: {name: Collection, collection: 8091, summary: 8091, poster: https://theposterdb.com/api/assets/1213}
trakt_list:
- https://trakt.tv/users/benreilly97/lists/alien?sort=released,desc
collection_order: alpha
collection_mode: hide_items

But the collection which is created has the tmdb poster.
Is it normal that the poster is not updated with the one supplied in poster: https://xxxxxxx ?
Should i use template: {name: Movie instead of collection ?

if you're just using the templates from the sample file then neither Movie nor Collection would do what you want.
This is the collection template and it only takes two variables, collection and movie, so anything else given to it is just ignored i.e. poster and summary
  Collection:
    optional:
      - collection
      - movie
    tmdb_collection_details: <<collection>>
    tmdb_movie: <<movie>>
    sync_mode: sync
    collection_order: release

if you want summary and poster to work in the template you can add it here also if you use tmdb_collection_details it will update the poster background and summary of the collection so doing a summary variable is redundant.
  Collection:
    optional:
      - collection
      - movie
      - poster
    tmdb_collection_details: <<collection>>
    tmdb_movie: <<movie>>
    url_poster: <<poster>>
    sync_mode: sync
    collection_order: release

Alien - Saga:
  template: {name: Collection, collection: 8091, poster: https://theposterdb.com/api/assets/1213}
  trakt_list:
    - https://trakt.tv/users/benreilly97/lists/alien?sort=released,desc
  collection_order: alpha
  collection_mode: hide_items

or if you didnt want to edit the template just use url_poster in the collection like so
Alien - Saga:
  template: {name: Collection, collection: 8091}
  trakt_list:
    - https://trakt.tv/users/benreilly97/lists/alien?sort=released,desc
  url_poster: https://theposterdb.com/api/assets/1213
  collection_order: alpha
  collection_mode: hide_items


git errors more often now

I know this is an evergreen but since we selfhost on the new git extension I see this a lot more frequent. It's usually enough to sync with ~5 incoming and ~2 outgoing changes
fatal: Unable to create '/Users/jrieken/Code/vscode/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.
could not detach HEAD

Yeah I see this a lot too.

cc @aeschli

@egamma This fix should also fix what you observed this morning.

Verified based not having seen this in a while...


Doomgrader dotnet 2705 abort

Hello,
I run the script and the depotdownloader has a problem with dotnet for some reason. Here is the output of the script:
/run/media/nvme/.doomgrader/depotdownloader /run/media/nvme/.doomgrader/doomgrader
--2020-05-17 22:40:53--  https://github.com/SteamRE/DepotDownloader/releases/download/DepotDownloader_2.3.4/depotdownloader-2.3.4.zip

Saving to: ‘depotdownloader_2.3.4.zip’

depotdownloader_2.3 100%[===================>]   2.10M  10.5MB/s    in 0.2s

2020-05-17 22:40:54 (10.5 MB/s) - ‘depotdownloader_2.3.4.zip’ saved [2201394/2201394]

depotdownloader_2.3 100%[===================>]   2.10M  10.5MB/s    in 0.2s

2020-05-17 22:40:54 (10.5 MB/s) - ‘depotdownloader_2.3.4.zip’ saved [2201394/2201394]

Archive:  depotdownloader_2.3.4.zip
replace depotdownloader? [y]es, [n]o, [A]ll, [N]one, [r]ename: A
 extracting: depotdownloader
 extracting: depotdownloader.bat
  inflating: DepotDownloader.deps.json
  inflating: DepotDownloader.dll
  inflating: DepotDownloader.dll.config
  inflating: DepotDownloader.pdb
  inflating: DepotDownloader.runtimeconfig.json
  inflating: LICENSE
  inflating: protobuf-net.dll
  inflating: README.md
  inflating: SteamKit2.dll
  inflating: SteamKit2.pdb
  inflating: System.Reflection.DispatchProxy.dll
  inflating: System.Security.Principal.Windows.dll
  inflating: System.ServiceModel.dll
  inflating: System.ServiceModel.Primitives.dll
Connecting to Steam3... Done!
Logging '' into Steam3...Unhandled exception. System.ArgumentException: LogOn requires a username and password to be set in 'details'.
   at SteamKit2.SteamUser.LogOn(LogOnDetails details) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\Handlers\SteamUser\SteamUser.cs:line 299
   at DepotDownloader.Steam3Session.ConnectedCallback(ConnectedCallback connected) in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 504
   at SteamKit2.Internal.Callback`1.Run(Object callback) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr_Internals.cs:line 81
   at SteamKit2.CallbackManager.<>c__DisplayClass10_0.<Handle>b__1(CallbackBase callback) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 142
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at SteamKit2.CallbackManager.Handle(ICallbackMsg call) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 143
   at SteamKit2.CallbackManager.RunWaitCallbacks(TimeSpan timeout) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 71
   at DepotDownloader.Steam3Session.WaitForCallbacks() in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 478
   at DepotDownloader.Steam3Session.WaitUntilCallback(Action submitter, WaitCondition waiter) in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 141
   at DepotDownloader.Steam3Session.WaitForCredentials() in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 154
   at DepotDownloader.ContentDownloader.InitializeSteam3(String username, String password) in C:\Projects\DepotDownloader\DepotDownloader\ContentDownloader.cs:line 356
   at DepotDownloader.Program.MainAsync(String[] args) in C:\Projects\DepotDownloader\DepotDownloader\Program.cs:line 196
   at DepotDownloader.Program.Main(String[] args) in C:\Projects\DepotDownloader\DepotDownloader\Program.cs:line 14
./depotdownloader: line 1:  2705 Aborted                 (core dumped) dotnet DepotDownloader.dll "$@"
me: A
 extracting: depotdownloader
 extracting: depotdownloader.bat
  inflating: DepotDownloader.deps.json
  inflating: DepotDownloader.dll
  inflating: DepotDownloader.dll.config
  inflating: DepotDownloader.pdb
  inflating: DepotDownloader.runtimeconfig.json
  inflating: LICENSE
  inflating: protobuf-net.dll
  inflating: README.md
  inflating: SteamKit2.dll
  inflating: SteamKit2.pdb
  inflating: System.Reflection.DispatchProxy.dll
  inflating: System.Security.Principal.Windows.dll
  inflating: System.ServiceModel.dll
  inflating: System.ServiceModel.Primitives.dll
Connecting to Steam3... Done!
Logging '' into Steam3...Unhandled exception. System.ArgumentException: LogOn requires a username and password to be set in 'details'.
   at SteamKit2.SteamUser.LogOn(LogOnDetails details) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\Handlers\SteamUser\SteamUser.cs:line 299
   at DepotDownloader.Steam3Session.ConnectedCallback(ConnectedCallback connected) in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 504
   at SteamKit2.Internal.Callback`1.Run(Object callback) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr_Internals.cs:line 81
   at SteamKit2.CallbackManager.<>c__DisplayClass10_0.<Handle>b__1(CallbackBase callback) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 142
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at SteamKit2.CallbackManager.Handle(ICallbackMsg call) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 143
   at SteamKit2.CallbackManager.RunWaitCallbacks(TimeSpan timeout) in C:\Projects\SteamKit\SteamKit2\SteamKit2\Steam\SteamClient\CallbackMgr\CallbackMgr.cs:line 71
   at DepotDownloader.Steam3Session.WaitForCallbacks() in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 478
   at DepotDownloader.Steam3Session.WaitUntilCallback(Action submitter, WaitCondition waiter) in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 141
   at DepotDownloader.Steam3Session.WaitForCredentials() in C:\Projects\DepotDownloader\DepotDownloader\Steam3Session.cs:line 154
   at DepotDownloader.ContentDownloader.InitializeSteam3(String username, String password) in C:\Projects\DepotDownloader\DepotDownloader\ContentDownloader.cs:line 356
   at DepotDownloader.Program.MainAsync(String[] args) in C:\Projects\DepotDownloader\DepotDownloader\Program.cs:line 196
   at DepotDownloader.Program.Main(String[] args) in C:\Projects\DepotDownloader\DepotDownloader\Program.cs:line 14
./depotdownloader: line 1:  2705 Aborted                 (core dumped) dotnet DepotDownloader.dll "$@"

What could be the issue with this? I think I set up the variables in the script right!
Thank you very much for your time!
EDIT: By looking at the output above, it looked like the username was not input into a variable. After further bash learning, the two read lines need this to them (lines 15-16):
read -p "Enter your Steam username:" STEAM_USERNAME
read -s -p "Enter your Steam password:" STEAM_PASSWORD

EDIT 2:
I am silly, I deleted the STEAM_USERNAME and STEAM_PASSWORD keywords, which is why it didn't work for me. You do not have to fill in your credentials on these lines, keep these variable names intact and the script will simply ask you for your credentials!

@brandflake11 I'm glad you got it working! Let me know if you get the game running :)

@lpww Thank you so much for this script. The script worked! I am never deleting these files of the older working-on-linux version of Doom Eternal.

Great news! Rip and tear baby


Build commands in CMakeList.txt

Why delete all ADD_EXECUTABLE in Cmakelists.txt? I can't build my change of dbt5/src

I'm guess this was my attempt to use loops.  Can you confirm which revision you are working with, and show the output.  Let's see if I can reproduce it.

This is my problem. I found that building of all code is moved into egen in this commit. By executing dbt5-build-egen, all code can be built.


feat(ui): show/hide node table columns

This PR provides the possibility to show / hide individual table columns.
Button to show the selection menu:

Menu to choose the columns from:

The headers array allows to customize each column using the properties active (is currently visible or not) and selectable (is selectable in the menu or not).
Some not so important columns are now not shown by default (see screenshot above).
Since this reflects my personal opinion I'm interested in feedback, what the defaults should be.
It lays the foundation to provide much more data in the nodes table (e.g. battery level, ...) without overloading the number of columns.

Pull Request Test Coverage Report for Build 527190713

0 of 88 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 21.362%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/nodes-table/nodes-table.js
0
88
0.0%

Totals

Change from base Build 527149526:
-0.2%

Covered Lines:
2044

Relevant Lines:
9798

💛  - Coveralls

@ahochsteger Feel free to squash and merge once ready

@robertsLando thanks for your feedback!
Are you ok with the defaults I've chosen to not show certain columns by default?

@ahochsteger I would keep all visible by default, and also I would store the shown in localstorage (like for the filters etc)

@robertsLando the list of columns is now stored in local storage and the full list of headers is shown by default.
As a consequence I've simplified the code a lot (eliminated the selectable and active header attributes since they are not really requred).

@ahochsteger In order to fix the UI problems I think we could set a max number of columns based on the screen size. What do you think about this?

@ahochsteger In order to fix the UI problems I think we could set a max number of columns based on the screen size. What do you think about this?

@robertsLando  I'm not sure, if the number of columns is a good solution here if you compare the width of the "ID" column with the "Product" column that shows for example something like "A real push button (switch) available in several colors." (which should be reduced in the device config anyway).

@ahochsteger What other solution do you propose to fix the other UI problem?

@ahochsteger What other solution do you propose to fix the other UI problem?

I'm not sure, which "other" UI problem you are referring to in this context?


Invite Façade URL should support encoding=JSON

Should respond with JSON body {invite, postTo}, see spec https://ssb-ngi-pointer.github.io/rooms2/#invites

@cryptix What if you take this issue and I take #189 ?

Yup! let's do that.


it seems get_open_orders should be two functions: get_open_orders and get_open_order

According to the official doc, there are 2 REST endpoints for getting open order:
GET /dapi/v1/openOrder (HMAC SHA256) and GET /dapi/v1/openOrders (HMAC SHA256)
The /dapi/v1/openOrder endpoint needs either orderId or clientOrderId. And the /dapi/v1/openOrders endpoint needs only symbol.
In the file binance/cm_futures/account.py , there is only 1 function get_open_orders while it acts like openOrder endpoint.
The right way should be 2 functions: get_open_orders() acts like /dapi/v1/openOrders, and get_open_order() acts like /dapi/v1/openOrders

Found a function get_orders()


Improve the Email Sent to ANET Approvers

"The ANET email sent to approvers needs nine changes:

Add information to the report: 1, 4, and 5
Meet terminology standards: 2, 6, and 7
3. Cosmetic: 3
Consistent capitalization: 8 and 9
Please see the attached PDF

Users often reply to the automated ANET notifications.

Change the email to state that it is an automated mail and you should not reply to it.

Secondly

On the automated emails for approvals, there is an option that says click here to approve.  However it is not clear that clicking that link approves the report without offering the opportunity to review it first.

Reference item 21

Remove the "Click Here to Approve this Report" link in the email body.  See update to reference item 21.

The report approver email should have only one link in the body that allows the report approver to open the report, and then either approve, reject or edit the report.


[CI/CD] Create templates for issues, pull requests, etc.

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository

I would like to work on this issue, Can you please assign it to me? @willroberts

Hi @harikanani , sure thing! Let us know if you have any questions.

Hi @harikanani , sure thing! Let us know if you have any questions.

I haven't any right now. If I have any further I'll ask. I do have some ideas about which kind of templates can be added, so i'll add that here. Please assign this issue to me.

I have created a pull request that resolves this issue. I am currently working on many other types of templates too. I'll update that once I complete it.

Sounds good! I think a feature request template and a pull request template would be sufficient here.

I have created another PR for feature_request template


[MB-6826] Add pricing params for Dest and Origin SIT Addl' days

Description
This PR adds pricer params to both the domestic origin and destination pricers for SIT additional days.
Setup
To get to the place where the DDASIT screenshot below was taken, do the following:

Build and start your local client/server (and make sure your DB is up-to-date)
Create a new move in the customer UI
As a TOO in the office UI, edit the orders to add the missing fields, then approve the move and the service items.  That should automatically add other service items like DLH.
Run the prime-api-demo script locally: prime-api-demo <move locator> local
Have it update dates/weights, create a payment request, and upload proof of service docs.
Go back to the office UI as a TIO and approve the payment request and its service items.
You should then have the ability to see the parameter details and pricing on the DDFSIT service item as shown below.
You should also be able to see the params by looking in the payment_service_item_params table or running a fetch-mto-updates via the prime-api-client.

Note: if you don't use the prime-api-demo client you might get an error for trying to create an additional sit service item when there's no first day sit service item on the shipment. If that happens you can follow these steps to create a DDF or DOF SIT service item
Otherwise we can also add a DDASIT (destination additional day SIT) to the move. Use this JSON template to make a file named create_ddasit_service_item.json, filling in the move ID and shipment ID (which you can find in your local database from the locator):
{
  "body": {
    "moveTaskOrderID": "<insert MTO ID>",
    "mtoShipmentID": "<insert shipment ID>",
    "modelType": "MTOServiceItemDestSIT",
    "reServiceCode": "DDASIT",
    "timeMilitary1": "1705Z",
    "firstAvailableDeliveryDate1": "2021-01-31",
    "timeMilitary2": "0719Z",
    "firstAvailableDeliveryDate2": "2021-02-03",
    "sitEntryDate": "2021-01-12"
  }
}

Create the DDASIT service item with this command:
prime-api-client --insecure create-mto-service-item --filename create_ddasit_service_item.json | jq

Now we should be able to issue a payment request for the DDASIT service item.  Create a create_payment_request.json file with these contents (looking up the IDs noted below in your database):
{
  "body": {
    "isFinal": false,
    "moveTaskOrderID": "<insert MTO ID>",
    "serviceItems": [
      {
        "id": "<insert DDASIT MTO service item ID>"
      }
    ]
  }
}

Now send this payload to create the payment request:
prime-api-client create-payment-request --insecure --filename create_payment_request.json | jq

Approve the payment request as a TIO to get to the point where you can see the DDASIT screenshot below.
Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots

Messages

:book:
:link: MB-6826

  Generated by :no_entry_sign: dangerJS against 6d17f1a9c7a947ca96d1fcc2ebb9267066fb9c76


Fixed formatting of the value 0

As the title says, this PR fixes formatting of the value 0.
The problem: When trying to format a value that is zero, the Base::integral_exponent_for gives the wrong result, because it uses log10/ log2 but the logarithm of zero returns -Inf. In a debug build, this panics because of integer overflow, in a release build it just gives the wrong result.
What I did to fix it: I added a check for the zero-case to Base::integral_exponent_for to have it return 0. I also added some tests that verify both this function and that formatting a zero Value give the correct results.
I hope you don't mind this PR, it seemed easy to fix but maybe this is not inline with how the library should be used? I really like this library and I'm trying to use it for progress logging in a tool that processes a large amount of data, but progress does start at zero, which is where I stumbled upon the bug.

Hi, thanks for the kind words and for your contribution!
I had a good laugh after realizing I had forgotten this super important use case 😅, good catch.
I'm happy to merge the PR and bump the version as soon as you fix the minor formatting issue.
Cool, thanks again!

LGTM, thanks again!

@Mortano, version 0.1.4 is now published 👍


Provide npmsearch.com also via HTTPS

It would be great to have http://npmsearch.com/ also available via HTTPS.
Free certificates are available via https://startssl.com/ and soon https://letsencrypt.org/.

:+1:

:+1:

There is also https://www.globalsign.com/en/ssl/ssl-open-source/

Seems to be a good amount of traction here and elsewhere, let's do this!
I've signed up for a beta account on letsencrypt.org
I don't want to judge startssl.com by it's cover but damn, that site's design is horrible.
this repo is currently unlicensed which is definitely a bad thing, so I'll fix that pending a letsencrypt email and submit to globalsign.
If anyone knows other options please let me know!

In an effort to be transparent (and not look too lazy!) I've gone off and retrieved an SSL cert.  I'll be looking at rolling out SSL support in the next day or two.  Thanks for the push!

Ok folks!  https://npmsearch.com is now alive and kicking

and the HTTP api also works under SSL
npmsearch‣ curl "https://npmsearch.com/query?q=dom&fields=id" -v | json
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>...
* Connected to npmsearch.com (<IP_ADDRESS>) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384
* Server certificate: www.npmsearch.com
* Server certificate: GlobalSign Domain Validation CA - SHA256 - G2
* Server certificate: GlobalSign Root CA
> GET /query?q=dom&fields=id HTTP/1.1
> Host: npmsearch.com
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Access-Control-Allow-Origin: *
< Date: Wed, 07 Oct 2015 22:36:58 GMT
< Transfer-Encoding: chunked
<
{ [288 bytes data]
100   276    0   276    0     0    984      0 --:--:-- --:--:-- --:--:--   985
* Connection #0 to host npmsearch.com left intact
{
  "results": [
    {
      "id": [
        "zero-dom-utils"
      ]
    },
    {
      "id": [
        "dom-iterators"
      ]
    },
    {
      "id": [
        "select-dom"
      ]
    },
    {
      "id": [
        "dom-anchor-testing"
      ]
    },
    {
      "id": [
        "dom-manipulations"
      ]
    },
    {
      "id": [
        "dom-value"
      ]
    },
    {
      "id": [
        "dom-stub"
      ]
    },
    {
      "id": [
        "dom-walk"
      ]
    },
    {
      "id": [
        "karma-chai-dom"
      ]
    },
    {
      "id": [
        "dom-select"
      ]
    }
  ],
  "total": 5857,
  "from": 0
}


Cascading Deletion

You can delete a group from identity, but that group is referenced by projects, so we need to cascade that delete and remove the association.  Preferably we break the chain before deleting the group to avoid races.

The other option here is to play it safe and block deletion to prevent things breaking unexpectedly.


Archive cert served instead of domains one

We had a strange behavior on 2 different servers.
The cert served come from the archive folder instead of domains folder.
Therefore the connection is considered insecure because the served cert is expired.
Moreover the behaviour is random, sometimes the valid cert is served.
Other cert renewal were not processed, without any error.
After removing the archive folders of the domain and restart Apache everything goes well.
This happened 2 month ago on a server, and today on another.
And we have no logs about what happened.
A little bit scary

It sounds more as if there was still an older child process hanging around after a restart and that answered some requests with the old configuration. That would explain the randomness in old/new answers to connections.
Otherwise, mod_md never reads certs from the archived directories, even when domains are missing.

It sounds more as if there was still an older child process hanging around after a restart and that answered some requests with the old configuration. That would explain the randomness in old/new answers to connections.
Otherwise, mod_md never reads certs from the archived directories, even when domains are missing.

Hello @icing !
I faced the same problem and you were right : a simple Apache restart fixed the issue.
=> It must be caused by a lingering child process.
For the record :
I tracked the error with the following openssl command :
openssl s_client -connect host.domain.com:443 -prexit -showcerts

=> This command randomly showed a different certificate.
I then greped the offending certificate in /etc/apache2/md
grep xxxx /etc/apache2/md/*/*/*.pem

=> It matched the last archived certificate for this domain !
I found this issue opened by @FV-Valraiso with the archive keyword.
I ran
sudo service apache2 restart

=> Fixed
Thank you 👍🏼

Glad that it worked for you!


XCode 12, swift 5.3 won't build

fatal error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: /tmp/icomoon-build/iphonesimulator/Icomoon.framework/Icomoon and /tmp/icomoon-build/iphoneos/Icomoon.framework/Icomoon have the same architectures (arm64) and can't be in the same fat output file

@TomerAvni
Created PR, Please checkout my branch and test.
git checkout https://github.com/nitrag/Icomoon.swift.git
make install
cd ~/Downloads/
icomoon-swift icomoon.zip

Then drag/drop into xcode (remove old framework first).
PR Pending your test #16

Thank you for your reply!
Unfortunately it doesn’t work for me. It builds ok (as was the previous version) but it keeps crying about:
fatal error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: /tmp/icomoon-build/iphonesimulator/Icomoon.framework/Icomoon and /tmp/icomoon-build/iphoneos/Icomoon.framework/Icomoon have the same architectures (arm64) and can't be in the same fat output file
Maybe it’s something local…
Any thoughts?
Thanks again,
Tomer.

On 19 Sep 2020, at 0:55, nitrag<EMAIL_ADDRESS>wrote:
@TomerAvni https://github.com/TomerAvni
Created PR, Please checkout my branch and test.
git checkout https://github.com/nitrag/Icomoon.swift.git
make install
cd ~/Downloads/
icomoon-swift icomoon.zip
Then drag/drop into xcode (remove old framework first).
PR Pending your test #16 https://github.com/schickling/Icomoon.swift/pull/16
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/schickling/Icomoon.swift/issues/15#issuecomment-695103217, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIXSAJ5X34QBFEBEIOV7RLSGPJMZANCNFSM4RR5WNUQ.

I believe it has to do with the updated build mechanism, I saw something similar here, but its Carthage:
https://github.com/Carthage/Carthage/issues/3019
Tomer.

On 19 Sep 2020, at 7:32, Tomer Avni<EMAIL_ADDRESS>wrote:
Thank you for your reply!
Unfortunately it doesn’t work for me. It builds ok (as was the previous version) but it keeps crying about:
fatal error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: /tmp/icomoon-build/iphonesimulator/Icomoon.framework/Icomoon and /tmp/icomoon-build/iphoneos/Icomoon.framework/Icomoon have the same architectures (arm64) and can't be in the same fat output file
Maybe it’s something local…
Any thoughts?
Thanks again,
Tomer.

On 19 Sep 2020, at 0:55, nitrag<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
@TomerAvni https://github.com/TomerAvni
Created PR, Please checkout my branch and test.
git checkout https://github.com/nitrag/Icomoon.swift.git https://github.com/nitrag/Icomoon.swift.git
make install
cd ~/Downloads/
icomoon-swift icomoon.zip
Then drag/drop into xcode (remove old framework first).
PR Pending your test #16 https://github.com/schickling/Icomoon.swift/pull/16
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/schickling/Icomoon.swift/issues/15#issuecomment-695103217, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIXSAJ5X34QBFEBEIOV7RLSGPJMZANCNFSM4RR5WNUQ.

@TomerAvni Sorry, it was on a branch and you I had you pull master. I merged, so now if you do git pull again and make install you should be good.

Perfect.
Thanks a lot again!

On 19 Sep 2020, at 8:26, nitrag<EMAIL_ADDRESS>wrote:
@TomerAvni https://github.com/TomerAvni Sorry, it was on a branch and you I had you pull master. I merged, so now if you do git pull again and make install you should be good.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/schickling/Icomoon.swift/issues/15#issuecomment-695167478, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIXSAIGOXXWZUSNM5S2ITLSGQ6JFANCNFSM4RR5WNUQ.


Fixed incongruity in the script

Now the script checks if it is run as root before using the sudo command
I made apt-get update quieter

Thanks for the contribution sir, great work!


[Speculative Decoding 1/2 ] Add typical acceptance sampling as one of the sampling techniques in the verifier

FILL IN THE PR DESCRIPTION HERE
FIX #5015  (link existing issues this PR will resolve)
In this PR we implement typical acceptance sampling as one of the alternate sampling techniques for the Speculative Decoding verifier. In this sampling we do the following

For every draft token we compare its probability in the target probability distribution with the entropy of the target distribution and if it meets the acceptance criteria then we accept it. More details on the acceptance criteria can be found in section 3.3.1 in this paper (https://arxiv.org/pdf/2401.10774)
We stop at the first token that is rejected and return
If no draft tokens are accepted  then we use greedy sampling to sample a token from the target distribution corresponding to k = 0. The paper also specifies sampling as the fallback option in case no draft tokens are selected
If all the draft tokens are selected then we append the bonus token to the end. The Medusa paper does not talk about the bonus token optimization. However we accept the bonus token since it is drawn from the same target distribution that we are using for accepting the rest of the tokens.

In this PR we make the following changes

Add typical_acceptance_sampler.py which implements the sampling logic described above. Also added test cases for the new sampler.
Create a base class spec_decode_base_sampler.py to be used by both the rejection_sampler and the new typical_acceptance_sampler.
Move some of the logic from rejection_sampler into spec_decode_base_sampler such that it can be used by both the samplers.

As part of this PR we introduce the new sampler. We will hook it up into the spec_decode_worker in a later pr

One thing we should check is that typical acceptance latency is ~= rejection sampling latency (and not 2x+ or something).

One thing we should check is that typical acceptance latency is ~= rejection sampling latency (and not 2x+ or something).

I modified the test_spec_decode_e2e_greedy_correctness_tiny_model_large_bs in tests/spec_decode/e2e/test_multistep_correctness.py and ran it both with rejection_sampling and typical_acceptance sampling with 68m model as the draft and 160m model as the target. The latencies of both are similar.
Below is the sample Spec Decode metrics that I got from the run. The avg time is the time spent in the forward pass in the 2 samplers averaged across all invocations.
This statistics is for temperature = 0.5, output length = 256 and batch = 64
rejection sampling
INFO 06-06 06:38:06 metrics.py:362] Speculative metrics: Draft acceptance rate: 0.765, System efficiency: 0.576, Number of speculative tokens: 5, Number of accepted tokens: 18180, Number of draft tokens tokens: 23780, Number of emitted tokens tokens: 16447, Avg Time: 0.0017990445097287495.
typical acceptance sampling
INFO 06-06 06:40:24 metrics.py:362] Speculative metrics: Draft acceptance rate: 0.843, System efficiency: 0.610, Number of speculative tokens: 5, Number of accepted tokens: 18946, Number of draft tokens tokens: 22470, Number of emitted tokens tokens: 16437, Avg Time: 0.0014608523675373622

LGTM. Thanks for running the performance test -- great that we're faster than rejection sampling :)
Only comment is to add a test for the typical acceptance hyperparameters, and to push the removal of the double-constructor invocation. OK to merge after!

@sroy745 can you rebase and then I'll merge ?

the models test is an fp8 failure that's unrelated

Auto merge enabled

@simon-mo can we get a force merge? the amd tests aren't running

@sroy745 one question about this new sampler (relates to work in #6034): it seems to me that the forward method of TypicalAcceptanceSampler is deterministic with respect to its inputs. e.g., if repeatedly call forward with the same input, we would always get the same output. In contrast, this is not the case for the RejectionSampler which will produce differnet output each time. Is this the correct understanding? Thanks!

@sroy745 one question about this new sampler (relates to work in #6034): it seems to me that the forward method of TypicalAcceptanceSampler is deterministic with respect to its inputs. e.g., if repeatedly call forward with the same input, we would always get the same output. In contrast, this is not the case for the RejectionSampler which will produce differnet output each time. Is this the correct understanding? Thanks!

@tdoublep yes the TypicalAcceptanceSampler should be deterministic as opposed to the RejectionSampler. All the computation in the sampler is deterministic once the input is fixed.


Fix RC4 based ciphersuites

Commit be9d82b inadvertently disabled ciphersuite testing. This masked two separate issues in the RC4 ciphersuites.
In 1 issue RC4 based ciphersuites fail the properly extract the MAC from decrypted TLS data.
In the 2nd issue the RC4-MD5 stitched ciphersuites fails to adjust the length of the decrypted data appropriately to account for the MAC.
This PR fixes both of those issues, as well as fixing the ciphersuite testing. Note that in fixing the testing I had to make some adjustments to take account of the FIPS module. This required some changes to the DH handling in the test. Currently that test uses the old "low-level" DH object instead of an EVP_PKEY - therefore this PR does the same in order to play nicely with the existing code. That issue is fixed by another PR (#13368). Whichever PR goes in second (this one or #13368) will have to be adjusted to use EVP_PKEY instead of DH.
Fixes #13363

Travis is relevant.

Fixup pushed to address Travis failure

This pull request is ready to merge

Unfortunately #13368 was merged which touched the same ssltest_old.c code as this PR. Therefore I had to rewrite some of that code.
@paulidale - please can you reconfirm your approval?

Travis failures are not relevant.
Ping @paulidale.

This pull request is ready to merge

Pushed. Thanks.


Difficult to reopen screen keyboard on Android

On Android, close screen keyboard, it is difficult to reopen it in CodeMirror area. I tried so many times, found a workarround. Tap some area outside the CodeMirror editor, then tap the CodeMirror area again, screen keyboard come back. I hope to bring back screen keyboard by tapping the CodeMirror area twice. Or some better actions. Thanks a lot.

Should be fixed now that the mobile/contenteditable code landed.


Missing permissions parameter in ExchangeInformation method

Reproduction steps
permissions | curl -X GET "https://api.binance.com/api/v3/exchangeInfo?permissions=SPOT"orcurl -X GET "https://api.binance.com/api/v3/exchangeInfo?permissions=["MARGIN"%2C"LEVERAGED"]"orcurl -g -X GET 'https://api.binance.com/api/v3/exchangeInfo?permissions=["MARGIN","LEVERAGED"]'
Expected behavior
https://binance-docs.github.io/apidocs/spot/en/#exchange-information
Actual Behavior

.NET version
Other (specify in environment)
Operating system
Linux
Environment
No response
dotnet --info
No response

+1


Settings: Allow disabling archive page

Context
See #4862 for history / context on this matter.
Summary
We want to allow users to disable the default /web-stories archive page.
Relevant Technical Choices
As there are only two options for now, enabled and disabled, made this PR simpler and just made the dashboard setting a checkbox. If we have more options in the future, at that point change to a dropdown.
To-do

User-facing changes
New setting in dashboard

Archive link option does not show for blocks

When archive is disabled, link should not rendered on the front end

Options should render in widget form to show archive links

Archive url, should 404.

Testing Instructions

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:
Enable the "Disable Archive" experience is enabled.
Test dashboard setting

Go to dashboard > settings.
Ensure Stories Archive option is present.
Disable / Enable archives, reload page, ensure setting is saved.

Test archive is really disabled

Diable Stories Archive ( see above ).
Go to /web-stories on the front end.
Ensure page is a 404.

Test widget presents settings are not rendered

Diable Stories Archive ( see above ).
Enable classic widget plugin.
Create a web stories widget.
Ensure no options for archives are present.
Save widget.
Ensure no link archive is rendered on front end.

Test block settings are not rendered

Diable Stories Archive ( see above ).
Create a new post
Create a web stories block
Select Latest stories.
Select any layout style
Ensure no options for archives are present.
Save post.
Ensure no link archive is rendered on front end.

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8811

@swissspidy The karma test failure seem unrelated here. An idea why they might have broken.

After a chat with @swissspidy, I am changing this to be a dropdown instead of a checkbox to allow for future dev work.

@swissspidy This is ready for review again

(I'm removing my 'needs work' because I'm out tomorrow and Monday. Tag Samuel for pea pod review of this when you are ready for more eyes on it)

@googlebot I consent


chore(typed-redux-saga): demonstrating without macros

To demonstrate test errors while using typed-redux-saga instead of typed-redux-saga/macros

Closing now that it's been done in https://github.com/valora-inc/wallet/pull/3961 🎉


Character limit deve ser refatorado para componente principal de main text

Describe the feature
A clear and short description of the feature.

Description

CharacterLimitMainText deve ser excluido e a função deve ser incorporada no Character limit em um componente só, nome precisa ser alterado para se adequar a nova função do componente

Use Case

[Explain the use for this feature and how it might benefits the project.]

Implementation Details

[Provide any details or suggestions on how this feature could be implemented.]

Visual Concepts

[Include any visual representations or concepts if those are available and applicable.]

Additional Information

[Provide any additional information, such as context that might be relevant to the implementation of this specific feature.]

Eu quero!!!


Mseger statsig automated rollouts

including images for scheduled rollouts

🚀 Deployed on https://63644aba3030330ad2dc55b6--cozy-fox-0defec.netlify.app


[Feature] Open crew character sheet through vehicle sheet 🚚

Now we can add actors into the crew slots and then assign them as pilot/gunner or other roles, but we can't open the actor sheet up from that list. Can you investigate this feature alongside possible this feature: : https://github.com/StarWarsFoundryVTT/StarWarsFFG/issues/1281

Done, thanks for the suggestion. Will be released in 1.805.


Add DynDNS provider SPDNS (IPv4 and IPv6)

SPDNS is a free DynDNS service from the German company Securepoint
Security Solutions

Before this pull request can be accepted you must first sign a CLA as described at https://www.pfsense.org/about-pfsense/#cla. Please read for more details.

CLA has been submitted


run system tasks periodically

Currently, system tasks (heartbeat and global timer) are only triggered upon receiving an HTTP request. This MR adds a new execution thread that periodically triggers these tasks every second.

I’m worried that using threads here rather than deterministic inputs will make it harder to use ic-ref as a deterministic environment to write test, as we do in Motoko. (Ok, there we use ic-ref-test, which could do something else). Just a thought.

I’m worried that using threads here rather than deterministic inputs will make it harder to use ic-ref as a deterministic environment to write test, as we do in Motoko. (Ok, there we use ic-ref-test, which could do something else). Just a thought.

My motivation to write this PR was actually that @ggreif could not use ic-ref to test the global timers in Motoko.

@nomeata Would it make sense to you to only start this thread if a corresponding flag is passed?

That might help indeed. And for ic-ref-test (and ideally also drun) I would suggest a new “skript command” command “run heartbeats now”, maybe even for a specific canister, to give the test harnesses good control over when the heartbeat runs. This allows to reliably check, for example, how a canister behaves if a heatbeat does or does not run between two other events.

And for ic-ref-test (and ideally also drun) I would suggest a new “skript command” command “run heartbeats now”, maybe even for a specific canister, to give the test harnesses good control over when the heartbeat runs. This allows to reliably check, for example, how a canister behaves if a heatbeat does or does not run between two other events.

This suggestion would not work for @ggreif 's test because the global timer would need to run during the execution of an update call, i.e., after it's submitted and before it returns.

Ah, yes, the old problem of how to influence the message scheduler from tests. It already ariess without heartbeats, if one wants to test various interleavings.
With @ielashi I discussed a possible approach a while ago, with barriers that hold back responses to messages, and thus allow you to do other things in between two specific messages, and I think he even had a prototype, but it never landed.
I am wary of trying to solve “we need better control over the scheduling to run specific tests” with “let’s introduce concurrency and timer-based interaction and hope for the best”, which I fear will lead to flaky (timing sensitive) and slower (because you need to poll or even have fixed delays) tests.
I don’t have much to offer, though, besides the barrier based approach (maybe @ielashi can share documents we wrote back then, I don’t have access. Might have been an issue on the old ic-ref repo.)

Steering system tasks in ic-ref is difficult without exposing a side-channel. In ic-ref-run, it's easier, but there the IC is not under heavy load (every test has its own IC instance) so the tests should not be flaky. I think it makes sense to add a flag specifying how often you want to run heartbeats. This should give enough determinism to the tests (although it could make their execution slower).

Heads-up: a tail-call test now times out in the Motoko test suite under ic-ref...


SQL Server database adapter

Hello,
We have produced a Scenic adapter for SQL Server, taking other third party adapters as inspiration.  I hope you'll consider it worthy of a link from your repo.
Best wishes,
Ben

Done in 05be54f


Add steps for building executables in Xcode

Previous doc only contained instructions for running in an Xcode playground. This was not sufficient to get code to compile and link.

I signed it.

On Aug 14, 2018, at 4:07 PM, googlebot<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers
It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data https://cla.developers.google.com/clas and verify that your email is set on your git commits https://help.github.com/articles/setting-your-email-in-git/.
Corporate signers
Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot http://go/cla#troubleshoot (Public version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data https://cla.developers.google.com/clas and verify that your email is set on your git commits https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must also be attached to your GitHub account https://github.com/settings/emails.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tensorflow/swift/pull/56#issuecomment-412998318, or mute the thread https://github.com/notifications/unsubscribe-auth/AJkLoWVJxoXOrSxBjuyzHp1gh3gzsn4Bks5uQy4RgaJpZM4V9Fym.

It still says CLA check is failing for some reason. Did you use a different email address to sign?

The CLA has been approved now. My email address was private when I submitted the PR but is now public. Do I need to resubmit the PR?

I’m sorry for the delayed response! If you don’t mind, could you start a new PR?


Revert some additions of free variables to type restrictions

Resolves #11244. Reverts part of #10467 where generic type restrictions like Foo were changed into Foo(T) forall T, as they are not equivalent.

It would be nice to have some specs so we make sure nobody adds them back again. Or maybe a comment in the source code.

It looks reasonable to include in 1.2.0, since this fixes an issue from #10467 which is part of 1.2.0 milestone.


Use of args.scale

Hello! At the inference_realesrgan.py there are the following lines of code:
        if args.scale == 2:
            mod_scale = 2
        elif args.scale == 1:
            mod_scale = 4

For me, it seems like there is a misprint and there should be elif args.scale == 4 as the default args.scale  value equals to 4 and there is a zero-padding up to mod_scale multipliers. Is it actually a misprint or I just did not understand your code?

For reference, this is the part of the code:
https://github.com/xinntao/Real-ESRGAN/blob/0573f32dd04a6ff9dbafff67df23f808dffc8811/inference_realesrgan.py#L40-L53


hpa enhancement

support default behaviors
support custom metrics

https://github.com/vesoft-inc/nebula-operator/pull/483
https://github.com/vesoft-inc/nebula-operator/pull/484


Add Error Pages

I've moved the current available pages to examples/src/pages/ to clear up the src folder.
Following Error Pages created:

400
401
403
404
500
503

Changes applied.
Here is the Error404 page.

Awesome!!! Thanks for contributing 👍

My pleasure, I'll keep working on the pages. I saw @jonthomp's SiteWrapper.react.js which will help me greatly with the rest of the pages. More to follow.


How to update the purchase order in auto-recharge scenarios for EA customers?

The documentation says for EA customers, "you will also be given an opportunity to specify a purchase order number to associate with the enterprise agreement number (if applicable)". But in scenario where there is "auto-recharge" enabled and over years the specified purchase order is no more valid, it seems it cannot be updated. What is the process in that case to ensure it can be invoiced on correct new purchase order? Thanks.

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

ID: 277cf140-5c6c-82c7-cc56-490ef2188754
Version Independent ID: 72e7de29-fd71-23bf-15a7-172634c2a5fd
Content: Set up Communications Credits for your organization - Microsoft Teams
Content Source: Teams/set-up-communications-credits-for-your-organization.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@olamer thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Docker instance of Traefik shutting down on it’s own

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
i'm setting up my homelab with Traefik through the official docker image, simple reverse proxy from subdomain to internal ports through a docker network, the plan is for traefik to run 24/7 serving my services on external network. I'm currently using portainer (2.16.2) to deploy the container.
What did you see instead?
When i leave my instance of Traefik up for a long-ish period of time it stops itself out of nowhere, the server wasn't being ssh into, traefik wasn't receiving any requests and there aren't any configurations defining resources limitations on the docker-compose.yml.
What version of Traefik are you using?

Traefik yaml:
version: '3'
services:
  traefik:
    image: "traefik:latest"
    container_name: "traefik"
    command:
      - "--log.level=DEBUG"
      - "--api.dashboard=true"
      - "--api.insecure=true"
      - "--providers.docker=true"
      - "--providers.docker.network=reverseproxy-nw"
      - "--providers.docker.exposedbydefault=false"
      - "--entrypoints.web.address=:80"

      #- "--entrypoints.websecure.address=:443"

      # cert resolver
      #- "--certificatesresolvers.myresolver.acme.tlschallenge=true"
      #- "--certificatesresolvers.myresolver.acme.caserver=https://acme-v02.api.letsencrypt.org/directory"
      #-<EMAIL_ADDRESS>      #- "--certificatesresolvers.myresolver.acme.storage=/letsencrypt/acme.json"
    ports:
      - "443:443"
      - "80:80"
      - "8080:8080"
    volumes:
      - "/home/gustavo/aplications/traefik/letsencrypt:/letsencrypt"
      - "/var/run/docker.sock:/var/run/docker.sock:ro"
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.dashboard.rule=Host(`traefik.domain.com`)"
      - "traefik.http.routers.dashboard.entrypoints=web"
      - "traefik.http.services.dashboard.loadbalancer.server.port=8080"
    networks:
      - reverseproxy-nw

networks:
  reverseproxy-nw:
    name: reverseproxy-nw
    external: true

PS: Commented lines are for enabling SSL certificates once i can lay down the foundation and iron out all the "unwanted behaviours".
Example service that uses traefik proxy:
version: '3.3'
services:
    portainer:
        labels:
            - "traefik.http.routers.portainer.rule=Host(`portainer.domain.com`)"
            - "traefik.http.routers.portainer.entrypoints=web"
            - "traefik.http.services.portainer.loadbalancer.server.port=9000"
            - "traefik.enable=true"
        container_name: portainer
        command: -H unix:///var/run/docker.sock
        ports:
            - '8000:8000'
            - '9000:9000'
        restart: always
        volumes:
            - '/var/run/docker.sock:/var/run/docker.sock'
            - './portainer_data:/data'
        image: portainer/portainer-ce
        environment:
            - VIRTUAL_HOST=portainer.domain.com
            - VIRTUAL_PORT=9000
        networks:
            - reverseproxy-nw

networks:
  reverseproxy-nw:
    external: true

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
time="2023-01-03T17:34:12Z" level=error msg="Error while Peeking first byte: read tcp <IP_ADDRESS>:80-><IP_ADDRESS>:42276: read: connection timed out"
time="2023-01-03T17:59:52Z" level=debug msg="'499 Client Closed Request' caused by: context canceled"
time="2023-01-03T18:32:27Z" level=info msg="I have to go..."
time="2023-01-03T18:32:27Z" level=info msg="Stopping server gracefully"
time="2023-01-03T18:32:28Z" level=debug msg="Waiting 10s seconds before killing connections." entryPointName=traefik
time="2023-01-03T18:32:28Z" level=debug msg="Waiting 10s seconds before killing connections." entryPointName=web
time="2023-01-03T18:32:28Z" level=error msg="accept tcp [::]:8080: use of closed network connection" entryPointName=traefik
time="2023-01-03T18:32:28Z" level=error msg="accept tcp [::]:80: use of closed network connection" entryPointName=web
time="2023-01-03T18:32:28Z" level=error msg="close tcp [::]:80: use of closed network connection" entryPointName=web
time="2023-01-03T18:32:28Z" level=debug msg="Entry point web closed" entryPointName=web
time="2023-01-03T18:32:28Z" level=error msg="close tcp [::]:8080: use of closed network connection" entryPointName=traefik
time="2023-01-03T18:32:28Z" level=debug msg="Entry point traefik closed" entryPointName=traefik
time="2023-01-03T18:32:28Z" level=info msg="Server stopped"
time="2023-01-03T18:32:28Z" level=info msg="Shutting down"

Hello @gustavodeon and thanks for your interest in Traefik,
To be sure, we looked again at the code path, and there are no reasons for Traefik to stop without receiving a signal. Thus, we think that the termination signal might come from the orchestrator (Docker) and that Traefik is behaving correctly.
We invite you to continue the discussion on the topic created in the forum. Feel free to reopen it, if you think that we missed something.


[FEAT] Stop / pause / play

like refresh button, will a great idea put Stop / pause / play

this is a feature stop / retry:
https://github.com/sunner/ChatALL/assets/428820/6449b869-6330-418b-a605-7cdc705d4dbc
this will usefull with claude  or other LMSYS when show a long answer

https://github.com/sunner/ChatALL/assets/428820/07effaf7-adce-4081-b441-c55bb8be1827
this stop / retry / puse / play feature is very important, because some models put a long answer meanwhile another question is requested

Yes, the stop is needed. Let me find a way.

@johnfelipe  @sunner 目前我自己实现了一版，一些细节还在优化中。点击【停止生成】后的效果如下：


GLIBC version requirement is incorrectly hardcoded in error message

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
Version: 1.86.0-insider (user setup)
Commit: fb769554405bee9be16e21ceb0a496bd29126941
Date: 2024-01-09T09:57:14.563Z
Electron: 27.2.1
ElectronBuildId: 26149897
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

OS Version: RHEL7.9 or CentOS 7

Steps to Reproduce:
Remote-SSH to RHEL/CentOS 7 fails to setup.
Running the remote manually shows a reason:
$ ./.vscode-server-insiders/code-insiders-fb769554405bee9be16e21ceb0a496bd29126941
[2024-01-09 09:20:19] error This machine not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= 2.17 (but found v2.17.0 instead) for GNU environments
  - find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Code Server in musl environments

The message is misleading however.
The system has GLIBC2.17 so it should not fail based on the message:
$ ldd --version|head -1
ldd (GNU libc) 2.17

The problem is that the check is really for MIN_LDD_VERSION which is now 2.28, It's only that '2.17' is incorrectly hardcoded in check_glibc_version() which is misleading,
See:
https://github.com/microsoft/vscode/blob/66bf39bd4e2e1822da2964f4b03bc364ce067c68/cli/src/util/prereqs.rs#L144

[root@xxxx ~]# cat /root/.vscode-server-insiders/.cli.fb769554405bee9be16e21ceb0a496bd29126941.log
[2024-01-10 09:16:45] error This machine not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= 2.17 (but found v2.17.0 instead) for GNU environments

find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Code Server in musl environments

遇到了同样的问题

Same issue here.
[2024-01-10 22:10:53] error This machine not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= 2.17 (but found v2.26.0 instead) for GNU environments

Same issue.
error This machine not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= 2.17 (but found v2.27.0 instead) for GNU environments

the same issue, but with additional lib:
error This machine not meet Visual Studio Code Server's prerequisites, expected either...:
  - find GLIBC >= 2.17 (but found v2.27.0 instead) for GNU environments
  - find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Code Server in musl environments
/root/.vscode-server-insiders/.cli.29bef5673ebb1fc3a59b90c9d8bfd2bc61c3f011.log

cc @roblourens or @connor4312 ? looks like a linux/c version issue?

same here, remote host is centos 7
└─╼ ~/.vscode-server-insiders $ ./code-insiders-81725794264e4c0fddbc1b3b728013744ae90840
[2024-01-15 17:56:40] error This machine not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= 2.17 (but found v2.17.0 instead) for GNU environments
  - find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Code Server in musl environments
┌─[17:56:40 mherk@faros]->(1) conda311
└─╼ ~/.vscode-server-insiders $ rpm -qa | grep glibc
glibc-headers-2.17-326.el7_9.x86_64
glibc-common-2.17-326.el7_9.x86_64
glibc-2.17-326.el7_9.x86_64
glibc-devel-2.17-326.el7_9.x86_64

I get this error also am on centos 7, I am trying to use an older version which I have but now joy is there a fix for this?
error This machine does not meet Visual Studio Code Server's prerequisites, expected either...:   - find GLIBC >= v2.28.0 (but found v2.17.0 instead) for GNU environments

find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Code Server in musl environments
Any help or advice or patch appreciated thanks

Same here. Having the same issue since this morning. I am on Amazon linux 2

same issue happening today, heres my computer info:
Version: 1.86.0
Commit: 05047486b6df5eb8d44b2ecd70ea3bdf775fd937
Date: 2024-01-31T10:27:46.147Z
Electron: 27.2.3
ElectronBuildId: 26495564
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.5.0-15-generic snap

Same here.

same issue happening here
error This machine does not meet Visual Studio Code Server
This machine does not meet Visual Studio Code Server's prerequisites, expected either...: - find GLIBC >= v2.28.0 (but found v2.17.0 instead) for GNU environments.

[2024-02-02 09:20:05] error This machine does not meet Visual Studio Code Server
's prerequisites, expected either...:   - find GLIBC >= v2.28.0 (but found v2.27
.0 instead) for GNU environments

find /lib/ld-musl-x86_64.so.1, which is required to run the Visual Studio Co
de Server in musl environments

same issue here.

Same here. Having the same issue since this morning. I am on Amazon linux 2

I have downgraded the VS code version from 1.86 back to 1.85, with that change now able to connect to my remote EC2 host.

https://github.com/microsoft/vscode/issues/203967#issuecomment-1923440629


Not seeing new/updated data in Flask

Hi,
I have a problem where data populated from a different session (CLI) doesn't show up in the server until I restart it. I initialize the Alchy object by passing in the Model and then running init_app in my Flask factory function.
As I understand Flask-SQLAlchemy will handle setting up and tearing down the sessions on each request. Do you have any idea of what this is? Have you seen it before?
This is my project: https://github.com/Clinical-Genomics/taboo/blob/master/taboo/server/ext.py
Database: MySQL

The teardown function that Flask-SQLAlchemy registers will call session.remove() at the end of all requests. So each new request should have a "fresh" session.
After you use your CLI to insert/update the database and see that the data is not shown by the web server, have you queried the database directly to see whether the data is there (e.g. using mysql CLI)? It could be that your CLI is not committing the transaction.
Otherwise, the issue could be any number of things:

Is the web page that shows the data being cached?
What kind of configuration is used for the CLI db connection vs the web db connection? Are the engine/session options used different?
How is the web app being deployed?  Is it the Flask development server?
etc.

For debugging, you could use the Flask development server and use something like import pdb; pdb.set_trace() within one of your route handlers to manually inspect the state of your database within the request. You could then do some dirty-debugging within that context to determine if the session is stale, if there's any way to remove/close/dispose of the session such that you can get to the data, etc.
But generally speaking when you have two "clients" reading/writing to the database and one of them can't see data committed by the other, then usually it's database transaction related where either the "writer" hasn't committed their transaction so no one else can see the changes or the "reader" is within a transaction and can't see any other transactions that committed after it started its own transaction.

Thank you for your extended answer! Really appreciate it 👍
I continue to debug my issue. I can provide a summary:

Data loaded via the CLI show up as expected in MySQL Workbench but not in my server
I can see the connection time resetting for every request which I guess means that part is working
I now noticed that I get different data depending if I use SomeModel.query.get(model_id) vs. db.session.query(SomeModel).get(model_id)

I think I'll use the global query for now

I now noticed that I get different data depending if I use SomeModel.query.get(model_id) vs. db.session.query(SomeModel).get(model_id)

That's a little scary actually because it implies you're potentially using multiple SQLAlchemy sessions within the same Flask web request. While it can work if you're consistent on which one you use, it's asking for trouble...
This sounds like it's not actually an issue with Flask-Alchy, but rather an issue with your app config, and probably a better fit for StackOverflow.

@robinandeer, you might want to check whether:
SomeModel.query.session == db.session
SomeModel.query.session.connection() == db.session.connection()

The Model.query property is bound to the same session as the Alchy(...).session object assuming you passed Alchy(..., Model=Model) and have SomeModel(Model). The session objects should be same whether you use Model.query or db.session.query(Model) so the fact that you are getting different results suggests you have two different database connections.
Since it appears you may have two different database connections, you may also be connecting to different databases. Might want to also check:
SomeModel.query.session.connection().engine
db.session.connection().engine

And see what databases they are connected to.
SomeModel.query.session.connection().engine.url
db.session.connection().engine.url

Okey, yeah that makes sense. I do make another database connection before I start the Flask server process. That seems to confuse the app. Skipping that makes both ways of querying return the same updated result!
You can close the issue but for completeness; comparing the sessions return false:
ipdb> db.session
<sqlalchemy.orm.scoping.scoped_session object at 0x1044418d0>
ipdb> SomeModel.query.session
<flask_sqlalchemy.SignallingSession object at 0x1051bec50>

However the connection is the same 👍
Thanks again @dgilland and @jeffwidman!