Clean up console prints for backend

Well, I accidentally left some prints in my last PR that are not needed, so I'll fix that tomorrow.
Also there is currently a mix of fmt.Println and log.Println throughout the code. Jaafar suggested to switch everything to log.Println, and also add some new useful prints, such as saying when a simulation started, to what file it was saved, how long did it take, etc

@VFjr FYSA you can now directly do log.Info() or log.Debug() or log.Error() etc... (log is the import name of logrus). That's a preference since it's colored on the console by default!


Add benchmarks for https://bugs.swift.org/browse/SR-11558

@swift-ci please test and merge

(I used the benchmark doctor locally to verify the benchmarks)

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


Fixes issue #188: problem with deprecated 'role in human social proce…

…sses' and new class that replaces it.

Verified the issue is fixed. I also made three minor updates.  This PR is ready to go.


dashboard: search for apps

Hi,
a search field for the apps would be nice to quickly find some. The applications list might display the apps with the entered string in the name.

Related to #3739


protractor 2.0 fails with Cannot read property '_failures' of undefined

I want to use the jasmine-reports with the protractor 2.0, but I'm getting an error:
/Dev/project/node_modules/jasmine-reporters/src/junit_reporter.js:154
            if (isFailed(spec)) { spec._suite._failures++; }
                                             ^
TypeError: Cannot read property '_failures' of undefined

I checked the 'spec' object, it's like this currently (no _suite property)
{ id: 'spec0',
  description: 'should contain basic identifiers',
  fullName: 'episode: TC14235 required fields should contain basic identifiers',
  failedExpectations:
   [ { matcherName: '',
       message: 'Failed: undefined is not a function',
       stack: 'Error: Failed: undefined is not a function\n    at stack (/Dev/project/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1441:17)\n    ...
       passed: false,
       expected: '',
       actual: '' } ],
  passedExpectations: [],
  status: 'failed',
  _endTime: Thu Apr 16 2015 10:42:18 GMT-0400 (EDT) }

May be I'm doing something not supported in the 2.0?
Thanks

Are there any other interesting aspects related to the tests that are failing? For example, fit and/or fdescribe? Last month I discovered problems with our handling of fit and fdescribe and released<EMAIL_ADDRESS>to address them. Since then, I've been successfully using protractor 2.0 with jasmine-reporters 2.0.5 without any issues.
Make sure you are using 2.0.5 of jasmine-reporters and that you've configured protractor with framework: 'jasmine2'. If you're still having problems, share any interesting information about how you are running your specs, your protractor config, environment, etc.

I'm getting the same error message on Windows 8.1 and in a Jenkins job running on Ubuntu.  I'm running my tests using grunt-protractor-runner and grunt-protractor-webdriver.
I'm using "xdescribe" and "xit" to ignore some suites and specs while I clean up the configuration of our test jobs.  My suites/specs look like this:
   describe('some test suites and specs', function() {

    // Some setup here

    describe('sample suite', function() {
        xit('ignores this spec', function() {
            // ... some expect() calls here
        });

        it('runs this spec', function() {
            // ... some expect() calls here
        });
    });

    xdescribe('ignored suite', function() {
        it('would run if the suite were enabled', function() {
            // ... some expect() calls here
        });
    });

});

When the ignored suites and spec are removed completely, I'm not seeing this error.

What version of jasmine-reporters are you using?

Using version 2.0.5.

Rolled back to 2.0.4, and I'm seeing the same issue.

Thanks, I'll take a deeper look in the next couple days.

@bloveridge My issues were resolved after updating from Node v.0.10.x to v.0.12.2.  I should have thought to check my Node version sooner.
I verified this with all tests running and passing, all tests running and failing, and with some tests/suites ignored.

I have encountered this problem as well.
In my test suite with protractor 2.0 there was raised on exception and the whole jasmine crashes with the exception above.
the structure of e2e test is like:
describe('', function () {
var session = require('...');
beforeEach(function () {
    session.login();
});

afterEach(function (done) {
    session.clearStorage().then(done);
});

describe('preserves state', function () {
    it('test case where it fails', function () {
        throw exception()...
    });
});

});
nodeJs 12.2
OS X,
protractor 2.0
jasmine reporters 2.0.5

i faced the same issue after upgrading to protractor 2.0.

I have been able to reproduce this error with the following in protractor.conf.js:
  framework: 'jasmine2',
  onPrepare: function() {
    var jasmineReporters = require('jasmine-reporters');
    browser.getProcessedConfig().then(function(config) {
      console.log('Executing capability', config.capabilities);
      jasmine.getEnv().addReporter(new jasmineReporters.JUnitXmlReporter({
        consolidateAll: true,
        filePrefix: 'xmloutput',
        savePath: 'testresults'
      }));
    });
  }

Here's the full stacktrace:
/Users/pgconreaux/WebstormProjects/E2E/node_modules/jasmine-reporters/src/junit_reporter.js:152
            if (isFailed(spec)) { spec._suite._failures++; }
                                             ^
TypeError: Cannot read property '_failures' of undefined
    at [object Object].self.specDone (/Users/pgconreaux/WebstormProjects/E2E/node_modules/jasmine-reporters/src/junit_reporter.js:152:46)
    at dispatch (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1869:28)
    at ReportDispatcher.specDone (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1852:11)
    at Spec.specResultCallback [as resultCallback] (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:813:18)
    at complete (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:362:12)
    at QueueRunner.clearStack (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:605:9)
    at QueueRunner.run (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1784:12)
    at /Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1801:16
    at /Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1745:9
    at Function.next.fail (/Users/pgconreaux/WebstormProjects/E2E/node_modules/protractor/node_modules/jasmine/node_modules/jasmine-core/lib/jasmine-core/jasmine.js:1807:9)

Environment:
OSX: 10.10.3
protractor: 2.1.0
node: 0.12.0
jasmine-reporters: 2.0.6

Same here, protractor version 2.1.0, jasmine-reporters 2.0.6.
Problem seems to occur if the spec has an error in it causing an exception to be thrown.
For whatever reason Jasmine 2 calls suiteStarted / specStarted only after the spec itself has run. This causes the error because if an exception is thrown in the spec, suiteStarted & specStarted are never called which causes suite._specs & currentSuite to be undefined. For the same reason spec._suite is not defined (as it is defined in specStarted).
However, specDone is called even after an exception. This method then tries to access spec._suite and throws the exception given by the others above because, well, spec._suite is undefined.
I'm not sure what the correct way to fix this would be, simply & easy fix (which, maybe, is not the way this module should work) would be to replace
if (isSkipped(spec)) { spec._suite._skipped++; }
if (isDisabled(spec)) { spec._suite._disabled++; }
if (isFailed(spec)) { spec._suite._failures++; }
with
if (spec._suite && isSkipped(spec)) { spec._suite._skipped++; }
if (spec._suite && isDisabled(spec)) { spec._suite._disabled++; }
if (spec._suite && isFailed(spec)) { spec._suite._failures++; }
@larrymyers Let me know if I can help with some further debugging.

Probably the error is related to configuring the reports on the getProcessedConfig promise.
In my specific scenario, this code produces the error:
  onPrepare: function() {
    var jasmineReporters = require('jasmine-reporters');
    browser.getProcessedConfig().then(function(config){
      var capabilities = config.capabilities;
      var prefix = capabilities.browserName + '-';
      jasmine.getEnv().addReporter(new jasmineReporters.JUnitXmlReporter({
        savePath: './test-results/e2e',
        filePrefix: prefix
      }));
    });
  }

While this code works fine:
  onPrepare: function() {
    var jasmineReporters = require('jasmine-reporters');
    var prefix = process.pid + '-';
    jasmine.getEnv().addReporter(new jasmineReporters.JUnitXmlReporter(
      savePath: './test-results/e2e',
      filePrefix: prefix
    }));
  }

+1 I've got exactly the same issue as c3s4r
Is something not initialized right due to the async nature of the promise?

Thanks for the details so far and sorry it has taken so long to get to this, hopefully I will get a chance to look into this more in another week or so.
From what I can tell, protractor's asynchronous getProcessedConfig logic is doing non-standard things with jasmine that the framework itself does not plan for and have support for.
For clarification, does the error only occur when using getProcessedConfig and then an exception is thrown inside a test? Or anytime you try to use fit/fdescribe? or when using xit/xdescribe? or something else? The reports kind of seem all over the place right now.

