fixed undefined hash and added test

Fixed undefined hash when in "yield" there are an unknown component

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   96.72%   96.72%
=======================================
  Files          11       11
  Lines         214      214
=======================================
  Hits          207      207
  Misses          7        7

Impacted Files
Coverage Δ

lib/models/node.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SDK Automation - JS] Improve the logging to be concise and actionable

The user is having difficulty to understand the error reported in the SDK generation log and is unsure of how to proceed.
Refine the error logs to enhance clarity and conciseness, provide guidance on corrective actions, and include relevant document links as necessary. The logging should include three parts information if possible.

error code or error message
next steps or suggested actions
associated document links if have

@MaryGao

Will do in next week


Convert readthedocs link for their .org -> .io migration for hosted projects

As per their blog post of the 27th April ‘Securing subdomains’:

Starting today, Read the Docs will start hosting projects from subdomains on the domain readthedocs.io, instead of on readthedocs.org. This change addresses some security concerns around site cookies while hosting user generated data on the same domain as our dashboard.

Test Plan: Manually visited all the links I’ve modified.

The link on the Github repo description also needs updating.

Coverage remained the same at 98.212% when pulling 5b507659f49a43c39c2f0c459e25aac06aeedf4f on adamchainz:readthedocs.io into f215dd394fc42720bf2f23fbd2cce76138860d48 on KeepSafe:master.

Thanks


Issue with Gauntlet Items

When using the Gauntlet armour (like LEGS:23894 (Crystal legs (perfected))), There are issues with clipping due to the legs extending down into the boot slot. Also, when wearing capes/backpacks with the (HEAD:23888 (Crystal helm (perfected))), the neck usually disappears on the character. I suspect this is because you do not wear capes at the gauntlet, however. The legs should be fixable though.
I hope this gets resolved soon!
Brifrost

I have a feeling these are issues with the models themselves, since they weren't intended to be worn with items outside of the gauntlet. The plugin doesn't do any sort of model manipulation, that would involve a lot more than what the RuneLite API can do.

I have a feeling these are issues with the models themselves, since they weren't intended to be worn with items outside of the gauntlet. The plugin doesn't do any sort of model manipulation, that would involve a lot more than what the RuneLite API can do.

Is it possible to hide the boot slot then?  Would be nice to figure out the workaround. I did put on Elven boots and they look better so far, but would love to just go nothing on (if possible with the plugin).

Yeah, if you select a boot style in the "base" tab, it'll override any item equipped there. Basically the same effect as hiding the item.

Yeah just selecting a base feet look still makes the clipping weird.  Just wanted to point it out to you if you didn't know :)

Oh, you're right. It looks like this is one of the only items in the game that does that. I'll look into hiding boots when those are equipped.

Let me know what you find out!


Leaderboard 2.0: Re-add model searchbar from 1.0

Currently, there is no way to search for specific models in the leaderboard. Re-adding the search functionality from v1 would be a great addition

will do

Hey! So I've been looking into this. Just out of curiosity, can someone explain a couple of things about the original search bar's functionality?
Is it correctly understood that there is an OR relationship between queries?
And if so, do you think I should implement this the same way?
Since str.contains() is already Regex sensitive, couldn't we just leave it as is, and let users use Regex to handle more complex queries?
if search_query:
            names = df["Model"].map(lambda x: re.match("<a .+?>(.+)</a>", x).group(1))
            masks = []
            for query in search_query.split(";"):
                masks.append(names.str.lower().str.contains(query.lower()))
            df = df[reduce(lambda a, b: a | b, masks)]

I'm just gonna tag you guys, cause you worked on the OG leaderboard (@orionw, @Muennighoff )

I wasn’t part of the original design so can’t answer fully, but my guess is that it’s just an artifact.
I think using a regex is a great idea and is more expressive!

cc @tomaarsen who designed the search bar IIRC

Is it correctly understood that there is an OR relationship between queries?

Yeah, that was my original design - that way you can filter for a few models of interest. Regex is also quite useful to have, I like it.


Vault Audit log only shows cluster IP address?

I have audit logging turned on in Vault and it's logging to StackDriver perfectly fine.
However, for each entry, the remote_address is always the Cluster Node IP's address, not the source address.
I'm guessing the Google Load Balancer doesn't pass the originating IP address?  Is there a way to see the actual source IP address?

My guess is that we need to set x_forwarded_for_hop_skips to 2? Can you try that?

How would I go about changing that without disrupting Vault?  I added it to the Terraform vault.yaml file but it doesn't appear to change when I run terraform apply and it lists the actions it's going to perform.
Still new to Terraform.

Can you share how you enabled the audit log device and which device you're using?

vault audit enable file file_path=stdout
I also saw that there are no headers being logged, so I did this too:
vault read sys/config/auditing/request-headers
Key        Value

headers    map[x-forwarded-for:map[hmac:false]]
Which is supposed to add the header to the audit logs, but it still doesn't show up - even after disabling and re-enabling audit logging.
{"time":"2019-01-04T15:41:13.83762549Z","type":"response","auth":{"client_token":"hmac-sha256:75bcb41b7e9dca088a870732d1838dda9d379cfe29ec7303a93eef470ecca1ee","accessor":"hmac-sha256:75a521bef0b698982e1cbee91ec1784637b59ddeb03e42e2eca69c26cd44d344","display_name":"root","policies":["root"],"token_policies":["root"],"metadata":null,"entity_id":"","token_type":"service"},"request":{"id":"0b0892ce-0cab-d56a-c0e6-a64f257313f1","operation":"update","client_token":"hmac-sha256:75bcb41b7e9dca088a870732d1838dda9d379cfe29ec7303a93eef470ecca1ee","client_token_accessor":"hmac-sha256:75a521bef0b698982e1cbee91ec1784637b59ddeb03e42e2eca69c26cd44d344","namespace":{"id":"root","path":""},"path":"sys/audit/file","data":{"description":"hmac-sha256:bb2f3c41481a5d8c69eb385818d3ce889231f31a26df06f4d2dcedb6af513284","local":false,"options":{"file_path":"hmac-sha256:17d533775ea8022c09bfbd2891e60ba9f2b1758bb833a56ab2f80d64bd5a68e2"},"type":"hmac-sha256:4f1e67473eca6bdfed2f938e741b1388ddb70e9643a6f242ebff63269ac62875"},"policy_override":false,"remote_address":"<IP_ADDRESS>","wrap_ttl":0,"headers":{}},"response":{},"error":""}

Okay - try removing all the x_forwarded_for things in the Vault config and add this to the service:
---
apiVersion: v1
kind: Service
metadata:
  name: vault
  labels:
    app: vault
spec:
  type: LoadBalancer
  loadBalancerIP: ${load_balancer_ip}
+  externalTrafficPolicy: Local
  selector:
    app: vault
  ports:
  - name: vault-port
    port: 443
    targetPort: 8200
    protocol: TCP


GEODE-2440: Switch hashcode() return type.

Convert from uint32_t to int32_t to match server.

Please close this pull request and resubmit when you have cleaned up all the places that use hashcode.

Close for now. Resubmit after implementing jbarrett's recommendations.


Automated test integration

Automate tests on push via GitHub workflow

will require small test datasets

will require small test datasets


Enum types

Enum types can be easily simulated by doing something like this /
C++:
enum Direction {
    UP = 0,
    RIGHT,
    DOWN,
    LEFT,
};

void showEnum(int direction) {
    static_cast<Direction>(direction);
}

lua.set_function("showEnum", &showEnum);

Lua:
local Direction = {
    UP = 0,
    RIGHT = 1,
    DOWN = 2,
    LEFT = 3
}

showEnum(Direction.DOWN)

--
It's not a great thing, i know
But, this will be possible in future releases? /
C++:
enum Direction {
    UP = 0,
    RIGHT,
    DOWN,
    LEFT,
};

void showEnum(Direction direction) {
    direction;
}

lua.new_usertype<Direction>("Dir",
                            "UP", &Direction::UP,
                            "RIGHT", &Direction::RIGHT,
                            "DOWN", &Direction::DOWN,
                            "LEFT", &Direction::LEFT);

Lua:
showEnum(Direction.UP)

enum class Direction {
     Up,
     Right,
     Down,
     Left
}

sol::state lua;

lua["Direction"] = lua.create_table_with( "Up", Direction::Up, "Down", Direction::Down, "Left", Direction::Left, "Right", Direction::Right );

Should work? If it doesn't, let me know.

Yes, it worked, of course, nice :)
Maybe a note in the tutorial about it?
probably others will question about it like me ;-/

I've tried this too and this works:
sol::object obj = lua["Direction"]["Up"];
auto dir = obj.as<Direction>();

but obj.is<Direction> returns false. If I try to register Direction as usertype, obj.as<Direction>() fails. What can be done about this?

auto dir = static_cast<Direction>(obj.as<int>());

That was a small oversight of mine. I have forgotten traits specifically for converting enum class types and the like (and plain enum, without the class, were treated like integers). Pushing the fix today, along with about 20 other things...

It also seems like a convenience function
lua.create_enum( "enum_name",
    "val1", my_enum::val1,
    "val2", my_enum::val2,
    "val3", my_enum::val3,
    "val4", my_enum::val4
    // ...
);

Wow, thanks! That was quicker then I expected. I'll test it later. :)
create_enum function would indeed be very useful. It would be also great to automatically make "enum_name" table read only. Accidentally changing or adding/removing some values in that table will lead to some bad stuff.

