Validate using python object

Having the validation format in a single place makes the validation
constrained by placing the content only on a single file,
on the local disk.
There are many applicabilities where we need to test with remote
content, or any other source.
This is making the validator more flexible

I think this is a good idea.
Do we have any unit tests for these cases (validation from a file and then the new case validation from source directly provided to function)?

Right, we should maybe have a bot that pings me everytime I submit a PR to remind me about tests... I will write next week.

@mirceaulinic I can do the ping ;)
I like the PR. It would it make easier to integrate it in salt. Can I help something?

we just need tests to merge it. One should suffice.

I just had some time to look into.
I think we should also update the driver function itself. https://github.com/napalm-automation/napalm-base/blob/enh-validate/napalm_base/base.py#L1541
    def compliance_report(self, validation_file='validate.yml'):
        """
        Return a compliance report.
        Verify that the device complies with the given validation file and writes a compliance
        report file. See https://napalm.readthedocs.io/en/latest/validate.html.
        """
        return validate.compliance_report(self, validation_file=validation_file)

Questen 1: Do we need the default value validate.yml? If yes, the validation_file will always be set and we should invert the logic of testing if validation_file or validation_source is set. https://github.com/napalm-automation/napalm-base/blob/enh-validate/napalm_base/validate.py#L158
if validation_file:
        validation_source = _get_validation_file(validation_file)

@dbarrosop mentioned one test should suffice. An other option would be to extend each validation test with a second assert. https://github.com/napalm-automation/napalm-base/blob/enh-validate/test/unit/validate/test_validate.py#L31
assert expected_report == actual_report, yaml.safe_dump(actual_report)

validate_source = _read_yaml(os.path.join(mocked_data, "validate.yml"))
actual_report_from_source = device.compliance_report(validation_source=validate_source)
assert expected_report == actual_report, yaml.safe_dump(actual_report_from_source)

Questen 2: Do you like more then one assert per test function or not?
I'm sorry for the mess. I'm trying to get the feeling of the community and I will contribute more and more in the future. You all do a great job!

I think we should also update the driver function itself. https://github.com/napalm-automation/napalm-base/blob/enh-validate/napalm_base/base.py#L1541

You are totally right. We should do the same change there.

Do we need the default value validate.yml? If yes, the validation_file will always be set and we should invert the logic of testing if validation_file or validation_source is set

I think we should make it consistent with other methods like this one:
    def load_merge_candidate(self, filename=None, config=None):

So we should probably do in the class:
    def compliance_report(self, filename=None, compliance_rules=None):

It's going to be a backwards incompatible change so we should mark it as such.

Questen 2: Do you like more then one assert per test function or not?

I don't mind multiple asserts if the asserts are related to what you are testing. We have tests with many asserts already. The problem with your solution is that a test testing X might break by something unrelated which is confusing and misleading. For example, if I break loading the rules from a string somehow I will break the tests that verify that the strict mode works fine as well.


Update scala-collection-compat to 2.4.4

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

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

labels: library-update, semver-minor

Superseded by #245.


Update version and CHANGELOG for new release: 0.6.1.

Codecov Report

Merging #368 (d55cfda) into main (3c37734) will decrease coverage by 4.89%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              main     #368      +/-   ##
===========================================
- Coverage   100.00%   95.10%   -4.90%
===========================================
  Files           53       53
  Lines         3043     3043
===========================================
- Hits          3043     2894     -149
- Misses           0      149     +149

Impacted Files
Coverage Δ

statick_tool/__init__.py
100.00% <100.00%> (ø)

...tatick_tool/plugins/tool/shellcheck_tool_plugin.py
53.33% <0.00%> (-46.67%)
:arrow_down:

...tatick_tool/plugins/tool/uncrustify_tool_plugin.py
66.66% <0.00%> (-33.34%)
:arrow_down:

statick_tool/plugins/tool/cppcheck_tool_plugin.py
85.10% <0.00%> (-14.90%)
:arrow_down:

...tatick_tool/plugins/tool/perlcritic_tool_plugin.py
87.03% <0.00%> (-12.97%)
:arrow_down:

...tatick_tool/plugins/tool/clang_tidy_tool_plugin.py
87.32% <0.00%> (-12.68%)
:arrow_down:

...tick_tool/plugins/tool/clang_format_tool_plugin.py
90.16% <0.00%> (-9.84%)
:arrow_down:

statick_tool/plugins/tool/cpplint_tool_plugin.py
90.32% <0.00%> (-9.68%)
:arrow_down:

...tatick_tool/plugins/tool/flawfinder_tool_plugin.py
91.11% <0.00%> (-8.89%)
:arrow_down:

statick_tool/plugins/tool/spotbugs_tool_plugin.py
91.56% <0.00%> (-8.44%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Sitecore 9 TrackingField

Hello,
When I try to use TrackingField in SiteCore 9 I get exception
System.InvalidOperationException: 'No service for type 'Sitecore.Marketing.Definitions.DefinitionManagerFactory' has been registered.'
How can I solve this issue? It used to work in Sitecore 8

Hello, post your test please so that I could reproduce it. Thanks.
Надіслано з iPhone
13 бер. 2019 р. о 18:30 MokoPolo<EMAIL_ADDRESS>пише:

Hello,
When I try to use TrackingField in SiteCore 9 I get exception
System.InvalidOperationException: 'No service for type 'Sitecore.Marketing.Definitions.DefinitionManagerFactory' has been registered.'
How can I solve this issue? It used to work in Sitecore 8
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'll have something for you shortly I had to look at other failing unit tests so don't close this issue yet :)

Here you go I hope this helps:
    [TestMethod]
    public void foo()
    {
        var trackItemId = new ID();
        var trackingValue = @"<tracking></tracking>";

        var db = new Db()
        {
            new DbItem("Tracked_Item", trackItemId)
            {
                new DbField("__Tracking") { Value = trackingValue }
            }
        };

        var item = db.GetItem(trackItemId);
        var field = item.Fields["__Tracking"];
        var trackingField = new TrackingField(field);
    }


RancherOS stops accepting SSH key

RancherOS Version: (ros os version)
V1.1.0 - Latest
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
I'm running it in a KVM/QEMU virtual machine in Proxmox 5.0.
After about 3 hours of not touching the VM, it just stops accepting my SSH key and I can no longer access the VM via SSH. I've done three fresh installs and the same thing happens over and over. My ssh id_rsa isn't changing, the way I'm connecting isn't changing, my IPs aren't changing - nothing is changing. It just decides they key isn't good enough anymore, apparently. None of my other virtual machines or containers have any issues even remotely like this.

I'm just going to stop using RancherOS and use a different OS for my Docker needs. Sorry for creating this issue and closing it so abruptly.


Hardware guidance to run a delta node

Hardware recommendations for running a delta node, so that clients who want to run their own delta node know how much infra to provision.

Yes. Thank you for raising this. Delta is lightweight so the hardware requirement to run is not demanding.
I'll add it here and on our set up docs.

To run delta

Processor: Any multi-core processor with a clock speed of at least 1.5 GHz. AMD based processors recommended (for commp).
RAM: RAM requirement will vary based on usage but At least 1 GB of RAM to run and process small datasets.
Hard Drive: Storage requirement will vary based on the dataset that Delta will process. At least 50 GB of storage space.
Operating System: Tested on MacOS, Linux. Reportedly works as expected on Windows.
Networking: A standard network interface card (NIC) with a speed of at least 100 Mbps. May vary depending on how large the data will be since online/e2e deals can use considerable amount of egress.

Note that these requirements may vary depending on the specific needs of your application.

https://github.com/application-research/delta/blob/main/docs/getting-started-run-delta.md

Awesome, thanks @alvin-reyes


Git provider non interactive

Non-Interactive GitProvider
Description
Added the flags to pass in non-interactive mode for the gitprovider add command.

[x] This change requires a documentation update
[x] I have made corresponding changes to the documentation

Related Issue(s)
This PR fixes #1352
Screenshots
Notes

@quest-bot loot #1352

Hi @Abiji-2020, noticed that your PR is very similar to mine (https://github.com/daytonaio/daytona/pull/1377), with only slight modifications. It's not fair or appropriate to copy someone else's work.
cc: @Tpuljak

@tarunrajput I have noticed this. No worries, we review PRs as they come in plus the quality of work. This PR doesn't hasnt addressed the issue entirely.

@Abiji-2020 Please share screenshots and screen recording of how your submission works.

see below

@Abiji-2020 submitting code from someone else's solution is not acceptable. Please don't let this happen again.


add options to VectorDBQAChain.fromLLM

allows for passing VectorDBQAChain options into VectorDBQAChain.fromLLM.
updated the doc for discoverability
sorry about https://github.com/hwchase17/langchainjs/pull/287 I was struggling with my fork 🤓

ci failed on this and I couldn't easily determine why. I made a style change that will hopefully work.


[druid] fixing the having clause in the explore view

The having filters were not working right since I did a FilterControl refactor. This PR addresses it.

LGTM


Wip 239 opendp feature

Coverage report
 Click to see where and how coverage changed
FileStatementsMissingCoverageCoverage(new stmts)Lines missing

  server/lomas_server
  app.py

  constants.py

  server/lomas_server/dp_queries/dp_libraries
  opendp.py

  server/lomas_server/tests
  test_api.py

  server/lomas_server/utils
  config.py

Project Total

This report was generated by python-coverage-comment-action

Nice! Good idea to have it as a parameter.
Two things that would need to be done:

Update the version of the helm chart so that the github workflow publishes it
Update the helm chart for onyxia and bump its version so that it's also published and usable from the datalab.

see here for onyxia deployment https://github.com/InseeFrLab/helm-charts-miscellaneous/pull/22


Add memory for stepwise observations

I'm using custom plugins on Chat Copilot with Stepwise planner.
My custom plugin returns semantic responses to the planner.
The stepwise planner works well and give me expected result while creating requests to the custom plugin.
However, I'd like to store the stepwise observation (results from the plugins) to the short-term or long-term memory.
This would give me the possibility to ask more details about the responses it provides without requesting more the plugin.
Does it make sense ?
Regards,

Hey @kbeaugrand, I'm working on Saved Plans functionality this sprint and as an extension. I'll look into saving Plan results for recall as well and will keep in mind this specific scenario. You can track that work here: https://github.com/microsoft/semantic-kernel/issues/2581
Closing this issue as a dupe

Hey @kbeaugrand, I'm working on Saved Plans functionality this sprint and as an extension. I'll look into saving Plan results for recall as well and will keep in mind this specific scenario. You can track that work here: https://github.com/microsoft/semantic-kernel/issues/2581
Closing this issue as a dupe

Hi,
Thank you for this answer ! Really happy to note that you are working on that ;)
I was also starting to think about the plan saving and retrieval by their objectives in SK. I think you will do it better than me