Sorry it had taken

@bloveridge: The issue happens when the reporter is added using the getProcessedConfig promise. It doesn't matter if you are using or not using fit/fdescribe or xit/xdescribe.

@c3s4r It happens all the time when using getProcessedConfig, or only when an exception is encountered? I'm thinking specifically about this earlier statement: "seems to occur if the spec has an error in it causing an exception to be thrown."

@bloveridge you're right, it doesn't happen when all tests pass. It only happens if at least one test fails.

@bloveridge while testing the changes made in my pull request I saw that the "fallback" spec names focused specs were used in my result file name. Maybe this gives you a hint where things start to go wrong?

Yeah, I guess that makes sense. Before deciding on the right fix for this, first I need to try and discover what Jasmine is doing under the hood when run by protractor in this way -- if some reporter methods are being called twice, others not, sharing data between different instances, etc.
Though I would not expect any data to be bleeding across into different instances, since each instance creates its own closure variables inside the constructor. I can only surmise that something in the way protractor is using jasmine is causing it to behave abnormally when calling reporter methods.
Definitely needs more investigation. sigh

@bloveridge It seems to me that the cause for this is simply caused by the fact the reporter is added after the tests have already started to run. By the time getProcessedConfig() promise is resolved, the specStarted had already happened and so when specDone is called, it fails to find spec._suite since the reporter never got the specStarted and never assigned spec._suite.

The solution for this is pretty simple: onPrepare should return the promise and so protractor will wait for the promise to resolve before going on to running the tests:
  framework: 'jasmine2',
  onPrepare: function() {
    var jasmineReporters = require('jasmine-reporters');
    return browser.getProcessedConfig().then(function(config) {
      jasmine.getEnv().addReporter(new jasmineReporters.JUnitXmlReporter({...}));
    });
  }

@shahata
You're right, returning the promise fixes it. I changed the README.md example in my pull-request.

@shahata Thanks for that, I just finished reading the protractor release notes based on your comment and planned to look into returning the getProcessedConfig promise this morning. Thanks @reppners for confirming it works.
I'll close this issue once the README has been updated.

The README has been updated to show how to make Protractor wait until getProcessedConfig() has happened and how to get unique output files for each capability.
2.0.7 has also been released, which adds a new modifySuiteName option to JUnitXmlReporter which will help you to get unique suite names for each capability as well.
Thanks everyone who submitted reports, info, and patches.

@bloveridge, I am getting error when using with protractor 3.1.1
Selenium standalone server started at http://<IP_ADDRESS>:54685/wd/hub
Started
F[launcher] Process exited with error code 1
C:\Users\username\AppData\Roaming\npm\node_modules\protractor-jasmine2-html-reporter\index.js:146
if (isFailed(spec)) { spec._suite._failures++; }
^
TypeError: Cannot read property '_failures' of undefined
at Jasmine2HTMLReporter.self.specDone (C:\Users\username\AppData\Roaming\npm\node_modules\protractor-jasmine2-html-reporter\index.js:146:42)
at dispatch (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:1966:28)
at ReportDispatcher.specDone (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:1949:11)
at Spec.specResultCallback [as resultCallback] (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:903:18)
at complete (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:368:12)
at QueueRunner.clearStack (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:660:9)
at QueueRunner.run (D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:1881:12)
at D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:1898:16
at D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasmine-core\lib\jasmine-core\jasmine.js:1842:9
at D:\worskspace_callcenter\ProtractorDemo\node_modules\protractor\node_modules\jasminewd2\index.js:18:5

check this https://github.com/larrymyers/jasmine-reporters/issues/94#issuecomment-113963903

Still problem is not resolved.
I am using "protractor-jasmine2-html-reporter" with Jasmine2 Framework in conf.js
onPrepare: function() {
var capsPromise = browser.getProcessedConfig();
capsPromise.then(function(caps){
var browserName = caps.capabilities.browserName.toUpperCase();
var browserVersion = caps.capabilities.browser_version;
var platform = process.platform;
var prePendStr=browserName+""+browserVersion+""+platform;
browser.manage().timeouts().pageLoadTimeout(600000),
browser.manage().timeouts().implicitlyWait(300000),
jasmine.getEnv().addReporter(new Jasmine2HtmlReporter({
savePath: _dirname+'/../TestOutput/Automation_reports'+timeStamp+'/',
takeScreenshotsOnlyOnFailures: true,
screenshotsFolder: './screenshots',
filePrefix: prePendStr+'report'
}));});
}
getting error:


Argument "date" in showMore callback function

Feature request.
Please add argument "date" to callback function "showMore" in property "messages".
We would like to display 1 million+ events in calendar without loading and passing them to Calendar. We load some small amount of events for each day and pass them to calendar. We also know how really many events each date have on server. We would like to render real count of events in showMore (not only how many events were passed to Calendar). Now we can't render real amount of events in day because in callback function we don't know what date is rendering.
We would also like to have argument with amount of rendered events in day. This will help to calculate the real more amount:
more amount = real count - rendered events.
Is it possible?

I don't really understand the ask here, but I'm not likely to implement it any time soon. I'm happy to take PR's tho if you want to put something together that is generally useful for folks!


feat: add pretty-printed logs. Closes #3

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.85%. Comparing base (e6ffe16) to head (2fd8ffc).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   96.85%   96.85%
=======================================
  Files          28       28
  Lines         381      382    +1
  Branches       70       70
=======================================
+ Hits          369      370    +1
  Misses         11       11
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Trying to copy or move (rename) files results in 403 with "Paths cannot be empty!" message

I'm trying to rename a file with aiodav library and I'm getting a 403 error with my WebDAV server.
I'm not sure if the WebDAV implementation is to blame or the library, but after getting past #17 I can see that the response from the server is "Paths cannot be empty".
I'm calling it like this
client.move("Some/Old/Path.txt", "Some/Old/Path_new.txt")

Thanks for your colaboration.
Hi @eblis, sorry for late response and for this in case. I'll not able to fix this for a while, I'm very busy right now. If you have time and you know how to do it, you can give me a hand with this, it will be awesome

Hi,
Turns out the problem is in our WebDAV server implementation, it doesn't work if you specify the port in the copy URL, you have to use an URL without the port (so https://example.com:443 but without :443)


compare scenarios

do work for this issue in a branch called issue-97.

scenarios and projections are the same thing and compare projections page already exists. renaming to "'Manage Projections' page mockup"


Barchart right side cut bar

Here is the screenshot we are facing issue with bar is cut when we increases width bars around space increases also so, Please help with some solutions. Thank you in advance

Same issue here. Can't seem to figure it out.

@indiespirit any ideas here?

The only solution I've found is to not show more than 8 bars.

ok

Hello, for me I use barPercentage props based on data.length so something like this:
barPercentage: 1.5 - (data.length * 0.1)


Getting a jwt fails if the user didnt authorize one of the scopes

The problem is the following (as stated in the documentation):
...the scope parameter needs to be given to prevent consumers to accidentally handing out `user:admin` or `organization:owner` scoped tokens to third party services.

But since you do not know the authorized scopes there is no way to know which ones to include (besides making a roundtrip over a normal access token).


Have rounds of spawning With Bosses in between

Instead of constant spawnin, how about after every 30 enemies spawn, stop for about a minute, to give the player a breather
Suggested by my buddy Julian @julianpoy Because his little bitty lady fingers hurt

a minute is quite long

Forsure @Heebe I'll just mess around with the values, maybe like 30 seconds? I'll figure it out!

With bosses in between

Going to Save this for another alpha, since this will work with bosses

Starting this now

This is finished, I just need to scale the bosses stats accordingly

a6909f6b579b762e6ca36a7464343d6adfdf01a0
The commit above is the point where bosses are fully inputted into the game, heres a screenshot:

And also, here is a .gif of me celebrating


Enhancement: Scroll Bar for System Notes field

Our corp uses the system notes to record intel, unfortunately that occasionally takes up quite a bit of space, and it currently gets cut off, a scroll bar would be needed to see all the notes.

I made some improvements to system description field:

The height of the description text container depends on the saved description text
No scrollbars required to see the complete description
The <textarea> has a dynamic with -> larger screens will benefit from this change
<textarea> can be resized (vertical)
Added a "character counter" while edit dialog is active
Max char count is 512. (database column limit)
Added maxlength="512" HTML attribute to <textarea>

