text:
dht.get always returns null for mutable content

Trying to get mutable content examples working, but dht.get always returns null without any error.
Setting content code:
var ed = require('ed25519-supercop');
var keypair = ed.createKeyPair(ed.createSeed());

var value = new Buffer(100).fill('helloworld'); // the payload you want to send
var opts = {
    k: keypair.publicKey,
    seq: 0,
    v: value,
    sign: function (buf) {
        return ed.sign(buf, keypair.publicKey, keypair.secretKey)
    }
};

var DHT = require('bittorrent-dht');
var dht = new DHT();

dht.listen(function () {
    console.log('dht is listening');
});

dht.on('ready', function() {
    console.log('dht is ready');
    dht.put(opts, function (err, hash, nodesAccepted) {
        console.error('error=', err);
        console.log('hash=', hash.toString('hex'), 'nodes accepted:', nodesAccepted);
    })
});

Then output is something like this:
dht is listening
dht is ready
error= null
hash= 87283a9f5e7bb584b367a95fa01577a8e4f0a730 nodes accepted: 14

Then I leave this script launched and use the same hash to launch the second script:
var DHT = require('bittorrent-dht');
var dht = new DHT();

dht.listen(function() {
    console.log('dht is listening:');
});

dht.on('ready', function () {
    console.log('dht is ready');
    dht.get('87283a9f5e7bb584b367a95fa01577a8e4f0a730', function (err, res) {
        if (err) {
            console.error('failed to get reply:', err);
        } else {
            console.log('get reply:', res);
        }
    });
});

The response is always null but without any error:
dht is listening:
dht is ready
get reply: null

I also tried many variations with exact UDP ports (which were forwarded), also with setInterval where put and get executed periodically in the both scripts, tried to launch the second script on another computer with different external IP, but dht.get reply was always null.

By the way I used node v4.4.4 on 64-bit debian linux for the "put" script and the "get" was launched on the same machine first and then on 64-bit Windows with node v4.4.3. The results were always the same.

Sorry I didn't checked closed issues before posting a new one:
The problem was resolve in the https://github.com/feross/bittorrent-dht/issues/104
The answer is to use on the "get" side:
var ed = require('ed25519-supercop');
var dht = new DHT({verify: ed.verify});

Probabbly it's a good idea to update readme in order to reflect this important point. Also it would be great if some error is generated in similar case.

@mihv you wanna send a PR that adds that? :)

@mihv Yes, PR welcome!

@mafintosh, feross Guys, sorry for the delay. I sligtly updated the readme in order to notice this very important thing and created a pull request.
By the way, thank you for writting such great software, your work is invaluable!

authors: feross mafintosh mihv

repo: feross/bittorrent-dht


text:
Update README.md

Fix server's example code

thanks!

source: gharchive/pull-request

authors: feross harshjv

repo: feross/bittorrent-tracker

license: mit

license_type: permissive

license_source: bigquery


text:
Merge in jscissr/net-chrome-apps

Some time ago I made jscissr/net-chrome-apps without knowing that you already did exactly the same thing... until jscissr/net-chrome-apps#1.
Because there is really no big difference (except that you have tests), I decided to merge in the parts of my version which are better, and I also made some other improvements in the process.
This patch results in higher compatibility to the original node module. For example the error codes of Chrome are now mapped to the UV/Node codes. Servers only emit close when they are closed, not starting up and no clients are connected anymore.
More edge cases are caught; there is now a check that the id is still the same as before in every callback (unexpected things may happen between an async call).
Tests now run directly in the Chrome App, the tape output is simply piped through a socket to the console.
I replaced ipaddr.js with a (complicated) regex, I don't think there should be another library in the deps just for some rarely used function for compatibility.

Thanks for the excellent PR!
Since this is a huge change, existing programs that rely on the current behavior could break. So, out of an abundance of caution, I released this as a major version, v3.0.0.
I also added you as a GitHub collaborator. Thanks again!

authors: feross jscissr

repo: feross/chrome-net


text:
Add eslint-plugin-node to Usage documentation

Update documentation to add eslint-plugin-node to the Usage documentation, as node is specified as a plugin in eslintrc.json but is missing from the documentation.

Thanks!

authors: feross jleft

repo: feross/eslint-config-standard


text:
ensure video is paused before hiding window

sendSync is throwing an error currently as it does not exist.
This PR adds logic to signal the video to stop playing and only hide after we're sure the video is paused.
Feedback appreciated!

LGTM!

I still can't help but feel that there's gotta be a cleaner way than this, but at least it works now :)

authors: Flet feross

repo: feross/webtorrent-app


text:
WebTorrent-Desktop Cannot Show SoftSub :(

What version of WebTorrent Desktop?
Version 0.17.2 (0.97.2) (64-bit)
What operating system and version?
Windows 10 Home (Versión 1607)
What did you do?
I tried to play a movie with softsub
What did you expect to happen?
Play the movie with softsub
What actually happened?
The movie play without subtitles

Closing as duplicate of #526.

source: gharchive/issue

authors: MarcosxSenpai demoneaux

repo: feross/webtorrent-desktop


text:
Move error definitions to errors.js

Moved most of the error definitions to dedicated errors.js file, which was hanging unused until now
Created separate classes for different types of errors to increase verbosity and get rid of vague errors
Fixed a typo in error name inside playback-controller.js possibly resulting in a TypeError

LGTM. @feross  ?

LGTM

authors: Goldob dcposch feross


text:
Add impl Clone for EncoderState

This allows to efficiently split the encoding of common-then-diverging streams.

Hi @3442, I've merged this into https://github.com/jamesmunns/cobs.rs, and this change is now in the cobs v0.2.2 release.

license_source: github-api


text:
Update .stderr files

With compiler 1.60.0:
$ rustc --version
rustc 1.60.0 (7737e0b5c 2022-04-04)
The contents of teh .stderr files is slightly different so the .stderr
files needed updating using TRYBUILD=overwrite cargo test.
Without this change I was getting errors like the following, with the
change the tests pass.
  test tests/ui/has-arguments.rs ... mismatch

  EXPECTED:
  ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
  error: this attribute takes no arguments

    = help: use `#[contracts]`

   --> $DIR/has-arguments.rs:3:1
    |
  3 | #[contracts(a, b)]
    | ^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in the attribute macro `contracts` (in Nightly builds, run with -Z macro-backtrace for more info)
  ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈

  ACTUAL OUTPUT:
  ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
  error: this attribute takes no arguments

           = help: use `#[contracts]`

   --> tests/ui/has-arguments.rs:3:1
    |
  3 | #[contracts(a, b)]
    | ^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in the attribute macro `contracts` (in Nightly builds, run with -Z macro-backtrace for more info)
  ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
  note: If the actual output is the correct output you can bless it by rerunning
        your test with the environment variable TRYBUILD=overwrite

  test tests/ui/has-expr-argument.rs ... mismatch

Hmmm, I did use TRYBUILD=overwrite cargo test. I'll try again and if unsuccessful will manually change it. I'm on an Arch Linux system running rust 1.60.0.

One other thing that was odd, I couldn't use Visual Studio Code in that repo, it started but "hung" with a blank window, is that expected?

I don't see any issue with vs code 1.67.1 and rust-analyzer 0.3.1051 using revision 60a36e1 . it may be an issue with your vs code setup; maybe try other Rust projects and see if they run into the same issue.

I've forced push the change after manually changing the path to $DIR/... not sure why it changes for me :(
Also, I booted my desktop computer installed the latest vs code, previously I had 1.67.1-1 now I have 1.67.1-2 from commit da15b6f and 2022-05-06. I then cd to my fork of your repo on my branch and ran code ., worked like a charm. I then checked my rust-analyzer and have I the same as you v0.3.1051.
So, maybe it was 1.67.1-2 vs 1.67.1-1 but my guess having rebooted my system "fixed" something.
Anyway, on the vs code front all is well and the tests are working for me :)
wink@3900x 22-05-16T16:24:46.538Z:~/prgs/rust/myrepos/ferrous-systems-testing-proc-macros (Update-stderr-files)
$ cargo test
    Finished test [unoptimized + debuginfo] target(s) in 0.01s
     Running unittests (target/debug/deps/contracts-ba10f924ddbf31e0)

running 5 tests
test lower::tests::produces_assertion_for_precondition ... ok
test analyze::tests::can_extract_precondition ... ok
test analyze::tests::non_dsl_attributes_are_preserved ... ok
test codegen::tests::output_is_function_item ... ok
test parse::tests::valid_syntax ... ok

test result: ok. 5 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/precondition.rs (target/debug/deps/precondition-7e5da3776f7abacd)

running 2 tests
test fail - should panic ... ok
test pass ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/ui.rs (target/debug/deps/ui-b0c3547267355bbf)

running 1 test
    Checking contracts-tests v0.0.0 (/home/wink/prgs/rust/myrepos/ferrous-systems-testing-proc-macros/target/tests/contracts)
    Finished dev [unoptimized + debuginfo] target(s) in 0.07s

test tests/ui/has-arguments.rs ... ok
test tests/ui/has-expr-argument.rs ... ok
test tests/ui/item-is-not-a-function.rs ... ok
test tests/ui/precondition-is-not-an-expression.rs ... ok
test tests/ui/zero-contracts.rs ... ok

test ui ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.56s

   Doc-tests contracts

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

authors: japaric winksaville

repo: ferrous-systems/testing-proc-macros


text:
add export entry for wasm and worker files

To enable usage like
import ffmpegWasm from '@ffmpeg/core/wasm';

this will be convenient when using file-loader in webpack.

Thanks for the contribution!

authors: jeromewu jichang

license: MIT


text:
Firebase

Descrição da Issue
Configurar o Firebase para ter as notificações do sistema.
Tasks:

[ ] Configurar o firebase
[ ] Ter exemplo de uma notifricação.

Critérios de Aceitação:

[ ] Firebase configurado
[ ] Permitir fazer uma notificação

Uma alternativa para quando formos automatizar o envio é conectá-lo a gateway com esse Server Integration

