[Okcoin] Withdraw crypto not working

There are a few things wrong with the current implementation.
First it's the currency symbol. It should be changed to something like this. We also need to differentiate okcoin.com (International) with btc_usd and okcoin.cn with btc_cny.
--- a/xchange-okcoin/src/main/java/org/knowm/xchange/okcoin/service/OkCoinAccountService.java
+++ b/xchange-okcoin/src/main/java/org/knowm/xchange/okcoin/service/OkCoinAccountService.java
@@ -44,15 +44,27 @@ public class OkCoinAccountService extends OkCoinAccountServiceRaw implements Acc
   }

   @Override
-  public String withdrawFunds(Currency currency, BigDecimal amount, String address) throws IOException {
-    String okcoinCurrency = currency == Currency.BTC ? "btc_usd" : "btc_ltc";
+  public String withdrawFunds(Currency currency, BigDecimal amount, String withdrawAddress) throws IOException {
+         //    String okcoinCurrency = currency == Currency.BTC ? "btc_usd" : "btc_ltc";

-    OKCoinWithdraw result = withdraw(null, okcoinCurrency, address, amount);
+         String withdrawCurrency = null;

-    if (result != null)
-      return result.getWithdrawId();
+         if (currency == Currency.BTC) {
+                 withdrawCurrency = "btc_cny";
+         } else if (currency == Currency.LTC) {
+                 withdrawCurrency = "ltc_cny";
+         } else if (currency == Currency.ETH) {
+                 withdrawCurrency = "eth_cny";
+         } else {
+                 throw new IllegalArgumentException("Unsupported currency");
+         }

-    return "";
+         OKCoinWithdraw result = withdraw(null, withdrawCurrency, withdrawAddress, amount);
+
+         if (result != null)
+                 return result.getWithdrawId();
+
+         return "";
   }

Then it's the fee. They have different required fees for currencies. It can be changed into something like this -
+++ b/xchange-okcoin/src/main/java/org/knowm/xchange/okcoin/service/OkCoinAccountServiceRaw.java
@@ -1,14 +1,14 @@

 public class OkCoinAccountServiceRaw extends OKCoinBaseTradeService {
   private final String tradepwd;

@@ -38,9 +38,22 @@ public class OkCoinAccountServiceRaw extends OKCoinBaseTradeService {
     return returnOrThrow(futuresUserInfoCross);
   }

-  public OKCoinWithdraw withdraw(String assetPairs, String assets, String key, BigDecimal amount) throws IOException {
-    OKCoinWithdraw withdrawResult = okCoin.withdraw(exchange.getExchangeSpecification().getApiKey(), assets, signatureCreator, "0.0001", tradepwd,
-        key, amount.toString());
+  public OKCoinWithdraw withdraw(String assetPairs, String assets, String withdrawAddress, BigDecimal amount) throws IOException {
+         String fee;
+
+         if (assets.equals("btc_cny")) {
+                 fee = "0.0001";
+         } else if (assets.equals("ltc_cny")) {
+                 fee = "0.001";
+         } else if (assets.equals("eth_cny")) {
+                 fee = "0.01";
+         } else {
+                 throw new IllegalArgumentException("Unsupported currency");
+         }
+         fee = "0";
+
+    OKCoinWithdraw withdrawResult = okCoin.withdraw(apikey, assets, fee, tradepwd,
+                   withdrawAddress, amount.toString(), "okcom", signatureCreator);

     return returnOrThrow(withdrawResult);
   }

The withdraw method in OkCoin.java is missing a parameter "target" as required in the API doc
https://www.okcoin.com/about/rest_api.do
target: withdraw address type. okcoin.cn:"okcn" okcoin.com:"okcom" okes.com："okex" outer address:"address"

After I fix those, I keep getting a 10007 error code meaning "Signature does not match". I'm using the same API key and secrete key for trading, which works fine.
Then I'm stuck because I don't know how to debug the signing.
Thanks in advanced if anyone can help.

The signing it sometimes tough to debug, and is more or less trial and error. Is it only failing for certain endpoints or all endpoints in general?

@timmolter Just failing for withdrawing crypto. Account info and Trading service are working fine. Trying to fix this for a while and even reached out to their technical team but couldn't figure out the signing. Hope to get some help here.

Can you submit a PR for the fixes you've already implemented? Then I can look at the withrow impl. You're saying the following is not working right?
POST /api/v1/withdraw BTC/LTC/ETH Withdraw from https://www.okcoin.com/about/rest_api.do

That's correct. I'd follow up with a PR. Thanks in advance.

@timmolter Following up with #1717

Sorry, I looked into it, but I don't see any obvious mistakes. I don't know how to further help you on this! :(

@MartianTech
I don't have much confident with directly connect your issue but I experience a wrong signature issue before because of wrong encoding 'tradepwd'.  PR #1722

@odaken Thanks for working on this! I'd love to try it out. Does it work for you?

Btw, Okcoin is charging absurdly high network transaction fee - 0.002 for BTC. At the current price it's over $8 per withdraw. Comparable to bank wire transfer.

@odaken Hey man, I patched your PR and tested again. Although it was still failing, you were correct that it was due to encoding. My trade password had special symbols, which caused problem in encoding. I changed the trade password and now it works! Your insight is much appreciated.

@MartianTech I am grad hearing from you that it works. I did not confirm this patch works in trade password includes special characters but the patch is working without special characters password.
I totally agree Okcoin expensive trade fee. They did not accept withdrawal in case we gives lower fee and interestingly even specify higher fee. As far as I experiencet the fee has to be exactly adjusted up to date value.
@timmolter  Although this patch may not fix 100% cases but I think better to apply.

Yea what's even worse that the okcoin only apply a small portion of the fee they charge for actual transfer transaction. It means they kept the rest.
This PR is legit. We should merge it. Thanks for your work again.

Excellent work!


Navigation with path instead of hash

I am using Keycloak for authentication, which is using location.hash to transfer the tokens.
The Problem is that the hash value is changed before it can be used in the child application.
So it would be great to use location.path instead of location.hash.

I try to update with location?path but that's broken my browser ... Any idea?

I've thought about something like this:
registerApplication('menu', () => import('./menu/loader.js'), () => true);
registerApplication('home', () => import('./home/loader.js'), () => location.pathname === "" || location.pathname === "/" || pathPrefix('/home') );
registerApplication('app1', () => import('./app1/loader.js'), pathPrefix('/app1'));

start();

function pathPrefix(prefix) {
    return function(location) {
        return location.pathname.indexOf(`${prefix}`) === 0;
    }
}

That's the way they do it in the single-spa-example. But I could not test it yet.

Ok thanks you, I try this week.

👍 single-spa works with both hash routing and html5 history api routing (pushState/replaceState). A lot of my original examples had hash routing, but I have since updated them to use path routing, since that's the industry standard.
I think that it would be best for single-spa-angular-cli to support either form of routing, but default to path routing. If only one can be supported, path routing is probably the better default.

Hello I fix it on 0.2.3 :
singleSpaAngularCliRouter.setPathStrategy('pathname');
I introduce 2 breaking changes :
singleSpaAngularCliPlatform.mount('app1', Router).subscribe(({ props, attachUnmount }) => {
platformBrowserDynamic().bootstrapModule(AppModule).then((module) => {
attachUnmount(module);
// Do something with props if you want
// Ex : module.instance.setSomething(...)
});
});
It is now the name of the app, not the root selector : app1 replace app1-root
And { props, attachUnmount } to retreive the props
And One new feature on my exemple :
registerApplication('help', import('./loaders/help.js'), singleSpaAngularCliRouter.hasParameter('help', 'open'));
--> has parameter
Hope is ok for you tanks to this fix, fell free to close the issue if it is ok
Bye

sorry @joeldenning I do not use path as default because that's don't work for me, maybe it is my bad but I've not a lot of time to find the solution.
But next time when I will work on the project I will try to fix it and use path as default

@robinComa sorry to hear that. What were the issues that you ran into with path based routing?

I don't know, i switched quickly because I have no time.
I will try later!

Since version 2.0.0 my loaded apps are not bootstraped anymore. Neither with hash routing nor with path routing.  The files get loaded but the apps are not bootstraped
I've changed the main.ts files of the loaded apps the way you posted.
Do you have any ideas what could be wrong?
import { registerApplication, start } from 'single-spa';
import { singleSpaAngularCliRouter } from 'single-spa-angular-cli';
import 'babel-polyfill';
import 'zone.js';
import './style.scss';

registerApplication('menu', () => import('./menu/loader.js'), singleSpaAngularCliRouter.hashPrefix('/**'));
registerApplication('home', () => import('./home/loader.js'), singleSpaAngularCliRouter.hashPrefix('/home', true) );
registerApplication('app1', () => import('./app1/loader.js'), singleSpaAngularCliRouter.hashPrefix('/app1'));

start();

import singleSpaAngularCli from 'single-spa-angular-cli';

const lifecycles = singleSpaAngularCli({
  name: 'menu',
  selector: 'menu-root',
  baseScriptUrl: '/apps/menu',
  styles: ['styles.bundle.css'],
  scripts: ['inline.bundle.js', 'polyfills.bundle.js', 'vendor.bundle.js', 'main.bundle.js', 'scripts.bundle.js']
});

export const bootstrap = [lifecycles.bootstrap];

export const mount = [lifecycles.mount];

export const unmount = [lifecycles.unmount];

export const unload = [lifecycles.unload];

I have the same issue when I do the import inside the function, I know is it bad, but for now I do it
Could you try like this?
If your interrested, a new version is here :

It use the location.pathname, location.search and history.pushState
It simplify the loader
Reorganise the imports from the namespace
Allow ng serve for apps inside the poral

If I use http://localhost:8080, the loaded apps are still not bootstraped.
If I now reload the page with the path behinde the base URL the sub page scripts are loaded without the outputPath.

I've solved the reload Problem by configuring my node server.

Ok perfect I think you need the fallback history mode.
I close the issue
Thanks for the feedback


Fix typo in weak_ssl documentation

verification instead of verfication.

Hi @Safranil, thank you for your PR!


Refactor team list

Right now it will still load team multiple times when running in --all mode. This is hard to get away from because of the "load team and retry with repoll if we can't find member in it" mechanism and the fact that team loads happen concurrently. My attempts at making a team cache (in front of loader cache) did not have any significant impact. Also, the common case of just getting the teams current user is in is fast enough with warm caches. There will be another ticket for simpler version of it that frontend can benefit from (we can skip loading invites for frontend).
This PR introduces code cleanup when some of the functionality is moved to separate function, so indentation level has been dropped by one in List function. It also loads username/fullname only once if running in keybase team list-members or ``keybase team list-members -u username` mode.
It also adds list sorting on cli side, similar to what front-end does on its side already.

This looks like a good improvement of factoring out pieces of the function so it's more digestible. But I'm not sure it addresses the mandate of CORE-6251 to split the function in twain

list all the team memberships for a user
list all the teammate memberships for a team
(and possibly a third that does the --all thing if we really need that, probably by calling the others)
which may involve avdl changes.

I'd like to pull in @patrickxb who wrote that ticket

I tried to split them into two functions, where:

would handle --all, and would be managing some "team list" local cache so it wouldn't have to Load same team multiple times for multiple people. This turned out to be hard when concurrency is in play, and also Load turned out to be pretty fast.
that would handle team list-members and team list-members -u username, that didn't need "team list" because it would be only checking one membership per team and only load username/fullname once.

In the end, function (1) without local cache turned out to be very similar to (2) so I just combined them back together in commit https://github.com/keybase/client/pull/8976/commits/9fc15b80e8a99665c1e1e658ecdcf040a758094c
The biggest win here seems to be UIDMapper, on some tests with prod server I tend to get 2x speedup overall in keybase team list-members --all with warm caches, but the numbers are 400ms -> 200ms so it was pretty fast to begin with despite having to go through hundreds of users.
If there is some significant simplification that I've missed, I'll be happy to work on this more.

OK - removed change from uidmap.go.


Interactive apps like 'req' should spit output & prompts to stdout

Looking at the apps/ directory, I see bio_err being used in a lot of places that bio_out would be more suitable.
Is this worth scrubbing so that script writers can correctly discard stdout via >/dev/null without worrying about any useful diagnostics being lost?
I'm willing to do the PR but not if it's not going to be accepted...

I would be in favor of this kind of cleanup of stdout/stderr.  But it might have to wait for 1.2.0 if folks think “output compatibility” is implied.  Raising the issue here to discuss is exactly the right way forward.

Okay, good.  Let's start with the obvious.  How many tests read run actual commands and read their output from only STDERR, for instance.

I'm all for a cleanup. However, I disagree about prompts (specifically, password prompts) going to stdout, exactly because a redirection of stdout will make a prompting command appear to just hang and the user not quite knowing what to do next...

Stderr should be for error messages and diagnostics.
Stderr is not for prompts. That is what stdout is for.
You might find the reference https://www.jstorimer.com/blogs/workingwithcode/7766119-when-to-use-stderr-instead-of-stdout useful as it quotes Doug McIllroy

Doesn't matter, I just went back to the source and reminded myself that we do the password prompting directly with the tty anyway.
So [ahem] never mind what I said up there...

Tim Hudson wrote:

Stderr should be for error messages and diagnostics.
Stderr is not for prompts. That is what stdout is for.
Hmm, intentionally stderr is for prompts.

You might find the reference https://www.jstorimer.com/blogs/workingwithcode/7766119-when-to-use-stderr-instead-of-stdout useful as it quotes Doug McIllroy
User prompt is not mentioned in document above.

Manual page readpassphrase(3) refer for stderr if no tty is available.
Regards,
Roumen Petrov

Richard, why you give up?
For prompts OpenSSL user interface functionality  uses tty (by default) or stderr (as fail-back). Functionality can not be used in openssl commands as input if from tty and this is incompatible modification - it will break exiting scripts including those used in openssl regression tests.
I would like to propose issue to be closed.

Hmm, I only discussed password prompts, while @pprindeville seems to discuss other prompts (non-password ones). I have no comment on the latter.

Hmm, I only discussed password prompts, while @pprindeville seems to discuss other prompts (non-password ones). I have no comment on the latter.

Under what circumstances would it make sense to send some prompts to STDERR and still others to STDOUT?

I don't think @levitte is talking about anything other than password handling - and that is using the tty rather than stdin or stdout for handling things.
There are multiple views on that - but in terms of providing prompts for input on stdin those should be provided on stdout. And stderr was explicitly designed for diagnostic or error output - not for prompts in general.

Just to restate the problem: some programs (like Travis) collect output from stdout and stderr separately so they can highlight output from warnings/errors differently, or filter stderr differently for patterns indicative of critical warnings, etc.
All of these apps should be capable of being run with >/dev/null without losing any critical information... and indeed, with the only output being purely critical information (i.e. warnings and errors).

Marking as inactive, to be closed at the end of 3.4 dev barring further input

To be clear, I’d like results/feedback from nominal operations going to stdout and diagnostics going to stderr.

@pprindeville are you able to submit a PR for this issue?  I'm happy to mark it as a community effort

thank you!  Assigning it to you


[FIX] Total Contributors is invisible

This PR closes #68

See associated issue.

See associated issue.


[FLINK-11932] [table-planner-blink] Add support for generating optimized logical plan for 'select * from mytable'

What is the purpose of the change
Add support for generating optimized logical plan for 'select * from mytable'
Brief change log

add BatchTableEnvironment
introduce Optimizer interface and implements for batch and stream
add rules and programs for batch and stream
add registerTableSource, optimize, explain methods in TableEnvironment
add plan test infrastructure (is duplicated by https://issues.apache.org/jira/browse/FLINK-11685)
add RelTreeWriterImpl to make query plan easier to read (is duplicated by https://issues.apache.org/jira/browse/FLINK-11680)

Verifying this change
This change added tests and can be verified as follows:

Added tests that validates explain result
Added tests that validates optimized logical plan

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented)

+1 to merge


[Bug]: I can't lauch the game (slime rancher 2)

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
Hello each try to lauch slime rancher 2 with mod it say to update .NET 6.0 and I don't know how please help
Latest-Bootstrap.log
Did you attach your log file?
Yes, I attached my log file to the text box above.

Use this installer to install .net: https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-desktop-6.0.32-windows-x64-installer


Bug fix: resolve missing catalog_facet_url route

Spotlight appears to have a bug that prevents some facets displaying by default. This corrects the issue by:

Adds many facets to catalog_controller.rb
Overwrites Spotlight's default controller.rb concern to map the catalog_facet_url (errors) call to facet_catalog_url instead.
Will add issue upstream in Spotlight

Fixes #11

Looks good.  The fatal error is has been eliminated, and facet.field params are added to solr request as expected.


Allow for Reproducible Builds

Hello,
it would be great if you could allow Android users to run reproducible builds to verify that the version downloaded from Google Play is 100% equivalent to the source code here on Github.
Here is an example how the Signal App does this via docker:
https://github.com/signalapp/Signal-Android/blob/master/ReproducibleBuilds.md
Thanks, Marc

Thank you very much for your suggestion! We'll let the development team know and move this issue to the android app repository as soon as it is published.

Thanks for open this up @hirschnase and thanks SAP&Telekom to making this OpenSource 👍
I'm pretty sure the iOS community is embracing reproducible builds as well. This link might be worth looking, to check how Telegram is doing it for Android&iOS.

Have a look at the documentation, how F-Droid handles reproducible builds: https://f-droid.org/en/docs/Reproducible_Builds/
There is also translation into German: https://f-droid.org/de/docs/Reproducible_Builds/

Reproducible builds will probably be an essential feature for an open source app.
When having open source community builds (see #5), the app usually is signed by the maintainer's key, as the maintainer does not have (and should not have) the developer's key.
Google announced to only allow one app per country. This will probably boil down to allowing access based on the developer's signature and prevent any community builds.
But when the app can be built reproducible, the community can build the app and get an identitic binary. For this binary the signature will still match and the community is sure that the official binary (that is the same as the community binary) is built from the same source code and be able to trust the official binaries more.
This does of course not solve issues like community builds that want to remove trackers (like crashlytics, firebase analytics, etc.), but that's nothing you can solve, but a problem with Google's rule only to allow one app.

Because of "Reflections on Trusting Trust",  you also need to have a reproducible compiler, and that compiler should have been compiled reproducibly as well. Downloading some binary compiler from the Internet only gives you the same bytes, not any certainty that nothing shady is going on.
Nix/Guix might be tools that help deal with these issues

@jellelicht I think this is out of scope here. I would not suspect that the person who uploads some app has manipulated my compiler.
Reflections on trusting trust is about trusting a compiler itself when you distrust the person who built the initial compiler binary. That scenario would only matter, when the author of the app was the author of the compiler that you're using as well.
And we're here not talking about a secret agency infiltrating your whole OS, but about people wanting to verify the binary before including it into an app catalog for apps that were built from source.

@allo I'm fairly certain that's a bit handwavy, especially if there is no proper risk model.
I agree that going that far seems out of scope for now though.

And to clarify: The author of the app would only need to meddle with any of the compilers leading to the compiler you currently use. That is the core argument of the paper, and makes me look ever so slightly less paranoid for bringing it up in the first place

Thank you very much for your suggestion! We'll let the development team know and move this issue to the android app repository as soon as it is published.

Hi @tkowark, you mentioned above this issue will be moved. As it was discussed a lot in other issues here or on social media, I wanted to clarify the status.
I also found one example how to do it with Maven, although I am not an app developer at all. https://t.co/YLJusFoqIO

As this issue was initially only targeted at reproducible builds for Android, I initially wanted to move it to that repo.
In the meantime, we also got similar requests for iOS and therefore, we decided to keep the issue centrally in the documentation repository since it applies to both apps and reproducibility in general. We'll let you know here, centrally, once we have any updates on that matter.

On Mon, Jun 15, 2020 at 11:14:55AM -0700, Christoph Fritzsche wrote:

I also found one example how to do it with Maven, although I am not an app developer at all. https://t.co/YLJusFoqIO

if https://zlika.github.io/reproducible-build-maven-plugin/ doesn't give you
sufficient information, feel free to come around either at
<EMAIL_ADDRESS>or also on irc on #reproducible-builds
on irc.oftc.net.
I'm very much looking forward to personally verify the corona app shipped is
the one built from the sources it's being said to be built from! ;)
I'm also willing to help if+where I can.
--
cheers,
Holger

           holger@(debian|reproducible-builds|layer-acht).org
   PGP fingerprint: B8BF 5413 7B09 D35C F026 FE9D 091A B856 069A AA1C

Another approach that may be helpful in this context is Gitian (https://gitian.org/). It was developed for Bitcoin to produce deterministic builds. I do not know if this is applicable for Android and iOS builds though.

I'm working on monitoring wallet apps for Android in an automated fashion and would like to extend it to corona apps as there appears to be more demand than for Bitcoin wallets for reasons I don't fully understand.
Anyway, what I have so far:

An Android app that monitors updates and gets triggered by reproducible apps (hand selected 6 Bitcoin wallets out of 155 analyzed apps which almost all were not open source or reproducible).
App update gets hashed.
Hash gets sent to the server.
If server doesn't know that hash, app update gets sent to server.

I have to manually run this test script on the files and as no app provider committed to not break my scripts so far, it often requires changes in the test script to get to a result like:
Results for
appId:          de.schildbach.wallet
apkVersionName: 8.02
apkVersionCode: 802
apkHash:        f01e4028778bc2036902af2253522b7de0eb40ca3bff50f51a8c0918737fd6b4

Diff:
Files /tmp/fromPlay_de.schildbach.wallet_802/apktool.yml and /tmp/fromBuild_de.schildbach.wallet_802/apktool.yml differ
Only in /tmp/fromPlay_de.schildbach.wallet_802/original/META-INF: BITCOIN-.RSA
Only in /tmp/fromPlay_de.schildbach.wallet_802/original/META-INF: BITCOIN-.SF
Files /tmp/fromPlay_de.schildbach.wallet_802/original/META-INF/MANIFEST.MF and /tmp/fromBuild_de.schildbach.wallet_802/original/META-INF/MANIFEST.MF differ

which I then publish on this website.
Now my focus are Bitcoin Wallets but I released almost all as open source, so if the findings are interesting but somebody else wants to step up and run a coronascrutiny service, that would also be an option. I just hope there are synergies. I would also be very interested in adding it to the same site.
Currently the biggest hurdle is probably to get the binary updates for iPhone in a comparable way as I hope to soon collect the Android updates: On user devices as they happen. The problem is that those marketplaces can release different apps to people depending on device, language, country, ... and basically could even release a modified version to named individuals (not that Google would need a Corona app to spy on you).

The swiss covid app allows for reproducible builds now:
https://github.com/DP-3T/dp3t-app-android-ch/blob/master/REPRODUCIBLE_BUILDS.md

wow, very neat. the instructions are also pretty clear and short and feature downloading the apk with adb from the phone and comparing that with the rebuild. really very cool! hoping we'll get that for the german corona app as well!

Any news on this?

Is there any ETA for this? It's the one thing preventing me from installing the app.

If the Swiss App supports reproducible builds, why doesn't this one?

Because government doesn't care/pay for it? Didn't know that people would
care and doesn't change mind if they have realised?
The maintainers of this repo aren't typical open source maintainers. They
are only really accountable to their gov client not the users/dev
community. Hence, if it doesn't matter to gov it won't happen.
Unfortunately, the end client, i.e. gov, doesn't take part in discussions
here.
And gov will never authorise anyone to issue opinionated statements on
here. So it's a dead end. Maintainers say out of scope, talk to gov and gov
doesn't respond (too busy fighting pandemic!). Just look at country app
stores. Turkey, US, India, Russia, Israel etc are still not included
despite significant pressure on GitHub, Twitter and media.
On Thu, 9 Jul 2020, 22:28 Andreas Kuckartz<EMAIL_ADDRESS>wrote:

If the Swiss App supports reproducible builds, why doesn't this one?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/corona-warn-app/cwa-documentation/issues/14#issuecomment-656335022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF77AQOOA34KXQRA6I7OBS3R2YR6JANCNFSM4M7XMWJA
.

While what your saying is not wrong, @corneliusroemer, there is a trivial additional/alternative explanation: Priorities. If you check the backlog repo, interoperability, time-to-warning and UX improvements are being worked on. Doing that before the reproducible build is understandable to me.
Also from the RKI perspective, getting some more information on symptom onset or opportunistic scanning in crowds would be more important.
Furthermore, in the release press conference, the top SAP executive indicated this reproducible build topic is important for him. So don’t discount it as dead yet either.


feat: design ens verification flow

[sc-714]

[sc-714]

[sc-714]

[sc-714]

Just to recap our convo, did we say there was an issue with using subdomain to verify instead?
I.e., in order for someone to add the ENS, they first go to ENS and add a TXT record saying nfa. (e.g., nfa.fleek.eth) and then we check for that. And that way we don't have to worry about transfers because it's up to the new recipient of the ENS if they want to keep it or not.

I think I did not get exactly about the transfer part. This is a single time validation, by now we do nothing if the NFA or ENS is transferred. By the way it checks by owner of the domain/subdomain, if an ENS owner want someone to create e NFA using his domain, he will need to create a subdomain for this someone to use it (e.g. nfa.fleek.eth). I think this is the simples to way to verify because the owner doesn't need to change nothing in his ENS to be able to mint a NFA using it, what do you think?

Just to recap our convo, did we say there was an issue with using subdomain to verify instead?
I.e., in order for someone to add the ENS, they first go to ENS and add a TXT record saying nfa. (e.g., nfa.fleek.eth) and then we check for that. And that way we don't have to worry about transfers because it's up to the new recipient of the ENS if they want to keep it or not.

I think I did not get exactly about the transfer part. This is a single time validation, by now we do nothing if the NFA or ENS is transferred. By the way it checks by owner of the domain/subdomain, if an ENS owner want someone to create e NFA using his domain, he will need to create a subdomain for this someone to use it (e.g. nfa.fleek.eth). I think this is the simples to way to verify because the owner doesn't need to change nothing in his ENS to be able to mint a NFA using it, what do you think?

Yes, this definitely works for the mint part. I was wondering if the ENS gets transferred after the minting, then it might not map to the site anymore but the NFA will still point to the ENS. So wondering if we can address that somehow ...

I think handling the transfer is a big piece. We can also leverage our substreams module to handle transfers. Please, let me know when this is being discussed next time so that I can join the discussion :)


Read(timeout) stuck when no data from device

Hi Contributors,
Before I get into my issue, thank you very much for the library.
I am writing a simple winform app to write and read data from my custom HID device (keyboard). It work fine when connect, write and read data in order. But when I does not write data to the device and call Read(timeouts) function, the app will stuck because the read func is waiting for new data form IO.
I have tried to edit read function by add a Task and cancel when timeout occurs but I just created new issue that write feature is not work anymore. I think because the read method is still holding the IO.
I googled about my problem and figure out that the read method can set the timeout but it set for C++ platform, so I dont know how to deal with it and still stuck in C#.
So I raise a issue and hope that anyone has faced this issue and knew how to deal with it in C# library (nativemethod wrapping).
Thank you,
Trong Truong

I'm facing the same issue currently.


Index out of range error

Currently there’s a /PokeDB/utils/DBUtils.py" line 41, in get_device_name error spitting out. Being looked into. All commands should still work as normal

Error should be resolved now


Does not work with custom context path

Setting a different context path than / for the application results in missing resources. Some files are still available under the root context.
Reproducible with vaadin spring-boot-starter and setting server.servlet.context-path: /example or the app-layout-annotation-example and following configuration:
<plugins>
  <plugin>
    <groupId>org.eclipse.jetty</groupId>
    <artifactId>jetty-maven-plugin</artifactId>
    <version>${jetty.version}</version>
    <configuration>
      <webApp>
        <contextPath>/example</contextPath>
      </webApp>
    </configuration>
  </plugin>
</plugins>```

#154 seems to be the same problem.

Tried prefixing all imports with context://frontend/..... and it seems to work. But I'm not sure if this is the right fix.

I will put some effort into this tomorrow. Sorry for the delay!

I have not tried it yet, but not sure if this is the right solution. According to this blogpost https://vaadin.com/blog/vaadin-10-and-static-resources Vaadin does some magic with the frontend-directory but your solution would prevent that. Still trying to understand this new way of resource handling in vaadin flow.


Fix security vulnerability

Fix security vulnerability in the dependencies.

Fixed in commit https://github.com/jfehrman/pomodoro/commit/da8ef12983fe1146c6775066348f37785b27fe41.


Content organization

Take inspiration from Diátaxis to organize the content into the following categories.

Guides
Recipe
References
Crash Courses

Done with v2


Mynah-ui parse paragraph as code block.

Problem
mynah-ui parse paragraph as code block.
Actual input
Hi, I'm the Amazon Q Developer Agent for software development.

    I can generate code to implement new functionality across your workspace. We'll start by discussing an implementation plan, and then we can review and regenerate code based on your feedback.

    To get started, describe the task you are trying to accomplish.

Steps to reproduce the issue

Go to https://aws.github.io/mynah-ui/
Enter

Hi, I'm the Amazon Q Developer Agent for software development.

    I can generate code to implement new functionality across your workspace. We'll start by discussing an implementation plan, and then we can review and regenerate code based on your feedback.

    To get started, describe the task you are trying to accomplish.

Same thing happens in Amazon Q /dev

Expected behavior
Parse as paragraph.
System details (run the AWS: About Toolkit command)

OS:
Visual Studio Code version:
AWS Toolkit version:

This is is reverted back to keep markdown default behavior as is without overriding the parsing mechanism. The reason behind that is that markdown reads the empty spaces as code blocks.
Can be shown in some markdown generators like Lorem Markdown.
Here's a basic markdown output it generates for a code block:

Since the support of the markdown in the prompt input field is crucial to give the ability to user writing markdown elements like code blocks and inline codes, as discussed with UX team the decision is to keep the prompt input to support default markdown formats without overriding the rendering.

Another useful information:


New tokenizers support

Do you have plans to use the new 的 tokenizers? Transformers are too slow; it takes 10 seconds to start on ax630C. If possible, could you refer to the implementation at: https://github.com/huggingface/tokenizers.git? This one imports very quickly.

I tried to write a tokenizer service, which might be useful. You can find it at: https://github.com/m5stack/StackFlow/blob/dev/projects/llm_framework/main_llm_tokenizer/_tokenizer.py


Fix some tests that use deprecated MLJBase code

Codecov Report

Merging #191 (38e79d2) into dev (87f370a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##              dev     #191   +/-   ##
=======================================
  Coverage   86.44%   86.44%
=======================================
  Files          13       13
  Lines         649      649
=======================================
  Hits          561      561
  Misses         88       88


Deleting multiple folders in WSL2 makes blue screen and does not release the space

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

VS Code Version: Version: 1.75.1 (user setup)
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:32:34.589Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No
OS Version: WIN11 22H2 X64, WSL2 (Ubuntu 22.04)

Steps to Reproduce:

Run Ubuntu 22.04 via WSL2 in VSCODE
Create multiple folders (at least 10) including big data inside.
Delete all folders at the same time.
Move your mouse and wait around 5s. Blue screen will show.

Restart your PC. Restart WSL2. Find all the folders were gone, but the space was not released.
Try >diskpart, select ext4 file path, >compact vdisk, >detach vdisk. Didn't work. Space not released.

Thanks!

Thanks for reporting. Can you report this this against https://github.com/microsoft/WSL ?
Probably a duplicate of https://github.com/microsoft/WSL/issues/7335

Yes, that's a duplicate. Thanks!


TracingInterceptor tests fails in Micronaut Reactor

@dstepanov TraceInterceptorSpec.test trace mono test fails. Can you investigate?

Hi @sdelamo, changing test code to makes test pass:
        @NewSpan("trace-mono")
        Publisher<String> mono(@SpanTag("more.stuff") String name) {
            return ReactivePropagation.propagate(
                    PropagatedContext.get(),
                    Mono.fromCallable({
                        spanCustomizer.tag("foo", "bar")
                        return name
                    }).subscribeOn(Schedulers.boundedElastic())
            )
        }

I guess the issue is with subscribeOn part because it happens on another thread.

I will check

@dstepanov can you check this?

This is not supported by default, but the propagation using Micrometer should work


Enhancement

I want to add new features on it in which

the admin can calculate the total working hours or active hours of the employee which will show the total active hours and,
improve the UI of the system
Can I work on this??

Actually I want to add a feature call admin in which view employees working or active hours. It will be be extra feature to this project
I am currently working on the code .
I can proceed if you accept..can you proceed??

Can I procced??

@sarasij190  could you explain a bit how is this related to rollcall ?


Mapping from double? to complex type generates wrong code

Describe the bug
When attempting to map from a class that has a property of type double? to a class with the same named property, it generates incorrect code. We expect the code that is generated, to either have a null check on the property before passing it or just passing the value fully, instead of calling .Value on it. It will generate warnings/errors depending on the csproj setup when nullable is enabled for that. However, calling .Value does not make sense since a method has been provided on the mapper that should be able to cope with the double? type.
In the following snippet you can see the DtoMapper that has been declared where the above scenario is being attempted.
[Mapper]
public partial class DtoMapper
{
    public partial NotNullableType To(TypeWithNullableProperty y);

    public Wrapper Map(double? source)
    {
        if (!source.HasValue) return null;
        return new Wrapper()
        {
            Test = source.Value
        };
    }
}

public class Wrapper
{
    public double Test { get; set; }
}

public class TypeWithNullableProperty
{
    public double? Test { get; set; }
}

public class NotNullableType
{
    public Wrapper Test { get; set; }
}

Generated code
This is the code that has been generated
public partial NotNullableType? To(TypeWithNullableProperty? y)
{
    if (y == null)
        return default;
    var target = new NotNullableType();
    target.Test = Map(y.Test.Value);
    return target;
}

Expected behavior¨
I guess the following code is what we actually expected that would happen in this case.
public partial NotNullableType? To(TypeWithNullableProperty? y)
{
    if (y == null)
        return default;
    var target = new NotNullableType();
    target.Test = Map(y.Test);
    return target;
}

Environment (please complete the following information):

Mapperly Version: [2.2.0]
.NET Version: [6.0.102]
Target Framework: [.net6.0]
OS: [Windows 11 22000.556]

Thank you for the report. We will look into it.

Thanks for the quick fix. Will check it out!

Could you make either a pre release or normal release on this so that it is possible to download through nuget?

Currently we release pre-releases only to the github nuget server. This document describes, how you can add the nuget server. The owner is riok, the pre-release package is available here: Riok.Mapper-v2.2.1-next.1.


How to abort a file upload

There seems to be code in file and http-request to handle aborting an upload but I could not find the right way to do so.
If this is doable with the current code base, then doc is probably missing.
If it is not doable, then it might be useful adding it.

file.upload returns a cancelable promise, which you can cancel using cancel.
so,
let task = file.upload();
task.cancel();

@tim-evans, unfortunately, this snippet does not work for me. task.cancel is undefined.
I've digged a bit in the code base and I think there might be a bug at https://github.com/tim-evans/ember-file-upload/blob/master/addon/file.js#L96.
This line returns a new promise, not the cancelable one. I have fixed it locally by doing the following
  let cancelablePromise = uploadFn(request, options);

  cancelablePromise.then(function (result) {
    set(file, 'state', 'uploaded');
    return result;
  }, function (error) {
    set(file, 'state', 'failed');
    return RSVP.reject(error);
  }).finally(function () {
    // Decrement for Ember.Test
    inflightRequests--;
  });

  return cancelablePromise;

What do you reckon ?

Could you fix this with a PR?

Thanks for finding this!


Base on senzingapi-runtime

Pull requests:

https://github.com/senzing-garage/senzing-tools/pull/174


[Developer Portal] - Usability / API credentials options after login

UI: Details view of the of an API

Point 1
Provided two options for the subscription can confuse the user.

KEY GENERATION WIZARD
SUBSCRIBE TO AN APPLICATION

Point 2
The terminology used to describe the second option is not accurate. In this case, it's not the Application that the user subscribes to but to an API using an Application.
Based on these 2 reasons, it would be better to re-design these two buttons as,
Fieldset title: "Subscribe with ..."
↳Button 1 title: "A New Application"
↳Button 2 title: "An Existing Application"

Fixed with the changes done with above PR


Update master with latest epinio changes

port of latest changes from epinio-dev
this will allow a cleaner merge of https://github.com/rancher/dashboard/pull/6001

@aalves08 Just the files outside of pkg is fine


Create welcome page for new users to we.hypha.app

Page is here
Purpose: Page is linked on Hypha Docs "Commune with us!" Page, intended to be first stop for new users who want to be introduced to the community.
Need to add:

guidelines for forum use
template for Introductions
Information on how to create an account (and figuring out who has what kind of access?)

I believe @Erioldoesdesign created this page, which has many of these things.
I edited slightly for readability.
Two things we may still want to add are:

guidelines for forum use (something about our forum culture, rules - does this exist anywhere else in Hypha's info, or is there a template for it I could start with?)
Info on how to make an account (not sure if this is redundant--would folks even be allowed to see we.hypha.app forum, and/or this specific post, without having created an account?)


Naming collisions with std::marker::Sync/Send

Bug Report

Version

tonic_example_broken_project v0.1.0 (/workspaces/tonic_example_broken_project/broken)
└── tonic v0.12.0
└── tonic-build v0.12.0

Platform
Linux f6ab1f425a44 6.5.0-41-generic #41~22.04.2-Ubuntu SMP PREEMPT_DYNAMIC Mon Jun  3 11:32:55 UTC 2 x86_64 GNU/Linux

Crates

tonic_build

Description
When using tonic_build and attempting to do codgen with a proto service called Sync or Send you encounter an error like this:
vscode ➜ /workspaces/tonic_example_broken_project (main) $ cargo build
   Compiling tonic_example_broken_project v0.1.0 (/workspaces/tonic_example_broken_project)
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.09s
vscode ➜ /workspaces/tonic_example_broken_project (main) $ cargo build
   Compiling tonic_example_broken_project v0.1.0 (/workspaces/tonic_example_broken_project)
error[E0391]: cycle detected when computing the super predicates of `telemetry_ingester::sync_server::Sync`
  --> /workspaces/tonic_example_broken_project/target/debug/build/tonic_example_broken_project-8247014933aa3ceb/out/my_proto.rs:95:28
   |
95 |     pub trait Sync: Send + Sync + 'static {}
   |                            ^^^^
   |
   = note: ...which immediately requires computing the super predicates of `telemetry_ingester::sync_server::Sync` again
note: cycle used when checking that `telemetry_ingester::sync_server::Sync` is well-formed
  --> /workspaces/tonic_example_broken_project/target/debug/build/tonic_example_broken_project-8247014933aa3ceb/out/my_proto.rs:95:5
   |
95 |     pub trait Sync: Send + Sync + 'static {}
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   = note: see https://rustc-dev-guide.rust-lang.org/overview.html#queries and https://rustc-dev-guide.rust-lang.org/query.html for more information

error[E0391]: cycle detected when computing the implied predicates of `telemetry_ingester::sync_server::Sync`
  --> /workspaces/tonic_example_broken_project/target/debug/build/tonic_example_broken_project-8247014933aa3ceb/out/my_proto.rs:95:28
   |
95 |     pub trait Sync: Send + Sync + 'static {}
   |                            ^^^^
   |
   = note: ...which immediately requires computing the implied predicates of `telemetry_ingester::sync_server::Sync` again
note: cycle used when computing normalized predicates of `telemetry_ingester::sync_server::SyncServer`
  --> /workspaces/tonic_example_broken_project/target/debug/build/tonic_example_broken_project-8247014933aa3ceb/out/my_proto.rs:97:5
   |
97 |     pub struct SyncServer<T: Sync> {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   = note: see https://rustc-dev-guide.rust-lang.org/overview.html#queries and https://rustc-dev-guide.rust-lang.org/query.html for more information

For more information about this error, try `rustc --explain E0391`.
error: could not compile `tonic_example_broken_project` (lib) due to 2 previous errors

Please see my example repository showing a minimal example and my fork of tonic_build fixing this issue.
I will submit a PR shortly.

Resolved at #1804.


Parameter handling is inconsistent

Currently, parameters that are objects are not encoded as strings, however, arrays are.
Object parameter: https://github.com/hyperledger/firefly-fabconnect/blob/main/internal/rest/utils/params_test.go#L91
Array parameter: https://github.com/hyperledger/firefly-fabconnect/blob/main/internal/rest/utils/params_test.go#L306
This is not consistent, and begs the question how to pass objects with nested arrays and arrays of objects.
Tests are missing for objectArray.
Firefly is sending objects encoded as strings, and this results in errors, see:
https://github.com/hyperledger/firefly/issues/1307

@DerekJarvis You have more insight into this?

@DerekJarvis You have more insight into this?

Sorry - haven't worked with Firefly for some time. I did a PR that tried to improve some type handling, but overall it doesn't seem like a current focus of Firefly to handle the complex data types that Hyperledger Fabric supports.


how would you suggest doing responsive images?

This implementation uses a plain old img tag, but I'm wondering how you'd go about handling different screen resolutions and display densities.

I've looked at srcset attribute and <picture> element but these don't seem well-suited to dynamically sized images; they all make display choices based on viewport width


Call same animation with a different target

Given that I create an animation like so:
var noticeMe = anime({
    targets: ('.focus'),
    autoplay: false,
    color: [
        { value: '#ccc', duration: 200, delay: 0, easing: 'easeInOutQuad' },
        { value: '#3333ff', duration: 400, delay: 200, elasticity: 100, easing: 'easeInOutQuad' },
        { value: '#000', duration: 3000, delay: 200, elasticity: 100, easing: 'easeInOutQuad' }
    ],
    translateY: [
        { value: -20, duration: 400, elasticity: 100, delay: 0, easing: 'easeInOutQuad' },
        { value: 0, duration: 200, delay: 200, elasticity: 100, easing: 'easeInOutQuad' }
    ],
    scale: [
        { value: '1.05', duration: 100, delay: 400, easing: 'easeInOutQuad' },
        { value: '1', duration: 30, delay: 0, easing: 'easeInOutQuad' },
    ]
});

After the animation is run, the DOM is manipulated by some other JavaScript and a new element now has the .focus class. However, calling noticeMe.restart() still affects the element that no longer has that class. Is there any way to call the animation with the correct target?

You can't, but you can wrap your animation in a function with an argument for the target.

Just what I was currently trying to do. Thanks for the prompt reply and the great resource 👍


Don't install freetype on OS X

It's not supposed to be used on this platform.

This change is

@bors-servo r+

:pushpin: Commit c67ba35 has been approved by KiChjang

:hourglass: Testing commit c67ba35 with merge 9c32d26...

:sunny: Test successful - travis


Listing tags

Some tags are not loaded by default and need to be specified in the styles file on etl step.
How about allowing to list the available ones with osmf?

I feel like this is out of scope. The possible tags are listed at https://wiki.openstreetmap.org/wiki/Map_features, which I reference in the usage text. Or did I misunderstand you?

some commonly used tags such as maxspeed are not included by default. By listing I meant "list what is currently loaded in the db".

OK, I now understand what you meant. However, I don't think that this is a needed feature. Someone who knows how to use a custom style file with osm2pgsql probably also knows how to do psql -d gis -c '\d planet_osm_point' to inspect the database.

Ok. How about at least changing the error msg on attempting to do a query on a non-existing tag?

I'm sorry, but I'm not going to change anything, unless I'm convinced the change will be useful for anyone. I feel like an error message like ERROR: column "foo" does not exist is self explanatory and doesn't need improvement.


User registration fails with GBIF username

We might have hit a bug with the user registration process in which the user has to use their email address the first time they login. Otherwise they get a 401 basic authentication error with this stack trace:
java.lang.IllegalArgumentException: User NicBailly does not exist
	at life.catalogue.dw.auth.IdentityService.get(IdentityService.java:83)
	at life.catalogue.dw.auth.IdentityService.authenticate(IdentityService.java:156)
	at life.catalogue.dw.auth.AuthFilter.doBasic(AuthFilter.java:229)
	at life.catalogue.dw.auth.AuthFilter.filter(AuthFilter.java:80)
	at org.glassfish.jersey.server.ContainerFilteringStage.apply(ContainerFilteringStage.java:108)
	at org.glassfish.jersey.server.ContainerFilteringStage.apply(ContainerFilteringStage.java:44)
	at org.glassfish.jersey.process.internal.Stages.process(Stages.java:173)
	at org.glassfish.jersey.server.ServerRuntime$1.run(ServerRuntime.java:247)
	at org.glassfish.jersey.internal.Errors$1.call(Errors.java:248)
	at org.glassfish.jersey.internal.Errors$1.call(Errors.java:244)
	at org.glassfish.jersey.internal.Errors.process(Errors.java:292)
	at org.glassfish.jersey.internal.Errors.process(Errors.java:274)
	at org.glassfish.jersey.internal.Errors.process(Errors.java:244)
	at org.glassfish.jersey.process.internal.RequestScope.runInScope(RequestScope.java:265)
	at org.glassfish.jersey.server.ServerRuntime.process(ServerRuntime.java:234)
	at org.glassfish.jersey.server.ApplicationHandler.handle(ApplicationHandler.java:680)
	at org.glassfish.jersey.servlet.WebComponent.serviceImpl(WebComponent.java:394)
	at org.glassfish.jersey.servlet.WebComponent.service(WebComponent.java:346)
	at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:366)
	at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:319)
	at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:205)
	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:799)
	at org.eclipse.jetty.servlet.ServletHandler$ChainEnd.doFilter(ServletHandler.java:1631)
	at io.dropwizard.servlets.ThreadNameFilter.doFilter(ThreadNameFilter.java:35)
	at org.eclipse.jetty.servlet.FilterHolder.doFilter(FilterHolder.java:193)
	at org.eclipse.jetty.servlet.ServletHandler$Chain.doFilter(ServletHandler.java:1601)
	at io.dropwizard.jersey.filter.AllowedMethodsFilter.handle(AllowedMethodsFilter.java:47)
	at io.dropwizard.jersey.filter.AllowedMethodsFilter.doFilter(AllowedMethodsFilter.java:41)
	at org.eclipse.jetty.servlet.FilterHolder.doFilter(FilterHolder.java:193)
	at org.eclipse.jetty.servlet.ServletHandler$Chain.doFilter(ServletHandler.java:1601)
	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:548)
	at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:233)
	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1434)
	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:188)
	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:501)
	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:186)
	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1349)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:127)
	at com.codahale.metrics.jetty9.InstrumentedHandler.handle(InstrumentedHandler.java:313)
	at io.dropwizard.jetty.RoutingHandler.handle(RoutingHandler.java:52)
	at org.eclipse.jetty.server.handler.gzip.GzipHandler.handle(GzipHandler.java:763)
	at org.eclipse.jetty.server.handler.RequestLogHandler.handle(RequestLogHandler.java:54)
	at org.eclipse.jetty.server.handler.StatisticsHandler.handle(StatisticsHandler.java:179)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:127)
	at org.eclipse.jetty.server.Server.handle(Server.java:516)
	at org.eclipse.jetty.server.HttpChannel.lambda$handle$1(HttpChannel.java:400)
	at org.eclipse.jetty.server.HttpChannel.dispatch(HttpChannel.java:645)
	at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:392)
	at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:277)
	at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:311)
	at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:105)
	at org.eclipse.jetty.io.ChannelEndPoint$1.run(ChannelEndPoint.java:104)
	at org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.runTask(EatWhatYouKill.java:338)
	at org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.doProduce(EatWhatYouKill.java:315)
	at org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.tryProduce(EatWhatYouKill.java:173)
	at org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.run(EatWhatYouKill.java:131)
	at org.eclipse.jetty.util.thread.ReservedThreadExecutor$ReservedThread.run(ReservedThreadExecutor.java:409)
	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:883)
	at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1034)
	at java.base/java.lang.Thread.run(Thread.java:834)