New edit mode (dynamic width):

Description text (multiple line breaks,...)

Awesome, great solution. Scrolls bars are terrible.


Models broken with sails since 0.12.2

It not works with sails since 0.12.2.
No errors, no warnings, models just not load.

Probably the issue: https://github.com/leeroybrun/sails-util-mvcsloader/issues/10

+1 I have the same problem in Gladys

This should technically be an easy one to fix, just need to run some checks & tests over it.

Fixed in #12
@leeroybrun can you please accept merge?

Thanks for your pull request.
Will it still works with previous versions of Sails?
I'm not using it (Sails) anymore, so I don't have any apps to test this.

@leeroybrun, actually, I did not check :)
I'll get back with tests results asap

sorry folks, suddenly I got a lot of other workload and can't get back to this
if someone can check this codechange on other versions of sails, please do

@leeroybrun why you not use sails anymore? Had you found another nice framework?

@dilame I'm now using Feathers. It's not the same kind of framework, Sails is a lot more complete but the structure and working of your app is imposed. I found myself limited by this (this is why I started working on sails-util-mvcsloader) and this is why I left to use Feathers instead as it let me be totally free.

Fixed in #12. Thanks @budmitr for your work.


[RUST] Rust DSO module

This PR depends on #2969
cc @ehsanmok
cc @kazimuth in reference to your question about being able to load PackedFuncs directly from Rust.

This adds support to just open compiled shared objects and load packed functions from them. Just curious, looks good overall.

ping @nhynes to see if you would like to address some of the comments :)

Yes, absolutely! It's literally on the top of my to-do list for this weekend :)

soft ping @nhynes

gee, it's sure been a long weekend hasn't it?

would like to handle [...] dev_mblob

Of the functions supported by DSOModule, dev_mblob seems to be the most complex. It might be worth saving that for a later time when there's more of a story around non-llvm targets in the static rust runtime. Do you think the basic functionality is okay for now?

Sure sounds good, please do fix the ci though :)

ping

Thanks, @nhynes @ehsanmok @kazimuth , this is now merged


TypeError: Cannot read property 'src' of null