A notificação está sendo enviada manualmente(criando lá no Firebase). Para as próximas issues de notificação deve-se configurar essa lógica de envio de notificação na gateway aparentemente tem um endpoint pra isso

authors: EsioFreitas youssef-md


text:
Configurações de integração e deploy

Descrição

Configuração de nginx como proxy reverso
Correção de configurações de testes e análise estática
Adição de makefile com principais comandos utilizados em desenvolvimento
Melhoria na segurança em relação à configuração de variáveis de ambiente

Tipo de mudança (Marque "x")

[ ] Correção de bug
[x] Nova funcionalidade
[ ] Documentação

Issue relacionada
Fix #109  #110 #111
Checklist

[x] Li o documento CONTRIBUTING(CONTRIBUTING.md)
[ ] Adicionei testes relacionados as minhas mudanças (se necessário)
[ ] Documentei minhas mudanças (se necessário)
[ ] Todos os testes estão passando
[x] Revisei minhas alterações
[x] Resolvi eventuais conflitos

Observações
A falha nos testes é de responsabilidade dos desenvolvedores. As mudanças contidas neste PR dizem respeito a configurações de integração e deploy.

O trecho version: "2" no arquivo .codeclimate.yml não se refere a versão da linguagem, mas a versão do próprio arquivo de configuração do codeclimate.
Referência: https://docs.codeclimate.com/docs/advanced-configuration#section-configuration-file-structure-and-content
Ao validar as configurações do arquivo utilizando o CLI do Codeclimate o seguinte warning é gerado caso a versão do arquivo seja omitida:
WARNING: missing 'version' key. Please add `version: "2".


text:
Proposed new action (log column change)

This action adds a comment that logs the new task column; like all comments, it contains the date and who performed the action. It is also possible to manually edit the comment to add detail, if needed. This means that one can just look at the comments and see the history of the task (what happened when, and when relevant, why). The action is triggered by the move-column event, so it is possible to control which ones are/not logged by enabling the action to just the columns "of interest".
This is a short patch so I'm not bothering with the full pull request mechanism:
diff --git a/app/Action/TaskLogMoveAnotherColumn.php b/app/Action/TaskLogMoveAnotherColumn.php
new file mode 100644
index 0000000..84579e4
--- /dev/null
+++ b/app/Action/TaskLogMoveAnotherColumn.php
@@ -0,0 +1,88 @@
+<?php
+
+namespace Action;
+
+use Model\GithubWebhook;
+use Model\Task;
+
+/**
+ * Add a log of the triggering event to the task description.
+ *
+ * @package action
+ * @author  Oren Ben-Kiki
+ */
+class TaskLogMoveAnotherColumn extends Base
+{
+    /**
+     * Get the list of compatible events
+     *
+     * @access public
+     * @return array
+     */
+    public function getCompatibleEvents()
+    {
+        return array(
+            Task::EVENT_MOVE_COLUMN,
+        );
+    }
+
+    /**
+     * Get the required parameter for the action (defined by the user)
+     *
+     * @access public
+     * @return array
+     */
+    public function getActionRequiredParameters()
+    {
+        switch ($this->event_name) {
+            default:
+                return array('column_id' => t('Column'));
+        }
+    }
+
+    /**
+     * Get the required parameter for the event
+     *
+     * @access public
+     * @return string[]
+     */
+    public function getEventRequiredParameters()
+    {
+        switch ($this->event_name) {
+            default:
+                return array('task_id', 'column_id');
+        }
+    }
+
+    /**
+     * Execute the action (append to the task description).
+     *
+     * @access public
+     * @param  array   $data   Event data dictionary
+     * @return bool            True if the action was executed or false when not executed
+     */
+    public function doAction(array $data)
+    {
+        $column = $this->board->getColumn($data['column_id']);
+        return (bool) $this->comment->create(array(
+            'comment' => "Moved to column {$column['title']}",
+            'task_id' => $data['task_id'],
+            'user_id' => $data['user_id'],
+        ));
+    }
+
+    /**
+     * Check if the event data meet the action condition
+     *
+     * @access public
+     * @param  array   $data   Event data dictionary
+     * @return bool
+     */
+    public function hasRequiredCondition(array $data)
+    {
+        switch ($this->event_name) {
+            default:
+                return $data['column_id'] == $this->getParam('column_id');
+        }
+    }
+}
diff --git a/app/Model/Action.php b/app/Model/Action.php
index 2204ad3..6aef81a 100644
--- a/app/Model/Action.php
+++ b/app/Model/Action.php
@@ -49,6 +49,7 @@ class Action extends Base
             'TaskAssignCategoryColor' => t('Assign automatically a category based on a color'),
             'CommentCreation' => t('Create a comment from an external provider'),
             'TaskCreation' => t('Create a task from an external provider'),
+            'TaskLogMoveAnotherColumn' => t('Add a comment logging moving the task between columns'),
             'TaskAssignUser' => t('Change the assignee based on an external username'),
             'TaskAssignCategoryLabel' => t('Change the category based on an external label'),
         );

Thanks, I merged your patch: https://github.com/fguillot/kanboard/commit/e6f7929f7780dc7ab06f6c4fbbbc86cd63115cd2
I just modified the code to have translated text.

authors: fguillot orenbenkiki

repo: fguillot/kanboard


text:
Error: getNamespaceValue: Node no longer exists

Sometimes I have this error ... maybe add a try catch ?
Warning: PicoFeed\Parser\XmlParser::getNamespaceValue(): Node no longer exists

Do you know which feed url is generating this error?

I am sorry no ... 3 or 4 in a list of 1000

maybe add a try catch

You can not catch warnings in php. Beside that, a proper solution is to fix warnings instead of hiding them.
With which Feedreader do you use picoFeed? In case it is miniflux, do you have the content grabber enabled?

I am sorry no ... 3 or 4 in a list of 1000

We need to know which feed causes the warning. Otherwise it is impossible to fix this.
Can you reliable reproduce the warning i.e. by exporting your subscriptions, removing all feeds from your Feedreader, import the subscriptions and updating the feeds? This way you can remove feeds from your subscriptions until you have found the faulty one.

I catch an example of this error:
bin/picofeed feed "http://www.tandfonline.com/action/showFeed?ui=0&mi=h7u81o&ai=1g8&jc=rcel20&type=etoc&feed=rss"
PHP Warning:  PicoFeed\Parser\XmlParser::getNamespaceValue(): Node no longer exists in vendor\fguillot\picofeed\lib\PicoFeed\Parser\XmlParser.php on line 274
Warning: PicoFeed\Parser\XmlParser::getNamespaceValue(): Node no longer exists in vendor\fguillot\picofeed\lib\PicoFeed\Parser\XmlParser.php on line 274
PHP Warning:  PicoFeed\Parser\XmlParser::getNamespaceValue(): Node no longer exists in vendor\fguillot\picofeed\lib\PicoFeed\Parser\XmlParser.php on line 274
Feed::id = http://www.tandfonline.com/action/showFeed?ui=0&mi=h7u81o&ai=1g8&jc=rcel20&type=etoc&feed=rss
Feed::title = http://www.tandfonline.com
Feed::feed_url = http://www.tandfonline.com/action/showFeed?ui=0&mi=h7u81o&ai=1g8&jc=rcel20&type=etoc&feed=rss
Feed::site_url = http://www.tandfonline.com
Feed::language =
Feed::description =
Feed::logo =
Feed::date = Thu, 14 May 15 13:32:27 +0000
Feed::isRTL() = false
Feed::items = 0 items

Bug analysis in a nutshell:

warning is related to a SimpleXMLElement::children($uri) call on an empty SimpleXMLElement element in getNamespaceValue
the rss 1.0 parser doesn't expect a namespaced channel element (dunno if namespaced channel elements are valid, but the feed validator doesn't complain)

@acrozes The issue is fixed in the latest picofeed development version. The is parsed correctly (items with content are returned) and the warning doesn't occur any longer.
Would you please test if it works for you as well and close the ticket afterwards!

authors: acrozes fguillot mkresin

repo: fguillot/picoFeed


text:
Latex Editor

Create, edit and compile Latex documents in the lab.
Additional context
Can be done with vscode + extension, or via an overleaf installation in the container.
Discussion Question
Integrate Latex in all images or use a separate one?
Latex install + editor can be quite heavy. A downside of splitting would be the inability to edit code/ review written code when writing a document about such.

How about a central Overleaf deployment? One key feature of overleaf is cooperative writing ... by using a separate installation per user, this feature would not be utilized.

authors: DeepCowProductions hgiefers


text:
Docs: Add quick reference for Blade icon usage

Abstract
Adds a line users can quickly copy to use the Blade icon component
<x-filament::icon icon="heroicon-o-check-circle" />

Description

[x] No code changes

Code style

[x] No code changes

Testing

[x] No code changes

Documentation

[x] Documentation is up-to-date.

It could be confusing why we use this internally, since normal Blade icon output works just fine. Hence why it is not documented. It only applies when you are using an icon alias.

It could be confusing why we use this internally, since normal Blade icon output works just fine. Hence why it is not documented. It only applies when you are using an icon alias.

Ah okay, understood!

authors: caendesilva danharrin

repo: filamentphp/filament


text:
ui has no obvious way to report a security bug privately

A user finding a possible security bug in plnkr.co might want to report it privately, to give the developer a chance to evaluate it and fix it before it becomes public knowledge and potentially harms other users.
But I am unable to find any guidance in the plnkr.co UI on how to report a bug in a non-public way.

You can contact me at filearts on gmail. Thanks for reaching out.

authors: ggoodman oresmus


text:
Would love to figure out a way to hide canaries in list

I know that you can do it with "--all" but these are system requests, it'd be great to figure out an alternative. Maybe an option to hide your thing from list --all? not sure

What about making sure labels can be specified from the command line on all jobs and then support some sort of query mechanism (include/exclude jobs with a label) - like how Kubernetes does selectors. Then the selector has a default value of "exclude canaries"?

conveniently, we have labels already :) just aren't sorting/filtering them

This is done!

authors: aronchick simonwo wjam

repo: filecoin-project/bacalhau


text:
Implementing karma-coverage

The other day I was on Slack, I came up with an issue that the exist method is being deprecated. Robert suggested that we should implement the karma-coverage, which I think is an interesting idea. Please let me know what you guys think about this idea, it would be much appreciated. If you agree I want to try to install the karma-runner dependency using this command: npm install karma karma-coverage --save-dev. I will test if the dependency can even be installed and later we can add any additional tests related to karma-coverage.
If you want more references this link tells you more about karma-coverage: https://github.com/karma-runner/karma-coverage. This is the same link that Robert shared on Slack. Thank you, Robert for offering such an interesting idea. I am willing to implement it, and I hope that most of us will be willing to work on this issue.
Thank you guys for reading this.

@MarkKrutik I think this is a great idea.  Quite a few students have asked for it.  If you want to give it a shot, I would be happy to review your PR.  I've played with this in the past, and run into some issues with parceljs and the way it bundles and minifies the code.  I think if we generate source maps for test builds, it should be possible.  Let's try it.

Go for it @MarkKrutik.  You'll want to play around with our settings in https://github.com/filerjs/filer/blob/master/karma.conf.js and https://github.com/filerjs/filer/blob/master/package.json.

I have installed karma by running this command:  npm install karma karma-coverage --save-dev. I have tried to fix the package.json and karma.conf.js, and it ran with the npm run karma-mocha. Is there anything else that you would recommend me to do?

I also ran this statement: istanbul cover index.js and got this message
Writing coverage object [C:\Users\markk\filer\tests\dist\coverage\coverage.json]
Writing coverage reports at [C:\Users\markk\filer\tests\dist\coverage]
=============================== Coverage summary ===============================
Statements   : 11.33% ( 710/6267 )
Branches     : 2.25% ( 119/5289 )
Functions    : 5% ( 125/2499 )
Lines        : 61.4% ( 105/171 )
C:\Users\markk\filer\tests\dist\index.js:9
__cov_da_OZ01mWkcCGFxHD8f19g.s['1']++;parcelRequire=function(e,r,n,t){__cov_da_OZ01mWkcCGFxHD8f19g.f['1']++;__cov_da_OZ01mWkcCGFxHD8f19g.s['2']++;var i=(__cov_da_OZ01mWkcCGFxHD8f19g.b['1'][0]++,'function'==typeof parcelRequire)&&(__cov_da_OZ01mWkcCGFxHD8f19g.b['1'][1]++,parcelRequire),o=(__cov_da_OZ01mWkcCGFxHD8f19g.b['2'][0]++,'function'==typeof require)&&(__cov_da_OZ01mWkcCGFxHD8f19g.b['2'][1]++,require);function u(n,t){__cov_da_OZ01mWkcCGFxHD8f19g.f['2']++;__cov_da_OZ01mWkcCGFxHD8f19g.s['4']++;if(!r[n]){__cov_da_OZ01mWkcCGFxHD8f19g.b['3'][0]++;__cov_da_OZ01mWkcCGFxHD8f19g.s['5']++;if(!e[n]){__cov_da_OZ01mWkcCGFxHD8f19g.b['4'][0]++;__cov_da_OZ01mWkcCGFxHD8f19g.s['6']++;var f=(__cov_da_OZ01mWkcCGFxHD8f19g.b['5'][0]++,'function'==typeof parcelRequire)&&(__cov_da_OZ01mWkcCGFxHD8f19g.b['5'][1]++,parcelRequire);__cov_da_OZ01mWkcCGFxHD8f19g.s['7']++;if((__cov_da_OZ01mWkcCGFxHD8f19g.b['7'][0]++,!t)&&(__cov_da_OZ01mWkcCGFxHD8f19g.b['7'][1]++,f)){__cov_da_OZ01mWkcCGFxHD8f
ReferenceError: describe is not defined
at Object.parcelRequire.k419.../../src (C:\Users\markk\filer\tests\dist\index.js:9:495518)
at u (C:\Users\markk\filer\tests\dist\index.js:9:2146)
at p (C:\Users\markk\filer\tests\dist\index.js:9:2386)
at Object.parcelRequire.Focm../spec/filer.spec (C:\Users\markk\filer\tests\dist\index.js:9:861267)
at u (C:\Users\markk\filer\tests\dist\index.js:9:2146)
at parcelRequire.rwRH (C:\Users\markk\filer\tests\dist\index.js:9:2962)
at Object. (C:\Users\markk\filer\tests\dist\index.js:9:3932)
at Module._compile (internal/modules/cjs/loader.js:689:30)
at Object.Module._extensions.(anonymous function) [as .js] (C:\Users\markk\AppData\Roaming\npm\node_modules\istanbul\lib\hook.js:107:24)
at Module.load (internal/modules/cjs/loader.js:599:32)

I am assuming from this message most of the code is not covered.

@MarkKrutik usually you wouldn't run istanbul directly, but rather let karma run it as a plugin.  You can see in their example at https://github.com/filerjs/filer/blob/master/package.json how they do it, by specifying a coverage reporter.

authors: MarkKrutik humphd

repo: filerjs/filer


text:
Languages are mixed when default

Description
When windows default langage,

Steps To Reproduce
from the start
Steps to fix
When I set it to Japanese in the settings, the date is also in Japanese.
Files Version
2.4.14.0, 2.4.15.0(pre)
Windows Version
windows11 HOME, 22H2, build 22621.963
Relevant Assets/Logs
no log.

Changed the image to a clearer one.

The date follows the system language, closing since this is intended.

repo: files-community/Files


text:
Code Quality: Remove INI Parser dependency

Description
We currently utilize the INI Parser solely within the Adaptive Layout Helper and can easily replace with other code.
Concerned code
Adaptive layout helper
Gains

Less dependencies
More control over the code

Requirements
No response
Comments
No response

Why not create a service, WindowsIniService.
public string GetValue(string section = string.Empty, string name = string.Empty);

If section is empty, returns the first item.
I found Win32 API for this but it is for 16-bit compat https://learn.microsoft.com/en-us/windows/win32/api/winbase/nf-winbase-getprivateprofilestring. But worth taking a look.

Something like this
var lines = File.ReadLines(iniPath);
			var keys = lines.Select(line => line.Split('='))
								.Where(parts => parts.Length == 2)
								.ToDictionary(parts => parts[0].Trim(), parts => parts[1].Trim());

yeah but putting this in places where it needs makes you hard to maintain. Also, I like services because I plan to create Unit Tests for services in the future.

Can I work on?

I meant that we can use that for the service.


text:
Copy and move operations always report success even on error

Description
On Windows Explorer, when you drag a file to another directory but the file is still open, it will prompt an error message that it can't move the file because it is currently open with an another program. In Files (Community) however, it doesn't show an error, but instead shows that the task has been completed.
Steps To Reproduce

Take one file and one folder
Important: Open the file first
Move the file to a folder
The file is still on the original place pre-"moving", but Files (Community) shows that the move has been complete

Expected behavior
I expected that one should see an error message saying that the file has not yet been transferred because the file in question is currently open and one should save all work before closing and trying again the task they tried to do.
Files Version
Version: 2.1.2.0 OS Version: 10.0.19043.1466
Windows Version
Windows 10 21H1 OS Build 19043.1466
Relevant Assets/Logs
https://user-images.githubusercontent.com/88814097/151035531-a4ea4fe8-3545-4505-b06b-76bb92332de8.mp4

I am new to the Open source community. For one of my courses for college, we have to contribute to a project - this seems like an issue that I may be able to help with. Can you assign this to me please? I'll update along the way as I troubleshoot

@lightningHoneypot is this still an issue in the preview version? https://files.community/download/preview

@lightningHoneypot is this still an issue in the preview version? https://files.community/download/preview

Nope! It seems that everything is good to go!
There is one issue that Files still shows a checkmark on the ongoing tasks icon that it makes it look like a file transfer is successful, but then the error box appears stating that a process is still using the file in question. However, it might as well be an inconvenience (see video on what am I saying, if there's any questions tell me).
(excuse the crappy editing in DaVinci Resolve, I am still in the beginner stages and that is the reason why it took so long for my response)
https://user-images.githubusercontent.com/88814097/167963318-842da106-8d91-400f-9773-a05b8838813c.mov

@lightningHoneypot thanks for the response, can you open a separate issue for the status icon? It's deffinitly something that should be fixed.


text:
SMB File

Description
After deleting a file located on an SMB folder that is linked to my computer as Z: drive, it crashes. Every time I make a file change in the SMB folder I have to refresh. If I change the name, I have to refresh... delete I have to refresh.
Steps To Reproduce
Navigate to an SMB mounted folder... rename it, then refresh, then delete it. Upon trying to refresh, it crashes.
Expected behavior
I expect any name change or deletion in an SMB folder that is mounted to automatically refresh the files listed.
Files Version
Version: 2.1.13.0 OS Version: 10.0.22000.493
Windows Version
Windows 11 Pro 21H2 22000.493
Relevant Assets/Logs
2022-02-15 14:22:26.8731|INFO|OnConnectionRequestReceived|Argument: FileOperation
2022-02-15 14:22:28.8796|INFO|OnConnectionRequestReceived|Argument: FileOperation
2022-02-15 14:22:34.6220|INFO|OnConnectionRequestReceived|Argument: NetworkDriveOperation
2022-02-15 14:22:34.6332|INFO|OnConnectionRequestReceived|Argument: ShellLibrary
2022-02-15 14:22:34.6335|INFO|OnConnectionRequestReceived|Argument: DetectCloudDrives
2022-02-15 14:22:34.6627|INFO|OnConnectionRequestReceived|Argument: GetSelectedIconsFromDLL
2022-02-15 14:22:34.7233|INFO|OnConnectionRequestReceived|Argument: GetSelectedIconsFromDLL
2022-02-15 14:22:34.7809|INFO|OnConnectionRequestReceived|Argument: GetIconWithoutOverlay
2022-02-15 14:22:34.7956|WARN|StartSTATask|The system cannot find the file specified. (0x80070002)
System.IO.FileNotFoundException: The system cannot find the file specified. (0x80070002)
at Vanara.PInvoke.HRESULT.ThrowIfFailed(String message)
at Vanara.PInvoke.Shell32.ShellUtil.GetShellItemForPath(String path)
at Vanara.Windows.Shell.ShellItem..ctor(String path)
at FilesFullTrust.Win32API.GetFileIconAndOverlay(String path, Int32 thumbnailSize, Boolean getOverlay, Boolean onlyGetOverlay)
at FilesFullTrust.MessageHandlers.Win32MessageHandler.<>c__DisplayClass3_0.b__1()
at FilesFullTrust.Win32API.<>c__DisplayClass0_0`1.b__0()
2022-02-15 14:22:35.0748|INFO|OnConnectionRequestReceived|Argument: GetIconWithoutOverlay
2022-02-15 14:22:35.1009|INFO|OnConnectionRequestReceived|Argument: GetIconWithoutOverlay
2022-02-15 14:22:35.1107|INFO|OnConnectionRequestReceived|Argument: DetectQuickLook
2022-02-15 14:22:35.1215|INFO|OnConnectionRequestReceived|Argument: GetNewContextMenuEntries
2022-02-15 14:22:35.6193|INFO|CheckQuickLookAvailability|QuickLook detected: False