wrong repo

wrong repo LOL

this is the right repo.

out of curiosity, what do you mean by the third point? the first two are doable in upstream, but the third point needs clarification

out of curiosity, what do you mean by the third point? the first two are doable in upstream, but the third point needs clarification

For some reason my brain wasn't working correctly, but in SS13, tails are rendered BEHIND the mob when facing south. I might be oblivious to this ability but currently it overlays ontop when facing south, and all other examples of tails have a human cutout of the south facing sprite.

out of curiosity, what do you mean by the third point? the first two are doable in upstream, but the third point needs clarification

For some reason my brain wasn't working correctly, but in SS13, tails are rendered BEHIND the mob when facing south. I might be oblivious to this ability but currently it overlays ontop when facing south, and all other examples of tails have a human cutout of the south facing sprite.

iirc this does require changes to sprites so that you can have a single layer that reorders itself, otherwise you would need two different layers (this is how it was before when lizards were first added)

out of curiosity, what do you mean by the third point? the first two are doable in upstream, but the third point needs clarification

For some reason my brain wasn't working correctly, but in SS13, tails are rendered BEHIND the mob when facing south. I might be oblivious to this ability but currently it overlays ontop when facing south, and all other examples of tails have a human cutout of the south facing sprite.

iirc this does require changes to sprites so that you can have a single layer that reorders itself, otherwise you would need two different layers (this is how it was before when lizards were first added)

That appears to be the case, I don't know what's better. The old system of needing two layers would prevent edge cases where more markings or parts are overlapped by the tail.

Direction dependent layer sorting is a feature that still needs to be added.

Direction dependent layer sorting is a feature that still needs to be added.

yeah i’d prefer this over doing double layers again (however, grouping multiple target layers to apply to in a single marking definition shouldn’t be impossible either)


strange behaviour with carousel in a fullscreen window on desktop

Today I noticed a strange behavior when the carousel is on a full-screen window (ubuntu 4.14) do not know if this strange behavior is due to the operating system or Kivy, the fact remains that when you put the carousel in full screen , slides behave in a strange way, the text inside of them sometimes become blurred, as if they had not yet completed its movement.
Try running this snippet, and put it in full screen on my computer slides lose correspondence with the tabs, as if the graphics do not was updated properly when the window is in full screen.
Snippet to reproduce the behaviour:
https://github.com/kivy/kivy/wiki/Embedding-a-Carousel-inside-a-TabbedPanel
PS: This strange behavior you can observe only with the carousel, but I wanted to use this snippet because it is more clear things

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23088628-strange-behaviour-with-carousel-in-a-fullscreen-window-on-desktop?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

@dennyb87 do you really mean fullscreen or just maximized? Could you try with kivy master, making sure that sdl2 is used as the window provider?

I mean fullscreen as in the screenshot i posted above.
I will try it as soon as possible as you said with the master branch https://github.com/kivy/kivy.git

The same problem as in my example for #2297

Let's track this at https://github.com/kivy/kivy/issues/2297.


Not all games works.

Tried to use it on Ragnarok Online but it doesn't work. Also i can't open your old version page.

Ragnarok uses an ancient software cursor method which isnt supported.
Modern games use hardware cursor. You can tell if a game uses software
cursor if it lags and shows up in screenshots.
On Fri, Mar 17, 2017 at 12:33 PM, sleeplesscat91<EMAIL_ADDRESS>wrote:

Tried to use it on Ragnarok Online but it doesn't work. Also i can't open
your old version page.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PandaTeemo/YoloMouse/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG3FMeYb89btd6vUQn-UtuULyuu4Da8gks5rmt_5gaJpZM4MhAyQ
.

Btw it still sorta works if you assign YM hotkey, then move cursor off
Ragnarok window then back (in windowed not fullscreen mode). But it will
only overlay and not replace the cursor.
On Sat, Mar 18, 2017 at 12:53 PM, Londo Mollari<EMAIL_ADDRESS>wrote:

Ragnarok uses an ancient software cursor method which isnt supported.
Modern games use hardware cursor. You can tell if a game uses software
cursor if it lags and shows up in screenshots.
On Fri, Mar 17, 2017 at 12:33 PM, sleeplesscat91<EMAIL_ADDRESS>
wrote:

Tried to use it on Ragnarok Online but it doesn't work. Also i can't open
your old version page.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PandaTeemo/YoloMouse/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG3FMeYb89btd6vUQn-UtuULyuu4Da8gks5rmt_5gaJpZM4MhAyQ
.


Better dependencies handling.

As discussed here.

move from setup.py to setup.cfg

Et un setup.cfg, 2021 \o/

Et un setup.cfg, 2021 \o/


Documentation Wrong for Asp.Net Core 2.2 Angular Template

As the ASP.Net Core 2.2 Template which ships with VS 2019 Preview has finally been upgraded to use angular 6, the documentation on ssr is wrong. Angular 6 does not use the angular-cli.json. There is a new configuration file.

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

ID: 32a18270-91cd-2a02-2eb8-2c8b1ed03737
Version Independent ID: c58d57e5-a0c2-201d-f6dd-076ad5971fcb
Content: Use the Angular project template with ASP.NET Core
Content Source: aspnetcore/client-side/spa/angular.md
Product: aspnet-core
GitHub Login: @SteveSandersonMS
Microsoft Alias: scaddie

Any idea how to make SSR work with 2.2? The docs are not useful.

So, first of all this guide is wrong. some steps are incorrect fo .net core 2.2 template. But full publishing project contain folder node_modules! As for me this implementation is not good idea. I think if you are using Angular SSR and .net core API you should to separate this two projects and add CORS to .net. So you have node.js with angular SSR witch call your .net core API by CORS.

@joshberry Thank you! You're the best!

@joshberry Thanks for the example, I get this error:
AngularCliBuilder: Error: Unknown option: '--watch'
Any ideas? Thanks!
Using: Net Core 3.0-preview (I know the example is in 2.2 but i'm doing some testing)
Angular: 7.3

app.UseSpa(spa =>
{
// To learn more about options for serving an Angular SPA from ASP.NET Core,
// see https://go.microsoft.com/fwlink/?linkid=864501
            spa.Options.SourcePath = "ClientApp";
            spa.UseSpaPrerendering(options =>
            {
                options.SupplyData = (context, data) =>
                {
                    // Creates a new value called isHttpsRequest that's passed to TypeScript code
                    data["isHttpsRequest"] = context.Request.IsHttps;
                };
            });

            if (env.IsDevelopment())
            {
                spa.UseAngularCliServer(npmScript: "start");
            }
        });

Hic, how to edit my comment on this docs?!!

Now using .netcore 3.0 it seems that SSR is not a thing in aspnet? or am i wrong?

@blogcraft for anyone who needs I had just updated my sample https://github.com/sketch7/dotnet-ngx-sample

using angular 8.2.9
netcore 3/also netcore 2.2

Unfortunately, I haven't seen @joshberry example before now - but its very similar to his
NOTE: theres a bit of a hack inorder for it to work, as latest angular doesnt work so I had to copy AngularCliBuilder and modify

2.2 is out of support.


#14982: Update Unary ops examples

Ticket
#14982
Problem description
Update Unary examples docs
What's changed
Updated Unary examples for the following ops :

ttnn.hardsigmoid
ttnn.hardswish
ttnn.selu

Checklist

[ ] Post commit CI

Doc screenshots

hardsigmoid sweep test :

hardswish sweep test :

https://github.com/tenstorrent/tt-metal/actions/runs/11993864842 - passed


Callbacks typing seems wrong

Describe the bug
The NamedCallbacks and Callback types seem wrong.
  NamedCallbacks: mapOf('PathItem'),

Note: Seems like it should be a mapOf('Callback').
And
const Callback = {
  properties: {},
  additionalProperties: 'PathItem',
};

This seems incorrect. The callback is a map of expression to PathItem. It seems it should be more similar to:
  Callback: mapOf('PathItem'),

To Reproduce
Steps to reproduce the behavior (minimal definition):
openapi: 3.1.0
info:
  title: Callback test
  version: 'alpha'
components:
  callbacks:
    resultCallback:
      '{$url}':
        post:
          requestBody:
            description: Callback payload
            content:
              'application/json':
                schema:
                  type: object
                  properties:
                    test:
                      type: string
          responses:
            '200':
              description: callback successfully processed

Run openapi lint.
Property `{$url}` is not expected here.

 6 | callbacks:
 7 |   resultCallback:
 8 |     '{$url}':
 9 |       post:
10 |         requestBody:

Error was generated by the spec rule.

Expected behavior
I expected that the callback is correct. If not, what is wrong?
Logs
Property `{$url}` is not expected here.

 6 | callbacks:
 7 |   resultCallback:
 8 |     '{$url}':
 9 |       post:
10 |         requestBody:

Error was generated by the spec rule.

openapi-cli Version(s)
1.0.0-beta.94
Node.js Version(s)
16
Additional context
None

Raising the priority as the fix is suggested in the issue itself and should take under 1 hour (with tests) to implement.


No resources loading

I'm trying to get Respond CMS working, I didn't modify the original .htaccess files - they (as I understand it) take anything behind /api/ and feed it to dispatch.php:

    RewriteEngine On
    RewriteCond %{REQUEST_URI} !dispatch\.php$
    RewriteCond %{REQUEST_FILENAME} !-f
    RewriteRule .* dispatch.php [L,QSA]

All I did was set-up my domain in config and that was it. When I do this simple install on other hosting, it works. But on this one it just throws from Application.php line 199 - Tonic\NotFoundException. Not even the hello (I have tried mydomain.com/api/hello).
I'm fairly new to this, so I'd appreciate any help :)

I have sorted it out, problem was in Zend OP Cache settings opcache.save_comments = 0 (thus all comments in anotations were ignored and resources were loaded without their URIs, so no way to call/find them)


Date and Time Pickers Not Opening

Describe the bug
Date and time picker are showing in search in the docs, but its not opening when clicking on them
To Reproduce
Steps to reproduce the behavior:

Go to 'https://actifyjs.com/components/'
Search for 'time'
Select time picker
See error

