[Python] Python enum types are not respected

Given a table with an Python Enum requires different access pattterns depending on if we filter on the database site via SQL (with the SQLAlchemy integration) or on the Python side just with OSO.
import enum
from sqlalchemy import coveting

class AccessLevel(enum.Enum):  # <- Python Enum
    public = "private"
    private = "public"

class Post(Model):
    __tablename__ = "posts"
    id = Column(Integer, primary_key=True)
    content = Column(String, nullable=False)
    access_level = Column(Enum(AccessLevel), nullable=False)
    # ^- Enum DB column, but with a Python Enum as value (SQLAlchemy does the translation automatically).

For just checking permissions on Python (authorize(user_obj, "read", post_obj)), I have to write this rule with .value, as post.access_level is of type enum.Enum.
allow(_: User, "read", post: Post) if
  post.access_level.value = "public";

However when I try to filter on the database side, with an authorized_sessionmaker, the rule doesn't work any more.
OSO parses the rule wrong, and believes that access_level is a relationship to another table with a column value.

Obviously it works if you define an Enum directly as in the https://docs.osohq.com/reference/frameworks/data_filtering/sqlalchemy.html tutorial, by using strings. With access_level = Column(Enum("public", "private"), nullable=False). However I think it's quite common to define the Enum separately on the Python side, and then reuse that enum in the ORM definition, and SQLAlchemy supports this since 1.1. It's just more safer, if you have Enums in the Python business logic, instead of having to deal with strings.

Probably a way too long bug report.
Just thought about it some more, and I guess the easiest solution is to just use:
class AccessLevel(str, enum.Enum):  # <- inherit both from str and Enum
    public = "private"
    private = "public"

Because then you can compare them by their string value as well.
>> AccessLevel.public == "public"
True

So you don't have to use .value in the policy file at all, and both methods work.
Not sure, if this should be documented somewhere.

Are you intentionally reversing the key and value names? SQLAlchemy uses the key name not the value unless you override the behavior:
https://docs.sqlalchemy.org/en/14/core/type_basics.html#sqlalchemy.types.Enum
Take a look at values_callable which I think is what you want

No. Was not intentionally. But I mostly let my Enums have the same names and values (as string enums), so I never noticed that default behavior from SQLAlchemy.
I think we can close this ticket.


Chromium HTML route not rendering provided CSS files

Chromium HTML route not rendering provided CSS files
Expected Behavior
Referencing the Gotenberg docs, I expect this to work:

I upload files (in my case, index.html, two css files, a png file, a js file, and a woff2 file) to the http://{hostname}:3000/forms/chromium/convert/html endpoint
Gotenberg generates a PDF document using these files, which should look like an invoice with a header including logo, page counters, and a table in the body.

Current Behavior

I upload files to the html endpoint
Gotenberg generates a blank PDF, 879 bytes, every time.

Investigation
Running Gotenberg with --log-level debug yielded these logs:
{
  "level": "debug",
  "ts":<PHONE_NUMBER>.8586862,
  "logger": "api.chromium.debug",
  "msg":"[0920/164714.858193:INFO:CONSOLE(0)] \"Access to XMLHttpRequest at 'file:///tmp/9c6c310e-b530-4120-820c-d239dae9df3a/bootstrap.min.css' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-untrusted, https.\", source: file:///tmp/9c6c310e-b530-4120-820c-d239dae9df3a/index.html (0)\n
[0920/164714.858303:INFO:CONSOLE(0)] \"Access to XMLHttpRequest at 'file:///tmp/9c6c310e-b530-4120-820c-d239dae9df3a/printable.css' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-untrusted, https.\", source: file:///tmp/9c6c310e-b530-4120-820c-d239dae9df3a/index.html (0)\n
[0920/164714.858367:INFO:CONSOLE(0)] \"Uncaught (in promise) #<ProgressEvent>\", source: file:///tmp/9c6c310e-b530-4120-820c-d239dae9df3a/index.html (0)\n",
  "trace": "f951bee9-bb1f-4e1a-94ea-303d935df037",
  "webhook": false
}

Indeed, this is the same error I get when opening index.html locally in Chrome on my PC:

Possible Solution
I have two ideas for solutions to this issue:
Solution 1: Run Chromium with override flag to use old behavior
After some searching, I found that Chromium needs to be run with the flag --allow-file-access-from-files to allow CORS, which is corroborated here. Testing this locally in Chrome on my PC did allow the document to render as desired. To me, this is the preferable solution, requiring no additional dependencies.
Solution 2: Run a small web server inside the Docker container with Gotenberg
If adding an additional flag to Chromium isn't desirable, running a small web server (nginx, etc.) using the /tmp directory as the web root would allow presumably allow this to work as well. I have not tested this, however.
Your Environment

Docker version 20.10.8, build 3967b7d
Windows Subsystem for Linux (WSL2)

Build: 18363
Branch: 19h1_release
Release: Fedora Remix for WSL
Kernel: Linux 5.4.72-microsoft-standard-WSL2

Hello @josephdicarlo1 👋
I just tested the HTML endpoint locally and via a demo URL, and it worked as expected (with these files).
Could you show me a sample of the index.html you're trying to convert 🤔 ?

Could you show me a sample of the index.html you're trying to convert 🤔 ?

Sure, here's a (redacted but otherwise unchanged) copy.

I also just tested it using https://live-demo-us.gotenberg.dev and got an identical result, down to the byte size.

The only effective difference I can find between my files and the example files is that I'm using a js file, so I tried removing it and attempting again, which resulted in a much larger file yet still 3 blank pages.

Thanks for the details!
That’s definitely weird though.. I mean, I don’t understand why it works just fine with my sample while with your example it does not work 🤔
Maybe because your CSS/JavaScript files are trying to call other local files?

@josephdicarlo1 After some investigations, it seems related to Paged.js, which requires A web server to let the polyfill access your CSS file.
I'll add a flag for --allow-file-access-from-files 👍

Cool, thanks! I spent a while yesterday trying to dig around for a cause but came up empty, so I'm glad you found an answer. 😄

BTW, I discovered that the client I was using for testing (Hoppscotch) was breaking the generated PDFs when I saved them, which explains why all the other examples were also showing up blank.
I did confirm to make sure that the cors issue was still the root cause of the original issue, but yeah, that led me to a lot of extra headache yesterday 🤦♂
Anyway, thanks again!

@josephdicarlo1 Gotenberg 7.0.7 has a new flag --chromium-allow-file-access-from-files that solves your issue 😄

I came to this post as we're attempting to use version 8 with pagedjs and we're encountering the dreaded blank pages. Has anyone else raised this issue recently? We've set the --chromium-allow-file-access-from-files=true in our configuration, but no change in the results. Any help would be appreciated.


Missing staticfiles manifest entry for 'reprohack/leaflet/leaflet.awesome-markers.css'

New sub-Issue of #37 (as of e977c45d8b0154beb78072b0c3e1c291064f94b1 on the dev branch):
django_1        |   File "/usr/local/lib/python3.8/site-packages/django/contrib/staticfiles/storage.py", line 423, in stored_name
django_1        |     raise ValueError("Missing staticfiles manifest entry for '%s'" % clean_name)
django_1        | ValueError: Missing staticfiles manifest entry for 'reprohack/leaflet/leaflet.awesome-markers.css'

Trying adding leaflet.awesome-markers 2.0.5 via package.json to see if that fixes this...

@annakrystalli @griff-rees The above resulted in npm sticking the missing file in node_modules/leaflet.awesome-markers/ beneath the Django root.  Does this dir need appending to STATICFILES_DIRS in config/settings/base.py?