Once they login with their email address, they seem to be able to login with the username.
Alternatively, it might be an issue with username case sensitivity during registration, because he was trying to login with NicBailly and the backend shows it as all lowercase and when he switched to camelCase the login worked but that was after he used his email address to successfully register.

I am not sure if I understand what happened. There are 2 Baillys in the system nicbailly and a much older nbailly.
I assume it's the nicbailly account that had problems. He did have a GBIF account NicBailly that he used to log into CLB for the first time. Using his username NicBailly resulted in aboves error? But when he logged in with his email it worked?

I think he lost access to the older account and couldn't recover it.
He hadn't logged into the prod server yet with NicBailly, and that's when he generated the above stack trace in the logs. I had him try again with his email address, and then after seeing that stack trace I asked him to log out and login with the username which he did with nicBailly and that worked. So I think either the GBIF registration IdentityService might be case sensitive or there's a new bug that requires logging in with an email address the first time.

A few remarks on how authentication works:

the GBIF account must be working - try it on the GBIF portal
only the username is accepted, never the email
username is case insensitive
other auth methods supported by GBIF like facebook, google, orcid are not yet supported by us
there is hardly any difference in logging in for the first time than subsequent log ins

As we moved to a new postgres14 server on march 23rd we could have missed first time log ins during a few hours on that day. This would explain the above error. It was inserted into the old still live pg13 db, we then switched 4 hours later and lost the new users during that time period. Any chance this happened then?