Expected behavior
It should open date and time picker
Screenshots

Okay, Tks, removed from algolia

I don't get why this was closed? has this been fixed? and is there an implementation of date and time picker available in actify?

No implementations, wait for Roadmap


Update configuration.rst

Documentation update - sample code correction - date format was incorrect.
Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines https://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (https://docs.wagtail.io/en/latest/contributing/developing.html#testing)
Does the code comply with the style guide? (Run make lint from the Wagtail root)
For Python changes: Have you added tests to cover the new/fixed behaviour?
For front-end changes: Did you test on all of Wagtail’s supported environments?

Please list the exact browser and operating system versions you tested.
Please list which assistive technologies you tested.

For new features: Has the documentation been updated accordingly?

@simmbiote thanks for finding that and fixing it up, we really appreciate it.
For smaller docs fixes we do not add them to the release notes but we have added you as a contributor, I have put you down as John Simms, please let me know if you wanted a different name listed instead.
Merged in via https://github.com/wagtail/wagtail/commit/40e6e1c803a7cdfc11799a26e748ef5196812582


Nothing but math is working

No currency or unit conversion

Same. I'll leave a traceback here:

No currency or unit conversion is working for me as well 😥

same for me, there has been #42  open since last Dec, but unfortunately no fix yet

now I am started thinking that this project has been abandoned
anyone here to say me that my statement is wrong?

I have the same issue.

Is there any hope for this to get fixed? It would be pretty sad for this great extension to die. but in the current state it is almost useless, at least for my use cases.

It is (was) a very useful extension, unfortunately just not with this bug 😢

2 weeks ago, this works perfectly, now it's just dead? I wonder if it's some dependency problem?

I can confirm this is a dupe of #42 for me - you can find the error dump if you execute ulauncher from CLI.

Can you try reinstalling the app and letting me know if it worked? I posed a fix #48 and also in #46 I explain what is happening here


Disable profiles with no pause ability

Hi,
I'm having trouble pausing the connected client ( iPad ) as it appears to have connected using the hfp profile which doesn't seem to have a pause method - at least in my case. Is there any harm in disabling most profiles in bluealsa e.g. headset, hands-free, audio sink leaving only A2DP due to the fact my device is pretty much only a BT speaker?
Thanks.

No, it's not a problem. In fact, that's why I've pushed this commit :) Compile fresh master and see bluealsa -h for more details.
I'm closing this issue, because it's not a bug report.

Thanks for the advice. One pain is I'm forced to keep ofono as well which
is hfp, taking it out my iPad just won't see BT at all...
On Wed, Apr 25, 2018 at 7:03 PM, Arkadiusz Bokowy<EMAIL_ADDRESS>wrote:

No, it's not a problem. In fact, that's why I've pushed this
https://github.com/Arkq/bluez-alsa/commit/c3a1c3520b1ec8bb9856492d26a73805dfa6b19c
commit :) Compile fresh master and see bluealsa -h for more details.
I'm closing this issue, because it's not a bug report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Arkq/bluez-alsa/issues/105#issuecomment-384380012,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZGht9N4O3Pkq9JuxYsYhomm6NV18h3wks5tsLqKgaJpZM4TjL_D
.


:sparkles: Add update/delete to migration waves table

Resolves https://github.com/konveyor/tackle2-ui/issues/854

Edit migration wave is not currently working. Opened  https://github.com/konveyor/tackle2-ui/issues/872


Udpate seo title

Context
SEO
Changes proposed in this pull request

Add in England into our SEO title
Remove SEO title from location page as this is our "homepage"

Guidance to review
n/a

The problem with this is that, this:
https://find-postgraduate-teacher-training.education.gov.uk/
And this:
https://www.gov.uk/guidance/find-postgraduate-teacher-training-courses-in-england
Now have the same title. The titles should be unique. The front door is the GOV.UK page. Our first page needs to be different.

Can we make it:
"Find a postgraduate teacher training course in England - search by location or training provider – GOV.UK"

@fofr the problem I saw is when you Google "Postgraduate Teacher Training" both pages show up on the first page so although we say our starting point is on GOV.UK its probably not the reality...

@fofr SEO is important for our service so we definitely need to think about it some more


Adapt to WSB v2.0.x

Creating new "2.0.x" branch in this repo.

Related PRs:

https://github.com/witnet/witnet-feeds/pull/30
https://github.com/witnet/witnet-solidity/pull/23
https://github.com/witnet/witnet-solidity-bridge/pull/407


Deployment & Test Tasks

Porting Deployment and Test tasks into github.
Tasks added:
Chef, CopyFilesToAzureBlob, CopyFilesToMachines, DeployAzureResourceGroup, Docker
Tasks Modified:
DeployVisualstudioTestAgent, RunPowerShellOntargetMachines, RunVisualstudioTestAgent

Have published addressing the comments. Please review.


fix: icons color inside navigation sub-menu

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #5988
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Could you please provide screenshots for this one? I am not sure we have to do that change


Added more tests to increase code coverage

This PR ...

added more tests to increase code coverage.

Checklist

Checklist for Contributors

[ ] Unit tests provided?
Unit tests are needed for new features or to reproduce the bug solved by this PR.
[ ] Documentation updated?
[ ] Unit tests successful
[ ] Code smells (flake8) checked

Checklist for Reviewers

[ ] Read PR description
[ ] Check linked issue
[ ] Check CI/CD (build, automated tests, linting, preview build)
[ ] Check documentation changes
[ ] Add labels to PR

Codecov Report

Merging #125 (5cc5f4d) into main (03c0629) will increase coverage by 5.59%.
The diff coverage is n/a.

:exclamation: Current head 5cc5f4d differs from pull request most recent head cfb7b76. Consider uploading reports for the commit cfb7b76 to get more accurate results

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   79.42%   85.01%   +5.59%
==========================================
  Files         104      104
  Lines        2522     2522
==========================================
+ Hits         2003     2144     +141
+ Misses        519      378     -141

Flag
Coverage Δ

pytest
85.01% <ø> (+5.59%)
:arrow_up:

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

Impacted Files
Coverage Δ

ensysmod/core/file_upload.py
87.09% <0.00%> (+0.80%)
:arrow_up:

ensysmod/crud/base_depends_component.py
88.46% <0.00%> (+3.84%)
:arrow_up:

ensysmod/crud/base_depends_dataset.py
100.00% <0.00%> (+12.50%)
:arrow_up:

ensysmod/api/endpoints/datasets.py
86.36% <0.00%> (+13.63%)
:arrow_up:

ensysmod/crud/energy_transmission_distance.py
85.71% <0.00%> (+14.28%)
:arrow_up:

ensysmod/crud/base_depends_timeseries.py
85.45% <0.00%> (+27.27%)
:arrow_up:

ensysmod/utils/encoders.py
92.20% <0.00%> (+70.12%)
:arrow_up:

ensysmod/core/file_download.py
100.00% <0.00%> (+78.26%)
:arrow_up:

Continue to review full report at Codecov.

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


Allow open basedir restrictions in installers op cache check

https://github.com/shopware/platform/blob/a7f1cbb7a46fde993e97bfcd5c5a4cb19e29e026/src/Recovery/Install/src/Requirements.php#L244
We should catch an exception and still add the check
See
https://github.com/shopware/shopware/pull/2052#issuecomment-501271424
In contrast to Shopware 5, Shopware 6 installation crashes here.
As reported here https://issues.shopware.com/issues/NEXT-10564

There was a fix added in shopware 5 which tries to catche an exception if open_basedir is set.
I believe catching the ErrorException does not really work - it would still throw a warning.
$ php7.4 -dopen_basedir=/home/ -a
Interactive mode enabled

php > try { echo fileinode('/'); } catch ( \ErrorException $e ) { echo 'does not work'; }
PHP Warning:  fileinode(): open_basedir restriction in effect. File(/) is not within the allowed path(s): (/home/) in php shell code on line 1
PHP Stack trace:
PHP   1. {main}() php shell code:0
PHP   2. fileinode() php shell code:1

is there any workaround to install Shopware6 now?
i get the following error
PHP Warning:  openssl_pkey_export_to_file(): cannot get key from parameter 1 in /var/customers/webs/customor/vendor/shopware/recovery/Common/src/Service/JwtCertificateService.php

@FrSelo  Why do you think this is related to this issue? You might want to create a new one or better ask in the support forum. It might be a OpenSSL configuration problem on your site. Good luck :)

Add an try catch around

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


[ACTION] Proposal 3 - Report

Task Description
Parent Issue https://github.com/cncf-tags/green-reviews-tooling/issues/83
This issue is about discussing a strategy on the reporting capabilities of a Green Review.
There are two open points to address:

Metrics: What kind of metrics do we select for a green review?
Long term storage: Where and how do we save the green metrics?

Metrics
At the moment we have agreed on the SCI score calculation, details of how we compute the SCI score can be found here.
Open points on the current approach
From the last milestone we had two open points:

Embodied carbon: plays a significant role in the SCI. This might not be relevant for the CNCF projects interested in the review, cause it is just dependent on the infrastructure we choose to use.
Functional unit: We choose a time functional unit of 15 minutes, are we fine with this or is there any other improvement we would like to do?

Possible extensions
This issue would also like to address possible extensions on the metrics we would like to consider:

Traditional SRE metrics (provider could be the simple k8s SDK)
More low level energy metrics (right now we have an aggregate at container level)
Any other metric that might be interesting for future release

Long Term storage
This section works under the assumption that we have already a chosen set of green metrics.
At the moment we don't store any metrics, the energy metric is just show live on the dashboard for the target container. We would like to store the metrics for future analytical purposes.
The proposal should suggest:

a long term storage technology
a process for storing the green metrics
integration with the current grafana dashboard

Goals to achieve

[ ] Draft the proposal for the metrics extension
[ ] Draft the proposal for the long term storage
[ ] Review the proposals and merge

Nice to have

[ ] Find experts for the metric topic
[ ] FInd experts for the long term storage topic

The 3 SRE metrics for CPU and memory requested by the Falco team are in this comment
We should consider these in the proposal to not lose the previous discussion.

@rossf7 and @rossf7 I am more than happy to think about the metrics part to begin with as it fits with my work right now too. I am less confident or experienced on ideas for the storage part, but it's also an excuse to explore some new database engines…