Could you upload the second debug file as well? (debug..og)

2022-02-15 14:22:28.9264|ERROR|AppUnhandledException|Sequence contains more than one matching element
System.InvalidOperationException: Sequence contains more than one matching element
at System.Linq.Enumerable.SingleOrDefault[TSource](IEnumerable1, Func2) + 0xe8
at Files.Filesystem.ShellFilesystemOperations.<>c__DisplayClass13_0.b__12(ShellOperationItemResult) + 0x6c
at System.Runtime.CompilerServices.ConditionalWeakTable2.CreateValueCallback.Invoke(TKey) + 0x2b    at System.Linq.Enumerable.WhereSelectListIterator2.MoveNext() + 0xa5
at System.Linq.Enumerable.WhereEnumerableIterator1.MoveNext() + 0x68    at System.Collections.Generic.List1.AddEnumerable(IEnumerable1) + 0x77    at System.Linq.Enumerable.ToList[TSource](IEnumerable1) + 0x122
at Files.Extensions.EnumerableExtensions.<>c__DisplayClass5_01.<ToListAsync>b__0() + 0x34    at System.Func1.Invoke() + 0x28
at System.Threading.Tasks.Task1.InnerInvoke() + 0x47    at System.Threading.Tasks.Task.Execute() + 0x1c --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38    at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17    at Files.Extensions.EnumerableExtensions.<ToListAsync>d__51.MoveNext() + 0x120
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at Files.Filesystem.ShellFilesystemOperations.d__13.MoveNext() + 0x1b8e
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at Files.Filesystem.FilesystemHelpers.d__17.MoveNext() + 0xe81
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at System.Runtime.CompilerServices.TaskAwaiter1.GetResult() + 0x10    at Files.Views.ModernShellPage.<KeyboardAccelerator_Invoked>d__111.MoveNext() + 0xf64 --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21    at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.<ThrowAsync>b__7_0(Object) + 0x1e    at System.Action1.Invoke(T) + 0x28
at System.Threading.WinRTSynchronizationContext.Invoker.InvokeCore() + 0x33
at System.Runtime.InteropServices.McgMarshal.ThrowOnExternalCallFailed(Int32, RuntimeTypeHandle) + 0x21
at __Interop.ComCallHelpers.Call(__ComObject, RuntimeTypeHandle, Int32) + 0xb8
at __Interop.ForwardComStubs.Stub_10[TThis](__ComObject, Int32) + 0x24
at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x3d
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x75
at System.EventHandler`1.Invoke(Object, TEventArgs) + 0x2e
at __Interop.Intrinsics.HasThisCall__23[TArg0](Object, IntPtr, Object, TArg0) + 0x36
at Files!+0x143a750
2022-02-15 14:22:34.1170|INFO|OnActivated|App activated by ToastNotification
2022-02-15 14:22:34.2650|WARN|MergeSettings|Merging settings failed
System.ArgumentException: The parameter is incorrect.
Error trying to write application data container value
at System.Runtime.InteropServices.WindowsRuntime.IMapSharedReferenceTypesStubClass.Insert[TKey, TValue](IDictionary`2, TKey, TValue) + 0xd1
at Windows.ApplicationModel.DataTransfer.DataPackagePropertySet.global::System.Collections.Generic.IDictionary<System.String,System.Object>.set_Item(String, Object) + 0x21
at Files.ViewModels.SettingsViewModel.Get[TValue](TValue, String) + 0x22d
at Files.Helpers.RegistryToJsonSettingsMerger.MergeSettings() + 0xaf
2022-02-15 14:22:34.7190|INFO|GetDrivesAsync|Drive added: C:, Fixed
2022-02-15 14:22:34.7800|INFO|GetDrivesAsync|Drive added: D:, Fixed
2022-02-15 14:22:34.8110|WARN|GetDrivesAsync|Unauthorized: Attempting to add the device, E:, failed at the StorageFolder initialization step. This device will be ignored.
2022-02-15 14:22:34.8570|INFO|GetDrivesAsync|Drive added: Z:, Network
2022-02-15 14:22:34.9050|INFO|DeviceAdded|Drive added: E:, Removable
2022-02-15 14:22:35.0740|INFO|EnumerateDrivesAsync|Adding cloud provider "Nextcloud" mapped to C:\Users\foxho\Nextcloud
2022-02-15 14:22:35.1000|INFO|EnumerateDrivesAsync|Adding cloud provider "OneDrive" mapped to C:\Users\foxho\OneDrive