We're seeing lot's of these error occurring for some of our users, to the tune of about 2000 errors per hour when it happens.  We can't determine the cause of the error as we've never repro'd it internally--we only know of it's existence because we log JS Errors in order to monitor the stability of our web app.
Wondering if we can get any insight on what might cause this.
Here's the stack trace we get from this error:
TypeError: Cannot read property 'src' of null
at Object.$d (https://apis.google.com//scs/apps-static//js/k=oz.gapi.en.CUXyo_wPfp0.O/m=auth/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCMYqx5cA6SdMRKSM5YaRVPo-xpcPg/cb=gapi.loaded_0:64:249)
at c (https://apis.google.com//scs/apps-static//js/k=oz.gapi.en.CUXyo_wPfp0.O/m=auth/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCMYqx5cA6SdMRKSM5YaRVPo-xpcPg/cb=gapi.loaded_0:45:399)
at e.invokeTask (https://dz0py21q5qt7s.cloudfront.net/js/publishDeps?compiled=1&2016-11-23_22-43-4:11:15126)
at n.runTask (https://dz0py21q5qt7s.cloudfront.net/js/publishDeps?compiled=1&2016-11-23_22-43-4:11:12486)
at invoke (https://dz0py21q5qt7s.cloudfront.net/js/publishDeps?compiled=1&2016-11-23_22-43-4:11:16239)
The somewhat beautified minified code where this is coming from looks like:
$d:function(a){
var c=t(a);
a=c ?
c:(c=l(a)) ?
C.origin : ".."==a ?
Rw.parent : window.document.getElementById(a).src; <== this is the 'src' in the error
return h(a);
}
function c(a){
var c=(0,.of)(a.data);
if(c && c.f){
(0,.Fc)("gadgets.rpc.receive("+window.name+"): "+a.data);
var g=_.T.$d(c.f); <== this is where the above function is called ====================
H && ("undefined"!==typeof a.origin) ?
a.origin!==g : a.domain!==/^.+://([^:]+).*/.exec(g)[1]) ?
_.sa("Invalid rpc message origin. "+g+" vs "+(a.origin||"")):f(c,a.origin)
}
}

Seems like this comes from the 'auth' module of the library. Are you using the library for signin users with Google OAuth?
Do you have stats on what browser this comes from?

Yes, we do use Google OAuth.  Every instance I've investigated is coming from Chrome, various versions (52,53,54).  It seems to happen most frequently on OSX, but some are coming from Win7, Win10 and ChromeOS.  It seems to have started happening in mid to late October and has been consistently producing copious quantities of errors since then.  If it was happening before then it was producing few enough errors as to be off the radar.  It seems to happen to relatively few people, but it spews a ton of errors when it happens, so much so that just by the raw numbers this is the #1 source of JS errors in our apps.  We don't know if it causes the user any noticeable problem or not.  We don't think that we've changed anything in regards to usage of gapi, but it certainly could be something that we've changed that has caused this to start happening.  Thanks in advance for any ideas you may have.

I believe this is due to a problem getting a reference to the hidden OAuth 2 IFRAME element from the DOM after receiving a postMessage from that IFRAME, but I do not understand why it would happen. Are impacted users able to provide any further details?

Users haven't reported any specific problems that we can connect to this error, it may be that users are unaware that the error is happening and aren't directly impacted in a noticeable way, but we don't know.  We're only aware of it through our error logs.  We could just mute this error, but didn't want to do that without trying to determine the cause and fixing if possible.  Is it possible this is happening because of a plugin that some users have installed in Chrome that is messing with the DOM or something?

Such a thing is certainly conceivable, but without more data from an affected user's browser I really can't make any sort of diagnosis. Please update this thread if you learn more and are able to share it.

I've noticed that the the code has changed around this error.  It looks like this code has been refactored.  The new error is:
h.postMessage is not a function at https://apis.google.com//scs/apps-static//js/k=oz.gapi.en.Ga51ZqnsCiU.O/m=auth/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCNpJI124YwOQHM7So5m4i2ZeouNcg/cb=gapi.loaded_0:48:445
I don't know if that provides any new information, but wanted to pass it along in case it's helpful.
Thanks again for your help!
--matt

+1 seeing many such errors as well, but not able to reproduce and haven't heard about it from users

Thanks for your additional reports. I don't have any cause analysis to share, but I do appreciate the additional data

Please try to move to the newer version of the sign-in API, and let us know if the problem persists:
https://developers.google.com/identity/sign-in/web/sign-in
https://developers.google.com/identity/sign-in/web/reference

We have not updated our gapi usage yet but by chance I noticed that the same users that have this error happen are also generating a lot of errors from some malware-ish plugins, DNSUnlocker and CounterFlix for example, but a few others as well.  I now suspect that the entire problem is caused by these plugins rather than an actual problem with anything in the gapi.

@carmatdaw Thanks for the info! Have you been able to migrate to gapi.auth2?

We haven't upgraded.  Other than this error, we aren't having any issues with the version we're using.  I'm not sure what immediate benefits we'd get from doing the work to update to the new version.  Is there are compelling reason for us to update to the new version?

@carmatdaw the newer version is more secure and reliable. It may also fit better your use case (it preserves the state of the user which makes it easier to use for many apps). However, if you rely on a "one-off" approach like gapi.auth.authorize, it will not suit you.
We are working on a new behavior in gapi.auth2 that will make the migration very easy as the behavior will be identical to gapi.auth.authorize, I will update this thread when I have more information.

@carmatdaw we released the new feature: https://developers.google.com/identity/sign-in/web/reference#advanced
You should now be able to very easily migrate to the latest version as the behavior is the same. Just make sure to update the configuration parameters, and read the correct ones in the response.
Let me know if that works!

@TMSCH, I'm taking the baton from @carmatdaw on this issue. We've updated from gapi.auth.authorize to gapi.auth2.authorize (with a flag to allow fast switching between the two), but have come across a problem with our Google Sheets integration, where users who authorized scopes with gapi.auth.authorize are not having the same permissions applied when using gapi.auth2.authorize, even though the requested scopes are identical. With auth2, they get an "IMMEDIATE_FAILED" error with detail "Not all scopes are approved." (Because of popup blockers, we do the initial authorize request with prompt: 'none').
If we instead make the gapi.auth2.authorize call result from a click (and therefore, we can leave prompt undefined without getting a blocked popup), we get a Google permissions dialog ("This site wants to [...] View and manage your spreadsheets in Google Drive") despite having granted both "View the files in your Google Drive" and "View and manage your spreadsheets in Google Drive" on this account back when we were using gapi.auth.authorize. When I check the actual "Apps connected to your account", I find that when only using gapi.auth.authorize, my site has access to Google Docs and Google Drive, but after approving the permissions dialog that pops up from using gapi.auth2.authorize, my site now has access to "Basic account info" as well.
So it seems like 1) the permissions dialog I'm being shown isn't actually showing the permission that will be granted when the dialog is submitted (it's asking for spreadsheet permissions, but it's actually granting Basic account info), and 2) gapi.auth2.authorize is asking for basic account info despite it not being included in my requested scopes (which is "https://www.googleapis.com/auth/drive.readonly https://www.googleapis.com/auth/spreadsheets").
This explains two other interesting phenomena related to this: 1) switching back to gapi.auth.authorize immediately makes everything work again (because it's not trying to request any unapproved scopes), and 2) users who performed their initial permission grant using gapi.auth2.authorize have no such problems, and can switch between auth and auth2 without any difficulty (because they have already granted Basic account info permissions, whether they meant to or not).
So, finally, my suspicion is that if gapi.auth2.authorize is made to not request basic info by default, or is given a fetch_basic_profile boolean option like gapi.auth2.init has so we can turn it off, then this changeover should work smoothly. Similarly, I'd guess that you will be unable to reproduce this issue with any app integration that already requested basic account info when using gapi.auth.authorize.
This may seem like nitpicking, but we want the upgrade to auth2 to be invisible to users; making them click a dialog that we pop up so that the Google permissions dialog doesn't get popup-blocked, so they can approve access to Google Sheets data that they think they've already approved access to, but isn't actually what they're approving, and what they're actually approving is access to basic account info that our application doesn't need and didn't ask for, is...pretty clunky.

Hi @timsama. Thanks for migrating to GSI v2!
The issue you're observing puzzles me, it shouldn't happen. It may be a regression. Could you look at the Network logs, and copy paste here the requests URL that point to https://accounts.google.com/o/oauth2/iframerpc?action.
Also, could you copy paste this:
window.open = (function() {
  var open = window.open;
  return function(url) {
    console.log(url);
    return open.apply(window, arguments);
  };
})();

In the console, and then start the authorize flow with GSI v2? It should display a URL in the console, please copy and paste it here too.
Thanks!

@TMSCH Sorry for the delay, I've been traveling for the last two weeks. Here's the request URLs:
`https://accounts.google.com/o/oauth2/iframerpc?action=listSessions&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive.readonly https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fspreadsheets&client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&ss_domain=https%3A%2F%2Flocalhost.office.lucidchart.com&origin=https%3A%2F%2Flocalhost.office.lucidchart.com
https://accounts.google.com/o/oauth2/iframerpc?action=listSessions&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive.readonly https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fspreadsheets&client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&ss_domain=https%3A%2F%2Flocalhost.office.lucidchart.com&origin=https%3A%2F%2Flocalhost.office.lucidchart.com
https://accounts.google.com/o/oauth2/iframerpc?action=listSessions&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive.readonly https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fspreadsheets&client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&ss_domain=https%3A%2F%2Flocalhost.office.lucidchart.com&origin=https%3A%2F%2Flocalhost.office.lucidchart.com`
Strangely, nothing is printed in the console when I run your code immediately before starting the authorize flow.

Although, if the point of that code is to get the URL I'm using to get the client JS, that's https://apis.google.com/js/client:auth2.js?onload=__gapiLoaded.

@timsama the point of the code I gave you is to get the URL that the popup is opened to, so I'm surprised nothing is logged. Did you do the sign-in flow with prompt=none?

@TMSCH Ah. Yes, I did.
In the console:
https://accounts.google.com/o/oauth2/auth?response_type=permission id_token&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive.readonly https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fspreadsheets&openid.realm=&redirect_uri=storagerelay%3A%2F%2Fhttps%2Flocalhost.office.lucidchart.com%3Fid%3Dauth446111&client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&ss_domain=https%3A%2F%2Flocalhost.office.lucidchart.com&gsiwebsdk=2
In case you need them, I also copied the URLs it sent me to during the flow.
Account selection page:
https://accounts.google.com/signin/oauth/oauthchooseaccount?client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&as=-6a2328019fc45753&nosignup=1&destination=https%3A%2F%2Flocalhost.office.lucidchart.com&approval_state=!ChRpcGo1d0lMZDdmLU5Mc2lISWUtYhIfNDVZaUpydGc2OVlkMEdRYlFmWE5LZUI4aEc1cHpCVQ∙ADiIGyEAAAAAWUqvbT3TuQ4KE_1Zfx-9rVJF8hsXI5Tc&xsrfsig=AHgIfE_h7-yeKAodBdAP0n9G_OKgfT-w6A&flowName=GeneralOAuthFlow
Permissions request page:
https://accounts.google.com/signin/oauth/consent?authuser=0&part=AJi8hAN1g08Pu3snxHFQzZJMa6nyksWutCJeOcT8sMAXN71D43231QSw9ZHPydWQebzLGlI9oOXG0PleRIF_ntq4RkT7uuGBt1edgBXD-lhGW5cxqiptT0GujQElaDWTw40BB6xdEGor877o_j8d1dSDK5xpiRraxjMmIDeayK0l-rYkNnJapUVRivgCgpqpLy79F7wv46N2fQFpqWwfDhvfpsN9Kp5qNu6DsJbj44Ek7F8PfdImsZIBxQ5nKchBZlUen1dPZ0iDoysC7XcLAmc-iAjHmeW2aeclIPI8ypz6zheGPlb22wTKDscF87K_cgNtME04jFPMhNomHzFhVhn7qDn4L0RjtokPsVnvjHcQLJWyjLWmW9sPtdcCvRGBPG8lgu4f8wjoznqLud_46q5krIdzJ2PuCgKhlLXEVotVmNewgmzQfc-RjCxDXyycfeJT92ayeGk6t9jqE5yPS0ZCQaltW91sJJZ3ara6vVWsWwwKNjyl-ulGurTfoFmCb9skXhiURk7O&hl=en&as=-6a2328019fc45753#

@timsama could you take a screenshot of the account selection page you're seeing? I'm not able to reproduce the issue.
If you're not using the id_token, you can also add response_type: 'permission' in the configuration to only request access_token. The id_token requires basic info grant.

The URL for the page is
https://accounts.google.com/signin/oauth/oauthchooseaccount?client_id=79338057834-7t65k3vt4c9bcguch7l6vr23ptff13lh.apps.googleusercontent.com&as=6f7a271bf3509438&nosignup=1&destination=https%3A%2F%2Flocalhost.office.lucidchart.com&approval_state=!ChRZRG5STXNJNlhYMFRVZGs3cVpPSxIfZzcwOWtiV3Z1YnNiMEdRYlFmWE5LZUNzeVVoc3pCVQ∙ADiIGyEAAAAAWUq7HJcDSuF8yKN3_DJTlpQck4joYjqn&xsrfsig=AHgIfE_G1phZPBExFathggMQQjA3T7Gkdw&flowName=GeneralOAuthFlow

If it helps reproduce this, I'm doing this in a private browsing session, and logging into lucidchart using username+password, not Google SSO, then connecting initially with auth before changing to auth2 as I've described before.

@timsama did you try adding response_type: 'permission' in the gapi.auth2.authorize call?

@TMSCH Yes, I tried adding , response_type: 'permission' to the authorize call, and it no longer gives me the account selection or increased permissions dialogs, but I'm still getting the "not all scopes are approved" error.

@timsama do you mean that you are getting the "not all scopes are approved" at the end of the sign in flow?

Actually, I'm not able to reproduce the "not all scopes are approved" error. It was being returned when I called authorize (with prompt: none), but it isn't happening now.
What is happening now is that the authorization appears to work, and I get an access token, but when I try to actually use the token to do a GET on the sheets data (with the authorization header set to "Bearer " + access_token), I get a 401 with the message "Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project."

@TMSCH Apologies for the confusing responses. It's been difficult to nail down exactly how to reproduce the 401, but I think I've got it now. So here's what I do to consistently reproduce this (this is with your recommended response_type: permission change):
Part 1: The Good

Open a new incognito browser
Log into Lucidchart (with username/password, not Google Sign-in)
Set the flag to use gapi.auth.authorize and open the page that loads from Google Sheets
Can't load Sheets data (because I'm not signed in to Google), Google sign-in dialog appears
Enter email and password, complete Google 2-factor authentication
Data from Sheets now loads correctly
Set the flag to use gapi.auth2.authorize and refresh the page
Data from Sheets still loads correctly

Part 2: The Bad
9) Delete Lucidchart's permissions from myaccount.google.com/permissions
10) Set the flag to use gapi.auth.authorize again and refresh the page
11) Can't load Sheets data, account selection dialog appears (not the same as the initial sign-in dialog)
12) Select my account
13) Permissions dialog appears
14) Approve permissions
15) Data from Sheets now loads correctly
16) Set the flag to use gapi.auth2.authorize again and refresh the page
17) Now the GET requests to sheets.googleapis.com return 401
Part 3: The Ugly
18) Delete Lucidchart's permissions from myaccount.google.com/permissions
19) Refresh the page
20) Can't load Sheets data, account selection dialog appears (not the same as the initial sign-in dialog)
21) Select my account
22) Permissions dialog appears
23) Approve permissions
24) GET requests to sheets.googleapis.com still return 401
If I then end my incognito browsing session and start over from step 1, I can continually reproduce this sequence. It also appears that Part 2 and Part 3 can be done in any order as long as they follow Part 1. Additionally, you can just skip the gapi.auth.authorize part of Part 1 in order to do this more quickly if needed.