@nikimanoledaki @ChrisChinchilla in the last meeting I said I'd look into long term storage options. Here is the write up. I'll present it at the meeting tomorrow.
We can also discuss then how best to integrate it into the proposal PR. Posting it here so its somewhere public. Sorry for the wall of text!
Git
Store results in JSON or Markdown with the file being committed by the GitHub Actions workflow.
Strengths

Simple and no infra to manage.
Results are publicly accessible.
Results can be migrated to more complex storage option later.

Weaknesses

Static data.
Not interactive so no ability to compare results.

Object Storage (S3 compatible)
Upload results to object storage from GitHub Actions workflow.
Strengths

Simple.
External backup outside of the cluster.

Weaknesses

Data is private and not publicly accessible.
Not interactive so no ability to compare results.

3 hosting options.
AWS S3
Store results in an S3 bucket.
Strengths

Simple and no infra to manage.
We already have S3 bucket in CNCF AWS account for storing cluster OpenTofu state files.

Weaknesses

Proprietary cloud service.
Need to check we can use existing bucket for storing more data.

MinIO
Run Equinix Metal node running MinIO.
Strengths

S3 compatible but not a proprietary service.
Open Source (AGPL license).

Weaknesses

Involves running an additional server.
Still needs external backup in case server fails.

Apache Ozone
Run Equinix Metal node running Apache Ozone.
Strengths

Same as MinIO.
Open Source (Apache 2 license).

Weaknesses

Same as MinIO.

In Cluster Storage with external backup
Current cluster runs K3s which can be configured to use local storage.
We still need an external backup but we could use object storage for this.
Strengths

Cluster already has Prometheus and Grafana deployed.
Dynamic queries like comparing results can be implemented.

Weaknesses

We still need an external backup of the results.
Needs a cluster with at least one node running 24/7.

Hey @rossf7! 👋 When you have time, could you add the option of using the following option that we discussed in the last meet, please? Thank you!

Postgres database used by DevStats


Erroring Scripts exit with Code 0 instead of 1

package.json
"scripts": {
  "foo": "per-env",
  "foo:development": "blahblah" //will error command not found
}

terminal
npm run foo
echo $? #0
npm run foo:development
echo $? #1

AGH!!

Fixed via d80ce4cd2c1cc0520f0a0d2f23ac08665e774f0f & in v1.0.2.


Deprecate QDE Documentation

What New Or Updated Would You Like To See?
Update documentation to convey that QDE is a deprecated enviornment solution from Chocolatey Software. And has been replaced by QSG.
Why Is It Needed?
Sometimes get asked if we offer the QDE solution. Also it is not mentioned in its documentation that is deprecated.
Additional Context?
NA
Tasks

[x] #642
[x] #651
[x] #653

Closing as work has been merged.


on maui how to attach offline map to ui?

I download some map title like map.osm and on offline navigation how to show on layer?
Thanks

You need to have an IProvider implementation for the specific format you are interested in. In the Mapsui repository there are provider from MBTiles and tiles on the file system, this is probably not what you need. If there is a C# library to read your format it may not be hard to implement an IProvider. Mapsui also support NTS. If there is a component to convert your format to NTS geometries that may be a good option.

All I have used so far for offline support is to use an mbtiles file. So, no direct pbf support or anything streaming. You would need to write something yourself for that. You could look at osmsharp to read a pdf and then convert it to NTS.

oh, is there any sample for mbtiles ?

Yes, in the file MbTilesSample.cs. If you start one of the sample apps you can see it as '1 MbTiles' in the category 'Data Formats'.

dear @pauldendulk still searching, but some rookie on that, can you share a very simple MbTilesSample project?

The Mapsui repository contains sample projects for the various platforms. All those sample platforms contain several categories with each several specific samples. One of these is a simple MBTiles sample, it is called '1 MbTiles' and is in the 'Data Formats' category. This is it: https://github.com/Mapsui/Mapsui/blob/master/Samples/Mapsui.Samples.Common/Maps/DataFormats/MbTilesSample.cs
So, you need to clone the Mapsui repository and build the Mapsui.sln, or one of the slnf's, and then run one of the samples, selected the 'Data Formats' category, and click on '1 MbTiles'.

Thanks so much mr Paul, i find it from samples 1mbtiles, and i download Turkey mbtiles from maptiler data, uploaded with visual studio, and changed buli daction embedded resource and copy always , i just see blank map, where should be problem at


Collect devices on Windows

See https://github.com/stephaneyfx/brightness/issues/8
This collects devices before returning a stream, so that the risk of getting errors when devices are connected/disconnected mid stream is reduced.

Thank you for the PR!


Checklist items for Workshop 1: Part 1

aCreate a project hosted on GitHub with a markdown file
with a draft of a project plan.
Checklist:

[x] Group of five students organized and recorded on Canvas.
[x] Git: name and email set in .gitconfig properly for all 5 students.
[x] Git: changes tracked through git.
[x] README.md with a project description.
[x] GitHub: project hosted on GitHub.
[x] GitHub: GitHub repository shared with Masoumeh ([mtaromi]).
[x] GitHub: GitHub repository link sent to Masoumeh by email or Canvas.
[x] README.md with a table/list of the 5 group members and their GitHub handles
[x] README.md: Declaration section built with an associated commit from each of the students. (See below)
[x] README.md: proper markdown with headings / etc.
[x] README.md with plan: What will you implement? How will it work? What kind of features will it have?
[ ] README.md with plan: which language(s) will you use?
[ ] README.md with plan: what build system will you use?
[x] Git: at least one commit from each student in the group.
[x] no force pushes to master/main (if you need to undo changes, use a new commit).

@AdeptiC, @Jakobloula, @Zeruxe, @Akbar-163, and @Zeruxe
I reviewed the checklist items for workshop 1: part 1 and and you seem to fulfill most of the items. Good job! 🎉
There are two minor items left to be done. Please check and complete them before the next workshop.


Searching algorithms

add Searching algorithms library

@romanZupancic I have merged your pull request on commit : 23b9d5dd7d56783b27e117e35a9837b98d16710a

@romanZupancic
There was a bug in the tests, particularly no test is running for binary search, as shown in the below image.

If the class is changed to binary search, all the tests fail, and the function always returns a False


DNS over HTTPS

Curious whether TrustKit is the right layer to enable features like OkHttp DNS over HTTPS support? It's hard to do anything like this in OkHttp directly, but there is a module supporting this in OkHttp.
https://github.com/square/okhttp/tree/master/okhttp-dnsoverhttps
n.b. I'm not an active user of this library, but mainly want to see additional adoption of DNS over HTTPS.  Please close if not relevant for the project.

Hello,
TrustKit is not really responsible for DNS resolution; it only customizes certificate chain validation. Hence it's not a great place to implement and enable DNS over HTTPS. Unfortunately I don't think a lot of progress toward encrypted DNS resolution will happen until it is part the Android OS and enabled by default...


Add SettingsPage component and update styling in navigators and dashboard

Add SettingsPage component and update styling in navigators and dashboard

:tada: This PR is included in version 1.14.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix(brew): resign modified dylibs

Fixes https://github.com/Homebrew/homebrew-core/issues/106008
Context
Based on this fix https://github.com/Homebrew/homebrew-core/pull/123682
Checklist

[ ] Review Contribution Guidelines.
[ ] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Update documentation when applicable.

Thank you!


Angel spider crash

https://pastebin.com/DPw579T5
forge 1.12.2-<IP_ADDRESS>055
mobultion-0.4.1
I didn't realize there was an update and I don't know how to recreate the crash.  Also how can I disable a mob in the config?  Would just removing all of the biomes in the mob spawn part of the config be enough?

There is no longer a way to disable a mob in the configs but as you said you can use the spawn changer stick (creative only item) and left click any mob you want and you can take away the spawn biomes causing the mob to never spawn (this does not require any restart to the world or the game).
The bug however is a null pointer that I forgot to fix.
An update will be released shortly.


Could you transfer it to WordPress plugin？

thx.

Hello,
I didn't work on WordPress for a few years, so it will hard for me to help on that topic.
Note, your request is quite vague 😊 would you mind explaining a bit more what you are trying to achieve? Just in case someone else runs into this issue and wants to help.
Thanks!


Websites where we can list our PyCon India 2024 event

Hi everyone,
As you might already know, PyCon India 2024 preparations have started already. We want to let people know when it's happening and want to list our event details in the places where maximum people get to know about it.
Please share your suggestion on this GitHub Discussion.
Regards,
Team PyCon India 2024

Tasks

[ ]  Prepare the Github discussion with the appropriate description.
[ ] Create a bitly URL for the GitHub discussion.


[1444] Floor Function Partition

Description
Resolves https://github.com/delta-io/delta/issues/1444
How was this patch tested?
added test in OptimizeGeneratedColumnSuite
Does this PR introduce any user-facing changes?

Hi @corymaklin, thanks for opening this. It looks like the scalastyle is failing, could you fix it?

please can I have more context ?


Default UI font no longer loads on Obsidian for iPadOS

**Before you open a bug report**
Disable plugins and snippets that could be causing a conflict. Update to the latest versions of the following, and restart Obsidian:

Minimal Theme (Done)
Minimal Theme Settings plugin (Not Installed)
Contextual Typography plugin (Not Installed)

Confirm that the bug is not present when Minimal theme is off
Fonts for all other themes work, including the built-in one and “Things”.

Describe the bug
On Obsidian for iPadOS, the default font used for Minimal no longer loads. Instead, a placeholder font (likely Times New Roman) is used for the UI font. However, the text font (what the user types) appears to be untouched.
Other themes do not appear to exhibit this behavior.
To reproduce
Update to the latest version of Obsidian for iOS/iPadOS and install the Minimal Theme from the Community Theme Manager.
The font should show as Times New Roman - I recall that on previous versions of Minimal, a non-TimesNewRoman font was used. This seems to be a bug caused in the latest version. I’m also using Obsidian on a brand-new install, with no other settings modified or plugins installed.
Expected behavior
I expected the Minimal default font to be displayed, instead of Times New Roman.
Screenshots

On Minimal, the font appears to be Times New Roman.

The default Obsidian theme does not exhibit the font issue.

Nor does another theme, Things.
Debug info (required):
Paste your debug info below, which can be accessed in Obsidian with Ctrl/Cmd-P, enter "Show debug info", and click "Copy to clipboard"
SYSTEM INFO:
	Operating system: ios 15.4.1 (Apple iPad)
	Login status: not logged in
	Live preview: on
	Legacy editor: on
	Base theme: dark
	Community theme: Minimal
	Snippets enabled: 0
	Safe mode: on