Thanks if this the full file? In not could you upload the full file as attachment? It's hard to read it like this, thanks

No it isn't the full file. Only what pertains to that timeframe. Other Log information was like 10 or 15 minutes prior or after the event.

Is this still an issue in the preview version? https://files.community/download/preview

authors: foxhoundv gave92 yaichenbaum


text:
Cannot open tab on current desktop

What's the Problem?
I opened two virtual desktops. Files was opened on one of them. When I pressed Win + E on the other desktop, I jumped to the desktop with files open.
Solution/Idea
No matter how many virtual desktops there are, open files only on the current desktop.
Alternatives
No matter how many virtual desktops there are, open files only on the current desktop.
Priorities

Capability
Priority

This proposal will allow developers to accomplish Y
Could

Files Version
2.1.13.0
Windows Version
Windows 10.0.19042.1586
Comments
No response

You can change this behavior by toggling the option to open a seperate instance.


text:
Code Quality: Reorganze namespaces

Description
Reorganize namespaces. For more information, see Details of Changes
Motivation and Context
I found that there're a lot of namespaces that need to be reorganized and there're namespaces that do not need to be separated and I merged them into the same.
Details of Changes

Previous
New
Rationale

Files.App.Controllers.IJson
Remove
Unused

Files.App.ValueConverters
Files.App.Converters
Same

Files.App.AppModels
Files.App.DataModels
Same

Files.App.CommandLine
Files.Backend.CommandLine
Can move to Backend

UserControls/Pane/PreviewPane
UserControls/PreviewPane
No need to be another folder

UserControls' ViewModels(Files.App.ViewModels.*)
Files.App.ViewModels.UserControls
Organized

Validation

[x] Built and ran the app
[ ] Tested the changes for accessibility

Ready to review :)

@onein528 could you look into the conflicts?

@d2dyno1 are you okay with these changes?


text:
Fix crash when closing combined properties

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #7918
Closes #7958

Details of Changes
Add details of changes here.

Fixed an issue causing crash when clicking OK button in Combined properties

Validation
How did you test these changes?

[x] Built and ran the app

/azp run


text:
Test run crashes and performance issues when plugin API is rendered during 'run mode' in CI

We identified an issue with a customer of ours where the performance of their test run is impacted when using the cypress-plugin-api plugin - specifically the browser renderer crashes. The performance issue is more pronounced when recording with Test Replay, since Cypress is capturing the entirety of the DOM from the cypress-plugin-api plugin that is rendered (like the entire response body within prism.js).
There is no value in rendering the entirety of the UI during run mode / during CI - as noted in this issue you opened previously. The user cannot interact with it during the run and they cannot interact with it when reviewing test Replay. This rendering DOES impact performance of the test run as the browser resources have to render.
I also wanted to link to this performance issue as we feel it's likely related.
Ask
It would be great if these hidden HTML elements were not rendered during run mode. You could also offer a flag to turn this off conditionally, as this rendering still impacts performance during open mode as well. Ideally, this UI would have some sort of virtual rendering where it renders on 'click' so that it does not impact performance of Cypress in any situation.
Let us know if you need any additional information and if any next steps are planned here. Thanks!

I am running into the same issue, as well, and agree that we do not need to use the plugin for CI runs. While it would be a nice-to-have in order to view conflicting JSON data on test failures between the request and the response bodies, I have added the following code to my e2e.ts file in order to only use this plugin when we are running tests locally, rather than in CI as well:
// We only want to utilize cypress-api-plugin locally, as we can't
// interact with the plugin within the test runner for CI jobs
// This also resolves a memory issue with the plugin itself
if (Cypress.env('CYPRESS_API_PLUGIN_ENABLED') === 'TRUE') {
  // eslint-disable-next-line @typescript-eslint/no-require-imports
  require('cypress-plugin-api');
}

This is a band-aid fix until I am able to hear back from a dev, or until cypress-plugin-api consumes less memory in the GH Actions run.

authors: danfooks jennifer-shehane

repo: filiphric/cypress-plugin-api

license: ISC


text:
Update maven central badge

Codecov Report

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

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   67.71%   67.71%
=======================================
  Files           8        8
  Lines         254      254
  Branches        3        3
=======================================
  Hits          172      172
  Misses         82       82

Continue to review full report at Codecov.

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

Thanks, was wondering why that wasn't working!

authors: codecov-io ilya-murzinov jeremyrsmith

repo: finagle/featherbed


text:
Update sbt to 1.5.5

Updates org.scala-sbt:sbt from 1.2.8 to 1.5.5.
Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-minor