I would close the issue as I cannot reproduce it and it might be related to the migration. Please reopen if you think differently


Should move_to_end mark the current file if it is not marked?

It could accept an optional parameter for this behaviour

I this this will be useful, same for move_to_start and move_to_index


Postgres 12 [Travis-CI] [Easy]

Related to: https://github.com/propelorm/Propel2/issues/1541
Travis-ci fails to install postgres 12, it looks like travis has some invalid configuration on their end. It should be possible to manually install via apt. Eg: https://github.com/spryker-shop/suite/blob/master/.travis.yml#L181

Tried to set it manually:
    - php: 7.3
      env: DB=pgsql POSTGRES=12 PGPORT=5433
      addons:
        postgresql: 12
        apt:
          packages:
            - postgresql-12
            - postgresql-client-12
      install:
        - sudo sed -i 's/md5/trust/g' /etc/postgresql/12/main/pg_hba.conf
        - sudo sed -i 's/peer/trust/g' /etc/postgresql/12/main/pg_hba.conf
        - sudo pg_ctlcluster --skip-systemctl-redirect 12 main restart
        - rm -f ~/.phpenv/versions/$(phpenv version-name)/etc/conf.d/xdebug.ini
        - composer install -v
        - composer show

Tests fail on travis (exit 255) but not docker.
No error message, needs more debugging. I suspect it's related to a file-open in the /tmp dir.