@timsama are you using gapi.client to load data from sheets? (if not, you should probably as it is easier to do such requests)
In Part 3., are you using gapi.auth2.authorize only?
Could you post here the GET requests URL done to the Google Sheets endpoint for each part and auth library?

@TMSCH No, I'm using our internal XHR client to construct a GET request. Yes, I am only using auth2 in Part 3. I'll follow up with a list of the GET request URLs once I'm finished constructing it.

@TMSCH It looks like the URLs are the same every time.
Part 1
6. https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk?ranges=Core Ex Bug Commitments&key=AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI&fields=properties.defaultFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern)%2Csheets(properties.sheetId%2Cdata(startRow%2CstartColumn%2CrowData.values(effectiveValue%2CeffectiveFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern))))

https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk?ranges=Core Ex Bug Commitments&key=AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI&fields=properties.defaultFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern)%2Csheets(properties.sheetId%2Cdata(startRow%2CstartColumn%2CrowData.values(effectiveValue%2CeffectiveFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern))))

Part 2
15. https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk?ranges=Core Ex Bug Commitments&key=AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI&fields=properties.defaultFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern)%2Csheets(properties.sheetId%2Cdata(startRow%2CstartColumn%2CrowData.values(effectiveValue%2CeffectiveFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern))))

https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk?ranges=Core Ex Bug Commitments&key=AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI&fields=properties.defaultFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern)%2Csheets(properties.sheetId%2Cdata(startRow%2CstartColumn%2CrowData.values(effectiveValue%2CeffectiveFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern))))

Part 3
24. https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk?ranges=Core Ex Bug Commitments&key=AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI&fields=properties.defaultFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern)%2Csheets(properties.sheetId%2Cdata(startRow%2CstartColumn%2CrowData.values(effectiveValue%2CeffectiveFormat(backgroundColor%2CtextFormat.foregroundColor%2CnumberFormat.type%2CnumberFormat.pattern))))

@timsama would you have any publicly accessible staging environment where I could reproduce the issue? Or a snippet of code that reproduces it?

@TMSCH Yes, I'll get it set up for you once my security analyst gives me the OK (which might not be today, since it's 5pm).
On another note, I tried using gapi.client.request to get the data from Sheets instead of creating the request with our XHR client, and I'm still seeing the same error.

@timsama Thanks. I am not able to reproduce such issue with Google a API. In the meantime, could you give me the snippet of code using gapi.client that requests the sheets endpoint?

Assuming you have an access token, this is what I was using (most values hardcoded for brevity)
return gapi.client.request({
    'path': 'https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk',
    'method': 'GET',
    'params': {
        'ranges': 'Core Ex Bug Commitments',
        'key': 'AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI',
        'fields': 'properties.defaultFormat(backgroundColor,textFormat.foregroundColor,numberFormat.type,numberFormat.pattern),sheets(properties.sheetId,data(startRow,startColumn,rowData.values(effectiveValue,effectiveFormat(backgroundColor,textFormat.foregroundColor,numberFormat.type,numberFormat.pattern))))'
    },
    'headers': {'Authorization': 'Bearer ' + this.accessToken}
}).then(parseJson).catch((error) => {
    console.log(error);
});

Actually, this version is probably better:
return gapi.client.init({
    'apiKey': 'AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI',
    'clientId': this.clientId,
    'scope': 'https://www.googleapis.com/auth/drive.readonly https://www.googleapis.com/auth/spreadsheets',
}).then(() => {
    return gapi.client.request({
        'path': 'https://sheets.googleapis.com/v4/spreadsheets/1fYfJlg8r2mPsgQSa57MjPEzbFGKHbZlC1ho_aTvtdkk',
        'method': 'GET',
        'params': {
            'ranges': 'Core Ex Bug Commitments',
            'key': 'AIzaSyAygv9ePDxCV1Ogdtq5PZ5Rnd1USYkpdWI',
            'fields':
                'properties.defaultFormat(backgroundColor,textFormat.foregroundColor,numberFormat.type,numberFormat.pattern),sheets(properties.sheetId,data(startRow,startColumn,rowData.values(effectiveValue,effectiveFormat(backgroundColor,textFormat.foregroundColor,numberFormat.type,numberFormat.pattern))))'
        },
        'headers': {'Authorization': 'Bearer ' + this.accessToken}
    });
}).then(parseJson);

@TMSCH Ok, the environment is set up. Can you email me to exchange information out of band<EMAIL_ADDRESS>

@TMSCH Some more weird things. If I perform Parts 1 and 2, and then open the same page in a new tab, everything works in the new tab, and stays broken in the old tab. Even when I refresh both tabs, the behavior is consistent. The requests to sheets.googleapis.com in both tabs appear to be identical except for the authorization header, in which the bearer tokens are different.

After offline talk and further analysis with @timsama, the problem appeared to be due to the local caching of tokens performed by gapi.auth2 (and not gapi.auth). After removing the permissions granted to the application by the user, the token remains in the local cache and will be used the next time gapi.auth2.authorize({...prompt: 'none'...}) is called, even if the token is not valid anymore. This is an edge case that can arise in QA or manual tests but is very unlikely to appear in real scenarios.

So what is the fix, because I am getting the same problem when trying to refresh token. {"error":"IMMEDIATE_FAILED","detail":"Not all scopes are approved."}

@GEMI could you obtain a new issue with a description of the issue you're facing, and a snippet of code? Thanks!