Superseded by #106.


text:
Add use client directive to fpjs-provider.tsx

As I understand it, this would allow users to use the <FpjsProvider>  in Next.js 13 server components without having to wrap it inside a client component first:
https://beta.nextjs.org/docs/rendering/server-and-client-components#rendering-third-party-context-providers-in-server-components

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: JuroUhlar ilfa

repo: fingerprintjs/fingerprintjs-pro-react


text:
StudioPanel - show console view if no "table" output

THE FOLLOWING DISCLAIMER APPLIES TO ALL SOFTWARE CODE AND OTHER MATERIALS CONTRIBUTED IN CONNECTION WITH THIS SOFTWARE:
THIS SOFTWARE IS LICENSED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS “AS IS” AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE AND ANY WARRANTY OF NON-INFRINGEMENT, ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. THIS SOFTWARE MAY BE REDISTRIBUTED TO OTHERS ONLY BY EFFECTIVELY USING THIS OR ANOTHER EQUIVALENT DISCLAIMER IN ADDITION TO ANY OTHER REQUIRED LICENSE TERMS.
ONLY THE SOFTWARE CODE AND OTHER MATERIALS CONTRIBUTED IN CONNECTION WITH THIS SOFTWARE, IF ANY, THAT ARE ATTACHED TO (OR OTHERWISE ACCOMPANY) THIS SUBMISSION (AND ORDINARY COURSE CONTRIBUTIONS OF FUTURES PATCHES THERETO) ARE TO BE CONSIDERED A CONTRIBUTION. NO OTHER SOFTWARE CODE OR MATERIALS ARE A CONTRIBUTION.

This should have had unit tests.

repo: finos/kdb-studio


text:
Add trino dependencies to server module

What type of PR is this?
Improvement
What does this PR do / why is it needed ?
Add trino dependencies to server module
Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?
No

we will also need to add the deps in execution-collection and generation-collection modules

authors: jinanisha vikask-gs

repo: finos/legend-engine


text:
Service Test Editor UI Imp: Label options in the connection-id dropdown in the service test editor

Summary

Label options in the connection-id dropdown in the service test editor will benefit users when there are identical connection ids from different runtimes.

How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

I think naming IdentifiedConnectionWithRuntime is a bit confusing because connections doesn't have a runtime. It's the other way around. Can we name it better? May be RuntimeConnections or something?
At other places I think we don't really have to mention that it is a connection from runtime. Why don't we add a note at the starting of the file regarding the assumption we make on all the connections are coming from a runtime as it is service?
I feel like we should just add a new field in NewConnectionDataState for runtime instead of introducing a type for this.
What do you think?

@YannanGao-gs
Don't really like the format of the label.
Maybe we could have a label for the runtime the connection belongs to
and another label for the store it contains.
We don't really need the connectionId again in the label since that is the main selection point.

repo: finos/legend-studio


text:
Running legend sdlc locally on https

Hi legend team,
For some reasons, our requirement is to run legend sdlc locally on https server. For this I installed cert and placed in my trust-store. I did following config change and getting this error.

The error message seems like a good hint, please try to read through this and follow:
https://www.dropwizard.io/en/latest/manual/configuration.html#default

The error message seems like a good hint, please try to read through this and follow: https://www.dropwizard.io/en/latest/manual/configuration.html#default

Hi @akphi, I did same config and running server with this command "java -jar C:/Users/maursho/LEGEND/legend-sdlc/legend-sdlc-server/target/legend-sdlc-server-0.91.2-SNAPSHOT-shaded.jar server C:/Users/maursho/LEGEND/legend-sdlc/legend-sdlc-server/src/test/resources/config-sample.yaml" , still same issue

The error message seems like a good hint, please try to read through this and follow: https://www.dropwizard.io/en/latest/manual/configuration.html#default

Hi @akphi  @epsstan @beekemarie , this issue is resovled but while doing curl I am getting this error

authors: akphi shomil-maurya-db

repo: finos/legend


text:
Waltz Project Meeting: 2024-07-08

Date
Monday 08 Jul 2024 - 8am EST / 1pm UK  30 mins
Occurs the second Monday of every month
Zoom
Join Zoom Meeting
https://zoom.us/j/98495304665
Meeting ID: 984 9530 4665
Passcode: 202795
Find your local number: https://zoom.us/u/aeyTu3Q7E8
Untracked attendees

Name
Firm
Comment

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact legal@finos.org with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene, roll call, welcome new people (5mins)
[ ] Display FINOS Antitrust Policy summary slide
[ ] Review Meeting Notices (see above)
[ ] 1.61/1.62 Summary (EUDAs, Aliases)
[ ] Plans

[ ] Admin screens
[ ] Documentation
[ ] Flow changes
[ ] Flow Visualisation

[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] ...

Action Items

[ ] ...

Project Microsite: https://waltz.finos.org

Hi, Jess here :)

Hi team
We would like to have better understanding on report grids and its extended use-cases. Few queries are listed below:

How to add more subjects in reporting grid feature?
How can more data entity can be added into subjects in reporting grid?

repo: finos/waltz


text:
Last part of getting started, pls help!

Hi Scott, thanks for putting this together... I'm trying to follow the webhooks instructions for Monzo
I got to the final furlong with this, but then got confused by the last part of the getting started guide, 'Connect to bank' (for Monzo). Here's what I put into the request body...
{
"account_id": "[from YNAB MONZO ID - as described, the final part of the URL]",
"url": "[APP URL?secret=SECRETSTRING, as instructed a random string]"
}
but then I'm not sure what to do after that. Do I need to save it? Does clicking 'SEND button save this? In any event, when I click 'SEND' I get the following error:
Sorry if completely asinine question - first time heroku user, but desperate to get this working to make YNAB easier! Thanks

Here's the error I get in the response body:
code:"forbidden.insufficient_permissions"
message:"Access forbidden due to insufficient permissions"

Can you just confirm where you are trying to send that data? In Monzo's dev portal?
The account_id in there is not your YNAB account id, it's your Monzo account ID, which you get by calling the "List Accounts" endpoint on that page.

Let me know if that does not work.

I have updated the wiki to make this a bit more clear

Thanks Scott. I got it working using this tutorial. Thanks for updating
though
https://community.monzo.com/t/automatically-import-transactions-to-ynab/38594/6
On Fri, Aug 17, 2018 at 9:55 PM, Scott Robertson notifications@github.com
wrote:

I have updated the wiki to make this a bit more clear
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fintech-to-ynab/fintech-to-ynab/issues/97#issuecomment-413985790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AofJizjggjQGajRNM5heuCJq3dW60vZPks5uRy2vgaJpZM4WAszD
.

repo: fintech-to-ynab/fintech-to-ynab


text:
FAILED_PRECONDITION: The Cloud Firestore API is not available for Datastore Mode projects

[REQUIRED] Codelab info
Project: Cloud Functions for Firebase: https://codelabs.developers.google.com/codelabs/firebase-cloud-functions/#0
[REQUIRED] Project setup
Directory: friendlychat/cloud-functions-start/functions
[REQUIRED] Describe the problem
Notification does not get sent when a new user logs in and there are errors in the logs:
FAILED_PRECONDITION: The Cloud Firestore API is not available for Datastore Mode projects

The notification should happen at step 8 of the tutorial

I have failed to follow all the steps. I have skipped step 8: create message section

repo: firebase/codelab-friendlychat-web


text:
Revert "Removing LOCATION param from all official extensions"

Reverts firebase/extensions#1520

HI @korturpazar
Can you explain the reason behind this PR? Thanks!

repo: firebase/extensions


text:
unmet peer dependency "@firebase/app-compat@0.x"

Describe your environment

Operating System version: macOS Monterey v12.3.1
Firebase SDK version: v10.1.0
Firebase Product: firebase-admin
Node.js version: v17.5.0
Yarn version: v1.22.18

Describe the problem
Steps to reproduce:
I get the following warning when installing the latest version of firebase-admin (10.1.0)
warning "firebase-admin > @firebase/database-compat@0.1.8" has unmet peer dependency "@firebase/app-compat@0.x".

Same cause as https://github.com/firebase/firebase-js-sdk/pull/2082 and potentially same fix (inside the Firebase JS SDK). When transitioning from v8 to v9, app (now app-compat) was moved back into peerDependencies. I think the same conditions still exist such that we can move it back to devDependencies.

This should be fixed in https://github.com/firebase/firebase-js-sdk/pull/6319
Thank you!

authors: hsubox76 lahirumaramba thomaswang

repo: firebase/firebase-admin-node


text:
App Store redirect not working for iOS Email Auth Dynamic Links

[READ] Step 1: Are you in the right place?
Yes
[REQUIRED] Step 2: Describe your environment

Operating System version: MacOS 10.15.3
Firebase SDK version: 8.10.0
Firebase Product: auth
Node.js version: 10.16.3
NPM version: 6.9.0

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
Using the link generated by
const link = await admin.auth().generateSignInWithEmailLink(userEmail, actionCodeSettings)
does not redirect the user to the App Store when the iOS app is not installed.
However, if I add the isi and remove the ifl params, this correctly redirects to the App Store and successfully logs the user in after they download the app.
My actionCode settings are correct and this works when the app is installed.
Is this a bug or is this intended behaviour? Because I was under the impression that users should be redirected to stores when the app is not installed, as they do on Android.
Thanks!

This is intended. ActionCodeSettings does not expose the option to install the iOS app when not installed. We did this on purpose because the process how these FDL redirects work is not reliable in certain versions of iOS. There is a low risk the link and the underlying code is intercepted by the wrong app. That is not an acceptable risk for sensitive auth operations.

Thanks for your reply. Do you have any more information around the circumstances under which this may occur? Which versions of iOS, in particular, does this affect?
Thanks

@bojeil-google I've been struggling to figure out this exact issue.
It would be great to have more information on which versions of iOS this affects as installing the app from a link and logging in the user is critical to our email based login workflow.

authors: antpuleo2586 athornz bojeil-google


text:
Adding google-api-core[grpc] dependency