The port number changed in the postgres 12 package, and symfony hides the errors for the command when a command is invoked from a test case.
Now the fix should be simple.

What is still left of this ticket?

It was merged


Add userns to run flags

Following the discussions in #20111, #19995 and #17409 we need a
mechinshem to enable skipping userns flag when userns re-mapping was
specified in the daemon level.
As suggest by #mrunalp, a coherent and consistent options is to add the
userns flag to the engine-api.
I've also moved the UT found at:
/docker/docker/runconfig/hostconfig_test.go next to hostconfig.go file
(since they test the specified functionality).

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "19995_skip_user_ns" git@github.com:twistlock/engine-api.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@calavera can you please direct me to the appropriate reviewer for this change?

TBH, I'd rather not bring these types here if they're not used in both sides, client and server. This is very similar to https://github.com/docker/engine-api/pull/56.

@calavera your comments is about DefaultDaemonNetworkMode not about the UsernsMode right?
Also, what about the unit test refactor, are you ok with it?

@calavera I've cleanup this change and removed all unrelated code.
The new UsernsMode should be used both in client and server.

@calavera @estesp can you please review this change?

LGTM - this is in support of the new flag to allow a privileged mode run when user namespaces are enabled. I believe it is used on both client and server as the flag parsing requires these functions.