Sorted with in 625d65afa8998cba2006113ec947a19bfaac2897. Ideally we do that for the rest of the javascript dependencies...


correct the comment for isRandomPort(). (#1256)

Signed-off-by: Jiangnan Jia<EMAIL_ADDRESS>Result:
Fixes #1256.

@EvenLjj  It looks something wrong with sofa-rpc-test-integration, maybe some test cases make the workflow error.

@EvenLjj It looks something wrong with sofa-rpc-test-integration, maybe some test cases make the workflow error.

Yes, try again first.


mypy complains about pre=True root validators

Bug
Please complete:

OS: Ubuntu 19.04
Python version import sys; print(sys.version): 3.7.3
Pydantic version import pydantic; print(pydantic.VERSION): 1.0

With the code found in the root validators documentation:
from pydantic import BaseModel, root_validator

class UserModel(BaseModel):
    username: str
    password1: str
    password2: str

    @root_validator(pre=True)
    def check_card_number_omitted(cls, values):  # type: ignore
        assert 'card_number' not in values, 'card_number should not be included'
        return values

Mypy has the following to say:
9: error: "classmethod" not callable

The following change pleases mypy, but of course that changes the semantics:
-    @root_validator(pre=True)
+    @root_validator

Thanks for reporting. Fixed by #985


The requested module<EMAIL_ADDRESS>is a CommonJS module, which may not support all module.exports as named exports.

I'm using sveltekit + vite, and when I run npm  run build, the following error came up. Here is an error I found:
import { ComputeEngine } from<EMAIL_ADDRESS>         ^^^^^^^^^^^^^
SyntaxError: Named export 'ComputeEngine' not found. The requested module<EMAIL_ADDRESS>is a CommonJS module, which may not support all module.exports as named exports.

Also npm run test:unit raises error:  (it uses vitest in the backend)

@Seancheey, which version of vite are you using?


base64: add base64url support

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Add base64url support in common utils, this is a dependency of JWT #2514 support.
Risk Level: Low
Testing: unit test
Docs Changes: N/A
Release Notes: N/A

Codewise, this seems fine. Is there a standard for this base64 variant that we can reference in the docs?

RFC4648 is the one, I'll add it to comment in header.

Add base64url support in common utils, this is a dependency of JWT #2514 support.
Risk Level: Low
Testing: unit test
Release Notes: N/A
Signed-off-by: <Lizan Zhou<EMAIL_ADDRESS>


Remove plugins

Which problem is this PR solving?

removing plugins
removing meta packages
updating readme
updating examples
downgrading karma from ver. 5 to 4 - it doesn't work locally

I think the history is brokee, there are commit from main on the PR :/

lgtm, just need to fix history

I had cherry picked the changes and created new PR -> https://github.com/open-telemetry/opentelemetry-js-contrib/pull/429


Correct spacing between closing delimiters and large operators

Closes #462
Low hanging fruit. This was just a missed case in the implementation of TeX spacing (TeXBook, p170).

Thanks!


Fixed #32296 -- Added --skip-checks option to runserver command.

https://code.djangoproject.com/ticket/32296

@hramezani Thanks :+1: I added docs changes.

@hramezani Thanks :+1: I added docs changes.

I fixed isolation of test_check_errors_catches_all_exceptions in a separate commit.

I fixed isolation of test_check_errors_catches_all_exceptions in a separate commit.


GDScript: Make annotations register statically in parser

So it only register the annotations once per run instead of doing it every time a script is parsed.

Thanks!


Events Segmentation Issue Not getting complete data

Hi,
I am passing all these data in the segmentation parameter of the recordEvent method but when i am checking on the console, i see always only 5 of these values.
val hashmap = HashMap<String, String>()                 hashmap["loginStatus"] = "No"                 hashmap["category"] = "Search"                 hashmap["deviceId"] = "74fae12e689a18c2"                 hashmap["action"] = "SearchType"                 hashmap["lastThreeSearches"] = "No"                 hashmap["city"] = "Delhi"                 hashmap["positionInEtype"] = "2"                 hashmap["section"] = "All"                 hashmap["userId"] = "4919999"
Countly.sharedInstance().events().recordEvent(key, newMap);
On the console:
{    "app_key": "Rahul_Android",    "hour": "15",    "dow": "1",    "events": [       {          "key": "[CLY]_view",          "count": "1",          "timestamp": "1636971932708",          "hour": "15",          "dow": "1",          "segmentation": {             "name": "HomePage"          },          "sum": "0"       },       {          "key": "[CLY]_view",          "count": "1",          "timestamp": "1636972057507",          "hour": "15",          "dow": "1",          "segmentation": {             "name": "D_You"          },          "sum": "0"       },       {          "key": "MYTESTSERACH",          "count": "1",          "timestamp": "1636972058190",          "hour": "15",          "dow": "1",          "segmentation": {             "deviceId": "74fae12e689a18c2",             "userId": "4919999",             "city": "Delhi",             "lastThreeSearches": "No",             "category": "Search",             "action": "SearchType"          },          "sum": "0"       }    ],    "sdk_version": "16.12.3",    "sdk_name": "java-native-android",    "device_id": "1de50d6294b196d1",    "checksum": "8628107250165465fc6d2d5dbf885426f7c39848",    "metrics": {       "_device": "EB2101",       "_os": "Android",       "_os_version": "11",       "_carrier": "airtel",       "_resolution": "1080x2156",       "_density": "other",       "_locale": "en_US",       "_app_version": "10.10.4"    },    "serverTimestamp":<PHONE_NUMBER>980 }
as you can see the event "MYTESTSERACH" has 9 values but i am getting only 5

Hello, please enable logging in the SDK and send the generated SDK logs. You would send the logs from the moment that the SDK is initialized, to slightly after you have recorded your event.

Fixed at my end


Compose fails to build backend

Application fails to start, datasource bean throws exception for invalid driver class.
Unsure if invalid beans are a cascading failure from datasource, of if there are multiple issues.
Steps to reproduce the behavior:
Run docker compose up -d to create stack
Review backend logs from EDPN stack in docker.

#44 is open for review


fix: remove empty CRD yaml

Fixes #4497

Release Note
Fix CRD installation when using Helm

thanks @squakez for taking a look!
could you elaborate on "completely removing" the files?
in the meanwhile i'll take a look at the failed tests

Tests were failing because of some infra problem that should have gone. Don't worry about them. What I meant is that the file should be completely removed. It seems to be a leftover from the previous renaming from KameletBinding to Binding which eventually became Pipe.

I'm colsing this in favour of #4510 - the problem is in the release process which is not taking in account properly the generation of charts when run from release branches.


Feature/default test for fixed parameters

We noticed for custom optimizer implementations that it could make sense to ensure that they respect fixed parameters, so this PR introduces a reference test that helps spot deviations from that expected behaviour.
I also added missing type hints and fixed docstring typos in the reference tests - if you'd like to de-clutter the diff, consider looking at the individual commits one by one, they are almost nicely curated (the one to spot the odd one out gets a drink :smile:).

I'll squash and sign-off to comply with DCO after the review.

Was there a specific incident that made you think that this might be broken or is this purely a safety measure?
Ideally, the optimizers don't see any fixed parameters, if they rely on the numerical representation. Everything should be handled in ParameterSpace.


Data requiring

I would like to request basic information regarding the control groups (consisting of a total of 349,539 cases) associated with aortic dissection or aortic aneurysm. Specifically, I am interested in obtaining information about the gender distribution and age demographics of these control groups. Thanks.

Hello,
Thanks for your question.
I am afraid this information is not directly accessible.
However, since these 2 endpoints have a rather small number of cases you can get a sense of the gender balance of their control groups by looking at a very broad endpoint.
For example, the endpoint ANY_HILMO has 209784 females and 164334 males.
Hope this helps!


feature: Pod sub-resources access via APIExport virtual workspace API server

Feature Description
It is now possible to query pods resources via the virtual workspace API server of an APIExport that claims permission on pods resources.
However, it is not possible to query sub-resources, like pods/log, pods/proxy, ...
Proposed Solution
It should be possible to claim permissions on pods sub-resources, so these are seamlessly accessible from the claiming APIExport virtual workspace API server, once accepted.
Alternative Solutions
No response
Want to contribute?

[ ] I would like to work on this issue.

Additional Context
No response

/transfer-issue contrib-tmc

/transfer-issue contrib-tmc


Added metrics endpoint - Closes #137

Added a new endpoint with Prometheus metrics using prometheus-net library.
A HTTP request middleware was created and placed into a new folder called Metrics. Required injections were added to Startup.cs

Reviewed the code in order to use prometheus-net built-in HTTP request middleware, instead of creating a personalized one.

@cesinhaugusto may I add your name to the contributors list in the readme page?

Of course @ivanpaulovich! I'm excited to contribute 😃

@all-contributors please add @cesinhaugusto for code.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/73293

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/73294

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/73802

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74011

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74484

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/75179

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/75218

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/75247

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/76079

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79375

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79504

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99322

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99891

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104813

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/105060

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/105096

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/105510

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/105682

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106334

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/118943


Supports Github commit suggestions?

I'm not familiar with Github Actions yet, but I have seen and used the Github suggestion codefence syntax, which lets PR reviews easily enable the author to apply the changes without copy/paste.
I'm familiar with CI tools having a lint stage/step or some projects having a pre-commit git hook to ensure code sticks to convention. Some tools like textlint however could be nicer to have brought up as suggestions after the PR is submitted and conditionally applied or discussed instead of overriding the contributors original commit data.
I saw your article that discusses your creation of this repo, and you had mentioned "If you use a GitHub suggestion", but I don't see anything that indicates that this project outputs the lint results as commit suggestions for a PR? Were you referring to something else?

When I spoke about Github suggestion I wanted to say only about the Github actions. In some place on the documentation (which I cant find right now), they put that for you to create the lint you need to do something like I wrote in my article and I made this repo just for abstracting some things.
I don't want to produce the same result as textlint (I didn't know this tool, it is very nice), my idea is to just create a first defense for PRs.