Taking a direct dependency on google-api-core[grpc] as recommended in https://github.com/googleapis/google-cloud-python/issues/5023
Skipping grpc installation on PyPy
Upgraded tox version to latest
Removing redundant dependency declarations from tox.ini (tox installs them via setup.py anyway)

Resolves #184

4 years later and this solved the ModuleNotFoundError: No module named 'google.auth' problem for me.
Specifically, I ran pip3 install 'google-api-core[grpc]', after which the error no longer occurred.

repo: firebase/firebase-admin-python


text:
FR: allow authDomain override in plist

[READ] Guidelines
Feature proposal

Firebase Component: Auth

Allow authDomain to be set in plist so custom domain can be used in the oauth flow.
This can already be done for web once you setup your custom domain.

Yes, please.

Hi team, can we get this prioritised please? As per this notification from Google:

We’re writing to let you know that end users of applications using Google Cloud Identity Platform may not be able to authenticate if you're using the signInWithRedirect method and your users are using one of the following browsers:

Safari 16.1+ on macOS
iOS 16.1+
Firefox 109+

What do you need to know?
Based on our records, you have an app that uses signInWithRedirect in the Web SDK API, meaning your users who use the platforms listed above are impacted.
What do you need to do?
While we cannot control the version/settings of the browser sending the login request, we want to offer mitigations for > authentication in your app to continue working for all users. Please follow our GCIP best practices guide to make sure users can use your app when additional browser platform updates are released.

However all feasible options require setting the authDomain which is impossible to do on mobile.

Hi team, can we get this prioritised please? As per this notification from Google:

We’re writing to let you know that end users of applications using Google Cloud Identity Platform may not be able to authenticate if you're using the signInWithRedirect method and your users are using one of the following browsers:

Safari 16.1+ on macOS
iOS 16.1+
Firefox 109+

What do you need to know?
Based on our records, you have an app that uses signInWithRedirect in the Web SDK API, meaning your users who use the platforms listed above are impacted.
What do you need to do?
While we cannot control the version/settings of the browser sending the login request, we want to offer mitigations for > authentication in your app to continue working for all users. Please follow our GCIP best practices guide to make sure users can use your app when additional browser platform updates are released.

However all feasible options require setting the authDomain which is impossible to do on mobile.

Thanks for filing this! The best practice needs to be applied only for web applications that use the signInWithRedirect web SDK API. No action is needed for mobile apps.

@prameshj thats great! Can you please elaborate why and explain what the best practice solution is for using a custom domain on mobile clients should then be?

@prameshj thats great! Can you please elaborate why and explain what the best practice solution is for using a custom domain on mobile clients should then be?

This guide applies to web applications that use redirect to login. The clients of this web app can be mobile clients (using a mobile browser) or web clients. The best practice(of setting authDomain or any other option) should be implemented in the web application.
This is needed because in redirect flow we 1) start at the app domain (call it domain1), 2) redirect to oauth helper code (in /__/auth/handler, call this domain 3), store some state in session storage of domain 2 3) redirect back to the app and open iframe to domain 2 and access session storage.
This cross origin storage access is disallowed in some browser policies, so all the best practices are towards making sure domain1 and domain2 are the same origin.
On mobile apps, a similar IDP sign in redirects to the browser, but there is no cross-origin storage access. Hence there is no action needed for mobile apps.

Similar to what I mentioned in the Android ticket, I guess our reasoning is not that it doesn't work, but that we would like to show our configured custom domain to the end user instead of the default *.firebaseapp.com domain for brand/trust reasons.

Hi @skion, we currently do not support custom domain on mobile. We have put this feature request on our radar, but we are unable to promise a timeline right now.

Similar to what I mentioned in the Android ticket, I guess our reasoning is not that it doesn't work, but that we would like to show our configured custom domain to the end user instead of the default *.firebaseapp.com domain for brand/trust reasons, during an OIDC or Microsoft login flow.
Is there a way to do this?

just a FYI - we dumped firebase and went with auth0; the lack of feature parity and prioritization for things like this was the motivator for moving off firebase auth.

At least indicate in the official documentation that using a custom domain leads to the breakdown of any oauth2 authorization in native applications (tested on github, apple, facebook).

Allowing the authDomain to be set in mobile is a must to prevent brute force attacks using the public key. This would allow us to host a proxy behind firewalls and WAF's that can mitigate such brute force attacks from hitting the google auth api's directly.

authors: NhienLam jostster prameshj psi-gh sebr siarheidudko skion zxed

repo: firebase/firebase-ios-sdk


text:
Remove deprecated token callback

The deprecated token callback has been replaced by messaging:didReceiveRegistrationToken:.

Done. Removed the unnecessary iOS 10 only label


text:
🐛 [firebase_storage] Firestore stale local cache issue

Problem.
Sometimes, when fetching a Firestore collection, it appears that the Firestore client only retrieves the old documents and does not fetch newly added documents. This issue is affecting the consistency of data retrieval from Firestore.
Expected Behavior:
When querying a Firestore collection, the Firestore client should retrieve all relevant documents, including both old and newly added documents. The data should be consistent with what is stored in the Firestore database.
Steps to Reproduce:
Attempt to fetch a Firestore collection.
Observe that the fetched data may not include newly added documents.

@nileshmsd12345
Can you provide flutter doctor -v, plugin versions, platform on which you are seeing the reported behavior and most importantly, a runnable reproducible code sample that shows it returns old documents and not new, so that we can properly address the issue.

repo: firebase/flutterfire


text:
firebase_core: Twitter Login Fails on Mobile When Firefox is the Default Browser

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Android
Description
There are two problems with Twitter login:

Initial Login Issue: If you are not logged into Twitter on the web browser of your mobile phone (or the web browser if you are on the web platform), the Twitter authentication pop-up does not close after logging in. Instead, it opens the Twitter web page in the pop up window. On the second attempt, since the Twitter session is already active, the pop-up closes correctly, and Firebase Auth login succeeds.

Browser-Specific Issue on Android: On Android (and likely on iOS as well), if Firefox is set as the default web browser, after logging in through the pop-up, it remains open and throws an error (see attached screenshot). In contrast, using Chrome as the default browser allows the pop-up to close after login, and authentication succeeds.

Reproducing the issue
Setup a normal twitter login setup through firebase.
Firebase Core version
3.2.0
Flutter Version
3.22.2
Relevant Log Output
No response
Flutter dependencies

Expand Flutter dependencies snippet

Replace this line with the contents of your `flutter pub deps -- --style=compact`.

Additional context and comments
No response

Hello @rsegecin, both of those issues are unrelated to FlutterFire directly. You can open an issue on the Android repository: https://github.com/firebase/firebase-android-sdk
Is see people have opened similar issues here: https://github.com/firebase/firebase-android-sdk/issues/6144

Hello @Lyokone, I hope you're well, thank you.

authors: Lyokone rsegecin


text:
UnimplementedError: subscribeToTopic() is not supported on the web clients

Is there an existing issue for this?

[x] I have searched the existing issues.

Which plugins are affected?
Messaging
Which platforms are affected?
Web
Description
DartError: UnimplementedError:       subscribeToTopic() is not supported on the web clients.
  To learn how to manage subscriptions for web users, visit the
  official Firebase documentation:

  https://firebase.google.com/docs/cloud-messaging/js/topic-messaging

dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 296:3  throw
packages/firebase_messaging_web/firebase_messaging_web.dart 171:5            subscribeToTopic
packages/firebase_messaging/src/messaging.dart 284:22                        subscribeToTopic
packages/sensewheel/main.dart 102:36                                         main
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future.dart 542:22
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 736:7                                    callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                             _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                              _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 181:7
Reproducing the issue
DartError: UnimplementedError:       subscribeToTopic() is not supported on the web clients.
  To learn how to manage subscriptions for web users, visit the
  official Firebase documentation:

  https://firebase.google.com/docs/cloud-messaging/js/topic-messaging

dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 296:3  throw
packages/firebase_messaging_web/firebase_messaging_web.dart 171:5            subscribeToTopic
packages/firebase_messaging/src/messaging.dart 284:22                        subscribeToTopic
packages/sensewheel/main.dart 102:36                                         main
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future.dart 542:22
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 736:7                                    callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                             _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                              _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 181:7
Firebase Core version
firebase_core: ^3.9.0
Flutter Version
flutter:3.24.0
Relevant Log Output
DartError: UnimplementedError:       subscribeToTopic() is not supported on the web clients.

      To learn how to manage subscriptions for web users, visit the
      official Firebase documentation:

      https://firebase.google.com/docs/cloud-messaging/js/topic-messaging

dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 296:3  throw_
packages/firebase_messaging_web/firebase_messaging_web.dart 171:5            subscribeToTopic
packages/firebase_messaging/src/messaging.dart 284:22                        subscribeToTopic
packages/sensewheel/main.dart 102:36                                         main
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50           <fn>
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future.dart 542:22                                        <fn>
dart-sdk/lib/async/zone.dart 1661:54                                         runUnary
dart-sdk/lib/async/future_impl.dart 163:18                                   handleValue
dart-sdk/lib/async/future_impl.dart 861:44                                   handleValueCallback
dart-sdk/lib/async/future_impl.dart 890:13                                   _propagateToListeners
dart-sdk/lib/async/future_impl.dart 666:5                                    [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 736:7                                    callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                             _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                              _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 181:7           <fn>

Flutter dependencies

Expand Flutter dependencies snippet

Replace this line with the contents of your `flutter pub deps -- --style=compact`.

Additional context and comments
No response

Hi @this-sunil, thanks for the report. As stated in your logs, the the JS SDK does not have support for subscribeTopic. To implement this functionality, you’ll need to handle it on the server side using the Admin SDK as outlined in the documentation.

authors: SelaseKay this-sunil


text:
Fix slight typo

Slight typo on comment

@paulentine thanks for this!   Happy to merge whenever the CLA bot says I can.

I signed it.
On Wed, Jul 17, 2019 at 3:30 PM googlebot notifications@github.com wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Ffirebase%2Ffriendlyeats-android%2Fpull%2F52
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/firebase/friendlyeats-android/pull/52?email_source=notifications&email_token=AKYR4IEY75QX6RNYCJVKOPTP76MRDA5CNFSM4IEVD672YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2GYVRA#issuecomment-512592580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYR4IAUW73QHC6SRHBERBDP76MRDANCNFSM4IEVD67Q
.

authors: paulentine samtstern

repo: firebase/friendlyeats-android


text:
Update all the Podfile.lock's

For Firebase 7.11.0 and GoogleUtilities 7.4.0

Isn't there something like dependency bot that should be able to update these for us?

Looks like it hasn't been run since January...

Yep. It was initially partially broken #1099 and now seems completely broken.

repo: firebase/quickstart-ios


text:
fix: wrong line number in stack trace

npm uninstall ts-node source-map-support @types/source-map-support
npm install --save-dev tsx
Issue
Using the following
# .gitlab-ci.yaml
---
job:
  image: test-err
  script:
    - echo 1

The stack trace returns the wrong line number.
which is possibly related to https://www.github.com/TypeStrong/ts-node/issues/2053, thus replacing ts-node with tsx

merge with master please @ANGkeith

@ANGkeith We need to ensure that stack traces in the pkg binaries still works, when you remove the sourcemap package.

opps, forgot about that..
but do we really need sourcemap for the binary ?
i thought sourcemap is more for dev/debugging

anyways,

@ANGkeith We need to ensure that stack traces in the pkg binaries still works, when you remove the sourcemap package.

should be fixed already

@ANGkeith We don't have any security concerns going on in this tool, so I can't think of a reason why we would want to exclude such epic piece of debug information, when users make bug reports.

authors: ANGkeith firecow

repo: firecow/gitlab-ci-local


text:
Sign Windows client with a valid EV code signing certificate

[ ] Procure certificate from a trusted party
[ ] Sign app and msi in CI using signtool
[ ] Verify that "Publisher: Firezone" is shown in the UAC dialog

Is there a secret manager we'd use to get a private key into CI for signing?

@ReactorScram Yep! GH has a secrets store we use for the other clients. We actually already had a code signing cert that just lapsed, so I'll take this on as a todo and get the secret available in CI

Tauri has something in its config for this but I'm not sure how it works https://tauri.app/v1/api/config/#windowsconfig

I wonder if that just wraps signtool. If so, this could be as simple as getting the right certs, installing / registering them in the GA runner, and configuring Tauri to sign with the cert fingerprint.

^ That's the process we use for signing the Apple and Android clients in CI.

I tried out SignTool with a self-signed cert and it doesn't seem too awful. Lots of GUI clicking in certmgr though. Key management will be annoying.

But I didn't try anything else because all the providers want hundreds of dollars for a cert and have no free trials.
The Electron docs say that Windows code signing EV certs are bound to an HSM? https://www.electron.build/code-signing.html#windows
What a strange process. I don't know what to try next for this one.

If I use certmgr to trust the self-signed cert, it looks correct in UAC. So maybe the process is feasible

Need to sign contained binaries not just the MSI

authors: ReactorScram jamilbk

repo: firezone/firezone


text:
Feat/lib 555 datepicker component

Jira

I was trying to use the 0.1.1 version, but I am having some problems:

The datepicker has a reddish background.

When selecting a value, the label goes over the icon (seems to be missing some padding on the left)

When used inside a FzConfirmDialog, the DatePicker dropdown creates an overflow. I would expect the container to always stay on top without going to affect the container

Nice to have:
Add a label to be passed via props, as other components do (such as FzInput, FzTextarea, etc.).

I was trying to use the 0.1.1 version, but I am having some problems:

The datepicker has a reddish background.
When selecting a value, the label goes over the icon (seems to be missing some padding on the left)

I will look into this: it seems that the input date is "wrong", maybe a formatting issue

When used inside a FzConfirmDialog, the DatePicker dropdown creates an overflow. I would expect the container to always stay on top without going to affect the container

This is an issue that can't be solved entirely in the Datepicker component. I can add move the datepicker to fixed position, but we also need to merge
https://github.com/fiscozen/design_system/pull/106
in order for that to work

Nice to have: Add a label to be passed via props, as other components do (such as FzInput, FzTextarea, etc.).
This will not be part of the first implementation as agreed in previous meetings: the datepicker will be using our FzInput therefore we don't want to reimplement things like label etc


text:
Fix broken remove_parameterization in firefly

Is this PR adding new feature or fix a BUG?
This is a developer-side bug fix that addresses an issue with VQGAN parameterization removal. This bug does not affect end users under normal circumstances, as remove_parameterization is not used in any other part of the codebase.
The following code (forked from tools/vqgan/inference.py) triggers the error:
model: FireflyArchitecture = load_model(config_name, checkpoint_path, device=device)
model.remove_parametrizations()

Error message:
ValueError: Module FishTransConvNet(
  (conv): ParametrizedConvTranspose1d(
    512, 256, kernel_size=(16,), stride=(8,)
    (parametrizations): ModuleDict(
      (weight): ParametrizationList(
        (0): _WeightNorm()
      )
    )
  )
) does not have a parametrization on weight

Is this pull request related to any issue? If yes, please link the issue.
No

LGTM

repo: fishaudio/fish-speech


text:
Displaced Hint with Bootstrap 4

Hi there,
Thanks for building this nifty autocompleter! I'm really enjoying using it with a little project I'm working on. The only hiccup I've run into so far is that the hint div and span end up getting displayed outside of the input box (specifically, below the input box). This ends up overlapping with the first row of the drop down list (see screenshot). I'm wondering if there is something with Bootstrap 4 (which I'm using) that is interfering with the placement of the hint? If you have any ideas for a fix, they would be greatly appreciated!
Thanks.

Forgive me for abandoning the project and answering you a few years later.
Bootstrap 4 example with a hint: https://codesandbox.io/s/jquery-autocompleter-bootstrap-4-died3v?file=/src/styles.css
It doesn't work well actually. The core idea here is to use CSS top property:
CSS
.custom .autocompleter-hint {
  top: -37px; /* <-- HERE */

  color: #6c757d;
  text-align: left;
  line-height: 1.5;
  left: 0;
  width: 100%;
  padding: 0.375rem 0.75rem;
  font-size: 1rem;
}

authors: anyhowes fitiskin

repo: fitiskin/jquery-autocompleter


text:
node_manager crashes on startup due to missing MessageFrame.ui

Since updating node_manager_fkie from 0.7.8 to 0.8, I can no longer open the node manager without it crashing. The traceback is as shown below.
[INFO] [1533888798.773351]: listen for logs on /rosout
[INFO] [1533888799.061694]: Start RPC-XML Server at ('0.0.0.0', 22622)
[INFO] [1533888799.062469]: Subscribe to parameter `/roslaunch/uris`
Traceback (most recent call last):
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/node_manager_fkie/main_window.py", line 779, in on_master_state_changed
    self.getMaster(msg.master.uri).master_state = msg.master
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/node_manager_fkie/main_window.py", line 572, in getMaster
    self.masters[masteruri] = MasterViewProxy(masteruri, self)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/node_manager_fkie/master_view_proxy.py", line 358, in __init__
    self.message_frame = MessageFrame()
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/node_manager_fkie/message_frame.py", line 137, in __init__
    loadUi(ui_file, self.frameui)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/python_qt_binding/binding_helper.py", line 247, in loadUi
    return _loadUi(uifile, baseinstance, custom_widgets)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/python_qt_binding/binding_helper.py", line 151, in _loadUi
    return uic.loadUi(uifile, baseinstance=baseinstance)
  File "/usr/lib/python2.7/dist-packages/PyQt5/uic/__init__.py", line 222, in loadUi
    return DynamicUILoader(package).loadUi(uifile, baseinstance, resource_suffix)
  File "/usr/lib/python2.7/dist-packages/PyQt5/uic/Loader/loader.py", line 72, in loadUi
    return self.parse(filename, resource_suffix, basedir)
  File "/usr/lib/python2.7/dist-packages/PyQt5/uic/uiparser.py", line 972, in parse
    document = parse(filename)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1182, in parse
    tree.parse(source, parser)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 647, in parse
    source = open(source, "rb")
IOError: [Errno 2] No such file or directory: '/opt/ros/kinetic/lib/python2.7/dist-packages/node_manager_fkie/MessageFrame.ui'

Thank you for reporting and sorry for trouble!
I released a new version, which should fix this problem.

I hope, the issue is solved.
If not, please reopen!

authors: atiderko samlcharreyron


text:
feat: Added HasGameReference mixin

Description
This almost exactly like the current HasGameRef mixin, except that:

The property is called game instead of gameRef (the "gameRef" violates Dart naming conventions against using abbreviations in variable names);
The template type T supports all Games, not only FlameGames;
Better integration with the SingleGameInstance mixin.

Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[-] I have updated/added relevant examples in examples.

Breaking Change

[-] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Related Issues

@erickzanardo Shouldn't we take a more careful approach here: instead of merging the two classes, merely mark the old one as deprecated? That way the users will be able to switch at their own pace, instead of being forced into it.

@st-pasha from what I see on your implementation, despite the name changes, everything could be accommodated on the current HasGameRef implementation.
The generics would change, but a FlameGame is a Game, so there wouldn't be a breaking change here. And how you made the retrievement of the game instance is an internal thing, just like on the HasGameRef, so that could go in there too.
I know that the naming convention may be off, but I would rather keep it instead of adding a big (big because this is widely used) just because of renaming, and avoid frutastions like this.

Ok, I've merged the two implementations then: the old name is marked as "Deprecated", but only in comments (so everything continues to work as before). We could upgrade the comment to "real" deprecate-annotation, either in this PR or at some later time (perhaps after the next release).

Sorry for popping in so late into the discussion, but I seriously think that we should slow down on breaking core API. I agree that HasGameReference is better than HasGameRef and a game property makes more sense than gameRef, but, renaming the core API has some serious trade-offs for example: making existing tutorial blogs and videos totally outdated, let alone all the projects people have already deployed
In my opinion, the juice doesn't worth the squeeze here. The rule of thumb on changing core API should be "we rename it only if it becomes misleading or sufficiently inaccurate".
All the other aspects of this PR are great and I appreciate all the work done here ❤

@renancaraujo I agree completely, breaking changes should be avoided at all costs, and when necessary introduced at a very slow pace. Which is why in the first submission I had the HasGameReference mixin completely separate from HasGameRef. This way we can transition from the old mixin to the new one reaaaally slow:

at first start mentioning in Discord that we prefer HasGameReference mixin now;
change our documentation/examples to use the new mixin only;
add a note somewhere that HasGameRef will eventually be deprecated;
deprecate it only at some future date, say in 6 months or in a year;
finally remove after it was deprecated for some long time.

This way we could still introduce code improvements, but with no disruption to the users.
The only con of this approach is that it results in some code duplication -- but I don't see it as that much of a big deal. Code duplication when migrating between different APIs is quite common, and doesn't result in increased maintenance cost.
Should I revert to that original version of the PR?

I agree with @renancaraujo and @erickzanardo, it seems unnecessary to rename the mixin and gameRef -> game just because it breaks the naming convention. The PR overall is good though, can't the HasGameRef just be changed to have the behaviour of HasGameReference, without changing any naming?

Alright, I've restored HasGameRef to the way it originally was. Now:

No changes for existing users whatsoever (not even deprecation);
If anyone gets annoyed by less-than-perfect grammar (like me), they can switch to HasGameReference and .game.

Thus, this PR is a net-positive for some users, and neutral for everyone else.

Alright, I've restored HasGameRef to the way it originally was. Now:
1. No changes for existing users whatsoever (not even deprecation);

2. If anyone gets annoyed by less-than-perfect grammar (like me), they can switch to `HasGameReference` and `.game`.

Thus, this PR is a net-positive for some users, and neutral for everyone else.

I don't think that we should keep HasGameReference, it will just lead to confusion having two mixins doing the same thing.

But that is exactly how code evolution works, if we want to avoid breaking changes.

This is not a necessary evolution though and we can avoid it. The code changes can be done in HasGameRef and the name change is not more important than avoiding a breaking change even though it would be a slow transition, the whole team has agreed on this.

This is not a necessary evolution though and we can avoid it. The code changes can be done in HasGameRef and the name change is not more important than avoiding a breaking change even though it would be a slow transition

So, what you're saying is that we want to avoid even a slow deprecation cycle here?
It's fine by me -- in fact, the PR is already structured that way. We could have the two mixins live side-by-side, giving the users an opportunity to choose which one they prefer. Perhaps I could add some more clarification in doc-comments about this, so as to avoid any potential confusion on the users part.

So, what you're saying is that we want to avoid even a slow deprecation cycle here?

No, what I'm saying is that the HasGameReference mixin shouldn't be introduced at all.
The changes done to the code in HasGameReference should be possible to add in a non-breaking way to HasGameRef, right?

No, what I'm saying is that the HasGameReference mixin shouldn't be introduced at all.

In that case I'm not exactly sure what is the objection against having this mixin? Given that it has some benefit (which appears minor for some people, and substantial for others), and no cost (anyone who has a question why there are two mixins can read the doc-comments and find the answer).

Having HasGameReference means that we would deprecate HasGameRef eventually, which we won't because of the reasons mentioned here, specially because of this one mentioned by Renan:

renaming the core API has some serious trade-offs for example: making existing tutorial blogs and videos totally outdated, let alone all the projects people have already deployed

Maybe this is something that we can consider rename when we start planning a Flame 2.0. But that is something for the long term future.

In that case I'm not exactly sure what is the objection against having this mixin? Given that it has some benefit (which appears minor for some people, and substantial for others), and no cost (anyone who has a question why there are two mixins can read the doc-comments and find the answer).

I don't understand, what are the benefits that can't be incorporated into the current mixin (except for the name changes)?
We shouldn't have two ways of doing things if we can avoid it, even if we can write docs about it, users are generally bad at reading the docs well.

Maybe this is something that we can consider rename when we start planning a Flame 2.0. But that is something for the long term future.

Agree.

I'll try to summarize the state of the discussion so far, so that it is easier to move forward:

I believe we all agree that, by itself, and not taking other factors into account, the name change gameRef -> game is good. There is a difference in opinion of how much of a benefit this creates, but at least no one had said that the old name was better.

We also definitely agree that breaking changes ought to be avoided, especially if done for minor reasons like changing a slightly imperfect name.

We do recognize that the current PR does not create any kind of breaking change. We want it to stay that way.

The only point of disagreement is whether there will be a substantial amount of confusion, and whether the drawbacks of that confusion outweigh the benefits of having a better name. I'm not sure how to quantify this confusion, but as far as I can see, the confusion is:

harmless, in the sense that even if someone picks one of the two mixins at random, their code would still work the same (up to a name change);
resolvable, meaning that if someone does see two mixins with similar names and the question of why there are two classes really bothers them, they would just go and check the documentation which would state the reasons plainly;
non-unique: there are other places in the code base where we are doing a transition from old way of doing things to the new way, this is not a "bug", this is the way code evolution works. If the users can understand this principle, it would make their life easier.

The only point of disagreement is whether there will be a substantial amount of confusion, and whether the drawbacks of that confusion outweigh the benefits of having a better name. I'm not sure how to quantify this confusion, but as far as I can see, the confusion is:
* **harmless**, in the sense that even if someone picks one of the two mixins at random, their code would still work the same (up to a name change);

It won't work the same if they choose the wrong one when they are following a tutorial, due to the name changes.

* **resolvable**, meaning that if someone does see two mixins with similar names and the question of why there are two classes really bothers them, they would just go and check the documentation which would state the reasons plainly;

They don't necessarily see that there are two classes, and just pick the wrong one and it won't work for them.

* **non-unique**: there are other places in the code base where we are doing a transition from old way of doing things to the new way, this is not a "bug", this is the way code evolution works. If the users can understand this principle, it would make their life easier.

We do recognize this, but this small change just isn't worth it since it is simply a change of naming (and the class is a very commonly used one), the whole team agrees on this.
Just like Erick said, this change can be something to be considered for v2.0, but it won't be merged with the name change before that (the code change is fine however, so that could just be put into HasGameRef instead).

I believe we all agree that, by itself, and not taking other factors into account, the name change gameRef -> game is good. There is a difference in opinion of how much of a benefit this creates, but at least no one had said that the old name was better.

Fully agree!

Fully agree!
I'm in favour of the rename and fully agree with @st-pasha . The current naming is not great since it violates effective dart (https://dart.dev/guides/language/effective-dart/design#avoid-abbreviations).
I'm all in in deprecating HasGameRef for HasGameReference.

It will be deprecated, but not pre-v2. This discussion is already settled.
We can create a ticket for it (with a v2 label) so that we don't forget about it.
What's left to do for this PR to get merged is to move the new code into the old mixin in a non-breaking way, could you do that @st-pasha?

It is very nice that this change will get in! Great news 🔥💙
Thanks @st-pasha for working on it!

It will be deprecated, but not pre-v2.

No one asked me, but this is the way. Tracking breaking changes and targeting them for v2 (possible years out, I assume) is definitely the ideal for a project with this many users

No one asked me, but this is the way. Tracking breaking changes and targeting them for v2 (possible years out, I assume) is definitely the ideal for a project with this many users

Your opinion is always valued @kurtome, and that's a good point actually, we need to start tracking what we want to put in V2.

we need to start tracking what we want to put in V2.

Perhaps we could have an issue with a task list of things, and then when the time comes we could quickly convert those task items into issues with a press of a button.

@spydon I've merged HasGameReference into the HasGameRef.

@spydon @st-pasha With this being merged into main, value_route.dart in _sphinx\flame\examples\lib is throwing errors.  I have synced my fork and ran pub get just in case for that yaml file, but I am getting errors like:
HasGameReference - Classes can only mix in mixins and classes.
Missing variable type for 'score'.
Try adding an explicit type, or remove implicit-dynamic from your analysis options file.
Undefined name 'game'.
Try correcting the name to one that is defined, or defining the name.

Surely I am missing something, right?  As it stands now, docs will not build on any updated branch.

Surely I am missing something, right? As it stands now, docs will not build on any updated branch. Reverting the file to the prior state of 12ce270 does solve the issue though.

I don't see any errors regarding this, try to run melos clean and then melos bs.

Tried it. I had @ufrshubham check and he had same issue. It only appears once you build the docs locally.

Solved.  Odd that I was able to get away without using melos for so long until all of a sudden, but none-the-less.

authors: alestiago erickzanardo kurtome munsterlander renancaraujo spydon st-pasha

repo: flame-engine/flame


text:
Metrics to measure hook execution times

counter of a hook executions
counter of binding contexts
a gauge with an execution time

Done in #176  and #180

repo: flant/shell-operator


text:
Blizzard: Patch notes are not being fetched

The bot responds with "I couldn't find any patch notes."  This is true for any game, whether or not the bot was online when new patch notes were posted.

Thanks, this entire cog has been due for a rework pretty much since I released it. See issue #2. I plan on:

Removing Pandoc requirement entirely.
Instead of posting the entire notes in chat, just post a link, or the first couple lines of the patch notes + a link to the full notes.
Add autodetection of new patch notes, with the option to post an alert in the channel when this happens (sort of like a webhook, in fact I should check to see if a suitable webhook already exists)

Thoughts?

Yep, it was the missing Pandoc dependency.  Those sound like fine ideas, though it'd definitely be helpful to post the entirety of the patch notes if they're very brief (one or two changes or a bugfix).  Links to prior patch notes would also be helpful as sometimes Blizzard will put out a lot of patches in quick succession.
The ability to specify between PTR and live patchnotes would also be helpful, for example checking out the recent Lucio changes on the Overwatch PTR.

Rework is complete. Pandoc is no longer required, and patch notes are working again (at least for all current notes -- please let me know if you encounter future patch notes that aren't working). I will add PTR notes to my roadmap, it's a good idea 👌

authors: HelmicNewciv flapjax

repo: flapjax/FlapJack-Cogs