RECOMMENDATIONS:
	Update installer: installer version too low, please download from https://obsidian.md/download and reinstall.
	Custom theme: for cosmetic issues, please first try updating your theme to latest. If still not fixed, please try to make the issue happen in the help vault or disable community theme and snippets.

Which version of the app are you using? You can see it in About > Settings
I believe this will be resolved if you update to the latest insider build, but I will issue a fix soon for reverse compatibility.

Which version of the app are you using? You can see it in About > Settings
I believe this will be resolved if you update to the latest insider build, but I will issue a fix soon for reverse compatibility.

Thanks for the fast reply - the About page indicates that I’m in 1.2.0, API 0.14.2.

Do you have the Minimal Theme Settings plugin turned on? If not does everything work correctly when you turn it on?

This is fixed if you update to the latest


net: failures with signal arrived during cgo execution

#!watchflakes
post <- goos == "freebsd" && pkg == "net" && `signal arrived during cgo execution`

One of these was reported in https://github.com/golang/go/issues/27992#issuecomment-1542912326, but doesn't match the prior failure mode for which that issue was created.
I saw another in a TryBot in https://storage.googleapis.com/go-build-log/55480854/freebsd-amd64-12_3_a94bfe2e.log.

signal arrived during cgo execution is just indicating that the crash happened in cgo, so the stack is going to be truncated.

Ah, I see. That makes it tricky to track down the actual failure, though — is there some way we can provide the equivalent of a default runtime.SetCgoTraceback for the cgo dependencies in the standard library? (Can we make assumptions about system libraries that allow for something simpler than the full generality of SetCgoTraceback that we need for user C code?)
(CC @ianlancetaylor)

If anything, system libraries are harder to get a traceback from, because they are always heavily optimized and because often the debug info is stored somewhere else.  (Without the debug info the traceback is close to useless, it's just a list of PC values.)
For debugging purposes a blank import of github.com/ianlancetaylor/cgosymbolizer will often get a C backtrace, but I really can't recommend making that part of the Go standard library.  It's 17,000 lines of C code.

Iiiiinteresting, all freebsd-amd64-13_0.
attn @golang/netbsd!

looking to see if i can reproduce locallay.

Okay -- so I'll see if i can reproduce but running the tests with  github.com/ianlancetaylor/cgosymbolizer and if I bump into it, I'll post the trace.
I'm curious: are we running FreeBSD 14 trybots?  It might be interesting to know if this was fixed in later releases -- might give me a place to start bisecting.

are we running FreeBSD 14 trybots?

Appears not. https://cs.opensource.google/go/x/build/+/master:env/freebsd-amd64/make.bash only shows versions up to 13.0-SNAPSHOT. (You're welcome to update the scripts, though — someone on release interrupts should be able to help you deploy the image.)

See issue https://github.com/golang/go/issues/55197 which appears to have the same flakes. Issue https://github.com/golang/go/issues/27992 has flakes for this same test which are different.

any progress on this? centos is have the same issue.

@enihcam Please post plain text as plain text, not as an image.  Images are much harder to read.  Also, please include all the text; your image seems to be missing the first line or two.  Thanks.
That said, the issue you are encountering does not seem to be the one that this bug report is about.  This issue is about a failure on FreeBSD, and you are using CentOS.  The logs in this issue are all about crashes in res_ninit.  Yours seems to be a crash in getaddrinfo.  So I suggest that you open a new issue.
When you open a new issue: does your problem repeat consistently?  Do you have a test case you could share?  Thanks.

@enihcam Please post plain text as plain text, not as an image. Images are much harder to read. Also, please include all the text; your image seems to be missing the first line or two. Thanks.
That said, the issue you are encountering does not seem to be the one that this bug report is about. This issue is about a failure on FreeBSD, and you are using CentOS. The logs in this issue are all about crashes in res_ninit. Yours seems to be a crash in getaddrinfo. So I suggest that you open a new issue.
When you open a new issue: does your problem repeat consistently? Do you have a test case you could share? Thanks.

issue resolved. it was due to glibc incompatible. I replaced it with musl libc.

@enihcam based on your description and the traceback, I suspect you may be running in to https://github.com/golang/go/issues/63567 . Any chance your program is calling os.Setenv() ? That specific crash won't happen with musl since its DNS resolver does not use environment variables.

@enihcam based on your description and the traceback, I suspect you may be running in to #63567 . Any chance your program is calling os.Setenv() ? That specific crash won't happen with musl since its DNS resolver does not use environment variables.

yes, you are right. the program compiled with an old-version glibc (with corresponding old-version libnss) crashes while running in an OS with newer version of glibc+libnss, because glibc loads libnss dynamically. musl has no such issues because musl uses its built-in function for resolving domain names, just like netdns=go.


Issue #388 - Use bash echo for git-credential-env

sh is not consistent in its support of the echo builtin's -e flag. On
debian sh is symlinked to dash, which does not support the flag
resulting in bad credential output. Unfortunately /bin/echo is also
nonstandard - e.g. on macs -e is not supported while it is on debian. I
think the best approach is to rely on bash's echo builtin which should
be consistent across systems (at the slight cost of requiring bash as a
dependency)
This solves issue #388

Thanks for the PR! I like it!
Congratulations on getting your first PR merged, @zymergen-luke! Hope to see more :) <3

Thank you, yuvipanda! I'll hopefully have more coming soon. I've got local changes that have enabled us to use Artifactory as a docker repo and access private git repositories.


[New Feature] Flask init utility

Every time we use flask there is some basic code we need to write and some directories we need to create. We can automate that particular cumbersome task by something like flask init which will let's say create an app.py file containing the basic imports statements, a basic hello world route, code to start the server on the default port, basic directories like a template, static etc.
Basically something like Django's django-admin startproject .

There are plenty of projects that do this already, some better than others. Flask does not require any specific layout, there are many decisions that should be made and understood by the developer.


How to update topic's router?

有2个broker，分别是173,174组成双master集群，如果173 shutdown，那么新创建的topic只会路由到174的broker，即使173后面重新开启，topic也不会自动更新路由信息，导致问题：
1.topic没有负载均衡
2.如果174 shutdown，那么这个topic将无法生产和消费
所以要怎样更新topic的路由信息？谢谢！
I have 2 broker,173 and 174 use 2master-noslave,If 173 shutdown, The new topic only has 174's router info, Even if 173 restart,The topic's router could't update,so has 2 questions:
1.the topic not load balancing
2.if 174 shutdown,the topic can not produce and consume
So how to update topic's router? Thinks!

Does your broker 173 use the same name server address after restart? Please check broker.log to verify your broker 173 connects and heartbeats normally to the name server instances.

Yes,Use the same name server address!
The question can also describe by "How to add a new broker?"
When start a new broker,The old topic should update router info include the new broker!

Please read the issue template carefully, If you have any question furthermore, please ask for our user email list. http://rocketmq.apache.org/about/contact/


support multiple spring contexts

Codecov Report

Merging #1436 into master will decrease coverage by <.01%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##             master    #1436      +/-   ##
============================================
- Coverage     48.45%   48.45%   -0.01%
- Complexity     1798     1800       +2
============================================
  Files           379      379
  Lines         10923    10928       +5
  Branches       1116     1121       +5
============================================
+ Hits           5293     5295       +2
  Misses         5244     5244
- Partials        386      389       +3

Impacted Files
Coverage Δ
Complexity Δ

...ip/framework/apollo/mockserver/EmbeddedApollo.java
87.01% <ø> (-0.94%)
17 <0> (ø)

...apollo/spring/config/PropertySourcesProcessor.java
92% <100%> (+3.53%)
16 <5> (ø)
:arrow_down:

...lo/spring/annotation/ApolloJsonValueProcessor.java
87.71% <100%> (ø)
10 <0> (ø)
:arrow_down:

...pring/property/AutoUpdateConfigChangeListener.java
92.85% <100%> (+1.78%)
17 <0> (+1)
:arrow_up:

...pring/property/SpringValueDefinitionProcessor.java
86.48% <72.72%> (-7.8%)
12 <8> (-1)

...apollo/spring/annotation/SpringValueProcessor.java
83.58% <87.5%> (-1.04%)
18 <3> (+1)

...rk/apollo/spring/property/SpringValueRegistry.java
92.85% <91.66%> (-7.15%)
5 <4> (+2)

...work/apollo/spring/property/PlaceholderHelper.java
81.53% <0%> (-3.08%)
24% <0%> (-2%)

.../apollo/internals/RemoteConfigLongPollService.java
77.91% <0%> (+0.61%)
28% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+0.02%) to 52.013% when pulling ccc3afb480dd26cd32c3c8bb8465395fa76e4744 on nobodyiam:fix-multiple-contexts into 2a926ea23a55283ea6fd982ade3961bdc58810e0 on ctripcorp:master.


az aks disable-addons failed (and worse)

Describe the bug
On AKS cluster version 1.12.6 (also tried 1.12.5) with OMS enabled through the portal when the cluster was created. Attempted to remove the monitoring addon using AZ CLI 2.0.61:
az aks disable-addons --addons monitoring -g myRg -n myCluster
Instead of removing the monitoring addon, the http application routing addon was installed to the cluster. Subsequently no removal of either addon has been possible but the az cli only reports success. I can disable both settings via the Azure Resource Explorer as per:
"addonProfiles": {
"httpApplicationRouting": {
"enabled": false,
"config": null
},
"omsagent": {
"enabled": false,
"config": null
}
},
but this has zero affect on the cluster. Both addons remain stubbornly in place.
To Reproduce
Steps to reproduce the behavior.
Expected behavior
az aks disable-addons removes the addons....

I've logged this as an AKS issue, further investigation showed the Http Application Routing add on is always added to the cluster on creation regardless of your selection and if not installed will be added if other addon removal is requested.
https://github.com/Azure/AKS/issues/886

Closing as this appeared to be a general AKS issue with Addons as per Azure/AKS#886 which is now resolved.


vinegar: 1.7.3 -> 1.7.4

Description of changes
Updated vinegar from 1.7.3 to 1.7.4.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 291815 run on x86_64-linux 1

1 package failed to build:

vinegar