I assume @polarathene was more meaning it could offer annotations or comments with suggested fixes, rather than forcing the user to do it manually.
FWIW @polarathene , when I looked at this before it doesn't seem very easy to do suggestions (compared to annotations, or even as part of them). From what I can tell they are only possible in comments. The only options I found before, which both suggest they may have flaws, are:
https://github.com/marketplace/actions/action-git-diff-suggestions
https://github.com/marketplace/actions/reviewdog-suggester-suggest-any-code-changes-based-on-diff-with-reviewdog


Gas opt: skip remaining calculation when not necessary

Motivation
Critical code paths for order fulfillment are preemptively computing duration, elapsed and remaining timestamps for all orders.
In particular, remaining can always be derived from the other two, and it's only actually necessary when startAmount != endAmount. For static-price orders, it ends up never actually being used (both _locateCurrentAmount and _applyFraction branch out and never use it).
This opens the door to a couple of optimizations:

we can remove this argument from internal calls, and derive it from duration and elapsed when necessary
we end up not actually computing it at all when startAmount == endAmount, which should be a significant number of orders

yarn coverage of the relevant section (before and after shown below) shows gains of 130 to 160 gas for all fulfill* and match* calls.

Now defering computation of duration and elapsed as well, as per my previous comment
For now deferred the decision of adding unchecked, since it warrants more review, and already generated some comments on #355