LGTM


distribute release, 403, invalid token, ValidationFailure

Description
I'm getting a 403 invalid token (ValidationFailure) on distribute release.
I think it could be because I'm using my personal token but I'm distributing a project from a team I'm included as an admin.
Manually uploading it also reports invalid token but it does got uploaded and I was able to distribute it manually.
Repro Steps
distribute release on a project owned by a team you're included using your personal token
appcenter login --token <your_personal_token>
appcenter distribute release --debug --app <team>/<project> --file <binary_file> --group <group_name>
Expected behavior
My build will get uploaded and distributed to selected group
Details
version 2.7.4
macOS BigSur 11.2.3
/bin/bash
node v15.0.1, npm 7.0.3
Logs
$ appcenter distribute release --debug --app alfred-at-mint-10b6/CVFR-mPOS-UAT --file "CVFR-mPos-Uat-1.0.0 (1).ipa" --group "Ian"
Using appcenter-cli version: 2.7.4
| Loading prerequisites...logFilter, request: {
  "rawResponse": false,
  "queryString": {},
  "url": "https://api.appcenter.ms/v0.1/apps/alfred-at-mint-10b6/CVFR-mPOS-UAT/distribution_groups/Ian/members",
  "method": "GET",
  "headers": {
    "Content-Type": "application/json; charset=utf-8",
    "user-agent": "appcenterCli/2.7.4 NodeJS/v15.0.1 darwin/20.3.0",
    "x-api-token": "08b9bbe7c02a6bada22eb2320f41b5d5d736f54d"
  },
  "body": null
}
\ Loading prerequisites...Response status code: 200
Body: [{"id":"42fc4413-7f58-4f09-8551-4e40786fd330","display_name":"Ian Lopez","email":"ian.lopez@mintpayments.com","name":"ian.lopez-mintpayments.com","avatar_url":null,"can_change_password":false,"created_at":"2021-03-01T01:45:01.000Z","origin":"appcenter"}]
logFilter, request: {
  "rawResponse": false,
  "queryString": {},
  "url": "https://api.appcenter.ms/v0.1/apps/alfred-at-mint-10b6/CVFR-mPOS-UAT/distribution_groups/Ian",
  "method": "GET",
  "headers": {
    "Content-Type": "application/json; charset=utf-8",
    "user-agent": "appcenterCli/2.7.4 NodeJS/v15.0.1 darwin/20.3.0",
    "x-api-token": "08b9bbe7c02a6bada22eb2320f41b5d5d736f54d"
  },
  "body": null
}
Response status code: 200
Body: {"id":"6b328015-1faf-459d-8af6-d3c5b2c24da9","name":"Ian","origin":"appcenter","display_name":"Ian","is_public":false}
logFilter, request: {
  "rawResponse": false,
  "queryString": {},
  "url": "https://api.appcenter.ms/v0.1/apps/alfred-at-mint-10b6/CVFR-mPOS-UAT/releases/3/groups",
  "method": "POST",
  "headers": {
    "Content-Type": "application/json; charset=utf-8",
    "user-agent": "appcenterCli/2.7.4 NodeJS/v15.0.1 darwin/20.3.0",
    "x-api-token": "08b9bbe7c02a6bada22eb2320f41b5d5d736f54d"
  },
  "body": "{\"id\":\"6b328015-1faf-459d-8af6-d3c5b2c24da9\",\"mandatory_update\":false,\"notify_testers\":true}"
}
Response status code: 403
Body: {"message":"invalid token","code":"ValidationFailure","statusCode":403}
Command Failure at Error: invalid token
    at /Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/dist/util/apis/generated/operations/releasesOperations.js:1791:19
    at retryCallback (/Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/systemErrorRetryPolicyFilter.js:89:9)
    at retryCallback (/Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/exponentialRetryPolicyFilter.js:140:9)
    at /Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/rpRegistrationFilter.js:59:14
    at handleRedirect (/Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/redirectFilter.js:39:9)
    at /Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/formDataFilter.js:23:14
    at /Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/filters/logFilter.js:21:14
    at Request._callback (/Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/ms-rest/lib/requestPipeline.js:125:16)
    at Request.self.callback (/Users/ian/.nvm/versions/node/v15.0.1/lib/node_modules/appcenter-cli/node_modules/request/request.js:185:22)
    at Request.emit (node:events:327:20)