Hi, thanks for this! Looks like the build isn't quite working correctly. One of the patches being carried now fails to apply. Are you able to investigate? Let me know if you need a hand.
include/wine/server_protocol.h updated
include/wine/server_protocol.h updated
server/trace.c updated
server/request.h updated
applying patch /nix/store/p2qhadv6sgc1dk60wqxnnnrpsh25ci1l-cert-path.patch
patching file dlls/crypt32/unixlib.c
Hunk #1 succeeded at 660 (offset 6 lines).
applying patch /nix/store/mhdh3kzkmsldjj1l8si043y83dqlx956-vinegar-wine-segregrevert.patch
patching file dlls/ntdll/signal_x86_64.c
Hunk #1 FAILED at 559.
1 out of 1 hunk FAILED -- saving rejects to file dlls/ntdll/signal_x86_64.c.rej
patching file dlls/ntdll/tests/exception.c
Hunk #1 succeeded at 1810 (offset -15 lines).
Hunk #2 FAILED at 3278.
Hunk #3 FAILED at 3618.
Hunk #4 FAILED at 3639.
Hunk #5 succeeded at 3800 (offset -603 lines).
3 out of 5 hunks FAILED -- saving rejects to file dlls/ntdll/tests/exception.c.rej
patching file dlls/ntdll/unix/signal_x86_64.c
Hunk #1 succeeded at 423 with fuzz 2 (offset 10 lines).
Hunk #2 succeeded at 827 (offset 14 lines).
Hunk #3 succeeded at 866 (offset 14 lines).
Hunk #4 succeeded at 898 (offset 14 lines).
Hunk #5 succeeded at 1072 (offset 15 lines).
Hunk #6 succeeded at 1143 (offset 10 lines).
Hunk #7 succeeded at 1376 (offset 6 lines).
Hunk #8 succeeded at 2793 (offset 245 lines).
Hunk #9 succeeded at 2813 with fuzz 1 (offset 247 lines).

For what it's worth I have the same patching errors when trying to build the previous PR in https://github.com/NixOS/nixpkgs/pull/289265
so I don't think it's necessarily this PR that's buggy.

Result of nixpkgs-review pr 291815 run on x86_64-linux 1

1 package failed to build:

vinegar

It seems like the patches done to wine-staging do not apply anymore. Perhaps we want to pin wine to a certain version or update the patches ourselves?

Oh, I completely forgot about this PR. I have another one up that pins the WINE version and applies all of the patches.
up to date PR: https://github.com/NixOS/nixpkgs/pull/305099


Angular 11 with webpack 5 breaks with webpackMissingModule on recompilation

Bug report

What is the current behavior?
An Angular 11 app with webpack 5 breaks after recompilation (for changes on certain files)
Cannot find module '@angular/core'
at webpackMissingModule ...

If the current behavior is a bug, please provide the steps to reproduce.

git clone https://github.com/difference-engine/ng-webpack5-missing-module.git
cd ng-webpack5-missing-module
yarn
yarn start
(go to http://localhost:4200/)
(edit app.module.ts and save)
refresh browser window, and you should see the error

(interestingly, doing the same with app.component.html or app.component.ts doesn't break things)

What is the expected behavior?
After recompilation, modules shouldn't go missing :sweat_smile:
PS: Honestly, I'm not entirely sure if this is a problem with webpack or how angular-cli uses webpack
Although, I noticed recently some changes were made to rebuildModule -> https://github.com/webpack/webpack/pull/11433
and AngularWebpackPlugin seems to rely on it
https://github.com/angular/angular-cli/blob/master/packages/ngtools/webpack/src/ivy/plugin.ts#L260
hence posting this here

Other relevant information:
webpack version:  5.19.0
Node.js version: 14.15.1
Operating System: Ubuntu
Additional tools: @angular-devkit/build-angular: ~0.1101.2

<EMAIL_ADDRESS>and @angular-devkit/build-webpack still require and use webpack@4 API, don't use resolutions in this case, it is unsafe, sorry we can't help, this tools should migrate on webpack@5 and change their API

I recommend to open issue(s) in them repos

Thanks for the quick response @alexander-akait, have opened an issue angular-cli :+1:


Please relax the over-specified constraint on base to allow building with ghc-8.10.1

The package works fine once the constraint is relaxed.

Thanks!


Refactors Checkbox: Component, Skin, and Theme. Adds theme API's for all components.

Adds composeTheme util function.

Updates Storybook to use refactored Checkbox (component, theme, and skin).

Adds a composed theme example of Checkbox in its stories (makes use of themeOverrides prop).

Removes Checkbox from raw folder.

Installs babel-plugin-transform-object-rest-spread and uses it in the npm script "js" and in storybook/webpack.config.js

Hey @MarcusHurney, what's the status on this PR? Can i start reviewing today/tomorrow? :)

Hi Dominik,
I had the logic board on my computer crash so there was a bit of a delay
there. I will be making the PR today (February 6th). I ported in the
changes from the latest PR as well.

Marcus

On Tue, Feb 6, 2018 at 2:25 AM Dominik Guzei<EMAIL_ADDRESS>wrote:

Hey @MarcusHurney https://github.com/marcushurney, what's the status on
this PR? Can i start reviewing today/tomorrow? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/react-polymorph/pull/35#issuecomment-363378778,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMusLuSd4Zger6kBHx6ibzP-zL2YybJ3ks5tSCi2gaJpZM4RU9Ej
.

@DominikGuzei The commits authored by Cat Tien Nguyen are mine as well. As I mentioned, my logic board crashed, so I am using my partner's computer. Besides necessary adjustments you find during review, this wraps up the refactor!


worksheet['!rows'] - row is undefined

124 row is undefined.
What is the reason?
Thank you.

Does the row have any value ? If I remember well, only rows id with at least one cell not empty has a key on this object.

console.log result =

not have ht so row is undefined
			if (opts && opts.cellStyles) {
				rowobj = {}; rowrite = false;
				if (tag.ht) { rowrite = true; rowobj.hpt = parseFloat(tag.ht); rowobj.hpx = pt2px(rowobj.hpt); }
				if (tag.hidden == "1") { rowrite = true; rowobj.hidden = true; }
				if (tag.outlineLevel != null) { rowrite = true; rowobj.level = +tag.outlineLevel; }
				if (rowrite) rows[tagr - 1] = rowobj;
			}

xlsx.min.flow.js
if (opts && opts.cellStyles) {
rowobj = {}; rowrite = false;
if (tag.ht) { rowrite = true; rowobj.hpt = parseFloat(tag.ht); rowobj.hpx = pt2px(rowobj.hpt); }
if (tag.hidden == "1") { rowrite = true; rowobj.hidden = true; }
if (tag.outlineLevel != null) { rowrite = true; rowobj.level = +tag.outlineLevel; }
if (rowrite) rows[tagr - 1] = rowobj;
}

console.log(row) ->

This row have not ht. so row is undefined. I don't know why, but it seems to be a matter of row. Data also existed in the row.

!rows holds row metadata like cell height.  If the height was not overridden and the row is not hidden, no object will be generated for that row.
The actual data in the row, if there are cells in the row, will be stored in the worksheet.  That has no bearing on the presence of row metadata


fix: title render

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


v5.3.1 version, run tpch workload, print lots of [advance.rs:273] ["check leader failed"]

Bug Report

What version of TiKV are you using?
v5.3.1
What operating system and CPU are you using?
linux
Steps to reproduce

install v5.3.1
/data-sdb/tools/go-tpc/bin/go-tpc tpch -H xxx.16.5.xxx -P 4000 -D tpch --sf 1 --analyze --tiflash prepare

What did you expect?
run normally
What did happened?
[2022/03/04 12:35:43.011 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:44.011 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efa22e690 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:44.011 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:45.012 +08:00] [INFO] [] ["New connected subchannel at 0x7f7e4a64c0f0 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:45.012 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:46.013 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efbb97200 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:46.013 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:47.015 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efb329ed0 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:47.015 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:48.015 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efaaa7790 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:48.016 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:49.016 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efa2939d0 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:49.016 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:50.017 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efbb83140 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:50.017 +08:00] [ERROR] [advance.rs:273] ["check leader failed"] [to_store=56] [store_id=1] [error="RpcFailure(RpcStatus { code: 12-UNIMPLEMENTED, message: "", details: [] })"]
[2022/03/04 12:35:51.019 +08:00] [INFO] [] ["New connected subchannel at 0x7f7efbb1f660 for subchannel 0x7f7e88e1d1c0"]
[2022/03/04 12:35:51.019 +08:00
log file
tikv.log

/component test

/remove-component test

/type bug
/assign Connor1996

/remove-type bug

/assign hicqu

/type question

/remove-assign Connor1996

It's expected. TiFlash doesn't implement the check_leader RPC. The error log should be omit for this case.

The printing frequency of this error is too high, maybe we can reduce it

/remove-type question
/type bug
/severity Moderate

Should be closed by #13312


Generic client, Browser + Node.js connections

Summary
Further efforts after #163 (different approach).
Introduce browser connection (using native fetch) with no Node.js modules in the common interfaces. No polyfills are required. It will also be possible to write new connections on top of @clickhouse/client-common if necessary.
The client was refactored into three packages:

@clickhouse/client-common: all possible platform-independent code, types and interfaces
@clickhouse/client-browser: new "browser" (or non-Node.js env) connection, uses native fetch. Depends on the common package.
@clickhouse/client: Node.js connection as it was before. Depends on the common package.

Related issues:

https://github.com/ClickHouse/clickhouse-js/issues/162
https://github.com/ClickHouse/clickhouse-js/issues/153

Jest was replaced with jasmine + jasmine-expect, cause it's easier to maintain the tests that way - Karma does not like Jest very much; however, one of few downsides of that approach is that retryOnFailure does not work anymore as jasmine matchers do not throw an exception, thus in some tests I had to use janky hardcoded sleep calls.
Current browser connection limitations:

Browser connection insert streams won't be supported atm because it does not work across all the platforms (see FF issue); arrays work fine though; select streams are also fine;
Custom Logger currently is not implemented for the browser connection;
Custom user-agent (with application name) is not implemented for browser connection (need to investigate what's actually possible there. Currently, I don't set it at all).

Checklist

[x] Unit and integration tests covering the common scenarios were added
[ ] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@mshustov, I added several follow-ups to reduce the scope of this one (sorted by priority):

https://github.com/ClickHouse/clickhouse-js/issues/179
https://github.com/ClickHouse/clickhouse-js/issues/180
https://github.com/ClickHouse/clickhouse-js/issues/176
https://github.com/ClickHouse/clickhouse-js/issues/178
https://github.com/ClickHouse/clickhouse-js/issues/177


task: automatically update atlas sdk go

Proposed changes
I have noticed that we are behind with sdk updates for the CLI.
This PR introduces job that will run once for a week and automatically reduce boilerplate work required for release.
Most of the times releases should work out of the box.
I also fixed bug in the golang proxy that improperly handles such large major versions :)
Verification
https://github.com/mongodb/mongodb-atlas-cli/pull/2394

LGTM! wondering if we'd like to set this up in terraform and cfn?

On my radar after testing with CLI


feat: clamp default values of parameter types to upper bound

Summary of Changes
The default value of a type parameter is now clamped to its upper bound during inference of type parameters for calls.

:tada: This PR is included in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.8.0
GitHub release

Your semantic-release bot :package::rocket:


introduce "uint24_t" data type and related operations

Is your feature request related to a problem? Please describe.
OpenThread doesn't have the data type uint24_t at the moment, and this makes it inconvenient when we need to define a 24 bits long data such as IEE Vendor OUI.
Describe the solution you'd like
Introduce uint24_t data type and related operations.
Additional context
IEEE Vendor OUI is 24 bits long which is being used in several places of OpenThread such as Vendor Stack Version TLV and Joiner Advertisement TLV.

I would not recommend adding general support for uint24_t integer type. It can add a lot of overhead to the code (since it is not directly supported by most toolchain/platforms).
I think it'd be easier to use uint32_t as underlying type to store such values . We can add helpers to  write/read it correctly in TLVs/buffers/messages when we would need it, e.g., we can  add  methods in encoding.hpp like WriteUint24(uint32_t value)  or uint32_t ReadUint24() (for both little/big endian encodings).

We can add helpers to write/read it correctly in TLVs/buffers/messages when we would need it, e.g., we can add methods in encoding.hpp like WriteUint24(uint32_t value) or uint32_t ReadUint24() (for both little/big endian encodings).

This was the comment that triggered this issue: https://github.com/openthread/openthread/pull/5299#discussion_r463926015

Submitted PR https://github.com/openthread/openthread/pull/5329 which should help address this.

Resolved by #5329


empty root CA certificate causes C++ server to incorrectly accept clients

hi,
i think there's a bug somewhere that is causing the c++ server to interpret an empty pem_root_certs to mean "don't verify clients" even though it's told to GRPC_SSL_REQUEST_AND_REQUIRE_CLIENT_CERTIFICATE_AND_VERIFY.
here are the three scenarios: the server uses the wrong root cert, the right root cert, and an empty root cert. the client does the same thing in all cases.
(i will attach the example and an optional patch for some logging)

when server uses an unrelated ca cert...
expected outcome: server rejects client
actual outcome: correct
server:
ssl-helloworld$ GRPC_VERBOSITY=INFO ./greeter_server unrelated-cert.pem server.key.pem server-self-signed.cert.pem
pem_root_certs length: 1919
I0810 09:46:01.689992524    1090 ev_epollsig_linux.c:71]     epoll engine will be using signal: 40
I0810 09:46:01.690109983    1090 server_builder.cc:269]      Synchronous server. Num CQs: 1, Min pollers: 1, Max Pollers: 2, CQ timeout (msec): 10000
I0810 09:46:01.691597364    1090 ssl_transport_security.c:573] ____ ca name [/C=AU/ST=Some-State/O=Internet Widgits Pty Ltd].
I0810 09:46:01.691607584    1090 ssl_transport_security.c:584] ____ num_roots 1.
I0810 09:46:01.691624128    1090 ssl_transport_security.c:1459] TSI_REQUEST_AND_REQUIRE_CLIENT_CERTIFICATE_AND_VERIFY!!!
Server listening on <IP_ADDRESS>:50051
E0810 09:46:03.419989514    1093 ssl_transport_security.c:1365] verify error:num=18:self signed certificate:depth=0:/C=AU/ST=Some-State/O=Internet Widgits Pty Ltd/CN=client
E0810 09:46:03.420031706    1093 ssl_transport_security.c:925] Handshake failed with fatal error SSL_ERROR_SSL: error:14089086:SSL routines:ssl3_get_client_certificate:certificate verify failed.