Hi , I just Have A Question About a problem I Have , And I Hope From to Help Me To Fix It .
In Fact , My Ionic App Show Me A message fills the screen when trying to test the app on my browser , it says : "Runtime Error Cannot Read property 'src' of null" And Also :
TypeError: Cannot read property 'src' of null
at Object.eval [as updateDirectives] (ng:///HomePageModule/HomePage.ngfactory.js:37:102)
at Object.updateDirectives (http://localhost:8100/build/vendor.js:13114:72)
at checkAndUpdateView (http://localhost:8100/build/vendor.js:12627:14)
at callViewAction (http://localhost:8100/build/vendor.js:12995:21)
at execEmbeddedViewsAction (http://localhost:8100/build/vendor.js:12953:17)
at checkAndUpdateView (http://localhost:8100/build/vendor.js:12628:5)
at callViewAction (http://localhost:8100/build/vendor.js:12995:21)
at execEmbeddedViewsAction (http://localhost:8100/build/vendor.js:12953:17)
at checkAndUpdateView (http://localhost:8100/build/vendor.js:12628:5)
at callViewAction (http://localhost:8100/build/vendor.js:12995:21)
at execEmbeddedViewsAction (http://localhost:8100/build/vendor.js:12953:17)
at checkAndUpdateView (http://localhost:8100/build/vendor.js:12628:5)
at callViewAction (http://localhost:8100/build/vendor.js:12995:21)
at execComponentViewsAction (http://localhost:8100/build/vendor.js:12927:13)
at checkAndUpdateView (http://localhost:8100/build/vendor.js:12633:5)
at callViewAction (http://localhost:8100/build/vendor.js:12995:21)
So , What Should I Do ???

@naoufalhim are you using gapi.auth2?


Documentation on AWS Marketplace listing

Share the steps on how one can use AWS marketplace app.

Hi @pranavrajs I am a technical author hoping to make documentation-related contributions to open source projects on GitHub as I am new to this platform. Is this something you need help with? Thanks, Salman

@pranavrajs I am not familiar with AWS market place but eager to learn.

@SalmanRashid1979 Thank you for your interest in contributing to Chatwoot. Since you are not familiar with AWS/ marketplace , let me take the lead on this one.
Meanwhile, you could start by reading through our existing documentation and trying out Chatwoot on your own. Let us know if you have questions and that will be an opportunity to improve our docs.


Compression broken in 3.0

I am opening this issue now and will update as I dig into it further but so far I am not able to use compression at all since the 3.0 release. For now, disabling it has fixed the issue.
The way I'm enabling compression is like this:
$websocket = new Websocket(
    $server,
    $logger,
    new Rfc6455Acceptor(),
    $clientHandler,
    new Rfc6455ClientFactory(
        new Rfc7692CompressionFactory(),
        new PeriodicHeartbeatQueue(),
        null //disable rate limiting
    )
);

Sending the header Sec-WebSocket-Extensions: permessage-deflate; client_max_window_bits. from the client results in no extensions header response from the server. Just
HTTP/1.1 101 Switching Protocols.
connection: upgrade.
upgrade: websocket.
sec-websocket-accept: 6rnHuC9ZEBsJy6Cn5SHQyIGuzLE=.

I'm pretty sure the issue is caused by the introduction ofRfc6455Acceptor. Rfc6455ClientFactory sets the header and in testing, I can verify that code is executing and setting the header on client creation however the actual HTTP Response does not have the header in it.

Potential fix: https://github.com/amphp/websocket-server/compare/3.x...fix-compression
The compression context factory will also need to be passed to the acceptor.

That fixes it, but there still seems to be an issue depending on the header sent by the client.
Working:
Sec-WebSocket-Extensions: permessage-deflate.

Not working:
Sec-WebSocket-Extensions: permessage-deflate; client_max_window_bits.

The difference in HTTP response is the presence of the Sec-WebSocket-Extensions header in the latter case.

What is the header sent by the server in the case which isn't working?

Full response below:
HTTP/1.1 101 Switching Protocols.
connection: upgrade.
upgrade: websocket.
sec-websocket-accept: y6cI8Vt16BtUOS53g14ZIVb6oeE=.

Ah, that's because the client_max_window_bits is requiring a value, but it seems that value is optional. This will need fixing in the factory.

I can reopen the issue there if you like. It seems the issue affecting this package is fixed in https://github.com/amphp/websocket-server/compare/3.x...fix-compression

Sure, it will remind me to take care of it.
FYI, after discussing this issue a bit with @kelunik we are considering deprecating the compression factory parameter to the client factory and not supporting compression with 3.0 and releasing a 4.0 with an interface which combines WebsocketAcceptor and WebsocketClientFactory.

What would it take to help keep compression working for now? Refactoring interfaces is fine but I am getting some pretty big bandwidth savings on compression of JSON messages > 80KB and most of my clients are mobile on unreliable networks so minimizing bandwidth is important.

You can keep compression working for now by creating a WebsocketAcceptor class like Rfc6455Acceptor in the https://github.com/amphp/websocket-server/compare/3.x...fix-compression branch. Pass the same compression context to the acceptor and the client factory.

@bennnjamin I just tagged v4.0.0-beta.1 which refactors how compression is supported. The WebsocketCompressionContextFactory instance should be passed to the Websocket constructor instead of the client factory constructor.
Please give this new version a try and provide any feedback you may have. Thank you!

Tagged 4.0.0 with fixed compression support.


Unable to see if there are updates or not

Seafile shows not available under version for some reason. Have tried force update but still won't show up-to-date.

Oh I've just posted a new issue about this before reading yours:
https://github.com/chirmstream/unraid-templates/issues/4#issue-2573824022
Update the template's repository to seafileltd/seafile-mc:11.0-latest

Worse than that, Unraid plugin "Fix Common Problems" is also warning:
seafile (seafileltd/seafile-mc:latest) has the following comments: No longer available on dockerHub Additionally, this application has been blacklisted from Community Applications for that reason.
Have applied manual fix, but this could trip people up.

Two reasons why I haven't fixed this yet, I'd like like to hear some of other people's thoughts on this.

I've been busy, ha
I have had issues in the past when I use using Nextcloud and updating before I was ready.  What happened with me was there was a Nextcloud update, say version C, that required you to be on version B before updating.  The problem was, I hadn't updated for a while and was still on version A.  I pulled down the new docker image without reading the changes, and Nextcloud wouldn't start because I skipped too many versions.  Pulling down my previous docker image wouldn't help, it was too late and it wouldn't run without manually restoring all the files, settings, etc.  In the meantime my household can't use it because it broke when I tried running the new docker image.

Personally, I'm just grateful to have SeaFile consistently working. I probably wouldn't have noticed the issue myself had I not had a power failure and UnRAID taken me straight to "Fix Common Problems" with red errors.
There's always going to be issues beyond your control, set-ups that aren't updated or using an unsupported/outdated DB to host. You can only do so much.
Whatever side the coin lands on, my personal thanks for your work on this project.

Worse than that, Unraid plugin "Fix Common Problems" is also warning: seafile (seafileltd/seafile-mc:latest) has the following comments: No longer available on dockerHub Additionally, this application has been blacklisted from Community Applications for that reason. Have applied manual fix, but this could trip people up.

What manual fix did you apply?

What manual fix did you apply? Are you talking about the :latest thing? My seafile shows up-to-date now, but I do still have the blacklisted message

My repo is :
seafileltd/seafile-mc:11.0-latest
That was the only change I've made. No issues on my Fix Common Problems since.

What manual fix did you apply? Are you talking about the :latest thing? My seafile shows up-to-date now, but I do still have the blacklisted message

My repo is : seafileltd/seafile-mc:11.0-latest
That was the only change I've made. No issues on my Fix Common Problems since.

Gotcha, I did update the repo setting and now it shows up to date, but I've been getting a new issue with the apps section. I get a notification in action centre anytime i go to apps with the following about Seafile.

Do you think it's better to have different templates for each Seafile version, or to setup different branches on one template for each version.
Option 1

Option 2

I haven't setup a template with multiple branches yet, but if someone wants to try, here is the documentation I found for it.

All,
I've added two new templates for seafile 10 and seafile 11 with the fixed docker registry. I was able to successfully do a fresh install and update to v11 on my unraid sever. I've closed this issue as it should be resolved now.
Thank you

Thanks for that! On a side note, is there anyway to fix the blacklist of the app or is that just expected from using a custom template to install it?

I think after a few days it will update and should no longer be blacklisted.


WISP 0.0.5 not on maven central

I realize this project is no longer under active development, but could you release 0.0.5 artifacts on the maven central repository? The current version is 0.0.4 https://repo1.maven.org/maven2/com/quantifind/wisp_2.10/
Please and thanks!

@ryanlecompte  :-)


Bigquery Query - Can not use DECLARE when a destinationTable is provided

Expected Behavior
When I use DECLARE to implement some functions or variables for my query, I want it to work with my query that will fill my table.
Actual Behaviour
Instead, it tells me that I cannot set destination table with Scripts

Steps To Reproduce

Create a bigquery.query task
Use a DECLARE statement in the query
Execute your flow

Environment Information

Kestra Version: 0.5.2
Plugin version: 0.5.0

Example flow

id: my_id
type: com.leroymerlin.dataplatform.gcp.bigquery.Query
destinationTable: project.dataset.table
sql: |
DECLARE X int64;
set X=3;
Select X

Expected behaviour,
You are using BigQuery Scripting, as soon as you are using this, you can't use destinationTable any more and you should do instead INSERT INTO tableA SELECT * FROM tableB


Can zv-1 be installed?  I really need to change the language!  Help！

I also bought the Sony ZV-1 in other places, and the system does not have my native language.  Can the new camera be used?  Is there a way to solve?  Although ZV-1 should not be considered a new machine w

In the latest development build, tweaks should now be supported for this camera, too. You can use pmca-console serviceshell or the "Start tweaking (service mode)" button in the Tweaks tab of pmca-gui. This will require the installation of custom USB drivers.


SelfQuery support for deeplake

Added support SelfQuery for Deeplake

lgtm, lint error aside!

fyi you can run make format and make lint to run formatting and linting locally

thanks @adolkhan!


Error import

root@kali:~/SigPloit# python sigploit.py
Traceback (most recent call last):
File "sigploit.py", line 16, in
from ss7.tracking import *
File "/root/SigPloit/ss7/tracking.py", line 14, in
import sigploit
File "/root/SigPloit/sigploit.py", line 21, in
from gtpmain import *
File "/root/SigPloit/gtpmain.py", line 12, in
import gtp.info
File "/root/SigPloit/gtp/info.py", line 18, in
from attacks.info import discover_gtp_nodes
ImportError: No module named attacks.info
This error is comming

same to me that error sigploit come and help us to fix

Hi dude. Write to the mail<EMAIL_ADDRESS>пт, 26 черв. 2020, 12:27 користувач aahmedy<EMAIL_ADDRESS>пише:

same to me that error sigploit come and help us to fix
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SigPloiter/SigPloit/issues/190#issuecomment-650083900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM7WOXR4G2554DDTM2XNYJTRYRSYZANCNFSM4MNVF5ZQ
.

root@kali:~/SigPloit# python sigploit.py
Traceback (most recent call last):
File "sigploit.py", line 16, in
from ss7.tracking import *
File "/root/SigPloit/ss7/tracking.py", line 14, in
import sigploit
File "/root/SigPloit/sigploit.py", line 21, in
from gtpmain import *
File "/root/SigPloit/gtpmain.py", line 12, in
import gtp.info
File "/root/SigPloit/gtp/info.py", line 18, in
from attacks.info import discover_gtp_nodes
ImportError: No module named attacks.info
This error is comming

i can solve this man >>.hit me up on
whatsapp=======+15203950788
<EMAIL_ADDRESS>icq===https://icq.im/rickorkins

same to me that error sigploit come and help us to fix

i can solve this man >>.hit me up on
whatsapp=======+15203950788
<EMAIL_ADDRESS>icq===https://icq.im/rickorkins


FrontEnd: CSS Layout_Positions

Description 📜
Here it's going to be a documentation.
It will explain what is CSS Layouts and all the information related to positions.

Domain of Contribution 📊

[x] Frontend Dev (HTML, CSS, JS)

/assign

/assign

/assign


Uncaught Error: dlopen(~/Desktop/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keybo...

getting this error each time I keydown.
Atom: 1.45.0 x64
Electron: 4.2.7
OS: Mac OS X 10.15.4
Thrown From: Atom Core
Stack Trace
Uncaught Error: dlopen(/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
At ELECTRON_ASAR.js:160

Error: dlopen(/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:14:2781650)
    at get_manager (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:14:2781752)
    at Object.getCurrentKeyboardLayout (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:14:2781878)
    at e.keystrokeForKeyboardEvent (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:14:1083882)
    at KeymapManager.keystrokeForKeyboardEvent (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:11:1251844)
    at KeymapManager.handleKeyboardEvent (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:11:1249829)
    at WindowEventHandler.handleDocumentKeyEvent (/Users/chrismorgan/Desktop/Atom.app/Contents/Resources/app/static/<embedded>:11:284496)

Commands
     -2:01.4.0 tree-view:remove-project-folder (span.name.icon.icon-repo)
     -1:17.5.0 application:add-project-folder (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-one-dark-syntax.theme-one-dark-ui)
  2x -0:49.3.0 tree-view:add-file (span.name.icon.icon-file-directory)

Non-Core Packages
atom-beautify 0.33.4
atom-html-preview 0.2.6
atom-ternjs 0.20.0
autoclose-html 0.23.0
busy-signal 2.0.1
csslint 1.2.0
emmet 2.4.3
hyperclick 0.1.5
intentions 1.1.5
language-babel 2.85.0
language-ejs 0.4.0
linter 2.3.1
linter-eslint 8.5.5
linter-ui-default 1.8.1
pigments 0.40.2
Sublime-Style-Column-Selection 1.7.5

Hey @ChrisGuy, thanks for reaching out
This seems like a duplicate of https://github.com/atom/atom/issues/20401, caused mainly by a faulty installation.
I will go ahead and close this one, feel free to comment or open a new issue if the problem still persists.


NWPS-1637-BE-Training-homepage-updates

Change Summary and Caption names to Subtitle Page and Programme Description. Change the maximum lengtH of summary to 120

This pull request is superseded by  new PR. https://github.com/Health-Education-England/hee-cms-platform/pull/496


Would like to see redirect_to use urls and not paths

The gem's redirects don't use full urls and also according to #188 it is ignoring options.
It would also be desirable that the urls method(s) were to default to using the default_url_options host and protocol settings.

Does wicked not use default_url_options? It builds links with url_for which should pull in those options.

Confirm/deny?


Add bindings to DirectShow stuff

Only failing on -gnu which can't find strmiids.lib :-/

I probably just need to update the bundled import libraries for the -gnu targets.

@retep998 any update on this?

@retep998 do the update thing.

MinGW import libraries have been updated to 10.0.18362.0

Also you should not be depending on strmiids.lib as that is a static library containing only GUIDs, and winapi prefers to define all GUIDs inline.

@retep998 the deed is done

Added some missing constants

Good job, really looking forward to this:)!


Adding space to connected k8s error

Summary
This pull request addresses a minor formatting issue in an error message within the project.
Description

Issue: The error message Problem loading the kubeconfig file.Invalid kube-config. /root/.kube/config file is empty lacks a space after the period in file.
Fix: Added a space to correct the formatting. The updated error message now reads: Problem loading the kubeconfig file. Invalid kube-config. /root/.kube/config file is empty

Impact
This change improves the readability and clarity of the error message displayed to users, ensuring better user experience and understanding of the issue.

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

az connectedk8s connect
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


prettify (format) PowerShell and XML blocks

fixing formatting of PowerShell and XML blocks for better readability, with appropriate line-breaks and code formatting tags.

@astaykov : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Merging in this repository is suspended until Tuesday, May 19. Merging will resume at 7:00 AM PT Tuesday. This PR is ready to merge. Thanks.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


setting options to function in the modules, but there is no way to get the options elsewhere

Environment
Working directory: /Users/elonehoo/Downloads/my-module                                                                                11:51:21
Nuxt project info: (copied to clipboard)                                                                                11:51:21

Operating System: Darwin
Node Version:     v20.10.0
Nuxt Version:     3.10.3
CLI Version:      3.10.1
Nitro Version:    2.9.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new                                                                                11:51:21
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction
https://github.com/elonehoo/nuxt-modules-issues
Describe the bug
after allowing it, there is no way to get to my Modules.plugins in plugin

Additional context
i'm trying to add a plugins method to options, but in plugins I can only get the undefine
Logs
No response

can get it when you first start up, but you can't get it in the plugin

hhhh，sorry, i completely missed this section, thank you very much for the answer, next it's time for me to get on with it


Decouple symfony

A start on #22, at least this way people can use the basics without pulling down all of symfony

Any movement on this?

This should really be merged in.

I'd really like to separate this out into a library and bundle ideally - that's why it hasn't been merged in yet. But if you're happy for it to be as-is without Symfony being a dependency then I'll re-look at it.

I would be happy with it as is without symfony. We are using this like a library right now and just autoloading everything and using the classes that we need. I think it would be a good solution in the mean time to drop symfony/symfony while it's being ported into a lib.

I've been using it for just over two years like this :)
Rebased if you do want to merge

OK that sounds good - thanks all :)

Can we tag this as a 1.6 release?

Done - 0.1.6


Filter - settings reorder filter sets

Widget
Filter
Version of widget
227c471
Bug or Enhancement
Bug
Repo Steps or Enhancement details

Add a new Filter widget
Specify a filter set name
Add new query task & choose a layer
Add a new filter set
Move the filter set up
Note the information has been inserted into the Query Tasks table

disregard


Add Finch

@vkostyukov, @rpless, @sergeykolbasov, what do you think? I've started using this for circe and some other projects and while it's not perfect yet, I definitely think it's going to be helpful.

I'm 👍on this! Wanted to do that for a long time. Thank you, Travis!


Add controlled version of this

Unfortunately, many useful libraries, including React, make use of this.
Squiggle shouldn't encourage the use of this, but it does need to be possible still.
I'm thinking a named/scoped this via a special function parameter:
let mia = {
    "_name": "Mia",
    "getName": fn(@self) self._name
}
in console.log(mia.getName())

This variable would be usage checked just like any others, so you'd know as soon as you look at a function whether it uses this or not!

Another idea I had was just using some kind of object transformer that modifies all the functions attached to an object, but that sounds more complicated, and harder to debug.
let mia = bind({
    "_name": "Mia",
    "getName": fn(self) self._name
})
in console.log(mia.getName())


[FEATURE] Utilize OpenFeature providers in flagd

Description
The goal here being that flagd would be able to plugin Go providers as the backend, as opposed to the JSON configuration.
Namely, the goal here would be to use the operator model built around the reference OpenFeature implementation, but collectively configure the provider at the flagd level instead of individual codebases.
Reasoning
For adoption of OpenFeature, this massively shifts the overhead for implementing providers per language to primarily Go, where that provider could be utilized in the sidecar instance of services using flagd.
For instance, we use Python, NodeJS, and PHP backed by Split. We would need to implement a Split OpenFeature provider in every language in order to adopt OpenFeature, and in addition to that we would have the initial overhead of migration from the Split SDK to OpenFeature, with no architectural benefit to doing so.
In the case of deploying to Kubernetes with the Flagd operator, we can immediately benefit from pod-local access to Flagd and configure the operator at a namespace (or even container level) which provider it would use.
The operator pattern, or just deploying of Flagd as a service available to the K8s namespace, would immensely improve our overall flexibility through OpenFeature. For example, the provider for OpenFeature can be optimized for asynchronous communication (re: Split's PHP provider is entirely synchronous) with Flagd doing the heavy lifting of individual request handling.
General food for thought, as this would likely be a blocker for our org adopting OpenFeature as an enhancement to our current architecture.

Could you create an OFEP for this?

I read that but I don't understand :(

@AlexsJones I think, basically, @tcarrio is suggesting something we've talked about a few times (I've heard it brought up independently by a few people): using go providers as evaluators in flagd.
Basically, since the interfaces are very similar (see the ievaluator.go, it should be possible to use a go provider as an evaluator. This would then basically mean flagd is powered by whatever provider you've plugged in (ie: flagd powered by LaunchDarkly or flagd powered by Flagsmith)

Exactly this @toddbaert. Where this really comes into play, as an interested party of the operator, is that we can now introduce any OpenFeature provider as the evaluator but automatically utilize deployment strategies of flagd by the operator (pod sidecars, node-local, etc).
The gains exist for flagd as its own service of course, where we can now utilize Split or LaunchDarkly in production but optimize the configuration of flagd deployments instead.

Could you create an OFEP for this?

I could look into this as well @AloisReitbauer 🙏

@AloisReitbauer @toddbaert @AlexsJones: drafted an OFEP for this enhancement here.

Closing as this is considered out of scope.


v1.13 Backports 2023-04-03

[ ] #24034 (@jibi)
[ ] #23935 (@jschwinger233)
only backporting c08189cfb1ec as a dependency of #24309
[x] #24575 (@mhofstetter)
[ ] #24484 (@jschwinger233)
[x] #24619 (@giorio94)
[ ] #24506 (@ti-mo)
[ ] #24309 (@chenyuezhou)
[x] #24645 (@tklauser)
skipping as the commit it fixes has been reverted in master
[ ] #24497 (@ferozsalam)
[ ] #24646 (@MrFreezeex)
[ ] #24538 (@ti-mo)
only backporting 072968feb415 on author's request

Once this PR is merged, you can update the PR labels via:
for pr in 24034 23935 24575 24484 24619 24506 24309 24497 24646 24538; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=24034,23935,24575,24484,24619,24506,24309,24497,24646,24538

see individual commits for list of conflicts

/test-backport-1.13

reviewed @ti-mo commits as they are small and without conflicts
test-1.24-4.9 is failing due to a known flake, https://github.com/cilium/cilium/pull/24723 should help with that once merged
ingress-conformance-test is passing although for some reason it failed to report the state

Marking as ready


Cannot get voice attack to trigger speech responder.

I am not super keen on how to get my voice attack to trigger a speech responder.  In fact I am having a hard time figuring out where to read documentation on the speech responder at all.  I understand that when a certain action is noted, the speechresponder will just do the TTS for you based on the script you provide it.  However I cannot for the life of me figure out how to set what triggers a specific speech responder script.  Are these hard coded?  If so how does creating a new script make much sense?
Anyway, say I wanted according to the voice attack docs:
This function uses EDDI's voice to read a Speech Responder script. The name of the script should be a text variable with the name 'Script'.

If you want to use a different voice to the standard one then you can set the name of the voice you want to use in a text variable with the name 'Voice'. Note that when you set this variable it will continue to be used until you unset it, at which point EDDI will use the voice configured in its text-to-speech settings.

To use this function in your own commands use the 'Execute an external plugin function' command with the plugin context set to 'speech'.

So I have tried to get a specific speech responder line to kick off.  For instance, when I scan a star I don't want EDDI to immediately tell me about it, but if I do want more information I want to be able to ask.  So in my voice command where I ask I have:
Set Text [Script] to 'Star scanned'
Execute external plugin, 'EDDI 2.2.1'

And I am executing the function 'speech' in teh external plugin call.  However it will not execute the "Star scanned" speech responder script, I hear nothing.  Do I have a fundamental misunderstanding?

It sounds like you have things set up properly but hard to tell without looking at the VoiceAttack command itself.  Please could you export the command from VoiceAttack and attach the .vap file here?

speechresponder-Profile.zip
There is a profile with my one speech responder commander.  My speech responder is just using the default personality so everything is enabled.
Also can you quickly explain how triggers are established.  I am quite confused by that.  If I click on the "entered signal source" responder command I see the following:

Now I understand how it generates the response, I looked up the references you have added on how to construct these responses.  What I am having a hard time grasping is how a particular speech responder script is triggered.  The description for this is that it is triggered when entering a signal source, but I don't know where or how that is set.  Is it merely hard coded?  And if it is how do you go about establishing a new trigger for creating a new script?

The VoiceAttack profile provided works okay on my system.  What version of VoiceAttack are you using?
Regarding events, the graphic at https://github.com/cmdrmcdonald/EliteDangerousDataProvider shows the basics of how EDDI works.  Events are triggered by monitors that watch the journal log, galnet feed, etc.  The responders receive the events and act on the information provided.  In the case of the speech responder it looks for a template with the same name as the event, and if found it feeds it to the speech processor.
You can add your own scripts and call them from VoiceAttack.  If you have more complex requirements you'd probably have to build an EDDI plugin to generate events.

I am using the latest version of VA: v1.6.2.  I am scratching my head on this one.  You just loaded the profile I gave you and BAM, it just worked.... hmmmm.
Also when I open EDDI it says "close this window when VA is running, you can configure it inside by running "configure EEDI".  But I cannot figure out how to access "configure EDDI".  Running it as an external plugin function doesn't seem to open the configuration prompt.
Is there a list of events that all the different responders generate anywhere?  I cannot seem to find it.

There is a VoiceAttack profile for EDDI in the installation directory, you can install that for the 'Configure EDDI' command and a few others.
The documentation at https://github.com/cmdrmcdonald/EliteDangerousDataProvider/blob/master/VoiceAttack.md#using-eddi-with-voiceattack provides a list of all current events, or you can look at the speech responder in-game as that will generate an entry for every known event.

Oh ok I didn't realize the name of the speech was a trigger lookup as well, everything is starting to make sense.  I will go home and try the profile ASAP, (ETA ~3 hours).  Hopefully it works but I suspect it won't.  I'll post once I know.  Thank you so much for the help.

I restarted my machine and now running the speech command does appear to be working.  Thank you for the help and further explanations.

One more thing, as I notice there are only a few trigger functions in the voice attack profile, some are missing.  When you say look at the speech responder in-game, do you just mean look at the configuration and all of teh triggers will be listed under the EDDI personality?