Error: Command 'distribute release --debug --app alfred-at-mint-10b6/CVFR-mPOS-UAT --file CVFR-mPos-Uat-1.0.0 (1).ipa --group Ian' failed with exception "invalid token"

Hi @jolen10! It is most probably a duplicate of https://github.com/microsoft/appcenter-cli/issues/1332
Feel free to reply if the issue is not resolved for you.


Add NextDNS

Add NextDNS
Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/issues/523

Installation failed suggesting that I need to install device software?

@KevinLaMS Could you please share the screenshot or details error message? I just tried installing, it went successful without throwing any error.

Not a malware confirmation.


Update thor gem dependency

Update thor to a more recent version: https://github.com/erikhuda/thor/blob/master/CHANGELOG.md

Thanks


AMI 2015-12-17.b not available in eu-central-1

The current version 2015-12-17.b is not available in region eu-central-1 yet and the README (https://github.com/weaveworks/integrations/blob/master/aws/ecs/README.md) points to the wrong AMI id (duplicate of eu-west-1).

Thanks for reporting this Fabian. I am on vacation until Jan 7, but I will
look into this as soon as I am back.
On Saturday, January 2, 2016, Fabian Fülling<EMAIL_ADDRESS>wrote:

The current version 2015-12-17.b is not available in region eu-central-1
yet and the README (
https://github.com/weaveworks/integrations/blob/master/aws/ecs/README.md)
points to the wrong AMI id (duplicate of eu-west-1).
—
Reply to this email directly or view it on GitHub
https://github.com/weaveworks/integrations/issues/54.

Great, thanks! Enjoy your vacations


build: Remove nest-asyncio from core dependencies

As the databinder module is deprecated and scheduled for removal (c.f. https://github.com/ssl-hep/ServiceX_frontend/issues/355#issuecomment-2030460171), remove nest-asyncio from the required core dependencies.
Add a 'databinder' extra that includes the nest-asyncio dependency, as the databinder module still exists and should be kept functional until it is removed or revised. (c.f. https://github.com/ssl-hep/ServiceX_frontend/issues/355#issuecomment-2032035624)

Amends PR https://github.com/ssl-hep/ServiceX_frontend/pull/349.

cc @kyungeonchoi, @BenGalewsky, @gordonwatts for review.

Thanks for the review and merge, Ben!


Releasing v4.18.0 to docs site

Updated the docs site for v4.18.0

@anselmbradford 😆 took me a minute to get that


How to exclude files gerneated from a particular child compiler from __WB_MANIFEST

I wrote a webpack plugin which will generate js files for SSR in the child compiler. But I do not want workbox-webpack-plugin to precache these js files generated from this child compiler. The actual behaviour is these js files get injected into self.__WB_MANIFEST. I try to avoid the injection by not specifying self.__WB_MANIFEST in the src. But it shows ERROR in Can't find self.__WB_MANIFEST in your SW source.
you may suggest I could use regex to filter the chunks that I do not want. This is not scalable. But I want to filter a set of js files genreated from a child compiler.
The second question is why it is a must to have self.__WB_MANIFEST in src? If I understand correctly, I can use this to do a on demand precache

I don't think you need to use the InjectManifest plugin at all. Or the GenerateSW one.
Just write your service worker with the logic you'd like, using the Workbox ES module imports as described here, and add your unbundled service worker as a new entry that will be compiled by webpack alongside your other entries.
The plugins are useful when you want to either generate an entire service worker automatically (GenerateSW) or when you want to use workbox-precaching to cache a bunch of assets in your compilation (InjectManifest). It sounds like you want more control than that, so just don't use them.


Log HTTP "499 client abort" in jabba logs and stop execution if user agent closed TCP connection from their end.

Is your feature request related to a problem? Please describe.
Currently when a user  agent closes the connection, the server does not log a 499 and defaults to whatever error handling it has in place.
Describe the solution you'd like
only when a user agent terminates the socket, and the server receives a FIN from user agent, before it finishes, should the server log a 499.

implemented in b164


Result queue clears slowly

With save_limit set to 0, the result queue is clearing ever more slowly with each batch of tasks added. Setting a fixed limit for save_limit is not feasible in my case (and would still be high enough to trigger the slowing down).
My test case adds around 1000 tasks (which process very quickly), then waits for the results using result_group.
With an empty tasks table, the result queue is emptying in a somewhat acceptable pace. However, the delay between the execution of the last task and the return of result_group is already long and noticeable (many seconds). Repeating this a few times, the delay keeps increasing.
A real case will involve tens of thousands of tasks and the delay will be in the sky.
Do you have a hunch where the issue might hide, where I might look further?

I can confirm the issue does not occur with 1.6.2, or at least is not as pronounced. The delay starts at ca. 20 seconds, and in subsequent runs increases only slightly to 20 - 27 seconds.
Both 1.7.4 and 1.7.3 are affected, where the delay starts at 30 seconds and increases around 30 seconds each run.
I was not able to test 1.7.2, as it hit a bug, related to prometheus if I saw correctly.

Uhm it may be somehow related to this discussion i opened a few weeks ago https://github.com/django-q2/django-q2/discussions/241 .


Revert the commit which disabled FSharpWebSite to unblock the mirror

b28002a9e530f4303ba216ee2dd670693b53e214

Reverted with https://github.com/aspnet/Mvc/commit/04f74edd8537bf524a9840b5f81cbfc19996f6e7


mysql80 package failing to start

Database change to mysql80 in magento2 template is failing to start.
I've changed package = pkgs.mariadb_106 to package = pkgs.mysql80;
first error I encountered was unknown variable 'query_cache_limit=1M'. I've fixed that by removing query_cache_size and query_cache_limit variables which have been removed in removed in MySQL 8.0.3.
# DATABASE
    services.mysql = {
        enable = true;
        #package = pkgs.mariadb_106;
	package = pkgs.mysql80;
        settings = {
            mysqld = {
                port = config.env.DEVENV_DB_PORT;
                innodb_buffer_pool_size = "2G";
                table_open_cache = "2048";
                sort_buffer_size = "8M";
                join_buffer_size = "8M";
                #query_cache_size = "256M";
                #query_cache_limit = "2M";
            };
        };
        initialDatabases = [{ name = "${config.env.DEVENV_DB_NAME}"; }];
        ensureUsers = [
            {
                name = "${config.env.DEVENV_DB_USER}";
                password = "${config.env.DEVENV_DB_PASS}";
                ensurePermissions = { "${config.env.DEVENV_DB_NAME}.*" = "ALL PRIVILEGES"; };
            }
        ];
    };

second error I see now Table 'mysql.plugin' doesn't exist

Did you boot the mariadb database before switching to mysql?
It can be that the state the mariadb left is not compatible with mysql8.
Ideally you make sure the environment is stopped, then remove the .devenv/state/mysql directory, and finally start it again


Creating an endpoint through API: Failure parse :: first path segment in URL cannot contain colon"

Description
Creating a new endpoint through the API creates the endpoint successfully but it cannot be accessed through the UI as it fails with the following error "Failure parse <IP_ADDRESS>:2376: first path segment in URL cannot contain colon"
Steps to reproduce the issue:

Create a new endpoint through the API

http POST long-portainer-s002/api/endpoints \
"Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MSwidXNlcm5hbWUiOiJhZG1pbiIsInJvbGUiOjEsImV4cCI6MTUyMjM1MjEyNn0.a3K4BIVSxzODyQWcmLV8Ro_SS4JnMWP0rKZzKSOF13M" \
Name="remote-docker" URL="<IP_ADDRESS>:2376"

----

HTTP/1.1 200 OK
Connection: keep-alive
Content-Length: 10
Content-Type: text/plain; charset=utf-8
Date: Thu, 29 Mar 2018 15:21:31 GMT
X-Content-Type-Options: nosniff
X-Frame-Options: DENY

{
    "Id": 63
}

Check config through the API

http GET long-portainer-s002/api/endpoints/63 "Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MSwidXNlcm5hbWUiOiJhZG1pbiIsInJvbGUiOjEsImV4cCI6MTUyMjM1MjEyNn0.a3K4BIVSxzODyQWcmLV8Ro_SS4JnMWP0rKZzKSOF13M"
HTTP/1.1 200 OK
Connection: keep-alive
Content-Length: 179
Content-Type: text/plain; charset=utf-8
Date: Thu, 29 Mar 2018 15:26:09 GMT
X-Content-Type-Options: nosniff
X-Frame-Options: DENY

{
    "AuthorizedTeams": [],
    "AuthorizedUsers": [],
    "Extensions": [],
    "Id": 63,
    "Name": "remote-docker",
    "PublicURL": "",
    "TLSConfig": {
        "TLS": false,
        "TLSSkipVerify": false
    },
    "URL": "<IP_ADDRESS>:2376"
}

Go to UI and select the newly added endpoint.

Go to Portainer Settings > Endpoints , select the newly added endpoint (here remote-docker) and just click the Update endpoint button

Endpoint will be successfully updated.

Now select as active endpoint the newly updated endpoint one and this time it works.

Why is this a problem
This is a bug as we currently attach the endpoints programmatically as we scale out our infrastructure and we need to "update" those endpoints before using them. It seems that the UI correctly splits the hostname/port parts of the address but the API doesn't
In our tests, we can recreate this even if we provide the PublicURL explicitly. It just adds ";" at the end of the hostname/ip address.
Technical details:

Portainer version: 1.16.4
Target Docker version (the host/cluster you manage):
Portainer running on:  Docker version 17.09.0-ce (ecs cluster)
Managing a host with: Docker version 18.03.0-ce
Platform (windows/linux): linux
Command used to start Portainer: docker run -p 8080:8080  --mount type=bind,source=/efs/portainer,target=/data  -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer
Browser: Version 65.0.3325.181 (Official Build) (64-bit)

Hi @mihneaspirescu try using URL="tcp://<IP_ADDRESS>:2376"

@mihneaspirescu right, you need to specify the tcp:// protocol in the endpoint URL.


Add BDSO as a imported component to PCL

BDSO should produce an artefact that includes base + custom imports (NS forest markers)
all other imports should come from PCL (CL, NCBITaxon, UBERON)
Blocked until BDSO can produce artefact for PCL

base-ext.owl = base.owl + gene ontologies releaed

Fixed with this release https://github.com/obophenotype/provisional_cell_ontology/releases/tag/v2021-12-03


fix  compilation of profiles via package offsets

fix  compilation of profiles via package offsets

I am moving this back to "In progress" (Project dashboard status) pending approval from one of the CODEOWNERS.
See https://github.com/finos/legend/wiki/Release-Management


#911 updated format function in derive_var_disposition_status to have…

… a meaningful par i.e. change 'x' to 'status'
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Code is formatted according to the tidyverse style guide
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue so that it closes after successful merging.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done!  Much love to your accomplishment!

That is a good catch.  @teckla-gsk do you mind updating the adsl.Rmd vignette as well?  Are you able to render the packagedown site on Desktop?  If not I can pull, render and check the site after you update it.

Can you put Closes #911 at the top of the tome that is our PR template.  Supposedly it will auto-close the issue once we merge into main.

@bms63  @kabis-ops should we update the vignette? The function that was updated is the format_eoxxstt_default() while the vignette is pointing users to create a custom function if the default is not used?

@bms63 @kabis-ops should we update the vignette? The function that was updated is the format_eoxxstt_default() while the vignette is pointing users to create a custom function if the default is not used?

Well then...I guess not!  :)  Can you update the top of the PR template with Closes #911.  @kabis-ops looks all good to me!

ok Vignette updated to replace the 'x' arguments to meaningful vars

Hi Teckla,
Some of the merge conflicts still came through.

@bms63 on my side I see the checks as incomplete

@kabis-ops  Hey Samia.  Are you able to re-review?  I changed one piece of the adsl vigneete to use var_input rather the name of the variable.  Let me know if you agree.  everything else was Teckla's update


Added all events and cleaned some code

Thanks a lot for your contribution! I'll merge it now. If further cleanup needs to be done, we can do it later.


[Vue warn]: Property or method "$_apollo" is not defined on the instance

Hi, i still getting this error for unknown reason:
[Vue warn]: Property or method "$_apollo" is not defined on the instance but referenced during render. Make sure that this property is reactive, either in the data option, or for class-based components, by initializing the property. See: https://vuejs.org/v2/guide/reactivity.html#Declaring-Reactive-Properties.

Here is my config:
import Vue from 'vue'
import VueApollo from 'vue-apollo'
import { ApolloClient } from 'apollo-client'
import { createHttpLink } from 'apollo-link-http'
import { setContext } from 'apollo-link-context'
import { InMemoryCache } from 'apollo-cache-inmemory'
import { LocalStorage } from 'ttl-localstorage'

const httpLink = createHttpLink({
  uri: 'http://example.test/graphql',
  credentials: 'include'
})

const authLink = setContext(async (_, { headers }) => {
  // get the authentication token from local storage if it exists
  let token = await LocalStorage.get('token')
  // return the headers to the context so httpLink can read them

  return {
    headers: {
      ...headers,
      Cookie: 'XDEBUG_SESSION=phpstorm',
      Authorization: token,
      'X-Requested-With': 'XMLHttpRequest',
      'Accept': 'application/json',
      'Content-Type': 'application/json'
    }
  }
})

const apolloClient = new ApolloClient({
  link: authLink.concat(httpLink),
  cache: new InMemoryCache()
})

const apolloProvider = new VueApollo({
  defaultClient: apolloClient
})
Vue.use(VueApollo)

window.vue = new Vue({
  ...App,
  router,
  store,
  apolloProvider
}).$mount('#app')

Any ideas what could be wrong? thx

Please provide a runnable reproduction so I can look into this issue, thanks!


Current method of timers for switches/encoders eats up >1.7kb of 2.5kb RAM budget

https://github.com/mastersta/407sim/blob/8a4eb561b845796872119923b619a594567dcb6c/407sim.ino#L226

Moving to inputs managed by AM software. Issue no longer relevant.


window.top is readonly - prevents us from isolating webviews

Looks like we can't actually completely isolate webviews while sharing React and papi. Either we must disallow same origin policy which would restrict us to communication via message passing only and nothing more, or we must completely distrust the frontend and do what we can to mitigate issues. Some options:

We could change our strategy with the offline-only mode to set content-src to 'self' only if we're offline. That way, even if some bad actor did get ahold of fetch or connected to some website with an image url or something, we would cut it off in offline mode. We lose security online, but at least we can maintain it offline.

Alternatively, we could set the content-src to 'self' always and just make all internet calls have to pass through the internet service. I'm not sure if that buys us anything that approach 1 doesn't.

Remove allow-same-origin from the iframe sandboxing. Then the webview can only communicate to the extension through message passing, which is what VSCode does. But we lose the ability to share React and the papi. Seems like it could get unwieldy.

Related: #85, #89

FYI on item 1 above, you can't robustly say you are offline, (online - yes, but not offline). Therefore there is always some time during transition where it's actually offline but we haven't detected it yet so fetch would still attempt fetching.

I was talking about a browser going offline because it loses access to the internet. However, I think this is referring to a user changing a setting to say "don't use the internet at all", right?

Some testing on master branch today. In evil.js, adding the following to the tryImports() method:
  try {
    // This should NOT work because `window` and `top` are excluded from the sandbox settings.
    window.top.fetch('https://bible-api.com/1%20thessalonians+5:17');
    logger.error('Evil: top.fetch is working.');
  } catch (e) {
    logger.warn(`Evil: Error on top.fetch! ${e}`);
  }

results in this error: Evil: Error on top.fetch! ReferenceError: window is not defined.

Opps, @tjcouch-sil pointed out that evil.js is only running in node and not in a webview.

Glenn and Matt's discussion about security: https://docs.google.com/document/d/1BeqVqgrtQbVp7HYYdi0mGEQG8jU81qpSdFQH3fPnw8k/edit?pli=1

Some thoughts on the Chrome extension stuff:
To my understanding, Electron supports a subset of Chrome extensions, and they must be manually downloaded and loaded in via the main.ts code to my understanding (I don't believe Electron tries to load Chrome extensions from your installed Chrome extensions or anything like that). Electron React Boilerplate starts us off with electron-devtools-installer and React DevTools Chrome extension by default if in development mode.

We decided we will throw out number 3 as it is likely only beneficial over the others in preventing malicious webviews from removing sandboxing, which does not have as significant consequences as content security policy. See the sandbox attribute for more info.
Let's make at least a start of a general security items list to present to Glenn to figure out how to move forward with security concerns. #277

We concluded that we will make an Electron protocol handler (option 4) that will pass whatever we want when online and do one of two things when offline:

Call the extension's offline asset backup function to get an offline version of the requested url
Reject the connection attempt

This will not work naturally if we make a web version, but we can go with option 1 in a web scenario. However, we don't know how we would make an offline backup function work in the online setting with arbitrary urls due to security restrictions on protocol handlers and service workers. Hopefully the number of extensions that really need that as an option will be very minimal.

Created #473 to implement the design

FYI all the following are not configurable in the renderer as determined by the following code entered into the console:
Object.keys(window).filter(key => !Object.getOwnPropertyDescriptor(window, key).configurable)

>  ['window', 'document', 'location', 'top', '__electronLog', 'electronAPI']


Don't try to encode TS token when null and DL encoding

Avoid NPE when encoding response with no timestamp
token and definate length encoding is used.

You might have to tell me what you're trying to do here. The impression I get is you have a situation where you need to use DLSequence(). Is that correct?

Thanks for the patch, merged with minor changes.


[SUGGESTION] Use sidebar theme-switching like cMenu

Hi, cMenu recently released an update which included a modal that folds out of the sidebar with its options. It is very convenient since you click it and the options are in your reach, no need to reach the top of the screen.
Maybe this could be implemented for theme-picker? Maybe use that roll up light/dark mode switching as well, into one status bar icon?
Thanks!

I like this because the modal that pops up now is still fairly large and hides a good portion of the UI – not as bad as the default settings modal though. It would be nice to have the theme picker as unobtrusive as possible.
One thing I like about the existing modal, though, is the ability to handle everything from the keyboard. If you have a lot of themes installed and start typing, it'll narrow down your search so you can quickly select the one you're looking for – nothing's stopping that from happening for a custom modal, but I think it's more work than I'm willing to do right now.
When I get the opportunity, I'll certainly look into it and see what the experience is like, though! And thanks for the suggestion!


Improve off-scale data hunches

I don't like the current off-scale implementation of data hunches. They don't look good and aren't all that effective.

Here's a different idea.

Use a sketchy arrow and point to a label that corresponds to the new data value. The brush can be of constant shape or height, so could be a glyph instead of a customly rendered arrow.
The above design shows two variations for more than one hunch; one with two arrows, one with one arrow, and multiple numbers shown in a list. Multiple numbers would make it easier to implement and probably more scalable.
Use a hand-written font for this, and for all other occurrences where we print text from a data hunch. For example https://www.1001fonts.com/gooddog-plain-font.html

I think the arrows should point sideways instead of downwards. And the text should be next to it, not under? The way this is right now it looks like 20, 19, and 40 are at these positions where the arrows end.

Yes, this is good. Let's try to give examples that are in sequential order, but no need to change anything with the implementation.


set wallpaper for xfce4 with xfce4-set-wallpaper

xfce4-set-wallpaper can set wallpaper to every monitor that contains image-path/last-image. It is available if you have "xapps", "xapps" or "xapps-common" installed.  Its project URL is https://github.com/linuxmint/xapp

Thanks for the PR. Tried it on xfce mint - works fine. Merged.


rainbow disables itself on Windows when Ruby version >= 2

Rainbow disables itself if it can't use win32console.  Beginning with Ruby 2 win32console is no longer needed and fails to load.  Rainbow should use native ANSI color support on Windows when Ruby version >= 2.

Didn't this get resolved by #23?

Yup, good catch.


Fixed 'substitution failure' related errors with _CalculateHash(arg1) f…

…unction.
Added missed function for enable_if  condition.
A few other compilation erros fixed to include Echo test code in ThunderNanoServices.

Found some issues during testing. So closing this PR.


Leetcode Question Number 4

Leetcode Question Number 4 soultion needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Hi, could this please be assigned to me


CIV-7839 evidence upload macro

JIRA link
https://tools.hmcts.net/jira/browse/CIV-7839
Change description
Framework "Upload documents"
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

we're missing tests on this PR, unit and the start of some functional?


Fix the warning and error for weather vc in sb

Fix the warning and error for weather vc in sb

Thanks @passerbyloo 👍👍👍

thx😆


keyword "ref"

Not sure if this keyword is supported, neither how it should be implemented.

This keyword is only implemented when it appears in the arguments of a function.
In C# 7.0 there can be local variables with this keyword. This feature is not supported yet.

Merged with issue #27.


FileStorage occasionally get corrupted. No way to recover

We've found using FileStorage that occasionally the files on disk get corrupted (i.e. file is truncated or otherwise not valid JSON).  Not quite sure what causes this condition. Perhaps when node process is prematurely stopped?
More importantly, there doesn't appear to be a graceful way to recover from this condition other than manually deleting the storage files from disk.   One can catch the failed reads, but there's no obvious way to clear the bad file.
Any suggestions?

Bigger picture, the Memory + File solutions we have now are just not rich enough to build a real bot atop. If nothing else, you can't build a scalable bot using either as state is affinitized to the local machine. The best choice - really the only choice - is to move to a more mature state solution. CosmosDB is where we anticipate most people storing their state.
We could certainly expose a hook in the FileStorage system that allows your code to decide what to do if things get corrupted.  This could even be a state flag you pass in. It would certainly also be good if the storage provider didn't corrupt itself...

Completely agree.  I created a Redis middleware plugin (which we show in one of our sample apps), but we need a quick way for users to get off the ground w/o having to create a storage account, provide keys, etc.   We default to FileStorage for that case.
Would be awesome such a flag were added so we can recover from corrupted data.  Currently our users get stuck and we have to tell them to manually delete the files

I can take a look at fixing this when I port the FileStorage provider over to the new M2 branch.  I've done just about everything but that actually.

Thanks!
On Feb 28, 2018 11:09 AM, "Steven Ickman"<EMAIL_ADDRESS>wrote:

I just reworked the FileStorage provider in my m2 branch. Now it simply
logs a warning if it can't read or parse a corrupted file for some reason.
Please let me know if you see any of these warnings. Not sure why things
would be getting corrupted.
https://github.com/Microsoft/botbuilder-js/blob/m2/
libraries/botbuilder/src/fileStorage.ts#L160
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/botbuilder-js/issues/104#issuecomment-369348628,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGOd5k69oYhiCfXE1Kcp5F0Ko-H0Ffzrks5tZaR_gaJpZM4SVudl
.

Thanks, Steve.  So the subsequent write will replace the corrupted file?
From: Steven Ickman<EMAIL_ADDRESS>Sent: Wednesday, February 28, 2018 11:10 AM
To: Microsoft/botbuilder-js<EMAIL_ADDRESS>Cc: Lars Liden<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/botbuilder-js] FileStorage occasionally get corrupted. No way to recover (#104)
I just reworked the FileStorage provider in my m2 branch. Now it simply logs a warning if it can't read or parse a corrupted file for some reason. Please let me know if you see any of these warnings. Not sure why things would be getting corrupted.
https://github.com/Microsoft/botbuilder-js/blob/m2/libraries/botbuilder/src/fileStorage.ts#L160https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-js%2Fblob%2Fm2%2Flibraries%2Fbotbuilder%2Fsrc%2FfileStorage.ts%23L160&data=04|01|Lars.Liden%40microsoft.com|2272e87133b64bdfacbe08d57ededa4f|72f988bf86f141af91ab2d7cd011db47|1|0|636554417990210122|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=WtBcjdZUrpunNI04B0phEm5P3mGSQtad5iJj3j0VS6g%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-js%2Fissues%2F104%23issuecomment-369348628&data=04|01|Lars.Liden%40microsoft.com|2272e87133b64bdfacbe08d57ededa4f|72f988bf86f141af91ab2d7cd011db47|1|0|636554417990210122|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=hhjgNSlgkkGCTD04lZppKozPdqREHj0wDxKot%2BIvUeA%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGOd5k69oYhiCfXE1Kcp5F0Ko-H0Ffzrks5tZaR_gaJpZM4SVudl&data=04|01|Lars.Liden%40microsoft.com|2272e87133b64bdfacbe08d57ededa4f|72f988bf86f141af91ab2d7cd011db47|1|0|636554417990210122|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=KbgZn68R3zy0Sn%2F2g3v9CMudLSowL8LBB%2F%2BeqixZUWo%3D&reserved=0.

yes... Now if a read error occurs we log it as a warning but then initialize the state to be empty. The next save should overwrite the corrupted file.


Make format.js work on Windows

The fix uses execSync to get the path output from npm bin rather than embedding the Bash-specific $(npm bin) in the command. I tested it successfully on Windows 10 and Ubuntu.

Coverage remained the same at 99.257% when pulling f62d17520cce921cd555e8a9e37107e46decb351 on richgieg:fix-format-on-windows into b9bdae41d3b0b331f62bc24825f45f350e7d71ac on pelotom:master.


Hitting back on signature page doesn't submit signature form [finishes #158447429]

Description
Our forms are set to save when hitting "Back". While this is great for many screens, it's not intended behavior for the final signature page. This PR allows the SignedCertification component to specify that it doesn't want the form saved when the user hits "Back"
Code Review Verification Steps

[ ] End to end tests pass (make e2e_test).
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
[ ] Any migrations/schema changes also update the diagram in docs/schema/dp3.sqs
[ ] There are no aXe warnings for UI.
[ ] This works in IE.
Any new client dependencies (Google Analytics, hosted libraries, CDNs, etc) have been:

[ ] Communicated to @willowbl00
[ ] Added to the list of network dependencies

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

If I hit back after entering my name, it clears my name. I'm not sure if that is a feature or a bug ;) Can you check with Jenny?


[Tooltip] Continously shows/hides on certain container structure

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Tooltip should show only once while being moused over
What is the current behavior?
When a tool-tip is positioned like the following it keeps show and hiding as you mouse over.
|-- div- fixed bottom right
        |-- div- relative
                |-- div- absolute
                        |-- a md-mini-fab mdTooltip

What are the steps to reproduce?
just mouse over the md-mini-fab
http://plnkr.co/edit/0qDDaLCMPeMYgElQeLhg
What is the use-case or motivation for changing an existing behavior?
N/A
Which versions of Angular, Material, OS, browsers are affected?
Failed on Chrome(latest), IE(Latest) and Edge(latest). ng-version 2.4.7
Is there anything else we should know?
No

+1

Same here

Tried this out with the latest version and couldn't reproduce, so it's likely been fixed
https://stackblitz.com/edit/angular-aswf7q?file=src/app/tooltip-auto-hide-example.html