client:
ssl-helloworld$ ./greeter_client server-self-signed.cert.pem client-key.pem client-self-signed-cert.pem
14: Connect Failed
Greeter received: RPC failed
ssl-helloworld$

when server uses the correct ca cert...
expected outcome: server accepts client, and client prints hello world
actual outcome: correct
server:
ssl-helloworld$ GRPC_VERBOSITY=INFO ./greeter_server client-self-signed-cert.pem server.key.pem server-self-signed.cert.pem
pem_root_certs length: 1968
I0810 09:47:14.952839284    1124 ev_epollsig_linux.c:71]     epoll engine will be using signal: 40
I0810 09:47:14.953103981    1124 server_builder.cc:269]      Synchronous server. Num CQs: 1, Min pollers: 1, Max Pollers: 2, CQ timeout (msec): 10000
I0810 09:47:14.957798359    1124 ssl_transport_security.c:573] ____ ca name [/C=AU/ST=Some-State/O=Internet Widgits Pty Ltd/CN=client].
I0810 09:47:14.957848062    1124 ssl_transport_security.c:584] ____ num_roots 1.
I0810 09:47:14.957885994    1124 ssl_transport_security.c:1459] TSI_REQUEST_AND_REQUIRE_CLIENT_CERTIFICATE_AND_VERIFY!!!
Server listening on <IP_ADDRESS>:50051
I0810 09:47:18.699935934    1131 ssl_transport_security.c:1368] depth=0:/C=AU/ST=Some-State/O=Internet Widgits Pty Ltd/CN=client

client:
ssl-helloworld$ ./greeter_client server-self-signed.cert.pem client-key.pem client-self-signed-cert.pem
Greeter received: Hello world
ssl-helloworld$

when server uses an EMPTY ca cert...
expected outcome: server rejects client
actual outcome: INCORRECT: client is accepted and prints hello world
server:
ssl-helloworld$ GRPC_VERBOSITY=INFO ./greeter_server empty.pem server.key.pem server-self-signed.cert.pem
pem_root_certs length: 0
I0810 09:50:16.470812718    1192 ev_epollsig_linux.c:71]     epoll engine will be using signal: 40
I0810 09:50:16.471159187    1192 server_builder.cc:269]      Synchronous server. Num CQs: 1, Min pollers: 1, Max Pollers: 2, CQ timeout (msec): 10000
Server listening on <IP_ADDRESS>:50051

client:
ssl-helloworld$ ./greeter_client server-self-signed.cert.pem client-key.pem client-self-signed-cert.pem
Greeter received: Hello world
ssl-helloworld$

What version of gRPC and what language are you using?
latest master at
commit faef8bcc6533ac5fbc6a8efa4ff964fe897f2b5c
Merge: f20bf78 e1418e4
Author: Vijay Pai<EMAIL_ADDRESS>Date:   Thu Aug 10 07:05:19 2017 -0700

    Merge pull request #12140 from vjpai/slicer

    Switch QPS generic RPC to idiomatic Slice API

What operating system (Linux, Windows, …) and version?
linux, ubuntu 16.04
Linux laptop 4.10.0-30-generic #34~16.04.1-Ubuntu SMP Wed Aug 2 02:13:56 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

What runtime / compiler are you using (e.g. python version or version of gcc)
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2

the cpp/ssl-helloworld example, including the keys/certs (client and server both use self-signed certs)
ssl-helloworld.zip

the patch to ssl_transport_security.c just for some logging
0001-some-logging-in-cert-verification.patch.txt

for convenience, here's the relevant code in the server
  auto client_ca_pem = get_file_contents(argv[1]); // for verifying clients
  auto my_key_pem = get_file_contents(argv[2]);
  auto my_cert_pem = get_file_contents(argv[3]);

  grpc::SslServerCredentialsOptions::PemKeyCertPair pkcp = {
    my_key_pem.c_str(), my_cert_pem.c_str()
  };

  grpc::SslServerCredentialsOptions ssl_opts(GRPC_SSL_REQUEST_AND_REQUIRE_CLIENT_CERTIFICATE_AND_VERIFY);
  ssl_opts.pem_root_certs = client_ca_pem;
  ssl_opts.pem_key_cert_pairs.push_back(pkcp);
  std::cout << "pem_root_certs length: " << ssl_opts.pem_root_certs.length() << std::endl;

  std::shared_ptr<grpc::ServerCredentials> creds = grpc::SslServerCredentials(ssl_opts);

  ServerBuilder builder;
  // Listen on the given address without any authentication mechanism.
  builder.AddListeningPort(server_address, creds);

and the client
  auto server_ca_pem = get_file_contents(argv[1]);
  auto my_key_pem = get_file_contents(argv[2]);
  auto my_cert_pem = get_file_contents(argv[3]);

  grpc::SslCredentialsOptions ssl_opts;
  ssl_opts.pem_root_certs = server_ca_pem;
  ssl_opts.pem_private_key = my_key_pem;
  ssl_opts.pem_cert_chain = my_cert_pem;

  std::shared_ptr<grpc::ChannelCredentials> creds = grpc::SslCredentials(ssl_opts);

  GreeterClient greeter(grpc::CreateChannel(
      "localhost:50051", creds));
  std::string user("world");
  std::string reply = greeter.SayHello(user);
  std::cout << "Greeter received: " << reply << std::endl;

the python server is behaving correctly when given the empty ca certificate: it rejects the client.
in particular, it is being clear reporting the error when loading the empty ca certificate.
server:
ssl-helloworld$ GRPC_VERBOSITY=INFO python ./greeter_server.py empty.pem server.key.pem server-self-signed.cert.pem
I0810 11:13:46.816352080    6010 ev_epollsig_linux.c:71]     epoll engine will be using signal: 40
client_ca_pem length: 0
E0810 11:13:46.824827204    6010 ssl_transport_security.c:588] Could not load any root certificate.
E0810 11:13:46.824862054    6010 ssl_transport_security.c:1437] Invalid verification certs.
E0810 11:13:46.824878103    6010 security_connector.c:902]   Handshaker factory creation failed with TSI_INVALID_ARGUMENT.
E0810 11:13:46.824903644    6010 server_secure_chttp2.c:81]  {"created":"@1502388826.824890226","description":"Unable to create secure server with credentials of type Ssl.","file":"src/core/ext/transport/chttp2/server/secure/server_secure_chttp2.c","file_line":60,"security_status":1}