Obligatory warning that making it "read-only" has a performance overhead etc. etc., if read-only bothers you call new_enum<false> to trigger a read/write table (that's just a regular table with no fancy metatable stuff).

I don't think this was ever documented.

You're right, it hasn't been, I forgot. I guess I'll get on it.

Is there a way to quickly register a large enum (that supports iteration and to_string conversion) without explicitly touching every element?

Unfortunately, no. Sol does not do auto-binding (yet).
The only way to do automatic iteration is to require the user to pick one of many available reflection libraries out there (Boost.Fusion / libmirror), force the user to annotate their types, and then hook into those structures. This is a bit of a daunting task.
The other way would be to use a parser of some sort to read C++ source and then just bind All The Things. The problem with this approach is that now I need to build a C++ parser into Sol at runtime. That would make the library a humongous pain for both end users to use (executable size and other things) and a problem for me (I'm building the early stages of a compiler. Aaahhh, the BUGS).
The good news is, Sol can be used with other generators. That is, there's no harm in grabbing, say, SWIG to do some rudimentary bindings for enums and other types while Sol just helps you manage state and register smaller, concise usertypes.
I know this answer isn't satisfactory. I don't like it myself, to be quite frank: problem is, my hands are tied until C++??, when I can take a hard dependency on built-in reflection. I'm going to hope it's before C++20.

Oh, ok thx :D @eliasdaler suggested the following to be. Maybe someone else will find it usefull, too:
sol::table enumTable = lua["SomeEnumName"];
for(auto& enumValue : enumValues)
    enumTable [to_string(enumValue)] = enumValue

Sorry to be that guy who bumped this CLOSED issue from 7 years ago... but as a new user of sol, I am reading the documentation and it seems this is STILL missing from the documentation. Just thought it should be known, because being able to handle enums so easily is an AMAZING feature, that is not documented anywhere as far as I know.

all the results returned there, are listing enums for sol api, not how to handle them with sol.

Not necessarily true. The last result points to the new_enum function which was implemented in relation to this issue I believe.
Though I guess it is a bit hidden inside the table members documentation.

Yes, please do use new_enum, that should solve it. If it doesn't work out
still, do let me know.

Though I guess it is a bit hidden inside the table members documentation.

Yeah. I've got a LOT of work to do for sol2 and I haven't started yet.
Highest on the list is automatically self-documenting members based on the
bindings people write, and just... entirely new documentation based on a
C++-parsing doc generator that can export it into the RST docs (so that I
don't have to keep forgetting to update the docs when I change the
function).
On Sun, Dec 3, 2023 at 2:15 PM Rochet2 @.***> wrote:

all the results returned there, are listing enums for sol api, not how to
handle them with sol.
Not necessarily true. The last result points to the new_enum function
which was implemented in relation to this issue I believe.
Though I guess it is a bit hidden inside the table members documentation.
—
Reply to this email directly, view it on GitHub
https://github.com/ThePhD/sol2/issues/103#issuecomment-1837572770, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWAWJBN4BW7SSIXYATTJFLYHTFTLAVCNFSM4CE2ILU2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBTG42TOMRXG4YA
.
You are receiving this because you were assigned.Message ID:
@.***>


90 fix rabbitmq username and password in clowder related containers

#88 and #83 both need this PR so merging first.
I have built containers with the tag "pr-91" and tested on smm-tester.


refactor(ls): greatly simplify ls implimentation

This greatly simplifies the implementation of ls, and probably makes it a good deal more performant.

@ariporad does this work for ls('-d')? I noticed you cut out the mention of options.directory, which makes me suspicious this could work.
Also, this breaks some additional tests, so those should be resolved first.

@nfischer: So, we appear to have some inconsistent ls behavior. If your pass in a file/pattern with no matches, should it have an error, or just return an array with nothing in it?

@ariporad I accounted for this in common.expand(). In bash:
$ ls this*file*doesnt*exist.txt
ls: cannot access this*file*doesnt*exist.txt: No such file or directory
# bash passes a literal 'this*file*doesnt*exist.txt' to the ls executable

@nfischer: So if it can't find the file it should error?

That's what ls() should do, yes. That's why I use common.expand() to inside common.wrap(). It will pass non-existing file names to commands (like ls()). Then those commands are expected to handle the non-existing file themselves (in ls()'s case, it should output an error message).

glob.sync() returns an empty array, I think, which probably isn't what you want

@ariporad pinging this to see if there's any work being done to re-add support for -l and -d.

@nfischer: Sorry, I've been sick. I'll work on that ASAP.

@nfischer: Both -l and -d should be back now.

@ariporad I'm going to make some adjustments to this branch and see if I can fix some of the errors we're seeing.

Ok, thanks! If not, I'll try to look into it tomorrow or when I get back.

@ariporad Take a look now. I fixed the bug you had for Windows (you forgot /g on a regex, that's all). Nice PR!
I tested performance and saw that this was very very slow... So I did my best to squeeze out some more performance. This is about 2x faster than your initial revision (and about 2x uglier...). This is still slower (also about 2x...) than the master branch, but it should be a lot cleaner.
So it's a tradeoff. More maintainable and a bit slower, or we can stick with the tried and true method and get faster.
As for the tests, only one of them needed to change (the one you changed for ls('...*.j')). I just changed some others because I realized I could format them a bit nicer, make them more robust (the tests using idx were actually written poorly, my fault), and let them catch more.
I also fixed a but where ls -l resources/ls would return values with names including resources/ls (they should be relative to resources/ls, not the current directory), so that's what's going on there.

@ariporad This should be rebased off master. If you can, see if you find ways to improve performance (or refactor things into a function), since we're still taking a performance hit for the normal use case, which isn't ideal.

@nfischer: I'm going to try something, give me a second.

@nfischer: Actually, let me talk this through with you first.
Personally I vote for slower and more maintainable, it's not like ls is a big bottleneck.
It also might be better to try to hijack fs.stat, so common.wrap looks something like this:
function wrap(cmd, fn, options) {
  return function() {
    var oldStat = fs.stat;
    var statResults = {};
    fs.stat = function (file) {
        if (!statResults[file]) statResults[file] = oldStat.apply(this, arguments);
        return statResults[file];
    }

    // Whatever common.wrap used to do

    fs.stat = oldStat;
  };
}

I bet this would speed things up a bit.

Nope, no it wouldn't.
But I did learn how to benchmark node things.

@ariporad I just pushed another unit test to ls() to make sure everything is working nicely. Assuming everything still passes (it should), does this look good to merge?

LGTM!

Oh, wait, I have to rebase this off master.

LGTM


images: Set GOPS_CONFIG_DIR in scratch images

This sets the GOPS_CONFIG_DIR environment variable in all images which
use gops and are based on a scratch image.
This fixes an issue with crun where due to missing environment
variables, cilium-operator and hubble-relay would error out when
starting the gops agent with the following error:
Error: failed to start gops agent: unable to get current user home
directory: os/user lookup failed; $HOME is empty

This only seems to occur on crun, which sets HOME to an empty
variable in scratch images. In non-scratch images, it sets HOME
according to /etc/passwd (see https://github.com/containers/crun/pull/104),
so this currently only affects the cilium-operator and hubble-relay images
when deployed with crun. Most other container runtimes
seem to set HOME to / in scratch images, which works for gops.
I have manually verified that this fixes the issue with podman --runtime /usr/bin/crun run
Fixes: #14301 (check this issue for additional context)

test-me-please

Maybe this is something that can be fixed in gops itself? (e.g. fall back to / if there's no HOME)
What about specifying ENV HOME=/ instead of something gops-specific?

Maybe this is something that can be fixed in gops itself? (e.g. fall back to / if there's no HOME)

Not sure what you mean by "fixing in gops". We could open a patch to upstream gops, but it seems like the current behavior of erroring is intentional  (see https://github.com/cilium/cilium/issues/14301#issuecomment-762241187 for the code path in gops).
I considered hard-coding a custom ConfigDir option in relay and operator, but that would change the behavior of gops outside of a scratch container, such in our dev VMs.

What about specifying ENV HOME=/ instead of something gops-specific?

I don't have a strong opinion on what variable we set. I decided to go with GOPS_CONFIG_DIR as it will have the least unexpected impact, but if you feel strongly that we should set HOME instead let me know and I can change it.

retest-4.19

if you feel strongly that we should set HOME

Probably not, at least like this it's clear this is a workaround for gops. :+1:


chore(main): release 1.1.1

:robot: I have created a release beep boop
1.1.1 (2024-12-03)
Bug Fixes

recompile for ILIAS 8.16 (9ed7de1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/iFadi/StudIP-ILIAS-Skin/releases/tag/v1.1.1 :sunflower:


fix: avoid updating CRD if TLS cert unchanged

In the injectPublicKeyIntoCRD function, the TLS cert is always
injected into the CRD conversion webhook. This can produce
conflicts with concurrent updates to the CRD.
This code ensures the update is only done when the TLS cert
is out of date, and includes a retry.
Closes #499
Signed-off-by: Jaime Silvela<EMAIL_ADDRESS>

/test


Investigate EAP fix

This is to look into a fix for #2279

@ivanporty updated, ptal.
If you have suggestions on how to specifically test this change to the projectSelector that would be helpful.

This is called when fetching project list over the network resulted in an error and should show built-in error message pane. Probably can test that by throwing exception in ProjectLoader? It could be really hard to actually emulate network or I/O issue. We can also add a unit test for that, but that can involve some heavy lifting with UI mocks.


Using this with Ollama + Open WebUI

Anyone successfully using this (very useful) plugin in connection to a locally served Open WebUI API? I'm struggling trying to interact with it without success. After having tested the WebUI API with curl commands looks like it's functional

Could you please be more specific and describe some context along with the steps to reproduce.

Hello,
I have a similar issue here.
I have openwebui installed and exposed, so I can now use ollama either through the openwebui or via the api with a token.
With VSCode addon, you can achieve this as openwebui and ollama have compatibility with open api specs. However you need to use the useLegacyCompletionsEndpoint to have it working properly for some completion usage.
When I set it up with this addon, following the same rule for the url and api key, I'm unable to list/select a model. I don't see any log coming on the ollama server. If I try to use a feature like fix/review document, its just spin but no error that I can see.

Thanks!

Nevermind, it working like a charm, the only change I had to do was switch the url from https://mydomain/ollama/v1 to https://mydomain/ollama

So I confirm that your addon is working fine so far with openwebui (helm chart v3.5.0).

Thanks to the comments above, I managed to have it work myself also! So anyone who might be in trouble with the same setup (local Ollama + WebUI service, API Key provided by WebUI) the endpoint to be specified should be set to something similar to the above.
Additionally, looks to me like whenever you try different formats for the endpoint address you'd better delete and rewrite again the API key. Dunno why but in my case changing the endpoint address to the correct one didn't work first; when I accidentally deleted the API Key and had to copy/paste again, everything started working. Might me a fault in my side so just a side note.


refactor(top-app-bar): top app bar's navigation icon should be a button, not an anchor

<a>nchors lead users to some url, <button>s indicate a user's action.
Currently nav icon in the top app bar is an anchor. But in the most cases it's used for opening / closing the drawer (action). Even if nav icon is a "back" button, it can be made with history.back() method, not with storing a url. And, unlike an anchor, button doesn't add unnecessary hash to the url (which can be a problem in SPAs routing).

Also, seems that since it changes the hash, it also interferes with vue-router, breaking the app. I tried .native, .prevent, anything that would prevent this behaviour but couldn't.


error: plugin failed to initialize free code camp

Can't run code because of this error error: plugin failed to initialize  it is showing in the lower left corner.  Tried different browsers, computer, and devices same error.  Have a really slow internet connection

Thanks for the report and connection speed detail. I am able to repro this by going to any challenge in Chrome with the devtools set to disable caching and throttling the network connection to 750KB/s or less.

Thanks I was hoping it wouldn't be the speed.
Larry
On Dec 1, 2015 6:44 PM, "Logan Tegman"<EMAIL_ADDRESS>wrote:

Thanks for the report and connection speed detail. I am able to repro this
by going to any challenge in Chrome with the devtools set to disable
caching and throttling the network connection to 750KB/s or less.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4882#issuecomment-161142724
.

Unfortunately, the we use web worker that pulls down a script from our servers. We have a timeout to check if the script has locked up. I have a PR coming up that will remove this dependency and should alleviate some of this pain.


Localize [Required] DataAnnotation

Is this a Bug or Feature request?:
Bug
Steps to reproduce (preferably a link to a GitHub repo with a repro project):
Create a project with this: https://docs.microsoft.com/en-us/aspnet/core/fundamentals/localization?view=aspnetcore-2.1#using-one-resource-string-for-multiple-classes
Description of the problem:
Translation for [Required] annotation doesn't work without providing "ErrorMessage" attribute.
Other DataAnnotation's works properly.
For example [EmailAddress] gets translated with this resx file, without providing "ErrorMessage" attribute to the annotation, but [Required] doesn't get translated without providing "ErrorMessage".
...
  <data name="The {0} field is required." xml:space="preserve">
    <value>A(z) {0} mező kötelező</value>
    <comment>Required</comment>
  </data>
  <data name="RequiredAttribute_ValidationError" xml:space="preserve">
    <value>A(z) {0} mező kötelező</value>
    <comment>Default Required</comment>
  </data>
  <data name="Invalid username or password!" xml:space="preserve">
    <value>Hibás felhasználónév vagy jelszó!</value>
    <comment>Required</comment>
  </data>
  <data name="The {0} field is not a valid e-mail address." xml:space="preserve">
    <value>Nem megfelelő email formátum.</value>
  </data>
  <data name="The input was not valid." xml:space="preserve">
    <value>Hibás érték.</value>
  </data>
...

I have tried this solution from StackOverflow: https://stackoverflow.com/a/50230263/4690796 but the custom GetAttributeAdapter function doesn't get called for [Required].
Version of Microsoft.AspNetCore.Mvc or Microsoft.AspNetCore.App or Microsoft.AspNetCore.App: 2.1.1

Thanks for contacting us, @MateEke.
@ryanbrandenburg, can you please look into this? Thanks!

This is a dupe of aspnet/Mvc#6696. Voicing your opinions there will help us see all the community feedback in one place.


isort Diagnostic Suggestion continues to appear after sorting and on non-import lines

VS Code Version: 1.73.1
OS Version: Windows 10 Version 10.0.19042 Build 19042

With the October release of Visual Studio Code, the isort extension appears to display its error diagnostic even after using the suggestion to correct the sorting of imports.  It also seems to appear on all lines of code, rather than just the imports.

For example, when you open a file and the imports do not follow isort standards, it will display an error diagnostic and provide a Code Action to fix the import order. In this release, we are migrating to use the standalone isort extension instead of the isort support built into the Python extension by automatically installing it alongside the Python extension.

Steps to Reproduce:

Make sure isort is enabled
In a Python file, sort your imports using the suggestion
After applying the suggestion, hover cursor over import again to see the diagnostic is still present
Hover cursor over a non-import line and see suggestion appear

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Updates COVID-19 screen

Changes in this PR
Updates the COVID-19 Apply screen in line with new designs.

Before

After

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Have all changes to any dependencies been deployed to both preprod and
production in a backwards compatible way? (This includes our API,
infrastructure, third-party integrations and any secrets or environment variables)
[ ] Has a required data migration been included in this change or been done in
advance?

Post merge checklist
Once we've merged we now need to release this through to production before
considering it done.
Find the build-and-deploy job in CircleCI
and work through the following steps:

[ ] Has the product manager asked to provide approval for this feature?

[ ] Has the product manager approved?

[ ] Manually approve release to preprod
[ ] Verify change released to preprod
[ ] Manually approve release to prod
[ ] Verify change released to production

Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

 Update COVID-19 screen


Remove use of JLine outside of accumulo-shell

JLine is used for a REPL command-line tool in accumulo-shell. However, we seem to also be using it for trivial line reading for non-REPL use cases. This is overkill, and makes it more likely that JLine upgrades will break code that doesn't need to depend on it. For many of these cases, a simple System.console().readLine() (or .readPassword()) will do.
JLine should be removed from all simple line reading parts of our code, and used only where it is actually needed (most likely, the only place it's actually needed is in accumulo-shell and test code).

@ctubbsii  - I can take this.


gitea logo

URL for the brand / site / app
https://gitea.io/en-us/
Logo resources for the icon
No response
Anything else?
No response

This is final warning, you are banned at next duplicate issue. Stop rushing at making the issues.
Already exists: https://aegis-icons.github.io/icons/primary/Gitea.svg

sorry i actually did not find that some times ago my mistake. i will be careful


14.12 bootstrapping downloads no longer work

If one downloads stdenv from binary cache, downloading with that one works, but one is unable to bootstrap (without prefetching manually). Example output:
curl: (6) Couldn't resolve host 'tarballs.nixos.org'

trying http://tukaani.org/xz/xz-5.0.7.tar.bz2
Warning: Transient problem: timeout Will retry in 1 seconds. 3 retries left.
Warning: Transient problem: timeout Will retry in 2 seconds. 2 retries left.
Warning: Transient problem: timeout Will retry in 4 seconds. 1 retries left.
curl: (6) Couldn't resolve host 'tukaani.org'
error: cannot download xz-5.0.7.tar.bz2 from any mirror

I also tried with bzip2, and it looks the same. Interestingly, just running the bootstrapping curl by hand fetches the files.
Noticed by @devhell on IRC.

I can not switch to release-14.12 too, but because MesaLib 10.2.9  is missing from ftp.freedesktop.org:
building path(s) ‘/nix/store/hx3xsi2z51w21hpdsy6mm0f07xd32l0s-MesaLib-10.2.9.tar.bz2’

trying ftp://ftp.freedesktop.org/pub/mesa/10.2.9/MesaLib-10.2.9.tar.bz2
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0
curl: (9) Server denied you to change to the given directory
error: cannot download MesaLib-10.2.9.tar.bz2 from any mirror
builder for ‘/nix/store/qp0px1v204x4ig5w4i07n1sbqwn8pv4x-MesaLib-10.2.9.tar.bz2.drv’ failed with exit code 1

@Sorokin: that should be fixed by a5444111bc1, contained in the current nixos-14.12 channel.

@vcunat probably I tried too old version. I confirm that it is fixed now.

I think this was fixed in master in 3d7f66e955b64735a6881c9d0fc36ede45f5168e. This problem should also only occur with Nix 1.9 (because it made fetchurl run in a chroot), but 1.8 is the default in NixOS 14.12. A workaround on 1.9 is to set build-use-chroot to relaxed.

Hmm, so only nix-1.9 and channel 14.12... that's probably not worth fixing. 14.12 will be well-supported only for a few months from now, I suppose.

No use to keep this open anymore...


Update high-resolution-photos.md

Issue #2716 - Missing exchange versions are updated.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin Following up on this PR, please let me know if there is anything I can do to assist. Thanks!

@sowjanya365, please commit the copy edit changes made by
@JohanFreelancer9 JohanFreelancer9.  I'll merge after those changes are commited. Thank you.

@sowjanya365, please commit the copy edit changes made by
@JohanFreelancer9. I'll merge after those changes are commited. Thank you.

Hi @LanaChin
Done. Thank you! :)


Fix bug for accept wildcard case in GraphqlService

Motivation:
If accept is a wildcard in GraphqlService it won't work.
Modifications:

Fix GraphqlUtil

Result:

Fix bug

Thanks, @heowc! ❤


Add hide on silent option

This adds an option to hide the bar when the voice is below what vrchat detects

This approach won't work if you change your mic sensitivity like you would with MicSensitivity. Something that could work is to check when the icon goes dark (not talking) or bright (talking). I'll take a look at it when i some spare time.


Fix duplicate chat message notifications

Contains
Fixes #2990
How to test

Start up 2 Terasology clients and connect to a server with client 1.
Connect to the server with client 2 and confirm that client 1 receives a popup notification.
Send a message from client 2 and confirm that client 1 receives a popup notification.
Open the chat window on client 1 (leave it open for the remaining steps).
Send a message from client 2 and confirm that client 1 sees the message, but does not receive a popup notification.
Disconnect client 2 from the server and confirm that client 1 sees the notification, but does not receive a popup notification.

Can one of the admins please verify this patch?

Belated thanks for the PR! Triggering our bot for testing :-)
@GooeyHub: ok to test
Anybody want to take this for a spin? Maybe @TheFlash98? :-)

Hooray Jenkins reported success with all tests good!

Hey @Cervator tested the PR exactly how the instructions have been given and it works well on my machine. 🙂

Thanks both! Appreciate the work. Merged :-)
@Derpthemeus added you to our credits, hoping to see more work from you!


Code completion selection background is inconsistent

The background colour of the selected item in the code completion popup is different depending on how the pop was triggered.
If the popup is triggered manually (by pressing ctrl+space) then the background of the selected item is lighter than the other items.

If the popup is triggered automatically as you're typing then the background of the selected item is darker than the other items (check the option "show suggestions as you type" to see this behaviour).

Having the selected item darker in one case and lighter in the other is very confusing.
Also the background of the selected item is too similar to the other items in both cases. It would be easier to see which item is selected if there was a bigger difference between the background colours.

In the second case, the background colour changes from dark to light when the selection changes.

Hi @cjkent :wave:, thanks for your contribution :+1:
Sorry for the delay, life keeps me busy.
Is the problem the same like described in #95? If so, a fiy has been submitted in #118 that'll be included in the next release version 0.10.0.

Sorry for the delay, life keeps me busy.
Don't apologise, thanks for the beautiful theme!

It's very similar, but it's not quite the same problem. In #95 the background for the selected item is missing (or is the same as unselected items) when the completion popup appears automatically.
The problem I see is that the background of the selected item is darker than unselected items when the popup appears automatically, and it is lighter than the background of unselected items when you press ctrl-space.
It seems possible that the underlying cause is the same and my problem will be fixed in 0.10.0.
If the problem is still there in 0.10.0 I'll comment again on this issue.

It's fixed in 0.10.0.
Thanks again for a great theme.


Bump deps (maintenance)

Trying to get ahead of dependabot.

@pastuxso fyi, might want to rebase your branch... finally upgrade to vitest v1.x and some other major deps


File upload is not possible when using custom roles

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I created a custom role using create custom_role;, but cannot upload the file.
To Reproduce

clone the following repository
https://github.com/harukaxq/role_test

supabase start

create a file in front-example/.env.local
NEXT_PUBLIC_SUPABASE_URL=http://localhost:54321
NEXT_PUBLIC_SUPABASE_ANON_KEY=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM 4MTI5OTZ9.CRXP1A7WOeoJeXxjNni43kdQwgnWNReilDMblYTn_I0

yarn dev in front-example

open http://localhost:3000/login and do a signup

update role to staff
update "auth"." users" set role='staff';.

open http://localhost:3000/login and login

open http://localhost:3000 and upload an image

an error is displayed

The above is reproduced in the local environment, but the same error occurs on supabase.com.
Expected behavior
Uploading is normal even after switching to staff role.
System information

OS: macOS
Browser (if applies) chrome
Version of supabase-js: 2.38.1
Version of Node.js: v18.15.0

Additional context
Similar problems occur in flutterflow.
I read the following issue and ran "grant staff to authenticator;grant anon to custom_role;" but that didn't solve the problem
https://github.com/supabase/storage/issues/369

Related to #369
You'll need to grant the following permissions to your custom role:
grant custom_role to authenticator;
grant anon to custom_role;


Allow overriding the Window Type

I think we can use this pattern:
https://github.com/AvaloniaUI/Avalonia/pull/2777/files#diff-fc612e8a5a5f1b90798b9431662b51e1R20
either that or some kind of DataTemplate?
Probably metro window shouldnt be part of this project, and user should be able to configure it or another one?

That's huge diff from AvaloniaUI, not sure what to look for.
Anyway, we already allow user to override Window type, see:
https://github.com/wieslawsoltes/Dock/blob/7660b73379b7ce4d9ec35ce4e70d1555191a6268/samples/AvaloniaDemo/Factories/DemoDockFactory.cs#L312-L328
The user Window class needs to only implement IDockHost interface and HostLocator will do the rest:
https://github.com/wieslawsoltes/Dock/blob/7660b73379b7ce4d9ec35ce4e70d1555191a6268/src/Dock.Model/Core/IDockHost.cs#L9
https://github.com/wieslawsoltes/Dock/blob/7660b73379b7ce4d9ec35ce4e70d1555191a6268/src/Dock.Avalonia/Controls/HostWindow.cs#L17

Also the window controls can be templated by user:
https://github.com/wieslawsoltes/Dock/blob/7660b73379b7ce4d9ec35ce4e70d1555191a6268/src/Dock.Avalonia.Themes.Default/HostWindow.xaml#L11
https://github.com/wieslawsoltes/Dock/blob/7660b73379b7ce4d9ec35ce4e70d1555191a6268/src/Dock.Avalonia.Themes.Default/MetroWindow.xaml#L6

@danwalmsley I can remove the MetroWindow style from both HostWindow and remove completely the MetroWindow or just move to separate sproject.

@danwalmsley I have done some refactoring and moved metro style for windows to separate project.
https://github.com/wieslawsoltes/Dock/tree/master/src/Dock.Avalonia.Themes.Metro
Now by default you will get standard window style, still I have left custom window types, as I thing we will need them anyway in future.

Closing as its already possible using HostLocator and default host window is templated.


Is there a way to make the $ref even more reusable with placeholders?

My use case is the following:
I have several schema definitions. The API should return these definitions always wrapped in a certain structure. So my idea would be to have something like this:
paths:
  /path
     responses:
       '200':
          content:
            application/json:
              schema:
                $ref:
                  scaffold: '#/components/schemas/dataContainer'
                  placeholders:
                     data: '#/components/schemas/pet'
components:
  schemas:
    dataContainer:
       type: object
       properties:
         data:
           type: placeholder
           name: data

    pet:
      type: object
      properties:
         arbitrary:
            type: string

Searching for placeholder, nested $ref, DRY, and similar variations did not lead me to satisfying solution.  I found the allOf keyword but this forces me to rebuild the structure that should be provided by the scaffold.
There is also something comparable for the servers key with url and variables.
Is something comparable possible to reuse schema scaffolds?
If not I would rephrase this issue as feature request.

This has been a fairly common request over the years.  Sometimes it is described as an envelope or wrapper for the internal data.  The issue is more of a schema related issue.  If you can find a schema language that supports this concept then you may be able to use that using with the proposed alternative schema feature.
From an API design perspective, I often find that envelopes around data are just duplicating what HTTP headers can already do.

https://github.com/json-schema-org/json-schema-spec/issues/322 is the JSON Schema issue closest to this.  It has not really found a champion, and it does open up considerable amounts of complexity.  But we haven't rejected it either.  It definitely won't go into draft-08, which is now wrapping up, but could possibly get into a future draft if someone can advocate for it successfully.
I'm personally a bit skeptical as I've been down this road of essentially implementing function calls in what was supposed to be a declarative system before, and it was mostly a sign that the original solution was being re-purposed in a way that didn't really fit, but feel free to comment on the issue :-)

This issue has been listed as a possible use case for Overlays. I don't think it would help here though - you could have overlays replace a value in the contract, but the replacement value would be fixed/static in the overlay, it would not be able to dynamically merge to elements.

Do $dynamicAnchor and $dynamicRef (available in OAS 3.1) help here?  They allow you to reference something that is determined at runtime.  Sadly they're a bit confusing- json-schema-org/json-schema-spec#1140 has a proposal for simplifying them.


Show settings after first install, new features list with images after update

Should show welcome page and settings.
Maybe show settings page with added descriptions.
Example:
chrome.runtime.onInstalled.addListener(function (object) {
    chrome.tabs.create({url: "http://yoursite.com/"}, function (tab) {
        console.log("New tab launched with http://yoursite.com/");
    });
});

Closing in favour of #1199 for second part.


[tbb] File does not have expected hash

The vcpkg package tbb fails to build on x64-linux and x86-windows (osx not tested).
File does not have expected hash:

File path: [ /home/sh/dev/vcpkg-2020.01/downloads/temp/intel-tbb-4bdba61bafc6ba2d636f31564f1de5702d365cf7.tar.gz ]
Expected hash: [ 0b00c9deefdac5dc1f4fbae314e91eb3513b54b47ff6dec08ed2460486fc7d211ab36d6130e5787bfd50523cb613c65f03f9217d967292ca9056e2d3f5010bf8 ]
Actual hash: [ f2a8d7e0476f846039390f4a79af3fe13770e23b01bf4741e738136f7ddb401357a0e50f35212e8d0fa5fc4cf1563418337309227d7243fc3676edd406ae652d ]

Package: tbb:x64-linux
Vcpkg version: 2020.01.17-unknownhash

@shauser67, coud you get latest source(git pull) and try to build tbb again? from your log, it seems you are using the older revision.
PS F:\vcpkg\src> ./vcpkg install tbb
Computing installation plan...
The following packages will be built and installed:
tbb[core]:x86-windows
Starting package 1/1: tbb:x86-windows
Building package tbb[core]:x86-windows...
-- Downloading https://github.com/oneapi-src/oneTBB/archive/427c252e0bb9e191767a62d8a744b21950c343f6.tar.gz...
-- Extracting source F:/VCPKG/src/downloads/oneapi-src-oneTBB-427c252e0bb9e191767a62d8a744b21950c343f6.tar.gz
-- Applying patch fix-static-build.patch
-- Applying patch terminate-when-buildtool-notfound.patch
-- Using source at F:/VCPKG/src/buildtrees/tbb/src/1950c343f6-81f8b97441
-- Building build/vs2013/makefile.sln for Release
-- Building build/vs2013/makefile.sln for Debug
-- Installing: F:/VCPKG/src/packages/tbb_x86-windows/share/tbb/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package tbb[core]:x86-windows... done
Installing package tbb[core]:x86-windows...
Installing package tbb[core]:x86-windows... done
Elapsed time for package tbb:x86-windows: 1.639 min
Total elapsed time: 1.643 min
The package tbb provides CMake targets:
find_package(TBB CONFIG REQUIRED)
target_link_libraries(main PRIVATE TBB::tbb)

Thank you, this solved the issue!


Making sense of the log.

I'm unable to understand the output of even specialized log plug-ins like metalsmith-logger.
I seem to end up with multiple [object Object] files in which I can't seem to resolve..
How does one figure out what data is being passed internally?
GraphQL does this part really well...

I'm not quite sure what you're asking. Can you post a MCVE of the problem you're experiencing?

@Zearin unfortunately I was using gulp-metalsmith so the debug ui wasn't working for me..

Can you try going without gulp-metalsmith just long enough to use metalsmith-debug-ui and determine the problem?  You can always figure it out and then go back to your normal setup afterwards....


fix: use contexts instead of extras

Additional Data (extras) is deprecated.
refs. https://docs.sentry.io/platforms/go/enriching-events/context/#additional-data

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


Docs: Tree with nested node examples do not work and crash

Bug
What is the expected behavior?
The examples for trees with nested nodes should work
What is the current behavior?
The docs app throws uncaught errors in the console, visit the section and look for yourself.
What are the steps to reproduce?
https://material.angular.io/components/tree/overview
https://material.angular.io/components/tree/examples
Visit these
What is the use-case or motivation for changing an existing behavior?
It speaks for itself
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
I'd assume the latest ng and mat versions
Using Chrome 67
Is there anything else we should know?
Nope.

Fix found here: https://github.com/angular/material2/blob/master/src/material-examples/cdk-tree-nested/cdk-tree-nested-example.ts
This works on the nested example and the checkbox example which is also currently broken.
Change this line:
private _getChildren = (node: FileNode) => node.children;

to this:
private _getChildren = (node: FileNode) => observableOf(node.children);

Change this import:
import {BehaviorSubject} from 'rxjs';

to this:
import {BehaviorSubject, of as observableOf} from 'rxjs';

The errors in the console for the examples don't appear anymore


[video_player] Formatting issue with uncommon aspect ratios.

Hi there,
I face an issue when using the video_player plugin with videos having uncommon aspect ratios.
For example, the video texture will vertically overflow its container when the video has a fairly small aspect ratio:

The red block on the right is a container having the aspect ration given by the VideoPlayerController, while the actual video on the left is displayed incorrectly.
It can be reproduced on the latest version using the example project in this fork.
Video to reproduce
https://user-images.githubusercontent.com/5921144/157257830-342e66cd-05e2-4e11-8ee8-50bbaf919007.mp4

flutter doctor -v

[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.22000.527], locale en-US)
    • Flutter version 2.8.1 at C:\tools\flutter_windows_2.2.3-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (3 months ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at C:\Users\Tom\AppData\Local\Android\sdk
• Platform android-31, build-tools 30.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 2021.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
[√] VS Code, 64-bit edition (version 1.64.2)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (3 available)
• Pixel 5 (mobile) • 08131FDD4003UU • android-arm64  • Android 12 (API 31)
• Chrome (web)     • chrome         • web-javascript • Google Chrome 98.0.4758.102
• Edge (web)       • edge           • web-javascript • Microsoft Edge 99.0.1150.30
• No issues found!

This problem doesn't happen on iOS. I only see it on Android, at least on Pixel and Honor phones.
Let me know if you need anything else to diagnostic this issue.

@TomVeniat
Thanks for the detailed report. It seems that the issue is video doesn't fit into the parent widget and goes out of the container, that leads me to this similar / related issue: https://github.com/flutter/flutter/issues/51250. Can you check this and see if it's same as your case ?

Thanks For the feedback @TomVeniat.
The stretch at the bottom is the same as you mentioned: https://github.com/flutter/flutter/issues/64233 in which you can follow-up for the fix.
As for the issue you reported, it seems to me that the video doesn't fit into the container goes out of the bounds, which seems to be similar to https://github.com/flutter/flutter/issues/51250. Would like to know how your issue is different than 51250 and if the behavior you reported can be followed up in the original issue.


⚠ Samsung has degraded performance

In 6d02b41, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3745 ms

Resolved: Samsung performance has improved in 5e8e9cf.


Adjustments for PHP 7.4

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
?

Fixed issues
#17698

Now the PHP is fixed. NodeJS part is broken though... not sure why yet if it wasn't changed.

Seems JS tests or code were not changed since last successful build. Still the set fails starting with:
jQuery.Deferred exception: document is not defined ReferenceError: document is not defined
    at initRedirectHandler (evalmachine.<anonymous>:368:11)
    at Object.init (evalmachine.<anonymous>:333:13)
    at Object.initModule (evalmachine.<anonymous>:322:24)
    at Document.<anonymous> (evalmachine.<anonymous>:532:16)
    at mightThrow (about:blank:3557:29)
    at process (about:blank:3625:12)
    at Timeout.callback [as _onTimeout] (C:\dev\yii2\node_modules\jsdom\lib\jsdom\browser\Window.js:628:19)
    at ontimeout (timers.js:436:11)
    at tryOnTimeout (timers.js:300:5)
    at listOnTimeout (timers.js:263:5) undefined

Here we go. Not sure why it was green before :)

Preferred less invasive solution: https://github.com/yiisoft/yii2/pull/17725


Fixed a tiny typo in our name in your docs

Otherwise...looks awesome & I'll add to our docs tonight.
What's your Postmark account so I can hook you up with some free credits?

Just noticed it myself.  Pushed a Fix just before I noticed you forked it.
account is<EMAIL_ADDRESS>Skylar S.
On Jan 29, 2012, at 5:22 PM, Alex Hillman wrote:

Otherwise...looks awesome & I'll add to our docs tonight.
What's your Postmark account so I can hook you up with some free credits?
You can merge this Pull Request by running:
git pull https://github.com/alexknowshtml/SSPostmark master
Or you can view, comment on it, or merge it online at:
https://github.com/skylarsch/SSPostmark/pull/1
-- Commit Summary --

Postmark misspelled

-- File Changes --
M README.md (2)
-- Patch Links --
https://github.com/skylarsch/SSPostmark/pull/1.patch
https://github.com/skylarsch/SSPostmark/pull/1.diff

Reply to this email directly or view it on GitHub:
https://github.com/skylarsch/SSPostmark/pull/1


fix: bump helloworld version to fix ci

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 41.499%

Files with Coverage Reduction
New Missed Lines
%

pkg/client/client.go
6
56.35%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.07%

Covered Lines:
3598

Relevant Lines:
8670

💛  - Coveralls


3074 UI unable to scroll click nfts in sidebar

Fixes #3074
With this fix, you can scroll and click the NFTs. You can drag and drop the NFTs to the world and it will upload the application metadata.
But, that upload doesn't put anything in the world.

@alisaad673 Yes those design updates should be a separate issue + PR.


Enhancement:  Project modularity

The project should be split into an SDK project and a separate application layer project.
The SDK should contain the existing service layer + model abstractions, the app layer will contain UI and UIViewControllers.
I'll give it a go over the weekend :)

Started working on this.  For the most part it is going pretty smooth.
Will require some refactoring as certain model/service entities depend on APIs that are not available to extensions and on view layer logic, but I don't expect it to be much work.

We've already been down that path with onebusaway-iphone-common. I'd love to know what your use cases are for this work that will prevent it from being another iteration of that.

I do remember seeing references to that in the past, but don't recall what it tried to achieve?
The idea behind this change is to split View + Controller logic from service/model layer and expose the service / model layer as a re-usable static library.
The app and the extensions would then link against this static library.
Sure, the code could technically be re-used through target inclusion, but that does not really scale tbh.

Ok sounds good and thanks. I think the idea behind OBA common was roughly along the same lines as this, but it sort of withered due to a number of factors.


[2645] Exclude trainees that are already present from the Apply import

Context
https://trello.com/c/GzTIv695/2645-s-exclude-trainees-that-are-already-present-from-the-apply-import
Changes proposed in this pull request

Add state enum to ApplyApplication to hold 3 possible states: importable, provider_a_hei, duplicate
Update ApplyApi::ImportApplication to set the correct state
Change guard clause in ApplyApi::ImportApplicationsJob so that it only creates trainees from importable apply applications
Update respective tests

 [S] Exclude trainees that are already present from the Apply import


Strange access issue with Ruby and network drives

Hi,
I'm trying to access 2 network drives from msys2 shell using ruby. One works, while the other doesn't.
The same command works with msys1.
ruby -e 'puts Dir.glob("//server1/some/path/*")' # works
ruby -e 'puts Dir.glob("//server2/other/path/*")' # works with msys1, fails with msys2

Needless to say, the same version of Ruby is used on both environments.
Process Monitor shows that for server2 CreateFile is invoked for //server2/other/path/ with a trailing slash, which results in Access Denied, while with msys1 or cmd, and for server1 even with msys2, there isn't a trailing slash.
The path length is the same for both servers (1 share name, 1 path under it).
I have no idea what type of servers these are.
Is there a way to debug this problem?

Where'd you get your ruby from?

RubyInstaller. Version 2.1.5p273.

Do both folders contain files or is one of them empty?

Can you show the output from:
ruby -e 'print ["//server2/other/path/"]'
and:
ruby -e 'print ["//server1/other/path/"]'

Both have files obviously.
Did you mean Dir["//server1/other/path/"]?

$ ruby -e 'print Dir["//server1/some/path/"]'
["//server1/some/path/"]
$ ruby -e 'print Dir["//server2/other/path/"]'
["//server2/other/path/"]

It is not obvious at all, and why would it be?
I wanted you to reply with what I asked for as I want to see if MSYS2 mangles your arguments.

The output is just the same without Dir.
If there weren't files and it would return an empty list, why would I file an issue?

Can you check the command lines that are used to invoke ruby in Process Monitor? (It's the "Process Start" event.)

You weren't clear about what "One works, while the other doesn't" meant.
Github mangled my "*"'s, I wanted to see the output of:
ruby -e 'print ["//server1/some/path/*"]'
ruby -e 'print ["//server2/other/path/*"]'

.. but the Process Start command lines should be enough anyway.

The Process Start looks the same for both paths:
D:\Ruby\bin\ruby.exe -e "print Dir[\"//server1/some/path/*\"]"
D:\Ruby\bin\ruby.exe -e "print Dir[\"//server2/other/path/*\"]"

Do they look the same when started from MSYS and MSYS2?

Exactly the same. It's the same also on cmd.

I wonder what else could be affecting the results. What terminals are you using?
Can you also try from Cygwin?

Doesn't work in cygwin (both 32 and 64 bit).

And terminals? Is it possible that the difference between the working environments and the non-woking ones is in mintty?

On cygwin I use the default terminal. Which other terminals are available?

The two basic options are the Windows console host and mintty. I don't know which one is the default for Cygwin, but MSYS2 uses mintty as default and AFAIK MSYS1 uses the Windows console.

The default of MSYS1 (which works) looks exactly like the default for Cygwin (which fails). MSYS2 uses mintty here, so it doesn't look related to the terminal type.

Cygwin and MSYS2 both use a very recent mintty, MSYS1 uses a very old mintty.

Is there a way to pick an alternative shell with cygwin?

From cmd.exe, do:
C:\msys64\usr\bin\bash --login -i

.. and see what results you get.

Fails.

.. and finally, same thing without --login -i

Fails.

 The WIP git-for-Windows one or ours?

Tried both. But official cygwin also fails.

I'm fairly out of ideas; if you are able to post detailed procmon logs somewhere then we can scan those for differences.

This is a full procmon log filtered by ruby.exe process name.
The working server is //netapp1/CM/CompilationResults, the failing one is //aclnas01/versions/CompilationResults.
The log contains 2 successful executions with MSYS1 which succeed (1 for each server), then a successful execution with MSYS2 to netapp1, and the last one is for aclnas01 which fails (notice ACCESS DENIED at line 9902).

The servers seem to have different access permissions.
Working one has:
"16:11:16.2122568","ruby.exe","6904","QueryNetworkOpenInformationFile","\\netapp1\CM\CompilationResults","SUCCESS","CreationTime: 20/04/2004 14:51:32, LastAccessTime: 07/05/2015 16:11:05, LastWriteTime: 13/04/2015 17:02:45, ChangeTime: 07/05/2015 00:41:01, AllocationSize: 01/01/1601 03:00:00, EndOfFile: 01/01/1601 03:00:00, FileAttributes: D"

Non working one has:
"16:11:18.9542818","ruby.exe","8676","QueryNetworkOpenInformationFile","\\aclnas01\versions\CompilationResults","SUCCESS","CreationTime: 27/01/2015 23:03:32, LastAccessTime: 07/05/2015 16:11:09, LastWriteTime: 28/01/2015 17:24:09, ChangeTime: 24/02/2015 16:23:19, AllocationSize: 01/01/1601 03:00:00, EndOfFile: 01/01/1601 03:00:00, FileAttributes: DA"

A is archiving .. are these both just normal smb shares? Can you remove archiving flag to see if it makes any difference?

This may be of interest too:
http://blogs.msdn.com/b/oldnewthing/archive/2009/02/19/9432917.aspx
A could mean "needs archiving" but it could also mean encrypted or compressed.

Also, MSYS1:
"16:11:09.1113553","ruby.exe","3036","CreateFile","\\aclnas01\versions\CompilationResults\","SUCCESS","Desired Access: Read Data/List Directory, Synchronize, Disposition: Open, Options: Directory, Synchronous IO Non-Alert, Attributes: n/a, ShareMode: Read, Write, Delete, AllocationSize: n/a, OpenResult: Opened"

vs MSYS2:
"16:11:18.9558312","ruby.exe","8676","CreateFile","\\aclnas01\versions\CompilationResults\","ACCESS DENIED","Desired Access: Read Data/List Directory, Synchronize, Disposition: Open, Options: Directory, Synchronous IO Non-Alert, Open For Backup, Attributes: n/a, ShareMode: Read, Write, Delete, AllocationSize: n/a"

I'm not sure what this "Open For Backup" means, but it does sound archiving-related.

This thread looks very interesting in this regard:
https://cygwin.com/ml/cygwin/2008-02/msg00076.html

I'm not sure what this "Open For Backup" means, but it does sound archiving-related.

AFAIK, this is primarily used for archiving purposes, but is also used by Cygwin to gain access to files that would be inaccessible without it (in case the user/process possesses SeBackupPrivilege), probably to emulate the all-doors-open rights of root accounts in the POSIX world.

https://cygwin.com/ml/cygwin/2008-02/msg00076.htm

Out of curiosity, I built the MSYS2 runtime with all occurrences of "Open for Backup" flag removed: msys-runtime, msys2-runtime.sig, msys-runtime-devel, msys2-runtime-devel.sig. I'm not sure if I actually succeeded, but give it a try.

@elieux: Thanks for your effort. Unfortunately, it doesn't solve the issue.

Can you remove archiving flag to see if it makes any difference?

@mingwandroid I only have read access to these servers. I can't test that.

Ok, I was able to reproduce with a minimal working example:
#include <windows.h>

int main()
{
    const char *name = "//aclnas01/versions/CompilationResults";
    WIN32_FIND_DATA fd;
    FindFirstFile(name, &fd); // ACCESS DENIED
    return 0;
}

Any ideas how to investigate it further?

2 days ago I posted this bug report to the cygwin mailing list. It still wasn't answered.
Some more insights follow.
Using Wireshark, I discovered that aclnas01 uses SMB, while netapp1 uses SMB2.
For some reason, when executed from command prompt, a FIND_FIRST2 request is sent and the server replies, while with cygwin a "NT Create AndX" request is sent and denied.
To minimize the captured packets, I executed the application twice, and captured only the second execution (to avoid session initiation).
Working capture
Failing capture
Does cygwin implement SMB on its own, or is this difference a
side-effect of the "Backup Intent" flag?

I just found another SMB1 linux server, which does work.
It first has a "NT Create AndX" request for path , which succeeds. Then it issues Trans2 FIND_FIRST2 request for the real path (\a).
This issue might be related to the Archive bit which is set on aclnas01.

Hi,
After investigation, I found that the root cause for this problem is
set_cygwin_privileges, which sets SE_RESTORE_PRIVILEGE and
SE_BACKUP_PRIVILEGE for the process during initialization.
Commenting out these 2 lines solves the problem for me.

Ok, I was able to fix the issue by removing the privileges in my script.
If anyone with this problem reaches here, the script follows:
  def remove_privileges
    return unless RUBY_PLATFORM.include?('mingw')
    require 'win32/api'

    _AdjustTokenPrivileges = Win32::API.new('AdjustTokenPrivileges', 'LBPLPP', 'B', 'advapi32')
    _OpenProcessToken  = Win32::API.new('OpenProcessToken', 'LLP', 'B', 'advapi32')
    _GetCurrentProcess = Win32::API.new('GetCurrentProcess', 'V', 'L', 'kernel32')
    _TOKEN_ADJUST_PRIVILEGES = 0x0020

    token = [0].pack('l')
    _OpenProcessToken.call(_GetCurrentProcess.call, _TOKEN_ADJUST_PRIVILEGES, token)
    token = token.unpack('l').first
    _AdjustTokenPrivileges.call(token, true, 0, 0, 0, 0)
  end


Added NPX Cheatsheet

What does this PR do?

Added npx cheatsheet for beginners

Description

NPX is a very handy replacement of npm for initiating react projects so, I have added npx cheatsheet.

Screenshots or Links

If applicable, add screenshots or a modified UI link to help understand your contribution.

Additional context

Add any other context about your PR.

Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please review and merge :)

Please merge, now I have resolved the conflicts...


Add UptoSite

Add UptoSite resources
Upto.Site is a free URL shortening service that allows users to create short URLs quickly and easily.
Be sure to complete and check the following steps:

[x] My changes were made in the source-web/src/assets/db folder.
[x] The resource(s) is completely or partially free.
[x] The resource(s) is not already in the database.
[x] All resources have an id, order, name, link and img.
[x] The ids and urls of the images are not repeated.

Thank you for your contributions, just a couple of notes:

When contributing multiple resources, try to unify them into a single pull request.
A tool always needs a tag or tags that describe its function.


Bindgen support

Closes #74
Can you direct me how to implement integration test, or could you do that?

Probably just add these command https://github.com/rust-cross/cargo-zigbuild/issues/74#issuecomment-1444862147 in https://github.com/rust-cross/cargo-zigbuild/blob/main/.github/workflows/CI.yml

Probably just add these command #74 (comment) in https://github.com/rust-cross/cargo-zigbuild/blob/main/.github/workflows/CI.yml

Will do that.

Added CI test. Would it suffice?

FYI, the unrelated test failures are tracked in https://github.com/rust-cross/cargo-zigbuild/issues/96.


Update readme with demo and use the up command

I've updated the README to include a running demo and to highlight the easy, one-liner up command over the 3 stage cluster bring-up.

Looks good to me.


Initial corpus for sv2_noise_cipher_roundtrip

Generated by running for about two hours on an AMD Ryzen 7950x (32 threads) with Ubuntu 23.10 using Clang 16.0.6 and libFuzzer.
I used --with-sanitizers=fuzzer. Merged using --m_dir into 1.3MB.
I'll do a longer run once https://github.com/bitcoin/bitcoin/pull/29346 has had more review.

Yea, this also seems a bit premature given the discussion in the linked (and related) pull requests.


That commit is intended and I thought it wouldn't affect Bazel but it looks like Tensorflow is still relying on the deprecated legacy whole archive behavior. The flag incompatible_remove_legacy_whole_archive was already flipped many releases ago but TF unflipped it in its bazelrc.
This behavior has to stay, without this bug fix, part of the reason for the existence of cc_shared_library disappears since one of its main goals was to avoid linking libraries into the shared object that didn't need to be linked. With the legacy whole archive behavior, anything may be linked unnecessarily.
For each error like this that you get you have two options, for both options you first have to find the cc_library that the missing symbol belongs to, then:

Add the cc_library as a direct dep of cc_shared library to cc_shared_library.deps (cc_shared_library.roots was renamed to cc_shared_library.deps)
Leave the library where it is and add alwayslink=True

@jakeharmon8

Looks like --features=-force_no_whole_archive is an workaround for this, but TF should really migrate off depending on the old behaviour.

How is the Bazel version upgrade relevant? In any case, I don't have the cycles to fix the TF build.
The longer TF remains broken, the harder it is to fix the breakage. TF is failing with a different error now: https://buildkite.com/bazel/bazel-at-head-plus-disabled/builds/1692#01889ec5-aa99-41a8-9e11-487dc4866b72

According to the churn policy most of the time spent on changes must be spent by the team mandating the change. TF is green at 6.1.0. It is broken at Bazel@HEAD. It seems reasonable to me that the Bazel team helps resolve the majority the issue.
CC @meteorcloudy

We have spent plenty of time to bisect TF and identify the breaking change (https://github.com/bazelbuild/bazel/commit/661ebef465b08697a82fc3ee4c9fd540d953dc84), and as @oquenchil explained, this is broken because TF still depends on the deprecated legacy feature (whole archive).
I found a workaround for TF by adding --features=-force_no_whole_archive as a build option, but eventually TF needs to migrate away from depending on the legacy behaviour.
I guess we can add first apply the workaround to stop the bleed here, sent https://github.com/tensorflow/tensorflow/pull/60837


[ICD] Increase ICD Monitoring buffer size to fit maximum sized entry

Description
ICD Monitoring table buffer size was not large enough to fix a entry with each field at its maximum size. PR increases the buffer to fit an entry with all its max values.
Tests
Added unit tests to validate the behavior change.

Combined the PR with https://github.com/project-chip/connectedhomeip/pull/35737/files


Feature Request: Allow managing firewall rules

After creating a cluster via terraform, I need to manually set the firewall rules. I would much prefer the safety of handling this in terraform

Thanks for the feedback, we'll increase the priority for this feature against the other API additions to the provider.


ip not updating

I set this up about a month ago, and it was working perfectly. I tested by going into my cf dashboard and changing the ip on my two A recs (www and mydomain.com) and withing a few seconds they were changed back to the correct ips. But today my ip actually changed from my isp for the first time since installing this and it did not update. The service is still running, but doesn't seem to be working. Any way to troubleshoot?
win10 64 with nginx
Edit: latest logs show cfddns updater is running every 5 minutes... just not updating anything

I'm having a similar issue. Tool was working fine until Cloudflare updated their security and it broke the tool. So I downloaded the newer version of the tool. (currently on <IP_ADDRESS>-dev) and it fixed the security issue but now the updater isn't actually updated the IP address with Cloudflare. Weird part is that it can see the correct IP at the top on the GUI but wont actually update the record to the new dynamic IP.
If I right click on the record in the tool and select "update ip" manually then give it a few minutes. It will update.
Not sure if it has to do with the version I'm on. Going to try the latest tool and see if that fixes it.


feat(bridge): Cleanup of obsolete feature in Bridge

This PR
Cleanup of obsolete feature in Bridge #7378
Related Issues
Removed the button Load more information about integrations and below label
Fixes #7378

Thanks @Brijesh-09
You now just need to fix some conflicts, ideally by rebasing your branch to keptn:master
and also please fix the DCO issue by signing your commits, as @thisthat mentioned in his comment.

@Brijesh-09 some of your commits are not signed yet.
Following steps should fix the DCO check:

In your local branch, run: git rebase HEAD~29 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin master

Signed the commits by running the git commands

Hey @ermin-muratovic can you help me  with merging the pull request , I have signed all the commits

Hi @Brijesh-09
there are still some commits that are unsigned and the DCO check is failing.
Please try following commands:

In your local branch, run: git rebase HEAD~30 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin master

@Brijesh-09
we still see the DCO check failing, because some commits are not signed. Please also make sure that for every new commit you use signoff.
Unfortunately you have now also some conflicting files, so you will need to rebase again to keptn:master for the latest changes.

 I did what was mentioned in the DCO still not able to pass the checks

@Brijesh-09 in the screenshot you posted, there were conflicts that you need to resolve before pushing.
The rebase didn't go through completely because of those conflicts.

how can I resolve those conflicts?

@Brijesh-09 you need to use a tool for conflict resolution, you probably don't want to do this on the command line. Have a look at this article about conflict resolution: My favorite tools to resolve git merge conflicts - hopefully those options help you with that.
If you need further support keep on asking here - we will do our best to help you out.

I am not able to find the conflicting files
installer/manifests/keptn/charts/control-plane/templates/core.yaml

That file moved to installer/manifests/keptn/templates/core.yaml, that's why you have a conflict :)

any suggestin on How to solve this conflict?

Git will probably say that the old file was deleted. Leave that file deleted and apply your changes from the old file manually in the new file after the rebase.

I have resolved the conflict between keptn : master branch with my own master branch still I am getting 1 conflict but when I check the code it matches with the master , any help?

Hey @Brijesh-09, I think the conflict is due to a missing new line at the end of the file


feat: add pem export

Description

Related Issue(s)

#{issue number}

Verification

[ ] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Converted to draft as this might not be merged/might not be what we want. Waiting to see what happens with the maskinporten interface.


[Bug Fix] fixed fd server use_paddle parameter parse policy

PR types(PR类型)

Description

fixed fd server use_paddle parameter parse policy

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DefTruth:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add: Height Checker

Codecov Report

Merging #561 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #561   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         175    176    +1
  Lines        1750   1758    +8
=====================================
+ Hits         1750   1758    +8

Impacted Files
Coverage Δ

problems/height-checker/height_checker.go
100% <100%> (ø)

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 3b4042b...24e30f9. Read the comment docs.


Logging is disabled while running tests. This makes it harder than necessary to disable.

Not really sure why, so I add this environment variable

I've used DEBUG=1 bundle exec rspec until now. Can you try if that works for you and let's use that in the documentation instead? I like the message 👍

I tested and it works. It's not perfect though:

The message is printed to the standard $stdout, which might be /tmp/spaceship_tests
there are 2 places with the following code:

unless ENV["DEBUG"]
  $stdout = File.open("/tmp/spaceship_tests", "w")
end

and the above code doesn't say what it does. It might be better to make it more explicit if we also added a message here before overriding $stdout. That way the user knows what is happening. Hence my second commit.

Note that I renamed the output file to /tmp/fastlane_tests

Seems like everything is clear now. Thanks for your contribution, @lacostej 🚀


ceaser decoder

Since I can't see the challenge it's hard to say about the WORD being needed just for tests. It has a code smell, but perhaps you have to do it that way, would need to see the challenge.

I was more so talking about my selection of words, I had attempted to import a list of the top 1000 english words, but ran into issues

I was more so talking about my selection of words

Yup, I see from the challenge that you need this global, a-okay.


[stable/factorio] Add home and icon

PR in the context of normalizing/improving metadata in official charts. Refs https://github.com/kubernetes/charts/issues/124 and https://github.com/helm/monocular/issues/93
NOTE: I have not bumped the chartVersion attribute in order to avoid race conditions/conflicts with existing PRs. Maintainers should be able to change this value before merging (let me know if you prefer that I update it instead)
cc/ @gtaylor

@migmartri Looks like some postgres related stuff ended up mixed in here. Is that intentional?

@migmartri @gtaylor That's happened in a few of these updates. I'm dropping that commit as I version bump.

@k8s-bot ok to test


Unable to do this with Teams Toolkit

This gives errors is you implement this with a brand new teams toolkit visual studio 2022 project. So how do you do graph api with teams toolkit?

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

ID: 896be20d-1321-0e2f-3efb-d0ce8f1e631a
Version Independent ID: f5e2b80f-9478-7500-2bc5-930237274f4b
Content: Use Graph API with ASP.NET Core Blazor WebAssembly
Content Source: aspnetcore/blazor/security/webassembly/graph-api.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @ziron999 ... I'm not familiar with "Teams Toolkit," nor do I understand why any tooling in VS would have anything to do with the coverage in the article.
Beyond that, this is an older article that will be receiving work (or replacement with an Azure document on MS Graph) in the holiday period next month into December, roughly between 🦃 Thanksgiving and the holidays 🎅🎆. That work is tracked by https://github.com/dotnet/AspNetCore.Docs/issues/24615, and I hope to have the work done by the EOY.
You're welcome to post the errors that you're seeing here for me to take a look at. If there's any work to be done, it would likely be performed a little later due to the upcoming release of .NET 7 in mid-November.

ok, i will as soon as i can. this is pretty critical for me to be able to make apps correctly in Teams.

Error with no provider:
System.AggregateException: 'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: Microsoft.Graph.IAuthenticationProvider Lifetime: Scoped ImplementationType: GraphClientExtensions+NoOpGraphAuthenticationProvider': Unable to resolve service for type 'Microsoft.AspNetCore.Components.WebAssembly.Authentication.IAccessTokenProvider' while attempting to activate 'GraphClientExtensions+NoOpGraphAuthenticationProvider'.)'
Error with provider shown in docs:
System.InvalidCastException: 'Unable to cast object of type 'Microsoft.AspNetCore.Components.Server.ServerAuthenticationStateProvider' to type 'Microsoft.AspNetCore.Components.WebAssembly.Authentication.IAccessTokenProvider'.'
and this happens in the GraphExetensions.cs here:
services.AddScoped(sp =>
{
return new GraphServiceClient(
sp.GetRequiredService(),
sp.GetRequiredService());
});

Everything is just as it shows in the doc at the start.
Program.cs has this added to it:
builder.Services.AddMsalAuthentication<RemoteAuthenticationState,
RemoteUserAccount>(options =>
{
builder.Configuration.Bind("AzureAd",
options.ProviderOptions.Authentication);
options.ProviderOptions.DefaultAccessTokenScopes.Add(
"https://graph.microsoft.com/User.Read");
})
.AddAccountClaimsPrincipalFactory<RemoteAuthenticationState, RemoteUserAccount,
CustomAccountFactory>();
builder.Services.AddGraphClient("https://graph.microsoft.com/User.Read");

This article only applies to standalone or hosted Blazor WebAssembly apps. If you're seeking to implement MS Graph in a Blazor Server app, then you follow guidance for implementing Graph API in a regular ASP.NET Core app.
There's a tutorial in the Azure docs at ...
https://learn.microsoft.com/azure/active-directory/develop/tutorial-blazor-server
... where they have a different project template (blazorserver2) that comes from installing a special set of project templates in Microsoft.Identity.Web.ProjectTemplates. It looks like the article and the sample is 3.1 era tho.
Their sample apps can be found here ...
https://learn.microsoft.com/azure/active-directory/develop/sample-v2-code#web-applications
... and they have a Call MS Graph sample app at ...
https://github.com/Azure-Samples/ms-identity-blazor-server/tree/main/WebApp-graph-user/Call-MSGraph
... but that's 3.1 era, too.
Here's a good one to look at: I found a recent sample over there for 6.0 🎉 at ...
https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2/tree/master/2-WebApp-graph-user/2-1-Call-MSGraph
I'd look at that one the closest. I think they may have missed updating their samples page link. When I reach my issue for work, I might open an issue with them to inquire on it. Anyway, that looks like the best one that they have.
Note that their sample doesn't use Minimal APIs, so you'll see the ancient 👴😄 Program.cs/Startup.cs bits. That's fine tho ... that approach to organizing startup code is still valid. Anyway , you'll see the right authn/z services setup and config for a 6.0 era app and should be able to translate that into a Minimal APIs-based app or get the right calls into the right spots in your existing Minimal APIs app.
There's also going to be new ASP.NET Core templates for 7.0 that have built-in Graph API. I haven't looked at them closely yet for how they enable/integrate Graph API, but it should be simpler to wire up Graph going forward. The new templates are in preview now and will be released along with the rest of .NET 7 in November.
One of things that I'll be doing on my updates to our Blazor Security node is get the best cross-links into our docs. The tutorials and sample apps that you see over there, along with a great deal of new MS Identity Platform and Azure hosting docs, were put up after all of the docs in this node were written. We probably will be able to drop and cross-link to their resources for these types of authn/z with Azure and Graph scenarios.
Side Note: For Blazor WebAssembly, they have a similar tutorial/samples for it in the same areas of their docs. Their 6.0 samples are more current than ours over here, too, for Blazor WebAssembly. I'd recommend that devs working on Blazor WebAssembly apps also look at their docs and samples more closely than ours over here ... at least until I can sort out this node with updates and cross-links later this year.
I'm going to close this as not actionable at this time, but I will cross-link this issue/discussion on my tracking issue to review it in November/December. If you require product support, we recommend the usual public support channels ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

Thanks, I will look into these solutions on Monday. I have yet to get anything working correctly with this stuff because of all the misinformation and wrong docs. It has been difficult to find an actual working starting point so I can just start doing my normal programming. It is very confusing how all this is structured to me, and C# is my main language. This isn't even recognizable as normal to me at all lol. Why do claims even need to be a thing for example? What's the point of that? Why isn't this just simple RESTful POST/GET? It seems like it has been overly complicated on design.

oh also, i need to mention that it must be blazor and a lot of these docs are about swagger as well which are all useless for me. The teams toolkit is based on blazor only.

misinformation and wrong docs

But if someone uses a doc written for Blazor WebAssembly to implement an API for Blazor Server, then that wouldn't be expected to work. I'm still not 100% clear on what hosting model you're using. You never said, but that error message makes reference to Blazor Server API, so that's what I assumed you were using.
I recommend going by the Azure docs that I mentioned, paying the most attention to that 6.0 Blazor Server sample app. I'll see about fixing up (or dropping 🔪  and cross-linking where I can) the Blazor node docs to what they have, including adding cross-links for those 6.0 Azure/AAD/Graph samples.
WRT ...

Why do claims even need to be a thing for example?
has been overly complicated on design
Why isn't this just simple RESTful POST/GET?

Those sorts of considerations are mostly out of scope for our work at this level. You're addressing industry-wide (OAuth 2.0/OIDC) and lower-level Microsoft platform features (MS Identity Platform).
WRT what is in scope here for how to implement the protocols and platform features, I won't know more about how our Blazor security node docs need to change until I reach the node for work in a few weeks. I was hoping to do it last year, which you can see on https://github.com/dotnet/AspNetCore.Docs/issues/19286. It wasn't reached 😢 ... just too 🏃🏃🏃 to reach this node last year. AFAICT, I will reach it this year. Still tho, the Blazor WebAssembly node docs can't be directly used for these scenarios in a Blazor Server apps, even after I seek to improve them.

oh also, i need to mention that it must be blazor and a lot of these docs are about swagger as well which are all useless for me. The teams toolkit is based on blazor only.

I don't understand. There's no Swagger in the Blazor docs. I think you're referring to the main doc set articles.
WRT tooling: There's nothing specific about the tooling that prevents the guidance here from working AFAIK. There was a problem with how VS was creating AAD/B2C Blazor apps with auth, but I think they either fixed it in .NET 6 or will have it fixed by .NET 7. Anyway, the docs don't use VS at all. They use the .NET CLI. I'll take a closer look at the Teams Toolkit. I don't know anything about it at this point, so I'll need to do some research on it. I wouldn't think that your issues have anything to do with it, but I can't say for sure not knowing what it is (yet! 😄).

Turns out, none of this is needed and I figure out how to do with the TeamsFx SDK. It is already tied like how it should be.


define* not considered as definition

The star-form of define (define*) is not considered a definition and their definitions are therefore not indexed.
https://github.com/absop/ST-Scheme/blob/0a21d269b5ac910d1f15da7abadbcecf8bb14ddd/syntax/Scheme.sublime-syntax#L764-L773
As a quick workaround I've modified (?:define(?:-(?:condition-type to (?:define\*?(?:-(?:condition-type, but I'm sure there are negative side-effects...

Sorry for the too late reply, what's the negative side-effects?


Implement a no projects check in the API request.

This resolves issue #308

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Added integer conversion in toBoolean functions

Implemented explicit integer conversion to boolean in both toBoolean and toBooleanList functions. Also made corresponding changes to regression tests and broke long lines to adhere to the 79-column line length limit in the toBooleanList function.
This PR makes the toBoolean() and toBooleanList() functions convert integers to boolean, as occurs in Neo4j. Examples are provided in issue #1171

@WendelLana Would it be possible to get this updated to the latest master?

@WendelLana If you would be so kind as to create PRs for PG16, PG15, PG14, & PG13 to make this change to those branches as well.

@jrgemignani Sure! I will shortly create the PRs


hotfix(dal): disable attribute override query temporarily

This query is hanging and blocking work, disabling it while I work on an alternative approach

bors r+


feat: add a layershellexample

I found that layershell is very similar with xdg, so I want to add one example of layershell with origin wayland-rs..because the are so similar

I just wanted to say, thank you very much for this example! It has really helped me out and guided me! Why exactly has it not yet been merged?? This is gold!


Add dapr useragent for Azure Service Bus and EventHubs

Description
Set custom user agent for dapr to be used with Azure Service Bus and EventHubs.
This change prepends dapr/ to the existing User Agents.
Issue reference
#1014
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly

The upstream components already have tests for setting custom user agents. No additional tests needed here.

FYI @greenie-msft

@CodeMonkeyLeet it does address that issue for Service Bus and Event Hubs, but not other services.
We have an email thread with that service team where we discussed this in depth:
TL;DR -- Service Bus and Event Hubs teams receive a product, version, user agent and a few other identifiers. An example they provided was the Python SDK.
@greenie-msft and I looked at the Python SDK in depth to see just how this is sent. It is transmitted via the AMQP message metadata properties -- this itself is set globally by the sender client constructor -- so the implementation is very specific to Service Bus and also Event Hubs.
The GO SDK for these services (just like the Python one) has the ability to prepend to the user agent that is transmitted. This change allows us to add a dapr specific identifier.
Now the much harder problem -- completely out of scope of this PR or anything on our (Dapr) team -- getting the logs from the Microsoft internal Service Bus / Event Hubs distributing tracing log store (Geneva). The team said they likely need to implement something to directly filter and forward events on their end. Something they need for business reasons anyway and this is on their backlog.


TelemetryClient assigned to Serilog not get passed when creating a contextual  logger for class

Currently, we are using Serilog together with ApplicationInsights to log the system. One issue we try to solve is attach a unique operation ID to the every log message in a single request. So an error message can be traced.
When build the ILogger for the container
serilogConfig.WriteTo.ApplicationInsightsTraces(telemetryClient);
The telemtryClient is registerd as instancePerRequest and assigned with a unique OperationID in one of the actionAttribute on the Controller
public override void OnActionExecuting(HttpActionContext actionContext)
{
    var requestScope = actionContext.Request.GetDependencyScope();
    var telemetryClient = requestScope.GetService(typeof(TelemetryClient)) as TelemetryClient;
    telemetryClient.Context.Operation.Id = Guid.NewGuid().ToString();
    base.OnActionExecuting(actionContext);
}

And inside the constructor of a controller requires logging
public ControllerA(ILogger logger)
{
    var contextLogger = logger.ForContext<ControllerA>();
}

Now the problem is that the operationID does not appear on the log message.
Even if we register the main ILogger as instancePerRequest, will the contextLogger still getting the same dependency object as the main one?
Additional comments:
If we were able to set the operationID on the TelemetryClient, ( which is supposed to be singleton). But this is not thread safe. One we request set operatinID and start logging, but in between, the another web request kicks in and reset this operationID.
Initially, we register TelemetryClient as singleton, but this operationID stay the same for each request, and it is also recommended to created telemtryClient instance per request
https://social.msdn.microsoft.com/Forums/en-US/fefbea33-2f0c-4cb0-9864-25cf9f1991d2/is-telemetryclient-threadsafe?forum=ApplicationInsights
We are using versin 2.1.0. Thanks

Hi @JonathonWei as far as I understand you are creating new logger instances per request & the problem is that by default the Serilog AI sink uses a singleton telemetry client?
You can solve this with the current version of the serilog sink by creating a custom Telemetry Client per Request yourself (and set the Operation ID you mentioned) and pass that instance into the serilog instance (also created per request) as shown i.e. in the  Readme.md.
The Sink itself doesn't know / care about the usage - whether it is a along-running service application or a web app that might require per-request instances.. but you can do it either way.
Does this help/solve your problem?

Hi @jbattermann , is there any possibility that ILogger will registered as singleton? I understand that int normal scenario the singleton should not depend on instancePerRequest.
However, from my understand and on the documentation, the ILogger.ForContext<>() will create a new contextual logger for the current web request already. This is already instance Per Request.
Is there anyway that even the ILogger is registered as singleton, the contextual logger will get the telemetryClient created in the attribute.OnActionExecuting()?

Additionally, the LoggerConfiguration only allow to createLogger() once.
If the Ilogger is registered instancePerRequest, then this LoggerConfiguration also needs to be instancePerRequest?


gardenlet uses "ambiguous" client certificate in nodeless development setup

How to categorize this issue?

/area security
/kind enhancement
What would you like to be added:
In the nodeless development setup, the gardenlet uses a kubeconfig containing a client certificate with the "ambiguous" user name:
"subject": {
  "common_name": "gardener.cloud:system:seed:\u003cambiguous\u003e",
  "organization": "gardener.cloud:system:seeds"
}

This file is maintained here: https://github.com/gardener/gardener/blob/master/hack/local-development/local-garden/kubeconfigs/gardenlet.conf
This is an artefact from the earlier times when a gardenlet was able to control multiple seed clusters. However, this feature was dropped as part of #4078.
Nowadays, when a gardenlet is started then it is required to specify the SEED_NAME environment variable containing the name of the Seed the gardenlet should act upon. This now allows us to drop this static kubeconfig and generate it on the fly such that is always contains the correct name of the seed cluster.
Why is this needed:
This allows a more accurate usage of the SeedAuthorizer feature in during local development (however, only in the nodeless setup; for other setups this is not (yet?) possible at all).

/assign


fix: Handle list[str] in function schema

Please describe the purpose of this pull request.
Allows the usage of list[str] as a parameter and return type in MemGPT functions
How to test
write a function that takes a list[str] and try and have MemGPT load it.
This works
def AI_command(self, command: str) -> str:

This Doesnt
def AI_command_batch(self, commands: list[str]) -> str:

error:
Error loading function set: Python type list[str] has no corresponding JSON schema type

Have you tested this PR?
Yes, after change a function with this prototype will load and run successfully:
def AI_command_batch(self, commands: list[str]) -> str:

Related issues or PRs
None
Is your PR over 500 lines of code?
No
Additional context
There is probably a better way to do this to include lists of other standard types such as int. My method is basically building out a long if-else chain/switch statement.

Rolling into #1110


:memo: feat: Created a consultancy-page.html file

Description
I created a consultancy-page.html file inside the features/pages directories #11
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Good attempt closing this pr as it is already done, please pick another issue


doc: obsolete screenshots should be replaced

please review using environment installed from master and replace obsolete screenshots in documentation.
https://qaprosoft.github.io/qps-infra/config-guide/#register-organization

https://github.com/qaprosoft/qps-infra/blob/master/docs/config-guide.md  - Config guide is updated

Images are updated according to 5.0.4 version.