@0age fixed all pending comments, and included underflow noticed as requested (actually re-used the notice from #355)
as for the test suite, it was failing to compile due to a "stack too deep error", which I fixed. However, the forge suite is failing for me locally, for reasons I haven't yet understood. It fails on main as well though, so I don't think it's a problem here. can we try re-runing the action?

Rerunning forge tests now!

While we're working out what's going on with the forge tests, just wanted to also flag an implication of this change: for orders with multiple ascending / descending amount items, these parameters will now be calculated multiple times. It does seem like most items won't have ascending or descending amounts, and the orders that do have an item with ascending or descending amounts usually won't have more than one, but just wanted to flag it and hear what you think.


test in demo

https://github.com/GIScience/Leaflet.Heightgraph

use properties https://github.com/opengeo-tech/geopicker/issues/8

https://github.com/stefanocudini/leaflet-locationpicker


[JOSS Review] Fix typos in docs

@frank1010111 This is a very tiny PR for https://github.com/openjournals/joss-reviews/issues/5255 that will fix some typos in the docs caught by codespell before we have you make a new release for archival purposes (hold off on that though until I request it in the review just in case anything else comes up).

Fix typos:

'assymptote' -> 'asymptote'
'availalbe' -> 'available'

Apply pre-commit hook updates that don't require code changes when applied.

Codecov Report

Merging #30 (a07ec24) into main (92fc483) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   89.69%   89.69%
=======================================
  Files          13       13
  Lines         786      786
=======================================
  Hits          705      705
  Misses         81       81

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I see codespell has improved its ability to check notebooks.

Actually that one I'm not sure on. I only found those by manually running
codespell .

from the top level of the repo, where it picks up also many false positives from the binary information in the notebook JSON from images. So without actually giving it some real thought I'm not sure how to improve that.

Yeah, sadly still an outstanding issue: https://github.com/codespell-project/codespell/issues/2138


Responsive mixins

Hello!
I'm wrote responsive mixin for my project:
same = false // synonym for better experience

browser(name, value)
    add-property name, value

tablet(name, value)
    @media (max-width: 979px)
        add-property name, value

phone(name, value)
    @media (max-width: 767px)
        add-property name, value

responsive(name, browser-value, tablet-value, phone-value = same)
    if s("%s", name) == "image"
        // hack for image support, not interested now
        -image-responsive browser-value tablet-value phone-value
    else
        browser name browser-value

        if tablet-value is not same and tablet-value != browser-value
            tablet name tablet-value

        if phone-value is not same and phone-value != tablet-value
            phone name phone-value

Usage example:
.logo
    responsive width 500px 360px 300px
    responsive height 170px 122px 102px
    responsive margin, 60px auto 65px, \
                       30px auto 45px, \
                       15px auto
    responsive image "/images/logo.png" "/images/logo-tablet.png" "/images/logo-phone.png"
    hide-text()

Example usage for same keyword:
.copyrights
    responsive font-size 32px same 18px

What are you think about include that mixin into nib?
Maybe you have any suggestions how to make that mixin better?

@vslinko i believe you saw https://github.com/jenius/rupture ?

rupture was released after this PR and after I'm moved from stylus, but thank you anyway :smile:

@vslinko you welcome ( :
what you using now?

@hems mostly other people :smile: but sometimes I can use pure css


Cypress test

Fixes ISSUE #xxx
Description
What will be the pain point if you don't make this change?
Tests
Before fix
screenshots, console outputs or logs are acceptable.
After fix
screenshots, console outputs or logs are acceptable.

Codecov Report
Base: 24.52% // Head: 43.70% // Increases project coverage by +19.17% :tada:

Coverage data is based on head (c596682) compared to base (da033aa).
 Consider uploading reports for the commit df5c543 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main       #2       +/-   ##
===========================================
+ Coverage   24.52%   43.70%   +19.17%
===========================================
  Files          37       10       -27
  Lines        1692     1167      -525
  Branches      103        0      -103
===========================================
+ Hits          415      510       +95
+ Misses       1265      624      -641
- Partials       12       33       +21

Flag
Coverage Δ

unittests
43.70% <ø> (+19.17%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...view/report/packedbubble/packedbubble.component.ts

...rontend/src/app/view/report/line/line.component.ts

...iew/assements/report-view/report-view.component.ts

...tor/src/frontend/src/app/service/policy.service.ts

...mesapce-histogram/namesapce-histogram.component.ts

...r/src/frontend/src/app/shard/shard/shard.module.ts

...w/insight/workload-page/workload-page.component.ts

...view/report/cluster-line/cluster-line.component.ts

...insight/namespace-page/namespace-page.component.ts

.../src/app/view/summary/cluster/cluster.component.ts

... and 35 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


(feat) Updated core protos to the changes required by TS SDK

Summary by CodeRabbit

New Features

Added min_notional field to enhance market information in Derivative and Spot Market protocols.
Introduced admin validation for Peggy and Exchange modules.

Improvements

Updated fee validation logic for better error reporting.
Simplified control flow in access control functions for WasmX module.

Bug Fixes

Corrected timestamp conversion in Oracle module to ensure accurate validation.

Updates

Adjusted listing fees to 20 INJ for Spot and Derivative markets.
Added new access configuration parameter in WasmX module.

@ThomasRalee what is this PR for? I see that the only chages are in the compiled protos that come from injective-core and injective-indexer. But I update those compiled protos regularly in the parent branch. I don't get the need for this.

Yes, you're right. let's close this PR


lollypop: 0.9.516 -> 0.9.519

Motivation for this change
An update
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build lollypop

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: lollypop
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: lollypop
Partial log (click to expand)
Wrapping program /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/bin/lollypop
Wrapping program /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/libexec/lollypop-sp
shrinking RPATHs of ELF executables and libraries in /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/lib  /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/libexec  /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/bin
patching script interpreter paths in /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519
checking for references to /build in /nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519...
wrapping `/nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/bin/.lollypop-wrapped'...
wrapping `/nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519/bin/.lollypop-cli-wrapped'...
/nix/store/w3ggqapsmncn4mrg9acn1a7xdy8wz66s-lollypop-0.9.519

Success on aarch64-linux (full log)
Attempted: lollypop
Partial log (click to expand)
Wrapping program /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/bin/lollypop
Wrapping program /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/libexec/lollypop-sp
shrinking RPATHs of ELF executables and libraries in /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/lib  /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/libexec  /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/bin
patching script interpreter paths in /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519
checking for references to /build in /nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519...
wrapping `/nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/bin/.lollypop-wrapped'...
wrapping `/nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519/bin/.lollypop-cli-wrapped'...
/nix/store/ml4wlgafqx8m8w422gsj3qjcqdf6z668-lollypop-0.9.519


Mounting /sys is broken in Docker 1.6.1

Docker 1.6.1 broke the ability to mount /sys into a container. This is breaking cAdvisor as it uses that for cgroup and network stats.
The fix was already merged to libcontainer: https://github.com/docker/libcontainer/pull/576
Can there be a Docker 1.6.2 with this fix?
/cc @vishh @rjnagal @crosbymichael

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

Fwiw I'm +1 a 1.6.2

Good finding , it works in Docker 1.6.0. I have to hold on my upgrade.
vmarmol : Why cAdvisor need a /var/run:rw ( read-write ) mount? where rest all are RO?

other users are reporting this as well; https://github.com/docker/docker/issues/5582#issuecomment-100241430 so +1 on a 1.6.2

@beeyeas that is to talk to the Docker daemon through it's socket

+1
[0] ~ >> TERM=xterm ssh core@core01
Last login: Sat May  9 09:38:32 2015 from <IP_ADDRESS>
CoreOS alpha (675.0.0)
core@core01 ~ $ docker --version
Docker version 1.6.1-rc2, build 17f157d-dirty

+1
It also hits mesos inside container.
sounds like someone did not predict how much projects depends on binded /sys.
Good consensus would be to allow bind anything, when --privileged is set.

@sielaq as far as mesos is concerned it is possible to run it inside container with docker 1.6.1 using --cgroups_hierarchy flag for mesos-slave. That's how I'm running it on CoreOS:
docker run --rm --net=host --name slave --privileged -v /usr/lib64/libdevmapper.so.1.02:/usr/lib/libdevmapper.so.1.02 -v /var/run/docker.sock:/var/run/docker.sock -v /usr/bin/docker:/usr/local/bin/docker -v /sys/fs/cgroup:/host/sys/fs/cgroup mesosphere/mesos-slave:0.22.1-1.0.ubuntu1404 --containerizers=docker,mesos --master=zk://${ZOOKEEPER_IPS}/mesos --cgroups_hierarchy=/host/sys/fs/cgroup

@gregory90 thx for the hint!

Binaries uploaded for testing found in this comment:
https://github.com/docker/docker/pull/13133#issuecomment-101079431
Feel free to test tomorrow if you have time then we will try to do the release on Wednesday.

tested and verified with docker-1.6.2-rc1 on centos 7 with the cadvisor image. thanks for the quick resolution!

@nathwill thanks for testing!

this is fixed in 1.6.2

Hi, is this version active on hub.docker.com? I still get those errors there. Building them locally with docker 1.6.2 works fine.
https://registry.hub.docker.com/u/feduxorg/centos-ruby/build_id/44153/code/bkz889ik2krjhqaqdcep7jx/
https://registry.hub.docker.com/u/feduxorg/centos-ruby/
[8] System error: "/var/lib/docker/btrfs/subvolumes/fb7e252e96df1225a8464844893af04d843760628855285fb56b840fee49d950/sys/fs/cgroup" cannot be mounted because it is located inside "/sys"

Sorry for the noise. Now it seems to be fixed on hub.docker.com. The issue does not occure anymore.

@maxmeyer no problem. next time you're having issues with Docker Hub, report those at<EMAIL_ADDRESS>because those issues are not handled through GitHub

Ok. Thanks for the information. Is there any info about this on docker hub? I was not aware that I need to report issues there.

Actually, good question. I think it's there, but will have to look if it needs to be more prominent :-)

Thanks. :-)

Had a look, and agree, it's not very clear where to contact support.
Looking at the current beta for Docker Hub, it has been improved a bit https://hub-beta.docker.com/help/, although a clear "support" link could be even better. Perhaps you can open an issue here (if you think it needs to improved further and no such issue exists): https://github.com/docker/hub-beta-feedback/issues


The dropzone SVG icon does not apply style in production

Expected:

Actual:

Adding styles provider does not solve the issue


Roctracer hip frontend 180826

@eshcherb LGTM. However, it would be nice if you can add some comments to hip_prof_api.h or any other header file which documents what we need to do to add profiling for any new APIs that we will support in the future. For example there are already some new APIs that are in the master branch (such as hipStreamCreateWithPriority) for which profiling support needs to be added.

I have created rebased PR#760
For new APIs you will include teh API name in HIP_INIT_API macro as the first argument and I'm going to provide a python script for regenerating of hip_api_str.h fille with callbacks structures and macros.
I will add commrnts to hip_prof_(str/api).h

closing this since #760 is a rebase of this PR.


Too much TLS?

It appears that net-contour will "promote" internal names like foo.my-ns.svc and foo.my-ns.svc.cluster.local to be HTTPS (and redirect HTTP callers to HTTPS) when combining the following features:

default-tls-secret is set to anything (causing the route[*].virtualhost.tls field to be set)
http-protocol set to Redirected or unset default (default is Redirected)

This is because resources/httpproxy.go computes a permitInsecure field and adds some annotations to the internal version of the HTTPProxy, but it doesn't recognize the cluster-internal URLs as needing any special TLS handling, which they probably do.
/good-first-issue
This should be relatively easy to add a test for, I don't know whether we'd want / need to add a flag to enable the old (broken) behavior; my guess is "no".

/triage accepted

/assign @jsanin-vmw


issue:43 [Documentation Update] : added the technologies.md

This PR addresses #43
The following files were added:

docs/technologies.md - contains all the documentation for tech stack of the project
public/tech.svg - an architecture diagram referred in the documentation

Let me know if there are any questions or further adjustments needed.

Thank you so much for your contribution @KrinsKumar!

Message if you are interested in joining our private Discord server.
We would be glad for you to join us @KrinsKumar! 😄

Thank you, sure I would love to join!!

Thank you, sure I would love to join!!

Awesome! 😃


Two Factor Authentication using Identity Server 4

How can I implement a two factor authentication using Identity Server 4.
Both the methods as per the sample does not allow to customize the end point:
var tokenClient = new TokenClient(disco.TokenEndpoint, "ro.client", "secret");
var tokenResponse = await<EMAIL_ADDRESS>"Pass123$", "api1");
Is it possible to achieve 2 factor authentication using either asp.net identity Or EF Core implementation?

This is something that your UI would be responsible for, not IdentityServer.

@brockallen thanks. But how can I make sure that the token is issued to request owner only after two factor authentication. Is it the responsibility of front end? Suppose front end does not take necessary precautions, and the token to access the API has breached security? Can anything be done with asp.net identity 2fa? Please help. Thanks.

I don't know how you'd do two factor while using the token endpoint directly -- that's not the normal flow. You should use a browser and redirect the user to the authorization endpoint.

@brockallen Thanks. What do you suggest as the normal flow to implement 2fa while using Identity Server for generating token for the APIs.

2fa is unrelated to flows. 2fa is part of authentication. see our samples: https://identityserver4.readthedocs.io/en/release/quickstarts/6_aspnet_identity.html

Thanks @brockallen Have gone through the document. Can I use the RO Grant to achieve that while giving full control to the client UI implementing that? Or I have to force the client UI to redirect to the Authentication page in which Identity Server resides?

Ok, now we're back to resource owner flow. I'll repeat:

This is something that your UI would be responsible for, not IdentityServer.

2fa is unrelated to IdentityServer. if you require the use of resource owner flow the your UI is responsible for figuring out how to get 2fa to work. I can't help you any more than that.

@brockallen  What grant type should I consider then while implementing this when the client is something like an SPA in angular js or other UI framework?

SPA = implicit flow

@leastprivilege Thanks. Will look into this.

@brockallen - I appreciate that it's not part of the flow, but it's hard to find documentation about what the best practices are about how I should design my user interface.
I'm trying to use IdentityServer4 as an authentication point for a couple of native iOS and Android apps, and I'd prefer to keep the UI entirely inside those apps. It seems like I'd just build a form inside the app, get the username and password, and submit them to the token endpoint for a password grant.
But if I want to add two factor authentication in - whats the best practice? Do I write a custom endpoint to trigger the sending of the 2fa code, then send that as a "acr_value" to the token endpoint?
I'm just a little lost - as a full stack developer, it's hard to find these resources. It'd be really helpful if IdentityServer4 had a little more documentation about how to set this up, or at least links to help me figure out how to put all the pieces together. I could hack something together, I'm sure, but I'd really like it to represent best practices, and I just can't figure out what the best practice is here.

what sort sorts of 2fa did you want to enable?  The simple Microsoft/google authenticator app, or yubi keys?  The former is in a pull request that was never merged. The later is a work in process.

Honestly, I was assuming I'd just send a text message and get a 5 or 6 digit code, but the Microsoft/Google Authenticator would probably work as well.

see issue 2079

@BuairtRi  I know this thread is dead, but I am trying to figure out the same thing here. I have implemented in IS4 the authenticator login flow for our website but our native windows app uses the token endpoint to authenticate. Do we need to make our own API for this and turn off the default token endpoint?
We would need a response indicating that the authenticator code is required, then we can modify the native client's UI to take a code and send that into IS to authenticate the user.

Shouldn't use implicit flow anymore as it is insecure. See the updated
guidelines nicely explained by Brock:
https://brockallen.com/2019/01/03/the-state-of-the-implicit-flow-in-oauth2/
For user facing native/mobile apps, use Auth code + PKCE flow, after
credential validation is completed by the UI.
On Tue, Oct 22, 2019 at 2:20 PM Corey Cunha<EMAIL_ADDRESS>wrote:

@BuairtRi https://github.com/BuairtRi I know this thread is dead, but I
am trying to figure out the same thing here. I have implemented in IS4 the
authenticator login flow for our website but our native windows app uses
the token endpoint to authenticate. Do we need to make our own API for this
and turn off the default token endpoint?
We would need a response indicating that the authenticator code is
required, then we can modify the native client's UI to take a code and send
that into IS to authenticate the user.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IdentityServer/IdentityServer4/issues/1046?email_source=notifications&email_token=AA7IJS6VYHH3OU5B7EWQ7CDQP477PA5CNFSM4DHEP2M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB6WWJQ#issuecomment-545090342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7IJS2M3K3PC6QFJ2KEWULQP477PANCNFSM4DHEP2MQ
.

Thanks @ghanashyaml . I'll have a look at that and the recommended flow of auth code + pkce and see where to go from there. I'm full stack but new to the authentication world, so most of the documentation on IS4 is very cryptic to me (and seems to be written with someone with authentication already known and internalized in mind).


Improvement: allow overriding a host for standalone-dev image.

Hello @tylertreat!
I'm trying to test the Rust client I'm currently working on, but the issue is that I don't run my docker host on localhost (docker-machine).
And the LIFTBRIDGE_HOST env variable does not seem to have any effect as there is no default config available in standalone-dev image.
It would be nice to be able to override the host without any extra actions.

I'm happy to contribute, just not sure where this image is coming from :/

Was not sure where to put this issue, so closing it in hopes that we can discuss this here.

@ruseinov Can you open an issue on the https://github.com/liftbridge-io/liftbridge repo? The dev-standalone image is built here: https://github.com/liftbridge-io/liftbridge/tree/master/docker/dev-standalone

Done!
On Mon, 17 Aug 2020 at 17:16, Tyler Treat<EMAIL_ADDRESS>wrote:

@ruseinov https://github.com/ruseinov Can you open an issue on the
https://github.com/liftbridge-io/liftbridge repo? The dev-standalone
image is built here:
https://github.com/liftbridge-io/liftbridge/tree/master/docker/dev-standalone
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/liftbridge-io/go-liftbridge/issues/68#issuecomment-674942721,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACP7ENDM2NMVJ3ZCCKZ3R3SBFCWTANCNFSM4P7MFKRQ
.


Xcode Command Lines Tools necessary?

We pulled xcode-select --install and sudo xcodebuild -license out of thoughtbot/laptop around when we were getting it ready to support Mavericks.
Our reasoning was:

Homebrew's install script checks whether the command line tools are installed,
and, only if necessary, will run xcode-select --install.

It looks like that's still true today:
https://github.com/Homebrew/homebrew/blob/master/bin/brew#L97-L121
The commit was https://github.com/thoughtbot/laptop/commit/3106976ff39f7c591138746a36badb95339d5449 but Homebrew link in commit is old / broken.
It looks like you're seeing a benefit to having them in your script. I see some Issues with conversation talking about having the user re-agree to the license when Xcode is upgraded. Is that the main reason to have them in or are there others?
Hope these questions aren't bugging you; just looking to learn if there are changes to our script we should make, too.

No, these aren't bugging me, happy to answer.
My reasoning is that we're not actually using Homebrew's installer so things could be done in a single script (maybe ironic given I'm a Homebrew maintainer) and I want to ensure that all of these things are installed without requiring user interaction.

Gotcha, good reasons. Not having user interaction is nice; we sort of gave up on that, ha.

:+1:


perf_analyzer: unrecognized option '--verbose-csv'

Hey! I'm new to model analyzer.
I was following the quick start guide mentioned at https://github.com/triton-inference-server/model_analyzer/blob/main/docs/quick_start.md .
Upon profiling the sample model, the model-analyzer gives the following output:
Profile complete. Profiled 0 configurations for models: [].
When i try to have a look at the output above i see this error:
INFO[perf_analyzer.py:264] Running perf_analyzer ['perf_analyzer', '-m', 'add_sub_config_8', '-b', '1', '-u', 'localhost:8001', '-i', 'grpc', '-f', 'add_sub_config_8', '--verbose-csv', '--concurrency-range', '1', '--measurement-mode', 'count_windows'] failed with exit status 1 : perf_analyzer: unrecognized option '--verbose-csv'

This looks to be an error coming from the underlying PA tool. Do you have an up to date version of PA as well?

I think the issue is that you are following quick start guide on the main branch. Could you try following the quick start on 22.03 branch and clone the 22.03 branch of the Model Analyzer instead of the main branch? As indicated in the README, main branch is under-development:

You are currently on the main branch which tracks under-development progress towards the next release. The latest release of the Triton Model Analyzer is 1.14.0 and is available on branch 22.03.

I think the issue is that you are following quick start guide on the main branch. Could you try following the quick start on 22.03 branch and clone the 22.03 branch of the Model Analyzer instead of the main branch? As indicated in the README, main branch is under-development:

You are currently on the main branch which tracks under-development progress towards the next release. The latest release of the Triton Model Analyzer is 1.14.0 and is available on branch 22.03.

I tried with the branch 22.03 but the same error pops up

I think the issue is that you are following quick start guide on the main branch. Could you try following the quick start on 22.03 branch and clone the 22.03 branch of the Model Analyzer instead of the main branch? As indicated in the README, main branch is under-development:

You are currently on the main branch which tracks under-development progress towards the next release. The latest release of the Triton Model Analyzer is 1.14.0 and is available on branch 22.03.

Thanks a lot! I cloned the r22.03 repository using the command git clone -b r22.03 https://github.com/triton-inference-server/model_analyzer.git and it worked fine!


remove unnecessary line

Fixes dotnet/docs#6303
Code from an earlier step is still included in the final sample.
While testing, upgrade CSPROJ file to .NET Core 2.1
/cc @markleebrian

bug number seems wrong though


AVSSDK: not able to connect to amazon server

Hi
While initialising the avs sdk , getting the below errors and not able to connect:
Delegate:onAuthStateChangeCalled:state=1,error=0
2017-12-19 12:04:03.311 [  1] 0 MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
2017-12-19 12:04:03.311 [  1] W HTTP2Transport:Unable to parse OpenSSL version!:version=GnuTLS/2.12.23,required=1.0.2
2017-12-19 12:04:03.312 [  1] 0 HTTP2StreamPool:getStream:streamId=1,numAcquiredStreams=1
2017-12-19 12:04:03.313 [  5] 9 PostConnectSynchronize:Entering postConnectLoop thread
2017-12-19 12:04:03.314 [  3] 1 AlertsCapabilityAgent:executeOnConnectionStatusChanged:status=PENDING,reason=ACL_CLIENT_REQUEST
#############################
Connecting...
#############################
2017-12-19 12:04:03.315 [  6] 9 SpeakerManager:provideStateCalled
2017-12-19 12:04:03.315 [  6] 0 AudioPlayer:provideState:stateRequestToken=1
2017-12-19 12:04:03.315 [  6] 9 SpeechSynthesizer:provideState:token=1
2017-12-19 12:04:03.316 [  7] 0 AudioPlayer:executeProvideState:sendToken=true,stateRequestToken=1
2017-12-19 12:04:03.316 [  8] 9 SpeakerManager:executeProvideStateCalled
2017-12-19 12:04:03.316 [  8] 9 MediaPlayer:getSpeakerSettingsCalled
2017-12-19 12:04:03.316 [  2] 9 MediaPlayer:handleGetSpeakerSettingsCalled
2017-12-19 12:04:03.316 [  9] 0 SpeechSynthesizer:executeProvideState:stateRequestToken=1
2017-12-19 12:04:03.317 [  8] 9 MediaPlayer:getSpeakerSettingsCalled
2017-12-19 12:04:03.317 [  2] 9 MediaPlayer:handleGetSpeakerSettingsCalled
2017-12-19 12:04:03.317 [  8] 9 SpeakerManager:validateSpeakerSettingsConsistencyResult:consistent=true
2017-12-19 12:04:03.317 [  8] 9 SpeakerManager:validateSpeakerSettings:volume=100,mute=false
2017-12-19 12:04:03.317 [  a] 0 ContextManager:updateStateLocked:action=updatedState,state={"wakeword" : ""},namespace=SpeechRecognizer,name=R
ecognizerState
2017-12-19 12:04:03.318 [  7] 0 ContextManager:updateStateLocked:action=updatedState,state={"token":"","offsetInMilliseconds":0,"playerActi
vity":"IDLE"},namespace=AudioPlayer,name=PlaybackState
2017-12-19 12:04:03.318 [  8] 0 ContextManager:updateStateLocked:action=updatedState,state={"volume":100,"muted":false},namespace=Speaker,na
me=VolumeState
2017-12-19 12:04:03.319 [  9] 0 ContextManager:updateStateLocked:action=updatedState,state={"token":"","offsetInMilliseconds":0,"playerActi
vity":"FINISHED"},namespace=SpeechSynthesizer,name=SpeechState
2017-12-19 12:04:03.319 [  6] 0 ContextManager:buildContextSuccessful:context={"context":[{"header":{"namespace":"Speaker","name":"VolumeS
tate"},"payload":{"volume":100,"muted":false}},{"header":{"namespace":"AudioPlayer","name":"PlaybackState"},"payload":{"token":""
,"offsetInMilliseconds":0,"playerActivity":"IDLE"}},{"header":{"namespace":"SpeechSynthesizer","name":"SpeechState"},"payload":{"toke
n":"","offsetInMilliseconds":0,"playerActivity":"FINISHED"}},{"header":{"namespace":"Alerts","name":"AlertsState"},"payload":{"allA
lerts":[],"activeAlerts":[]}},{"header":{"namespace":"SpeechRecognizer","name":"RecognizerState"},"payload":{"wakeword":""}}]}
2017-12-19 12:04:03.664 [  6] 0 EventBuilder:buildJsonEventString:messageId=943f346a-796c-4b7c-ab22-ec1175683da8,namespace=System,name=Synchron
izeState
2017-12-19 12:04:03.664 [  6] 0 PostConnectSynchronize:onContextAvailable : Send PostConnectMessage to transport
2017-12-19 12:04:03.664 [  6] 9 HTTP2Transport:enqueueRequest:jsonContent={"context":[{"header":{"namespace":"Speaker","name":"VolumeState
"},"payload":{"volume":100,"muted":false}},{"header":{"namespace":"AudioPlayer","name":"PlaybackState"},"payload":{"token":"","of
fsetInMilliseconds":0,"playerActivity":"IDLE"}},{"header":{"namespace":"SpeechSynthesizer","name":"SpeechState"},"payload":{"token":
"","offsetInMilliseconds":0,"playerActivity":"FINISHED"}},{"header":{"namespace":"Alerts","name":"AlertsState"},"payload":{"allAlert
s":[],"activeAlerts":[]}},{"header":{"namespace":"SpeechRecognizer","name":"RecognizerState"},"payload":{"wakeword":""}}],"event":
{"header":{"namespace":"System","name":"SynchronizeState","messageId":"943f346a-796c-4b7c-ab22-ec1175683da8"},"payload":{}}}
2017-12-19 12:04:04.567 [  b] I HTTP2Stream:libcurl:streamId=1,text=  Trying <IP_ADDRESS>...
2017-12-19 12:04:04.568 [  b] I HTTP2Stream:libcurl:streamId=1,text=TCP_NODELAY set
2017-12-19 12:04:05.074 [  b] I HTTP2Stream:libcurl:streamId=1,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#0)
2017-12-19 12:04:05.578 [  b] I HTTP2Stream:libcurl:streamId=1,text=found 143 certificates in /etc/ssl/certs/amazon.crt
2017-12-19 12:04:06.062 [  b] I HTTP2Stream:libcurl:streamId=1,text=gnutls_handshake() failed: Handshake failed
2017-12-19 12:04:06.063 [  b] I HTTP2Stream:libcurl:streamId=1,text=stopped the pause stream!
2017-12-19 12:04:06.063 [  b] I HTTP2Stream:libcurl:streamId=1,text=Closing connection 0
2017-12-19 12:04:06.094 [  b] 0 HTTP2StreamPool:releaseStream:streamId=1,numAcquiredStreams=0
2017-12-19 12:04:06.095 [  b] 0 HTTP2StreamPool:getStream:streamId=3,numAcquiredStreams=1
2017-12-19 12:04:06.095 [  b] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=0,retryBackoff=185
2017-12-19 12:04:06.296 [  b] I HTTP2Stream:libcurl:streamId=3,text=Hostname avs-alexa-na.amazon.com was found in DNS cache
2017-12-19 12:04:06.297 [  b] I HTTP2Stream:libcurl:streamId=3,text=  Trying <IP_ADDRESS>...
2017-12-19 12:04:06.297 [  b] I HTTP2Stream:libcurl:streamId=3,text=TCP_NODELAY set
2017-12-19 12:04:07.004 [  b] I HTTP2Stream:libcurl:streamId=3,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#1)
2017-12-19 12:04:07.512 [  b] I HTTP2Stream:libcurl:streamId=3,text=found 143 certificates in /etc/ssl/certs/amazon.crt
2017-12-19 12:04:08.355 [  b] I HTTP2Stream:libcurl:streamId=3,text=gnutls_handshake() failed: Handshake failed
2017-12-19 12:04:08.355 [  b] I HTTP2Stream:libcurl:streamId=3,text=stopped the pause stream!
2017-12-19 12:04:08.355 [  b] I HTTP2Stream:libcurl:streamId=3,text=Closing connection 1
2017-12-19 12:04:08.379 [  b] 0 HTTP2StreamPool:releaseStream:streamId=3,numAcquiredStreams=0
2017-12-19 12:04:08.381 [  b] 0 HTTP2StreamPool:getStream:streamId=5,numAcquiredStreams=1
2017-12-19 12:04:08.381 [  b] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=1,retryBackoff=1114
2017-12-19 12:04:09.506 [  b] I HTTP2Stream:libcurl:streamId=5,text=Hostname avs-alexa-na.amazon.com was found in DNS cache
2017-12-19 12:04:09.507 [  b] I HTTP2Stream:libcurl:streamId=5,text=  Trying <IP_ADDRESS>...
2017-12-19 12:04:09.507 [  b] I HTTP2Stream:libcurl:streamId=5,text=TCP_NODELAY set
2017-12-19 12:04:10.300 [  b] I HTTP2Stream:libcurl:streamId=5,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#2)
2017-12-19 12:04:10.740 [  b] I HTTP2Stream:libcurl:streamId=5,text=found 143 certificates in /etc/ssl/certs/amazon.crt
2017-12-19 12:04:11.529 [  b] I HTTP2Stream:libcurl:streamId=5,text=gnutls_handshake() failed: Handshake failed
2017-12-19 12:04:11.530 [  b] I HTTP2Stream:libcurl:streamId=5,text=stopped the pause stream!
2017-12-19 12:04:11.530 [  b] I HTTP2Stream:libcurl:streamId=5,text=Closing connection 2
2017-12-19 12:04:11.548 [  b] 0 HTTP2StreamPool:releaseStream:streamId=5,numAcquiredStreams=0
2017-12-19 12:04:11.550 [  b] 0 HTTP2StreamPool:getStream:streamId=7,numAcquiredStreams=1
2017-12-19 12:04:11.550 [  b] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=2,retryBackoff=3446
2017-12-19 12:04:15.016 [  b] I HTTP2Stream:libcurl:streamId=7,text=Hostname avs-alexa-na.amazon.com was found in DNS cache
2017-12-19 12:04:15.017 [  b] I HTTP2Stream:libcurl:streamId=7,text=  Trying <IP_ADDRESS>...
2017-12-19 12:04:15.017 [  b] I HTTP2Stream:libcurl:streamId=7,text=TCP_NODELAY set
2017-12-19 12:04:15.646 [  b] I HTTP2Stream:libcurl:streamId=7,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#3)
2017-12-19 12:04:16.075 [  b] I HTTP2Stream:libcurl:streamId=7,text=found 143 certificates in /etc/ssl/certs/amazon.crt
2017-12-19 12:04:16.752 [  b] I HTTP2Stream:libcurl:streamId=7,text=gnutls_handshake() failed: Handshake failed
2017-12-19 12:04:16.752 [  b] I HTTP2Stream:libcurl:streamId=7,text=stopped the pause stream!
2017-12-19 12:04:16.752 [  b] I HTTP2Stream:libcurl:streamId=7,text=Closing connection 3
2017-12-19 12:04:16.778 [  b] 0 HTTP2StreamPool:releaseStream:streamId=7,numAcquiredStreams=0
2017-12-19 12:04:16.779 [  b] 0 HTTP2StreamPool:getStream:streamId=9,numAcquiredStreams=1
2017-12-19 12:04:16.779 [  b] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=3,retryBackoff=4747
2017-12-19 12:04:21.536 [  b] I HTTP2Stream:libcurl:streamId=9,text=Hostname avs-alexa-na.amazon.com was found in DNS cache
2017-12-19 12:04:21.537 [  b] I HTTP2Stream:libcurl:streamId=9,text=  Trying <IP_ADDRESS>...
2017-12-19 12:04:21.537 [  b] I HTTP2Stream:libcurl:streamId=9,text=TCP_NODELAY set
2017-12-19 12:04:22.485 [  b] I HTTP2Stream:libcurl:streamId=9,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#4)
2017-12-19 12:04:22.935 [  b] I HTTP2Stream:libcurl:streamId=9,text=found 143 certificates in /etc/ssl/certs/amazon.crt
Failed to connect to AVS!
2017-12-19 12:04:23.326 [  1] I DirectiveSequencer:doShutdown
2017-12-19 12:04:23.328 [  1] 9 TemplateRuntime:getConfiguration
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=TemplateRuntime,name=RenderPlayerInfo,handler=
0x1c3fe8,policy=NON_BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=TemplateRuntime,name=RenderTemplate,handler=0x
1c3fe8,policy=NON_BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=Speaker,name=SetMute,handler=0x1c4820,policy=N
ON_BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=Speaker,name=AdjustVolume,handler=0x1c4820,pol
icy=NON_BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=Speaker,name=SetVolume,handler=0x1c4820,policy
=NON_BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=SpeechSynthesizer,name=Speak,handler=0x19c938,
policy=BLOCKING
2017-12-19 12:04:23.328 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=AudioPlayer,name=ClearQueue,handler=0x19d428,p
olicy=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=AudioPlayer,name=Stop,handler=0x19d428,policy=
NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=AudioPlayer,name=Play,handler=0x19d428,policy=
NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x1
9c158,policy=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=SpeechRecognizer,name=StopCapture,handler=0x19
c158,policy=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=Alerts,name=DeleteAlert,handler=0x19e008,polic
y=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=Alerts,name=SetAlert,handler=0x19e008,policy=N
ON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=System,name=ResetUserInactivity,handler=0x19bf
34,policy=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:removeDirectiveHandlers:action=removed,namespace=System,name=SetEndpoint,handler=0x1c5280,polic
y=NON_BLOCKING
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x1c3fe8
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x1c4820
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x19c938
2017-12-19 12:04:23.329 [  1] 9 SpeechSynthesizer:onDeregistered
2017-12-19 12:04:23.329 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x19d428
2017-12-19 12:04:23.330 [  1] 0 AudioPlayer:onDeregistered
2017-12-19 12:04:23.330 [  7] 1 AudioPlayer:executeStop:playNextItem=false,m_currentActivity=IDLE
2017-12-19 12:04:23.330 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x19c158
2017-12-19 12:04:23.330 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x19e008
2017-12-19 12:04:23.330 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x19bf34
2017-12-19 12:04:23.330 [  1] I DirectiveRouter:onDeregisteredCalled:handler=0x1c5280
2017-12-19 12:04:23.331 [  1] 0 ContextManager:setStateProvider:action=removedStateProvider,namespace=Speaker,name=VolumeState
2017-12-19 12:04:23.331 [  1] 1 AudioPlayer:removeObserver
2017-12-19 12:04:23.332 [  1] 1 AudioPlayer:executeStop:playNextItem=false,m_currentActivity=IDLE
2017-12-19 12:04:23.332 [  1] 9 MediaPlayer:setObserverCalled
2017-12-19 12:04:23.332 [  2] 0 MediaPlayer:handleSetObserverCalled
2017-12-19 12:04:23.332 [  1] 0 ContextManager:setStateProvider:action=removedStateProvider,namespace=AudioPlayer,name=PlaybackState
2017-12-19 12:04:23.332 [  1] 9 SpeechSynthesizer:doShutdown
2017-12-19 12:04:23.332 [  1] 9 MediaPlayer:setObserverCalled
2017-12-19 12:04:23.673 [  2] 0 MediaPlayer:handleSetObserverCalled
2017-12-19 12:04:23.674 [  1] 9 AlertsCapabilityAgent:releaseChannel
2017-12-19 12:04:23.675 [  1] 0 MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
2017-12-19 12:04:23.675 [  1] 0 PostConnectSynchronize:PostConnectSynchronizer::doShutdown().
2017-12-19 12:04:23.675 [  5] 9 PostConnectSynchronize:Exiting postConnectLoop thread
#####################################
Client not connected!
#####################################
2017-12-19 12:04:23.706 [  b] I HTTP2Stream:libcurl:streamId=9,text=stopped the pause stream!
2017-12-19 12:04:23.707 [  b] I HTTP2Stream:libcurl:streamId=9,text=Closing connection 4
2017-12-19 12:04:23.749 [  b] 0 HTTP2StreamPool:releaseStream:streamId=9,numAcquiredStreams=0
2017-12-19 12:04:23.751 [  b] 0 HTTP2StreamPool:getStream:streamId=11,numAcquiredStreams=1
2017-12-19 12:04:23.751 [  b] 0 HTTP2StreamPool:releaseStream:streamId=11,numAcquiredStreams=0
2017-12-19 12:04:23.752 [  b] 0 PostConnectSynchronize:onSendCompleted:status=NOT_CONNECTED
2017-12-19 12:04:23.752 [  b] 0 PostConnectSynchronize:onDisconnected()
2017-12-19 12:04:23.752 [  b] 0 PostConnectSynchronize:PostConnectSynchronizer::doShutdown().
2017-12-19 12:04:23.754 [  c] I CertifiedSender:CertifiedSender worker thread done.  exiting mainloop.
2017-12-19 12:04:23.822 [  1] 9 MediaPlayer:~MediaPlayerCalled
2017-12-19 12:04:23.829 [  1] 9 MediaPlayer:resetPipeline
2017-12-19 12:04:23.829 [  1] 9 MediaPlayer:~MediaPlayerCalled
2017-12-19 12:04:23.832 [  1] 9 MediaPlayer:resetPipeline
Failed to initialize SampleApplication
Failed to create to SampleApplication!
Please suggest the problem area.

@dhananjayj29 it looks like a certificate error. Take a look at these lines in your log:
2017-12-19 12:04:05.074 [ b] I HTTP2Stream:libcurl:streamId=1,text=Connected to avs-alexa-na.amazon.com (<IP_ADDRESS>) port 443 (#0)
2017-12-19 12:04:05.578 [ b] I HTTP2Stream:libcurl:streamId=1,text=found 143 certificates in /etc/ssl/certs/amazon.crt
2017-12-19 12:04:06.062 [ b] I HTTP2Stream:libcurl:streamId=1,text=gnutls_handshake() failed: Handshake failed
Did you make that amazon.crt yourself?
Ryan

Yes. I just renamed it.
I found the issue , this is problem of openssl.
Thanks for the help.

@dhananjayj29 glad to help the community when I can. Do you mind closing the issue if you are all set.

It appears the problem has been resolved. If this is not the case, please re-open the ticket.


Missing newline at the end of file

The VHDL formatter removes the newline from the last line of the file.
It would be great if all the formatted files have a newline character at the end.
I am using it in VSCode with this extension, but I think it's the same also here.

I'll fix this by this weekend.

this feature has been added now, but hasn't included into the vs extension because I don't own that extension.

Thanks a lot, I have also made an issue there.


Increase RawNotifications raw size and used defaults

Addresses #116

Use Plugin default settings
Increase size of raw notification stored: all the headers are included.

As I stated in #116, to me the headers are the least intereresting part of an email notification. I'd much rather see this plugin capturing the full email body (or even just the first 4k or 8k bytes, if it's a big email) in the RawNotification.

Yes, defaulting to 8K it's a good size to get good email representation without overwhelming the DB