client:
ssl-helloworld$ python ./greeter_client.py server-self-signed.cert.pem client-key.pem client-self-signed-cert.pem
Traceback (most recent call last):
  File "./greeter_client.py", line 43, in <module>
    run()
  File "./greeter_client.py", line 38, in run
    response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
  File "/home/cauthu/.local/lib/python2.7/site-packages/grpc/_channel.py", line 492, in __call__
    return _end_unary_response_blocking(state, call, False, deadline)
  File "/home/cauthu/.local/lib/python2.7/site-packages/grpc/_channel.py", line 440, in _end_unary_response_blocking
    raise _Rendezvous(state, None, None, deadline)
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with (StatusCode.UNAVAILABLE, Connect Failed)>
ssl-helloworld$

server code
from concurrent import futures
import time

import grpc

import helloworld_pb2
import helloworld_pb2_grpc

_ONE_DAY_IN_SECONDS = 60 * 60 * 24

class Greeter(helloworld_pb2_grpc.GreeterServicer):

  def SayHello(self, request, context):
    return helloworld_pb2.HelloReply(message='Hello, %s!' % request.name)

import sys

client_ca_pem = open(sys.argv[1]).read() # for verifying clients
my_key_pem = open(sys.argv[2]).read()
my_cert_pem = open(sys.argv[3]).read()

def serve():
  server_credentials = grpc.ssl_server_credentials(
      [(my_key_pem, my_cert_pem)],
      require_client_auth=True,
      root_certificates=client_ca_pem, # for verifying clients
  )

  print("client_ca_pem length: {}".format(len(client_ca_pem)))

  server = grpc.server(futures.ThreadPoolExecutor(max_workers=10))
  helloworld_pb2_grpc.add_GreeterServicer_to_server(Greeter(), server)
  server.add_secure_port('[::]:50051', server_credentials)
  server.start()
  try:
    while True:
      time.sleep(_ONE_DAY_IN_SECONDS)
  except KeyboardInterrupt:
    server.stop(0)

if __name__ == '__main__':
  serve()

client code
from __future__ import print_function

import grpc

import helloworld_pb2
import helloworld_pb2_grpc

import sys

server_ca_pem = open(sys.argv[1]).read()
my_key_pem = open(sys.argv[2]).read()
my_cert_pem = open(sys.argv[3]).read()

def run():
  credentials = grpc.ssl_channel_credentials(
    private_key=my_key_pem, certificate_chain=my_cert_pem,
    root_certificates=server_ca_pem,
  )

  channel = grpc.secure_channel('localhost:50051', credentials)
  stub = helloworld_pb2_grpc.GreeterStub(channel)
  response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
  print("Greeter client received: " + response.message)

if __name__ == '__main__':
  run()

@deepaklukose would you mind having a look at the issue? Thanks!

@jboeuf https://github.com/grpc/grpc/blob/master/src/core/tsi/ssl_transport_security.h#L144 does indeed claim that the pem_client_root_certs needs to be non-null for the server to validate the client certs. Let me update the code to get rid of this since rejecting the client seems like the safer option.

Thanks much Deepak!
On Thu, Aug 10, 2017 at 10:21 PM, deepaklukose<EMAIL_ADDRESS>wrote:

@jboeuf https://github.com/jboeuf https://github.com/grpc/grpc/
blob/master/src/core/tsi/ssl_transport_security.h#L144 does indeed claim
that the pem_client_root_certs needs to be non-null for the server to
validate the client certs. Let me update the code to get rid of this since
rejecting the client seems like the safer option.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/12146#issuecomment-321733863, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF-P7Znx9AnmI5JaG0RFyOm__mn7vnnBks5sW-TTgaJpZM4Ozw3g
.

That'd be great! Thanks, Deepak!
(sent from phone)
On Aug 10, 2017 10:24 PM, "jboeuf"<EMAIL_ADDRESS>wrote:
Thanks much Deepak!
On Thu, Aug 10, 2017 at 10:21 PM, deepaklukose<EMAIL_ADDRESS>wrote:

@jboeuf https://github.com/jboeuf https://github.com/grpc/grpc/

blob/master/src/core/tsi/ssl_transport_security.h#L144 does indeed claim
that the pem_client_root_certs needs to be non-null for the server to
validate the client certs. Let me update the code to get rid of this since
rejecting the client seems like the safer option.
—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/12146#issuecomment-321733863, or
mute
the thread
<https://github.com/notifications/unsubscribe-
auth/AF-P7Znx9AnmI5JaG0RFyOm__mn7vnnBks5sW-TTgaJpZM4Ozw3g>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/12146#issuecomment-321734128, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABHdpKgnC_38OfavEdWU2B20UZlhhgVOks5sW-WEgaJpZM4Ozw3g
.

What is the status of this issue? is this closed? if so, Can please share the commit ID in which the issue is fixed?
We are also facing similar issue in python when used grpc.dynamic_ssl_server_credentials() function to update the server credentials dynamically for hitless certificate rotation. The scenario is,

The server is started with valid certificate and client can connect with the server.
We deleted the roots.pem from the server.
Now attempted a new client connection with the same crt details as used in step-1. The client connection is accepted eventhough roots.pem is deleted.

grpc.dynamic_ssl_server_credentials() api notes says that
certificate_configuration_fetcher (callable) – A callable that takes no arguments and should return a ServerCertificateConfiguration to replace the server’s current certificate, or None for no change (i.e., the server will continue its current certificate config). The library will call this callback on every new client connection before starting the TLS handshake with the client, thus allowing the user application to optionally return a new ServerCertificateConfiguration that the server will then use for the handshake.

The fix has not been committed. The proposed fix (https://github.com/grpc/grpc/pull/12193) needs a lot more work to rebase for which I unfortunately don't have the time right now.

The server is started with valid certificate and client can connect with the server.
We deleted the roots.pem from the server.
Now attempted a new client connection with the same crt details as used in step-1

@kboyapa1 so i assume you want the server to reject all new clients after you delete the roots.pem? while i understand that technically, that is the behavior promised by the grpc.dynamic_ssl_server_credentials() api, i dont know why that would be useful in real life? is it a scenario where you want to essentially temporarily "take a server offline" but instead of shutting down the service/cutting off network, you want to keep the server running, so that it can continue serving existing clients and also avoid the subsequent server startup cost, and in the mean time just rejects new clients? then later you "bring the server back up" by simply restoring the roots.pem?

@cauthu Your understanding is correct. I am expecting server should reject all new client connections when roots.pem is deleted.
We are looking for hitless server certificate rotation. The existing client connections should not interrupted due to new certificates installation. All the new client connections should use new certificates to connect to the server.
using grpc.dynamic_ssl_server_credentials() api,
Scenario-1: (Working)

Server started with cert-1(private_key_certificate_chain_pairs + roots.pem) and client session-1 with cert-1 establisged.
Installed new certificate cert-2 (with different private_key_certificate_chain_pairs + roots.pem). The client session-2 with cert-1 should not connect. Its not connecting. The client session-1 should not terminate.  (Working)
Client session-2 with cert-2 will connect to Server. Client session-1 also continues to work.

Scenario-2 (Not Working):

Server started with cert-1(private_key_certificate_chain_pairs + roots.pem) and client session-1 with cert-1 establisged.
Remove the roots.pem while server is running. The client session-2 with cert-1 should not connect because roots.pem is deleted. But client session-2 with cert-1 is connecting. (Not Working).
The reason for this is, grpc.dynamic_ssl_server_credentials() returns None and gRPC server continues to use previously loaded certificate. So clients with old certificates still connect to server.

Scenario-3 (Not Working):

Server started with cert-1(private_key_certificate_chain_pairs + roots.pem) and client session-1 with cert-1 establisged.
Install new certificate cert-2 but invalid private_key_certificate_chain_pairs. The client session-2 with cert-1 should not connect because of invalid chain_pairs. But client session-2 with cert-1 is connecting. (Not Working).
Seeing error message "security_connector.cc:692]  Failed fetching new server credentials, continuing to use previously-loaded credentials."

@cauthu This should be fixed. Let me know if problem still exists. Thanks for patience.

Ah cool, thanks Jiangtao.
On Fri, Dec 14, 2018, 8:37 AM Jiangtao Li<EMAIL_ADDRESS>wrote:

@cauthu https://github.com/cauthu This should be fixed. Let me know if
problem still exists. Thanks for patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/12146#issuecomment-447379932, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABHdpPOqH0UipDwvfSkbSec18_rQwcj_ks5u49PlgaJpZM4Ozw3g
.


Address some feedback from API review

Remove constructors that we don't need
Change metadata property to IsDisabled
Rename PreLoad->Preload

Part of #112

/azp run


bug: sidebar is missing

Version
Version 0.2.2
Describe the bug
As what you can see in the demo. We lose the sidebar that generated by Vuepress.
Expected behavior
There should have a sidebar.
Additional context
https://github.com/vuejs/vuepress/blob/master/packages/%40vuepress/markdown/index.js

Version: 0.4.0,
Reopen this issue, because we can not generate header tokens during md.parse
https://github.com/vuejs/vuepress/blob/cbafe8f562df45f0e9399abf3298da80f5159c1a/packages/%40vuepress/shared-utils/src/extractHeaders.ts#L22-L37


Document how phonegap-app-developer can be used with Cordova

Hello,
Until now I was able to use phonegap-app-developer with cordova serve.
It worked fine until I did an Ajax but there used to be a bug with CORS and a need for a proxy: https://github.com/phonegap/phonegap-app-developer/issues/98
This proxy seems to have been implemented in Phonegap CLI but not in Cordova CLI
It would be nice to document if phonegap-app-developer can be used with a Cordova project.
Actually my project is a cordova project and it seems I'm able to use it (with the proxy) by creating an empty .cordova folder and running phonegap serve instead of cordova serve
Which is confusing because I was told by a Cordova dev to use cordova CLI unless I need Phonegap Build system, but it seems the Phonegap CLI is not only cordova + the build system commands

I believe the command has been implemented so that the cordova serve command can be used in conjunction with the "browser" target platform to run the www on a localhost server instance.. at least that's its purpose for me.
This enables devs to quickly check out if the weblayer is working in combination with plugins that have (test) implementations for the browser target as well...

Closing this out, we have documented how to use the correct commands for the app.


Hostname issues when using CDN

Temporary response:
https://github.com/mya-ake/portfolio/blob/7b80abd8b565a1e679b96bfe83fd785188f5f768/main.ts#L40-L54
Wait for this PR:
https://github.com/denoland/fresh/pull/1024

fix: #1161